Skip to content

fix: cern-magnum packaging - remove values.yaml from crd chart

Jack Charlie Munday requested to merge fix-cern-magnum-values into master

This PR reverts !237 (merged) and removes the values.yaml from the crd chart not the umbrella cern-magnum chart.

There was a second issue relating to the handling of un-tarring of charts/*.tgz when packaging the crds. None of the charts where being untarred (in the helm image used), however this was working fine on my mac. Why tar behaves differently on the two OS's I am not sure...

I have tested this twice already as it did not require rebuilding the magnum image.

$ openstack coe cluster create from-branch --cluster-template kubernetes-1.30.2-1 --merge-labels --labels cern_chart_version=0.18.0###https://gitlab.cern.ch/kubernetes/automation/releases/cern-magnum###fix-cern-magnum-values --keypair jmunday

$ openstack coe cluster show from-branch-tar | grep status

| status               | CREATE_COMPLETE 

$ helm list -n kube-system

NAME                    NAMESPACE       REVISION        UPDATED                                 STATUS          CHART                   APP VERSION
cern-magnum             kube-system     1               2024-08-27 13:22:03.863620103 +0000 UTC deployed        cern-magnum-0.16.0
cern-magnum-crds        kube-system     1               2024-08-27 13:22:00.59958142 +0000 UTC  deployed        cern-magnum-crds-0.16.0

Merge request reports