[Engine-patches] Change in ovirt-engine[master]: engine: Return Volume Rebalance status on rebalance stop

2013-09-11 Thread rnachimu
Ramesh N has posted comments on this change. Change subject: engine: Return Volume Rebalance status on rebalance stop .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/19109 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 6: (1 comment) File backend/manag

[Engine-patches] Change in ovirt-guest-agent[master]: agent: Fix python-2.4 compatibility

2013-09-11 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: agent: Fix python-2.4 compatibility .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19148 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-guest-agent[master]: Packaging: Add the missing dependencies

2013-09-11 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: Packaging: Add the missing dependencies .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/18889 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: engine: Engine has to delete image from DB after VDSM remove...

2013-09-11 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: engine: Engine has to delete image from DB after VDSM removed it .. Patch Set 1: (1 comment) File backend/

[Engine-patches] Change in ovirt-engine[master]: engine: Add audit log messages to GetAlignmentScanCommand

2013-09-11 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: engine: Add audit log messages to GetAlignmentScanCommand .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/19136 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: engine: Add audit log messages to GetAlignmentScanCommand

2013-09-11 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: engine: Add audit log messages to GetAlignmentScanCommand .. Patch Set 2: I fixed messages due to Cheryn review. -- To view, visit http://gerrit.ovirt.org/1913

[Engine-patches] Change in ovirt-engine[master]: engine: Add audit log messages to GetAlignmentScanCommand

2013-09-11 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: engine: Add audit log messages to GetAlignmentScanCommand .. Patch Set 1: (2 comments) File backend/manage

[Engine-patches] Change in ovirt-engine[master]: engine: Added profile logging to VM run/VNIC plug

2013-09-11 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine: Added profile logging to VM run/VNIC plug .. Patch Set 3: (1 comment) File backend/manager/modules

[Engine-patches] Change in ovirt-engine[master]: gluster: Gluster volume rebalance status entities

2013-09-11 Thread shtripat
Shubhendu Tripathi has posted comments on this change. Change subject: gluster: Gluster volume rebalance status entities .. Patch Set 24: (2 comments) File backend/manager/

[Engine-patches] Change in ovirt-engine[master]: gluster: Query for gluster volume rebalance status

2013-09-11 Thread shtripat
Shubhendu Tripathi has posted comments on this change. Change subject: gluster: Query for gluster volume rebalance status .. Patch Set 15: (1 comment) File backend/manager/

[Engine-patches] Change in ovirt-engine[master]: packaging: enable overriding default value of an option with...

2013-09-11 Thread iheim
Itamar Heim has posted comments on this change. Change subject: packaging: enable overriding default value of an option with commandline .. Patch Set 2: is this still relevant? -- To view, visit http://gerrit.ovirt.org/9115

[Engine-patches] Change in ovirt-engine[master]: engine: Integrate Atlassian Crowd Client as a new Authentica...

2013-09-11 Thread iheim
Itamar Heim has abandoned this change. Change subject: engine: Integrate Atlassian Crowd Client as a new Authentication Domain .. Abandoned no activity in a while, though will be happy if licensing is resolved. otherwise, may

[Engine-patches] Change in ovirt-engine[master]: packaging: validate MAC addresses range provided by the user

2013-09-11 Thread Itamar Heim
Itamar Heim has abandoned this change. Change subject: packaging: validate MAC addresses range provided by the user .. Abandoned -- To view, visit http://gerrit.ovirt.org/8670 To unsubscribe, visit http://gerrit.ovirt.org/sett

[Engine-patches] Change in ovirt-engine[master]: engine: Engine has to delete image from DB after VDSM remove...

2013-09-11 Thread sgotliv
Sergey Gotliv has uploaded a new change for review. Change subject: engine: Engine has to delete image from DB after VDSM removed it .. engine: Engine has to delete image from DB after VDSM removed it Engine has to delete image

[Engine-patches] Change in ovirt-engine[master]: engine: application touches a file to detect un orderly shut...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: application touches a file to detect un orderly shutdown .. Patch Set 10: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19102 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: engine: application touches a file to detect un orderly shut...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: engine: application touches a file to detect un orderly shutdown .. engine: application touches a file to detect un orderly shutdown Used to diagnose u

[Engine-patches] Change in ovirt-host-deploy[master]: packaging: Virtual DMI decode package

2013-09-11 Thread vitor . lima
Vitor de Lima has uploaded a new change for review. Change subject: packaging: Virtual DMI decode package .. packaging: Virtual DMI decode package This introduces a subpackage which provides the dmidecode dependency in platforms

[Engine-patches] Change in ovirt-engine[master]: core: lock VM for whole live snapshot process

2013-09-11 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: core: lock VM for whole live snapshot process .. core: lock VM for whole live snapshot process VM was used to be locked for the entire live snapshot process only i

[Engine-patches] Change in ovirt-engine[master]: core: revert the existing solution for bz 878041

2013-09-11 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: core: revert the existing solution for bz 878041 .. core: revert the existing solution for bz 878041 Before having the auto-start VMs runner job, the problem of HA

[Engine-patches] Change in ovirt-engine[master]: engine: application touches a file to detect un orderly shut...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: application touches a file to detect un orderly shutdown .. Patch Set 11: Verified+1 Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19102 To u

[Engine-patches] Change in ovirt-engine[master]: core: trigger auto-start VMs runner job after live snapshot ...

2013-09-11 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: core: trigger auto-start VMs runner job after live snapshot operation .. core: trigger auto-start VMs runner job after live snapshot operation Since live snapshot

[Engine-patches] Change in ovirt-engine[master]: Revert "engine : VdsUpdateRuntimeInfo updates the data of a ...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has uploaded a new change for review. Change subject: Revert "engine : VdsUpdateRuntimeInfo updates the data of a specific VDS." .. Revert "engine : VdsUpdateRuntimeInfo updates the data of a specific VDS." This

[Engine-patches] Change in ovirt-engine[master]: Revert "engine : VdsUpdateRuntimeInfo updates the data of a ...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: Revert "engine : VdsUpdateRuntimeInfo updates the data of a specific VDS." .. Patch Set 1: Verified+1 Code-Review+2 Verified by running some CI jobs with Eyal

[Engine-patches] Change in ovirt-engine[master]: Revert "engine : VdsUpdateRuntimeInfo updates the data of a ...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: Revert "engine : VdsUpdateRuntimeInfo updates the data of a specific VDS." .. Revert "engine : VdsUpdateRuntimeInfo updates the data of a specific VDS.

[Engine-patches] Change in ovirt-engine[engine_3.2]: core: Avoid XSS in RedirectServlet

2013-09-11 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Avoid XSS in RedirectServlet .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19153 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: core: Avoid XSS in RedirectServlet

2013-09-11 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Avoid XSS in RedirectServlet .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19152 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3]: core: Avoid XSS in RedirectServlet

2013-09-11 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Avoid XSS in RedirectServlet .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19155 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[ovirt-engine-3.3.0]: core: Avoid XSS in RedirectServlet

2013-09-11 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Avoid XSS in RedirectServlet .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19156 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 6: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19123 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: move unplugged disks while VM is running

2013-09-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: move unplugged disks while VM is running .. Patch Set 1: (7 comments) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-engine[master]: rest: VirtIO-SCSI enabled flag

2013-09-11 Thread derez
Daniel Erez has uploaded a new change for review. Change subject: rest: VirtIO-SCSI enabled flag .. rest: VirtIO-SCSI enabled flag * Added 'virtio_scsi_enabled' boolean to VM/Template resources (api.xsd). * Set 'VirtioScsiEnable

[Engine-patches] Change in ovirt-engine[master]: engine: sort domain list query results

2013-09-11 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: engine: sort domain list query results .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/18727 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: webadmin: Host interface manual refresh

2013-09-11 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: webadmin: Host interface manual refresh .. Patch Set 1: (1 comment) Commit Message Line 11: - Added manual re

[Engine-patches] Change in ovirt-engine[master]: engine: sort domain list query results

2013-09-11 Thread ecohen
Einav Cohen has submitted this change and it was merged. Change subject: engine: sort domain list query results .. engine: sort domain list query results Changed the domain list query results to be sorted alphabetically, with t

[Engine-patches] Change in ovirt-engine[master]: webadmin: unescape cell tooltips string values

2013-09-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: unescape cell tooltips string values .. Patch Set 2: Hmm, do you mean something similar to SafeHtmlWithSafeHtmlTooltipColumn as I've suggested? I.e. do

[Engine-patches] Change in ovirt-engine[master]: webadmin: unescape cell tooltips string values

2013-09-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: unescape cell tooltips string values .. Patch Set 2: Hmm, do you mean something similar to SafeHtmlWithSafeHtmlTooltipColumn as I've suggested? I.e. do

[Engine-patches] Change in ovirt-engine[master]: engine: application touches a file to detect un orderly shut...

2013-09-11 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: engine: application touches a file to detect un orderly shutdown .. Patch Set 10: Code-Review+1 (1 comment) ...

[Engine-patches] Change in ovirt-engine[master]: webadmin: unescape cell tooltips string values

2013-09-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: unescape cell tooltips string values .. Patch Set 2: Hmm, do you mean something similar to SafeHtmlWithSafeHtmlTooltipColumn as I've suggested? I.e. do

[Engine-patches] Change in ovirt-engine[master]: API: editing host add and update fields - exposing all store...

2013-09-11 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: API: editing host add and update fields - exposing all stored ssh fields .. Patch Set 5: Michael: What is need to be done for final verification for that? --

[Engine-patches] Change in ovirt-engine[master]: engine : Updated user information in OpenLDAP are not propag...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine : Updated user information in OpenLDAP are not propagated to engine .. Patch Set 5: No need for all the changes with DbUserCacheManager? Strange, it rea

[Engine-patches] Change in ovirt-engine[master]: engine: application touches a file to detect un orderly shut...

2013-09-11 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: engine: application touches a file to detect un orderly shutdown .. Patch Set 10: (1 comment) Commit Messa

[Engine-patches] Change in ovirt-engine[master]: engine: Cache spice support for os/compatibility version

2013-09-11 Thread fkobzik
Frank Kobzik has posted comments on this change. Change subject: engine: Cache spice support for os/compatibility version .. Patch Set 2: Verified+1 -- To view, visit http://gerrit.ovirt.org/19137 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-hosted-engine-setup[master]: build: ovirt-hosted-engine-setup-1.0.0-0.3.beta

2013-09-11 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: build: ovirt-hosted-engine-setup-1.0.0-0.3.beta .. Patch Set 14: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/18064 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: restapi: Update VM pool fail with NPE

2013-09-11 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: restapi: Update VM pool fail with NPE .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19142 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: webadmin: unescape cell tooltips string values

2013-09-11 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: webadmin: unescape cell tooltips string values .. Patch Set 2: For SafeHtmlCellWithTooltip, I think the answer to "is SafeHtml value in getTooltip method alread

[Engine-patches] Change in ovirt-hosted-engine-setup[master]: ux: describe better how to reboot VM

2013-09-11 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: ux: describe better how to reboot VM .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/18909 To unsubscribe, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: core: Adding job with auto_cleared false...

2013-09-11 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: Adding job with auto_cleared false... .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/19144 To unsubscribe, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-hosted-engine-setup[master]: man: few fixes in documentation

2013-09-11 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: man: few fixes in documentation .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/18929 To unsubscribe, visit http://gerrit.ovirt.org/setti

[Engine-patches] Change in ovirt-engine[master]: core: Adding job with auto_cleared false...

2013-09-11 Thread emesika
Eli Mesika has uploaded a new change for review. Change subject: core: Adding job with auto_cleared false... .. core: Adding job with auto_cleared false... Adding job with auto_cleared false, not working correct v_is_auto_clear

[Engine-patches] Change in ovirt-hosted-engine-setup[master]: packaging: setup: correctly parse stroage domain info

2013-09-11 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: packaging: setup: correctly parse stroage domain info .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19115 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-hosted-engine-setup[master]: bin: support --vm-status option

2013-09-11 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: bin: support --vm-status option .. Patch Set 7: (1 comment) see minor comment. File src/bin/hosted-engine.i

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 6: Ok, since the validator also saves the need to add another VdcBllMessage I've moved to us

[Engine-patches] Change in ovirt-engine[master]: engine: Cache spice support for os/compatibility version

2013-09-11 Thread fkobzik
Frank Kobzik has posted comments on this change. Change subject: engine: Cache spice support for os/compatibility version .. Patch Set 2: Deleted unused osRepository function hasSpiceSupport and related code. -- To view, visi

[Engine-patches] Change in ovirt-engine[master]: core: share a tranlation table of osinfo with DWH

2013-09-11 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: share a tranlation table of osinfo with DWH .. Patch Set 9: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/17767 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: share a tranlation table of osinfo with DWH

2013-09-11 Thread ydary
Yaniv Dary has posted comments on this change. Change subject: core: share a tranlation table of osinfo with DWH .. Patch Set 9: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/17767 To unsubscribe, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: engine: sort domain list query results

2013-09-11 Thread gshereme
Greg Sheremeta has posted comments on this change. Change subject: engine: sort domain list query results .. Patch Set 3: Verified+1 Einav, I believe this is ready for merge. -- To view, visit http://gerrit.ovirt.org/18727 To

[Engine-patches] Change in ovirt-engine[master]: engine: sort domain list query results

2013-09-11 Thread gshereme
Greg Sheremeta has posted comments on this change. Change subject: engine: sort domain list query results .. Patch Set 2: (1 comment) Commit Message Line 3: AuthorDate: 2013

[Engine-patches] Change in ovirt-engine[master]: engine: nullify EmulatedMachine field only on cluster level ...

2013-09-11 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: engine: nullify EmulatedMachine field only on cluster level change .. Patch Set 4: (1 comment) File front

[Engine-patches] Change in ovirt-engine[master]: webadmin, userportal: do not add default nic on edit vm

2013-09-11 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: webadmin,userportal: do not add default nic on edit vm .. Patch Set 4: Verified+1 addressed comments -- To view, visit http://gerrit.ovirt.org/17315 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: engine: Retain gluster task id on volume when task completes

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Retain gluster task id on volume when task completes .. Patch Set 4: -Code-Review Can you please elaborate more in the commit msg? Now that I took a sec

[Engine-patches] Change in ovirt-engine[master]: gluster: Gluster volume rebalance status entities

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: gluster: Gluster volume rebalance status entities .. Patch Set 24: Code-Review+1 (2 comments) Minor comments .

[Engine-patches] Change in ovirt-engine[master]: core, engine: SCSI CD-ROM on PPC64 VMs

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core, engine: SCSI CD-ROM on PPC64 VMs .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/18622 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 5: (1 comment) File backend/manager/mo

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 5: (1 comment) File backend/manager/mo

[Engine-patches] Change in ovirt-engine[master]: core: rename methods to lowercase in storage commands

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: rename methods to lowercase in storage commands .. Patch Set 2: Verified+1 Code-Review+2 Rebased, no code changes. -- To view, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 5: (1 comment) File backend/manag

[Engine-patches] Change in ovirt-engine[master]: core, engine: SCSI CD-ROM on PPC64 VMs

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core, engine: SCSI CD-ROM on PPC64 VMs .. Patch Set 1: Looks OK to me. Omer/Roy? -- To view, visit http://gerrit.ovirt.org/18622 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-engine[master]: engine: application touches a file to detect un orderly shut...

2013-09-11 Thread mtayer
mooli tayer has posted comments on this change. Change subject: engine: application touches a file to detect un orderly shutdown .. Patch Set 10: Verified+1 -- To view, visit http://gerrit.ovirt.org/19102 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: rename methods to lowercase in storage commands

2013-09-11 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: rename methods to lowercase in storage commands .. core: rename methods to lowercase in storage commands Change-Id: Ic90230c11cf11fe510c803c69ffa

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 5: (2 comments) File backend/mana

[Engine-patches] Change in ovirt-engine[master]: core: rename ExecuteQuery to executeQuery

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: rename ExecuteQuery to executeQuery .. Patch Set 1: (1 comment) Commit Message Line 4: Commit: A

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 5: (1 comment) File backend/manager/mo

[Engine-patches] Change in ovirt-engine[master]: core: Disallow force select SPM when Data Center is not up

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Disallow force select SPM when Data Center is not up .. Patch Set 5: Code-Review-1 (1 comment) File

[Engine-patches] Change in ovirt-engine[master]: webadmin: unescape cell tooltips string values

2013-09-11 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: unescape cell tooltips string values .. Patch Set 2: Not sure since htmlEscapeAllowEntities also escapes a string, but just leaving not double-escape ta

[Engine-patches] Change in ovirt-engine[master]: engine: Retain gluster task id on volume when task completes

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Retain gluster task id on volume when task completes .. Patch Set 4: Code-Review+1 needs rebasing, but looks good. -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: engine: Support for gluster task monitoring

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Support for gluster task monitoring .. Patch Set 7: (2 comments) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-engine[master]: core: rename methods to lowercase in storage commands

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: rename methods to lowercase in storage commands .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19139 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: core: rename ExecuteQuery to executeQuery

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: rename ExecuteQuery to executeQuery .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.ovirt.org/19138 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: gluster:VDS command to get gluster tasks list

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: gluster:VDS command to get gluster tasks list .. Patch Set 5: Code-Review+1 (1 comment) Minor comment + please resolve conflicts and rebase. Looks good besides

[Engine-patches] Change in ovirt-engine[master]: engine: nullify EmulatedMachine field only on cluster level ...

2013-09-11 Thread rgolan
Roy Golan has posted comments on this change. Change subject: engine: nullify EmulatedMachine field only on cluster level change .. Patch Set 4: (2 comments) File frontend

[Engine-patches] Change in ovirt-engine[master]: engine: nullify EmulatedMachine field only on cluster level ...

2013-09-11 Thread rgolan
Roy Golan has posted comments on this change. Change subject: engine: nullify EmulatedMachine field only on cluster level change .. Patch Set 3: (1 comment) File backend/m

[Engine-patches] Change in ovirt-engine[master]: engine: Cache spice support for os/compatibility version

2013-09-11 Thread fkobzik
Frank Kobzik has posted comments on this change. Change subject: engine: Cache spice support for os/compatibility version .. Patch Set 1: Note: if this patch is to be merged, HasSpiceSupport query should be removed, since ther

[Engine-patches] Change in ovirt-engine[master]: core: fix not closing statement resource leak

2013-09-11 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: fix not closing statement resource leak .. core: fix not closing statement resource leak Fix a resource leak caused by not closing a statement by

[Engine-patches] Change in ovirt-engine[master]: core: fix not closing statement resource leak

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: fix not closing statement resource leak .. Patch Set 2: Verified+1 Code-Review+2 Rebased, no code changes. -- To view, visit http://gerrit.ovirt.org/190

[Engine-patches] Change in ovirt-engine[master]: engine: Cache spice support for os/compatibility version

2013-09-11 Thread fkobzik
Frank Kobzik has uploaded a new change for review. Change subject: engine: Cache spice support for os/compatibility version .. engine: Cache spice support for os/compatibility version Patch 31903bda17f97c2d785238b3d0d557d3c24f40

[Engine-patches] Change in ovirt-engine[master]: engine: Add audit log messages to GetAlignmentScanCommand

2013-09-11 Thread sgotliv
Sergey Gotliv has posted comments on this change. Change subject: engine: Add audit log messages to GetAlignmentScanCommand .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.ovirt.org/19136 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: engine: Add audit log messages to GetAlignmentScanCommand

2013-09-11 Thread sgotliv
Sergey Gotliv has uploaded a new change for review. Change subject: engine: Add audit log messages to GetAlignmentScanCommand .. engine: Add audit log messages to GetAlignmentScanCommand Change-Id: I4400aea34bd848416e67b7bb77a92

[Engine-patches] Change in ovirt-engine[master]: core: ExternalSchedulerDiscoveryResult map iter

2013-09-11 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: ExternalSchedulerDiscoveryResult map iter .. core: ExternalSchedulerDiscoveryResult map iter Replaced keySet() iteration with entrySet() iteratio

[Engine-patches] Change in ovirt-engine[master]: core: ExternalSchedulerDiscoveryResult map iter

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: ExternalSchedulerDiscoveryResult map iter .. Patch Set 3: Verified+1 Code-Review+2 Rebased, no code changes -- To view, visit http://gerrit.ovirt.org/18

[Engine-patches] Change in ovirt-engine[master]: core: move unplugged disks while VM is running

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: move unplugged disks while VM is running .. Patch Set 1: @Cheryn, please take a look at backend/manager/modules/dal/src/main/resources/bundles/AppErrors.

[Engine-patches] Change in ovirt-engine[master]: core: move unplugged disks while VM is running

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: move unplugged disks while VM is running .. Patch Set 1: (7 comments) File backend/manager/modules/

[Engine-patches] Change in ovirt-engine[master]: core: AddTemplate: Remove unused dest sd handling

2013-09-11 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: AddTemplate: Remove unused dest sd handling .. core: AddTemplate: Remove unused dest sd handling Change-Id I523814cd3c1f64c1ae40624ba50f239f2ceba

[Engine-patches] Change in ovirt-engine[master]: core: AddTemplate: Remove unused dest sd handling

2013-09-11 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: AddTemplate: Remove unused dest sd handling .. Patch Set 3: Verified+1 Code-Review+2 Rebased, no code changes. -- To view, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: core: PM options for secondary agent are not saved

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: PM options for secondary agent are not saved .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19119 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: core: rename TemplateUser role

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: rename TemplateUser role .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.ovirt.org/18779 To unsubscribe, visit http://gerrit.ovirt.org/sett

[Engine-patches] Change in ovirt-engine[master]: engine : VdsUpdateRuntimeInfo updates the data of a specific...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine : VdsUpdateRuntimeInfo updates the data of a specific VDS. .. Patch Set 27: Verified+1 Code-Review+2 -- To view, visit http://gerrit.ovirt.org/16066 To

[Engine-patches] Change in ovirt-engine[master]: engine : VdsUpdateRuntimeInfo updates the data of a specific...

2013-09-11 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: engine : VdsUpdateRuntimeInfo updates the data of a specific VDS. .. engine : VdsUpdateRuntimeInfo updates the data of a specific VDS. This change mig

[Engine-patches] Change in ovirt-engine[master]: core: Fixing job dao test

2013-09-11 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fixing job dao test .. Patch Set 2: Verified+1 Code-Review+2 -- To view, visit http://gerrit.ovirt.org/19131 To unsubscribe, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: core: Fixing job dao test

2013-09-11 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fixing job dao test .. core: Fixing job dao test Job dao test relies that the first job is always the same - this assumption may cause a test to

  1   2   3   >