Merge branch 'master' into lazy_load

pkg/compiler/transformers/k8s/utils.go

@@ -86,8 +86,10 @@ func StripTypeMetadata(t *core.LiteralType) *core.LiteralType {
 
 	c := *t
 	c.Metadata = nil
-	c.Structure = nil
 	c.Annotation = nil
+	// Note that we cannot strip `Structure` from the type because the dynamic node output type is used to validate the
+	// interface of the dynamically compiled workflow. `Structure` is used to extend type checking information on
+	// differnent Flyte types and is therefore required to ensure correct type validation.
 
 	switch underlyingType := c.Type.(type) {
 	case *core.LiteralType_UnionType:

pkg/compiler/transformers/k8s/utils_test.go

@@ -240,6 +240,7 @@ func TestStripTypeMetadata(t *testing.T) {
 									Type: &core.LiteralType_Simple{
 										Simple: core.SimpleType_STRING,
 									},
+									Structure: &core.TypeStructure{Tag: "str"},
 								},
 							},
 						},