Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

Commit

Permalink
Merge pull request #916 from slagle/log-consistently
Browse files Browse the repository at this point in the history
Use consistent log message when setting Deployment.Status.Deployed
  • Loading branch information
openshift-merge-bot[bot] authored Jun 4, 2024
2 parents 7b11917 + e0a10d3 commit 4476a80
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion controllers/openstackdataplanedeployment_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -384,6 +384,7 @@ func (r *OpenStackDataPlaneDeploymentReconciler) Reconcile(ctx context.Context,

Log.Info("Set DeploymentReadyCondition true")
instance.Status.Conditions.MarkTrue(condition.DeploymentReadyCondition, condition.DeploymentReadyMessage)
Log.Info("Set Status.Deployed to true", "instance", instance)
instance.Status.Deployed = true
if version != nil {
instance.Status.DeployedVersion = version.Spec.TargetVersion
Expand All @@ -392,7 +393,6 @@ func (r *OpenStackDataPlaneDeploymentReconciler) Reconcile(ctx context.Context,
if err != nil {
Log.Error(err, "Error setting service hashes")
}
Log.Info("Set status deploy true", "instance", instance)
return ctrl.Result{}, nil
}

Expand Down

0 comments on commit 4476a80

Please sign in to comment.