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: flaky test ClusterMetadataManifestTests #9804

Merged
merged 1 commit into from
Sep 6, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 6fe9388 from #9728.

We're comparing for unequal manifests but randomBoolean() has a high
probability of getting clashed because of a small search-space, making
test flaky. Let's limit randomization here.

Signed-off-by: Rohit Ashiwal <rashiwal@amazon.com>
(cherry picked from commit 6fe9388)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Compatibility status:

Checks if related components are compatible with change 80da4bb

Incompatible components

Incompatible components: [https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/neural-search.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/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/performance-analyzer-rca.git]

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=pit/10_basic/Delete all}

@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Merging #9804 (80da4bb) into 2.x (affbd76) will increase coverage by 0.03%.
Report is 1 commits behind head on 2.x.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #9804      +/-   ##
============================================
+ Coverage     70.81%   70.84%   +0.03%     
- Complexity    58113    58150      +37     
============================================
  Files          4812     4812              
  Lines        274731   274738       +7     
  Branches      40375    40376       +1     
============================================
+ Hits         194549   194640      +91     
+ Misses        63480    63476       -4     
+ Partials      16702    16622      -80     
Files Changed Coverage Δ
...rg/opensearch/index/translog/RemoteFsTranslog.java 72.54% <100.00%> (+0.97%) ⬆️

... and 465 files with indirect coverage changes

@sachinpkale sachinpkale merged commit 9c707c9 into 2.x Sep 6, 2023
14 of 40 checks passed
@github-actions github-actions bot deleted the backport/backport-9728-to-2.x branch September 6, 2023 10:18
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.

2 participants