Merge pull request #91 from hbelmiro/RHOAIENG-13327

UPSTREAM: <carry>: Fixed importer component when podToPodTLS is true

Author: HumairAK

backend/src/v2/compiler/argocompiler/importer.go

@@ -17,6 +17,7 @@ package argocompiler
 import (
 	"fmt"
 	"github.com/kubeflow/pipelines/backend/src/apiserver/common"
+	"strconv"
 
 	wfapi "github.com/argoproj/argo-workflows/v3/pkg/apis/workflow/v1alpha1"
 	"github.com/kubeflow/pipelines/api/v2alpha1/go/pipelinespec"
@@ -79,6 +80,7 @@ func (c *workflowCompiler) addImporterTemplate() string {
 		fmt.Sprintf("$(%s)", component.EnvPodUID),
 		"--mlmd_server_address", common.GetMetadataGrpcServiceServiceHost(),
 		"--mlmd_server_port", common.GetMetadataGrpcServiceServicePort(),
+		"--metadataTLSEnabled", strconv.FormatBool(common.GetMetadataTLSEnabled()),
 	}
 	importerTemplate := &wfapi.Template{
 		Name: name,

backend/src/v2/compiler/argocompiler/testdata/importer.yaml

@@ -41,6 +41,8 @@ spec:
       - "metadata-grpc-service"
       - --mlmd_server_port
       - "8080"
+      - --metadataTLSEnabled
+      - "false"
       command:
       - launcher-v2
       env:

backend/src/v2/metadata/client.go

@@ -120,7 +120,9 @@ func NewClient(serverAddress, serverPort string, tlsEnabled bool) (*Client, erro
 
 	creds := insecure.NewCredentials()
 	if tlsEnabled {
-		config := &tls.Config{}
+		config := &tls.Config{
+			InsecureSkipVerify: true, // This should be removed by https://issues.redhat.com/browse/RHOAIENG-13871
+		}
 		creds = credentials.NewTLS(config)
 	}