[Engine-patches] Change in ovirt-engine[master]: core: removed unused fqn from VdsSelector

2012-10-08 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: removed unused fqn from VdsSelector .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: core: log should be final in VdsSelector

2012-10-08 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: log should be final in VdsSelector .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8403 To unsub

[Engine-patches] Change in ovirt-engine[master]: core: camel-case methods in VdsSelector

2012-10-08 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: camel-case methods in VdsSelector .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8402 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: break down spaghetti-code in VdsSelector

2012-10-08 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: break down spaghetti-code in VdsSelector .. Patch Set 1: not entirely convinced this is more readable I'll have to think about it for a while -- To view

[Engine-patches] Change in ovirt-engine[master]: core: VdsSelector cleanup

2012-10-08 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: VdsSelector cleanup .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: core: remove unused code from VdsSelector

2012-10-08 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: remove unused code from VdsSelector .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: core: search for disks on specific domains

2012-10-08 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: search for disks on specific domains .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8385 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: core: VmPayload serialVersionUID should be final

2012-10-08 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: VmPayload serialVersionUID should be final .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8405 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Reduce host connection timeout

2012-10-08 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Reduce host connection timeout .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8410 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: core: Reduce host connection timeout

2012-10-08 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: core: Reduce host connection timeout .. core: Reduce host connection timeout The timeout that we currently use to connect to hosts is by default very large, as

[Engine-patches] Change in ovirt-engine[master]: core: correcting max memory for scheduling VMs field

2012-10-08 Thread apahim
Amador Pahim has posted comments on this change. Change subject: core: correcting max memory for scheduling VMs field .. Patch Set 1: Ok. I got it. Let me argue a bit... without "guest_overhead", the "Max free Memory for sched

[Engine-patches] Change in ovirt-engine[master]: core, webadmin: Failure to reconnect authenticated iSCSI LUNs

2012-10-08 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: core,webadmin: Failure to reconnect authenticated iSCSI LUNs .. Patch Set 2: (4 inline comments) Thanks Allon and Doron, further comments inline. ...

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

2012-10-08 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 10: (3 inline comments) More comments. File backen

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

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

[Engine-patches] Change in ovirt-engine[master]: webadmin: Adding "Select All" checkbox header to CellTable

2012-10-08 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: webadmin: Adding "Select All" checkbox header to CellTable .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8282 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: core: Tar cleanup

2012-10-08 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Tar cleanup .. Patch Set 3: Thanks! If you still feel like the closure of stream should be modified, feel free to do this... My view is my personal view..

[Engine-patches] Change in ovirt-engine[master]: core: Tar cleanup

2012-10-08 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Tar cleanup .. Patch Set 3: I prefer to leave this as-is, the comment is there to say that this is intentional... well, comment was actually for Juan :) -

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

2012-10-08 Thread dgopal
Dhandapani Gopal has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 9: (13 inline comments) Incorporated Yair and Moti review comments. One in-line reply for Yair's comme

[Engine-patches] Change in ovirt-engine[master]: core: Tar cleanup

2012-10-08 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Tar cleanup .. Patch Set 3: Again There is no point in fixing the main() it is just for manual testing, never actually executed. And As I wrote I inte

[Engine-patches] Change in ovirt-engine[master]: bootstrap: allow customize bridge name for standard sequence

2012-10-08 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: bootstrap: allow customize bridge name for standard sequence .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: restapi: VM created from template -> "Bad volume specificati...

2012-10-08 Thread tjelinek
Tomas Jelinek has abandoned this change. Change subject: restapi: VM created from template -> "Bad volume specification" .. Patch Set 1: Abandoned http://gerrit.ovirt.org/8092 is a better solution for this -- To view, visit ht