Skip to content

Prod: chore: remove legacy per-platform charts and stale build artifacts (closes #70) (#83)#84

Merged
saadqbal merged 1 commit intomainfrom
develop
Apr 29, 2026
Merged

Prod: chore: remove legacy per-platform charts and stale build artifacts (closes #70) (#83)#84
saadqbal merged 1 commit intomainfrom
develop

Conversation

@saadqbal
Copy link
Copy Markdown
Contributor

@saadqbal saadqbal commented Apr 29, 2026

The chart unification (4 per-platform charts -> unified client/ chart) shipped in v1.1.0; the unified chart has now been at v1.2.x in production across stg + hasan-prod for several releases. Time to retire the legacy artifacts.

Removed:

  • aks/, bm/, eks/, oc/ chart directories — 75 files, ~330KB. Each had a DEPRECATED.md pointing at the unified chart for ~6 months.
  • 7 stale .tgz tarballs at repo root (aks-1.0.3, aks-1.0.4, bm-1.0.3, bm-1.0.4, eks-1.0.3, eks-1.0.4, oc-1.0.4). The release workflow publishes via gh-pages; these checked-in builds were dead weight.
  • Root index.yaml — stale snapshot listing only 1.0.3/1.0.4 of the legacy charts. The live index served at tracebloc.github.io/client is on the gh-pages branch and is the source of truth.
  • mysql.yaml at repo root — orphaned PVC manifest with hardcoded volume UUID and namespace. Audited: zero references anywhere in the repo.

Other:

  • Added *.tgz to .gitignore so chart packages don't sneak back in.
  • Updated client/MIGRATION.md Rollback section. The old "the legacy charts remain in aks/, bm/, eks/, oc/ and can be used at any time" was about to become a lie. Replaced with instructions to recover the directory from git history if anyone genuinely needs the old chart.

Verification:

  • helm lint --strict ./client -f client/ci/eks-values.yaml — clean (same invocation the release workflow runs on every tag)
  • helm unittest client — 105/105 still passing
  • helm package ./client -d /tmp — produces a valid client-1.2.3.tgz

Net diff: 86 files changed, 17 insertions(+), 3447 deletions(-).

Summary

Related

Type of change

  • Feature
  • Bug fix
  • Tech-debt / refactor
  • Docs
  • Security / hardening
  • Breaking change

Test plan

Screenshots / recordings

Deployment notes

Checklist

  • Tests added / updated and passing locally
  • Docs updated if behavior or config changed
  • No secrets / credentials in the diff
  • For security-sensitive paths: appropriate reviewer requested

Note

Low Risk
Mostly deletes unused chart directories and stale packaged artifacts; behavior only changes for anyone still relying on the removed in-repo legacy charts.

Overview
Removes the legacy per-platform Helm charts (aks/, bm/, eks/, oc/) and associated published artifacts from the repo (including index.yaml and an orphaned mysql.yaml PVC manifest), leaving the unified client/ chart as the only in-repo chart source.

Updates .gitignore to ignore Helm package tarballs (*.tgz) and adjusts client/MIGRATION.md rollback guidance to instruct recovering legacy chart directories from git history if ever needed.

Reviewed by Cursor Bugbot for commit bd74124. Bugbot is set up for automated code reviews on this repo. Configure here.

…loses #70) (#83)

The chart unification (4 per-platform charts -> unified client/ chart)
shipped in v1.1.0; the unified chart has now been at v1.2.x in production
across stg + hasan-prod for several releases. Time to retire the legacy
artifacts.

Removed:
- aks/, bm/, eks/, oc/ chart directories — 75 files, ~330KB. Each had a
  DEPRECATED.md pointing at the unified chart for ~6 months.
- 7 stale .tgz tarballs at repo root (aks-1.0.3, aks-1.0.4, bm-1.0.3,
  bm-1.0.4, eks-1.0.3, eks-1.0.4, oc-1.0.4). The release workflow
  publishes via gh-pages; these checked-in builds were dead weight.
- Root index.yaml — stale snapshot listing only 1.0.3/1.0.4 of the
  legacy charts. The live index served at tracebloc.github.io/client
  is on the gh-pages branch and is the source of truth.
- mysql.yaml at repo root — orphaned PVC manifest with hardcoded volume
  UUID and namespace. Audited: zero references anywhere in the repo.

Other:
- Added *.tgz to .gitignore so chart packages don't sneak back in.
- Updated client/MIGRATION.md Rollback section. The old "the legacy
  charts remain in aks/, bm/, eks/, oc/ and can be used at any time"
  was about to become a lie. Replaced with instructions to recover the
  directory from git history if anyone genuinely needs the old chart.

Verification:
- helm lint --strict ./client -f client/ci/eks-values.yaml — clean
  (same invocation the release workflow runs on every tag)
- helm unittest client — 105/105 still passing
- helm package ./client -d /tmp — produces a valid client-1.2.3.tgz

Net diff: 86 files changed, 17 insertions(+), 3447 deletions(-).

Co-authored-by: Lukas Wuttke <lukas@tracebloc.io>
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-authored-by: lukasWuttke <54042461+LukasWodka@users.noreply.github.com>
@saadqbal saadqbal changed the title chore: remove legacy per-platform charts and stale build artifacts (closes #70) (#83) Prod: chore: remove legacy per-platform charts and stale build artifacts (closes #70) (#83) Apr 29, 2026
@saadqbal saadqbal merged commit ab2331b into main Apr 29, 2026
14 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