-
Notifications
You must be signed in to change notification settings - Fork 15
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
huge refactor for the new datahub build/deploy CI/CD github actions workflow #117
huge refactor for the new datahub build/deploy CI/CD github actions workflow #117
Conversation
some examples below... help output:
deploy a hub with one image/profile to staging:
verbose output:
debug output:
using the hashes from artifact repo, deploying multiple images to the dev hub (debug output to see what it does):
the hacked config for the dev hub:
|
1) removed ability and code for building images (for now) 2) add support for image tagging (defined in hubploy.yaml, defaults to 'latest' if not specified 3) added verbose and debuging output 4) added --dry-run for helm actions 5) fixed almost every line > 80 chars 6) consistent use of " vs a mix of " and ' 7) did a ton of readability cleanup 8) maybe more?
9623735
to
88be478
Compare
i've also successfully built and pushed the |
added CLI help text for all the things:
|
i've tested this extensively on my own device and everything work perfectly!