Skip to content

ci: publish prepackaged Helm charts - #612

Merged
loveRhythm1990 merged 1 commit into
matrixorigin:mainfrom
loveRhythm1990:lr90/fix-chart-release-dependencies
Aug 22, 2026
Merged

ci: publish prepackaged Helm charts#612
loveRhythm1990 merged 1 commit into
matrixorigin:mainfrom
loveRhythm1990:lr90/fix-chart-release-dependencies

Conversation

@loveRhythm1990

@loveRhythm1990 loveRhythm1990 commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • prepackage both kruise-1.8.3.tgz and matrixone-operator-1.3.0-alpha.4.tgz with Helm
  • embed the local Kruise package in the parent Chart before packaging it
  • use chart-releaser only to upload the prepared archives and update the GitHub Pages index
  • preserve the existing release naming and skip-existing behavior

Why

The Chart Publish run triggered after #611 failed while chart-releaser v1.4.1 packaged the parent Chart:

Packaging chart 'charts/matrixone-operator'...
Error: can't get a valid version for repositories kruise

Although the workflow had already placed kruise-1.8.3.tgz under the parent Chart, cr package tried to resolve Kruise v1.8.3 from the remote MatrixOrigin Chart repository before that version had been published.

This change removes that publication-order dependency. Merging it to main will trigger Chart Publish again and should create:

  • chart-1.8.3
  • chart-1.3.0-alpha.4

Failed run: https://github.com/matrixorigin/matrixone-operator/actions/runs/32573542424

Validation

  • actionlint .github/workflows/release_chart.yml
  • generated both expected archives in a clean temporary Chart tree
  • verified the parent archive embeds Kruise v1.8.3
  • helm lint passed for the packaged parent Chart
  • confirmed the pinned chart-releaser action supports install_only
  • make verify

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@loveRhythm1990
loveRhythm1990 force-pushed the lr90/fix-chart-release-dependencies branch from d8488d2 to d7fdb0f Compare August 22, 2026 12:49
@loveRhythm1990
loveRhythm1990 enabled auto-merge (squash) August 22, 2026 13:04
@loveRhythm1990
loveRhythm1990 merged commit d261876 into matrixorigin:main Aug 22, 2026
4 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