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

controller deamonset ports are fixed #143

Closed
fga1794 opened this issue Jun 30, 2023 · 5 comments · Fixed by nginx/kubernetes-ingress#4252
Closed

controller deamonset ports are fixed #143

fga1794 opened this issue Jun 30, 2023 · 5 comments · Fixed by nginx/kubernetes-ingress#4252

Comments

@fga1794
Copy link

fga1794 commented Jun 30, 2023

we need to deploy nginx ingress. Howerver on openshift, there is a default controller that use default ports 80 & 443.
So we need to custom ports . in the chart helm, the parameters are not variable.

in template Breadcrumbsnginx-ingress-helm-operator/helm-charts/nginx-ingress/templates
/controller-daemonset.yaml

.....
{{ toYaml .Values.controller.lifecycle | indent 10 }}
{{- end }}
        ports:
        - name: http
          containerPort: 80
          hostPort: 80
        - name: https
          containerPort: 443
          hostPort: 443
{{ if .Values.controller.customPorts }}
{{ toYaml .Values.controller.customPorts | indent 8 }}
....

can you change it as variable ?

@lucacome
Copy link
Member

Hi @fga1794 do you need the hostPort to be configurable? If so we have an issue here nginx/kubernetes-ingress#3613 for that.

@fga1794
Copy link
Author

fga1794 commented Jun 30, 2023

Hello @lucacome i need all this to be configured
ports: - name: http containerPort: 80 hostPort: 80 - name: https containerPort: 443 hostPort: 443

@lucacome
Copy link
Member

@fga1794 can you explain your use case? Do you build a custom NGINX Ingress Controller image and change the ports for the container?

@fga1794
Copy link
Author

fga1794 commented Jul 3, 2023

openshift deploy a standard ingress controller that use port 80 & 443.
So when we tried to deploy nginx ingress controller we got error that the port 80 & 443 are used. So, we think that we need to change this port to used other ones for nginx ingress controller

@vincentor
Copy link

need the same feature too, when will it be merged?

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 a pull request may close this issue.

3 participants