Skip to content
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

Add new allow_restricted_indices field to the expected roles response result. #29055

Closed

Conversation

azasypkin
Copy link
Member

@azasypkin azasypkin commented Jan 21, 2019

Fixes test failures that are caused by newly introduced allow_restricted_indices field in ES roles response: elastic/elasticsearch#37577.

Master CI is green right now, but 6.x one is red already and tests fail locally on master too. Likely ES master snapshot haven't been updated yet or something?

Fixes: #29047

@azasypkin azasypkin added Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! test_api labels Jan 21, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security

@elasticmachine
Copy link
Contributor

💔 Build Failed

@azasypkin
Copy link
Member Author

retest

@elasticmachine
Copy link
Contributor

💔 Build Failed

@azasypkin
Copy link
Member Author

This fix was incorporated into #29184 that has just been merged. Closing.

@azasypkin azasypkin closed this Jan 30, 2019
@azasypkin azasypkin deleted the issue-29047-fix-roles-tests branch January 30, 2019 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! test_api
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants