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.

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

[TEST] Replace reference to blacklist with failed #75866

Merged
merged 2 commits into from
Jul 30, 2021
Merged

[TEST] Replace reference to blacklist with failed #75866

merged 2 commits into from
Jul 30, 2021

Conversation

tvernum
Copy link
Contributor

@tvernum tvernum commented Jul 30, 2021

This commit replaces use of "blacklist" with "failed" to better
reflect the behaviour of the test.

@tvernum
[TEST] Replace reference to blacklist with failed
b913886 
This commit replaces use of "blacklist" with "failed" to better
reflect the behaviour of the test
@tvernum tvernum 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 auto-backport Automatically create backport pull requests when merged v7.15.0 labels Jul 30, 2021
@tvernum tvernum requested a review from ywangd July 30, 2021 01:50
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Jul 30, 2021
@elasticmachine
Copy link
Collaborator

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

ywangd
ywangd approved these changes Jul 30, 2021
View reviewed changes
Copy link
Member

@ywangd ywangd left a comment

Choose a reason for hiding this comment

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

LGTM

@tvernum tvernum added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jul 30, 2021
@tvernum
Copy link
Contributor Author

tvernum commented Jul 30, 2021

@elasticmachine update branch

@elasticsearchmachine elasticsearchmachine merged commit 98554b4 into elastic:master Jul 30, 2021
@elasticsearchmachine elasticsearchmachine mentioned this pull request Jul 30, 2021
Merged
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
✅ 7.x

elasticsearchmachine pushed a commit to elasticsearchmachine/elasticsearch that referenced this pull request Jul 30, 2021
@tvernum @elasticmachine
[TEST] Replace reference to blacklist with failed (elastic#75866)
5efd374 
This commit replaces use of "blacklist" with "failed" to better
reflect the behaviour of the test

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
tvernum pushed a commit that referenced this pull request Jul 30, 2021
@elasticsearchmachine
[TEST] Replace reference to blacklist with failed (#75866) (#75868)
230f1ef 
This commit replaces use of "blacklist" with "failed" to better
reflect the behaviour of the test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ywangd ywangd ywangd approved these changes

Assignees
No one assigned
Labels
auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :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.15.0 v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tvernum @elasticmachine @elasticsearchmachine @ywangd @mark-vieira