Omer Frenkel has uploaded a new change for review.

Change subject: core: add marks for key-value in sysprep handler log (#870100)
......................................................................

core: add marks for key-value in sysprep handler log (#870100)

https://bugzilla.redhat.com/870100

Added marks ('') for key and value in the log,
to make the message more clear.
also changed level to warn as flow continues with defaults.

Change-Id: I3b0e4076e983a0d2bac60c4eaade5c84284c44d0
Signed-off-by: Omer Frenkel <ofren...@redhat.com>
---
M 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/90/8890/1

diff --git 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
index b6873f1..01012e6 100644
--- 
a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
+++ 
b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/SysprepHandler.java
@@ -163,7 +163,7 @@
     private static String useDefaultIfNull(String key, String value, String 
defaultValue,
             boolean printDefaultValue) {
         if (value == null && printDefaultValue) {
-            log.errorFormat("Could not find value for {0}. Going to use 
default value of: {1}",
+            log.warnFormat("Could not find value for key '{0}'. Going to use 
default value of: '{1}'",
                     key, defaultValue);
         }
         return value != null ? value : defaultValue;


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

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

Reply via email to