Skip to content

[Test] Unmute testRefreshingMultipleTimesWithinWindowSucceeds #64282

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

Merged
merged 2 commits into from
Oct 29, 2020

Conversation

ywangd
Copy link
Member

@ywangd ywangd commented Oct 28, 2020

The recent failure should be fixed by #64178. Unmuting to ensure things work as expected.

Resolves: #55816

@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) v7.11.0 labels Oct 28, 2020
@ywangd ywangd requested a review from jkakavas October 28, 2020 13:18
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Security Meta label for security team label Oct 28, 2020
@jkakavas
Copy link
Member

failure is unrelated, now that #64276 is merged we can try again to get a green CI run

@jkakavas
Copy link
Member

@elasticmachine update branch

Copy link
Member

@jkakavas jkakavas left a comment

Choose a reason for hiding this comment

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

LGTM!

@ywangd ywangd merged commit 6091dc2 into elastic:7.x Oct 29, 2020
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 v7.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants