cp-helm-charts icon indicating copy to clipboard operation
cp-helm-charts copied to clipboard

Chart does not work in Kubernetes 1.25. `PodDisruptionBudget` `v1beta` is _removed_

Open dhinojosa opened this issue 2 years ago • 15 comments

The chart no longer works in 1.25 The chart should use Helm's Capabilities. object to determine the version of Kubernetes and make the right PodDisruptionBudget. The new reference is here: https://kubernetes.io/docs/tasks/run-application/configure-pdb/

I suspect this is what the new one should look like.

apiVersion: policy/v1
kind: PodDisruptionBudget
metadata:
  name: confluent-oss-cp-zookeeper-pdb
spec:
  minAvailable: 2
  selector:
    matchLabels:
      app: cp-zookeeper

dhinojosa avatar Sep 22 '22 13:09 dhinojosa

Same issue here - version 0.6.x has the same behavior with k8s 1.25 and minikube. Downgrading to 1.24.1 fixes the problem.

markmixson avatar Sep 22 '22 15:09 markmixson

Same issue on microk8s and k8s v. 1.25.2

aseksenali avatar Oct 08 '22 08:10 aseksenali

Also having an issue related to this on Azure Kubernetes Service v1.23.8. ArgoCD (v2.5.0-rc2) returns an error on sync: ComparisonError: error calculating structured merge diff: error calculating diff: error while running updater.Apply: converting (v1beta1.PodDisruptionBudget) to (v1.PodDisruptionBudget): unknown conversion

vancwo avatar Oct 31 '22 20:10 vancwo

Same here

laran avatar Nov 10 '22 17:11 laran

PodDisruptionBudget of beta version is unavailable from 1.25 on. please replace it with policy/v1

leewoobin789 avatar Nov 13 '22 16:11 leewoobin789

Same problem here. @leewoobin789 solution worked fine for me

  1. Cloned the repo locally
  2. Edited charts/cp-zookeeper/templates/poddisruptionbudget.yaml and replaced policy/v1beta to policy/v1
  3. Install helm using the local repo

ymoreiratiti avatar Nov 16 '22 11:11 ymoreiratiti

This problem is going to get worse as the cloud clusters versions keep moving forwards, it's now a problem in EKS now 1.25 is out.

danielloader avatar Mar 08 '23 12:03 danielloader

Do we have some news about this issue?

ymoreiratiti avatar Apr 17 '23 20:04 ymoreiratiti

This problem is going to get worse as the cloud clusters versions keep moving forwards, it's now a problem in EKS now 1.25 is out.

To reply to myself we moved to 1.26 today 😅

danielloader avatar Apr 17 '23 20:04 danielloader

I don't think that confluent will ever care about this chart, since they have a commercial operator (CFK). To those who needs this type of chart for any purpose, I recommend you to use either CFK or Strimzi. I actually hoped that confluent defines this repository as deprecated, since there is no release since 2020.

leewoobin789 avatar Apr 18 '23 09:04 leewoobin789

Deleting the history for the chart from the secrets seems to fix it.

yoda avatar Jun 29 '23 05:06 yoda

Is there any change this will be fixed in the near future?

malibcia avatar Jul 19 '23 10:07 malibcia

hoped that confluent defines this repository as deprecated

It is, and they did (#638).

This is how you should be using Helm now

OneCricketeer avatar Aug 01 '23 21:08 OneCricketeer

hoped that confluent defines this repository as deprecated

It is, and they did (#638).

This is how you should be using Helm now Colleagues,

As of this day the issue still persists.

https://packages.confluent.io/helm is no longer available (/helm has been removed)

CFK 2.7.1 on AKS 1.26.6 giving PodDistributionBudget errors since v1beta1 is not available.

As of 2023-10-13 helm charts & CFK from official examples seem to be not working, which is really worrying since those are basically stopping anyone from deploying confluent platform to K8s 1.25+ without hacks which would be prohibited in a prod environment.

Any expectation on when Confluent would address that ?

RodionXedin avatar Oct 13 '23 13:10 RodionXedin

I'm not sure any Confluent employee watches this repo anymore

If the CFK instructions are broken, please contact support directly.

OneCricketeer avatar Oct 13 '23 13:10 OneCricketeer