jx3-gitops-repositories / jx3-kubernetes

Jenkins X 3.x GitOps repository for a vanilla Kubernetes cluster
Apache License 2.0
14 stars 46 forks source link

Kuberhealthy chart 60 uses deprecated API apiextensions.k8s.io/v1beta1 #18

Open wzcwts521 opened 2 months ago

wzcwts521 commented 2 months ago

https://github.com/jx3-gitops-repositories/jx3-kubernetes/blame/7d75ad00401eb3449bfc35f183f0452ad217b234/helmfiles/kuberhealthy/helmfile.yaml#L14

customresourcedefinition.apiextensions.k8s.io/tasks.tekton.dev serverside-applied
resource mapping not found for name: "khchecks.comcast.github.io" namespace: "" from "config-root/customresourcedefinitions/kuberhealthy/kuberhealthy/khchecks.comcast.github.io-crd.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "khjobs.comcast.github.io" namespace: "" from "config-root/customresourcedefinitions/kuberhealthy/kuberhealthy/khjobs.comcast.github.io-crd.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
resource mapping not found for name: "khstates.comcast.github.io" namespace: "" from "config-root/customresourcedefinitions/kuberhealthy/kuberhealthy/khstates.comcast.github.io-crd.yaml": no matches for kind "CustomResourceDefinition" in version "apiextensions.k8s.io/v1beta1"
ensure CRDs are installed first
ErikEngerd commented 2 weeks ago

encountering this as well

ErikEngerd commented 2 weeks ago

looks like the jx3-kubernetes template is one of the few templates (the only one?) that uses kuberhealthy. Perhaps it can simply be removed but don't know the implications of this. Looks like all it adds is health monitoring with prometheus metrics. It is also installing a very old version of the kuberhealthy helm chart.

Can any one confirm that kuberhealthy is safe to remove? And perhaps also a good idea to check other versions. Is there another template that is better to use as a starting point?

ErikEngerd commented 1 week ago

See also this pull request: https://github.com/jx3-gitops-repositories/jx3-kubernetes/pull/14, but that gives different errors.