Change in ovirt-engine[ovirt-engine-3.6]: core: 'revert' of 2f471d83d145d7bc6674c6039bf7f8d1ba9ed137

tnisan at redhat.com tnisan at redhat.com
Thu Nov 26 09:48:17 UTC 2015


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 at 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 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: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list