[Engine-patches] Change in ovirt-engine[master]: core, webadmin: [WIP] Unifying Move and LiveMigrate

2012-10-11 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: core,webadmin: [WIP] Unifying Move and LiveMigrate .. core,webadmin: [WIP] Unifying Move and LiveMigrate Provide a simpler API for moving disks. Created a sing

[Engine-patches] Change in ovirt-engine[master]: core: Query GetVdsGroupsByNetworkId (WIP)

2012-10-11 Thread msalem
Muli Salem has posted comments on this change. Change subject: core: Query GetVdsGroupsByNetworkId (WIP) .. Patch Set 1: (3 inline comments) File backend/manager/dbscripts/vd

[Engine-patches] Change in ovirt-engine[master]: engine: Refresh gluster data periodically

2012-10-11 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 20: Here we go - removed the test-time dependency on dal. Testing the functionality now with loads of da

[Engine-patches] Change in ovirt-engine[master]: core: [WIP]: Live Storage Migration commands

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: [WIP]: Live Storage Migration commands .. Patch Set 17: (2 inline comments) Commit Message Line 14: 2

[Engine-patches] Change in ovirt-engine[master]: core: [WIP]: Live Storage Migration commands

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: [WIP]: Live Storage Migration commands .. Patch Set 17: Changes since last patchset: 1. Updated to newer VDSM API - removed the replicateExtend concept 2.

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] Adding VDSM API for LiveStorageMigration

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: [WIP] Adding VDSM API for LiveStorageMigration .. Patch Set 12: Updated to newer VDSM API - removed the replicateExtend concept -- To view, visit http://

[Engine-patches] Change in ovirt-engine[master]: core: Introducing SyncImageGroupVDSCommand

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Introducing SyncImageGroupVDSCommand .. Patch Set 13: Updated to newer VDSM API - removed the replicateExtend concept -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 13: Liron/Yair - will address, don't worry. Also, need to encapsulate the API between commands and han

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread mpastern
Michael Pasternak has submitted this change and it was merged. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. restapi: user cannnot take a vm from pool via restapi #864438 This patch implements

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread mpastern
Michael Pasternak has submitted this change and it was merged. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. restapi: Allow copying template disks to another storage domain #851099

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: restapi: add disk.name property for disk.add() to ease < 3.1...

2012-10-11 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: restapi: add disk.name property for disk.add() to ease < 3.1 clients migration .. restapi: add disk.name property for disk.add() to ease < 3.1 clients migra

[Engine-patches] Change in ovirt-engine[master]: engine: Removing unneeded read query from SetVdsStatusVDSCom...

2012-10-11 Thread abaron
Ayal Baron has posted comments on this change. Change subject: engine: Removing unneeded read query from SetVdsStatusVDSCommand .. Patch Set 1: (3 inline comments) File back

[Engine-patches] Change in ovirt-engine[master]: engine: Removing all unneeded staff from VdsIdVDSCommandBase

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Removing all unneeded staff from VdsIdVDSCommandBase .. Patch Set 1: Looks good to me, approved I'm quite surprised that such a code even existed. -- T

[Engine-patches] Change in ovirt-engine[master]: engine:cannot start stateless VM

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: engine:cannot start stateless VM .. Patch Set 2: (1 inline comment) File backend/manager/modules/bll/src/m

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 1: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: Engine: Rhev-H upgrade list show downgraded versions (#84276...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: Engine: Rhev-H upgrade list show downgraded versions (#842767) .. Patch Set 7: Verified Please note the commit message, we added a new file for the compatibility

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 1: (1 inline comment) .

[Engine-patches] Change in ovirt-engine[master]: engine:cannot start stateless VM

2012-10-11 Thread rgolan
Roy Golan has posted comments on this change. Change subject: engine:cannot start stateless VM .. Patch Set 2: (1 inline comment) File backend/manager/modules/bll/src/main/j

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 2: (1 inline comment) .

[Engine-patches] Change in ovirt-engine[master]: core: Query GetVdsByNetworkId (WIP)

2012-10-11 Thread msalem
Muli Salem has uploaded a new change for review. Change subject: core: Query GetVdsByNetworkId (WIP) .. core: Query GetVdsByNetworkId (WIP) This patch adds the query above as part of the Network Main Tab feature. Change-Id: If5

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 1: (3 inline comments) File

[Engine-patches] Change in ovirt-guest-agent[master]: replaced agent license to apache software license (asl) 2.0.

2012-10-11 Thread ghammer
Gal Hammer has posted comments on this change. Change subject: replaced agent license to apache software license (asl) 2.0. .. Patch Set 1: (1 inline comment) File ovirt-gues

[Engine-patches] Change in ovirt-engine[master]: core: avoid having SD active on vdsm and not in engine

2012-10-11 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: avoid having SD active on vdsm and not in engine .. Patch Set 2: (2 inline comments) File backend/mana

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. Patch Set 2: (1 inline comment) File b

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove domain column from template main grid (#837...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: remove domain column from template main grid (#837025) .. Patch Set 5: Verified -- To view, visit http://gerrit.ovirt.org/8229 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: webadmin: rename Allocation to Space Allocation in new VM di...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: rename Allocation to Space Allocation in new VM dialog (#837025) .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/8228 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. Patch Set 2: (1 inline comment) File backend/ma

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. Patch Set 2: (2 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: engine: Moved generic methods from GlusterCoreUtil

2012-10-11 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: engine: Moved generic methods from GlusterCoreUtil .. engine: Moved generic methods from GlusterCoreUtil Generic methods like objectsEqual, listsEquel

[Engine-patches] Change in ovirt-engine[master]: engine: Moved generic methods from GlusterCoreUtil

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine: Moved generic methods from GlusterCoreUtil .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8434 To unsubs

[Engine-patches] Change in ovirt-engine[master]: engine: Removing all unneeded staff from VdsIdVDSCommandBase

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine: Removing all unneeded staff from VdsIdVDSCommandBase .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8496 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: engine: Removing all unneeded staff from VdsIdVDSCommandBase

2012-10-11 Thread mkublin
Michael Kublin has uploaded a new change for review. Change subject: engine: Removing all unneeded staff from VdsIdVDSCommandBase .. engine: Removing all unneeded staff from VdsIdVDSCommandBase 1. Removed garbage comments 2. Rem

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. Patch Set 2: (2 inline comments) File backend/m

[Engine-patches] Change in ovirt-guest-agent[master]: replaced agent license to apache software license (asl) 2.0.

2012-10-11 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: replaced agent license to apache software license (asl) 2.0. .. Patch Set 1: (1 inline comment) File ovirt-gu

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 2: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: engine: Clean up of VDS class

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine: Clean up of VDS class .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8494 To unsubscribe, visit http://gerrit.ovirt.org/settings Ge

[Engine-patches] Change in ovirt-engine[master]: engine: Clean up of VDS class

2012-10-11 Thread mkublin
Michael Kublin has uploaded a new change for review. Change subject: engine: Clean up of VDS class .. engine: Clean up of VDS class Removed unneeded staff Change-Id: I9613bbd2e5000cb507839fd313bfdfb6087835ec Signed-off-by: Mich

[Engine-patches] Change in ovirt-engine[master]: engine: Passwordless ssh status query

2012-10-11 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: Passwordless ssh status query .. Patch Set 1: No I don't know any. But I don't understand when you throw SshConnectionFailed for example... I see you de

[Engine-patches] Change in ovirt-engine[master]: engine: Passwordless ssh status query

2012-10-11 Thread sesubram
Selvasundaram has posted comments on this change. Change subject: engine: Passwordless ssh status query .. Patch Set 1: Following use case requires the passwordless ssh setup status, Gluster volume 'Geo-replication' requires p

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 1: (4 inline comments)

[Engine-patches] Change in ovirt-engine[master]: engine: Get Volume Advanced Details Query

2012-10-11 Thread dgopal
Dhandapani Gopal has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 15: Verified -- To view, visit http://gerrit.ovirt.org/7807 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: engine: Get Volume Advanced Details Query

2012-10-11 Thread dgopal
Dhandapani Gopal has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 14: (2 inline comments) Incorporated review comments.

[Engine-patches] Change in ovirt-engine[master]: engine: Passwordless ssh status query

2012-10-11 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: Passwordless ssh status query .. Patch Set 1: Not that I have something to contribute... I don't see the actual ssh usage, so I guess this is code reuse.

[Engine-patches] Change in ovirt-engine[master]: engine:cannot start stateless VM

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: engine:cannot start stateless VM .. Patch Set 2: (1 inline comment) see inline File backend/manager/modul

[Engine-patches] Change in ovirt-engine[master]: restapi: Allow copying template disks to another storage dom...

2012-10-11 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: Allow copying template disks to another storage domain #851099 .. Patch Set 1: (4 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: core: Add support for iLO2 and iLO4 as...(#837907)

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Add support for iLO2 and iLO4 as...(#837907) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8492 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: restapi: Don't raise 404 if slave not found

2012-10-11 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: restapi: Don't raise 404 if slave not found .. restapi: Don't raise 404 if slave not found If slave wasn't found, then just map without template, otherw

[Engine-patches] Change in ovirt-engine[master]: restapi: Don't raise 404 if slave not found

2012-10-11 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: restapi: Don't raise 404 if slave not found .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8490 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: restapi: If slave id is not set, lookup by name

2012-10-11 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: restapi: If slave id is not set, lookup by name .. restapi: If slave id is not set, lookup by name If slave ID for bonding is not set, then need to look

[Engine-patches] Change in ovirt-engine[master]: restapi: If slave id is not set, lookup by name

2012-10-11 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: restapi: If slave id is not set, lookup by name .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8489 To unsubscribe, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: core: fix module name in fencing logs(#652581)

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: fix module name in fencing logs(#652581) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8491 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: restapi: Don't raise 404 if slave not found

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Don't raise 404 if slave not found .. Patch Set 1: Looks good to me, approved agreed with mkolesni, that BE will validate slaves existence. -- To v

[Engine-patches] Change in ovirt-engine[master]: engine: Removing unneeded read query from SetVdsStatusVDSCom...

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine: Removing unneeded read query from SetVdsStatusVDSCommand .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8493 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: engine: Removing unneeded read query from SetVdsStatusVDSCom...

2012-10-11 Thread mkublin
Michael Kublin has uploaded a new change for review. Change subject: engine: Removing unneeded read query from SetVdsStatusVDSCommand .. engine: Removing unneeded read query from SetVdsStatusVDSCommand The following query was us

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove domain column from template main grid (#837...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: remove domain column from template main grid (#837025) .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/8229 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: webadmin: rename Allocation to Space Allocation in new VM di...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: rename Allocation to Space Allocation in new VM dialog (#837025) .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/8228 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: webadmin: rename Allocation to Space Allocation in new VM di...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: rename Allocation to Space Allocation in new VM dialog (#837025) .. Patch Set 2: (2 inline comments) .

[Engine-patches] Change in ovirt-engine[master]: core: Add support for iLO2 and iLO4 as...(#837907)

2012-10-11 Thread emesika
Eli Mesika has uploaded a new change for review. Change subject: core: Add support for iLO2 and iLO4 as...(#837907) .. core: Add support for iLO2 and iLO4 as...(#837907) Add support for iLO2 and iLO4 as a fencing options(#837907

[Engine-patches] Change in ovirt-engine[master]: [RFE] Integrate smartcard support

2012-10-11 Thread rgolan
Roy Golan has posted comments on this change. Change subject: [RFE] Integrate smartcard support .. Patch Set 3: (1 inline comment) what about import / export flows? File ba

[Engine-patches] Change in ovirt-engine[master]: restapi: If slave id is not set, lookup by name

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: If slave id is not set, lookup by name .. Patch Set 1: Looks good to me, approved (1 inline comment) minor comment ...

[Engine-patches] Change in ovirt-engine[master]: engine:cannot start stateless VM

2012-10-11 Thread rgolan
Roy Golan has posted comments on this change. Change subject: engine:cannot start stateless VM .. Patch Set 2: Allon - please have a look here and see this sits with your last refactoring Frank - can you please verify by runni

[Engine-patches] Change in ovirt-engine[master]: restapi: Don't raise 404 if slave not found

2012-10-11 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Don't raise 404 if slave not found .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: core: fix module name in fencing logs(#652581)

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: fix module name in fencing logs(#652581) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8491 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: fix module name in fencing logs(#652581)

2012-10-11 Thread emesika
Eli Mesika has uploaded a new change for review. Change subject: core: fix module name in fencing logs(#652581) .. core: fix module name in fencing logs(#652581) This patch changes the term "Fencing" to "Fence" in all log keys &

[Engine-patches] Change in ovirt-engine[master]: core: Removing reschedule of monitoring of vdsm by another v...

2012-10-11 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: Removing reschedule of monitoring of vdsm by another vdsm .. Patch Set 1: good catch. but we may still want to preserve that behaviour in case the interval of

[Engine-patches] Change in ovirt-engine[master]: engine: Execution messages for Gluster Volume Actions (#8561...

2012-10-11 Thread emesika
Eli Mesika has posted comments on this change. Change subject: engine: Execution messages for Gluster Volume Actions (#856102) .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7967 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: Don't add the cdrom device if iso_path is empty

2012-10-11 Thread emesika
Eli Mesika has posted comments on this change. Change subject: Don't add the cdrom device if iso_path is empty .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8092 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: core: Removing reschedule of monitoring of vdsm by another v...

2012-10-11 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: Removing reschedule of monitoring of vdsm by another vdsm .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: restapi: Don't raise 404 if slave not found

2012-10-11 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: restapi: Don't raise 404 if slave not found .. restapi: Don't raise 404 if slave not found If slave wasn't found, then just map without template, otherwise the

[Engine-patches] Change in ovirt-engine[master]: restapi: If slave id is not set, lookup by name

2012-10-11 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: restapi: If slave id is not set, lookup by name .. restapi: If slave id is not set, lookup by name If slave ID for bonding is not set, then need to look for the

[Engine-patches] Change in ovirt-engine[master]: restapi: user cannnot take a vm from pool via restapi #86443...

2012-10-11 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: restapi: user cannnot take a vm from pool via restapi #864438 .. restapi: user cannnot take a vm from pool via restapi #864438 This patch implements new act

[Engine-patches] Change in ovirt-engine[master]: engine: PM configuration during host installation sometime f...

2012-10-11 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: engine: PM configuration during host installation sometime fails due to a race .. engine: PM configuration during host installation sometime fails due

[Engine-patches] Change in ovirt-engine[master]: engine: PM configuration during host installation sometime f...

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: PM configuration during host installation sometime fails due to a race .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 12: (1 inline comment) File backend/manager/modul

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove domain column from template main grid (#837...

2012-10-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: remove domain column from template main grid (#837025) .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8229 To unsub

[Engine-patches] Change in ovirt-engine[master]: engine: PM configuration during host installation sometime f...

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine: PM configuration during host installation sometime fails due to a race .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8485 To unsub

[Engine-patches] Change in ovirt-engine[master]: engine: PM configuration during host installation sometime f...

2012-10-11 Thread mkublin
Michael Kublin has uploaded a new change for review. Change subject: engine: PM configuration during host installation sometime fails due to a race .. engine: PM configuration during host installation sometime fails due to a rac

[Engine-patches] Change in ovirt-engine[master]: webadmin: rename Allocation to Space Allocation in new VM di...

2012-10-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: rename Allocation to Space Allocation in new VM dialog (#837025) .. Patch Set 2: (2 inline comments) ...

[Engine-patches] Change in ovirt-engine[master]: engine: Get Volume Advanced Details Query

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 14: (2 inline comments) File backend/manager/module

[Engine-patches] Change in ovirt-engine[master]: [RFE] Integrate smartcard support

2012-10-11 Thread tjelinek
Tomas Jelinek has uploaded a new change for review. Change subject: [RFE] Integrate smartcard support .. [RFE] Integrate smartcard support This patch integrates the smartcard support into the frontend and engine. The behavior o

[Engine-patches] Change in ovirt-engine[master]: engine:Cannot remove iscsi master storage right after removi...

2012-10-11 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: engine:Cannot remove iscsi master storage right after removing the storage pool .. engine:Cannot remove iscsi master storage right after removing the s

[Engine-patches] Change in ovirt-engine[master]: engine:Cannot remove iscsi master storage right after removi...

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine:Cannot remove iscsi master storage right after removing the storage pool .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-10-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 12: (1 inline comment) File backend/manager/modul

[Engine-patches] Change in ovirt-engine[master]: engine:Cannot remove iscsi master storage right after removi...

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine:Cannot remove iscsi master storage right after removing the storage pool .. Patch Set 2: Verified rebased -- To view, visit http://gerrit.ovirt.org/836

[Engine-patches] Change in ovirt-engine[master]: webadmin: rename Allocation to Space Allocation in new VM di...

2012-10-11 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: rename Allocation to Space Allocation in new VM dialog (#837025) .. Patch Set 1: (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: core: Removing reschedule of monitoring of vdsm by another v...

2012-10-11 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Removing reschedule of monitoring of vdsm by another vdsm .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8483 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: core: Removing reschedule of monitoring of vdsm by another v...

2012-10-11 Thread mkublin
Michael Kublin has uploaded a new change for review. Change subject: core: Removing reschedule of monitoring of vdsm by another vdsm .. core: Removing reschedule of monitoring of vdsm by another vdsm The following patch will rem

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-10-11 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 12: (1 inline comment) File backend/manager/modules

[Engine-patches] Change in ovirt-engine[master]: core: QuotaManager Test - test 5, 6

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: QuotaManager Test - test 5,6 .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8465 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: core: QuotaManager Test - test 2, 3, 4

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: QuotaManager Test - test 2,3,4 .. Patch Set 2: I would prefer that you didn't submit this Same issues I had with the previous patch: 1. Find better names

[Engine-patches] Change in ovirt-engine[master]: core: QuotaManager Test - test 1

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: QuotaManager Test - test 1 .. Patch Set 4: I would prefer that you didn't submit this (1 inline comment) -1 on the naming, +1 on everything else. Consid

[Engine-patches] Change in ovirt-engine[master]: core : QuotaMnager Test - basic structure

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core : QuotaMnager Test - basic structure .. Patch Set 7: (6 inline comments) Sorry, missed some issues in the previous patchset - see inline.

[Engine-patches] Change in ovirt-engine[master]: webadmin, core: Use NFS v3 as default

2012-10-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin, core: Use NFS v3 as default .. Patch Set 2: Looks good to me, but someone else must approve (2 inline comments) Looks OK, see suggestions inline. ..