-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Parameterize ITs to run with concurrent search enabled #9441
Parameterize ITs to run with concurrent search enabled #9441
Conversation
Compatibility status:Checks if related components are compatible with change ba78d93 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/security-analytics.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git] |
Gradle Check (Jenkins) Run Completed with:
|
server/src/internalClusterTest/java/org/opensearch/search/sort/FieldSortIT.java
Outdated
Show resolved
Hide resolved
server/src/internalClusterTest/java/org/opensearch/threadpool/SimpleThreadPoolIT.java
Outdated
Show resolved
Hide resolved
server/src/internalClusterTest/java/org/opensearch/cluster/settings/ClusterSettingsIT.java
Outdated
Show resolved
Hide resolved
...rc/internalClusterTest/java/org/opensearch/search/scroll/SearchScrollWithFailingNodesIT.java
Show resolved
Hide resolved
@harishbhakuni thanks for the changes, it would be good to add the classes for which we are adding the parameterization in the pr description. |
Gradle Check (Jenkins) Run Completed with:
|
server/src/internalClusterTest/java/org/opensearch/indices/stats/IndexStatsIT.java
Show resolved
Hide resolved
4706fc1
to
88fe7d0
Compare
Gradle Check (Jenkins) Run Completed with:
|
88fe7d0
to
98902c5
Compare
Compatibility status:Checks if related components are compatible with change acbd7b5 Incompatible componentsSkipped componentsCompatible componentsCompatible 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/sql.git, https://github.com/opensearch-project/job-scheduler.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/k-nn.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git] |
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #9441 +/- ##
============================================
+ Coverage 71.03% 71.18% +0.14%
- Complexity 58071 58115 +44
============================================
Files 4824 4824
Lines 273918 273915 -3
Branches 39918 39917 -1
============================================
+ Hits 194591 194988 +397
+ Misses 63017 62544 -473
- Partials 16310 16383 +73 |
server/src/internalClusterTest/java/org/opensearch/search/pit/DeletePitMultiNodeIT.java
Outdated
Show resolved
Hide resolved
@harishbhakuni There is a conflict with |
Signed-off-by: Harish Bhakuni <[email protected]>
Signed-off-by: Harish Bhakuni <[email protected]>
Signed-off-by: Harish Bhakuni <[email protected]>
98902c5
to
acbd7b5
Compare
Gradle Check (Jenkins) Run Completed with:
|
below tests are failing in gradle check and are marked flaky (not able to reproduce them in local): |
* Parameterize ITs to run with concurrent search enabled Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Changes Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Comments Signed-off-by: Harish Bhakuni <[email protected]> --------- Signed-off-by: Harish Bhakuni <[email protected]> Co-authored-by: Harish Bhakuni <[email protected]> (cherry picked from commit 7d1fa7d) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Parameterize ITs to run with concurrent search enabled * Address PR Changes * Address PR Comments --------- (cherry picked from commit 7d1fa7d) Signed-off-by: Harish Bhakuni <[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> Co-authored-by: Harish Bhakuni <[email protected]>
…oject#9441) * Parameterize ITs to run with concurrent search enabled Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Changes Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Comments Signed-off-by: Harish Bhakuni <[email protected]> --------- Signed-off-by: Harish Bhakuni <[email protected]> Co-authored-by: Harish Bhakuni <[email protected]>
…oject#9441) * Parameterize ITs to run with concurrent search enabled Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Changes Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Comments Signed-off-by: Harish Bhakuni <[email protected]> --------- Signed-off-by: Harish Bhakuni <[email protected]> Co-authored-by: Harish Bhakuni <[email protected]> Signed-off-by: Ivan Brusic <[email protected]>
…oject#9441) * Parameterize ITs to run with concurrent search enabled Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Changes Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Comments Signed-off-by: Harish Bhakuni <[email protected]> --------- Signed-off-by: Harish Bhakuni <[email protected]> Co-authored-by: Harish Bhakuni <[email protected]>
…oject#9441) * Parameterize ITs to run with concurrent search enabled Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Changes Signed-off-by: Harish Bhakuni <[email protected]> * Address PR Comments Signed-off-by: Harish Bhakuni <[email protected]> --------- Signed-off-by: Harish Bhakuni <[email protected]> Co-authored-by: Harish Bhakuni <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
Parameterizing first batch of ITs with concurrent search enabled as part of this PR.
List of ITs Parameterized in this PR:
Related Issues
#9443
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.