Skip to content

Restructure chart for PatchMon v2 monolith architecture#5

Draft
MartinHell wants to merge 1 commit intomainfrom
feat/patchmon-v2
Draft

Restructure chart for PatchMon v2 monolith architecture#5
MartinHell wants to merge 1 commit intomainfrom
feat/patchmon-v2

Conversation

@MartinHell
Copy link
Copy Markdown
Contributor

  • Restructure chart for PatchMon v2 single Go binary (patchmon-server) replacing separate backend + frontend deployments
  • Add guacd sidecar for Windows RDP, new v2 secrets (SESSION_SECRET, AI_ENCRYPTION_KEY), and all new env vars (auth/lockout, password policy, body limits, Redis TLS)
  • Simplify routing to single service, add gatewayAPI/ingress mutual exclusion guard, bump chart to 0.3.0 / appVersion 2.0.0

Test plan

  • helm template renders cleanly with default values
  • helm lint passes
  • Verify internal DB mode produces correct DATABASE_URL and POSTGRES_HOST/USER/DB env vars
  • Verify external DB mode (URI, URI-from-secret, host+password) all render correctly
  • Verify enabling both gatewayAPI and ingress fails with clear error
  • Deploy to test cluster and confirm server pod starts with guacd sidecar
  • Verify Valkey connectivity from server pod

PatchMon v2 is a single Go binary (patchmon-server) serving both
frontend and API on port 3000. This replaces the separate Node.js
backend + frontend deployments.

- Merge backend/frontend into single server deployment and service
- Add guacd sidecar container for Windows RDP support
- Add new v2 secrets: SESSION_SECRET, AI_ENCRYPTION_KEY
- Add new env vars: auth/lockout, password policy, body limits,
  Redis TLS, password rate limiting
- Simplify HTTPRoute/Ingress to single service (no /api split)
- Add mutual exclusion guard for gatewayAPI + ingress
- Use redis helper consistently instead of inline logic
- Scope PDB selector to server component
- Bump chart to 0.3.0, appVersion 2.0.0

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

1 participant