Skip to content

Restrict nodes for testClusterPrimariesActive2#90184

Merged
elasticsearchmachine merged 1 commit intoelastic:mainfrom
DaveCTurner:2022-09-21-fix-testClusterPrimariesActive2
Sep 21, 2022
Merged

Restrict nodes for testClusterPrimariesActive2#90184
elasticsearchmachine merged 1 commit intoelastic:mainfrom
DaveCTurner:2022-09-21-fix-testClusterPrimariesActive2

Conversation

@DaveCTurner
Copy link
Copy Markdown
Member

This test sets up one green and one yellow index and requires that the green index is the one that gets rebalanced. This may not be true in future, but we can make it true by restricting the nodes that can hold shards of the yellow index.

This test sets up one `green` and one `yellow` index and requires that
the `green` index is the one that gets rebalanced. This may not be true
in future, but we can make it true by restricting the nodes that can
hold shards of the `yellow` index.
@DaveCTurner DaveCTurner added >test Issues or PRs that are addressing/adding tests :Distributed/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) v8.5.0 labels Sep 21, 2022
@elasticsearchmachine elasticsearchmachine added the Team:Distributed Meta label for distributed team. label Sep 21, 2022
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Sep 21, 2022
@DaveCTurner
Copy link
Copy Markdown
Member Author

@elasticmachine please run elasticsearch-ci/packaging-tests-windows-sample

@DaveCTurner DaveCTurner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Sep 21, 2022
@elasticsearchmachine elasticsearchmachine merged commit 08957ab into elastic:main Sep 21, 2022
@DaveCTurner DaveCTurner deleted the 2022-09-21-fix-testClusterPrimariesActive2 branch September 21, 2022 11:17
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Sep 21, 2022
This test sets up one `green` and one `yellow` index and requires that
the `green` index is the one that gets rebalanced. This may not be true
in future, but we can make it true by restricting the nodes that can
hold shards of the `yellow` index.

Also reverts elastic#90184 which applied this fix to the wrong test :/
elasticsearchmachine pushed a commit that referenced this pull request Sep 21, 2022
This test sets up one `green` and one `yellow` index and requires that
the `green` index is the one that gets rebalanced. This may not be true
in future, but we can make it true by restricting the nodes that can
hold shards of the `yellow` index.

Also reverts #90184 which applied this fix to the wrong test :/
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Sep 22, 2022
* main: (186 commits)
  [DOCS] Add 8.5 release notes and fix links (elastic#90201)
  Mute DownsampleActionSingleNodeTests.testCannotRollupWhileOtherRollupInProgress (elastic#90198)
  Bump to version 8.6.0
  Increase the minimum size of the management pool to 2 (elastic#90193)
  Speed `getIntLE` from `BytesReference` (elastic#90147)
  Restrict nodes for testClusterPrimariesActive1 (elastic#90191)
  Fix bug with `BigIntArray` serialization (elastic#90142)
  Synthetic _source: test _source filtering (elastic#90138)
  Modernize cardinality agg tests (elastic#90114)
  Mute failing test (elastic#90186)
  Move assertion in ES85BloomFilterPostingsFormat to fix test (elastic#90150)
  Restrict nodes for testClusterPrimariesActive2 (elastic#90184)
  Batch index delete cluster state updates (elastic#90033)
  Register stable plugins in ActionModule (elastic#90067)
  Mute failing test (elastic#90180)
  [HealthAPI] Disk: Use _ for diagnosis id (elastic#90179)
  [HealtAPI] Disk: use shorter help URLs (elastic#90178)
  Fixing disk health indicator unit tests (elastic#90175)
  Enable the health node and the disk health indicator elastic#84811 (elastic#90085)
  Add missing Disk Indicator health api IDs (elastic#90174)
  ...
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Sep 22, 2022
* main: (121 commits)
  [DOCS] Add 8.5 release notes and fix links (elastic#90201)
  Mute DownsampleActionSingleNodeTests.testCannotRollupWhileOtherRollupInProgress (elastic#90198)
  Bump to version 8.6.0
  Increase the minimum size of the management pool to 2 (elastic#90193)
  Speed `getIntLE` from `BytesReference` (elastic#90147)
  Restrict nodes for testClusterPrimariesActive1 (elastic#90191)
  Fix bug with `BigIntArray` serialization (elastic#90142)
  Synthetic _source: test _source filtering (elastic#90138)
  Modernize cardinality agg tests (elastic#90114)
  Mute failing test (elastic#90186)
  Move assertion in ES85BloomFilterPostingsFormat to fix test (elastic#90150)
  Restrict nodes for testClusterPrimariesActive2 (elastic#90184)
  Batch index delete cluster state updates (elastic#90033)
  Register stable plugins in ActionModule (elastic#90067)
  Mute failing test (elastic#90180)
  [HealthAPI] Disk: Use _ for diagnosis id (elastic#90179)
  [HealtAPI] Disk: use shorter help URLs (elastic#90178)
  Fixing disk health indicator unit tests (elastic#90175)
  Enable the health node and the disk health indicator elastic#84811 (elastic#90085)
  Add missing Disk Indicator health api IDs (elastic#90174)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Distributed/Allocation All issues relating to the decision making around placing a shard (both master logic & on the nodes) Team:Distributed Meta label for distributed team. >test Issues or PRs that are addressing/adding tests v8.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants