Skip to content

fix: Make ingress service port configurable via Helm values, consistent wi… - #174

Open
rtalmaci wants to merge 1 commit into
plexinc:masterfrom
rtalmaci:bugfix/ingress-service-port
Open

fix: Make ingress service port configurable via Helm values, consistent wi…#174
rtalmaci wants to merge 1 commit into
plexinc:masterfrom
rtalmaci:bugfix/ingress-service-port

Conversation

@rtalmaci

Copy link
Copy Markdown

Issue Summary:

When setting service.port to any other value than 32400, the ingress port number is not in-sync.
eg:

service:
  type: LoadBalancer
  port: 12345

Change

Change Ingress template to use the service port number from values rather than 32400 hardcoded

Other notes:

  • No changes necessary to httproute, it already references the correct port number
  • Used .Values.service.port for consistency with httproute, rather than referencing by port name

@rtalmaci
rtalmaci requested a review from a team as a code owner August 18, 2026 13:52
@rtalmaci
rtalmaci requested review from cookandy and lotooo and removed request for a team August 18, 2026 13:52
@lotooo
lotooo requested a review from cilindrox August 19, 2026 07:26

@cilindrox cilindrox left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks @rtalmaci - this will ship alongside a couple of other pending changes soon-ish.

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.

2 participants