Skip to content

feat: auto-apply extraPorts to service templates#111

Merged
joshleecreates merged 2 commits intoAltinity:mainfrom
fajarhide:main
Feb 19, 2026
Merged

feat: auto-apply extraPorts to service templates#111
joshleecreates merged 2 commits intoAltinity:mainfrom
fajarhide:main

Conversation

@fajarhide
Copy link
Copy Markdown
Contributor

@fajarhide fajarhide commented Jan 7, 2026

Summary

  • Auto-apply clickhouse.extraPorts from values.yaml to ClickHouse service templates.

  • Keeps container ports and service ports in sync with a single configuration.

Changes

  • Update charts/clickhouse/templates/chi.yaml

    • Add extraPorts to the regular service template.

    • Ensure extraPorts in the LoadBalancer service template maps containerPort → port and targetPort.

Usage

Define once in values.yaml:

clickhouse:
  extraPorts:
    - name: custom-port
      containerPort: 8080

Result

custom-port is exposed on:

  • The ClickHouse container
  • The regular Service
  • The LoadBalancer Service (when enabled)

@fkywong fkywong mentioned this pull request Feb 14, 2026
Open
joshleecreates
joshleecreates approved these changes Feb 19, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@joshleecreates joshleecreates left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@joshleecreates joshleecreates merged commit fceac54 into Altinity:main Feb 19, 2026
joshleecreates added a commit that referenced this pull request Feb 19, 2026
@joshleecreates
ClickHouse Chart Release 0.3.9
bab8e46 
Changes:
- #111: adds extraPorts to service templates
- #109: improves user profiles, mount points
- #119: uses securityContext for Keeper pods
@joshleecreates joshleecreates mentioned this pull request Feb 19, 2026
Merged
joshleecreates added a commit that referenced this pull request Feb 19, 2026
@joshleecreates
ClickHouse Chart Release 0.3.9
edcb31c 
Changes:
- #111: adds extraPorts to service templates
- #109: improves user profiles, mount points
- #119: uses securityContext for Keeper pods
fkywong added a commit to fkywong/helm-charts that referenced this pull request Feb 20, 2026
@fkywong
make tls service ports compatible with Altinity#111
113e40b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joshleecreates joshleecreates joshleecreates approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@fajarhide @joshleecreates