Mount slurm-scripts to nodesets worker - #2715
Conversation
0a97520 to
08bcc5a
Compare
|
@andriishestakov I understand the problem, but ... The injected entries are unconditional and unguarded. If a NodeSet's values already contain a mount named slurm-scripts at /opt/slurm_scripts, the rendered pod gets duplicate volume names and duplicate mountPaths. Both rejected by API validation (must be unique), so the StatefulSet never rolls. CM naming duplicated across charts, and not identically. slurm-cluster computes it as default .Chart.Name .Values.clusterName | trunc 63 | trimSuffix "-" + -slurm-scripts. The new helper skips both the .Chart.Name fallback and the trunc 63, so an empty or >63-char clusterName produces divergent names (silent no-mount, or a reference to a CM that doesn't exist → pods stuck in ContainerCreating). An explicit slurmScriptsConfigMapName value that defaults to the convention would decouple the two charts. |
08bcc5a to
0fe8dd2
Compare
|
Added checks for slurm-cluster and nodesets. |
0fe8dd2 to
2df8985
Compare
Mount %s-slurm-scripts configmap to worker if cluster name is set