Skip to content

[8.0] Always re-run Feature migrations which have encountered errors (#83918)#84138

Merged
elasticsearchmachine merged 2 commits intoelastic:8.0from
AthenaEryma:backport/8.0/pr-83918
Feb 22, 2022
Merged

[8.0] Always re-run Feature migrations which have encountered errors (#83918)#84138
elasticsearchmachine merged 2 commits intoelastic:8.0from
AthenaEryma:backport/8.0/pr-83918

Conversation

@AthenaEryma
Copy link
Copy Markdown
Contributor

Backports the following commits to 8.0:

@AthenaEryma AthenaEryma added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport labels Feb 17, 2022
@cla-checker-service
Copy link
Copy Markdown

cla-checker-service bot commented Feb 22, 2022

💚 CLA has been signed

…ic#83918)

This PR addressed the behavior described in elastic#83917, in which Feature migrations
which have encountered errors are not re-run in some cases. As of this PR, Features
which have encountered errors during migration are treated the same as Features 
requiring migration.

This PR also adds a test which artificially replicates elastic#83917.
@AthenaEryma AthenaEryma force-pushed the backport/8.0/pr-83918 branch from 3589c5a to b7479c4 Compare February 22, 2022 18:42
@AthenaEryma
Copy link
Copy Markdown
Contributor Author

CLA business was due to forgetting to configure my git author name & email on my new laptop. Hence the force push.

@elasticsearchmachine elasticsearchmachine merged commit 90dc01a into elastic:8.0 Feb 22, 2022
@AthenaEryma AthenaEryma deleted the backport/8.0/pr-83918 branch February 22, 2022 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport v8.0.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants