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

Allow warnings about 'global' template in upgrade tests #59242

Merged
merged 1 commit into from
Jul 8, 2020

Conversation

dakrone
Copy link
Member

@dakrone dakrone commented Jul 8, 2020

These tests sometimes install a template so they can be compatible with older versions, but they run
amok of the occasionally installed "global" template which changes the default number of shards.

This commit adds allowedWarnings and allows these warnings to be present, but doesn't fail if they
are not (since the global template is only randomly installed).

Resolves #58807
Resolves #58258

These tests sometimes install a template so they can be compatible with older versions, but they run
amok of the occasionally installed "global" template which changes the default number of shards.

This commit adds `allowedWarnings` and allows these warnings to be present, but doesn't fail if they
are not (since the global template is only randomly installed).

Resolves elastic#58807
Resolves elastic#58258
@dakrone dakrone added >test Issues or PRs that are addressing/adding tests :Data Management/Indices APIs APIs to create and manage indices and templates v7.9.0 v7.8.2 labels Jul 8, 2020
@dakrone dakrone requested a review from andreidan July 8, 2020 17:42
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features (:Core/Features/Indices APIs)

@elasticmachine elasticmachine added the Team:Data Management Meta label for data/management team label Jul 8, 2020
@dakrone
Copy link
Member Author

dakrone commented Jul 8, 2020

@elasticmachine run elasticsearch-ci/2

(test failed with the test.fixtures.gcs-fixture-other.tcp.80 problem)

@dakrone
Copy link
Member Author

dakrone commented Jul 8, 2020

@elasticmachine run elasticsearch-ci/1

(I opened #59245 for the failure)

Copy link
Contributor

@andreidan andreidan left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for stabilising these

@dakrone dakrone merged commit bb1c53a into elastic:7.x Jul 8, 2020
@dakrone dakrone deleted the itv2-allow-template-warnings branch July 8, 2020 19:40
dakrone added a commit to dakrone/elasticsearch that referenced this pull request Jul 8, 2020
These tests sometimes install a template so they can be compatible with older versions, but they run
amok of the occasionally installed "global" template which changes the default number of shards.

This commit adds `allowedWarnings` and allows these warnings to be present, but doesn't fail if they
are not (since the global template is only randomly installed).

Resolves elastic#58807
Resolves elastic#58258
dakrone added a commit that referenced this pull request Jul 8, 2020
#59246)

These tests sometimes install a template so they can be compatible with older versions, but they run
amok of the occasionally installed "global" template which changes the default number of shards.

This commit adds `allowedWarnings` and allows these warnings to be present, but doesn't fail if they
are not (since the global template is only randomly installed).

Resolves #58807
Resolves #58258
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates Team:Data Management Meta label for data/management team >test Issues or PRs that are addressing/adding tests v7.8.2 v7.9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants