bottlerocket-os / bottlerocket

An operating system designed for hosting containers
https://bottlerocket.dev
Other
8.78k stars 519 forks source link

1.20.x: kernel updates, statically linked modprobe #3997

Closed larvacea closed 5 months ago

larvacea commented 5 months ago

Description of changes:

Testing done:

Kernel changes: our standard sonobuoy tests and manual inspection of patch and configuration changes. modprobe: see [#3981] for details.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

larvacea commented 5 months ago

I have incorporated @ginglis13's suggested changes.

obirhuppertz commented 5 months ago

Would it be possible to publish a 1.20.1 release with just these changes so we can select this AMI in our launch template? Reading through the merged issues, cilium should be working again with these commits.

Thanks @vigh-m (and any unmentioned contributor) for investigating and fixing this issue!

yeazelm commented 5 months ago

Hello @obirhuppertz, we did release 1.20.2 which provides a fix for CIlium. Please check that release and follow up on https://github.com/bottlerocket-os/bottlerocket/issues/3968 if you experience more issues with Cilium. Thanks!

obirhuppertz commented 5 months ago

@yeazelm @vigh-m can confirm cilium is still working on 1.20.2 bottlerocket nodes having the workaround via extraMounts and volumes removed in values.yaml. For me the issue is fixed now. Great work! :partying_face: