DevFW-CICD/ingress-nginx-helm
12
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Update comment about restart of pod

Browse source
This commit is contained in:
Manuel Alejandro de Brito Fontes 2020-06-23 16:07:48 -04:00
parent d7f7e16cc8
commit 14acc186f0
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
internal/ingress/controller/nginx.go
View file

@ -337,13 +337,9 @@ func (n *NGINXController) Start() {
return
}
// if the nginx master process dies the workers continue to process requests,
// passing checks but in case of updates in ingress no updates will be
// reflected in the nginx configuration which can lead to confusion and report
// issues because of this behavior.
// To avoid this issue we restart nginx in case of errors.
// if the nginx master process dies, the workers continue to process requests
// until the failure of the configured livenessProbe and restart of the pod.
if process.IsRespawnIfRequired(err) {
// release command resources
return
}