UPSTREAM: <carry>: feat: mount EmptyDir volumes for launcher write lo…

…cations pt 2 follows up on dfe6edc, which does an EmptyDir mount for /.local and /.cache. /.config is another common write location, so EmptyDir mount that too.
red-hat-data-services · Apr 3, 2024 · e428f16 · e428f16
1 parent 680e445
commit e428f16
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 0 deletions.
diff --git a/backend/src/v2/compiler/argocompiler/container.go b/backend/src/v2/compiler/argocompiler/container.go
@@ -42,6 +42,8 @@ const (
 	dotLocalScratchName     = "dot-local-scratch"
 	dotCacheScratchLocation = "/.cache"
 	dotCacheScratchName     = "dot-cache-scratch"
+	dotConfigScratchLocation = "/.config"
+	dotConfigScratchName     = "dot-config-scratch"
 )
 
 func (c *workflowCompiler) Container(name string, component *pipelinespec.ComponentSpec, container *pipelinespec.PipelineDeploymentConfig_PipelineContainerSpec) error {
@@ -289,6 +291,12 @@ func (c *workflowCompiler) addContainerExecutorTemplate() string {
 					EmptyDir: &k8score.EmptyDirVolumeSource{},
 				},
 			},
+			{
+				Name: dotConfigScratchName,
+				VolumeSource: k8score.VolumeSource{
+					EmptyDir: &k8score.EmptyDirVolumeSource{},
+				},
+			},
 		},
 		InitContainers: []wfapi.UserContainer{{
 			Container: k8score.Container{
@@ -338,6 +346,10 @@ func (c *workflowCompiler) addContainerExecutorTemplate() string {
 					Name:      dotCacheScratchName,
 					MountPath: dotCacheScratchLocation,
 				},
+				{
+					Name:      dotConfigScratchName,
+					MountPath: dotConfigScratchLocation,
+				},
 			},
 			EnvFrom: []k8score.EnvFromSource{metadataEnvFrom},
 			Env:     commonEnvs,

diff --git a/backend/src/v2/compiler/argocompiler/testdata/create_mount_delete_dynamic_pvc.yaml b/backend/src/v2/compiler/argocompiler/testdata/create_mount_delete_dynamic_pvc.yaml
@@ -152,6 +152,8 @@ spec:
         name: dot-local-scratch
       - mountPath: /.cache
         name: dot-cache-scratch
+      - mountPath: /.config
+        name: dot-config-scratch
     initContainers:
     - command:
       - launcher-v2
@@ -188,6 +190,8 @@ spec:
       name: dot-local-scratch
     - emptyDir: { }
       name: dot-cache-scratch
+    - emptyDir: { }
+      name: dot-config-scratch
   - dag:
       tasks:
       - arguments:

diff --git a/backend/src/v2/compiler/argocompiler/testdata/hello_world.yaml b/backend/src/v2/compiler/argocompiler/testdata/hello_world.yaml
@@ -135,6 +135,8 @@ spec:
         name: dot-local-scratch
       - mountPath: /.cache
         name: dot-cache-scratch
+      - mountPath: /.config
+        name: dot-config-scratch
     initContainers:
     - command:
       - launcher-v2
@@ -171,6 +173,8 @@ spec:
       name: dot-local-scratch
     - emptyDir: {}
       name: dot-cache-scratch
+    - emptyDir: {}
+      name: dot-config-scratch
   - dag:
       tasks:
       - arguments: