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.

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

Revert "Commits related to using default distribution for all tests #94968 and #100244" #101183

Closed
wants to merge 3 commits into from

Conversation

tylersmalley
Copy link
Contributor

@tylersmalley tylersmalley commented Jun 2, 2021

Reverts #100244 and #94968

There are substantial changes needed for this to make it back to 7.x. In 7.x, when using the default distribution, a migration is required for all esArchives in what once was oss. Almost ALL of these migrations fail. Instead of fixing those migrations, we should be migrating any usage of esArchiver with the .kibana index to using kbnArchiver. Reverting these PR's allows us to merge these changes into master and backport them piecemeal, instead of what is now looking like ~26k line change.

@tylersmalley tylersmalley changed the title Revert "[FTR] Use importExport for saved_object/basic archive" Revert "[FTR][CI] Use default distribution for all tests #94968" Jun 3, 2021
@tylersmalley tylersmalley reopened this Jun 3, 2021
@tylersmalley tylersmalley changed the title Revert "[FTR][CI] Use default distribution for all tests #94968" Revert "Commits related to using default distribution for all tests #94968 and #100244" Jun 3, 2021
Signed-off-by: Tyler Smalley <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@tylersmalley
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

merge conflict between base and head

@tylersmalley tylersmalley deleted the revert-100244-ftr-so-basic branch October 14, 2021 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants