Skip to content

ci: publish release bundles under direct VERSION (uniform across tools)#120

Merged
a-spiker merged 1 commit into
mainfrom
tools/release-bundle-direct-version
Jun 19, 2026
Merged

ci: publish release bundles under direct VERSION (uniform across tools)#120
a-spiker merged 1 commit into
mainfrom
tools/release-bundle-direct-version

Conversation

@a-spiker

Copy link
Copy Markdown
Contributor

What

Drop the branch-prefixed BUNDLE_VERSION and publish every JFrog release bundle under the SemVer VERSION directly, matching aerospike-admin (asadm). This makes the release-bundle version scheme uniform across all five tools.

Why

Non-release / feature-branch publishes were doubling the branch name, e.g. main-0.21.1-main.<sha>, because VERSION already embeds <branch>.<sha> as a SemVer pre-release tag and BUNDLE_VERSION prepended <branch>- again. The SHA already disambiguates, so the prefix was redundant.

Resulting version pattern (now identical for all 5 tools)

  • Real release (main, not skip_tagging): 0.21.1
  • Non-release / feature / skip_tagging: 0.21.1-main.<sha> (branch appears once)

Note

This branch also carries earlier related CI commit(s) that were already on it. Draft for review.

Removes the branch-prefixed BUNDLE_VERSION so non-release / feature-branch publishes no longer double the branch name (e.g. master-4.5.6-master.<sha>). All publishes now use the SemVer VERSION directly, matching aerospike-admin: real release -> X.Y.Z; non-release/feature -> X.Y.Z-<branch>.<sha>.
@a-spiker a-spiker force-pushed the tools/release-bundle-direct-version branch from f62f53c to 090b97c Compare June 19, 2026 19:02
@a-spiker a-spiker marked this pull request as ready for review June 19, 2026 19:03
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.05%. Comparing base (fec998e) to head (090b97c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #120   +/-   ##
=======================================
  Coverage   81.05%   81.05%           
=======================================
  Files          15       15           
  Lines        2101     2101           
=======================================
  Hits         1703     1703           
  Misses        296      296           
  Partials      102      102           

☔ View full report in Codecov by Harness.
📢 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.

@a-spiker a-spiker requested a review from pvinh-spike June 19, 2026 19:16

@pvinh-spike pvinh-spike left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@a-spiker a-spiker merged commit b13cd15 into main Jun 19, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants