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

Mute SegmentReplicationRemoteStoreIT #7654

Merged

Conversation

sachinpkale
Copy link
Member

@sachinpkale sachinpkale commented May 22, 2023

Description

We are observing flaky tests in SegmentReplicationRemoteStoreIT class. Few examples: #7643, #7592

Muting this test class until tests are fixed

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

Signed-off-by: Sachin Kale <kalsac@amazon.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

Merging #7654 (ad58235) into main (3732ddb) will increase coverage by 0.04%.
The diff coverage is 73.75%.

@@             Coverage Diff              @@
##               main    #7654      +/-   ##
============================================
+ Coverage     70.75%   70.80%   +0.04%     
- Complexity    56084    56126      +42     
============================================
  Files          4676     4676              
  Lines        265853   265927      +74     
  Branches      39043    39053      +10     
============================================
+ Hits         188107   188291     +184     
+ Misses        61800    61730      -70     
+ Partials      15946    15906      -40     
Impacted Files Coverage Δ
...rg/opensearch/common/settings/ClusterSettings.java 92.85% <ø> (ø)
...c/main/java/org/opensearch/index/IndexService.java 74.04% <ø> (+0.45%) ⬆️
...x/remote/RemoteRefreshSegmentPressureSettings.java 94.44% <ø> (-5.56%) ⬇️
...ain/java/org/opensearch/indices/IndicesModule.java 96.46% <0.00%> (-1.74%) ⬇️
...ensearch/action/bulk/TransportShardBulkAction.java 75.36% <14.28%> (-0.82%) ⬇️
...ex/remote/RemoteRefreshSegmentPressureService.java 77.50% <25.00%> (-4.65%) ⬇️
...ch/indices/cluster/IndicesClusterStateService.java 78.30% <33.33%> (+4.73%) ⬆️
...arch/index/remote/RemoteRefreshSegmentTracker.java 88.65% <63.63%> (+0.23%) ⬆️
...in/java/org/opensearch/index/shard/IndexShard.java 70.73% <66.66%> (+0.09%) ⬆️
...search/index/shard/RemoteStoreRefreshListener.java 84.95% <83.20%> (+3.39%) ⬆️
... and 1 more

... and 426 files with indirect coverage changes

@gbbafna gbbafna merged commit 54813db into opensearch-project:main May 22, 2023
bharath-techie pushed a commit to bharath-techie/OpenSearch that referenced this pull request May 23, 2023
Signed-off-by: Sachin Kale <kalsac@amazon.com>
Signed-off-by: Bharathwaj G <bharath78910@gmail.com>
ashking94 pushed a commit to ashking94/OpenSearch that referenced this pull request May 23, 2023
Signed-off-by: Sachin Kale <kalsac@amazon.com>
Signed-off-by: Ashish Singh <ssashish@amazon.com>
gbbafna pushed a commit that referenced this pull request May 23, 2023
…re settings & Mute flaky test (#7692)

* Add support to run SegRep integ tests using remote store settings (#7361)

Signed-off-by: Sachin Kale <kalsac@amazon.com>
Signed-off-by: Ashish Singh <ssashish@amazon.com>

* Mute SegmentReplicationRemoteStoreIT (#7654)

Signed-off-by: Sachin Kale <kalsac@amazon.com>
Signed-off-by: Ashish Singh <ssashish@amazon.com>

---------

Signed-off-by: Sachin Kale <kalsac@amazon.com>
suranjay pushed a commit to suranjay/OpenSearch that referenced this pull request May 29, 2023
Signed-off-by: Sachin Kale <kalsac@amazon.com>
stephen-crawford pushed a commit to stephen-crawford/OpenSearch that referenced this pull request May 31, 2023
Signed-off-by: Sachin Kale <kalsac@amazon.com>
shiv0408 pushed a commit to Gaurav614/OpenSearch that referenced this pull request Apr 25, 2024
Signed-off-by: Sachin Kale <kalsac@amazon.com>
Signed-off-by: Shivansh Arora <hishiv@amazon.com>
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.

2 participants