Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: publish latest tag based on matrix #1

Merged
merged 2 commits into from
Jan 13, 2023

Conversation

toast-gear
Copy link
Contributor

@toast-gear toast-gear commented Jan 13, 2023

@Link- the latest tag appears to have been added to the non-latest step which has broken many deployments as the 22.04 is signficantly different to the 20.04 image

We were going to deprecate the latest tag entirely actions/actions-runner-controller#2056 as we can't shift people without breaking lots of deployments and we don't really have a means for communicating with people so it seemed like it wasn't viable. The tags ubuntu-20.04 and ubuntu-22.04 were to act as latest for each specific OS for people who are happy to just pull the latest and can then migrate to the newer OS in a controlled manner

@toast-gear
Copy link
Contributor Author

Additionally the GitHub Packages publish doesn't appear to work https://github.com/actions/actions-runner-controller/pkgs/container/actions-runner-controller%2Factions-runner. The workflow run seems happy but no packages, I wonder if the fact it's redirect from actions-runner-controller to actions is related.

@florianmutter
Copy link

Not sure if it belongs to this PR or not but I think updating the ubuntu-22-04 tag is missing. If I don't want to use latest I would have to use a version. Is this intended?

@kuuji
Copy link

kuuji commented Jan 13, 2023

Not sure if it belongs to this PR or not but I think updating the ubuntu-22-04 tag is missing. If I don't want to use latest I would have to use a version. Is this intended?

Same thing with the 20.04 tag I think. I noticed it was last pushed a bit ago (17 days).

Copy link
Contributor

@Link- Link- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @toast-gear for addressing this so swiftly

@Link- Link- merged commit 29379f0 into actions-runner-controller:main Jan 13, 2023
@toast-gear toast-gear deleted the patch-1 branch January 13, 2023 18:07
@Link-
Copy link
Contributor

Link- commented Jan 13, 2023

Rebuilding and pushing the images again after this change: https://github.com/actions-runner-controller/releases/actions/runs/3913800459

@Link-
Copy link
Contributor

Link- commented Jan 13, 2023

@toast-gear in that workflow you referenced, GHCR terminated the connection for one of the jobs (we're looking into this). This shouldn't be the norm though.

@vanZeben vanZeben linked an issue Jan 13, 2023 that may be closed by this pull request
@toast-gear
Copy link
Contributor Author

Yeh when I said looked fine I meant more the other runs, we've had GHCR close connections from time to time since we started publishing to GHCR (it would be great if it to work everytime though ofc!). The other runs succeeded so I would the expect to see those images in GHCR, for example ghcr.io/actions-runner-controller/actions-runner-controller/actions-runner:v2.300.2-ubuntu-20.04 completed fine but you don't see it in https://github.com/actions/actions-runner-controller/pkgs/container/actions-runner-controller%2Factions-runner.

@Link-
Copy link
Contributor

Link- commented Jan 13, 2023

Yeh when I said looked fine I meant more the other runs, we've had GHCR close connections from time to time since we started publishing to GHCR (it would be great if it to work everytime though ofc!). The other runs succeeded so I would the expect to see those images in GHCR, for example ghcr.io/actions-runner-controller/actions-runner-controller/actions-runner:v2.300.2-ubuntu-20.04 completed fine but you don't see it in https://github.com/actions/actions-runner-controller/pkgs/container/actions-runner-controller%2Factions-runner.

That's because the images are here:
https://github.com/actions-runner-controller/actions-runner-controller/pkgs/container/actions-runner-controller%2Factions-runner

They're still in the old organisation as they're owned by it. That's part of the reason we had to jump through the hoops to build these split workflows. The naming problems are a known issue that the team is addressing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Latest tag moved to 22.04 on the last runner release
4 participants