Change in ovirt-engine[master]: engine: HostsetupNetworksCommand- small bug fixes
Alona Kaplan has submitted this change and it was merged.
Change subject: engine: HostsetupNetworksCommand- small bug fixes
......................................................................
engine: HostsetupNetworksCommand- small bug fixes
1. The attachment was always compared to the base nic.
2. completion of the networkId and network name of the exisiting
attahcments.
3. Fix sync error message.
4. MapNetworkAttachments mapped between nicName to single attachment.
It is wrong since single base nic can have multiple attachments.
Change-Id: I5c249dda94b572291bf0428e279ae8bc61549ba0
Signed-off-by: Alona Kaplan <alkaplan(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksCommand.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidator.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/MapNetworkAttachments.java
3 files changed, 30 insertions(+), 16 deletions(-)
Approvals:
Alona Kaplan: Verified; Passed CI tests
Moti Asayag: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/43722
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I5c249dda94b572291bf0428e279ae8bc61549ba0
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org