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

sysprep: Remove could-init #682

Closed

Conversation

dominikholler
Copy link
Contributor

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.

Copy link

@mykaul mykaul left a 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]>
@dominikholler
Copy link
Contributor Author

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.

@mykaul
Copy link

mykaul commented Dec 9, 2017 via email

@dominikholler
Copy link
Contributor Author

dominikholler commented Dec 9, 2017 via email

@ifireball
Copy link

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.

@gbenhaim
Copy link
Member

  1. There is a WIP of adding cloud-init support to lago: [WIP] clod-init: Adding cloud init support #651
    (actually its almost ready, just need to add some unit and functional tests).

  2. I don't think that disabling cloud-init should be hard coded.

  3. We can use the build section in lago's init file in order to disable cloud-init. http://lago.readthedocs.io/en/latest/BUILD.html

  4. The following list of symlinks can be used in order to disable cloud-init:

link /dev/null:/etc/systemd/system/cloud-init.service
link /dev/null:/etc/systemd/system/cloud-init-local.service
link /dev/null:/etc/systemd/system/cloud-config.service
link /dev/null:/etc/systemd/system/cloud-final.service

@dominikholler
Copy link
Contributor Author

  1. We can use the build section in lago's init file in order to disable cloud-init.
    http://lago.readthedocs.io/en/latest/BUILD.html

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.

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.

4 participants