Skip to content

[release-v2.10] Move helm charts from rancher/charts to /charts dir #356

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: rancher-charts-release/v2.10
Choose a base branch
from

Conversation

pratikjagrut
Copy link

PR Description

We're integrating the Auto Chart Bumps feature into the System Upgrade Controller (SUC) repository to streamline chart maintenance and align with Rancher's release structure.

Why This Change?

  • Currently, the SUC chart resides in rancher/charts, but due to strict requirements, we need a separate repo for managing SUC chart.
  • One key requirement is maintaining three active release branches for Rancher, which makes it difficult to continue hosting the chart in the rancher/charts repository.
  • Instead of creating a new repository, we've decided to host the SUC chart in a dedicated branch within the existing SUC repo.
  • We'll use the same branch for all three active releases instead of three branches.

What’s Changing?

  • A new branch, rancher-charts-release/v2.10, will be used exclusively for SUC charts.
  • All new PRs for the Rancher SUC chart will be created in this branch instead of rancher/charts.
  • This branch will only contain the SUC chart files, not the SUC source code.
  • The auto chart bump mechanism will automatically watch this branch and update charts.

Related issue: #49087.

catalog.cattle.io/rancher-version: '>= 2.10.0-0'
catalog.cattle.io/release-name: system-upgrade-controller
apiVersion: v1
appVersion: v0.14.2
Copy link
Member

Choose a reason for hiding this comment

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

v0.14.2 is from October, can we use something newer?

Copy link
Collaborator

Choose a reason for hiding this comment

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

This matches the version used in v2.10.x release line (https://github.com/rancher/charts/blob/dev-v2.10/charts/system-upgrade-controller/105.0.1/values.yaml#L8) so we're just matching that.
Bumping SUC version is out of scope for this effort.

@brandond brandond changed the title Move helm charts from rancher/charts to /charts dir [release-v2.10] Move helm charts from rancher/charts to /charts dir Mar 19, 2025
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