Greg Padgett has uploaded a new change for review.

Change subject: engine: fix vmJob logging
......................................................................

engine: fix vmJob logging

Make logging more clear and less spammy for vmJobs updates made in
VdsUpdateRunTimeInfo.

Change-Id: I0f5a9851c347366413496fb6e6375e7d44d1a30b
Bug-Url: https://bugzilla.redhat.com/1110394
Signed-off-by: Greg Padgett <gpadg...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/06/28906/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
index b76e156..26b6907 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsUpdateRunTimeInfo.java
@@ -1076,7 +1076,7 @@
 
             if (vmInternalData.getValue().getVmStatistics().getVmJobs() == 
null) {
                 // If no vmJobs key was returned, we can't presume anything 
about the jobs; save them all
-                log.info("No vmJob data returned, preserving existing jobs");
+                log.debug("No vmJob data returned from VDSM, preserving 
existing jobs");
                 continue;
             }
 
@@ -1086,10 +1086,10 @@
                         // Same data, no update needed.  It would be nice if a 
caching
                         // layer would take care of this for us.
                         vmJobIdsToIgnore.add(jobFromVds.getId());
-                        log.infoFormat("VM job {0}: Preserving (no change)", 
jobFromVds.getId());
+                        log.infoFormat("VM job {0}: In progress (no change)", 
jobFromVds.getId());
                     } else {
                         vmJobsToUpdate.put(jobFromVds.getId(), jobFromVds);
-                        log.infoFormat("VM job {0}: Updating", 
jobFromVds.getId());
+                        log.infoFormat("VM job {0}: In progress, updating", 
jobFromVds.getId());
                     }
                 }
             }


-- 
To view, visit http://gerrit.ovirt.org/28906
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0f5a9851c347366413496fb6e6375e7d44d1a30b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadg...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to