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

Make cloudsql-proxy into a proper sidecar container #588

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mortenlj
Copy link
Contributor

This makes sure that cloudsql-proxy will start before the application, and not be stopped until the main application container has exited.

@mortenlj mortenlj self-assigned this Jan 28, 2025
@mortenlj mortenlj requested a review from a team as a code owner January 28, 2025 08:24
@kimtore
Copy link
Contributor

kimtore commented Jan 29, 2025

Refs #568

@jhrv
Copy link
Contributor

jhrv commented Feb 4, 2025

Er det noe som gjør at vi ikke kan/bør merge denne nå @mortenlj ?

@mortenlj
Copy link
Contributor Author

mortenlj commented Feb 4, 2025

Er det noe som gjør at vi ikke kan/bør merge denne nå

Jeg ble litt betenkt da jeg leste issuet som det linkes til i det issuet @kimtore linket til.
Første tanke er at Noen(tm) burde sjekke om dette vil treffe noe av det som ikke er fikset i 1.31 som vi kjører, og det hadde jeg ikke tid til her om dagen. Mulig @kimtore eller @tronghn har bedre oversikt over issuet og kan si på strakere arm enn meg om det er noe vi bør være bekymret for.

@tronghn
Copy link
Contributor

tronghn commented Feb 4, 2025

Det er en del småissues/bugs som jeg lenket til i det andre issuet. Bekymringene jeg uttrykte gjaldt primært innkommende trafikk fra kilder utenfor podden, spesielt med "named ports". Det er nok kun relevant for Wonderwall og evt. andre sidecars som eksponerer interne lyttere for scraping av metrikker e.l.

Jeg tipper at cloudsql-proxy fungerer helt fint som native sidecar, men det kan jo være lurt å teste det litt i ett av miljøene våre hvis vi er bekymret for å brekke noe.

This makes sure that cloudsql-proxy will start before the application, and not be stopped until the main application container has exited.
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 this pull request may close these issues.

4 participants