Fix Artifact type to a pointer

Prior to this, the Artifacts in TaskRunStatusFields was not set to
a pointer of the underlying struct. As a result, it was incompatible
with `tkn`. Here we fix that issue by updating it to a `pointer`.

(cherry picked from commit c910594)
Signed-off-by: Vincent Demeester <vdemeest@redhat.com>

pkg/apis/pipeline/v1/artifact_types.go

@@ -52,29 +52,30 @@ type Artifacts struct {
 	Outputs []Artifact `json:"outputs,omitempty"`
 }
 
-func (a *Artifacts) Merge(another Artifacts) {
+func (a *Artifacts) Merge(another *Artifacts) {
 	inputMap := make(map[string][]ArtifactValue)
 	var newInputs []Artifact
 
 	for _, v := range a.Inputs {
 		inputMap[v.Name] = v.Values
 	}
-
-	for _, v := range another.Inputs {
-		_, ok := inputMap[v.Name]
-		if !ok {
-			inputMap[v.Name] = []ArtifactValue{}
-		}
-		for _, vv := range v.Values {
-			exists := false
-			for _, av := range inputMap[v.Name] {
-				if cmp.Equal(vv, av) {
-					exists = true
-					break
-				}
+	if another != nil {
+		for _, v := range another.Inputs {
+			_, ok := inputMap[v.Name]
+			if !ok {
+				inputMap[v.Name] = []ArtifactValue{}
 			}
-			if !exists {
-				inputMap[v.Name] = append(inputMap[v.Name], vv)
+			for _, vv := range v.Values {
+				exists := false
+				for _, av := range inputMap[v.Name] {
+					if cmp.Equal(vv, av) {
+						exists = true
+						break
+					}
+				}
+				if !exists {
+					inputMap[v.Name] = append(inputMap[v.Name], vv)
+				}
 			}
 		}
 	}
@@ -92,31 +93,33 @@ func (a *Artifacts) Merge(another Artifacts) {
 		outputMap[v.Name] = v
 	}
 
-	for _, v := range another.Outputs {
-		_, ok := outputMap[v.Name]
-		if !ok {
-			outputMap[v.Name] = Artifact{Name: v.Name, Values: []ArtifactValue{}, BuildOutput: v.BuildOutput}
-		}
-		// only update buildOutput to true.
-		// Do not convert to false if it was true before.
-		if v.BuildOutput {
-			art := outputMap[v.Name]
-			art.BuildOutput = v.BuildOutput
-			outputMap[v.Name] = art
-		}
-		for _, vv := range v.Values {
-			exists := false
-			for _, av := range outputMap[v.Name].Values {
-				if cmp.Equal(vv, av) {
-					exists = true
-					break
-				}
+	if another != nil {
+		for _, v := range another.Outputs {
+			_, ok := outputMap[v.Name]
+			if !ok {
+				outputMap[v.Name] = Artifact{Name: v.Name, Values: []ArtifactValue{}, BuildOutput: v.BuildOutput}
 			}
-			if !exists {
+			// only update buildOutput to true.
+			// Do not convert to false if it was true before.
+			if v.BuildOutput {
 				art := outputMap[v.Name]
-				art.Values = append(art.Values, vv)
+				art.BuildOutput = v.BuildOutput
 				outputMap[v.Name] = art
 			}
+			for _, vv := range v.Values {
+				exists := false
+				for _, av := range outputMap[v.Name].Values {
+					if cmp.Equal(vv, av) {
+						exists = true
+						break
+					}
+				}
+				if !exists {
+					art := outputMap[v.Name]
+					art.Values = append(art.Values, vv)
+					outputMap[v.Name] = art
+				}
+			}
 		}
 	}
 

pkg/apis/pipeline/v1/artifact_types_test.go

@@ -182,7 +182,7 @@ func TestArtifactsMerge(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			tc.a1.Merge(tc.a2)
+			tc.a1.Merge(&tc.a2)
 			got := tc.a1
 			if d := cmp.Diff(tc.expected, got, cmpopts.SortSlices(func(a, b Artifact) bool { return a.Name > b.Name })); d != "" {
 				t.Errorf("TestArtifactsMerge() did not produce expected artifacts for test %s: %s", tc.name, diff.PrintWantGot(d))

pkg/apis/pipeline/v1/swagger.json

@@ -2125,7 +2125,6 @@
         },
         "artifacts": {
           "description": "Artifacts are the list of artifacts written out by the task's containers",
-          "default": {},
           "$ref": "#/definitions/v1.Artifacts",
           "x-kubernetes-list-type": "atomic"
         },
@@ -2220,7 +2219,6 @@
       "properties": {
         "artifacts": {
           "description": "Artifacts are the list of artifacts written out by the task's containers",
-          "default": {},
           "$ref": "#/definitions/v1.Artifacts",
           "x-kubernetes-list-type": "atomic"
         },

pkg/apis/pipeline/v1/taskrun_types.go

@@ -282,7 +282,7 @@ type TaskRunStatusFields struct {
 	// Artifacts are the list of artifacts written out by the task's containers
 	// +optional
 	// +listType=atomic
-	Artifacts Artifacts `json:"artifacts,omitempty"`
+	Artifacts *Artifacts `json:"artifacts,omitempty"`
 
 	// The list has one entry per sidecar in the manifest. Each entry is
 	// represents the imageid of the corresponding sidecar.

pkg/pod/status.go

@@ -260,7 +260,7 @@ func setTaskRunStatusBasedOnStepStatus(ctx context.Context, logger *zap.SugaredL
 			logger.Errorf("Failed to set artifacts value from sidecar logs: %v", err)
 			merr = multierror.Append(merr, err)
 		} else {
-			trs.Artifacts = tras
+			trs.Artifacts = &tras
 		}
 	}
 
@@ -343,8 +343,8 @@ func setTaskRunStatusBasedOnStepStatus(ctx context.Context, logger *zap.SugaredL
 						logger.Errorf("error setting step artifacts in taskrun %q: %v", tr.Name, err)
 						merr = multierror.Append(merr, err)
 					}
-					trs.Artifacts.Merge(tras)
-					trs.Artifacts.Merge(sas)
+					trs.Artifacts.Merge(&tras)
+					trs.Artifacts.Merge(&sas)
 				}
 				msg, err = createMessageFromResults(filteredResults)
 				if err != nil {