-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add chart_deploy into workflow.yml #3415
Conversation
- Added missing OWNERS and README files - Added maintainers section to Chart.yaml - Changed NOTES.txt so it points to the installation of the CLI - Set the proxy-init version to v1.1.0 in values.yaml Ref #3256 Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
… the Identity.Issuer.CrtExpiryAnnotation value Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Changed `bin/helm-build`'s `(-stable|-edge)` flag to `deploy`. It will automatically extract the repo (edge or stable) and version from git. Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Ref linkerd/website#502 Adds a new final step into `workflow.yml` to generate and push the Helm artifacts. This depends on the changes made to `bin/helm-build` in #3292 so this branch is based on that one till it gets merged. Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
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.
nice! a few nits then shipit. 👍 🚢
also can you update the # Jobs by event type and dependencies:
section at the top to include this new job?
also also were you able to test this? if not, you could disable all those if: startsWith(github.ref, 'refs/tags')
checks on your job for just one run.
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.
Excellent 👍, pending changes per @siggy comments!
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
@siggy I could finally test this successfully, by setting a fake tag like |
Followup to #3415 Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Followup to #3415 Signed-off-by: Alejandro Pedraza <alejandro@buoyant.io>
Ref linkerd/website#502
Adds a new final step into
workflow.yml
to generate and push the Helmartifacts.
This depends on the changes made to
bin/helm-build
in #3292 so thisbranch is based on that one till it gets merged.