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

renovate: fix the way we manually install Go inside the runner #1584

Merged
merged 1 commit into from
Oct 12, 2023

Conversation

mtardy
Copy link
Member

@mtardy mtardy commented Oct 12, 2023

This is a follow-up of #1579 because #1577 is broken.

This simplifies a49c4bc that causes issues in the case of the new Go toolchain feature. It was manually installing a version off-by-one, like 1.21.2 for 1.21.3 present in the just updated go.mod and thus triggering the automatic download of the 1.21.3 toolchain. But in the renovate container env, GOSUMDB is set to off for some reason which blocks the download of the toolchain.

This simplifies the manual install by reading the go.mod file to determine the correct version, allowing to remove the bump of the number in the renovate config itself.

This simplify a49c4bc that causes
issues in the case of the new Go toolchain feature. It was manually
installing a version off-by-one, like 1.21.2 for 1.21.3 present in the
just updated go.mod and thus triggering the automatic download of the
1.21.3 toolchain. But in the renovate container env, GOSUMDB is set to
off for some reason which blocks the download of the toolchain.

This simplifies the manual install by reading the go.mod file to
determine the correct version, allowing to remove the bump of the number
in the renovate config itself.

Signed-off-by: Mahe Tardy <mahe.tardy@gmail.com>
@mtardy mtardy added area/ci Related to CI release-note/ci This PR makes changes to the CI. labels Oct 12, 2023
@mtardy mtardy requested a review from a team as a code owner October 12, 2023 08:15
@mtardy mtardy requested a review from jrfastab October 12, 2023 08:15
@mtardy mtardy merged commit 2a79de5 into cilium:main Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci Related to CI release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants