-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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: run verifier tests with proper Go toolchain version #27857
Conversation
/ci-verifier |
/test |
254d6ba
to
8439b0f
Compare
/test |
Currently, the verifier tests are built and run with whatever Go toolchain version the LVH images ship. To make sure these tests are run with the same Go toolchain version as the rest of the tests install a copy of that Go toolchain before running the test. This wasn't caught in CI on #27820 because the verifier workflow wasn't run passed on the ariane exclusion list. Reported-by: Julian Wiedmann <jwi@isovalent.com> Signed-off-by: Tobias Klauser <tobias@cilium.io>
8439b0f
to
c166510
Compare
/ci-verifier |
Marked for backport to all stable branches (except v1.15 which already includes it), as we are starting to hit the same issue there too: https://github.com/cilium/cilium/actions/runs/7623179184/job/20762689062 |
The workflow does not appear to exist in v1.12 and v1.13. Removing the needs-backport label for these branches. |
Currently, the verifier tests are built and run with whatever Go toolchain version the LVH images ship. To make sure these tests are run with the same Go toolchain version as the rest of the tests install a copy of that Go toolchain before running the test.
This wasn't caught in CI on #27820 because the verifier workflow wasn't run passed on the ariane exclusion list.
Reported-by: @julianwiedmann