Skip to content

chore: update chart readme to recommend newer versions of Helm (not 3.2)#161

Merged
vinzscam merged 3 commits intobackstage:mainfrom
nickboldt:RHIDP-976
Mar 8, 2024
Merged

chore: update chart readme to recommend newer versions of Helm (not 3.2)#161
vinzscam merged 3 commits intobackstage:mainfrom
nickboldt:RHIDP-976

Conversation

@nickboldt
Copy link
Copy Markdown
Contributor

@nickboldt nickboldt commented Jan 25, 2024

chore: update chart readme to recommend newer versions of Helm (not 3.2)

Signed-off-by: Nick Boldt nboldt@redhat.com

For context:

The documentation for deploying RHDH is missing information for both upstream (backstage/chart) and backstage-showcase docs regarding the required Helm CLI version. This has led to parsing errors during deployment, particularly with Helm 3.6.

As per the discussion with Tomas Kral suggested fix:

The recommended version is last stable version (3.13) and minimum version should be 3.10.

@nickboldt nickboldt requested a review from a team as a code owner January 25, 2024 16:13
@nickboldt
Copy link
Copy Markdown
Contributor Author

@kadel can you approve this? or suggest who can approve it?

@nickboldt nickboldt changed the title chore: update chart readme to recommend... chore: update chart readme to recommend newer versions of Helm (not 3.2) Feb 26, 2024
@vinzscam
Copy link
Copy Markdown
Member

sorry @nickboldt, missed the PR! Is it possible to get more information about the warnings? Are the warnings caused by this chart?

@nickboldt
Copy link
Copy Markdown
Contributor Author

nickboldt commented Feb 29, 2024

@nickboldt
Copy link
Copy Markdown
Contributor Author

Note downstream update: https://github.com/redhat-developer/rhdh-chart/pull/8/files

nickboldt and others added 3 commits March 8, 2024 14:49
….2) - RHIDP-976

Signed-off-by: Nick Boldt <nboldt@redhat.com>
Co-authored-by: Nick Boldt <nboldt@redhat.com>
Signed-off-by: Vincenzo Scamporlino <vincenzos@spotify.com>
Signed-off-by: Vincenzo Scamporlino <vincenzos@spotify.com>
@vinzscam vinzscam merged commit fd3a453 into backstage:main Mar 8, 2024
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