Add juju status and get-constraints outputs

Signed-off-by: Ante Karamatic <ante.karamatic@canonical.com>
Rearrange openstack plugin so that distribution OpenStack class
contains all non-distribution specific details. Also dropped all
logrotate parts since those are collected by logrotate plugin.

Signed-off-by: Ante Karamatic <ante.karamatic@canonical.com>
cluster plugin: add obfuscation of luci secrets

Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
cluster plugin: restrict data collected for luci

Notably, avoid server cert being added.  On the other hand, allow
collection of rotated log files for luci.

Signed-off-by: Jan Pokorný <jpokorny@redhat.com>
Merge pull request #167 from ivoks/juju

Refactored OpenStack plugin
Merge pull request #168 from jnpkrn/patch-luci

cluster plugin: restrictions wrt. luci
Fix API usage in openstack plug-in

One call in openstack.py used the old addCopySpecs() method. This
will fail with current sos-3.x code. Update it to use the new
add_copy_specs() method instead.

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
Fix broken binary checks in openstack.py

The OpenStack plug-in attempts to check for the presence of
binaries before running them. To do this it passes a relative
path to os.path.exists(); clearly this will never succeed.

Ideally plug-ins should never need to do these checks. If the
binary exists we'll run it and if not we won't (with no harmful
side-effects). This is how sos is currently designed and if there
is a need for plug-ins to ever do this we should add a method to
the base plug-in class e.g. Plugin.find_command() to do this
properly (i.e. evaluating the policy-supplied PATH).

For now, just make all of this unconditional.

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
Delete bogus dist checks from RedHatOpenStack

We already know we are running on RHEL or Fedora if we are
executing a plug-in class tagged with RedHatPlugin. Do not check
for /etc/*-release.

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
Fix add_copy_specs(string) bugs in openstack.py

OpenStack was full of the add_copy_specs(string) anti-pattern.
While we should address this properly these instances all need
fixing to avoid collecting the whole file system.

They should also have been tested before being put forward as
a pull request.

Signed-off-by: Bryn M. Reeves <bmr@redhat.com>