-
Notifications
You must be signed in to change notification settings - Fork 39
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
sysprep: Remove could-init #682
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.
I'd mask it instead, as it's faster.
Cloud images for VMs usually are prepared to be used with cloud-init. Because lago does not utilize could-init, it should be disabled to avoid waiting could-init for configuration during booting. This change improves the usage of cloud images in lago. Signed-off-by: Dominik Holler <[email protected]>
f54feb6
to
fdc047a
Compare
I am not sure what means 'mask', but I disabled cloud-init in a simple and probably distribution independent way. |
delete /etc/systemd/system/multi-user.target.wants/kdump.service
delete /etc/systemd/system/multi-user.target.wants/postfix.service
Are two examples of masking. It is using systemd, but I assume all our
images do (?!)
Y.
…On Sat, Dec 9, 2017 at 10:57 PM, Dominik Holler ***@***.***> wrote:
I'd mask it instead, as it's faster.
I am not sure what means 'mask', but I disabled cloud-init in a simple and
probably distribution independent way.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#682 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEcJ6dKrxEXtKpK7JBSgw1qwgjD0oOG8ks5s-vQkgaJpZM4Q8OFw>
.
|
delete /etc/systemd/system/multi-user.target.wants/kdump.service
delete /etc/systemd/system/multi-user.target.wants/postfix.service
Are two examples of masking.
This way of disabling are used for specif el or fc versions.
The change proposed here would affect all linux distributions.
It is using systemd, but I assume all our
images do (?!)
I see no reason here to have a dependency to systemd, even it would not
hurt.
Additional there are 4 *.service links for cloud-init
in /etc/systemd/system/multi-user.target.wants and I have no idea if
they depend on cloud-init version.
Because the [cloud-init
documentation](http://cloudinit.readthedocs.io/en/latest/topics/boot.html#generator)
suggest to disable cloud-init by the /etc/cloud/cloud-init.disabled or
kernel command line, I think we should disable cloud-init like this.
|
I'd really rather someone goes ahead and implements full cloud-init support for Lago rather then workarounds. I think various pieces of this work had already been done in the past. |
|
This is great! This way we can disable could-init per image and this makes this change obsolete. Thanks for the discussion, I will go on in https://gerrit.ovirt.org/#/c/85170/, because no change to lago seems required. |
This change improves the usage of cloud images in lago.
If we think the usage of generic cloud images in OST would be a good idea, this change would prepare this.