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

Add shareProcessNamespace to bootloader helm chart #282

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

mrhovunping
Copy link

What

Solved: #31697

In case you need to implement a sidecar pod that needs to be killed once the bootloader is completed. Example sidecar to connect to db (gcp auth proxy):
Create a pod with 3 containers:

  • Bootloader container
  • Cloud SQL Auth Proxy container that creates a secured tunnel to communicate with managed GCP database
  • Additional container that scan processes to killed Cloud SQL Auth Proxy container when bootloader container is done
    If you don't have this mechanism bootloader pod will still be ready, even if bootloader container is in completed state because Cloud SQL Auth Proxy container is still up and airbyte deployment process will be stuck.

How

Add shareProcessNamespace feature into the bootloader helm chart (https://kubernetes.io/docs/tasks/configure-pod-container/share-process-namespace/)

Recommended reading order

Can this PR be safely reverted / rolled back?

🚨 User Impact 🚨

No

@mrhovunping mrhovunping changed the title Add Add shareProcessNamespace to bootloader helm chart Oct 25, 2023
@CLAassistant
Copy link

CLAassistant commented May 7, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ mrhovunping
❌ sho34215
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants