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

[Backport 2.x] Fix ScalingThreadPoolTest to handle new pool #10533

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 506065c from #10427.

This was a bad use of randomization. The test is super fast so it can be
run against every scaling thread pool every time.

Signed-off-by: Andrew Ross <[email protected]>
(cherry picked from commit 506065c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Compatibility status:

Checks if related components are compatible with change 69045ab

Incompatible components

Incompatible components: [https://github.com/opensearch-project/performance-analyzer.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/neural-search.git]

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Oct 10, 2023

Codecov Report

Merging #10533 (69045ab) into 2.x (42f10d5) will decrease coverage by 0.02%.
Report is 4 commits behind head on 2.x.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##                2.x   #10533      +/-   ##
============================================
- Coverage     70.82%   70.81%   -0.02%     
- Complexity    58462    58470       +8     
============================================
  Files          4829     4829              
  Lines        276380   276384       +4     
  Branches      40579    40579              
============================================
- Hits         195747   195714      -33     
- Misses        63970    64001      +31     
- Partials      16663    16669       +6     
Files Coverage Δ
...ibs/core/src/main/java/org/opensearch/Version.java 83.62% <100.00%> (+0.50%) ⬆️
...c/main/java/org/opensearch/index/IndexService.java 75.43% <ø> (ø)
.../org/opensearch/index/remote/RemoteStoreUtils.java 96.55% <ø> (-0.12%) ⬇️
...earch/index/store/RemoteSegmentStoreDirectory.java 88.88% <100.00%> (-0.95%) ⬇️
...ex/translog/transfer/TranslogTransferMetadata.java 85.00% <100.00%> (+0.38%) ⬆️
...in/java/org/opensearch/index/shard/IndexShard.java 69.91% <0.00%> (+0.81%) ⬆️

... and 432 files with indirect coverage changes

@dblock dblock merged commit 649d1bb into 2.x Oct 10, 2023
38 of 64 checks passed
@github-actions github-actions bot deleted the backport/backport-10427-to-2.x branch October 10, 2023 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant