parameter and literal map construction wrong

Signed-off-by: Yee Hing Tong <[email protected]>

flyteadmin_config.yaml

@@ -60,6 +60,10 @@ flyteadmin:
     - "metadata"
     - "admin"
   useOffloadedWorkflowClosure: false
+artifacts:
+  host: localhost
+  port: 50051
+  insecure: true
 database:
   postgres:
     port: 30001

pkg/artifacts/artifact_client.go

@@ -0,0 +1,37 @@
+package artifacts
+
+import (
+	"context"
+	"crypto/tls"
+	"github.com/flyteorg/flyteidl/gen/pb-go/flyteidl/artifact"
+	"github.com/flyteorg/flytestdlib/logger"
+	"google.golang.org/grpc"
+	"google.golang.org/grpc/credentials"
+)
+
+func NewArtifactConnection(_ context.Context, cfg *Config, opts ...grpc.DialOption) (*grpc.ClientConn, error) {
+	if opts == nil {
+		// Initialize opts list to the potential number of options we will add. Initialization optimizes memory
+		// allocation.
+		opts = make([]grpc.DialOption, 0, 5)
+	}
+
+	if cfg.Insecure {
+		opts = append(opts, grpc.WithInsecure())
+	} else {
+		tlsConfig := &tls.Config{} //nolint
+		creds := credentials.NewTLS(tlsConfig)
+		opts = append(opts, grpc.WithTransportCredentials(creds))
+	}
+
+	return grpc.Dial(cfg.Host, opts...)
+}
+
+func InitializeArtifactClient(ctx context.Context, cfg *Config, opts ...grpc.DialOption) artifact.ArtifactRegistryClient {
+	conn, err := NewArtifactConnection(ctx, cfg, opts...)
+	if err != nil {
+		logger.Panicf(ctx, "failed to initialize Artifact connection. Err: %s", err.Error())
+		panic(err)
+	}
+	return artifact.NewArtifactRegistryClient(conn)
+}

pkg/artifacts/config.go

@@ -0,0 +1,7 @@
+package artifacts
+
+type Config struct {
+	Host     string `json:"host"`
+	Port     int    `json:"port"`
+	Insecure bool   `json:"insecure"`
+}