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

🌱 Revise questions.yaml and add missing options #454

Merged
merged 2 commits into from
Apr 3, 2024

Conversation

Danil-Grigorev
Copy link
Contributor

@Danil-Grigorev Danil-Grigorev commented Mar 28, 2024

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:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@Danil-Grigorev Danil-Grigorev requested a review from a team as a code owner March 28, 2024 13:02
@Danil-Grigorev Danil-Grigorev force-pushed the update-questions-yaml branch 12 times, most recently from 477a625 to eab058b Compare April 2, 2024 09:56
.github/workflows/test_chart.yaml Outdated Show resolved Hide resolved
.github/workflows/test_chart.yaml Outdated Show resolved Hide resolved
charts/rancher-turtles/questions.yml Outdated Show resolved Hide resolved
@Danil-Grigorev Danil-Grigorev force-pushed the update-questions-yaml branch 6 times, most recently from 1188955 to 236f810 Compare April 2, 2024 13:37
@Danil-Grigorev Danil-Grigorev force-pushed the update-questions-yaml branch 2 times, most recently from 0931565 to 6739cd1 Compare April 2, 2024 15:22
@Danil-Grigorev
Copy link
Contributor Author

Screencast from 2024-04-02 17-36-24.webm
Here is a demo of install + uninstall + re-install running locally

Copy link
Contributor

@richardcase richardcase left a 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.

charts/rancher-turtles/questions.yml Outdated Show resolved Hide resolved
charts/rancher-turtles/questions.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@richardcase richardcase left a 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.

charts/rancher-turtles/questions.yml Outdated Show resolved Hide resolved
charts/rancher-turtles/questions.yml Outdated Show resolved Hide resolved
charts/rancher-turtles/questions.yml Outdated Show resolved Hide resolved
.github/workflows/test_chart.yaml Show resolved Hide resolved
Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
- Test install, uninstall and re-install in helm chart testing

Signed-off-by: Danil Grigorev <danil.grigorev@suse.com>
@richardcase richardcase merged commit c4c40b3 into rancher:main Apr 3, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update questions yaml
4 participants