Bug 864346

Summary: [Backend] If VM import fails, allocated MAC is not freed
Product: Red Hat Enterprise Virtualization Manager Reporter: Jakub Libosvar <jlibosva>
Component: ovirt-engineAssignee: lpeer <lpeer>
Status: CLOSED CURRENTRELEASE QA Contact: Martin Pavlik <mpavlik>
Severity: high Docs Contact:
Priority: high    
Version: 3.1.0CC: dyasny, gklein, hateya, iheim, italkohe, lpeer, mavital, michal.skrivanek, mkolesni, Rhev-m-bugs, sgrinber, yeylon, ykaul
Target Milestone: ---Keywords: ZStream
Target Release: 3.2.0   
Hardware: x86_64   
OS: Linux   
Whiteboard: network
Fixed In Version: sf1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 884248 (view as bug list) Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
backend log none

Description Jakub Libosvar 2012-10-09 08:20:41 UTC
Created attachment 623961 [details]
backend log

Description of problem:
MAC for imported VM is allocated when import is about to start. If import fails, MAC is not freed.

Version-Release number of selected component (if applicable):
rhevm-backend-3.1.0-20.el6ev.noarch

How reproducible:
Always

Steps to Reproduce:
1. Start import VM and fail it
  
Actual results:
MAC is allocated but not freed

Expected results:
MAC is allocated and freed on unsuccessful import 

Additional info:
Backend log attached, watch MAC 00:1a:4a:32:3f:00
Not sure if regression, I'd say not

Comment 7 Muli Salem 2012-11-18 13:01:48 UTC
Proposed fix in:

http://gerrit.ovirt.org/#/c/9302/

Comment 10 Martin Pavlik 2013-01-02 15:30:29 UTC
works in sf2.1

2013-01-02 16:28:21,497 INFO  [org.ovirt.engine.core.bll.MacPoolManager] (pool-3-thread-50) MacPoolManager::freeMac(mac = 00:1a:4a:e0:3f:05) - entered

Comment 12 Itamar Heim 2013-06-11 08:40:26 UTC
3.2 has been released

Comment 13 Itamar Heim 2013-06-11 08:40:26 UTC
3.2 has been released

Comment 14 Itamar Heim 2013-06-11 08:40:42 UTC
3.2 has been released

Comment 15 Itamar Heim 2013-06-11 08:48:04 UTC
3.2 has been released