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.

Sign up for GitHub

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

Jump to bottom

[7.17] Stabilize testRerouteRecovery throttle testing (#100788) #100858

Merged
merged 2 commits into from
Oct 16, 2023
Merged

[7.17] Stabilize testRerouteRecovery throttle testing (#100788) #100858

merged 2 commits into from
Oct 16, 2023

Conversation

DiannaHohensee
Copy link
Contributor

Refactor testRerouteRecovery, pulling out testing of shard recovery
throttling into separate targeted tests. Now there are two additional
tests, one testing source node throttling, and another testing target
node throttling. Throttling both nodes at once leads to primarily the
source node registering throttling, while the target node mostly has
no cause to instigate throttling.

(cherry picked from commit 323d936)

@DiannaHohensee
Stabilize testRerouteRecovery throttle testing (elastic#100788)
b1523c1 
Refactor testRerouteRecovery, pulling out testing of shard recovery
throttling into separate targeted tests. Now there are two additional
tests, one testing source node throttling, and another testing target
node throttling. Throttling both nodes at once leads to primarily the
source node registering throttling, while the target node mostly has
no cause to instigate throttling.

(cherry picked from commit 323d936)
@DiannaHohensee DiannaHohensee self-assigned this Oct 13, 2023
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v7.17.15 labels Oct 13, 2023
@DiannaHohensee DiannaHohensee added >test Issues or PRs that are addressing/adding tests :Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. backport Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. and removed needs:triage Requires assignment of a team area label labels Oct 13, 2023
@DiannaHohensee DiannaHohensee changed the title Stabilize testRerouteRecovery throttle testing (#100788) [7.17] Stabilize testRerouteRecovery throttle testing (#100788) Oct 16, 2023
@DiannaHohensee DiannaHohensee merged commit 5a8b6fc into elastic:7.17 Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@DiannaHohensee DiannaHohensee

Labels
backport :Distributed Indexing/Recovery Anything around constructing a new shard, either from a local or a remote source. Team:Distributed (Obsolete) Meta label for distributed team (obsolete). Replaced by Distributed Indexing/Coordination. >test Issues or PRs that are addressing/adding tests v7.17.15
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@DiannaHohensee @elasticsearchmachine