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

[Archive Migrations] don't use es_archiver/empty_kibana in Shakespeare tests #137797

Merged
merged 6 commits into from
Aug 3, 2022

Conversation

LeeDr
Copy link
Contributor

@LeeDr LeeDr commented Aug 1, 2022

Summary

Fixes: #137327
Also part of: #102552

This PR replaces calls to esArchiver.load('test/functional/fixtures/es_archiver/empty_kibana', with a call to kibanaServer.savedObjects.cleanStandardList() to clean up in the before and after methods.

This PR is not removing es_archiver/empty_kibana as there are still a lot of places using it

@LeeDr
Copy link
Contributor Author

LeeDr commented Aug 2, 2022

@elasticmachine merge upstream

@LeeDr LeeDr changed the title don't use es_archiver/empty_kibana, speed up steps [Archive Migrations] don't use es_archiver/empty_kibana in Shakespeare tests Aug 2, 2022
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@LeeDr LeeDr added Team:QA Team label for QA Team test_ui_functional v8.5.0 labels Aug 2, 2022
@LeeDr LeeDr requested a review from wayneseymour August 2, 2022 21:51
@LeeDr LeeDr marked this pull request as ready for review August 2, 2022 22:42
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-qa (Team:QA)

@LeeDr LeeDr added the release_note:skip Skip the PR/issue when compiling release notes label Aug 2, 2022
@LeeDr LeeDr merged commit 148381b into elastic:main Aug 3, 2022
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Aug 3, 2022
@LeeDr LeeDr added v8.4.0 auto-backport Deprecated - use backport:version if exact versions are needed backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) and removed backport:skip This commit does not require backporting labels Aug 4, 2022
kibanamachine pushed a commit that referenced this pull request Aug 4, 2022
…e tests (#137797)

* don't use es_archiver/empty_kibana, speed up steps

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

* revert change to listing page

* revert this change and do in another PR

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 148381b)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.4

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@LeeDr LeeDr removed the auto-backport Deprecated - use backport:version if exact versions are needed label Aug 4, 2022
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Aug 5, 2022
…e tests (elastic#137797) (elastic#138176)

* don't use es_archiver/empty_kibana, speed up steps

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

* revert change to listing page

* revert this change and do in another PR

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 148381b)

Co-authored-by: Lee Drengenberg <lee.drengenberg@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:QA Team label for QA Team test_ui_functional v8.4.0 v8.5.0
Projects
None yet
5 participants