fix(orchestrator-infra): Rename the chart in the README [RHDHBUGS-2021]#237
Merged
rm3l merged 1 commit intoredhat-developer:release-1.7from Sep 10, 2025
Conversation
|
There was a problem hiding this comment.
Hey there - I've reviewed your changes - here's some feedback:
- The change to the install command in README.md should also be applied to README.md.gotmpl so future doc generations don’t revert the new chart name.
- Don’t forget to regenerate the Helm repo index after renaming the chart so the updated path is published for users.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The change to the install command in README.md should also be applied to README.md.gotmpl so future doc generations don’t revert the new chart name.
- Don’t forget to regenerate the Helm repo index after renaming the chart so the updated path is published for users.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
rm3l
commented
Sep 10, 2025
Member
Author
rm3l
left a comment
There was a problem hiding this comment.
This is just a simple fix in the README - so merging to close the issue.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Description of the change
Manual cherry-pick of #236
The chart was renamed in [1][2], but the README was still pointing to the
previous chart, which caused users to install an outdated version of the
chart when following this doc.
[1] https://issues.redhat.com/browse/RHIDP-6169
[2] #131
Which issue(s) does this PR fix or relate to
How to test changes / Special notes to the reviewer
Checklist
Chart.yamlaccording to Semantic Versioning.values.yamland added to the corresponding README.md. The pre-commit utility can be used to generate the necessary content. Usepre-commit run -ato apply changes. The pre-commit Workflow will do this automatically for you if needed.pre-commithook.ct lintcommand.Summary by Sourcery
Rename the orchestrator-infra chart reference in README, bump chart version to 0.1.2, and align documentation with the renamed chart to prevent users installing the outdated version
Bug Fixes:
Enhancements:
Documentation:
Chores: