Alon Bar-Lev has uploaded a new change for review.

Change subject: host-deploy: appropriate message for kdump detection
......................................................................

host-deploy: appropriate message for kdump detection

1. a system that does not support kdump is expected, so it cannot
   be warning.

2. messages with '!' are not polite, user did not do anything to
   justify that.

Change-Id: Ie10ebdbbc0b5148e4cd6e1b2deb251f96a2f56c7
Signed-off-by: Alon Bar-Lev <alo...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/69/29369/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
index 7e76ae2..b356e7a 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
@@ -574,8 +574,8 @@
         new Callable<Boolean>() { public Boolean call() throws Exception {
             if (_vds.isPmKdumpDetection() && !fenceKdumpSupported) {
                 _messages.post(
-                        InstallerMessages.Severity.WARNING,
-                        "Kdump detection is not supported by host!"
+                        InstallerMessages.Severity.INFO,
+                        "Disabling Kdump support"
                 );
             }
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie10ebdbbc0b5148e4cd6e1b2deb251f96a2f56c7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to