diff options
| author | Jose Castillo <jose.mfcastillo@gmail.com> | 2020-02-12 14:14:59 +0100 |
|---|---|---|
| committer | Pavel Moravec <pmoravec@redhat.com> | 2019-11-13 11:37:57 +0100 |
| commit | 43d156200ba679ec72f2a591d0b3c39041a7869b (patch) | |
| tree | f65d43b0780137be3e035886f65960c948c09755 | |
| parent | e530f9443506f833b9aa2dfbe067333cd34d3536 (diff) | |
| download | sos-43d156200ba679ec72f2a591d0b3c39041a7869b.tar.gz | |
[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 <jose.mfcastillo@gmail.com>
Signed-off-by: Pavel Moravec <pmoravec@redhat.com>
| -rw-r--r-- | sos/plugins/openstack_horizon.py | 4 |
1 files changed, 3 insertions, 1 deletions
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" |