wekan / charts

Wekan Helm Charts to deploy to Kubernetes
MIT License
2 stars 15 forks source link

Support k8s >= 1.22 ingress API #1

Closed varac closed 2 years ago

varac commented 2 years ago

https://kubernetes.io/blog/2021/08/04/kubernetes-1-22-release-announcement/#removal-of-several-deprecated-beta-apis

    ❯ helm lint .
    ==> Linting .
    [WARNING] templates/ingress.yaml: networking.k8s.io/v1beta1 Ingress is deprecated in v1.19+, unavailable in v1.22+; use networking.k8s.io/v1 Ingress

    1 chart(s) linted, 0 chart(s) failed
varac commented 2 years ago

This is needed in k8s 1.22, and compatible with version >=1.19

varac commented 2 years ago

Ready for review!

varac commented 2 years ago

Oh noes, this is so confusing. I submitted my PR to this repo, but we use the helm chart from https://github.com/wekan/wekan/tree/master/helm/wekan. Please consolidate this / maintain only one helm chart (and delete the other one), otherwise they will diverge.

I know you're in the process of publishing the helm chart to artifacthub, hope you'll get it done soon :crossed_fingers:

varac commented 2 years ago

@xet7 So which location will recieve updates from your point of view ?

xet7 commented 2 years ago

@varac

I know nothing about helm. And I don't know what repo to add to artifacthub and how. Pull requests etc info very welcome.

I wish having all at https://github.com/wekan/wekan could be setup somehow.

varac commented 2 years ago

@xet7 I created https://github.com/wekan/charts/pull/3 to fix the helm repo