Conversation
WalkthroughDocumentation version bumps for Helm charts (Cloudprem 4.0.0-beta.21→4.0.0-beta.22, Membership 3.0.0-beta.8→3.0.0-beta.9) with corresponding Dex image updates (v1.0.4→v2.0.0-beta.2), new Dex role field in staticPasswords, wizard configuration blocks, and template restructuring in helpers. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Fix all issues with AI agents
In `@charts/membership/README.md`:
- Line 14: Update the chart README and default values to document and require
explicit migration and new helm values for Bitnami PostgreSQL v18: note that
major-version migration (pg_dumpall/pg_upgrade) is required for moving from
15→18 and must be performed outside a helm upgrade; update references to the
NetworkPolicy keys to the new split structure for primary and readReplicas
(e.g., primary/networkPolicy and readReplicas/networkPolicy) and mark them
enabled by default; document the image verification flag
global.security.allowInsecureImages=true for custom images (or require use of
official Bitnami images) and add guidance about the image distribution change
and supported registries; update README examples and any default values
referencing global.postgresql.* to reflect these new keys and migration steps.
- Line 140: The Helm chart bumps dex.image.tag to v2.0.0-beta.2 which is a
breaking major upgrade; update all Dex connector configurations under the values
key dex.connectors to include the new required id field for each connector,
remove any unsupported role fields from dex.staticPasswords (only email, hash,
username, userID allowed), and audit any deployment/storage assumptions against
Dex v2 architecture docs to ensure the chart’s storage, config-loading and
binary expectations are compatible with v2.
No description provided.