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] Reduce amount of docs ingested for SegRep ITs #11237

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport e3bd0c3 from #10584.

This change reduces the amount of docs we ingest inside SegmentReplicationIT. These tests
were often ingesting 1-200 docs where it was not required. Many only required a few so that
segments are created.
This speeds up tests when run with remote store through SegmentReplicationUsingRemoteStore.
Also add Testlogging annotation on the replication package for known flaky tests.

Signed-off-by: Marc Handalian <handalm@amazon.com>
(cherry picked from commit e3bd0c3)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

Compatibility status:

Checks if related components are compatible with change e4a051e

Incompatible components

Incompatible components: [https://github.com/opensearch-project/performance-analyzer.git]

Skipped components

Compatible components

Compatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/performance-analyzer-rca.git]

Copy link
Contributor

❌ Gradle check result for e4a051e: 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?

@Poojita-Raj
Copy link
Contributor

❌ Gradle check result for e4a051e: 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?

#3603
#10154

Copy link
Contributor

✅ Gradle check result for e4a051e: SUCCESS

Copy link

codecov bot commented Nov 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a6748e0) 70.92% compared to head (e4a051e) 70.96%.
Report is 19 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #11237      +/-   ##
============================================
+ Coverage     70.92%   70.96%   +0.04%     
- Complexity    58958    59012      +54     
============================================
  Files          4867     4867              
  Lines        278457   278457              
  Branches      40810    40810              
============================================
+ Hits         197485   197607     +122     
+ Misses        64142    64116      -26     
+ Partials      16830    16734      -96     

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

@andrross andrross merged commit b79904d into 2.x Nov 22, 2023
43 of 69 checks passed
@github-actions github-actions bot deleted the backport/backport-10584-to-2.x branch November 22, 2023 17:53
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.

3 participants