kubernetes/ingress-nginx

Distinguish wait-shutdown command from standard k8s SIGTERM

Offen

#6.287 geöffnet am 06.10.2020

Auf GitHub ansehenContributor Guide
 (32 Kommentare) (1 Reaktion) (1 zugewiesene Person)Go (7.888 Forks)batch import
good first issuehelp wantedkind/featureneeds-triagepriority/backlog

Repository-Metriken

Stars
 (15.199 Sterne)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

As of v0.26.0, we can specify /wait-shutdown as a pre-stop hook in our deployment spec. As currently implemented, it sends SIGTERM to the nginx-ingress-controller process. https://github.com/kubernetes/ingress-nginx/blob/59a7f51cd4f0c41f214c520ca8b2a502946ed08e/cmd/waitshutdown/main.go#L29

Since SIGTERM is also what k8s sends when shutting down a pod, there is no way for us to really tell for sure from the logs whether the pre-stop hook was properly configured or not. For this reason, it would be better if wait-shutdown used a different signal.

/kind feature

Contributor Guide

Research-Richtung
Ändern Sie den wait shutdown Befehl, um ein anderes Signal (z. B. SIGUSR1) anstelle von SIGTERM zu senden. Aktualisieren Sie den Code in cmd/waitshutdown/main.go, um das neue Signal zu verwenden, und aktualisieren Sie die Dokumentation, um die Änderung widerzuspiegeln. Stellen Sie außerdem sicher, dass der Controller zwischen dem Pre Stop Hook Signal und dem standardmäßigen SIGTERM von k8s unterscheiden kann.
Tech Stack
go
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitGo
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70