Create a PR in upstream for the 0.10 release.
wwitzel3
All 18 comments
I'm building a helm chart for use internally at my company. I've run into a failure due to a CRD not being immediately available. The prereqs file creates the BackupStorageLocation and VolumeStorageLocation CRDs but to create a resource of kind BSL or VSL fails because those CRDs are not found.
Helm is aware of this:
https://docs.helm.sh/chart_best_practices/#install-a-crd-declaration-before-using-the-resource
mehemken
on 30 Nov 2018
A third option would be to extend the current hook-deploy.yaml file and create the custom resources with a job. The downside is that the custom resources are not managed by the helm release then.
wiegandf
on 1 Dec 2018
Helm's support for CRDs is a known problem. I was pointed to https://github.com/helm/helm/issues/4863 as something we should look into which also references this community proposal. We should be very careful until the Helm community works this out.
rosskukulinski
on 4 Dec 2018
and FWIW, Istio & cert-manager are also facing similar challenges
rosskukulinski
on 4 Dec 2018
Any update on this. is it still in progress?
vidhyashankarpaysafe
on 8 Jan 2019
I have not had a chance to really get started on this yet, but it is something we are still planning to do. I will try to make some progress on this, this week.
wwitzel3
on 8 Jan 2019
I'm looking to deploy ark on bare-metal and I guess the helm chart is the way to go? Only thing is that it currently only supports 0.9.0 so having the chart updated would be nice :-)
jonaskello
on 13 Jan 2019
https://github.com/helm/charts/commit/3c34df71ac7da7b490427622b5b90151aee79e14
^ Got this merged already
agolomoodysaada
on 19 Feb 2019
@agolomoodysaada thanks! when @wwitzel3 is back in the office I'm sure he'd like to chat with you about maintenance of the chart going forward.
skriss
on 19 Feb 2019
Sounds good. You can find me as saada on the velero slack channel
agolomoodysaada
on 20 Feb 2019
@nrb FYI - andy had a PR to update some docs links that never got merged. Could you ensure this stuff is updated as part of this PR?
skriss
on 8 Mar 2019
@skriss Yeah, I had seen several open PRs for v0.10 for quality of life improvements. I'll make sure they're rolled up into the v0.11 change.
nrb
on 8 Mar 2019
https://github.com/helm/charts/pull/12031 is in flight, I'm going to help review this since I was a bit behind.
nrb
on 14 Mar 2019
@nrb nice. keep an eye out for any UX issues with it that we might want to tackle. Particularly interested in if the CRD creation race condition has been addressed?
Probably also useful to add a brief doc to our repo on what a typical "upgrade the Helm chart" PR looks like so going fwd we have a guide for what to do each release.
skriss
on 19 Mar 2019
https://github.com/helm/charts/pull/12031 is merged.
nrb
on 19 Apr 2019
skriss
on 23 Apr 2019
IMO, once 13209 is merged, we can close this issue and make a separate one for our own documentation. And another one for getting the chart ready for 1.0.
nrb
on 24 Apr 2019
Docs PR is merged - closing this issue. We'll open 1+ new issues for additional 1.0 work.
skriss
on 25 Apr 2019
Related issues
Chams91
路
4Comments
Berndinox
路
3Comments
akgunjal
路
3Comments
my1990
路
3Comments
carlisia
路
4Comments
Most helpful comment
@skriss Yeah, I had seen several open PRs for v0.10 for quality of life improvements. I'll make sure they're rolled up into the v0.11 change.