Skip to content

Commit acdf31f

Browse files
committed
fix: default the image tag for sackd in a nodeset pod correctly
1 parent ae08235 commit acdf31f

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ All notable changes to this project will be documented in this file.
1212

1313
- Fixed token job `ttlSecondsAfterFinished` being too low for helm
1414
`--wait-for-jobs`.
15+
- Fixed nodeset pod's sackd image tag default value.
1516

1617
### Changed
1718

helm/slurm/templates/compute/compute-nodeset.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ SPDX-License-Identifier: Apache-2.0
1010
{{- $imageRef := printf "%s:%s" $repository $tag }}
1111
{{- $name := printf "%s-%s" (include "slurm.compute.name" $) $nodeset.name }}
1212
{{- $authcredRepository := include "slurm.authcred.image.repository" $ }}
13-
{{- $authcredTag := include "slurm.authcred.image.repository" $ }}
13+
{{- $authcredTag := include "slurm.authcred.image.tag" $ }}
1414
{{- $authcredImagePullPolicy := $nodeset.imagePullPolicy | default (include "slurm.imagePullPolicy" $) }}
1515
{{- $authcredResources := $.Values.authcred.resources }}
1616
{{- with $nodeset.authcred -}}

0 commit comments

Comments
 (0)