-
Notifications
You must be signed in to change notification settings - Fork 981
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
Publish helm chart as release asset #2189
Conversation
Hi @ddelange. Thanks for your PR. I'm waiting for a kserve member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
5a9b6a7
to
d596a8b
Compare
18ebc70
to
ae47c34
Compare
@ddelange Thanks for the work with helm! My question is that currently crd is part of one helm chart, as far as I know when we upgrade the chart helm does not update the crd if anything is changed in this case. Should we create a separate chart for crd as helm recommended? |
Oh interesting, I didn't know! I'm guessing that can be a separate issue, this workflow will attempt to |
Indeed looks like istio also splits it up! https://istio.io/latest/docs/setup/install/helm/#installation-steps |
yes let's split the crd out then. |
I can give it a shot. Should it be a separate PR? If yes, should it be merged before this one? |
Yes we can merge this one first, thanks! |
ae47c34
to
907902d
Compare
907902d
to
54565fe
Compare
@yuzisun could you trigger CI for this PR? |
/ok-to-test |
Signed-off-by: ddelange <14880945+ddelange@users.noreply.github.com>
54565fe
to
5f4a92c
Compare
@yuzisun pushed, there was no conflict according to |
Thanks @ddelange! will test out this with the 0.9.0-rc0 release. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ddelange, yuzisun The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: ddelange <14880945+ddelange@users.noreply.github.com> Signed-off-by: alexagriffith <agriffith96@gmail.com>
What this PR does / why we need it:
Allow installing the helm chart without the user having to clone, nor kserve having to set up a helm repository.
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 #2188
Type of changes
Please delete options that are not relevant.
Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevent result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.
Workflow succeeded https://github.com/ddelange/kserve/runs/6525109187?check_suite_focus=true
Asset uploaded https://github.com/ddelange/kserve/releases/tag/v0.8.0-helm
Logs
Special notes for your reviewer:
Checklist:
Release note: