Issue with helm chart and extraContainers field

Slack Knowledge Base Q&A Platform Questions
, , , ,
1

Summary

Having trouble deploying Airbyte using the extraContainers field in the helm chart, encountering errors related to converting YAML to JSON.


Question

Hi guys!

Anyone successfuly using the “*.extraContainers” part of the helm chart?
Trying to deploy with a sidecar pattern, starting with the cron.extraContainers
followed the examples but the chart is not liking it: errors with: `error converting YAML to JSON: yaml: line 166: did not find expected key``


This topic has been created from a Slack thread to give it more visibility.
It will be on Read-Only mode here. Click here if you want
to access the original thread.

Join the conversation on Slack

["helm-chart", "extracontainers", "sidecar-pattern", "yaml-to-json", "deployment-error"]
2 
    - name: ecExample
      image: <http://gcr.io/endpoints-release/my-extra-container:dev|gcr.io/endpoints-release/my-extra-container:dev>
      args: [
          "--listener_port=8081",
          "--backend=127.0.0.1:8001",
      ]
      ports:
        - containerPort: 8081
      volumeMounts:
        - name: VolumeStuff
          mountPath: /path/mounth
          readOnly: true```
Do you mind sharing how you are configuring the extra container?
3

I found out <@U01MMSDJGC9>

cron:
enabled: true
replicaCount: 1
extraContainers:
- name: "sqlproxy"
image: "<http://gcr.io/cloud-sql-connectors/cloud-sql-proxy:2.12.0|gcr.io/cloud-sql-connectors/cloud-sql-proxy:2.12.0>"
volumeMounts: []
ports:
- containerPort: 3000
args:
- "--structured-logs"
- "--address=0.0.0.0"
- "--port=3000"
- "--auto-iam-authn"
- "MY_INSTANCE"

global:

4

it’s the cron chart
https://github.com/airbytehq/airbyte-platform/blob/d7d1d2ca3ae49fec8615866f6ac8fde10a463ded/charts/airbyte-cron/templates/deployment.yaml#L179
this needs to be ``
nindent8

5

I’ll check the other charts

6

It need to update the chart? :thinking_face:

7

yeah, it seems to be an issue only with the “cron” chart

8

hey <@U047LU2HAPK>
any plans to release the chart? :3

9

PR open: https://github.com/airbytehq/airbyte-platform/pull/347
Do you guys need me to open an issue?

10

if so just let me know <@U01MMSDJGC9> :slightly_smiling_face:

11

<@U047LU2HAPK> would be possible to update the Helm Chart to fix this?

12

its always an nindentissue :rolling_on_the_floor_laughing:

13

I’ll help get this merged

14

It has been merged, we need to release a new version of the chart

15

awesome :slightly_smiling_face:
will try to start with the other containers until we release the chart :3

16

Yes, let me cut a release