Eli Mesika has uploaded a new change for review.

Change subject: core: handle PM proxy failure correctly
......................................................................

core: handle PM proxy failure correctly

The fenceSucceeded is set to true and remain as that if the proxy
selection is failed, causing mismatch of fencing operation success/failure 
reporting as described in the bug

Change-Id: I554edd6451d0a7c913ad610528a1153e6641e3ba
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1184768
Signed-off-by: emesika <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/48/37648/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
index a22d342..892f5ba 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/InitVdsOnUpCommand.java
@@ -77,7 +77,7 @@
  */
 @NonTransactiveCommandAttribute
 public class InitVdsOnUpCommand extends 
StorageHandlingCommandBase<HostStoragePoolParametersBase> {
-    private boolean fenceSucceeded = true;
+    private boolean fenceSucceeded = false;
     private boolean vdsProxyFound;
     private List<StorageDomainStatic> problematicDomains;
     private boolean connectPoolSucceeded;


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I554edd6451d0a7c913ad610528a1153e6641e3ba
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Eli Mesika <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to