Skip to content

Bug: Models ConfigMap checksum missing from deployment annotations #3

@dhilgaertner

Description

@dhilgaertner

Description

templates/deployment.yaml checksums env.yaml and secrets.yaml in pod annotations but does not checksum the citadel-models ConfigMap. When modelsConfig changes via helm upgrade, the ConfigMap updates but the pod doesn't restart — leaving stale model configuration running until the pod is manually restarted.

Expected Behavior

Changing modelsConfig in values.yaml and running helm upgrade should trigger a pod rollout automatically, just like changes to env or secret values do.

Fix

Add a checksum/models annotation in templates/deployment.yaml alongside the existing checksums:

checksum/models: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}

Found During

Local testing of v0.2.1 on Docker Desktop Kubernetes.

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions