Skip to content
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

Cilium sysext #114

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Cilium sysext #114

wants to merge 1 commit into from

Conversation

reavessm
Copy link

Install Cilium via sysext

This PR adds the cilium sysext which downloads the cilium binary and creates a systemd unit install it to a cluster on boot. Additional args are provided via an env var.

How to use

Run create_cilium_sysext.sh v0.16.24 cilium. This creates the binary in cilium/usr/local/bin/cilium as well as the systemd unit file to run cilium install on boot.

Testing done

I ran this against a demo cluster running on libvirt with the kubernetes sysext.

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

Signed-off-by: Stephen Reaves <[email protected]>
@reavessm
Copy link
Author

At some point I'd like to update the usage docs to include an example of a drop-in to get this to run only after kubeadm.service, but I didn't want to make that a default dependency since you technically could install k8s from a couple different ways.

@t-lo
Copy link
Member

t-lo commented Feb 13, 2025

At some point I'd like to update the usage docs to include an example of a drop-in to get this to run only after kubeadm.service, but I didn't want to make that a default dependency since you technically could install k8s from a couple different ways.

Using the bakery kubernetes sysext in a more complex configuration might be a nice and self-contained way to document this.

@reavessm
Copy link
Author

I mean I have a full HA k8s setup using crio, cilium and HAProxy+Keepalived using a variation of this sysext in multiple butane configs. I can make another PR adding some docs on that if you think it'd be worth it

@t-lo
Copy link
Member

t-lo commented Feb 14, 2025

I mean I have a full HA k8s setup using crio, cilium and HAProxy+Keepalived using a variation of this sysext in multiple butane configs. I can make another PR adding some docs on that if you think it'd be worth it

Yes, I think it's absolutely worth it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants