Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

connectors errors in 5.8.6 #1321

Closed
jason-da-redpanda opened this issue May 24, 2024 · 0 comments · Fixed by #1322
Closed

connectors errors in 5.8.6 #1321

jason-da-redpanda opened this issue May 24, 2024 · 0 comments · Fixed by #1322

Comments

@jason-da-redpanda
Copy link

jason-da-redpanda commented May 24, 2024

What happened?

seems there is something up for connectors in helm/operator. with 5.8.6 ..

When applying with the following yaml

apiVersion: cluster.redpanda.com/v1alpha1 kind: Redpanda metadata: name: redpanda spec: chartRef: chartVersion: 5.8.6 clusterSpec: storage: persistentVolume: enabled: true size: "27Gi" connectors: enabled: true logging: level: info external: domain: customredpandadomain.local statefulset: initContainers: setDataDirOwnership: enabled: true

Encounter this error when it tries to create the connectors ..
,"error":"template: redpanda/templates/console/configmap-and-deployment.yaml:84:37: executing \"redpanda/templates/console/configmap-and-deployment.yaml\" at <$.Subcharts.connector>: wrong type for value; expected map[string]interface {}; got interface {}"}

Same yaml works fine in 5.8.5

Applying this way reports the same :

helm upgrade --install redpanda redpanda/redpanda --namespace jbarlow --create-namespace --set connectors.enabled=true 
Error: UPGRADE FAILED: template: redpanda/templates/console/configmap-and-deployment.yaml:84:37: executing "redpanda/templates/console/configmap-and-deployment.yaml" at <$.Subcharts.connector>: wrong type for value; expected map[string]interface {}; got interface {}

These are logs from a customer that report it too

customer logs:

install.go:200: [debug] Original chart version: ""
install.go:217: [debug] CHART PATH: /Users/pmw/Library/Caches/helm/repository/redpanda-5.8.6.tgz


Error: template: redpanda/templates/console/configmap-and-deployment.yaml:84:37: executing "redpanda/templates/console/configmap-and-deployment.yaml" at <$.Subcharts.connector>: wrong type for value; expected map[string]interface {}; got interface {}
helm.go:84: [debug] template: redpanda/templates/console/configmap-and-deployment.yaml:84:37: executing "redpanda/templates/console/configmap-and-deployment.yaml" at <$.Subcharts.connector>: wrong type for value; expected map[string]interface {}; got interface {}

What did you expect to happen?

To run without error and create the connector pod

How can we reproduce it (as minimally and precisely as possible)?. Please include values file.

$ helm get values <redpanda-release-name> -n <redpanda-release-namespace> --all
# paste output here

Anything else we need to know?

No response

Which are the affected charts?

No response

Chart Version(s)

$ helm -n <redpanda-release-namespace> list 
# paste output here

Cloud provider

JIRA Link: K8S-238

chrisseto added a commit that referenced this issue May 24, 2024
In commit 5d44bb5, a typo was accidentally
introduced that broke the connectors integration of the redpanda chart. This
issue went unnoticed until discovered by an end user due to a lack of any test
case that specified `connectors.enabled=true`.

This commit corrects the typo and adds a minimal regression test. A manual
check was also performed to ensure that the behavior of
`connectors.enabled=true` is the same prior to
`5d44bb5a3866608c435ce57ffbcc172dd91ec74c`

Fixes #1321
chrisseto added a commit that referenced this issue May 24, 2024
In commit 5d44bb5, a typo was accidentally
introduced that broke the connectors integration of the redpanda chart. This
issue went unnoticed until discovered by an end user due to a lack of any test
case that specified `connectors.enabled=true`.

This commit corrects the typo and adds a minimal regression test. A manual
check was also performed to ensure that the behavior of
`connectors.enabled=true` is the same prior to
`5d44bb5a3866608c435ce57ffbcc172dd91ec74c`

Fixes #1321
RafalKorepta pushed a commit that referenced this issue May 24, 2024
In commit 5d44bb5, a typo was accidentally
introduced that broke the connectors integration of the redpanda chart. This
issue went unnoticed until discovered by an end user due to a lack of any test
case that specified `connectors.enabled=true`.

This commit corrects the typo and adds a minimal regression test. A manual
check was also performed to ensure that the behavior of
`connectors.enabled=true` is the same prior to
`5d44bb5a3866608c435ce57ffbcc172dd91ec74c`

Fixes #1321
RafalKorepta pushed a commit to redpanda-data/redpanda-operator that referenced this issue Dec 3, 2024
In commit 82a7e51, a typo was accidentally
introduced that broke the connectors integration of the redpanda chart. This
issue went unnoticed until discovered by an end user due to a lack of any test
case that specified `connectors.enabled=true`.

This commit corrects the typo and adds a minimal regression test. A manual
check was also performed to ensure that the behavior of
`connectors.enabled=true` is the same prior to
`82a7e519af7ad9591a5f55ac245022e0886ba22b`

Fixes redpanda-data/helm-charts#1321
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant