-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix flaky testEqualsAndHashcode in SearchRequestTests #10649
Conversation
Signed-off-by: David Zane <[email protected]>
Thanks @dzane17 ! |
Compatibility status:Checks if related components are compatible with change 9dffac7 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/common-utils.git] |
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #10649 +/- ##
============================================
+ Coverage 71.03% 71.16% +0.12%
- Complexity 58414 58444 +30
============================================
Files 4853 4853
Lines 275670 275670
Branches 40118 40118
============================================
+ Hits 195831 196187 +356
+ Misses 63455 63032 -423
- Partials 16384 16451 +67 |
The backport to
To backport manually, run these commands in your terminal: # Navigate to the root of your repository
cd $(git rev-parse --show-toplevel)
# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add ../.worktrees/OpenSearch/backport-2.x 2.x
# Navigate to the new working tree
pushd ../.worktrees/OpenSearch/backport-2.x
# Create a new branch
git switch --create backport/backport-10649-to-2.x
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 d46a012c9fb01f7b3fb58a1c8fff1e352fe9efb8
# Push it to GitHub
git push --set-upstream origin backport/backport-10649-to-2.x
# Go back to the original working tree
popd
# Delete the working tree
git worktree remove ../.worktrees/OpenSearch/backport-2.x Then, create a pull request where the |
@dzane17 could you please backport to |
@reta We have not backported the original PR to 2.x yet. I will manually backport this one at the same time. |
…ject#10649) Signed-off-by: David Zane <[email protected]> Signed-off-by: Siddhant Deshmukh <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]>
Signed-off-by: David Zane <[email protected]> (cherry picked from commit d46a012) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
(cherry picked from commit d46a012) Signed-off-by: David Zane <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…ject#10649) Signed-off-by: David Zane <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]>
…ject#10649) Signed-off-by: David Zane <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
Fix for flaky
SearchRequestTests.testEqualsAndHashcode
Related Issues
Resolves #10643
Caused by #10351
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.