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

[so-migrations] Integration test fixes: Use default distribution for cleanup fixture creation #101698

Merged

Conversation

TinaHeiligers
Copy link
Contributor

Resolves #98352

Use the default distribution for integration test fixtures creation.

Checklist

For maintainers

@TinaHeiligers TinaHeiligers added Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc release_note:skip Skip the PR/issue when compiling release notes v7.8.0 v7.14.0 project:ResilientSavedObjectMigrations Reduce Kibana upgrade failures by making saved object migrations more resilient labels Jun 9, 2021
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

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

@TinaHeiligers TinaHeiligers marked this pull request as ready for review June 9, 2021 14:58
@TinaHeiligers TinaHeiligers requested a review from a team as a code owner June 9, 2021 14:58
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-core (Team:Core)

@@ -78,7 +77,7 @@ describe.skip('migration v2', () => {
adjustTimeout: (t: number) => jest.setTimeout(t),
settings: {
es: {
license: 'trial',
license: 'basic',
Copy link
Contributor

Choose a reason for hiding this comment

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

btw, we can remove it since basic is the default value

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's good to know. There's no harm it keeping it though, as a reminder to use the default distro.

@TinaHeiligers TinaHeiligers added the auto-backport Deprecated - use backport:version if exact versions are needed label Jun 10, 2021
@TinaHeiligers TinaHeiligers merged commit 1fe7840 into elastic:master Jun 10, 2021
@TinaHeiligers TinaHeiligers deleted the so-migrations/fix-cleanup-test branch June 10, 2021 13:44
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 10, 2021
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
7.x
7.8 Commit could not be cherrypicked due to conflicts

Successful backport PRs will be merged automatically after passing CI.

To backport manually run:
node scripts/backport --pr 101698

kibanamachine added a commit that referenced this pull request Jun 10, 2021
…cleanup fixture creation (#101698) (#101897)

Co-authored-by: Christiane (Tina) Heiligers <christiane.heiligers@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed project:ResilientSavedObjectMigrations Reduce Kibana upgrade failures by making saved object migrations more resilient release_note:skip Skip the PR/issue when compiling release notes Team:Core Core services & architecture: plugins, logging, config, saved objects, http, ES client, i18n, etc v7.14.0 v8.0.0
Projects
None yet
4 participants