-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add 1.16.3 #12
base: main
Are you sure you want to change the base?
Add 1.16.3 #12
Conversation
We're adding Rock images for Cilium 1.16.3.
e716e6c
to
5a80a64
Compare
Building cilium-envoy currently fails because of a checksum validation error, need to investigate further.
|
We're hitting this: envoyproxy/envoy#36563 |
Some dependency checksums have changed, which causes the envoy build to fail. We'll apply a patch that updates these checksums.
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.
Great work, I believe in 1.16
Cilium switched to LLVM 17 and Go 1.22. Can we check for dependencies to make sure they are matching? You can refer to https://github.com/cilium/cilium/tree/v1.16.3/images
d233c26
to
02ea635
Compare
@berkayoz one problem though: there are no llvm-17 packages for ubuntu 22.04, we'll have to add the repos or switch to 24.04, which may break other dependencies. |
We're updating the cilium dependencies according to the upstream 1.16.3 release.
02ea635
to
8c027b8
Compare
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.
We're almost there, thanks!
The "OUTPUT" parameter isn't properly handled by the bpftool make script, so we'll copy it over from the default directory.
ae63931
to
eaafef3
Compare
We're adding Rock images for Cilium 1.16.3.