Skip to content

Commit

Permalink
feat: don't add configmap volumemounts in wait container(argoproj#13089)
Browse files Browse the repository at this point in the history
  • Loading branch information
tooptoop4 authored May 25, 2024
1 parent 4d8f972 commit 8fc54ab
Showing 1 changed file with 10 additions and 0 deletions.
10 changes: 10 additions & 0 deletions workflow/controller/workflowpod.go
Original file line number Diff line number Diff line change
Expand Up @@ -1063,6 +1063,16 @@ func addOutputArtifactsVolumes(pod *apiv1.Pod, tmpl *wfv1.Template) {
if util.IsWindowsUNCPath(mnt.MountPath, tmpl) {
continue
}
isConfigMapVolume := false
for _, vol := range pod.Spec.Volumes {
if vol.Name == mnt.Name && vol.ConfigMap != nil {
isConfigMapVolume = true
break
}
}
if isConfigMapVolume {
continue
}
mnt.MountPath = filepath.Join(common.ExecutorMainFilesystemDir, mnt.MountPath)
// ReadOnly is needed to be false for overlapping volume mounts
mnt.ReadOnly = false
Expand Down

0 comments on commit 8fc54ab

Please sign in to comment.