Skip to content

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

Merged
cilindrox merged 1 commit into
plexinc:masterfrom
rtalmaci:bugfix/ingress-service-port
Aug 27, 2026
Merged

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

Conversation

@rtalmaci

Copy link
Copy Markdown
Contributor

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.

@cilindrox cilindrox self-assigned this Aug 27, 2026
@cilindrox
cilindrox merged commit 5878896 into plexinc:master Aug 27, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants