Skip to content

Commit e0aae1c

Browse files
AlexsJonesclaude
andcommitted
fix(install): disable chart namespace template to avoid collision
The CLI creates the sympozium-system namespace via Helm's install.CreateNamespace flag, while the chart's templates/namespace.yaml also rendered a Namespace resource (createNamespace=true by default). Both racing produced "namespaces \"sympozium-system\" already exists" on install, leaving a failed release with orphaned resources. Default createNamespace=false in buildHelmValues so the chart template is skipped; --set can still override for standalone chart use. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 5b3cf42 commit e0aae1c

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

cmd/sympozium/main.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1240,6 +1240,10 @@ func newHelmConfig(ns string) (*action.Configuration, error) {
12401240
// buildHelmValues constructs a Helm values map from CLI flags.
12411241
func buildHelmValues(imageTag string, setValues []string) (map[string]interface{}, error) {
12421242
vals := make(map[string]interface{})
1243+
// The CLI manages namespace creation itself via Helm's install.CreateNamespace
1244+
// flag, so disable the chart's own Namespace template to avoid the two
1245+
// racing and producing an "already exists" error on install.
1246+
vals["createNamespace"] = false
12431247
if imageTag != "" {
12441248
vals["image"] = map[string]interface{}{
12451249
"tag": imageTag,

0 commit comments

Comments
 (0)