Martin Peřina has uploaded a new change for review. Change subject: core: Fix FencingPolicy copy constructor ......................................................................
core: Fix FencingPolicy copy constructor Properly initializes hostsWithBrokenConnectivityThreshold when instance is created using copy constructor with null parameter. Change-Id: I3b1445e404887704562d9d8bbfdf733b75924f22 Bug-Url: https://bugzilla.redhat.com/1120829 Signed-off-by: Martin Perina <[email protected]> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/FencingPolicy.java 1 file changed, 2 insertions(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/54/31654/1 diff --git a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/FencingPolicy.java b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/FencingPolicy.java index 6813c8c..f8adeb6 100644 --- a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/FencingPolicy.java +++ b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/FencingPolicy.java @@ -20,10 +20,7 @@ private int hostsWithBrokenConnectivityThreshold; public FencingPolicy() { - fencingEnabled = true; - skipFencingIfSDActive = false; - skipFencingIfConnectivityBroken = false; - hostsWithBrokenConnectivityThreshold = 50; + this(null); } public FencingPolicy(FencingPolicy fencingPolicy) { @@ -31,6 +28,7 @@ fencingEnabled = true; skipFencingIfSDActive = false; skipFencingIfConnectivityBroken = false; + hostsWithBrokenConnectivityThreshold = 50; } else { fencingEnabled = fencingPolicy.fencingEnabled; skipFencingIfSDActive = fencingPolicy.skipFencingIfSDActive; -- To view, visit http://gerrit.ovirt.org/31654 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I3b1445e404887704562d9d8bbfdf733b75924f22 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Martin Peřina <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
