Skip to content

chore(k8s): make MADFAM CV invite-only (INVITE_ONLY_CLIENTS) - #748

Open
aldoruizluna wants to merge 1 commit into
mainfrom
chore/invite-only-madfam-cv
Open

aldoruizluna wants to merge 1 commit into
mainfrom
chore/invite-only-madfam-cv

Conversation

@aldoruizluna

Copy link
Copy Markdown
Collaborator

What

This is the on-switch for the cv beta. It adds INVITE_ONLY_CLIENTS=<cv-app client>=cv to janua-api. The client id is the one already pinned in enclii's config/ecosystem-oidc-provision.yaml. Once it is on, the cv-app client admits only people entitled to the cv product.

Merge order

Merge after #747 is promoted and live. Argo syncs this env change on merge, but it does nothing until the running image reads the setting.

Right after it is live, invite the owner's test account and the first beta user. Use POST /api/v1/internal/product-invites (runbook: docs/runbooks/invite-only-clients.md). Until someone is invited, nobody can sign in to cv, operators included.

Verification

kubectl kustomize k8s/overlays/production builds, and the YAML parses.

🤖 Generated with Claude Code

The on-switch for the cv beta: the cv-app client admits only people entitled
to the `cv` product. Inert until the janua-api image that reads the setting is
promoted; merge after that, then invite the first people right away.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

📚 Documentation checks found issues. Please review the maintenance script output.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Code quality checks found issues. Please review the maintenance script output.

@github-actions

Copy link
Copy Markdown
Contributor

Security Scan Summary

Date: Fri Oct 23 01:46:14 UTC 2026
Commit: d7094a9

Scan Results

  • CodeQL: success
  • Python Security: success
  • JavaScript Security: success
  • Secrets Scan: success
  • Container Scan: success
  • DAST ZAP: skipped

@github-actions

Copy link
Copy Markdown
Contributor

🎭 Playwright E2E Test Results

  • Total: 11
  • Passed: 11 ✅
  • Failed: 0 ❌
  • Flaky: 0 ⚠️
  • Skipped: 0 ⏭️

Full Report

This branch has not been deployed

No deployments
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