Skip to content

feat(chart): add service properties for gangplank #1050

Merged
oliverbaehler merged 2 commits into
projectcapsule:mainfrom
oliverbaehler:feat/gangplank-val
Jun 9, 2026
Merged

feat(chart): add service properties for gangplank #1050
oliverbaehler merged 2 commits into
projectcapsule:mainfrom
oliverbaehler:feat/gangplank-val

Conversation

@oliverbaehler

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: Oliver Baehler <oliver@sudo-i.net>
Signed-off-by: Oliver Baehler <oliver@sudo-i.net>
Copilot AI review requested due to automatic review settings June 8, 2026 21:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the Helm chart’s Gangplank configuration to allow customizing the Gangplank Service’s metadata and ports, aligning it more closely with the chart’s main Service configurability.

Changes:

  • Add Gangplank service values for annotations, labels, and nodePort, and improve port description.
  • Update the Gangplank Service template to render the new labels/annotations and support nodePort.
  • Update the values JSON schema and Helm chart README to document the new Gangplank service options.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
charts/capsule-proxy/values.yaml Adds Gangplank service config knobs (port/nodePort/labels/annotations) and inline docs.
charts/capsule-proxy/values.schema.json Extends Gangplank service schema with the new fields and descriptions.
charts/capsule-proxy/templates/gangplank/service.yaml Renders the new service metadata fields and supports NodePort behavior.
charts/capsule-proxy/README.md Documents the new Gangplank service values in the parameters table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +721 to +723
# Add these labels to have capsule-proxy show up in `kubectl cluster-info`
# kubernetes.io/cluster-service: "true"
# kubernetes.io/name: "Capsule Proxy"
@oliverbaehler oliverbaehler merged commit d224015 into projectcapsule:main Jun 9, 2026
13 checks passed
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.

3 participants