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

Fix test failure for watcher with API key in upgraded cluster #63044

Merged
merged 1 commit into from
Sep 30, 2020

Conversation

ywangd
Copy link
Member

@ywangd ywangd commented Sep 30, 2020

Depending on the execution order of different tests, the watcher service may be stopped when the failed test runs.
This PR ensure the watcher service is started before test assetions.

@ywangd ywangd added >test Issues or PRs that are addressing/adding tests :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) v8.0.0 labels Sep 30, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (:Security/Authentication)

@elasticmachine elasticmachine added the Team:Security Meta label for security team label Sep 30, 2020
@ywangd ywangd merged commit a025345 into elastic:master Sep 30, 2020
ywangd added a commit to ywangd/elasticsearch that referenced this pull request Sep 30, 2020
…c#63044)

Depending on the execution order of different tests, the watcher service may be stopped 
when the failed test runs. This PR ensure the watcher service is started before test assetions.
ywangd added a commit to ywangd/elasticsearch that referenced this pull request Sep 30, 2020
…c#63044)

Depending on the execution order of different tests, the watcher service may be stopped 
when the failed test runs. This PR ensure the watcher service is started before test assetions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants