mirror of
https://github.com/elastic/kibana.git
synced 2025-04-24 01:38:56 -04:00
[Infra] Fix failing test by changing read role permission (#205707)
Closes #203740 ## Summary This PR fixes failing test by changing read role permission to include `streams` and `apm`. As I mentioned in this [comment](https://github.com/elastic/kibana/issues/203740#issuecomment-2574907832) I saw some 403 errors related to some `streams` and `apm` APIs requests so this should fix the test as it was meant to test if the dashboards tab behaves correctly based on the admin/read-only role. However, we should think about a solution to those errors in case we have this scenario (read-only user role without apm/streams access) and have a better error message/explanation of what is missing instead of only showing the error toasts - I saw that we reverted (https://github.com/elastic/kibana/pull/202418) already a solution (https://github.com/elastic/kibana/issues/200151) for APM because of other issues but now that we also include the `streams` (https://github.com/elastic/kibana/pull/200060) (not sure if we need to do the request in infra but that's probably a different discussion) it's something we can revisit at one point to improve the user experience.
This commit is contained in:
parent
d86a966ceb
commit
6fc90d0410
1 changed files with 2 additions and 0 deletions
|
@ -236,7 +236,9 @@ export default ({ getPageObjects, getService }: FtrProviderContext) => {
|
|||
{
|
||||
feature: {
|
||||
infrastructure: ['read'],
|
||||
apm: ['read'],
|
||||
advancedSettings: ['read'],
|
||||
streams: ['read'],
|
||||
},
|
||||
spaces: ['*'],
|
||||
},
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue