[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-20 Thread msalem
Muli Salem has submitted this change and it was merged. Change subject: engine: Release Mac from Pool .. engine: Release Mac from Pool This patch makes sure the Mac address is released from pool in case the Update/Add VmInterfa

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-20 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10697 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-17 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 9: Looks good to me, but someone else must approve +1 for frontend -- To view, visit http://gerrit.ovirt.org/10697 To

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-16 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/10697 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-16 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 8: After reviewing the error issue, it seems that VdsmErrors is the right place to put the error messages. -- To view,

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-13 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 8: I don't think the vdsm errors properties file is the correct place for the errors. Please move them to AppErrors.pro

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-13 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 6: (3 inline comments) File backend/manager/modules/bll/src/main/ja

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-10 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 6: (3 inline comments) Basically the changes address the main issue. See few comments inside. .

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-10 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 5: (2 inline comments) File backend/manager/modules/bll/src/main/ja

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-10 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 5: (2 inline comments) File backend/manager/modules/bll/src/main/j

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-09 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 4: (1 inline comment) File backend/manager/modules/bll/src/main/jav

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-08 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 4: (1 inline comment) What about the mac address committed to the pool in the can-do-action? How would it be released?

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-07 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 1: (1 inline comment) File backend/manager/modules/bll/src/main/jav

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-06 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Release Mac from Pool .. Patch Set 1: (1 inline comment) File backend/manager/modules/bll/src/main/ja

[Engine-patches] Change in ovirt-engine[master]: engine: Release Mac from Pool

2013-01-06 Thread msalem
Muli Salem has uploaded a new change for review. Change subject: engine: Release Mac from Pool .. engine: Release Mac from Pool This patch makes sure the Mac address is released from pool in case the Update/Add VmInterfaceComman