fix(helm): per-agent image as string, fallback to appVersion, persistence.size default 1Gi

README.md

@@ -123,7 +123,7 @@ working_dir = "/home/agent"
 [agent]
 command = "claude-agent-acp"
 args = []
-working_dir = "/home/agent"
+working_dir = "/home/node"
 
 # Gemini
 [agent]

charts/openab/templates/_helpers.tpl

@@ -41,11 +41,22 @@ app.kubernetes.io/component: {{ .agent }}
 {{- printf "%s-%s" (include "openab.fullname" .ctx) .agent | trunc 63 | trimSuffix "-" }}
 {{- end }}
 
-{{/* Resolve image: agent-level override → global default */}}
+{{/* Resolve image: agent-level string override → global default (repository:tag, tag defaults to appVersion) */}}
 {{- define "openab.agentImage" -}}
-{{- $repo := .ctx.Values.image.repository }}
-{{- $tag := .ctx.Values.image.tag }}
-{{- if and .cfg.image .cfg.image.repository (ne .cfg.image.repository "") }}{{ $repo = .cfg.image.repository }}{{ end }}
-{{- if and .cfg.image .cfg.image.tag (ne .cfg.image.tag "") }}{{ $tag = .cfg.image.tag }}{{ end }}
-{{- printf "%s:%s" $repo $tag }}
+{{- if and .cfg.image (kindIs "string" .cfg.image) (ne .cfg.image "") }}
+{{- .cfg.image }}
+{{- else }}
+{{- $tag := default .ctx.Chart.AppVersion .ctx.Values.image.tag }}
+{{- printf "%s:%s" .ctx.Values.image.repository $tag }}
+{{- end }}
+{{- end }}
+
+{{/* Resolve imagePullPolicy: global default (per-agent image string has no pullPolicy) */}}
+{{- define "openab.agentImagePullPolicy" -}}
+{{- .ctx.Values.image.pullPolicy }}
+{{- end }}
+
+{{/* Persistence enabled: default true unless explicitly set to false */}}
+{{- define "openab.persistenceEnabled" -}}
+{{- if and . .persistence (eq (.persistence.enabled | toString) "false") }}false{{ else }}true{{ end }}
 {{- end }}

charts/openab/templates/deployment.yaml

@@ -1,6 +1,6 @@
 {{- range $name, $cfg := .Values.agents }}
 {{- $d := dict "ctx" $ "agent" $name "cfg" $cfg }}
-{{- $pvcEnabled := and $cfg.persistence $cfg.persistence.enabled }}
+{{- $pvcEnabled := not (eq (include "openab.persistenceEnabled" $cfg) "false") }}
 ---
 apiVersion: apps/v1
 kind: Deployment
@@ -31,7 +31,7 @@ spec:
       containers:
         - name: openab
           image: {{ include "openab.agentImage" $d | quote }}
-          imagePullPolicy: {{ $.Values.image.pullPolicy }}
+          imagePullPolicy: {{ include "openab.agentImagePullPolicy" $d }}
           {{- with $.Values.containerSecurityContext }}
           securityContext:
             {{- toYaml . | nindent 12 }}

charts/openab/templates/pvc.yaml

@@ -1,5 +1,5 @@
 {{- range $name, $cfg := .Values.agents }}
-{{- if and $cfg.persistence $cfg.persistence.enabled }}
+{{- if not (eq (include "openab.persistenceEnabled" $cfg) "false") }}
 {{- $d := dict "ctx" $ "agent" $name "cfg" $cfg }}
 ---
 apiVersion: v1
@@ -11,11 +11,11 @@ metadata:
 spec:
   accessModes:
     - ReadWriteOnce
-  {{- if $cfg.persistence.storageClass }}
+  {{- if and $cfg.persistence $cfg.persistence.storageClass }}
   storageClassName: {{ $cfg.persistence.storageClass }}
   {{- end }}
   resources:
     requests:
-      storage: {{ $cfg.persistence.size }}
+      storage: {{ (and $cfg.persistence $cfg.persistence.size) | default "1Gi" }}
 {{- end }}
 {{- end }}

charts/openab/values.yaml

@@ -1,6 +1,7 @@
 image:
   repository: ghcr.io/openabdev/openab
-  tag: "78f8d2c"
+  # tag defaults to .Chart.AppVersion
+  tag: ""
   pullPolicy: IfNotPresent
 
 podSecurityContext:
@@ -44,9 +45,8 @@ agents:
     #   nodeSelector: {}
     #   tolerations: []
     #   affinity: {}
-    image:
-      repository: ""
-      tag: ""
+    #   image: "ghcr.io/openabdev/openab-claude:latest"
+    image: ""
     command: kiro-cli
     args:
       - acp
@@ -68,7 +68,7 @@ agents:
     persistence:
       enabled: true
       storageClass: ""
-      size: 1Gi
+      size: 1Gi  # defaults to 1Gi if not set
     agentsMd: ""
     resources: {}
     nodeSelector: {}