From 43d156200ba679ec72f2a591d0b3c39041a7869b Mon Sep 17 00:00:00 2001 From: Jose Castillo Date: Wed, 12 Feb 2020 14:14:59 +0100 Subject: [openstack_horizon] fix useless forbidden paths The code has currently the following line: self.add_forbidden_path("*.py[co]") But add_forbidden_path() needs a directory prefix, so this won't work. This line was introduced via commit 4d75385ad to solve issue #856, that discussed the postproc method in openstack_horizon. It seems that the python object and source files are present in /etc/openstack-dashboard/local_settings.d/ used to customize the dashboard. This patch adds that directory to the add_forbidden_path() line. Closes: #1846 Signed-off-by: Jose Castillo Signed-off-by: Pavel Moravec --- sos/plugins/openstack_horizon.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/sos/plugins/openstack_horizon.py b/sos/plugins/openstack_horizon.py index 68999aa2..5acf1102 100644 --- a/sos/plugins/openstack_horizon.py +++ b/sos/plugins/openstack_horizon.py @@ -41,7 +41,9 @@ class OpenStackHorizon(Plugin): self.var_puppet_gen + "/horizon/etc/httpd/conf.modules.d/*.conf", self.var_puppet_gen + "/memcached/etc/sysconfig/memcached" ]) - self.add_forbidden_path("*.py[co]") + self.add_forbidden_path( + "/etc/openstack-dashboard/local_settings.d/*.py[co]" + ) def postproc(self): var_puppet_gen = self.var_puppet_gen + "/horizon" -- cgit