Skip to content

Commit 32eb32f

Browse files
Scott Shambargermarkgoddard
authored andcommitted
horizon: Correct location of monitoring_policy file
Patch to correctly copy monitoring_policy.json into /etc/openstack-dashboard. Policy was misplaced, and not being enforced. Note that by current default policy, admin doesn't not have Monitoring access. Closes-Bug: #1928408 Change-Id: I4faabdfa9c273fc61b536e6ce88b8d71ab2fc581 (cherry picked from commit f68dfb8)
1 parent c87f82f commit 32eb32f

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

docker/horizon/extend_start.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ function config_monasca_ui {
160160
"${SITE_PACKAGES}/openstack_dashboard/local/enabled/_50_admin_add_monitoring_panel.py"
161161
config_dashboard "${ENABLE_MONASCA:-no}" \
162162
"${SITE_PACKAGES}/monitoring/conf/monitoring_policy.json" \
163-
"${SITE_PACKAGES}/openstack_dashboard/conf/monitoring_policy.json"
163+
"/etc/openstack-dashboard/monitoring_policy.json"
164164
}
165165

166166
function config_murano_dashboard {
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
fixes:
3+
- |
4+
Fixes location of monitoring_policy in Horizon, so access
5+
policy is correctly enforced. Note that by current default,
6+
admin doesn't not have Monitoring access.
7+
`LP#1928408 <https://launchpad.net/bugs/1928408>`__

0 commit comments

Comments
 (0)