chore(deploy): retire Helm chart + Helm-based GHA workflows#62
Merged
Conversation
GitOps via cfp-sandbox-cluster + the local Kustomize manifests at
deploy/kustomize/ are now the canonical deploy path. The Helm chart and
the two Helm-driving workflows are dead weight.
- Delete deploy/charts/codeforphilly/ (entire chart + values files)
- Delete .github/workflows/deploy-{staging,production}.yml
- Rewrite docs/operations/deploy.md to describe the Kustomize+GitOps flow
(preserving boot sequence, probes, env var reference, bucket reqs,
rollback semantics). The Helm-specific install/upgrade/values sections
are gone.
- runbook.md: replace `helm history/rollback/upgrade` recovery commands
with GitOps deploy-PR revert + `kubectl set image` hotfix; reword the
CFP_DATA_REMOTE boot-failure row to point at the SealedSecret.
- secrets.md: update the "where they live in the cluster" section to
describe SealedSecrets in the cluster repo; drop the trailing
helm-install example.
- monitoring.md, cutover.md: small touch-ups removing Helm references.
plans/deploy.md is left untouched per the plans protocol (frozen at
status: done).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 task
themightychris
added a commit
that referenced
this pull request
May 18, 2026
…l-docs docs(ops): operational doc updates that were dropped from #62
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
GitOps via
cfp-sandbox-cluster+ this repo's Kustomize manifests atdeploy/kustomize/are now the canonical deploy path. The Helm chart and the two Helm-driving GHA workflows are dead weight; delete them and update the docs that reference them.Closes the cleanup half of the Helm→Kustomize migration that landed in #60 (spec) and #61 (impl).
Diff overview
deploy/charts/codeforphilly/(chart + 3 values files),.github/workflows/deploy-{staging,production}.ymldocs/operations/deploy.md(preserves boot sequence, probes, env var reference, bucket reqs, rollback semantics — drops Helm install/upgrade/values sections)docs/operations/{runbook,secrets,monitoring,cutover}.md— replacehelm history|rollback|upgradewith GitOps deploy-PR revert +kubectl set imagehotfix; reword references to the Helm chart in favor of the kustomize manifests + GitOps cluster repo.plans/deploy.mdis left untouched per the plans protocol (frozen atstatus: done).Test plan
🤖 Generated with Claude Code