Skip to content

Commit 96a2e86

Browse files
committed
Improve Ironic Horizon out of the box experience
Policy override was causing lots of error messages in the Ironic view of Horizon. I've asked upstream if we can get this enabled by default: https://bugs.launchpad.net/horizon/+bug/2102214 But are yet to see any traction.
1 parent 7074489 commit 96a2e86

File tree

3 files changed

+13
-1
lines changed

3 files changed

+13
-1
lines changed
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
SYSTEM_SCOPE_SERVICES = ['ironic']

etc/kayobe/kolla/config/ironic/policy.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,4 @@
44
# GET /nodes/detail
55
# Intended scope(s): system, project
66
# Overridden: added role:admin
7-
"baremetal:node:list_all": "role:admin or (role:reader and system_scope:all) or (role:service and system_scope:all) or rule:service_role"
7+
"baremetal:node:list_all": "role:baremetal_node_list_all or (role:reader and system_scope:all) or (role:service and system_scope:all) or rule:service_role"
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
---
2+
upgrade:
3+
- |
4+
Users with the admin role can can no longer list all baremetal nodes by
5+
default. This broke horizon as it tries to collect data for any nodes
6+
that it can see. Please add the baremetal_node_list_all role
7+
to any users to want this capability; it is not recommened since
8+
Horizon will remain broken for them.
9+
fixes:
10+
- |
11+
Fixes Horizon Ironic integration which was broken by custom policy.

0 commit comments

Comments
 (0)