Eli Mesika has uploaded a new change for review.

Change subject: core: PM options for secondary agent are not saved
......................................................................

core: PM options for secondary agent are not saved

Adding needed code to save the secondary PM options in VDS and VdsStatic
code.

Change-Id: Ic4ff49f7991d06140a8d7d01778b7731b0458736
Changoe-Id: I8df59ff12faa5a710905b6b2188d5cab446d0c85
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1001572
Signed-off-by: Eli Mesika <[email protected]>
---
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
2 files changed, 2 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/19/19119/1

diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
index ad193f9..dd2baee 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDS.java
@@ -160,6 +160,7 @@
         vds.setPmSecondaryIp(getPmSecondaryIp());
         vds.setPmSecondaryType(getPmSecondaryType());
         vds.setPmSecondaryPort(getPmSecondaryPort());
+        vds.setPmSecondaryOptions(getPmSecondaryOptions());
         vds.setPmSecondaryUser(getPmSecondaryUser());
         vds.setPmSecondaryPassword(getPmSecondaryPassword());
         vds.setPmSecondaryConcurrent(isPmSecondaryConcurrent());
diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
index 3c1ed64..7d6e5fd 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VdsStatic.java
@@ -436,6 +436,7 @@
 
     public void setPmSecondaryOptionsMap(HashMap<String, String> value) {
         pmSecondaryOptionsMap = value;
+        pmSecondaryOptions = PmOptionsMapToString(value);
     }
 
     public long getOtpValidity() {


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

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

Reply via email to