Change in ovirt-engine[ovirt-engine-3.6.3]: core: custom parameters couldn't be updated and wasnt reported

oourfali at redhat.com oourfali at redhat.com
Tue Feb 16 11:59:39 UTC 2016


Oved Ourfali has submitted this change and it was merged.

Change subject: core: custom parameters couldn't be updated and wasnt reported
......................................................................


core: custom parameters couldn't be updated and wasnt reported

due to bug in removing custom parameters from VdsNetworkInterface user
was unable to update custom parameters using old SetupNetworksCommand,
and those custom properties wasn't reported on REST as well.

Change-Id: I4d8a9e9346997e05e5f90b7e80abe2ba924b74e0
Bug-Url: https://bugzilla.redhat.com/1304787
Bug-Url: https://bugzilla.redhat.com/1308336
Bug-Url: https://bugzilla.redhat.com/1304784
Signed-off-by: Martin Mucha <mmucha at redhat.com>
Signed-off-by: Alona Kaplan <alkaplan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ChangeVDSClusterCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/cluster/AttachNetworkToClusterInternalCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksCommand.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/CustomPropertiesForVdsNetworkInterface.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/UserConfiguredNetworkData.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/HostNetworkAttachmentsPersister.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/HostNetworkTopologyPersisterImpl.java
M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/vdsbroker/HostNetworkAttachmentsPersisterTest.java
8 files changed, 292 insertions(+), 49 deletions(-)

Approvals:
  Martin Mucha: Verified
  Alona Kaplan: Looks good to me, approved
  Oved Ourfali: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/53575
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d8a9e9346997e05e5f90b7e80abe2ba924b74e0
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6.3
Gerrit-Owner: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list