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
build: fix master version helm values replacement #13877
Conversation
824b28b
to
83d34d7
Compare
deploy/charts/rook-ceph/values.yaml
Outdated
# @default -- `master` | ||
tag: master | ||
# @default -- `"master"` | ||
tag: "master" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of changing the script, what if we just update the ceph links to the latest
links? The master
links are anyway outdated and we should be using latest
. Then there is no risk or need to test the change to the build script.
83d34d7
to
c8434d5
Compare
Signed-off-by: Alexander Trost <galexrt@googlemail.com>
c8434d5
to
6f61f46
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
build: fix master version helm values replacement (backport #13877)
Issue resolved by this Pull Request:
Resolves #13872
Checklist: