-
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
test/k8sT/manifests: use image hash with cilium-builder image #13982
test/k8sT/manifests: use image hash with cilium-builder image #13982
Conversation
de4061a
to
905df9e
Compare
test-me-please failed due to docker registry rate limiting 😞 |
retest-net-next |
retest-4.9 |
retest-4.19 |
retest-gke |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍. Typo in commit msg: refernce
Will fix up once all tests passed and this is ready to be merged. |
retest-4.19 previous failure: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Kernel/3778/, looks like #13552 |
Tags can be arbitrarily changed without the user noticing, potentially making the tag point to a completely different image. Instead, reference the cilium-builder image by its unique sha256 hash which is already done in all other places using the image. While at it, also bump the image to the latest version which was bumped to Go 1.15.4 in #13945. Fixes: 417cded ("test: Move RuntimeVerifier to K8sVerifier") Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
905df9e
to
f2eb0eb
Compare
Test have previously passed, force pushed to fix the typo in the commit message pointed out by @christarazi: #13982 (review) |
Tags can be arbitrarily changed without the user noticing, potentially
making the tag point to a completely different image. Instead, reference
the cilium-builder image by its unique sha256 hash which is already done
in all other places using the image.
While at it, also bump the image to the latest version which was bumped
to Go 1.15.4 in #13945.
Fixes: 417cded ("test: Move RuntimeVerifier to K8sVerifier")
Note that the manifests in the
v1.8
andv1.9
release branches already use the image hash.