-
Notifications
You must be signed in to change notification settings - Fork 16
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
🌱 Revise questions.yaml and add missing options #454
🌱 Revise questions.yaml and add missing options #454
Conversation
477a625
to
eab058b
Compare
1188955
to
236f810
Compare
236f810
to
4bc569b
Compare
4bc569b
to
9a6665d
Compare
0931565
to
6739cd1
Compare
Screencast from 2024-04-02 17-36-24.webm |
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.
Would be good to make these changes as well as part of this.
6739cd1
to
622bf80
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.
This looks good to me. Just a few minor suggestions.
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
622bf80
to
d21ec24
Compare
- Test install, uninstall and re-install in helm chart testing Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
What this PR does / why we need it:
We need to provide best UX with
questions.yaml
while installing turtles from UI. This change refactors the manifest and adds missing helm chart options, or features which may force users to fall back to helm cli installation.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #451
Special notes for your reviewer:
Checklist: