Merge pull request #380 from fao89/jobmsg

openshift-merge-bot[bot] · web-flow · commit 0b7bee230fa3 · 2024-06-17T14:27:18.000Z
Fix job status message
diff --git a/controllers/openstack_ansibleee_controller.go b/controllers/openstack_ansibleee_controller.go
@@ -244,7 +244,7 @@ func (r *OpenStackAnsibleEEReconciler) Reconcile(ctx context.Context, req ctrl.R
 	// the sub conditions
 	if instance.Status.Conditions.AllSubConditionIsTrue() {
 		instance.Status.Conditions.MarkTrue(
-			condition.ReadyCondition, condition.ReadyMessage)
+			condition.ReadyCondition, condition.JobReadyMessage)
 	}
 	Log.Info(fmt.Sprintf("Reconciled AnsibleEE '%s' successfully", instance.Name))
 	return ctrl.Result{}, nil

Original file line number	Diff line number	Diff line change
`@@ -244,7 +244,7 @@ func (r *OpenStackAnsibleEEReconciler) Reconcile(ctx context.Context, req ctrl.R`
`244`	`244`	`// the sub conditions`
`245`	`245`	`if instance.Status.Conditions.AllSubConditionIsTrue() {`
`246`	`246`	`instance.Status.Conditions.MarkTrue(`
`247`		`- condition.ReadyCondition, condition.ReadyMessage)`
	`247`	`+ condition.ReadyCondition, condition.JobReadyMessage)`
`248`	`248`	`}`
`249`	`249`	`Log.Info(fmt.Sprintf("Reconciled AnsibleEE '%s' successfully", instance.Name))`
`250`	`250`	`return ctrl.Result{}, nil`