Skip to content

Conversation

@Sukriti0717
Copy link
Contributor

@Sukriti0717 Sukriti0717 commented Sep 3, 2025

Implemented a comprehensive metrics framework that captures 8 key metrics across node-level and shard-level operations:

  1. Total scheduler execution time
  2. Total Merges Triggered
  3. Total Merges Skipped
  4. Skips count due to Node Validator
  5. Total failed force merges
  6. Shard specific
    1. Time taken by force merge
    2. Shard size
    3. Segment count

@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2025

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

@Sukriti0717 Sukriti0717 force-pushed the staggered-merges-metrics branch 2 times, most recently from fe55acb to 42b7925 Compare September 7, 2025 14:46
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2025

❕ Gradle check result for 42b7925: 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.

@codecov
Copy link

codecov bot commented Sep 7, 2025

Codecov Report

❌ Patch coverage is 98.38710% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.97%. Comparing base (6fdb010) to head (56bee61).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...ch/index/autoforcemerge/AutoForceMergeManager.java 97.05% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #19226      +/-   ##
============================================
+ Coverage     72.90%   72.97%   +0.06%     
- Complexity    69915    69916       +1     
============================================
  Files          5675     5676       +1     
  Lines        320841   320923      +82     
  Branches      46387    46392       +5     
============================================
+ Hits         233909   234189     +280     
+ Misses        68031    67786     -245     
- Partials      18901    18948      +47     

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

@Sukriti0717 Sukriti0717 force-pushed the staggered-merges-metrics branch from 42b7925 to 440b472 Compare September 8, 2025 08:18
@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025

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

@github-actions
Copy link
Contributor

github-actions bot commented Sep 8, 2025

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

@Sukriti0717 Sukriti0717 force-pushed the staggered-merges-metrics branch from c27b84e to 9735956 Compare September 19, 2025 06:42
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

✅ Gradle check result for a81b1df: SUCCESS

@Sukriti0717 Sukriti0717 force-pushed the staggered-merges-metrics branch 2 times, most recently from 3397b56 to fef6db0 Compare September 22, 2025 04:36
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

❌ Gradle check result for fef6db0: null

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?

@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

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

Sukriti Sinha added 2 commits September 25, 2025 09:37
Signed-off-by: Sukriti Sinha <[email protected]>
Signed-off-by: Sukriti Sinha <[email protected]>
@Sukriti0717 Sukriti0717 force-pushed the staggered-merges-metrics branch from fef6db0 to 56bee61 Compare September 25, 2025 04:08
@github-actions
Copy link
Contributor

✅ Gradle check result for 56bee61: SUCCESS

@gbbafna gbbafna merged commit b256549 into opensearch-project:main Sep 25, 2025
33 checks passed
vinaykpud pushed a commit to vinaykpud/OpenSearch that referenced this pull request Sep 26, 2025
karenyrx pushed a commit to karenyrx/OpenSearch that referenced this pull request Sep 29, 2025
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