Skip to content

Commit 85b3853

Browse files
ELENAGERraghavendra-talur
authored andcommitted
Shorten too long log messages
Signed-off-by: Elena Gershkovich <[email protected]>
1 parent afd4271 commit 85b3853

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

controllers/drplacementcontrol.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -1712,7 +1712,7 @@ func (d *DRPCInstance) EnsureCleanup(clusterToSkip string) error {
17121712
return nil
17131713
}
17141714

1715-
d.log.Info(fmt.Sprintf("PeerReady Condition %v", condition))
1715+
d.log.Info(fmt.Sprintf("PeerReady Condition is %s, msg: %s", condition.Status, condition.Message))
17161716

17171717
// IFF we have VolSync PVCs, then no need to clean up
17181718
homeCluster := clusterToSkip
@@ -2012,7 +2012,7 @@ func (d *DRPCInstance) ensureVRGDeleted(clusterName string) bool {
20122012
return false
20132013
}
20142014

2015-
d.log.Info(fmt.Sprintf("VRG not deleted(%v)", vrg.ObjectMeta))
2015+
d.log.Info(fmt.Sprintf("VRG not deleted(%s)", vrg.Name))
20162016

20172017
return false
20182018
}

controllers/drplacementcontrol_controller.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -1719,7 +1719,7 @@ func (r *DRPlacementControlReconciler) updateResourceCondition(
17191719
vrg, err := r.MCVGetter.GetVRGFromManagedCluster(drpc.Name, vrgNamespace,
17201720
clusterName, annotations)
17211721
if err != nil {
1722-
log.Info("Failed to get VRG from managed cluster", "errMsg", err)
1722+
log.Info("Failed to get VRG from managed cluster", "errMsg", err.Error())
17231723

17241724
drpc.Status.ResourceConditions = rmn.VRGConditions{}
17251725
} else {
@@ -1967,7 +1967,7 @@ func (r *DRPlacementControlReconciler) createOrUpdatePlacementDecision(ctx conte
19671967
return fmt.Errorf("failed to update placementDecision status (%w)", err)
19681968
}
19691969

1970-
r.Log.Info("Created/Updated PlacementDecision", "PlacementDecision", plDecision)
1970+
r.Log.Info("Created/Updated PlacementDecision", "PlacementDecision", plDecision.Status.Decisions)
19711971

19721972
return nil
19731973
}

0 commit comments

Comments
 (0)