Skip to content

Helm: Add extraContainers to airbyte-cron #20506

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

Closed
wants to merge 5 commits into from

Conversation

rob-pomelo
Copy link

What

Add extra containers to airbyte-cron, similar to #19703.

This is needed for using cloud sql auth proxy. All other charts support piping these fields.

@CLAassistant
Copy link

CLAassistant commented Dec 15, 2022

CLA assistant check
All committers have signed the CLA.

@marcosmarxm
Copy link
Member

@rob-pomelo did you checked if the nident is correct?

@marcosmarxm
Copy link
Member

Hello 👋:skin-tone-2: and thank you for your contribution!

Airbyte has instituted a code freeze between 19 and 30 December, to make sure there are no disruptions during the holidays.
Because of this, reviewing and merging your contribution may take longer than usual.
We apologize for the delay, but we want everyone to have a quiet and happy holiday.

If you have any questions or need further clarification, please don't hesitate to ping via Slack.

@nightmareartist
Copy link

👋🏻 Any updates on this? The charts are unusable with SQL proxy at the moment.

@marcosmarxm
Copy link
Member

Close, continuing the work in airbyte-platform.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform issues related to the platform community kubernetes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants