Skip to content

Extract rollover tests into their own test class #69295

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 4 commits into from
Feb 23, 2021

Conversation

joegallo
Copy link
Contributor

Related to #68917. As requested in this comment.

@joegallo joegallo added >non-issue :Data Management/ILM+SLM Index and Snapshot lifecycle management v8.0.0 v7.13.0 labels Feb 19, 2021
@joegallo joegallo requested a review from andreidan February 19, 2021 18:52
@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Feb 19, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (Team:Core/Features)

Copy link
Contributor

@andreidan andreidan left a comment

Choose a reason for hiding this comment

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

❤️ Thanks for this refactoring

@joegallo joegallo merged commit 2f570c4 into elastic:master Feb 23, 2021
@joegallo joegallo deleted the extract-rollover-tests branch February 23, 2021 18:29
joegallo added a commit to joegallo/elasticsearch that referenced this pull request Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/ILM+SLM Index and Snapshot lifecycle management >non-issue Team:Data Management Meta label for data/management team v7.13.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants