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

Consider switch to Gardener OperatingSystemConfig extension. #75

Open
ppalucki opened this issue Oct 24, 2022 · 0 comments
Open

Consider switch to Gardener OperatingSystemConfig extension. #75

ppalucki opened this issue Oct 24, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request question Further information is requested

Comments

@ppalucki
Copy link
Contributor

ppalucki commented Oct 24, 2022

Iinitial approach was based on how gvisor extension works (installation daemon set).

But there is another approach - we can try https://gardener.cloud/docs/gardener/extensions/operatingsystemconfig/

According the docs, we could just replace "installation DaemonSet" with OSC that will containt nesseary unit & files for running CRI-RM.

There is whole infrastructre already in place here: it is just enough to provide "generator" that will preapre files & units:

like in this gardener-extension OSC for garden linux: https://github.com/gardener/gardener-extension-os-gardenlinux/blob/master/cmd/gardener-extension-os-gardenlinux/app/app.go#L45
and then just prepare cloud-init template:

as examples for

those templates could be just replacement for our installation-script (only installation part)

Advantages:

  • it is setup during OS provisioning (passsed as userData to VM) so seems more natural for critical units configuration

Downsides:

  • if we want "uninstallation" feature - we still need "uninstallation daemonset" that will restore kubelet configuration to original version
  • userData will be quiet big: cri-rm exectuable is 64mb but gardener work arounded this by using "downloader script"
@ppalucki ppalucki added enhancement New feature or request question Further information is requested labels Oct 24, 2022
@ppalucki ppalucki added this to the future/undecided milestone Oct 24, 2022
@ppalucki ppalucki self-assigned this Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants