Skip to content

Commit df3101b

Browse files
authored
Merge pull request redhat-appstudio#248 from gabemontero/openshift-perm-fixes
fix lookup-artifact-location and process-build-requests permission issues running on openshfit
2 parents e1dd73f + 21411e0 commit df3101b

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

pkg/reconciler/artifactbuild/artifactbuild.go

+4
Original file line numberDiff line numberDiff line change
@@ -524,6 +524,7 @@ func createLookupScmInfoTask(gav string, config map[string]string) *pipelinev1be
524524
recipes = recipes + "," + additional
525525
}
526526

527+
zero := int64(0)
527528
return &pipelinev1beta1.TaskSpec{
528529
Results: []pipelinev1beta1.TaskResult{
529530
{Name: PipelineResultScmUrl},
@@ -554,6 +555,9 @@ func createLookupScmInfoTask(gav string, config map[string]string) *pipelinev1be
554555
"--gav",
555556
gav,
556557
},
558+
SecurityContext: &corev1.SecurityContext{
559+
RunAsUser: &zero,
560+
},
557561
},
558562
},
559563
},

pkg/reconciler/dependencybuild/dependencybuild.go

+4
Original file line numberDiff line numberDiff line change
@@ -634,6 +634,7 @@ func createLookupBuildInfoPipeline(build *v1alpha1.DependencyBuildSpec, config m
634634
if len(path) == 0 {
635635
path = "."
636636
}
637+
zero := int64(0)
637638
return &pipelinev1beta1.PipelineSpec{
638639
Results: []pipelinev1beta1.PipelineResult{{Name: BuildInfoPipelineMessage, Value: "$(tasks." + artifactbuild.TaskName + ".results." + BuildInfoPipelineMessage + ")"}, {Name: BuildInfoPipelineBuildInfo, Value: "$(tasks." + artifactbuild.TaskName + ".results." + BuildInfoPipelineBuildInfo + ")"}},
639640
Tasks: []pipelinev1beta1.PipelineTask{
@@ -664,6 +665,9 @@ func createLookupBuildInfoPipeline(build *v1alpha1.DependencyBuildSpec, config m
664665
"--build-info",
665666
"$(results." + BuildInfoPipelineBuildInfo + ".path)",
666667
},
668+
SecurityContext: &v1.SecurityContext{
669+
RunAsUser: &zero,
670+
},
667671
},
668672
},
669673
},

0 commit comments

Comments
 (0)