Repository navigation
Commit b671dc5
committed
Fix dashboard-resources Role typo excluding the alertmanager WorkloadMonitor
The dashboard-resources Role whitelisted '- alermanager' (misspelled) in its
workloadmonitors resourceNames, so the real 'alertmanager' WorkloadMonitor was
never RBAC-covered — a tenant's dashboard gets forbidden reading its status and
the alertmanager health tile breaks. This PR edits that exact resourceNames list
(it adds the otel-traces entry a few lines below), so the defect is in the block
it owns. Correct the name and add assertions that the Role covers alertmanager by
its real name and not the typo; mutation-verified (restoring the typo reds it).
Signed-off-by: Alexey Artamonov <[email protected]>1 parent b000ef7 commit b671dc5
2 files changed
Lines changed: 11 additions & 1 deletion
File tree
- packages/extra/monitoring
- templates
- tests
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
35 | 35 | | |
36 | 36 | | |
37 | 37 | | |
38 | | - | |
| 38 | + | |
39 | 39 | | |
40 | 40 | | |
41 | 41 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
71 | 81 | | |
72 | 82 | | |
73 | 83 | | |
| |||
0 commit comments