Skip to content

[CU-86b3j8fqg] Move wallet config from cloud-init to K8s CRDs#154

Open
jstromsky wants to merge 3 commits intomainfrom
Move-wallet-config-to-k8s-crds-CU-86b3j8fqg
Open

[CU-86b3j8fqg] Move wallet config from cloud-init to K8s CRDs#154
jstromsky wants to merge 3 commits intomainfrom
Move-wallet-config-to-k8s-crds-CU-86b3j8fqg

Conversation

@jstromsky
Copy link
Copy Markdown
Contributor

Summary

  • Migrates 2 wallet clients (data-connect-trino, e2e-test) with all grants from cloud-init to WalletClient CRDs in helm/templates/clients.yaml
  • Migrates 1 wallet policy (4 statements) to WalletPolicy CRD in helm/templates/policies.yaml
  • Trims cloud-init to setupAppDatabase + labelSecret
  • Deletes bootstrap-config/ directory

Secrets compatibility

Existing secrets created by old cloud-init will NOT be overwritten — the lookup pattern skips secrets without helm annotations.

Test plan

  • Deploy to dev and verify wallet operator reconciles CRDs (kubectl get walletclients,walletpolicies for SYNCED)
  • Verify cloud-init still runs for DB setup and labelSecret
  • Run e2e tests to confirm grants and policies work

🤖 Generated with Claude Code

…a-connect-trino

Migrate wallet clients, grants, and policy from cloud-init REST API
calls to declarative Helm-managed WalletClient and WalletPolicy CRDs.

New Helm templates:
- clients.yaml: 2 WalletClient CRDs (data-connect-trino, e2e-test)
  with K8s Secrets using lookup-based preservation
- policies.yaml: 1 WalletPolicy CRD (data-connect-trino-access-policy)

Cloud-init now only runs setupAppDatabase and labelSecret.
Deleted bootstrap-config directory.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@platform-automation-dnastack
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Test Results

49 tests  ±0   49 ✅ ±0   11s ⏱️ ±0s
 5 suites ±0    0 💤 ±0 
 5 files   ±0    0 ❌ ±0 

Results for commit 270eedb. ± Comparison against base commit bff122d.

♻️ This comment has been updated with latest results.

jstromsky and others added 2 commits April 7, 2026 09:49
…ption hook

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…cies

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants