Revert "feat(backend): Source ObjStore Creds from Env in Tekton Template"

Author: gmfrasca

backend/src/apiserver/common/config.go

@@ -48,7 +48,7 @@ const (
 	MoveResultsImage                        string = "MOVERESULTS_IMAGE"
 	Path4InternalResults                    string = "PATH_FOR_INTERNAL_RESULTS"
 	ObjectStoreAccessKey                    string = "OBJECTSTORECONFIG_ACCESSKEY"
-	ObjectStoreSecretKey                    string = "OBJECTSTORECONFIG_SECRETACCESSKEY"
+	ObjectStoreSecretKey                    string = "OBJECTSTORECONFIG_SECRETKEY"
 )
 
 func IsPipelineVersionUpdatedByDefault() bool {
@@ -145,14 +145,6 @@ func GetArtifactImage() string {
 	return GetStringConfigWithDefault(ArtifactImage, DefaultArtifactImage)
 }
 
-func GetObjectStoreAccessKey() string {
-	return GetStringConfig(ObjectStoreAccessKey)
-}
-
-func GetObjectStoreSecretKey() string {
-	return GetStringConfig(ObjectStoreSecretKey)
-}
-
 func GetMoveResultsImage() string {
 	return GetStringConfigWithDefault(MoveResultsImage, DefaultMoveResultImage)
 }

backend/src/apiserver/template/tekton_template.go

@@ -258,8 +258,6 @@ func (t *Tekton) injectArchivalStep(workflow util.Workflow, artifactItemsJSON ma
 		artifacts, hasArtifacts := artifactItemsJSON[task.Name]
 		archiveLogs := common.IsArchiveLogs()
 		trackArtifacts := common.IsTrackArtifacts()
-		objectStoreAccessKey := common.GetObjectStoreAccessKey()
-		objectStoreSecretKey := common.GetObjectStoreSecretKey()
 		stripEOF := common.IsStripEOF()
 		injectDefaultScript := common.IsInjectDefaultScript()
 		copyStepTemplate := common.GetCopyStepTemplate()
@@ -342,8 +340,8 @@ func (t *Tekton) injectArchivalStep(workflow util.Workflow, artifactItemsJSON ma
 					t.getObjectFieldSelector("PIPELINERUN", "metadata.labels['tekton.dev/pipelineRun']"),
 					t.getObjectFieldSelector("PODNAME", "metadata.name"),
 					t.getObjectFieldSelector("NAMESPACE", "metadata.namespace"),
-					t.getEnvVar("AWS_ACCESS_KEY_ID", objectStoreAccessKey),
-					t.getEnvVar("AWS_SECRET_ACCESS_KEY", objectStoreSecretKey),
+					t.getSecretKeySelector("AWS_ACCESS_KEY_ID", "mlpipeline-minio-artifact", "accesskey"),
+					t.getSecretKeySelector("AWS_SECRET_ACCESS_KEY", "mlpipeline-minio-artifact", "secretkey"),
 					t.getEnvVar("ARCHIVE_LOGS", strconv.FormatBool(archiveLogs)),
 					t.getEnvVar("TRACK_ARTIFACTS", strconv.FormatBool(trackArtifacts)),
 					t.getEnvVar("STRIP_EOF", strconv.FormatBool(stripEOF)),