search

nais / naiserator

Naiserator creates a full set of Kubernetes application infrastructure based on a single application spec.
MIT License
41 stars 12 forks source link

add quitquitquit for cloudsql-proxy #502

Closed Muni10 closed 8 months ago

Starefossen commented 8 months ago

The Proxy includes support for an admin server on localhost. By default, the the admin server is not enabled. To enable the server, pass the --debug or --quitquitquit flag. This will start the server on localhost at port 9091. To change the port, use the --admin-port flag.

When --quitquitquit is set, the admin server adds an endpoint at /quitquitquit. The admin server exits gracefully when it receives a POST request at /quitquitquit.

https://github.com/GoogleCloudPlatform/cloud-sql-proxy?tab=readme-ov-file#localhost-admin-server

Starefossen commented 8 months ago

Kaller vi /quitquitquit fra noe sted når dette er slått på?

Kyrremann commented 8 months ago

Kaller vi /quitquitquit fra noe sted når dette er slått på?

Tipper Hahaha skal kalle det endepunktet.

Muni10 commented 8 months ago

Det stemmer @Kyrremann, hahaha skal kalle dette endepunktet. Etter oppgradering til cloudsql-proxy@v2 så fungerer ikke lenger sigint som vi har brukt fra hahaha, da får vi en feilmelding.i pod'en som kjører og jobber tror de har feilet og restarter i de tilfellene restartPolicy er satt. For å unngå dette legger vi inn quitquitquit endepunktet og kaller dette fra hahaha. Det negative er at det startes en localhost admin server på 9091. Alternativet er et flagg --exit-zero-on-sigterm, som overstyrer det docker gir (143 eller 130) og exiter containeren med 0 ved alle TERM signaler. Jeg synes det virker litt skummelt i tilfelle containeren får TERM fra andre kilder, som vi kanskje vil undersøke hvis vi oppdager det. Da tenker jeg det er bedre å be den avslutte pent via quitquitquit.