Skip to content

fix(startup): only show the "using latest tag" warning on relevant images#2718

Closed
TPXP wants to merge 3 commits intoactions:masterfrom
TPXP:fix-latest-warning
Closed

fix(startup): only show the "using latest tag" warning on relevant images#2718
TPXP wants to merge 3 commits intoactions:masterfrom
TPXP:fix-latest-warning

Conversation

@TPXP
Copy link

@TPXP TPXP commented Jul 5, 2023

We'll set the SHOW_LATEST_TAG_WARN env var for images build with the latest tag only (see PR on the releases repo).

Since the warning is now only shown in relevant cases, bump its severity.

See corresponding PR for the build pipeline actions-runner-controller/releases#11

…ages

We'll set the SHOW_LATEST_TAG_WARN env var for images build with the latest
tag only (see PR on the releases repo).

Since the warning is now only shown in relevant cases, bump its severity.
@TPXP
Copy link
Author

TPXP commented Aug 21, 2023

@Link- Hello, thanks for your work on this repo! This PR has been open for a while, is there anything I can do to help getting it merged?

@Link-
Copy link
Member

Link- commented Sep 7, 2023

@TPXP - these runner images including the startup script are maintained by @mumoshu. My 2 cents are that this type of warnings is not good to ship. We expect a certain level of experience when adopting a solution like ARC and we don't want to be throwing warnings for every trivial matter.

It's valid to use the latest tag, albeit not advisable, and I expect end-users to be aware what they're getting into here.

@Link- Link- added the community Community contribution label Sep 7, 2023
@Link- Link- closed this Sep 7, 2023
@TPXP
Copy link
Author

TPXP commented Sep 11, 2023

@Link- Thanks for your feedback, I just wish to clarify that the warning is currently raised unconditionally in the current version, which adds confusion over unneeded noise in the logs. If you prefer not to have any warning raised even when using the latest tag, could we just drop the warning from the script?

log.notice "WARNING LATEST TAG HAS BEEN DEPRECATED. SEE GITHUB ISSUE FOR DETAILS:"
log.notice "https://github.com/actions/actions-runner-controller/issues/2056"

@Link-
Copy link
Member

Link- commented Sep 11, 2023

I'll let @mumoshu weigh in on this ☝️

@TPXP
Copy link
Author

TPXP commented Nov 15, 2023

@mumoshu Hello there, any updates?

@mumoshu
Copy link
Collaborator

mumoshu commented Dec 14, 2023

@TPXP Sorry for the delay! Yeah, let's drop it and update our defaultRunnerImage default to not point to the latest image.
That's the way to go and it's just that I didn't have much time to do it myself!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Community contribution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants