-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
docs: update helm documentation #12932
Conversation
Codecov Report
@@ Coverage Diff @@
## master #12932 +/- ##
==========================================
- Coverage 77.39% 68.94% -8.45%
==========================================
Files 928 1025 +97
Lines 47024 48765 +1741
Branches 5812 5241 -571
==========================================
- Hits 36393 33623 -2770
- Misses 10488 15008 +4520
+ Partials 143 134 -9
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
noting that |
Look like a re doing of #11763 |
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.
Left inline comment. Please also remove requiments.yaml / requirements.locl in favor of keeping everything in Chart.yaml.
* docs: update helm documentation * docs: fix for end-of-file-fixer pre-commit-config * update: bump chart apiVersion to v2 and remove requirements.yaml
SUMMARY
Noticed #11363 and updated documentation to reflect a success full deployment on GKE with helm3.
ADDITIONAL INFORMATION
Can publish these on merge wrt https://github.com/apache/superset/blob/master/docs/README.md#to-publish