Skip to content

CI:add release script#71

Merged
RongGu merged 4 commits intosgl-project:mainfrom
cheyang:update_release_shell
Oct 26, 2025
Merged

CI:add release script#71
RongGu merged 4 commits intosgl-project:mainfrom
cheyang:update_release_shell

Conversation

@cheyang
Copy link
Copy Markdown
Collaborator

@cheyang cheyang commented Oct 25, 2025

Ⅰ. Motivation

This pull request introduces an automated Helm chart version update script to streamline release processes. The key motivations are:

  1. Eliminate manual errors in Helm chart versioning and image tagging
  2. Enforce branch naming conventions for semantic versioning
  3. Automate synchronization between Git branches, Helm charts, and container images
  4. Reduce release overhead by replacing error-prone manual updates with a single command

Ⅱ. Modifications

  1. Added new script: rbg/tools/release/version-update.sh
  2. Core functionality:
    • Auto-detects semantic versions from Git branch names (0.5.0, v1.2.3-rc.1)
    • Updates Helm chart fields:
      Chart.yaml:
      - version: 0.5.0        # From branch name
      - appVersion: 0.5.0-abc123  # From image tag (without 'v')
      
      values.yaml:
      - image.tag: v0.5.0-abc123  # From Makefile VERSION + Git SHA
    • Validates:
      • Branch naming conventions
      • Makefile variable presence
      • File existence
  3. Path handling:
    • Works from any directory in repository
    • Correctly resolves paths:
      CHARTS_DIR="deploy/helm"  # Relative to repo root

Ⅲ. Does this pull request fix one issue?

NONE

Ⅳ. List the added test cases

No tests added because:

  1. This is a release utility script run manually
  2. Contains built-in validations and error checks:
    • Branch format validation
    • Makefile variable checks
    • File existence verification
  3. Outputs clear error messages for all failure cases
  4. Will be validated during actual release processes

Note: Manual testing has been performed with:

  • Valid branches (0.5.0, v1.2.3-rc1)

@cheyang cheyang requested a review from Syspretor October 25, 2025 16:11
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@cheyang cheyang changed the title add release script CI:add release script Oct 25, 2025
@cheyang cheyang requested a review from RongGu October 26, 2025 07:08
Copy link
Copy Markdown
Collaborator

@RongGu RongGu left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@RongGu RongGu merged commit 83b9542 into sgl-project:main Oct 26, 2025
3 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