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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

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?

CHANGELOG.md Outdated
@@ -31,6 +31,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
- Add support for `matched_fields` with the unified highlighter ([#18164](https://github.com/opensearch-project/OpenSearch/issues/18164))
- [repository-s3] Add support for SSE-KMS and S3 bucket owner verification ([#18312](https://github.com/opensearch-project/OpenSearch/pull/18312))
- Added File Cache Stats - Involves Block level as well as full file level stats ([#17538](https://github.com/opensearch-project/OpenSearch/issues/17479))
- Add integration tests for Bulk and Search GRPC endpoints ([#18366](https://github.com/opensearch-project/OpenSearch/issues/18366))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can probably skip the changelog for a test-only change

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense, removed

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 20 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.

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