-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Bump Linux minimum version to 4.19.57 (or equivalent) #23124
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
Conversation
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.
Nice!
I've bumped the minimum here with a recommendation for 4.19.57 "or equivalent" with the understanding that sometimes distributions may choose to backport significant amounts of newer kernel functionality into older releases, such as RHEL. The daemon itself will still rely on feature probes for the features that it requires in order to operate, so at runtime it should continue to work on older kernels as long as the kernel functionality is there. However, over time we will begin to remove and deprecate support for kernels older than this target version. I've also bumped the distribution versions to more recently supported versions of distributions which match this kernel dependency. Signed-off-by: Joe Stringer <joe@cilium.io>
These features rely on LPM maps, so they printed a helpful message in case the user was running one of those older kernels. Now that we don't support those kernel versions, we don't need to emit such tips. Signed-off-by: Joe Stringer <joe@cilium.io>
173f43e
to
26e6c0e
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.
Yay! It'll be good to bring this up in the next community meeting, especially in the context of datapath testing.
@aditighag yep we can revisit it this Wednesday 👍 . For now, this isn't actually removing the code that supports those older versions; CI is still depending on 4.9 for now for instance. But this is signalling that we will not support 4.9 for the full lifecycle of the Cilium 1.13.x release series. |
FWIW - with |
Cool, thanks for the update Julian. We discussed this item briefly also during the community meeting today, and agreed that there are subsequent issues to follow up on with regards to the way that the Cilium codebase relies on v4.9 to do testing. Those items are already listed in #22116 so we can continue to follow up on that over time. For now, we think it makes sense to nudge users onto newer kernel versions by documenting the higher minimum version, then over time we will actually remove support for older kernels. |
I've bumped the minimum here with a recommendation for 4.19.57 "or
equivalent" with the understanding that sometimes distributions may
choose to backport significant amounts of newer kernel functionality
into older releases, such as RHEL. The daemon itself will still rely on
feature probes for the features that it requires in order to operate, so
at runtime it should continue to work on older kernels as long as the
kernel functionality is there. However, over time we will begin to
remove and deprecate support for kernels older than this target version.
I've also bumped the distribution versions to more recently supported
versions of distributions which match this kernel dependency.
Fixes: #22586