Tal Nisan has submitted this change and it was merged.
Change subject: core: 'revert' of 2f471d83d145d7bc6674c6039bf7f8d1ba9ed137
......................................................................
core: 'revert' of 2f471d83d145d7bc6674c6039bf7f8d1ba9ed137
patch 2f471d83d145d7bc6674c6039bf7f8d1ba9ed137 was entirelly wrong.
One must not consult macPool there, since interface here already has
MAC address obtained from pool, so check here always failed.
It probably cannot be reverted nicely, since there were other patches
fixing some of problems this patch introduced.
note: bug
https://bugzilla.redhat.com/show_bug.cgi?id=1212461
related to patch being reverted, was reverified as well and I did not reproduce it.
Change-Id: I0c3b9f9a4cb2f8515823d42293a7928485f4cedf
Bug-Url:
https://bugzilla.redhat.com/1270583
Signed-off-by: Martin Mucha <mmucha(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java
1 file changed, 6 insertions(+), 31 deletions(-)
Approvals:
Martin Mucha: Verified
Alona Kaplan: Looks good to me, approved
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/49154
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0c3b9f9a4cb2f8515823d42293a7928485f4cedf
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Mucha <mmucha(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: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>