-
Notifications
You must be signed in to change notification settings - Fork 8.4k
Helm chart not creating controller-configmap-tcp.yaml when tcp field specified in values #9079
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
Comments
@cbc02009: This issue is currently awaiting triage. If Ingress contributors determines this is a relevant issue, they will accept it by applying the The Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/remove-kind bug Works for me ;
|
Chart that you have installed is not published by this project https://kubernetes.github.io/ingress-nginx/deploy/#quick-start . Default install you have seems to have some init container but the default install of the chart published by this project does not have any such init container
Please try the suggested process to tnstall the controller. If you find a bug or a problem that is related to the code of this project, then kindly re-open this issue with the data on the bug or the problem you found. /close |
@longwuyuan: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@longwuyuan I am using the official chart from the project. I have a kyverno policy that adds init containers to all my pods to add timezone data. I can try disabling the kyverno policy, but I don't think that will make a difference, since this happens well before the policy is applied. Edit: Here's the repo definition to prove I'm using the correct chart: https://github.com/cbc02009/k8s-home-ops/blob/main/cluster/repositories/helm/ingress-nginx-charts.yaml |
Please see my post above with copy/pasted data from a cluster created using kind. It shows that there is no problem with the functionality of opening a port for TCP, in this case 2222. The reason for closing the issue is to help reduce the use of resources for tracking support issues. If the functionality of configuring a TCP port as documented is broken, then I would not have been able to create that in my test. My suggestion is that you try your to create the same TCP service in a kind or a minikube cluster without any customization or kyverno policies or extra settings. Just try the default out-of-the-box install of ingress-nginx-controller with the values.yaml fror tcp port. That way you can see if the function is broken in all clusters or not. Also there are many more engineers on kubernetes slack, so its better to get support there because very very few people will offer support here on github for all issues. Its not possible to reply to closed issues so kindly post bug related data as proof of a problem to solve(and reopen the issue) or discuss this in kubernetes slack. |
What happened:
I am trying to add a tcp service to the nginx controller following by adding
to the values for the helm chart. (chart source available here: https://github.com/cbc02009/k8s-home-ops/blob/c5f17c8833f32ddec3d7e3f9a91b697c4099a1c4/cluster/manifests/network/ingress-nginx/helmrelease.yaml)
however the appropriate resources are not being generated from the field existing.
note: the quotes around
2222
are added to try to work around kubernetes-sigs/kustomize#3446What you expected to happen:
According to the helm source it should add the args with
and should also create the controller-configmap-tcp.yaml file
I'm honestly not sure why this would be the case, since the helm chart is pretty explicit, but you can see from the following kubectl outputs that the resources aren't being created.
NGINX Ingress controller version (exec into the pod and run nginx-ingress-controller --version.):
Kubernetes version (use
kubectl version
):Environment:
Kernel (e.g.
uname -a
):Linux <hostname> 5.19.9-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 15 Sep 2022 16:08:26 +0000 x86_64 GNU/Linux
Install tools: fluxcd using helm and kustomize. bootstrapped with kubeadm.
Please mention how/where was the cluster created like kubeadm/kops/minikube/kind etc.
Basic cluster related info:
kubectl version
:kubectl get nodes -o wide
:How was the ingress-nginx-controller installed: fluxcd
all values and settings from flux available in the gitops repo: https://github.com/cbc02009/k8s-home-ops/blob/c5f17c8833f32ddec3d7e3f9a91b697c4099a1c4/cluster/manifests/network/ingress-nginx/helmrelease.yaml
Current State of the controller:
kubectl describe ingressclasses
:kubectl -n <ingresscontrollernamespace> get all -A -o wide
:kubectl -n <ingresscontrollernamespace> describe po <ingresscontrollerpodname>
:kubectl -n <ingresscontrollernamespace> describe svc <ingresscontrollerservicename>
:The helm output is really long, so I've added it in a text file here:
ingress-nginx-helm-output.txt
The text was updated successfully, but these errors were encountered: