Skip to content
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

[v2.8] Add charts url setting (#43530) #45699

Merged
merged 1 commit into from
Jun 13, 2024

Conversation

weyfonk
Copy link
Contributor

@weyfonk weyfonk commented Jun 6, 2024

Backport of #43530 to release/v2.8.
Refers to #45677.

  • Add default chart URL setting

This adds new settings to control the origin of system, partner and RKE2 charts, which can now be tested with custom versions prior to releasing corresponding RCs to their respective repositories:

  • ChartDefaultURL (rancher/charts)
  • PartnerChartDefaultURL (rancher/partner- charts)
  • RKE2ChartDefaultURL (rancher/rke2- charts)

These options are intended for internal use only. When in use, Rancher issues a warning suggesting to unset them should any issues be encountered when fetching charts.

  • Group settings by repo type

This moves each default branch setting, for the partner and RKE2 charts repositories, next to the default URL settings for the respective repository, to make configuration easier and improve documentation.

* Add default chart URL setting

This adds new settings to control the origin of system, partner and RKE2
charts, which can now be tested with custom versions prior to releasing
corresponding RCs to their respective repositories:
* `ChartDefaultURL` (`rancher/charts`)
* `PartnerChartDefaultURL` (`rancher/partner- charts`)
* `RKE2ChartDefaultURL` (`rancher/rke2- charts`)

These options are intended for internal use only. When in use, Rancher
issues a warning suggesting to unset them should any issues be
encountered when fetching charts.

* Group settings by repo type

This moves each default branch setting, for the partner and RKE2 charts
repositories, next to the default URL settings for the respective
repository, to make configuration easier and improve documentation.
Copy link
Collaborator

@snasovich snasovich left a comment

Choose a reason for hiding this comment

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

Straight cherry-pick of #43530 and build is (almost) passing - looks like a flaky provisioning test. Restarted...

@weyfonk weyfonk merged commit 62e6a3f into rancher:release/v2.8 Jun 13, 2024
1 of 2 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.

None yet

3 participants