Make results in Run status optional

If it's optional in the spec, it should probably be optional in the
Status, since the behaviour would probably be not to upload the logs
anywhere if no endpoint is provided.

Possibly in the future these could become mandator, depending how we
design the results store interface.

pkg/apis/pipeline/v1alpha1/pipelinerun_types.go

@@ -112,7 +112,8 @@ type PipelineTriggerRef struct {
 type PipelineRunStatus struct {
 	Conditions duckv1alpha1.Conditions `json:"conditions"`
 	// In #107 should be updated to hold the location logs have been uploaded to
-	Results Results `json:"results"`
+	// +optional
+	Results *Results `json:"results,omitempty"`
 	// map of TaskRun Status with the taskRun name as the key
 	//+optional
 	TaskRuns map[string]TaskRunStatus `json:"taskRuns,omitempty"`

pkg/apis/pipeline/v1alpha1/taskrun_types.go

@@ -105,7 +105,8 @@ type TaskRunStatus struct {
 	Conditions duckv1alpha1.Conditions `json:"conditions,omitempty"`
 
 	// In #107 should be updated to hold the location logs have been uploaded to
-	Results Results `json:"results"`
+	// +optional
+	Results *Results `json:"results,omitempty"`
 
 	// PodName is the name of the pod responsible for executing this task's steps.
 	PodName string `json:"podName"`