Skip to content

Comments

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

Closed
pratikjagrut wants to merge 1 commit intorancher:rancher-charts-release/v2.10from
pratikjagrut:rancher-charts-release-automation
Closed

[release-v2.10] Move helm charts from rancher/charts to /charts dir#356
pratikjagrut wants to merge 1 commit intorancher:rancher-charts-release/v2.10from
pratikjagrut:rancher-charts-release-automation

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
@snasovich
Copy link
Collaborator

This has been deprioritized, closing for now.

@snasovich snasovich closed this Jul 1, 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