Skip to content

Conversation

@andrross
Copy link
Member

Check List

  • Functionality includes testing.

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.

@andrross andrross requested a review from a team as a code owner June 17, 2025 22:18
@andrross
Copy link
Member Author

@peterzhuamazon There was previously automation that opened this PR, right? Do we need to update something for the new branching strategy?

@github-actions
Copy link
Contributor

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

@cwperks
Copy link
Member

cwperks commented Jun 18, 2025

There is an issue with the version increment automation and the new branching strategy in 3.x where we cut 3.x branches off of main instead of 3.x like was done in previous major versions. I believe there's an issue in the workflow around here:

BASE=$(IFS=. ; echo "${CURRENT_VERSION_ARRAY[*]:0:2}")
BASE_X=$(IFS=. ; echo "${CURRENT_VERSION_ARRAY[*]:0:1}.x")

@github-actions
Copy link
Contributor

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

Signed-off-by: Andrew Ross <andrross@amazon.com>
@github-actions
Copy link
Contributor

❕ Gradle check result for fc1ead8: 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 Jun 18, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.70%. Comparing base (9dc2a88) to head (fc1ead8).
Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #18544      +/-   ##
============================================
- Coverage     72.76%   72.70%   -0.06%     
+ Complexity    68147    68135      -12     
============================================
  Files          5540     5540              
  Lines        313378   313379       +1     
  Branches      45472    45472              
============================================
- Hits         228032   227850     -182     
- Misses        66772    67001     +229     
+ Partials      18574    18528      -46     

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

@reta reta merged commit 7c8f01b into opensearch-project:main Jun 19, 2025
31 checks passed
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
Signed-off-by: Andrew Ross <andrross@amazon.com>Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
Signed-off-by: Andrew Ross <andrross@amazon.com>
tandonks pushed a commit to tandonks/OpenSearch that referenced this pull request Aug 5, 2025
Signed-off-by: Andrew Ross <andrross@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.

3 participants