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

Change nginx status port to 18080 to avoid conflicts with port 8080

Browse source
This commit is contained in:
Manuel de Brito Fontes 2016-07-19 16:15:48 -04:00
parent 0d1dd32567
commit 3ef038c557
3 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
controllers/nginx/nginx.tmpl
View file

@ -291,7 +291,7 @@ http {
# default server, including healthcheck # default server, including healthcheck
server { server {
listen 8080 default_server reuseport backlog={{ .backlogSize }}; listen 18080 default_server reuseport backlog={{ .backlogSize }};
location /healthz { location /healthz {
access_log off; access_log off;

2
controllers/nginx/nginx/command.go
View file

@ -101,7 +101,7 @@ func (ngx Manager) Name() string {
// Check returns if the nginx healthz endpoint is returning ok (status code 200) // Check returns if the nginx healthz endpoint is returning ok (status code 200)
func (ngx Manager) Check(_ *http.Request) error { func (ngx Manager) Check(_ *http.Request) error {
res, err := http.Get("http://127.0.0.1:8080/healthz") res, err := http.Get("http://127.0.0.1:18080/healthz")
if err != nil { if err != nil {
return err return err
} }

6
controllers/nginx/rc.yaml
View file

@ -93,10 +93,10 @@ spec:
hostPort: 80 hostPort: 80
- containerPort: 443 - containerPort: 443
hostPort: 443 hostPort: 443
# we expose 8080 to access nginx stats in url /nginx-status # we expose 18080 to access nginx stats in url /nginx-status
# this is optional # this is optional
- containerPort: 8080 - containerPort: 18080
hostPort: 8080 hostPort: 18080
args: args:
- /nginx-ingress-controller - /nginx-ingress-controller
- --default-backend-service=default/default-http-backend - --default-backend-service=default/default-http-backend