Skip to content

[stack_functional_integration] remove esArchiver service#141842

Merged
spalger merged 1 commit intoelastic:mainfrom
spalger:fix/stack_functional_integration-config
Sep 26, 2022
Merged

[stack_functional_integration] remove esArchiver service#141842
spalger merged 1 commit intoelastic:mainfrom
spalger:fix/stack_functional_integration-config

Conversation

@spalger
Copy link
Copy Markdown
Contributor

@spalger spalger commented Sep 26, 2022

With #140680 we moved the esArchiver service into a package and changed the way it worked. This ended up breaking the x-pack/test/stack_functional_integration/configs/config.stack_functional_integration_base.js FTR config because it used a custom esArchiver provider to customize the esArchiver baseDir. Rather than continue replacing the esArchiver service this PR adds the ability to configure this path via config instead.

@spalger spalger added Team:Operations Kibana-Operations Team Team:QA Platform QA t// release_note:skip Skip the PR/issue when compiling release notes labels Sep 26, 2022
@spalger spalger requested a review from a team as a code owner September 26, 2022 18:48
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@elasticmachine
Copy link
Copy Markdown
Contributor

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

@spalger spalger force-pushed the fix/stack_functional_integration-config branch from 4474db3 to a11f87f Compare September 26, 2022 18:50
@spalger spalger requested a review from marius-dr September 26, 2022 18:50
Copy link
Copy Markdown
Member

@marius-dr marius-dr left a comment

Choose a reason for hiding this comment

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

LGTM. gave it a run on integration-test as well and it passed.

@spalger spalger enabled auto-merge (squash) September 26, 2022 19:26
@spalger spalger merged commit c494c04 into elastic:main Sep 26, 2022
@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@spalger spalger deleted the fix/stack_functional_integration-config branch September 26, 2022 19:50
@kibanamachine kibanamachine added v8.6.0 backport:skip This PR does not require backporting labels Sep 26, 2022
spalger pushed a commit that referenced this pull request Sep 28, 2022
@LeeDr LeeDr added backport:prev-minor and removed backport:skip This PR does not require backporting labels Oct 20, 2022
@kibanamachine
Copy link
Copy Markdown
Contributor

💔 All backports failed

Status Branch Result
8.5 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 141842

Questions ?

Please refer to the Backport tool documentation

LeeDr pushed a commit to LeeDr/kibana that referenced this pull request Oct 20, 2022
@LeeDr
Copy link
Copy Markdown

LeeDr commented Oct 20, 2022

💚 All backports created successfully

Status Branch Result
8.5

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

Questions ?

Please refer to the Backport tool documentation

LeeDr pushed a commit that referenced this pull request Oct 20, 2022
Co-authored-by: Spencer <spencer@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Operations Kibana-Operations Team Team:QA Platform QA t// v8.5.0 v8.6.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants