Apply suggestions from code review

Co-Authored-By: hardikdr <[email protected]>

pkg/apis/cluster/v1alpha1/machine_types.go

@@ -154,13 +154,12 @@ type MachineStatus struct {
 
 	// LastOperation describes the last-operation performed by the machine-controller.
 	// This API should be useful as a history in terms of the latest operation performed on the
-	// specific machine. It should also convey the state of the latest-operation for example if // it is still on-going, failed or completed successfully.
+	// specific machine. It should also convey the state of the latest-operation for example if
+	// it is still on-going, failed or completed successfully.
 	// +optional
-	LastOperation LastOperation `json:"lastOperation,omitempty"`
+	LastOperation *LastOperation `json:"lastOperation,omitempty"`
 
-	// Phase represents an ongoing-phase a machine is going through. It is expected to be
-	// transient. Higher-level controllers should rely on MachinePhases to take the right
-	// decision such as replacing the Failed machines.
+	// Phase represents the current phase of machine actuation.
 	// Eg. Pending, Running, Terminating, Failed etc.
 	// +optional
 	Phase *string `json:"phase,omitempty"`
@@ -169,7 +168,7 @@ type MachineStatus struct {
 // LastOperation represents the detail of the last performed operation on the MachineObject.
 type LastOperation struct {
 	// Description is the human-readable description of the last operation.
-	Description string `json:"description,omitempty"`
+	Description *string `json:"description,omitempty"`
 
 	// LastUpdateTime is the timestamp at which LastOperation API was last-updated.
 	LastUpdateTime *metav1.Time `json:"lastUpdateTime,omitempty"`