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

fix(ingress-nginx): remove tcp stuff #3555

Merged
merged 1 commit into from
Oct 17, 2024
Merged

fix(ingress-nginx): remove tcp stuff #3555

merged 1 commit into from
Oct 17, 2024

Conversation

rxbn
Copy link
Owner

@rxbn rxbn commented Oct 10, 2024

No description provided.

@rxbn rxbn self-assigned this Oct 10, 2024
Copy link

Path: infra/ingress-nginx/helmrelease.yaml

@@ -27,21 +27,6 @@
 namespace: default
 automountServiceAccountToken: true
 ---
-# Source: ingress-nginx/templates/controller-configmap-tcp.yaml
-apiVersion: v1
-kind: ConfigMap
-metadata:
- labels:
- app.kubernetes.io/name: ingress-nginx
- app.kubernetes.io/instance: ingress-nginx
- app.kubernetes.io/part-of: ingress-nginx
- app.kubernetes.io/managed-by: Helm
- app.kubernetes.io/component: controller
- name: ingress-nginx-tcp
- namespace: default
-data:
- "22": gitlab/gitlab-gitlab-shell:22
----
 # Source: ingress-nginx/templates/controller-configmap.yaml
 apiVersion: v1
 kind: ConfigMap
@@ -364,10 +349,6 @@
 protocol: TCP
 targetPort: https
 appProtocol: https
- - name: 22-tcp
- port: 22
- protocol: TCP
- targetPort: 22-tcp
 selector:
 app.kubernetes.io/name: ingress-nginx
 app.kubernetes.io/instance: ingress-nginx
@@ -446,7 +427,6 @@
 - --controller-class=k8s.io/ingress-nginx
 - --ingress-class=nginx
 - --configmap=$(POD_NAMESPACE)/ingress-nginx-controller
- - --tcp-services-configmap=$(POD_NAMESPACE)/ingress-nginx-tcp
 - --validating-webhook=:8443
 - --validating-webhook-certificate=/usr/local/certificates/cert
 - --validating-webhook-key=/usr/local/certificates/key
@@ -507,9 +487,6 @@
 - name: webhook
 containerPort: 8443
 protocol: TCP
- - name: 22-tcp
- containerPort: 22
- protocol: TCP
 volumeMounts:
 - name: webhook-cert
 mountPath: /usr/local/certificates/

@rxbn rxbn merged commit 6b74f27 into master Oct 17, 2024
2 checks passed
@rxbn rxbn deleted the fix/ingress-nginx branch October 17, 2024 18:57
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 this pull request may close these issues.

1 participant