Merge pull request #3933 from zedongh/chore-preempt-victims-log-with-…

…details chore: use Infof for slice taskinfo instead of InfoS log less details
volcano-sh · Dec 28, 2024 · 8b08e85 · 8b08e85
2 parents 8962bf8 + 06d01ba
commit 8b08e85
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/scheduler/plugins/capacity/capacity.go b/pkg/scheduler/plugins/capacity/capacity.go
@@ -138,7 +138,7 @@ func (cp *capacityPlugin) OnSessionOpen(ssn *framework.Session) {
 			allocated.Sub(reclaimee.Resreq)
 			victims = append(victims, reclaimee)
 		}
-		klog.V(4).InfoS("Victims from capacity plugin", "victims", victims, "reclaimer", reclaimer)
+		klog.V(4).Infof("Victims from capacity plugin, victims=%+v reclaimer=%s", victims, reclaimer)
 		return victims, util.Permit
 	})
 

diff --git a/pkg/scheduler/plugins/gang/gang.go b/pkg/scheduler/plugins/gang/gang.go
@@ -99,7 +99,7 @@ func (gp *gangPlugin) OnSessionOpen(ssn *framework.Session) {
 			}
 		}
 
-		klog.V(4).InfoS("Victims from Gang plugins", "victims", victims, "preemptor", preemptor)
+		klog.V(4).Infof("Victims from Gang plugins, victims=%+v preemptor=%s", victims, preemptor)
 
 		return victims, util.Permit
 	}
-Original file line number
+Diff line change
@@ Expand Up / @@ -99,7 +99,7 @@ func (gp *gangPlugin) OnSessionOpen(ssn *framework.Session) { @@
     			}
     		}
-    		klog.V(4).InfoS("Victims from Gang plugins", "victims", victims, "preemptor", preemptor)
+    		klog.V(4).Infof("Victims from Gang plugins, victims=%+v preemptor=%s", victims, preemptor)
     		return victims, util.Permit
     	}
@@ Expand Down @@