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

[Rest Api Compatibility] Clean up test blocklist #76179

Merged
merged 4 commits into from
Aug 19, 2021
Merged

[Rest Api Compatibility] Clean up test blocklist #76179

merged 4 commits into from
Aug 19, 2021

Conversation

pgomulka
Copy link
Contributor

@pgomulka pgomulka commented Aug 5, 2021

v7compatibilityNotSupportedTests was introduced to make it easier to
track tests that have been identified as not needing compatible changes
and those that still need to be checked.
We have checked all tests now and the separate list is no longer needed.

relates #51816
relates #73912

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

@pgomulka
[Rest Api Compatibility] Clean up blacklist
1d31dcc 
v7compatibilityNotSupportedTests was introduced to make it easier to
track tests that have been identified as not needing compatible changes
and those that still need to be checked.
We have checked all tests now and the separate list is no longer needed.

relates elastic#51816
relates elastic#73912
@pgomulka pgomulka added the :Delivery/Build Build or test infrastructure label Aug 5, 2021
@pgomulka pgomulka requested review from joegallo and jakelandis August 5, 2021 14:20
@pgomulka pgomulka self-assigned this Aug 5, 2021
@elasticmachine elasticmachine added the Team:Delivery Meta label for Delivery team label Aug 5, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

@pgomulka pgomulka added the v8.0.0 label Aug 5, 2021
@pgomulka
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/bwc

@pgomulka
Copy link
Contributor Author

@elasticmachine update branch

@pgomulka
Copy link
Contributor Author

@elasticmachine update branch

@pgomulka pgomulka changed the title [Rest Api Compatibility] Clean up blacklist [Rest Api Compatibility] Clean up test blocklist Aug 17, 2021
@pgomulka pgomulka merged commit 127015e into elastic:master Aug 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pugnascotia pugnascotia pugnascotia approved these changes

@joegallo joegallo Awaiting requested review from joegallo

@jakelandis jakelandis Awaiting requested review from jakelandis

Assignees
No one assigned
Labels
:Delivery/Build Build or test infrastructure >non-issue Team:Delivery Meta label for Delivery team v8.0.0-alpha2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@pgomulka @elasticmachine @pugnascotia @jakelandis