Alona Kaplan 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@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; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47899 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c3b9f9a4cb2f8515823d42293a7928485f4cedf Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha@redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra@redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>