druid-io / druid-operator

Druid Kubernetes Operator
Other
205 stars 93 forks source link

PodDisruptionBudget API changed #346

Open domenicbove opened 8 months ago

domenicbove commented 8 months ago

Hi! We recently upgraded K8s versions (to 1.28). And the operator is giving this error:

1.704997223775299e+09   ERROR   druid_operator_handler  failed to list [PodDisruptionBudget] due to [no matches for kind "PodDisruptionBudget" in version "policy/v1beta1"]     {"name": "druid", "namespace": "druid", "error": "failed to list [PodDisruptionBudget] due to [no matches for kind \"PodDisruptionBudget\" in version \"policy/v1beta1\"]"}
github.com/druid-io/druid-operator/controllers/druid.deleteUnusedResources
        /workspace/controllers/druid/handler.go:636
github.com/druid-io/druid-operator/controllers/druid.deployDruidCluster
        /workspace/controllers/druid/handler.go:346
github.com/druid-io/druid-operator/controllers/druid.(*DruidReconciler).Reconcile
        /workspace/controllers/druid/druid_controller.go:66
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:114
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:311
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:227

From the code, it looks like yall are using v1beta1: https://github.com/druid-io/druid-operator/blob/9cec3f2814d36eb54af554372c4c45eedafc63bd/controllers/druid/handler.go#L1369

But the PDBs now are policy/v1

Is there anyway to skip the creation of PDBs while this is getting resolved?

AdheipSingh commented 8 months ago

@domenicbove this repo is deprecated. Pls migrate to this repo https://github.com/druid-io/druid-operator?tab=readme-ov-file#deprecation-notice