Skip to content

Re-enable Kerberos + anonymous access test. #55377

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jan 21, 2020
Merged

Re-enable Kerberos + anonymous access test. #55377

merged 1 commit into from
Jan 21, 2020

Conversation

@azasypkin
Copy link
Member

@azasypkin azasypkin commented Jan 21, 2020 •
edited
Loading

Since elastic/elasticsearch#51042 is merged we can finally re-enable skipped test as soon as we use fresh ES snapshot on CI. For now tested locally with:

$ KBN_ES_SNAPSHOT_USE_UNVERIFIED=1 node scripts/functional_tests_server.js \
   --config test/kerberos_api_integration/anonymous_access.config.ts
-------
$ KBN_ES_SNAPSHOT_USE_UNVERIFIED=1 node scripts/functional_test_runner.js \
  --config test/kerberos_api_integration/anonymous_access.config.ts

Fixes: #52969

@azasypkin azasypkin added test chore Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// Feature:Security/Authentication Platform Security - Authentication v7.7.0 v7.6.0 v7.6.1 labels Jan 21, 2020
@elasticmachine
Copy link
Contributor

elasticmachine commented Jan 21, 2020

Pinging @elastic/kibana-security (Team:Security)

@azasypkin azasypkin added the release_note:skip Skip the PR/issue when compiling release notes label Jan 21, 2020
@kibanamachine
Copy link
Contributor

kibanamachine commented Jan 21, 2020

💚 Build Succeeded

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@azasypkin azasypkin requested a review from legrego January 21, 2020 13:48
kobelb
kobelb approved these changes Jan 21, 2020
View reviewed changes
Copy link
Contributor

@kobelb kobelb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks for taking care of this, Aleh!

@azasypkin azasypkin merged commit be31198 into elastic:master Jan 21, 2020
@azasypkin azasypkin deleted the issue-52969-kerberos-tests branch January 21, 2020 14:56
This was referenced Jan 21, 2020
Merged
Merged
@azasypkin azasypkin removed the v7.6.1 label Jan 21, 2020
azasypkin added a commit to azasypkin/kibana that referenced this pull request Jan 21, 2020
@azasypkin
Re-enable Kerberos + anonymous access test. (elastic#55377)
44007e9
@azasypkin
Copy link
Member Author

azasypkin commented Jan 21, 2020

7.6/7.6.0: 64dc134
7.x/7.7.0: 0c62eba

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kobelb kobelb kobelb approved these changes

@legrego legrego Awaiting requested review from legrego

Assignees

No one assigned

Labels

backported chore Feature:Security/Authentication Platform Security - Authentication release_note:skip Skip the PR/issue when compiling release notes Team:Security Platform Security: Auth, Users, Roles, Spaces, Audit Logging, etc t// test v7.6.0 v7.7.0

Projects

None yet

Milestone

No milestone

4 participants

@azasypkin @elasticmachine @kibanamachine @kobelb