Skip to content

[7.2] Fixing isMigrated bug, when migrations failed we incorrectly ignored it. (#37674)#38042

Merged
kobelb merged 1 commit into
elastic:7.2from
kobelb:backport/7.2/pr-37674
Jun 4, 2019
Merged

[7.2] Fixing isMigrated bug, when migrations failed we incorrectly ignored it. (#37674)#38042
kobelb merged 1 commit into
elastic:7.2from
kobelb:backport/7.2/pr-37674

Conversation

@kobelb
Copy link
Copy Markdown
Contributor

@kobelb kobelb commented Jun 4, 2019

Backports the following commits to 7.2:

…it. (elastic#37674)

Prior to this change, when migrations failed (for example when a
.kibana_1 index already exists and no alias does) the awaitMigration
promise would resolve incorrectly. This allowed subsequent plugins,
including Spaces, to proceed using the SavedObjectsClient/Repository
to create documents. This commonly caused a .kibana index to be created,
which would then block the creation of a .kibana alias, and everything
would grind to a halt.
@kobelb kobelb added the backport This PR is a backport of another PR label Jun 4, 2019
@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

@kobelb kobelb merged commit 990657c into elastic:7.2 Jun 4, 2019
@kobelb kobelb deleted the backport/7.2/pr-37674 branch June 4, 2019 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants