Eli Mesika has submitted this change and it was merged. Change subject: core: fix PM settings in new host ......................................................................
core: fix PM settings in new host This patch fixes the following issue: 1) Add a new host to cluster, PM tab has these settings: Enable PM: true Kdump integration: false Type: ipmilan UserName, Password: filled in 2) Click to OK, host is deployed with success 3) Check the DB, that host doesn't have any records in fence_agents table 4) Try to Edit the host, PM tab contains these settings: Enable PM: true Kdump integration: false Type: apc UserName, Password: empty 5) Fill in data again and click OK 6) fence_agents table contains record for the host Change-Id: I13ebea573105b7e8f6f8e7a33251e40ec11c165b Signed-off-by: emesika <emes...@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVdsCommand.java 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Eli Mesika: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/36584 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I13ebea573105b7e8f6f8e7a33251e40ec11c165b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Eli Mesika <emes...@redhat.com> Gerrit-Reviewer: Martin Peřina <mper...@redhat.com> Gerrit-Reviewer: Ori Liel <ol...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches