GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Add Helm chart by andrew · Pull Request #250 · git-pkgs/proxy

Conversation

@andrew

@andrew andrew commented Aug 13, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Adds a Helm chart for deploying proxy with persistent SQLite and filesystem storage by default. It supports existing ConfigMaps and PVCs, environment-based secret overrides, ingress, and hardened pod settings.

The chart is checked in CI and published to oci://ghcr.io/git-pkgs/charts/proxy with its version and appVersion synchronized to the container release. The release flow follows the example shared by @acidghost in #226.

Closes #226

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@andrew andrew mentioned this pull request Aug 13, 2026
Provide an Helm chart #226
Closed
@andrew
andrew requested a lite review from Copilot August 13, 2026 09:40
Copilot started reviewing on behalf of andrew August 13, 2026 09:40
View session
Copilot AI reviewed Aug 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Pull request overview

Adds a first-party Helm chart to deploy git-pkgs/proxy to Kubernetes, with defaults geared toward persistent SQLite + filesystem-backed storage, and integrates chart linting/rendering in CI plus publishing the chart as an OCI artifact to GHCR during releases (closing #226).

Changes:

  • Introduces a Helm chart (deploy/charts/proxy) with ConfigMap/PVC support, hardened pod/container security settings, and optional ingress.
  • Updates CI to lint and template the chart (including a non-default values variant).
  • Extends the release publishing workflow to package and push the chart to GHCR with version/appVersion synced to the release tag.

Reviewed changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
README.md Documents Helm install and points to chart values.
deploy/charts/proxy/Chart.yaml Adds chart metadata with placeholder version/appVersion for release-time injection.
deploy/charts/proxy/values.yaml Defines default chart values (SQLite + filesystem + PVC, probes, security contexts, ingress toggles).
deploy/charts/proxy/.helmignore Adds Helm packaging ignore rules.
deploy/charts/proxy/templates/_helpers.tpl Adds naming/label helpers and image reference helper (tag vs digest).
deploy/charts/proxy/templates/configmap.yaml Renders a ConfigMap containing config.yaml unless an existing ConfigMap is provided.
deploy/charts/proxy/templates/deployment.yaml Deploys the proxy container with config/data volumes, probes, and hardened settings.
deploy/charts/proxy/templates/service.yaml Exposes the proxy via a ClusterIP Service.
deploy/charts/proxy/templates/pvc.yaml Creates a PVC when persistence is enabled and no existing claim is provided.
deploy/charts/proxy/templates/ingress.yaml Adds optional Ingress support driven by values.
deploy/charts/proxy/templates/NOTES.txt Provides post-install guidance (base_url, port-forward, persistence warning).
.github/workflows/ci.yml Adds Helm lint + template rendering job in CI.
.github/workflows/publish.yml Adds chart packaging + push-to-GHCR job gated on tags.
.gitignore Adjusts ignore patterns for built artifacts.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
Comment thread deploy/charts/proxy/values.yaml
Comment on lines +54 to +57
persistence:
enabled: true
mountPath: /data
existingClaim: ""
Comment thread deploy/charts/proxy/templates/ingress.yaml
Comment on lines +1 to +2
{{- if .Values.ingress.enabled }}
apiVersion: networking.k8s.io/v1
Comment thread deploy/charts/proxy/values.yaml
@@ -0,0 +1,133 @@
replicaCount: 1
andrew added 2 commits September 3, 2026 17:05
@andrew
Add Helm chart
7b03b1b
@andrew
Fail early on ingress without hosts and document replica/mount coupling
e98a0ee
@andrew
andrew force-pushed the issue-226-helm-chart branch from 116f763 to e98a0ee Compare September 3, 2026 16:09
@andrew
andrew marked this pull request as ready for review September 3, 2026 16:11
@andrew
andrew merged commit 230fa7b into main Sep 4, 2026
11 checks passed

Uh oh!

There was an error while loading. Please reload this page.
@andrew
andrew deleted the issue-226-helm-chart branch September 4, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Provide an Helm chart

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@andrew
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.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.