Skip to content
This repository was archived by the owner on Mar 27, 2023. It is now read-only.

Commit 23afcba

Browse files
author
Yi Jiang
committed
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.
1 parent 0c9bf03 commit 23afcba

File tree

2 files changed

+31
-1
lines changed

2 files changed

+31
-1
lines changed

api/v1alpha1/condition_consts.go

Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,3 +106,33 @@ const (
106106
// is not prepared for VMService consumption.
107107
VirtualMachineImageV1Alpha1NotCompatibleReason = "VirtualMachineImageV1Alpha1NotCompatible"
108108
)
109+
110+
// Condition.Reason for Conditions related to VirtualMachinePublishRequest.
111+
const (
112+
// VirtualMachinePublishRequestSourceVMNotFoundReason (Severity=Error) documents that the source VM is not found.
113+
VirtualMachinePublishRequestSourceVMNotFoundReason = "SourceVMNotFound"
114+
115+
// VirtualMachinePublishRequestSourceVMNotCreatedReason (Severity=Error) documents that the source VM
116+
// hasn't been fully created yet.
117+
VirtualMachinePublishRequestSourceVMNotCreatedReason = "SourceVMNotCreated"
118+
119+
// VirtualMachinePublishRequestSourceVMUniqueIDNotReadyReason (Severity=Error) documents that the source VM
120+
// has empty unique ID in its status.
121+
VirtualMachinePublishRequestSourceVMUniqueIDNotReadyReason = "SourceVMUniqueIDNotReady"
122+
123+
// VirtualMachinePublishRequestTargetLocationNotFoundReason (Severity=Error) documents that the target location
124+
// content library is not found.
125+
VirtualMachinePublishRequestTargetLocationNotFoundReason = "TargetContentLibraryNotFound"
126+
127+
// VirtualMachinePublishRequestTargetItemAlreadyExistsReason (Severity=Error) documents that an item with the
128+
// same target name already exists in the content library.
129+
VirtualMachinePublishRequestTargetItemAlreadyExistsReason = "TargetItemAlreadyExists"
130+
131+
// VirtualMachinePublishRequestUploadingReason (Severity=Info) documents that the VM publish work is still in progress.
132+
// The target item is uploading to the content library.
133+
VirtualMachinePublishRequestUploadingReason = "Uploading"
134+
135+
// VirtualMachinePublishRequestUploadFailureReason (Severity=Error) documents that the VM publish work failed.
136+
// The target item failed to be uploaded to the content library.
137+
VirtualMachinePublishRequestUploadFailureReason = "UploadFailure"
138+
)

api/v1alpha1/virtualmachinepublishrequest_types.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -340,7 +340,7 @@ func (vmpr *VirtualMachinePublishRequest) markCondition(
340340
}
341341
if reason == "" && status == corev1.ConditionTrue {
342342
reason = VirtualMachinePublishRequestConditionSuccess
343-
} else {
343+
} else if reason == "" {
344344
reason = string(status)
345345
}
346346

0 commit comments

Comments
 (0)