traefik-helm-chart icon indicating copy to clipboard operation
traefik-helm-chart copied to clipboard

Add notes about CRDs not applied when upgrading chart

Open mnorrsken opened this issue 4 years ago • 3 comments

#343

Tested locally:

NOTES:
######## Upgrade information ########

If upgrading from Traefik version 2.3 or earlier please reinstall helm chart or apply CRDs manually:

kubectl apply -f https://raw.githubusercontent.com/traefik/traefik-helm-chart/v9.13.0/traefik/crds/serverstransports.yaml
$ kubectl apply -f https://raw.githubusercontent.com/traefik/traefik-helm-chart/v9.13.0/traefik/crds/serverstransports.yaml
Warning: apiextensions.k8s.io/v1beta1 CustomResourceDefinition is deprecated in v1.16+, unavailable in v1.22+; use apiextensions.k8s.io/v1 CustomResourceDefinition
customresourcedefinition.apiextensions.k8s.io/serverstransports.traefik.containo.us unchanged

Disclaimer: English is not my first language.

Also unsure about the {{ .Chart.Version }} variable in the URL name

mnorrsken avatar Jan 24 '21 02:01 mnorrsken

I’m not sure why CI is not reporting green, is it something with LINT not recognizing the NOTES? Anyway this could propably be handled by adding some explicit documentation to traefik docs too.

mnorrsken avatar Jan 29 '21 12:01 mnorrsken

Because it needs a minor version bump to 9.14.3

SantoDE avatar Feb 03 '21 15:02 SantoDE

Hey @mnorrsken,

can we maybe make the download instructions to download all files in the directory, rather then just 1 file? That would be better maintainable for the future.

SantoDE avatar Mar 02 '21 14:03 SantoDE

Hello @mnorrsken,

With PR #649, we added more information on how to upgrade CRDs and we provided a one-command way to do it.

Before closing this PR, is there anything we missed and you think we should add on this topic ?

mloiseleur avatar Oct 10 '22 16:10 mloiseleur

Norhing to add, excellent work :)

mån 10 okt. 2022 kl. 18:13 skrev mloiseleur @.***>:

Hello @mnorrsken https://github.com/mnorrsken,

With PR #649 https://github.com/traefik/traefik-helm-chart/pull/649, we added more information on how to upgrade CRDs and we provided a one-command way to do it.

Before closing this PR, is there anything we missed and you think we should add on this topic ?

— Reply to this email directly, view it on GitHub https://github.com/traefik/traefik-helm-chart/pull/344#issuecomment-1273543337, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALCB73EEKN4GUNMGMM7ZJVDWCQ6CNANCNFSM4WQE23LA . You are receiving this because you were mentioned.Message ID: @.***>

mnorrsken avatar Oct 10 '22 16:10 mnorrsken