/sos

[cloud-image] Added Ubuntu plugin for Cloud-Images #548

Open
wants to merge 1 commit into
from

Conversation

Reviewers
No reviews
Assignees

@battlemidget battlemidget

Labels
Changes Needed
Projects
None yet
Milestone
No milestone
3 participants
@utlemming @battlemidget @bmr-cymru
@utlemming

utlemming commented Apr 9, 2015

Ubuntu Cloud Images are defined by the presence of cloud-init being
installed. This plugin gathers the base logs for cloud-init and is
useful for determining boot failures in the cloud.

Signed-off-by: Ben Howard ben.howard@ubuntu.com
[cloud-image] Added Ubuntu plugin for Cloud-Images 
  Ubuntu Cloud Images are defined by the presence of cloud-init being
  installed. This plugin gathers the base logs for cloud-init and is
  useful for determining boot failures in the cloud.

Signed-off-by: Ben Howard <ben.howard@ubuntu.com>
e71908b
@battlemidget
Owner

battlemidget commented Apr 9, 2015

ACK @bmr-cymru

@battlemidget battlemidget self-assigned this Apr 9, 2015

@bmr-cymru

bmr-cymru commented on e71908b Apr 13, 2015

Since this is packaged for Fedora we might as well add RedHatPlugin to the tagging classes form the start.

$ yum info cloud-init
Loaded plugins: auto-update-debuginfo, langpacks, product-id, refresh-packagekit, subscription-manager
Available Packages
Name        : cloud-init
Arch        : noarch
Version     : 0.7.2
Release     : 7.fc20
Size        : 325 k
Repo        : fedora/20/x86_64
Summary     : Cloud instance init scripts
URL         : http://launchpad.net/cloud-init
Licence     : GPLv3
Description : Cloud-init is a set of init scripts for cloud instances.  Cloud instances
            : need special scripts to run during initialization to retrieve and install
            : ssh keys and to let the user run various scripts.

Name        : cloud-init
Arch        : x86_64
Version     : 0.7.2
Release     : 10.fc20
Size        : 326 k
Repo        : updates/20/x86_64
Summary     : Cloud instance init scripts
URL         : http://launchpad.net/cloud-init
Licence     : GPLv3
Description : Cloud-init is a set of init scripts for cloud instances.  Cloud instances
            : need special scripts to run during initialization to retrieve and install
            : ssh keys and to let the user run various scripts.

bmr-cymru replied Apr 13, 2015

Doesn't this also want DebianPlugin? (it's unusual to have UbuntuPlugin and not DebianPlugin).

bmr-cymru replied Apr 13, 2015

Ships in wheezy, jessie & sid at least.

bmr-cymru replied Apr 13, 2015

Lastly: why "cloudimage"/"CloudImage" not "cloudinit"/"CloudInit"? We generally call things after the upstream project names or packages that they involve. The launchpad and RTD pages are called 'cloud init' - Cloud Images seems to be the name of the product/service built on top of that at cloud-images.ubuntu.com.
Owner

utlemming replied Apr 13, 2015

bmr-cymru replied Apr 13, 2015

That's not really the model we use in sos plugins. Plugins generally map to components (either upstream projects or packages) not to the particular way things are integrated in any one instance.

So in this case the right way to do it is probably to have a cloud-init plugin that collects general data for that component and a separate cloudimages plugin that addresses any additional up-the-stack integration in that piece.

These aren't rigid rules and we're flexible when something doesn't entirely fit but where things are used by multiple distros we don't want to tie the plugin to the particular specifics used by one distro over the others.
@battlemidget
Owner

battlemidget commented May 5, 2015

@bmr-cymru agreed we should split this into 2 plugins. do you happen to know if cloud-init on RHEL/Fedora/CentOS keeps data in /run as well?
@bmr-cymru
Owner

bmr-cymru commented Sep 29, 2015

@battlemidget not off hand but I would imagine so - Fedora (via systemd) was one of the main distributions driving the /run adoption.

Anyway: we're getting close to a new release - it would be good to get this tidied up and ready for that but it's been four months now with no progress on the suggested split.

If we don't have something to merge to master soon it will not make 3.3.

@battlemidget battlemidget added the Changes Needed label Sep 29, 2015

@bmr-cymru
Owner

bmr-cymru commented Oct 21, 2016

Any updates to this PR? If not then I think we should archive the discussion in an issue and close it.
@battlemidget
Owner

battlemidget commented May 30, 2017 •
Edited 1 time

@bmr-cymru Yea lets archive this and close it
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment