Add VirtualMachinePublishRequest reason.

This change includes
- Introduce VirtualMachinePublishRequest reason.
- Update MarkCondition function. It should set the reason to the input parameter if it is not empty.

Author: Yi Jiang

api/v1alpha1/condition_consts.go

@@ -106,3 +106,33 @@ const (
 	// is not prepared for VMService consumption.
 	VirtualMachineImageV1Alpha1NotCompatibleReason = "VirtualMachineImageV1Alpha1NotCompatible"
 )
+
+// Condition.Reason for Conditions related to VirtualMachinePublishRequest.
+const (
+	// VirtualMachinePublishRequestSourceVMNotFoundReason (Severity=Error) documents that the source VM is not found.
+	VirtualMachinePublishRequestSourceVMNotFoundReason = "SourceVMNotFound"
+
+	// VirtualMachinePublishRequestSourceVMNotCreatedReason (Severity=Error) documents that the source VM
+	// hasn't been fully created yet.
+	VirtualMachinePublishRequestSourceVMNotCreatedReason = "SourceVMNotCreated"
+
+	// VirtualMachinePublishRequestSourceVMUniqueIDNotReadyReason (Severity=Error) documents that the source VM
+	// has empty unique ID in its status.
+	VirtualMachinePublishRequestSourceVMUniqueIDNotReadyReason = "SourceVMUniqueIDNotReady"
+
+	// VirtualMachinePublishRequestTargetLocationNotFoundReason (Severity=Error) documents that the target location
+	// content library is not found.
+	VirtualMachinePublishRequestTargetLocationNotFoundReason = "TargetContentLibraryNotFound"
+
+	// VirtualMachinePublishRequestTargetItemAlreadyExistsReason (Severity=Error) documents that an item with the
+	// same target name already exists in the content library.
+	VirtualMachinePublishRequestTargetItemAlreadyExistsReason = "TargetItemAlreadyExists"
+
+	// VirtualMachinePublishRequestUploadingReason (Severity=Info) documents that the VM publish work is still in progress.
+	// The target item is uploading to the content library.
+	VirtualMachinePublishRequestUploadingReason = "Uploading"
+
+	// VirtualMachinePublishRequestUploadFailureReason (Severity=Error) documents that the VM publish work failed.
+	// The target item failed to be uploaded to the content library.
+	VirtualMachinePublishRequestUploadFailureReason = "UploadFailure"
+)

api/v1alpha1/virtualmachinepublishrequest_types.go

@@ -139,13 +139,13 @@ type VirtualMachinePublishRequestTarget struct {
 	// item.
 	//
 	// +optional
-	Item VirtualMachinePublishRequestTargetItem `json:"item,omitempty"`
+	Item *VirtualMachinePublishRequestTargetItem `json:"item,omitempty"`
 
 	// Location contains information about the location to which to publish
 	// the VM.
 	//
 	// +optional
-	Location VirtualMachinePublishRequestTargetLocation `json:"location,omitempty"`
+	Location *VirtualMachinePublishRequestTargetLocation `json:"location,omitempty"`
 }
 
 // VirtualMachinePublishRequestSpec defines the desired state of a
@@ -166,7 +166,7 @@ type VirtualMachinePublishRequestSpec struct {
 	// a resource exists, then it is the source of the publication.
 	//
 	// +optional
-	Source VirtualMachinePublishRequestSource `json:"source,omitempty"`
+	Source *VirtualMachinePublishRequestSource `json:"source,omitempty"`
 
 	// Target is the target of the publication request, ex. item
 	// information and a ContentLibrary resource.
@@ -184,7 +184,7 @@ type VirtualMachinePublishRequestSpec struct {
 	// will be marked in error.
 	//
 	// +optional
-	Target VirtualMachinePublishRequestTarget `json:"target,omitempty"`
+	Target *VirtualMachinePublishRequestTarget `json:"target,omitempty"`
 
 	// TTLSecondsAfterFinished is the time-to-live duration for how long this
 	// resource will be allowed to exist once the publication operation
@@ -355,7 +355,7 @@ func (vmpr *VirtualMachinePublishRequest) markCondition(
 	}
 	if reason == "" && status == corev1.ConditionTrue {
 		reason = VirtualMachinePublishRequestConditionSuccess
-	} else {
+	} else if reason == "" {
 		reason = string(status)
 	}