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

Helm Chart - Multiple kong releases - ingress classes overrides #5677

Open
1 task done
MostefaKamalLala opened this issue Mar 4, 2024 · 3 comments
Open
1 task done
Labels
bug Something isn't working pending author feedback stale Will be closed unless advocated for within 7 days

Comments

@MostefaKamalLala
Copy link

MostefaKamalLala commented Mar 4, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

I have a helm release of kong version 2.6.3 already deployed.
I also have another need, which is deploying another kong release in the same cluster.
I am aware of the configuration that create a specific ingress class name for each.
I left the first release with default value for the ingress class name which is ingressController.ingressClass: "kong" and the second release I changed the value to ingressController.ingressClass: "kong-2".
However the issue is that when I check the ingressClasses in Lens, I see only one ingressClass, the last one that I deployed.
I validated the with functional tests that the right ingress class is being used successfully.
I am currently testing it in non-production environment but I am not keen on deploying it in production environment yet until I make sure that it is not a helm chart bug or Lens bug. Unfortuantly there is no kubectl command to list directly the ingress classes, and using kubectl get ingress --all-namespaces -o jsonpath='{.items[*].spec.ingressClassName}' | tr ' ' '\n' | sort -u
shows only kong-2 ingress class.

Expected Behavior

Two ingressClasses

Steps To Reproduce

Deploy two helm releases. One with default ingressClassName and the other with a custom name.

Kong Ingress Controller version

v2.0

Kubernetes version

AKS v1.27.7

Anything else?

I have noticed a changed in the template for ingressClass in the version 2.7.0 Would it fixe my issue?

Also the previous ingress with the default helm release are still working.

@MostefaKamalLala MostefaKamalLala added the bug Something isn't working label Mar 4, 2024
@MostefaKamalLala MostefaKamalLala changed the title Multiple kong releases - ingress classes overrides Helm Chart - Multiple kong releases - ingress classes overrides Mar 4, 2024
@randmonkey
Copy link
Contributor

@MostefaKamalLala What was your Kong charts version? The latest version of kong/kong charts (2.38.0) will not remove your existing ingressClass. charts 2.6.3 and KIC 2.0 is quite old now and has reached the end of support.

@MostefaKamalLala
Copy link
Author

MostefaKamalLala commented Mar 18, 2024

@randmonkey I am using chart version 2.6.3. So I tryied to gradually update I reallized that the issue is fixed in subsequent versions but I faced another issue. For some reason, even if I follow the update guide, the KIC doesnt spin up.
I get an error saying: error=“resource not yet configured in the data-plane” logger=controllers.Ingress.netv1

Also it is not a big deal if it's overriden I can always add it manually without affecting the exising binding of IngressClass and other K8s objects.

Copy link

stale bot commented Apr 22, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Will be closed unless advocated for within 7 days label Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working pending author feedback stale Will be closed unless advocated for within 7 days
Projects
None yet
Development

No branches or pull requests

2 participants