Skip to content

Conversation

cwperks
Copy link
Member

@cwperks cwperks commented Oct 9, 2025

Description

This PR skirts around a 30 day retention period in sonatype by publishing snapshots weekly in addition to on PR merge. This will ensure that snapshots are always available for branches that contain this change.

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • Public documentation issue/PR created.

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

codecov bot commented Oct 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.49%. Comparing base (a05ebbc) to head (e603090).
⚠️ Report is 2 commits behind head on main.

❌ Your project status has failed because the head coverage (70.49%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #847      +/-   ##
==========================================
- Coverage   70.63%   70.49%   -0.15%     
==========================================
  Files          42       42              
  Lines        2057     2057              
  Branches      165      165              
==========================================
- Hits         1453     1450       -3     
- Misses        493      497       +4     
+ Partials      111      110       -1     

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

Signed-off-by: Craig Perkins <[email protected]>
@cwperks cwperks closed this Oct 9, 2025
@cwperks cwperks reopened this Oct 9, 2025
@github-project-automation github-project-automation bot moved this from ✅ Done to 🏗 In progress in Engineering Effectiveness Board Oct 9, 2025
@cwperks
Copy link
Member Author

cwperks commented Oct 9, 2025

FYI for anyone interested in INTEG_TEST vs ARCHIVE see opensearch-project/query-insights#441 (comment).

I needed to change that bc the tests with security needs all of the core modules, not only the transport-* ones.

@github-project-automation github-project-automation bot moved this from 🏗 In progress to 👀 In Review in Engineering Effectiveness Board Oct 9, 2025
@dbwiddis dbwiddis merged commit 5699707 into opensearch-project:main Oct 9, 2025
16 of 17 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in Engineering Effectiveness Board Oct 9, 2025
opensearch-trigger-bot bot pushed a commit that referenced this pull request Oct 9, 2025
* Run maven snapshot publication weekly as well as on PR merge

Signed-off-by: Craig Perkins <[email protected]>

* Switch to ARCHIVE

Signed-off-by: Craig Perkins <[email protected]>

---------

Signed-off-by: Craig Perkins <[email protected]>
(cherry picked from commit 5699707)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants