-
Notifications
You must be signed in to change notification settings - Fork 71
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
podvm-mkosi: remove cloud-init #1532
podvm-mkosi: remove cloud-init #1532
Conversation
5346d85
to
dd8acc2
Compare
We have our own mechanism to fetch metadata from IMDS. This is far more flexible than using cloud init, especially regarding the upcoming switch to an read-only root FS. Thus removing cloud init support for mkosi images with this commit. Signed-off-by: Paul Meyer <49727155+katexochen@users.noreply.github.com>
Some additional debug utils. Signed-off-by: Paul Meyer <49727155+katexochen@users.noreply.github.com>
dd8acc2
to
608879c
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.
thanks, tested successfully
iputils | ||
curl | ||
wget |
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.
unrelated?
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.
/lgtm
Thanks @katexochen
This is based on #1524, which should be reviewed and merged first.
Fixes #1467 (for x86, we should check with the IBM folks if they want to remove cloud init for s390x as well)
This requires
DISABLE_CLOUD_CONFIG=true
in yourinstall/overlay/${CSP}/kustomization.yaml