Skip to content

Release: check the pom.xml version against the release tag - #122

Merged
tuhaihe merged 1 commit into
apache:mainfrom
tuhaihe:release-check-pom-version
Oct 12, 2026
Merged

tuhaihe merged 1 commit into
apache:mainfrom
tuhaihe:release-check-pom-version

Conversation

@tuhaihe

@tuhaihe tuhaihe commented Oct 12, 2026

Copy link
Copy Markdown
Member

The release script checked the VERSION file against the tag but never compared it with the <version> in pom.xml, which ships in the source tarball. The go-libs and pxf 2.2.0-incubating-rc2 tarballs went out with pom.xml still at 2.1.0-incubating because their scripts had the same gap; this one was right only because pom.xml had been bumped by hand.

Read the project version with xmllint, which the script already requires, and stop unless it is <base version>-incubating.

Assisted-by: Claude Code

closes: #ISSUE_Number


Change logs

Describe your change clearly, including what problem is being solved or what document is being added or updated.

Contributor's checklist

Here are some reminders before you submit your pull request:

The release script checked the VERSION file against the tag but never
compared it with the <version> in pom.xml, which ships in the source
tarball.  The go-libs and pxf 2.2.0-incubating-rc2 tarballs went out
with pom.xml still at 2.1.0-incubating because their scripts had the
same gap; this one was right only because pom.xml had been bumped by
hand.

Read the project version with xmllint, which the script already
requires, and stop unless it is <base version>-incubating.

Assisted-by: Claude Code
@tuhaihe

tuhaihe commented Oct 12, 2026

Copy link
Copy Markdown
Member Author

verified:

[gpadmin@cdw cloudberry-backup]$ ./cloudberry-backup-release.sh --stage --tag 2.2.0-incubating-rc3 -S -k --repo ~/cloudberry-backup/
INFO: GPG signing has been intentionally skipped (--skip-signing).

=================================================================
>> Platform and Tool Detection
=================================================================
Platform detected: Linux

[OK] SHA-512 tool: sha512sum
[OK] Tar tool: tar
- GPG tool: skipped (--skip-signing enabled)
[OK] XML tool: xmllint

Press Enter or type y/yes to continue, or 'n' to exit: y

=================================================================
>> Validating Version Consistency
=================================================================
Version validation strategy:
  Tag: 2.2.0-incubating-rc3
  Base version (for source files): 2.2.0
Version consistency verified
    Release Tag   : 2.2.0-incubating-rc3
    Base Version  : 2.2.0
    VERSION file  : 2.2.0
    pom.xml       : 2.2.0-incubating

Press Enter or type y/yes to continue, or 'n' to exit: y

@MisterRaindrop MisterRaindrop left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@tuhaihe
tuhaihe merged commit 5a0d9df into apache:main Oct 12, 2026
11 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.

2 participants