Blame SOURCES/sos-bz1463635-openstack-containerized-tripleo.patch

50f7da
From 665b85fcf25c016cbb467f8d9e3e93c2005b0fb9 Mon Sep 17 00:00:00 2001
50f7da
From: Bogdan Dobrelya <bdobreli@redhat.com>
50f7da
Date: Wed, 28 Jun 2017 21:34:39 +0200
50f7da
Subject: [PATCH] [plugins] Tripleo specific containerized services logs
50f7da
 (#1046)
50f7da
50f7da
Add Tripleo Pike opinionated logs paths to be collected
50f7da
for services, when running in containers. This is a temporary
50f7da
and will be reworked for Queens, like switching those to syslog
50f7da
or fluentd shipping logs to Elasticsearch cluster.
50f7da
50f7da
Partial bug: https://bugs.launchpad.net/tripleo/+bug/1700909
50f7da
Related blueprint: https://review.openstack.org/#/c/462900
50f7da
50f7da
Signed-off-by: Bogdan Dobrelya <bdobreli@redhat.com>
50f7da
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
50f7da
---
50f7da
 sos/plugins/mongodb.py              |  3 ++-
50f7da
 sos/plugins/mysql.py                |  2 ++
50f7da
 sos/plugins/openstack_ceilometer.py |  7 +++++--
50f7da
 sos/plugins/openstack_cinder.py     |  8 ++++++--
50f7da
 sos/plugins/openstack_glance.py     |  8 ++++++--
50f7da
 sos/plugins/openstack_heat.py       |  8 ++++++--
50f7da
 sos/plugins/openstack_horizon.py    |  8 ++++++--
50f7da
 sos/plugins/openstack_instack.py    | 12 ++++++++----
50f7da
 sos/plugins/openstack_ironic.py     |  8 ++++++--
50f7da
 sos/plugins/openstack_keystone.py   |  8 ++++++--
50f7da
 sos/plugins/openstack_manila.py     |  6 ++++--
50f7da
 sos/plugins/openstack_neutron.py    |  8 ++++++--
50f7da
 sos/plugins/openstack_nova.py       |  8 ++++++--
50f7da
 sos/plugins/openstack_sahara.py     |  8 ++++++--
50f7da
 sos/plugins/openstack_swift.py      |  8 ++++++--
50f7da
 sos/plugins/openstack_trove.py      |  8 ++++++--
50f7da
 sos/plugins/pacemaker.py            |  3 ++-
50f7da
 sos/plugins/rabbitmq.py             |  3 ++-
50f7da
 sos/plugins/redis.py                |  6 ++++--
50f7da
 19 files changed, 95 insertions(+), 35 deletions(-)
50f7da
50f7da
diff --git a/sos/plugins/mongodb.py b/sos/plugins/mongodb.py
50f7da
index 5d7d601..46b0251 100644
50f7da
--- a/sos/plugins/mongodb.py
50f7da
+++ b/sos/plugins/mongodb.py
50f7da
@@ -30,7 +30,8 @@ class MongoDb(Plugin, DebianPlugin, UbuntuPlugin):
50f7da
     def setup(self):
50f7da
         self.add_copy_spec([
50f7da
             "/etc/mongodb.conf",
50f7da
-            "/var/log/mongodb/mongodb.log"
50f7da
+            "/var/log/mongodb/mongodb.log",
50f7da
+            "/var/log/containers/mongodb/mongodb.log"
50f7da
         ])
50f7da
 
50f7da
     def postproc(self):
50f7da
diff --git a/sos/plugins/mysql.py b/sos/plugins/mysql.py
50f7da
index b750ed3..7bf933a 100644
50f7da
--- a/sos/plugins/mysql.py
50f7da
+++ b/sos/plugins/mysql.py
50f7da
@@ -40,12 +40,14 @@ class Mysql(Plugin):
50f7da
             # Required for MariaDB under pacemaker (MariaDB-Galera)
50f7da
             "/var/log/mysqld.log",
50f7da
             "/var/log/mysql/mysqld.log",
50f7da
+            "/var/log/containers/mysql/mysqld.log",
50f7da
             "/var/log/mariadb/mariadb.log",
50f7da
         ])
50f7da
 
50f7da
         if self.get_option("all_logs"):
50f7da
             self.add_copy_spec([
50f7da
                 "/var/log/mysql*",
50f7da
+                "/var/log/containers/mysql*",
50f7da
                 "/var/log/mariadb*"
50f7da
             ])
50f7da
 
50f7da
diff --git a/sos/plugins/openstack_ceilometer.py b/sos/plugins/openstack_ceilometer.py
50f7da
index 8e3e8e9..2c3a809 100644
50f7da
--- a/sos/plugins/openstack_ceilometer.py
50f7da
+++ b/sos/plugins/openstack_ceilometer.py
50f7da
@@ -32,9 +32,12 @@ class OpenStackCeilometer(Plugin):
50f7da
         # Ceilometer
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/ceilometer/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/ceilometer/",
50f7da
+                                "/var/log/containers/ceilometer/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/ceilometer/*.log",
50f7da
+            self.add_copy_spec(["/var/log/ceilometer/*.log",
50f7da
+                                "/var/log/containers/ceilometer/*.log"],
50f7da
                                sizelimit=self.limit)
50f7da
         self.add_copy_spec("/etc/ceilometer/")
50f7da
         if self.get_option("verify"):
50f7da
diff --git a/sos/plugins/openstack_cinder.py b/sos/plugins/openstack_cinder.py
50f7da
index 0383a8a..ffec62b 100644
50f7da
--- a/sos/plugins/openstack_cinder.py
50f7da
+++ b/sos/plugins/openstack_cinder.py
50f7da
@@ -40,9 +40,13 @@ class OpenStackCinder(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/cinder/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/cinder/",
50f7da
+                                "/var/log/containers/cinder/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/cinder/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/cinder/*.log",
50f7da
+                                "/var/log/containers/cinder/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
diff --git a/sos/plugins/openstack_glance.py b/sos/plugins/openstack_glance.py
50f7da
index 1aa0cb7..88d06e3 100644
50f7da
--- a/sos/plugins/openstack_glance.py
50f7da
+++ b/sos/plugins/openstack_glance.py
50f7da
@@ -38,9 +38,13 @@ class OpenStackGlance(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/glance/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/glance/",
50f7da
+                                "/var/log/containers/glance/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/glance/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/glance/*.log",
50f7da
+                                "/var/log/containers/glance/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec("/etc/glance/")
50f7da
 
50f7da
diff --git a/sos/plugins/openstack_heat.py b/sos/plugins/openstack_heat.py
50f7da
index 451bad3..2dad83f 100644
50f7da
--- a/sos/plugins/openstack_heat.py
50f7da
+++ b/sos/plugins/openstack_heat.py
50f7da
@@ -45,9 +45,13 @@ class OpenStackHeat(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/heat/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/heat/",
50f7da
+                                "/var/log/containers/heat/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/heat/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/heat/*.log",
50f7da
+                                "/var/log/containers/heat/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec("/etc/heat/")
50f7da
 
50f7da
diff --git a/sos/plugins/openstack_horizon.py b/sos/plugins/openstack_horizon.py
50f7da
index f778b91..75ad3ce 100644
50f7da
--- a/sos/plugins/openstack_horizon.py
50f7da
+++ b/sos/plugins/openstack_horizon.py
50f7da
@@ -32,9 +32,13 @@ class OpenStackHorizon(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/horizon/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/horizon/",
50f7da
+                                "/var/log/containers/horizon/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/horizon/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/horizon/*.log",
50f7da
+                                "/var/log/containers/horizon/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec("/etc/openstack-dashboard/")
50f7da
         self.add_forbidden_path("*.py[co]")
50f7da
diff --git a/sos/plugins/openstack_instack.py b/sos/plugins/openstack_instack.py
50f7da
index 3c87de8..87ec59f 100644
50f7da
--- a/sos/plugins/openstack_instack.py
50f7da
+++ b/sos/plugins/openstack_instack.py
50f7da
@@ -34,14 +34,18 @@ class OpenStackInstack(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/mistral/",
50f7da
+            self.add_copy_spec(["/var/log/mistral/",
50f7da
+                                "/var/log/containers/mistral/"],
50f7da
                                sizelimit=self.limit)
50f7da
-            self.add_copy_spec("/var/log/zaqar/",
50f7da
+            self.add_copy_spec(["/var/log/zaqar/",
50f7da
+                                "/var/log/containers/zaqar/"],
50f7da
                                sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/mistral/*.log",
50f7da
+            self.add_copy_spec(["/var/log/mistral/*.log",
50f7da
+                                "/var/log/containers/mistral/*.log"],
50f7da
                                sizelimit=self.limit)
50f7da
-            self.add_copy_spec("/var/log/zaqar/*.log",
50f7da
+            self.add_copy_spec(["/var/log/zaqar/*.log",
50f7da
+                                "/var/log/containers/zaqar/*.log"],
50f7da
                                sizelimit=self.limit)
50f7da
 
50f7da
         vars = [p in os.environ for p in [
50f7da
diff --git a/sos/plugins/openstack_ironic.py b/sos/plugins/openstack_ironic.py
50f7da
index f225427..e4e320d 100644
50f7da
--- a/sos/plugins/openstack_ironic.py
50f7da
+++ b/sos/plugins/openstack_ironic.py
50f7da
@@ -31,9 +31,13 @@ class OpenStackIronic(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/ironic/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/ironic/",
50f7da
+                                "/var/log/containers/ironic/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/ironic/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/ironic/*.log",
50f7da
+                                "/var/log/containers/ironic/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_cmd_output('ls -laRt /var/lib/ironic/')
50f7da
 
50f7da
diff --git a/sos/plugins/openstack_keystone.py b/sos/plugins/openstack_keystone.py
50f7da
index cb8a626..5a8f4e2 100644
50f7da
--- a/sos/plugins/openstack_keystone.py
50f7da
+++ b/sos/plugins/openstack_keystone.py
50f7da
@@ -37,9 +37,13 @@ class OpenStackKeystone(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/keystone/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/keystone/",
50f7da
+                                "/var/log/containers/keystone/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/keystone/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/keystone/*.log",
50f7da
+                                "/var/log/containers/keystone/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
diff --git a/sos/plugins/openstack_manila.py b/sos/plugins/openstack_manila.py
50f7da
index ce50a31..c1e6609 100644
50f7da
--- a/sos/plugins/openstack_manila.py
50f7da
+++ b/sos/plugins/openstack_manila.py
50f7da
@@ -29,10 +29,12 @@ class OpenStackManila(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/manila/*",
50f7da
+            self.add_copy_spec(["/var/log/manila/*",
50f7da
+                                "/var/log/containers/manila/*"],
50f7da
                                sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/manila/*.log",
50f7da
+            self.add_copy_spec(["/var/log/manila/*.log",
50f7da
+                                "/var/log/containers/manila/*.log"],
50f7da
                                sizelimit=self.limit)
50f7da
 
50f7da
     def postproc(self):
50f7da
diff --git a/sos/plugins/openstack_neutron.py b/sos/plugins/openstack_neutron.py
50f7da
index 62fd1c3..b0f6699 100644
50f7da
--- a/sos/plugins/openstack_neutron.py
50f7da
+++ b/sos/plugins/openstack_neutron.py
50f7da
@@ -29,9 +29,13 @@ class OpenStackNeutron(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/neutron/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/neutron/",
50f7da
+                                "/var/log/containers/neutron/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/neutron/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/neutron/*.log",
50f7da
+                                "/var/log/containers/neutron/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec("/etc/neutron/")
50f7da
         self.add_copy_spec("/var/lib/neutron/")
50f7da
diff --git a/sos/plugins/openstack_nova.py b/sos/plugins/openstack_nova.py
50f7da
index 14bdfbd..3dda7af 100644
50f7da
--- a/sos/plugins/openstack_nova.py
50f7da
+++ b/sos/plugins/openstack_nova.py
50f7da
@@ -63,9 +63,13 @@ class OpenStackNova(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/nova/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/nova/",
50f7da
+                                "/var/log/containers/nova/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/nova/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/nova/*.log",
50f7da
+                                "/var/log/containers/nova/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec("/etc/nova/")
50f7da
 
50f7da
diff --git a/sos/plugins/openstack_sahara.py b/sos/plugins/openstack_sahara.py
50f7da
index adbad32..adaedf9 100644
50f7da
--- a/sos/plugins/openstack_sahara.py
50f7da
+++ b/sos/plugins/openstack_sahara.py
50f7da
@@ -32,9 +32,13 @@ class OpenStackSahara(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/sahara/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/sahara/",
50f7da
+                                "/var/log/containers/sahara/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/sahara/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/sahara/*.log",
50f7da
+                                "/var/log/containers/sahara/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
diff --git a/sos/plugins/openstack_swift.py b/sos/plugins/openstack_swift.py
50f7da
index 790a4cb..9906b94 100644
50f7da
--- a/sos/plugins/openstack_swift.py
50f7da
+++ b/sos/plugins/openstack_swift.py
50f7da
@@ -31,9 +31,13 @@ class OpenStackSwift(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/swift/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/swift/",
50f7da
+                                "/var/log/containers/swift/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/swift/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/swift/*.log",
50f7da
+                                "/var/log/containers/swift/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec("/etc/swift/")
50f7da
 
50f7da
diff --git a/sos/plugins/openstack_trove.py b/sos/plugins/openstack_trove.py
50f7da
index 4d526b0..5183ad1 100644
50f7da
--- a/sos/plugins/openstack_trove.py
50f7da
+++ b/sos/plugins/openstack_trove.py
50f7da
@@ -30,9 +30,13 @@ class OpenStackTrove(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/trove/", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/trove/",
50f7da
+                                "/var/log/containers/trove/"],
50f7da
+                               sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/trove/*.log", sizelimit=self.limit)
50f7da
+            self.add_copy_spec(["/var/log/trove/*.log",
50f7da
+                                "/var/log/containers/trove/*.log"],
50f7da
+                               sizelimit=self.limit)
50f7da
 
50f7da
         self.add_copy_spec('/etc/trove/')
50f7da
 
50f7da
diff --git a/sos/plugins/pacemaker.py b/sos/plugins/pacemaker.py
50f7da
index 9775284..5780c81 100644
50f7da
--- a/sos/plugins/pacemaker.py
50f7da
+++ b/sos/plugins/pacemaker.py
50f7da
@@ -36,7 +36,8 @@ class Pacemaker(Plugin, DebianPlugin, UbuntuPlugin):
50f7da
             "/var/lib/pacemaker/cib/cib.xml",
50f7da
             self.defaults,
50f7da
             "/var/log/pacemaker.log",
50f7da
-            "/var/log/pcsd/pcsd.log"
50f7da
+            "/var/log/pcsd/pcsd.log",
50f7da
+            "/var/log/pacemaker/bundles/*/",
50f7da
         ])
50f7da
         self.add_cmd_output([
50f7da
             "crm_mon -1 -A -n -r -t",
50f7da
diff --git a/sos/plugins/rabbitmq.py b/sos/plugins/rabbitmq.py
50f7da
index d9871a0..8d73835 100644
50f7da
--- a/sos/plugins/rabbitmq.py
50f7da
+++ b/sos/plugins/rabbitmq.py
50f7da
@@ -29,6 +29,7 @@ class RabbitMQ(Plugin, RedHatPlugin, DebianPlugin, UbuntuPlugin):
50f7da
         self.add_cmd_output("rabbitmqctl list_policies")
50f7da
 
50f7da
         self.add_copy_spec("/etc/rabbitmq/*")
50f7da
-        self.add_copy_spec("/var/log/rabbitmq/*",
50f7da
+        self.add_copy_spec(["/var/log/rabbitmq/*",
50f7da
+                            "/var/log/containers/rabbitmq/*"],
50f7da
                            sizelimit=self.get_option('log_size'))
50f7da
 # vim: set et ts=4 sw=4 :
50f7da
diff --git a/sos/plugins/redis.py b/sos/plugins/redis.py
50f7da
index befa3d3..9ac2397 100644
50f7da
--- a/sos/plugins/redis.py
50f7da
+++ b/sos/plugins/redis.py
50f7da
@@ -32,10 +32,12 @@ class Redis(Plugin, RedHatPlugin):
50f7da
         self.limit = self.get_option("log_size")
50f7da
         self.add_cmd_output("redis-cli info")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/redis/redis.log*",
50f7da
+            self.add_copy_spec(["/var/log/redis/redis.log*",
50f7da
+                                "/var/log/containers/redis/redis.log*"],
50f7da
                                sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/redis/redis.log",
50f7da
+            self.add_copy_spec(["/var/log/redis/redis.log",
50f7da
+                                "/var/log/containers/redis/redis.log"],
50f7da
                                sizelimit=self.limit)
50f7da
 
50f7da
     def postproc(self):
50f7da
-- 
50f7da
2.7.5
50f7da
50f7da
From 5bbe32f49f3406368ac794e638b61a3234da1d11 Mon Sep 17 00:00:00 2001
50f7da
From: stuggi <github@schuppert.net>
50f7da
Date: Wed, 28 Jun 2017 21:35:45 +0200
50f7da
Subject: [PATCH] [openstack plugins] run openstack command when correct ENV is
50f7da
 set (#1048)
50f7da
50f7da
With Tripleo Pike the default ENV changed from OS_TENANT_NAME to
50f7da
OS_PROJECT_NAME. The commands of the openstack plugins against
50f7da
the env should run when either OSP_TENANT_NAME or OS_PROJECT_NAME
50f7da
is set.
50f7da
50f7da
Signed-off-by: Martin Schuppert mschuppe@redhat.com
50f7da
Signed-off-by: Adam Stokes <battlemidget@users.noreply.github.com>
50f7da
---
50f7da
 sos/plugins/gnocchi.py            | 10 +++++++---
50f7da
 sos/plugins/openstack_glance.py   | 10 +++++++---
50f7da
 sos/plugins/openstack_heat.py     | 10 +++++++---
50f7da
 sos/plugins/openstack_instack.py  | 10 +++++++---
50f7da
 sos/plugins/openstack_ironic.py   | 10 +++++++---
50f7da
 sos/plugins/openstack_keystone.py | 10 +++++++---
50f7da
 sos/plugins/openstack_neutron.py  | 10 +++++++---
50f7da
 sos/plugins/openstack_nova.py     | 10 +++++++---
50f7da
 8 files changed, 56 insertions(+), 24 deletions(-)
50f7da
50f7da
diff --git a/sos/plugins/gnocchi.py b/sos/plugins/gnocchi.py
50f7da
index 14baf55..dc79a0f 100644
50f7da
--- a/sos/plugins/gnocchi.py
50f7da
+++ b/sos/plugins/gnocchi.py
50f7da
@@ -44,9 +44,13 @@ class GnocchiPlugin(Plugin, RedHatPlugin):
50f7da
             self.add_copy_spec("/var/log/gnocchi/*.log",
50f7da
                                sizelimit=self.limit)
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_glance.py b/sos/plugins/openstack_glance.py
50f7da
index 88d06e3..6202c49 100644
50f7da
--- a/sos/plugins/openstack_glance.py
50f7da
+++ b/sos/plugins/openstack_glance.py
50f7da
@@ -51,9 +51,13 @@ class OpenStackGlance(Plugin):
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_heat.py b/sos/plugins/openstack_heat.py
50f7da
index 2dad83f..0f457d9 100644
50f7da
--- a/sos/plugins/openstack_heat.py
50f7da
+++ b/sos/plugins/openstack_heat.py
50f7da
@@ -34,9 +34,13 @@ class OpenStackHeat(Plugin):
50f7da
             suggest_filename="heat_db_version"
50f7da
         )
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_instack.py b/sos/plugins/openstack_instack.py
50f7da
index 87ec59f..f990476 100644
50f7da
--- a/sos/plugins/openstack_instack.py
50f7da
+++ b/sos/plugins/openstack_instack.py
50f7da
@@ -48,9 +48,13 @@ class OpenStackInstack(Plugin):
50f7da
                                 "/var/log/containers/zaqar/*.log"],
50f7da
                                sizelimit=self.limit)
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_ironic.py b/sos/plugins/openstack_ironic.py
50f7da
index e4e320d..0396cc6 100644
50f7da
--- a/sos/plugins/openstack_ironic.py
50f7da
+++ b/sos/plugins/openstack_ironic.py
50f7da
@@ -44,9 +44,13 @@ class OpenStackIronic(Plugin):
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_keystone.py b/sos/plugins/openstack_keystone.py
50f7da
index 5a8f4e2..82e987a 100644
50f7da
--- a/sos/plugins/openstack_keystone.py
50f7da
+++ b/sos/plugins/openstack_keystone.py
50f7da
@@ -48,9 +48,13 @@ class OpenStackKeystone(Plugin):
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_neutron.py b/sos/plugins/openstack_neutron.py
50f7da
index b0f6699..d2bfe6b 100644
50f7da
--- a/sos/plugins/openstack_neutron.py
50f7da
+++ b/sos/plugins/openstack_neutron.py
50f7da
@@ -42,9 +42,13 @@ class OpenStackNeutron(Plugin):
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
diff --git a/sos/plugins/openstack_nova.py b/sos/plugins/openstack_nova.py
50f7da
index 3dda7af..c0e8109 100644
50f7da
--- a/sos/plugins/openstack_nova.py
50f7da
+++ b/sos/plugins/openstack_nova.py
50f7da
@@ -35,9 +35,13 @@ class OpenStackNova(Plugin):
50f7da
         self.add_cmd_output("nova-manage fixed list")
50f7da
         self.add_cmd_output("nova-manage floating list")
50f7da
 
50f7da
-        vars = [p in os.environ for p in [
50f7da
-                'OS_USERNAME', 'OS_PASSWORD', 'OS_TENANT_NAME']]
50f7da
-        if not all(vars):
50f7da
+        vars_all = [p in os.environ for p in [
50f7da
+                    'OS_USERNAME', 'OS_PASSWORD']]
50f7da
+
50f7da
+        vars_any = [p in os.environ for p in [
50f7da
+                    'OS_TENANT_NAME', 'OS_PROJECT_NAME']]
50f7da
+
50f7da
+        if not (all(vars_all) and any(vars_any)):
50f7da
             self.soslog.warning("Not all environment variables set. Source "
50f7da
                                 "the environment file for the user intended "
50f7da
                                 "to connect to the OpenStack environment.")
50f7da
-- 
50f7da
2.7.5
50f7da
50f7da
From f379776da6d2d3f4e3f31bf34b12908674dc122d Mon Sep 17 00:00:00 2001
50f7da
From: Martin Schuppert <mschuppert@redhat.com>
50f7da
Date: Wed, 28 Jun 2017 16:21:05 +0200
50f7da
Subject: [PATCH] [gnocchi] Tripleo specific containerized services logs
50f7da
50f7da
This is an addition to pull request 1046 to collect gnocchi logs
50f7da
correct when service is running in a container.
50f7da
50f7da
Signed-off-by: Martin Schuppert mschuppert@redhat.com
50f7da
---
50f7da
 sos/plugins/gnocchi.py | 14 ++++++++++----
50f7da
 1 file changed, 10 insertions(+), 4 deletions(-)
50f7da
50f7da
diff --git a/sos/plugins/gnocchi.py b/sos/plugins/gnocchi.py
50f7da
index dc79a0f..724e41b 100644
50f7da
--- a/sos/plugins/gnocchi.py
50f7da
+++ b/sos/plugins/gnocchi.py
50f7da
@@ -38,11 +38,17 @@ class GnocchiPlugin(Plugin, RedHatPlugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec("/var/log/gnocchi/",
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/gnocchi/*",
50f7da
+                "/var/log/containers/gnocchi/*"],
50f7da
+                sizelimit=self.limit
50f7da
+            )
50f7da
         else:
50f7da
-            self.add_copy_spec("/var/log/gnocchi/*.log",
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/gnocchi/*.log",
50f7da
+                "/var/log/containers/gnocchi/*.log"],
50f7da
+                sizelimit=self.limit
50f7da
+            )
50f7da
 
50f7da
         vars_all = [p in os.environ for p in [
50f7da
                     'OS_USERNAME', 'OS_PASSWORD']]
50f7da
-- 
50f7da
2.7.5
50f7da
50f7da
From 55800f71bdcfdea22b1d6e1c68848e58eb087d1c Mon Sep 17 00:00:00 2001
50f7da
From: Martin Schuppert <mschuppert@redhat.com>
50f7da
Date: Sun, 9 Jul 2017 15:22:30 +0200
50f7da
Subject: [PATCH] [openstack plugins] Tripleo specific containerized services
50f7da
 configs
50f7da
50f7da
Add Tripleo Pike opinionated config paths to be collected
50f7da
for services, when running in containers. Each service has a
50f7da
config dir in /var/lib/config-data/puppet-generated on the host
50f7da
which gets bind mounted read only into the container.
50f7da
50f7da
Closes #1054
50f7da
50f7da
Signed-off-by: Martin Schuppert <mschuppe@redhat.com>
50f7da
Signed-off-by: Bryn M. Reeves <bmr@redhat.com>
50f7da
---
50f7da
 sos/plugins/gnocchi.py              | 18 +++++++++++++-
50f7da
 sos/plugins/haproxy.py              |  6 ++++-
50f7da
 sos/plugins/iscsi.py                |  2 ++
50f7da
 sos/plugins/mongodb.py              | 15 +++++++++++-
50f7da
 sos/plugins/mysql.py                |  5 ++--
50f7da
 sos/plugins/openstack_ceilometer.py | 24 ++++++++++++------
50f7da
 sos/plugins/openstack_glance.py     | 25 +++++++++++++------
50f7da
 sos/plugins/openstack_heat.py       | 49 ++++++++++++++++++++++++++++++-------
50f7da
 sos/plugins/openstack_horizon.py    | 48 ++++++++++++++++++++++++++----------
50f7da
 sos/plugins/openstack_keystone.py   | 28 +++++++++++++++------
50f7da
 sos/plugins/openstack_manila.py     | 25 +++++++++++++------
50f7da
 sos/plugins/openstack_neutron.py    | 27 ++++++++++++++------
50f7da
 sos/plugins/openstack_nova.py       | 37 ++++++++++++++++++++++------
50f7da
 sos/plugins/openstack_sahara.py     | 24 ++++++++++++------
50f7da
 sos/plugins/openstack_swift.py      | 30 +++++++++++++++++------
50f7da
 sos/plugins/openstack_trove.py      | 25 +++++++++++++------
50f7da
 sos/plugins/rabbitmq.py             | 22 +++++++++++++----
50f7da
 sos/plugins/redis.py                | 40 ++++++++++++++++++++++++------
50f7da
 18 files changed, 345 insertions(+), 105 deletions(-)
50f7da
50f7da
diff --git a/sos/plugins/gnocchi.py b/sos/plugins/gnocchi.py
50f7da
index 724e41b..c7a9709 100644
50f7da
--- a/sos/plugins/gnocchi.py
50f7da
+++ b/sos/plugins/gnocchi.py
50f7da
@@ -33,8 +33,17 @@ class GnocchiPlugin(Plugin, RedHatPlugin):
50f7da
 
50f7da
     requires_root = False
50f7da
 
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/gnocchi"
50f7da
+
50f7da
     def setup(self):
50f7da
-        self.add_copy_spec("/etc/gnocchi/")
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/gnocchi/*",
50f7da
+            self.var_puppet_gen + "/etc/gnocchi/*",
50f7da
+            self.var_puppet_gen + "/etc/httpd/conf/*",
50f7da
+            self.var_puppet_gen + "/etc/httpd/conf.d/*",
50f7da
+            self.var_puppet_gen + "/etc/httpd/conf.modules.d/wsgi.conf",
50f7da
+            self.var_puppet_gen + "/etc/my.cnf.d/tripleo.cnf"
50f7da
+        ])
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
@@ -76,5 +85,12 @@ class GnocchiPlugin(Plugin, RedHatPlugin):
50f7da
             r"password=(.*)",
50f7da
             r"password=*****",
50f7da
         )
50f7da
+        self.do_file_sub(
50f7da
+            self.var_puppet_gen + "/etc/gnocchi/"
50f7da
+            "gnocchi.conf",
50f7da
+            r"password=(.*)",
50f7da
+            r"password=*****",
50f7da
+        )
50f7da
+
50f7da
 
50f7da
 # vim: set et ts=4 sw=4 :
50f7da
diff --git a/sos/plugins/haproxy.py b/sos/plugins/haproxy.py
50f7da
index 6ec98d2..9084506 100644
50f7da
--- a/sos/plugins/haproxy.py
50f7da
+++ b/sos/plugins/haproxy.py
50f7da
@@ -27,7 +27,11 @@ class HAProxy(Plugin, RedHatPlugin, DebianPlugin):
50f7da
     packages = ('haproxy',)
50f7da
 
50f7da
     def setup(self):
50f7da
-        self.add_copy_spec("/etc/haproxy/haproxy.cfg")
50f7da
+        var_puppet_gen = "/var/lib/config-data/puppet-generated/haproxy"
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/haproxy/haproxy.cfg",
50f7da
+            var_puppet_gen + "/etc/haproxy/haproxy.cfg"
50f7da
+        ])
50f7da
         self.add_copy_spec("/etc/haproxy/conf.d/*")
50f7da
         self.add_cmd_output("haproxy -f /etc/haproxy/haproxy.cfg -c")
50f7da
 
50f7da
diff --git a/sos/plugins/iscsi.py b/sos/plugins/iscsi.py
50f7da
index 7324b26..66aa632 100644
50f7da
--- a/sos/plugins/iscsi.py
50f7da
+++ b/sos/plugins/iscsi.py
50f7da
@@ -30,9 +30,11 @@ class RedHatIscsi(Iscsi, RedHatPlugin):
50f7da
 
50f7da
     def setup(self):
50f7da
         super(RedHatIscsi, self).setup()
50f7da
+        var_puppet_gen = "/var/lib/config-data/puppet-generated/iscsid"
50f7da
         self.add_copy_spec([
50f7da
             "/etc/iscsi/iscsid.conf",
50f7da
             "/etc/iscsi/initiatorname.iscsi",
50f7da
+            var_puppet_gen + "/etc/iscsi/initiatorname.iscsi",
50f7da
             "/var/lib/iscsi"
50f7da
         ])
50f7da
         self.add_cmd_output([
50f7da
diff --git a/sos/plugins/mongodb.py b/sos/plugins/mongodb.py
50f7da
index 46b0251..8c89517 100644
50f7da
--- a/sos/plugins/mongodb.py
50f7da
+++ b/sos/plugins/mongodb.py
50f7da
@@ -25,11 +25,18 @@ class MongoDb(Plugin, DebianPlugin, UbuntuPlugin):
50f7da
     profiles = ('services',)
50f7da
 
50f7da
     packages = ('mongodb-server',)
50f7da
-    files = ('/etc/mongodb.conf',)
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/mongodb"
50f7da
+
50f7da
+    files = (
50f7da
+        '/etc/mongodb.conf',
50f7da
+        var_puppet_gen + '/etc/mongod.conf'
50f7da
+    )
50f7da
 
50f7da
     def setup(self):
50f7da
         self.add_copy_spec([
50f7da
             "/etc/mongodb.conf",
50f7da
+            self.var_puppet_gen + "/etc/",
50f7da
+            self.var_puppet_gen + "/etc/systemd/system/mongod.service.d/",
50f7da
             "/var/log/mongodb/mongodb.log",
50f7da
             "/var/log/containers/mongodb/mongodb.log"
50f7da
         ])
50f7da
@@ -41,6 +48,12 @@ class MongoDb(Plugin, DebianPlugin, UbuntuPlugin):
50f7da
             r"mms-token = ********"
50f7da
         )
50f7da
 
50f7da
+        self.do_file_sub(
50f7da
+            self.var_puppet_gen + "/etc/mongodb.conf",
50f7da
+            r"(mms-token\s*=\s*.*)",
50f7da
+            r"mms-token = ********"
50f7da
+        )
50f7da
+
50f7da
 
50f7da
 class RedHatMongoDb(MongoDb, RedHatPlugin):
50f7da
 
50f7da
diff --git a/sos/plugins/mysql.py b/sos/plugins/mysql.py
50f7da
index 7bf933a..decca69 100644
50f7da
--- a/sos/plugins/mysql.py
50f7da
+++ b/sos/plugins/mysql.py
50f7da
@@ -93,9 +93,10 @@ class RedHatMysql(Mysql, RedHatPlugin):
50f7da
         super(RedHatMysql, self).setup()
50f7da
         self.add_copy_spec([
50f7da
             "/etc/ld.so.conf.d/mysql-*.conf",
50f7da
-            "/etc/ld.so.conf.d/mariadb-*.conf"
50f7da
+            "/etc/ld.so.conf.d/mariadb-*.conf",
50f7da
+            "/etc/my.cnf.d/*",
50f7da
+            "/var/lib/config-data/puppet-generated/mysql/etc/my.cnf.d/*"
50f7da
         ])
50f7da
-        self.add_copy_spec("/etc/my.cnf.d/*")
50f7da
 
50f7da
 
50f7da
 class DebianMysql(Mysql, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_ceilometer.py b/sos/plugins/openstack_ceilometer.py
50f7da
index 2c3a809..b3c6bd8 100644
50f7da
--- a/sos/plugins/openstack_ceilometer.py
50f7da
+++ b/sos/plugins/openstack_ceilometer.py
50f7da
@@ -27,19 +27,25 @@ class OpenStackCeilometer(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller', 'openstack_compute')
50f7da
 
50f7da
     option_list = []
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/ceilometer"
50f7da
 
50f7da
     def setup(self):
50f7da
         # Ceilometer
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/ceilometer/",
50f7da
-                                "/var/log/containers/ceilometer/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/ceilometer/*",
50f7da
+                "/var/log/containers/ceilometer/*"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/ceilometer/*.log",
50f7da
-                                "/var/log/containers/ceilometer/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
-        self.add_copy_spec("/etc/ceilometer/")
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/ceilometer/*.log",
50f7da
+                "/var/log/containers/ceilometer/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/ceilometer/*",
50f7da
+            self.var_puppet_gen + "/etc/ceilometer/*"
50f7da
+        ])
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
 
50f7da
@@ -54,6 +60,10 @@ class OpenStackCeilometer(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/ceilometer/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/ceilometer/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianCeilometer(OpenStackCeilometer, DebianPlugin,
50f7da
diff --git a/sos/plugins/openstack_glance.py b/sos/plugins/openstack_glance.py
50f7da
index 6202c49..0a2bdc1 100644
50f7da
--- a/sos/plugins/openstack_glance.py
50f7da
+++ b/sos/plugins/openstack_glance.py
50f7da
@@ -28,6 +28,7 @@ class OpenStackGlance(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
 
50f7da
     option_list = []
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/glance_api"
50f7da
 
50f7da
     def setup(self):
50f7da
         # Glance
50f7da
@@ -38,15 +39,21 @@ class OpenStackGlance(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/glance/",
50f7da
-                                "/var/log/containers/glance/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/glance/",
50f7da
+                "/var/log/containers/glance/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/glance/*.log",
50f7da
-                                "/var/log/containers/glance/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/glance/*.log",
50f7da
+                "/var/log/containers/glance/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
-        self.add_copy_spec("/etc/glance/")
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/glance/",
50f7da
+            self.var_puppet_gen + "/etc/glance/",
50f7da
+            self.var_puppet_gen + "/etc/my.cnf.d/tripleo.cnf"
50f7da
+        ])
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -73,6 +80,10 @@ class OpenStackGlance(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/glance/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/glance/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianGlance(OpenStackGlance, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_heat.py b/sos/plugins/openstack_heat.py
50f7da
index 0f457d9..a73916b 100644
50f7da
--- a/sos/plugins/openstack_heat.py
50f7da
+++ b/sos/plugins/openstack_heat.py
50f7da
@@ -26,6 +26,7 @@ class OpenStackHeat(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
 
50f7da
     option_list = []
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/heat"
50f7da
 
50f7da
     def setup(self):
50f7da
         # Heat
50f7da
@@ -49,15 +50,31 @@ class OpenStackHeat(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/heat/",
50f7da
-                                "/var/log/containers/heat/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/heat/",
50f7da
+                "/var/log/containers/heat/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/heat/*.log",
50f7da
-                                "/var/log/containers/heat/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
-
50f7da
-        self.add_copy_spec("/etc/heat/")
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/heat/*.log",
50f7da
+                "/var/log/containers/heat/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
+
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/heat/",
50f7da
+            self.var_puppet_gen + "/etc/heat/",
50f7da
+            self.var_puppet_gen + "/etc/my.cnf.d/tripleo.cnf",
50f7da
+            self.var_puppet_gen + "_api/etc/heat/",
50f7da
+            self.var_puppet_gen + "_api/etc/httpd/conf/",
50f7da
+            self.var_puppet_gen + "_api/etc/httpd/conf.d/",
50f7da
+            self.var_puppet_gen + "_api/etc/httpd/conf.modules.d/*.conf",
50f7da
+            self.var_puppet_gen + "_api/var/spool/cron/heat",
50f7da
+            self.var_puppet_gen + "_api_cfn/etc/heat/",
50f7da
+            self.var_puppet_gen + "_api_cfn/etc/httpd/conf/",
50f7da
+            self.var_puppet_gen + "_api_cfn/etc/httpd/conf.d/",
50f7da
+            self.var_puppet_gen + "_api_cfn/etc/httpd/conf.modules.d/*.conf",
50f7da
+            self.var_puppet_gen + "_api_cfn/var/spool/cron/heat",
50f7da
+        ])
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -69,7 +86,21 @@ class OpenStackHeat(Plugin):
50f7da
         ]
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
-        self.do_path_regex_sub("/etc/heat/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            "/etc/heat/*",
50f7da
+            regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/heat/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "_api/etc/heat/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "_api_cfn/etc/heat/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianHeat(OpenStackHeat, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_horizon.py b/sos/plugins/openstack_horizon.py
50f7da
index 75ad3ce..a9a7fab 100644
50f7da
--- a/sos/plugins/openstack_horizon.py
50f7da
+++ b/sos/plugins/openstack_horizon.py
50f7da
@@ -27,36 +27,58 @@ class OpenStackHorizon(Plugin):
50f7da
     plugin_name = "openstack_horizon"
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
     option_list = []
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated"
50f7da
 
50f7da
     def setup(self):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/horizon/",
50f7da
-                                "/var/log/containers/horizon/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/horizon/",
50f7da
+                "/var/log/containers/horizon/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/horizon/*.log",
50f7da
-                                "/var/log/containers/horizon/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
-
50f7da
-        self.add_copy_spec("/etc/openstack-dashboard/")
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/horizon/*.log",
50f7da
+                "/var/log/containers/horizon/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
+
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/openstack-dashboard/",
50f7da
+            self.var_puppet_gen + "/horizon/etc/openstack-dashboard/",
50f7da
+            self.var_puppet_gen + "/horizon/etc/httpd/conf/",
50f7da
+            self.var_puppet_gen + "/horizon/etc/httpd/conf.d/",
50f7da
+            self.var_puppet_gen + "/horizon/etc/httpd/conf.modules.d/*.conf",
50f7da
+            self.var_puppet_gen + "/memcached/etc/sysconfig/memcached"
50f7da
+        ])
50f7da
         self.add_forbidden_path("*.py[co]")
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
 
50f7da
     def postproc(self):
50f7da
+        var_puppet_gen = self.var_puppet_gen + "/horizon"
50f7da
         protect_keys = [
50f7da
             "SECRET_KEY", "EMAIL_HOST_PASSWORD"
50f7da
         ]
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
-        self.do_path_regex_sub("/etc/openstack-dashboard/.*\.json",
50f7da
-                               regexp, r"\1*********")
50f7da
-        self.do_path_regex_sub("/etc/openstack-dashboard/local_settings/ + \
50f7da
-                               .*\.conf.*",
50f7da
-                               regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            "/etc/openstack-dashboard/.*\.json",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            var_puppet_gen + "/etc/openstack-dashboard/.*\.json",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            "/etc/openstack-dashboard/local_settings/.*\.conf.*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            var_puppet_gen + "/etc/openstack-dashboard/.*\.conf.*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianHorizon(OpenStackHorizon, DebianPlugin):
50f7da
diff --git a/sos/plugins/openstack_keystone.py b/sos/plugins/openstack_keystone.py
50f7da
index 82e987a..bdcb61d 100644
50f7da
--- a/sos/plugins/openstack_keystone.py
50f7da
+++ b/sos/plugins/openstack_keystone.py
50f7da
@@ -26,24 +26,34 @@ class OpenStackKeystone(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
 
50f7da
     option_list = [("nopw", "dont gathers keystone passwords", "slow", True)]
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/keystone"
50f7da
 
50f7da
     def setup(self):
50f7da
         self.add_copy_spec([
50f7da
             "/etc/keystone/default_catalog.templates",
50f7da
             "/etc/keystone/keystone.conf",
50f7da
             "/etc/keystone/logging.conf",
50f7da
-            "/etc/keystone/policy.json"
50f7da
+            "/etc/keystone/policy.json",
50f7da
+            self.var_puppet_gen + "/etc/keystone/*.conf",
50f7da
+            self.var_puppet_gen + "/etc/keystone/*.json",
50f7da
+            self.var_puppet_gen + "/etc/httpd/conf/",
50f7da
+            self.var_puppet_gen + "/etc/httpd/conf.d/",
50f7da
+            self.var_puppet_gen + "/etc/httpd/conf.modules.d/*.conf",
50f7da
+            self.var_puppet_gen + "/var/spool/cron/",
50f7da
+            self.var_puppet_gen + "/etc/my.cnf.d/tripleo.cnf"
50f7da
         ])
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/keystone/",
50f7da
-                                "/var/log/containers/keystone/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/keystone/",
50f7da
+                "/var/log/containers/keystone/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/keystone/*.log",
50f7da
-                                "/var/log/containers/keystone/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/keystone/*.log",
50f7da
+                "/var/log/containers/keystone/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -71,6 +81,10 @@ class OpenStackKeystone(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/keystone/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/keystone/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianKeystone(OpenStackKeystone, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_manila.py b/sos/plugins/openstack_manila.py
50f7da
index c1e6609..92bca6f 100644
50f7da
--- a/sos/plugins/openstack_manila.py
50f7da
+++ b/sos/plugins/openstack_manila.py
50f7da
@@ -24,18 +24,25 @@ class OpenStackManila(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
     option_list = []
50f7da
 
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/manila"
50f7da
+
50f7da
     def setup(self):
50f7da
-        self.add_copy_spec("/etc/manila/")
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/manila/",
50f7da
+            self.var_puppet_gen + "/etc/manila/"
50f7da
+        ])
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/manila/*",
50f7da
-                                "/var/log/containers/manila/*"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/manila/*",
50f7da
+                "/var/log/containers/manila/*"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/manila/*.log",
50f7da
-                                "/var/log/containers/manila/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/manila/*.log",
50f7da
+                "/var/log/containers/manila/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
     def postproc(self):
50f7da
         protect_keys = [
50f7da
@@ -47,6 +54,10 @@ class OpenStackManila(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/manila/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/manila/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianManila(OpenStackManila, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_neutron.py b/sos/plugins/openstack_neutron.py
50f7da
index d2bfe6b..b6116c7 100644
50f7da
--- a/sos/plugins/openstack_neutron.py
50f7da
+++ b/sos/plugins/openstack_neutron.py
50f7da
@@ -25,19 +25,26 @@ class OpenStackNeutron(Plugin):
50f7da
     plugin_name = "openstack_neutron"
50f7da
     profiles = ('openstack', 'openstack_controller', 'openstack_compute')
50f7da
 
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/neutron"
50f7da
+
50f7da
     def setup(self):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/neutron/",
50f7da
-                                "/var/log/containers/neutron/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/neutron/",
50f7da
+                "/var/log/containers/neutron/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/neutron/*.log",
50f7da
-                                "/var/log/containers/neutron/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
-
50f7da
-        self.add_copy_spec("/etc/neutron/")
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/neutron/*.log",
50f7da
+                "/var/log/containers/neutron/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
+
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/neutron/",
50f7da
+            self.var_puppet_gen + "/etc/neutron/"
50f7da
+        ])
50f7da
         self.add_copy_spec("/var/lib/neutron/")
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -75,6 +82,10 @@ class OpenStackNeutron(Plugin):
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
 
50f7da
         self.do_path_regex_sub("/etc/neutron/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/neutron/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianNeutron(OpenStackNeutron, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_nova.py b/sos/plugins/openstack_nova.py
50f7da
index c0e8109..75dd372 100644
50f7da
--- a/sos/plugins/openstack_nova.py
50f7da
+++ b/sos/plugins/openstack_nova.py
50f7da
@@ -29,6 +29,8 @@ class OpenStackNova(Plugin):
50f7da
     plugin_name = "openstack_nova"
50f7da
     profiles = ('openstack', 'openstack_controller', 'openstack_compute')
50f7da
 
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/nova"
50f7da
+
50f7da
     def setup(self):
50f7da
         # commands we do not need to source the environment file
50f7da
         self.add_cmd_output("nova-manage db version")
50f7da
@@ -67,15 +69,28 @@ class OpenStackNova(Plugin):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/nova/",
50f7da
-                                "/var/log/containers/nova/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/nova/",
50f7da
+                "/var/log/containers/nova/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/nova/*.log",
50f7da
-                                "/var/log/containers/nova/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/nova/*.log",
50f7da
+                "/var/log/containers/nova/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
-        self.add_copy_spec("/etc/nova/")
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/nova/",
50f7da
+            self.var_puppet_gen + "/etc/nova/",
50f7da
+            self.var_puppet_gen + "/etc/my.cnf.d/tripleo.cnf",
50f7da
+            self.var_puppet_gen + "_placement/var/spool/cron/nova",
50f7da
+            self.var_puppet_gen + "_placement/etc/nova/",
50f7da
+            self.var_puppet_gen + "_placement/etc/httpd/conf/",
50f7da
+            self.var_puppet_gen + "_placement/etc/httpd/conf.d/",
50f7da
+            self.var_puppet_gen + "_placement/etc/httpd/conf.modules.d/*.conf",
50f7da
+            self.var_puppet_gen + "_placement/etc/my.cnf.d/tripleo.cnf",
50f7da
+            self.var_puppet_gen + "/../memcached/etc/sysconfig/memcached"
50f7da
+        ])
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -92,6 +107,14 @@ class OpenStackNova(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/nova/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/nova/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "_placement/etc/nova/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianNova(OpenStackNova, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_sahara.py b/sos/plugins/openstack_sahara.py
50f7da
index adaedf9..68cf0b3 100644
50f7da
--- a/sos/plugins/openstack_sahara.py
50f7da
+++ b/sos/plugins/openstack_sahara.py
50f7da
@@ -23,22 +23,28 @@ class OpenStackSahara(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
 
50f7da
     option_list = []
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/sahara"
50f7da
 
50f7da
     def setup(self):
50f7da
-        self.add_copy_spec("/etc/sahara/")
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/sahara/",
50f7da
+            self.var_puppet_gen + "/etc/sahara/"
50f7da
+        ])
50f7da
         self.add_journal(units="openstack-sahara-all")
50f7da
         self.add_journal(units="openstack-sahara-api")
50f7da
         self.add_journal(units="openstack-sahara-engine")
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/sahara/",
50f7da
-                                "/var/log/containers/sahara/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/sahara/",
50f7da
+                "/var/log/containers/sahara/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/sahara/*.log",
50f7da
-                                "/var/log/containers/sahara/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/sahara/*.log",
50f7da
+                "/var/log/containers/sahara/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -52,6 +58,10 @@ class OpenStackSahara(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/sahara/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/sahara/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianSahara(OpenStackSahara, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_swift.py b/sos/plugins/openstack_swift.py
50f7da
index 9906b94..1cfff2b 100644
50f7da
--- a/sos/plugins/openstack_swift.py
50f7da
+++ b/sos/plugins/openstack_swift.py
50f7da
@@ -27,19 +27,29 @@ class OpenStackSwift(Plugin):
50f7da
 
50f7da
     option_list = []
50f7da
 
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated"
50f7da
+
50f7da
     def setup(self):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/swift/",
50f7da
-                                "/var/log/containers/swift/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/swift/",
50f7da
+                "/var/log/containers/swift/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/swift/*.log",
50f7da
-                                "/var/log/containers/swift/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
-
50f7da
-        self.add_copy_spec("/etc/swift/")
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/swift/*.log",
50f7da
+                "/var/log/containers/swift/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
+
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/swift/",
50f7da
+            self.var_puppet_gen + "/swift/etc/*",
50f7da
+            self.var_puppet_gen + "/swift/etc/swift/*",
50f7da
+            self.var_puppet_gen + "/swift/etc/xinetd.d/*",
50f7da
+            self.var_puppet_gen + "/memcached/etc/sysconfig/memcached"
50f7da
+        ])
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -54,6 +64,10 @@ class OpenStackSwift(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/swift/.*\.conf.*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/swift/etc/swift/.*\.conf.*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianSwift(OpenStackSwift, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/openstack_trove.py b/sos/plugins/openstack_trove.py
50f7da
index 5183ad1..093e5d5 100644
50f7da
--- a/sos/plugins/openstack_trove.py
50f7da
+++ b/sos/plugins/openstack_trove.py
50f7da
@@ -26,19 +26,26 @@ class OpenStackTrove(Plugin):
50f7da
     profiles = ('openstack', 'openstack_controller')
50f7da
     option_list = []
50f7da
 
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/trove"
50f7da
+
50f7da
     def setup(self):
50f7da
 
50f7da
         self.limit = self.get_option("log_size")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/trove/",
50f7da
-                                "/var/log/containers/trove/"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/trove/",
50f7da
+                "/var/log/containers/trove/"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/trove/*.log",
50f7da
-                                "/var/log/containers/trove/*.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/trove/*.log",
50f7da
+                "/var/log/containers/trove/*.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
-        self.add_copy_spec('/etc/trove/')
50f7da
+        self.add_copy_spec([
50f7da
+            '/etc/trove/',
50f7da
+            self.var_puppet_gen + '/etc/trove/'
50f7da
+        ])
50f7da
 
50f7da
         if self.get_option("verify"):
50f7da
             self.add_cmd_output("rpm -V %s" % ' '.join(self.packages))
50f7da
@@ -53,6 +60,10 @@ class OpenStackTrove(Plugin):
50f7da
 
50f7da
         regexp = r"((?m)^\s*(%s)\s*=\s*)(.*)" % "|".join(protect_keys)
50f7da
         self.do_path_regex_sub("/etc/trove/*", regexp, r"\1*********")
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/trove/*",
50f7da
+            regexp, r"\1*********"
50f7da
+        )
50f7da
 
50f7da
 
50f7da
 class DebianTrove(OpenStackTrove, DebianPlugin, UbuntuPlugin):
50f7da
diff --git a/sos/plugins/rabbitmq.py b/sos/plugins/rabbitmq.py
50f7da
index 8d73835..2c7e428 100644
50f7da
--- a/sos/plugins/rabbitmq.py
50f7da
+++ b/sos/plugins/rabbitmq.py
50f7da
@@ -20,7 +20,11 @@ class RabbitMQ(Plugin, RedHatPlugin, DebianPlugin, UbuntuPlugin):
50f7da
     """
50f7da
     plugin_name = 'rabbitmq'
50f7da
     profiles = ('services',)
50f7da
-    files = ('/etc/rabbitmq/rabbitmq.conf',)
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/rabbitmq"
50f7da
+    files = (
50f7da
+        '/etc/rabbitmq/rabbitmq.conf',
50f7da
+        var_puppet_gen + '/etc/rabbitmq/rabbitmq.config'
50f7da
+    )
50f7da
     packages = ('rabbitmq-server',)
50f7da
 
50f7da
     def setup(self):
50f7da
@@ -28,8 +32,16 @@ class RabbitMQ(Plugin, RedHatPlugin, DebianPlugin, UbuntuPlugin):
50f7da
         self.add_cmd_output("rabbitmqctl cluster_status")
50f7da
         self.add_cmd_output("rabbitmqctl list_policies")
50f7da
 
50f7da
-        self.add_copy_spec("/etc/rabbitmq/*")
50f7da
-        self.add_copy_spec(["/var/log/rabbitmq/*",
50f7da
-                            "/var/log/containers/rabbitmq/*"],
50f7da
-                           sizelimit=self.get_option('log_size'))
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/rabbitmq/*",
50f7da
+            self.var_puppet_gen + "/etc/rabbitmq/*",
50f7da
+            self.var_puppet_gen + "/etc/security/limits.d/",
50f7da
+            self.var_puppet_gen + "/etc/systemd/"
50f7da
+        ])
50f7da
+        self.add_copy_spec([
50f7da
+            "/var/log/rabbitmq/*",
50f7da
+            "/var/log/containers/rabbitmq/*"
50f7da
+        ], sizelimit=self.get_option('log_size'))
50f7da
+
50f7da
+
50f7da
 # vim: set et ts=4 sw=4 :
50f7da
diff --git a/sos/plugins/redis.py b/sos/plugins/redis.py
50f7da
index 9ac2397..088a9a2 100644
50f7da
--- a/sos/plugins/redis.py
50f7da
+++ b/sos/plugins/redis.py
50f7da
@@ -1,4 +1,5 @@
50f7da
 # Copyright (C) 2015 Red Hat, Inc., Abhijeet Kasurde <akasurde@redhat.com>
50f7da
+# Copyright (C) 2017 Red Hat, Inc., Martin Schuppert <mschuppe@redhat.com>
50f7da
 
50f7da
 # This program is free software; you can redistribute it and/or modify
50f7da
 # it under the terms of the GNU General Public License as published by
50f7da
@@ -25,20 +26,32 @@ class Redis(Plugin, RedHatPlugin):
50f7da
     profiles = ('services',)
50f7da
 
50f7da
     packages = ('redis',)
50f7da
-    files = ('/etc/redis.conf', '/var/log/redis')
50f7da
+    var_puppet_gen = "/var/lib/config-data/puppet-generated/redis"
50f7da
+    files = (
50f7da
+        '/etc/redis.conf',
50f7da
+        '/var/log/redis',
50f7da
+        var_puppet_gen + '/etc/redis.conf'
50f7da
+    )
50f7da
 
50f7da
     def setup(self):
50f7da
-        self.add_copy_spec("/etc/redis.conf")
50f7da
+        self.add_copy_spec([
50f7da
+            "/etc/redis.conf",
50f7da
+            self.var_puppet_gen + "/etc/redis*",
50f7da
+            self.var_puppet_gen + "/etc/redis/",
50f7da
+            self.var_puppet_gen + "/etc/security/limits.d/"
50f7da
+        ])
50f7da
         self.limit = self.get_option("log_size")
50f7da
         self.add_cmd_output("redis-cli info")
50f7da
         if self.get_option("all_logs"):
50f7da
-            self.add_copy_spec(["/var/log/redis/redis.log*",
50f7da
-                                "/var/log/containers/redis/redis.log*"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/redis/redis.log*",
50f7da
+                "/var/log/containers/redis/redis.log*"
50f7da
+            ], sizelimit=self.limit)
50f7da
         else:
50f7da
-            self.add_copy_spec(["/var/log/redis/redis.log",
50f7da
-                                "/var/log/containers/redis/redis.log"],
50f7da
-                               sizelimit=self.limit)
50f7da
+            self.add_copy_spec([
50f7da
+                "/var/log/redis/redis.log",
50f7da
+                "/var/log/containers/redis/redis.log"
50f7da
+            ], sizelimit=self.limit)
50f7da
 
50f7da
     def postproc(self):
50f7da
         self.do_file_sub(
50f7da
@@ -51,5 +64,16 @@ class Redis(Plugin, RedHatPlugin):
50f7da
             r"(requirepass\s).*",
50f7da
             r"\1********"
50f7da
         )
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/redis.conf*",
50f7da
+            r"(masterauth\s).*",
50f7da
+            r"\1*********"
50f7da
+        )
50f7da
+        self.do_path_regex_sub(
50f7da
+            self.var_puppet_gen + "/etc/redis.conf*",
50f7da
+            r"(requirepass\s).*",
50f7da
+            r"\1*********"
50f7da
+        )
50f7da
+
50f7da
 
50f7da
 # vim: set et ts=4 sw=4 :
50f7da
-- 
50f7da
2.7.5
50f7da