-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Reporting] Correct CSV and ILM tests when run against ES 8.0 #123981
[Reporting] Correct CSV and ILM tests when run against ES 8.0 #123981
Conversation
fdfd4b1
to
d8f2eb6
Compare
d8f2eb6
to
5f33d98
Compare
@elasticmachine merge upstream |
@elasticmachine merge upstream |
{ | ||
names: ['.reporting-*'], | ||
privileges: ['all', 'manage'], | ||
allow_restricted_indices: true, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be forward-ported to 8.x
💚 Build Succeeded
Metrics [docs]
History
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-app-services (Team:AppServicesUx) |
Pinging @elastic/kibana-reporting-services (Team:Reporting Services) |
allow_restricted_indices: true, | ||
} as SecurityHasPrivilegesIndexPrivilegesCheck, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should also be forward ported (and the import statement above)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah nvm, this is already the case in main 🤦🏻
Summary
Closes #123082