Skip to content

[GRPC] Add integration tests for Bulk and Search GRPC endpoints #18366

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

Merged
merged 3 commits into from
Jun 4, 2025

Conversation

karenyrx
Copy link
Contributor

Description

Add some integration tests for the DocumentService/Bulk and SearchService/Search GRPC endpoints.

Related Issues

#16783
#16784

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

Copy link
Contributor

❌ Gradle check result for 5693e8f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

✅ Gradle check result for 05d693a: SUCCESS

Copy link

codecov bot commented May 31, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.46%. Comparing base (1d97bb0) to head (7301603).
Report is 26 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #18366      +/-   ##
============================================
- Coverage     72.54%   72.46%   -0.08%     
+ Complexity    67467    67461       -6     
============================================
  Files          5496     5496              
  Lines        311449   311449              
  Branches      45250    45250              
============================================
- Hits         225942   225702     -240     
- Misses        67052    67334     +282     
+ Partials      18455    18413      -42     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@karenyrx karenyrx marked this pull request as ready for review May 31, 2025 03:10
@karenyrx karenyrx requested a review from a team as a code owner May 31, 2025 03:10
Copy link
Contributor

github-actions bot commented Jun 2, 2025

❕ Gradle check result for 7301603: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

@karenyrx
Copy link
Contributor Author

karenyrx commented Jun 4, 2025

@yupeng9 @andrross let me know if any other changes are required. If not would you mind helping to merge this? Thanks!

@andrross andrross merged commit bee0749 into opensearch-project:main Jun 4, 2025
30 checks passed
Gagan6164 pushed a commit to Gagan6164/OpenSearch that referenced this pull request Jun 8, 2025
…search-project#18366)

* Add integration tests for Bulk and Search GRPC endpoints

Signed-off-by: Karen Xu <[email protected]>

* fix assertion for shard info and remove changelog entry
Signed-off-by: Karen Xu <[email protected]>

* use a common GRPC test base

Signed-off-by: Karen Xu <[email protected]>

---------

Signed-off-by: Karen Xu <[email protected]>
Gagan6164 pushed a commit to Gagan6164/OpenSearch that referenced this pull request Jun 8, 2025
…search-project#18366)

* Add integration tests for Bulk and Search GRPC endpoints

Signed-off-by: Karen Xu <[email protected]>

* fix assertion for shard info and remove changelog entry
Signed-off-by: Karen Xu <[email protected]>

* use a common GRPC test base

Signed-off-by: Karen Xu <[email protected]>

---------

Signed-off-by: Karen Xu <[email protected]>
rgsriram pushed a commit to rgsriram/OpenSearch that referenced this pull request Jun 9, 2025
…search-project#18366)

* Add integration tests for Bulk and Search GRPC endpoints

Signed-off-by: Karen Xu <[email protected]>

* fix assertion for shard info and remove changelog entry
Signed-off-by: Karen Xu <[email protected]>

* use a common GRPC test base

Signed-off-by: Karen Xu <[email protected]>

---------

Signed-off-by: Karen Xu <[email protected]>
abhita pushed a commit to abhita/OpenSearch that referenced this pull request Jun 9, 2025
…search-project#18366)

* Add integration tests for Bulk and Search GRPC endpoints

Signed-off-by: Karen Xu <[email protected]>

* fix assertion for shard info and remove changelog entry
Signed-off-by: Karen Xu <[email protected]>

* use a common GRPC test base

Signed-off-by: Karen Xu <[email protected]>

---------

Signed-off-by: Karen Xu <[email protected]>
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
…search-project#18366)

* Add integration tests for Bulk and Search GRPC endpoints

Signed-off-by: Karen Xu <[email protected]>

* fix assertion for shard info and remove changelog entry
Signed-off-by: Karen Xu <[email protected]>

* use a common GRPC test base

Signed-off-by: Karen Xu <[email protected]>

---------

Signed-off-by: Karen Xu <[email protected]>Signed-off-by: TJ Neuenfeldt <[email protected]>
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
…search-project#18366)

* Add integration tests for Bulk and Search GRPC endpoints

Signed-off-by: Karen Xu <[email protected]>

* fix assertion for shard info and remove changelog entry
Signed-off-by: Karen Xu <[email protected]>

* use a common GRPC test base

Signed-off-by: Karen Xu <[email protected]>

---------

Signed-off-by: Karen Xu <[email protected]>
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.

4 participants