Repository navigation
docs: Document NIC Helm additional controller Services - #2374
Merged
JTorreG merged 2 commits intoOct 6, 2026
Merged
Conversation
Document controller.service.additionalServices from nginx/kubernetes-ingress#10478, which lets one NGINX Ingress Controller deployment be exposed through several Kubernetes Services. Add rows for the map and each per-Service field, the naming and type rules, and the single-Service status behavior.
6 tasks done
✅ Deploy Preview will be available once build job completes!
|
JTorreG
approved these changes
Oct 6, 2026
JTorreG
left a comment
Contributor
There was a problem hiding this comment.
looks good from a TW pov
ciarams87
approved these changes
Oct 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Proposed changes
Documents nginx/kubernetes-ingress#10478, which ships in the next NGINX Ingress Controller release. The change isn't on the
release-5.6line, so this PR targetsnic-release-next.The Helm chart adds
controller.service.additionalServices, so one NGINX Ingress Controller workload can sit behind several Kubernetes Services. Each Service has its own type, ports, annotations, labels, and load balancer settings. This PR adds rows for the new parameter and its fields to the Helm chart parameters table, right aftercontroller.service.customPorts. The rows cover:type(ClusterIP,NodePort, orLoadBalancer) andportsfields, which need at least one portcontroller.service.createisfalsecontroller.reportIngressStatus.externalServiceThe new rows follow the wording of the existing
controller.service.*rows. This may be worth a release note entry. This PR doesn't change any release notes.Checklist
Before sharing this pull request, I completed the following checklist:
Footnotes
Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content. ↩