[Engine-patches] Change in ovirt-engine[master]: engine: VM Linux Boot options aren't parsing properly

2013-01-15 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: engine: VM Linux Boot options aren't parsing properly .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9550 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: packaging: Application Mode option in installer

2013-01-15 Thread avishwan
Aravinda VK has posted comments on this change. Change subject: packaging: Application Mode option in installer .. Patch Set 5: (3 inline comments) Will submit new patch set with the changes. ..

[Engine-patches] Change in ovirt-engine[master]: engine: Disable gluster hook command

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Disable gluster hook command .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/11073 To unsubscr

[Engine-patches] Change in ovirt-engine[master]: core: add support for tunnel migration

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: add support for tunnel migration .. Patch Set 1: (1 inline comment) File backend/manager/modules/com

[Engine-patches] Change in ovirt-engine[master]: engine: Removing using of latch all around a code

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: Removing using of latch all around a code .. Patch Set 1: (2 inline comments) File backend/manager

[Engine-patches] Change in ovirt-engine[master]: core:Async Tasks cleaner utility...

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core:Async Tasks cleaner utility... .. Patch Set 5: Eli, bare in mind that the utility of 3.0 -> 3.1 presented the tasks by groups of commands - stored procedu

[Engine-patches] Change in ovirt-engine[master]: engine : User is UNKNOWN in async task completion events

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine : User is UNKNOWN in async task completion events .. Patch Set 3: (1 inline comment) File backend/m

[Engine-patches] Change in ovirt-engine[master]: engine: Disable gluster hook command

2013-01-15 Thread sesubram
Selvasundaram has uploaded a new change for review. Change subject: engine: Disable gluster hook command .. engine: Disable gluster hook command Disable the gluster hook in all available up servers in the gluster cluster. -

[Engine-patches] Change in ovirt-engine[master]: core: add support for tunnel migration

2013-01-15 Thread danken
Dan Kenigsberg has posted comments on this change. Change subject: core: add support for tunnel migration .. Patch Set 1: (2 inline comments) File backend/manager/dbscripts/u

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add DC column to main tab Cluster view

2013-01-15 Thread gpadgett
Greg Padgett has uploaded a new change for review. Change subject: webadmin: Add DC column to main tab Cluster view .. webadmin: Add DC column to main tab Cluster view Add Data Center column to main tab Cluster view. Change-Id:

[Engine-patches] Change in ovirt-iso-uploader[master]: packaging: Fixed creating objects in an insecure way

2013-01-15 Thread kroberts
Keith Robertson has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 2: Do not submit Please fix this in the API. The specific line of code you want to look at i

[Engine-patches] Change in ovirt-engine[master]: gluster: Make set option a step of create volume

2013-01-15 Thread masayag
Moti Asayag has posted comments on this change. Change subject: gluster: Make set option a step of create volume .. Patch Set 2: (1 inline comment) File backend/manager/modu

[Engine-patches] Change in ovirt-iso-uploader[master]: packaging: Fixed creating objects in an insecure way

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 2: I would prefer that you didn't submit this (1 inline comment) ..

[Engine-patches] Change in ovirt-engine[master]: tools: Adding Setting ldap servers for domain

2013-01-15 Thread yzaslavs
Yair Zaslavsky has uploaded a new change for review. Change subject: tools: Adding Setting ldap servers for domain .. tools: Adding Setting ldap servers for domain This patch adds support for setting ldap servers for a given dom

[Engine-patches] Change in ovirt-engine[master]: core: Supporting hard coded ldap servers per domain

2013-01-15 Thread yzaslavs
Yair Zaslavsky has uploaded a new change for review. Change subject: core: Supporting hard coded ldap servers per domain .. core: Supporting hard coded ldap servers per domain Adding support for hard coded ldap server list per d

[Engine-patches] Change in ovirt-engine[master]: core: Adding ldap server ordering algorithms factory

2013-01-15 Thread yzaslavs
Yair Zaslavsky has uploaded a new change for review. Change subject: core: Adding ldap server ordering algorithms factory .. core: Adding ldap server ordering algorithms factory The following adds ldap server ordering logic as p

[Engine-patches] Change in ovirt-engine[master]: core:Async Tasks cleaner utility...

2013-01-15 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core:Async Tasks cleaner utility... .. Patch Set 5: Verified -- To view, visit http://gerrit.ovirt.org/11005 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-iso-uploader[master]: packaging: Fixed creating objects in an insecure way

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 2: (1 inline comment) File src/engine-iso-u

[Engine-patches] Change in ovirt-engine[master]: webadmin: fetch bookmarks only when tab is shown

2013-01-15 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: webadmin: fetch bookmarks only when tab is shown .. Patch Set 1: Looks good to me, but someone else must approve Looks good. If other widgets in the panel need a

[Engine-patches] Change in ovirt-engine[master]: engine: only supported models should use search paging

2013-01-15 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: engine: only supported models should use search paging .. Patch Set 1: (2 inline comments) File backend/manag

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: (1 inline comment) File frontend/webadmin/modules/uic

[Engine-patches] Change in ovirt-iso-uploader[master]: packaging: Fixed creating objects in an insecure way

2013-01-15 Thread kroberts
Keith Robertson has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 2: No score; No score (1 inline comment) ...

[Engine-patches] Change in ovirt-iso-uploader[master]: packaging: Fixed creating objects in an insecure way

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 2: (1 inline comment) File src/engine-iso-u

[Engine-patches] Change in ovirt-iso-uploader[master]: packaging: Fixed creating objects in an insecure way

2013-01-15 Thread kroberts
Keith Robertson has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 2: Verified; Looks good to me, approved (1 inline comment) .

[Engine-patches] Change in ovirt-engine[master]: webadmin: fetch bookmarks only when tab is shown

2013-01-15 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: fetch bookmarks only when tab is shown .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/11060

[Engine-patches] Change in ovirt-engine[master]: core: TrybackToAllSnapshots vmDown validation

2013-01-15 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: TrybackToAllSnapshots vmDown validation .. Patch Set 2: (1 inline comment) File backend/manager/module

[Engine-patches] Change in ovirt-engine[master]: Send http 400 when the cpu pinning is wrong

2013-01-15 Thread dfediuck
Doron Fediuck has posted comments on this change. Change subject: Send http 400 when the cpu pinning is wrong .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/11059 To un

[Engine-patches] Change in ovirt-engine[master]: packaging: updated setup to fail immediately if adding host ...

2013-01-15 Thread alourie
Alex Lourie has posted comments on this change. Change subject: packaging: updated setup to fail immediately if adding host fails .. Patch Set 6: Verified -- To view, visit http://gerrit.ovirt.org/10578 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: VM is down check when adding template

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: VM is down check when adding template .. Patch Set 2: Looks good to me, approved Still can't figure out how you manage to count the booleans in each of the pa

[Engine-patches] Change in ovirt-engine[master]: core: add support for tunnel migration

2013-01-15 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: add support for tunnel migration .. Patch Set 1: Fails didn't verify yet -- To view, visit http://gerrit.ovirt.org/11062 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: core: add support for tunnel migration

2013-01-15 Thread ofrenkel
Omer Frenkel has uploaded a new change for review. Change subject: core: add support for tunnel migration .. core: add support for tunnel migration with 'tunnel migration' the migration process is using libvirt-to-libvirt commun

[Engine-patches] Change in ovirt-engine[master]: core: Early return RemoveDiskCommand.canDoAction()

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Early return RemoveDiskCommand.canDoAction() .. Patch Set 2: Looks good to me, but someone else must approve Looks good, but tests are a good idea -- To view,

[Engine-patches] Change in ovirt-engine[master]: core: Consolidate vm down check when removing disk

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Consolidate vm down check when removing disk .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11013 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: RemoveSnapshot vmDown validation

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: RemoveSnapshot vmDown validation .. Patch Set 2: Looks good to me, approved (1 inline comment) File back

[Engine-patches] Change in ovirt-engine[master]: core: TrybackToAllSnapshots vmDown validation

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: TrybackToAllSnapshots vmDown validation .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11015 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-engine[master]: core: ExportVm vmDown validation

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: ExportVm vmDown validation .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11016 To unsubscribe, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: packaging: updated setup to fail immediately if adding host ...

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: updated setup to fail immediately if adding host fails .. Patch Set 6: Looks good to me, but someone else must approve -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: > going fw, we need to support paging for this field. not sure about the paging (could be), but we'd might

[Engine-patches] Change in ovirt-engine[master]: core: MoveVm vmDown validation

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: MoveVm vmDown validation .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11017 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: Do not submit probably will abandon this patch. on a second thought, going fw, we need to support paging

[Engine-patches] Change in ovirt-engine[master]: engine: only supported models should use search paging

2013-01-15 Thread gchaplik
Gilad Chaplik has uploaded a new change for review. Change subject: engine: only supported models should use search paging .. engine: only supported models should use search paging Currently only main tabs support paging, that m

[Engine-patches] Change in ovirt-engine[master]: webadmin: fetch bookmarks only when tab is shown

2013-01-15 Thread gchaplik
Gilad Chaplik has uploaded a new change for review. Change subject: webadmin: fetch bookmarks only when tab is shown .. webadmin: fetch bookmarks only when tab is shown And stop fetching when the tab is hidden. Change-Id: Iebc1

[Engine-patches] Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: Verify vdsm cluster level support before send getHardwareInfo request .. Patch Set 4: (1 inline comment) F

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: there is a slight problem with over-crowding the UI that we need to solve. and actually this patch will

[Engine-patches] Change in ovirt-engine[master]: core: Remove checkVmDown from PerformImagesChecks

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Remove checkVmDown from PerformImagesChecks .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11018 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: core: remove unused code and fix param name

2013-01-15 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: remove unused code and fix param name .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11022 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: Looks good to me, but someone else must approve Gilad - any chance to address Alona's comment about the one

[Engine-patches] Change in ovirt-engine[master]: core: Remove checkVmDown from PerformImagesChecks

2013-01-15 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove checkVmDown from PerformImagesChecks .. Patch Set 2: Verified rebased, no code changes -- To view, visit http://gerrit.ovirt.org/11018 To unsubscr

[Engine-patches] Change in ovirt-engine[master]: core: MoveVm vmDown validation

2013-01-15 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: MoveVm vmDown validation .. Patch Set 2: Verified rebased, no code changes -- To view, visit http://gerrit.ovirt.org/11017 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: Verify vdsm cluster level support before send getHardwareInfo request .. Patch Set 3: (3 inline comments)

[Engine-patches] Change in ovirt-engine[master]: core: ExportVm vmDown validation

2013-01-15 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: ExportVm vmDown validation .. Patch Set 2: Verified rebased, no code changes -- To view, visit http://gerrit.ovirt.org/11016 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-engine[master]: gluster: Make set option a step of create volume

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: gluster: Make set option a step of create volume .. Patch Set 1: (2 inline comments) Will send new patch-set, incorporating Moti's suggestions. ...

[Engine-patches] Change in ovirt-dwh[master]: history: fixed post upgrade enum change issue

2013-01-15 Thread ydary
Yaniv Dary has submitted this change and it was merged. Change subject: history: fixed post upgrade enum change issue .. history: fixed post upgrade enum change issue Change-Id: I9e2139f436a42135ded32e14d02687c1057e8cd3 Signed-

[Engine-patches] Change in ovirt-dwh[master]: history: fixed post upgrade enum change issue

2013-01-15 Thread ydary
Yaniv Dary has posted comments on this change. Change subject: history: fixed post upgrade enum change issue .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11055 To unsubscribe, v

[Engine-patches] Change in ovirt-dwh[master]: history: fixed post upgrade enum change issue

2013-01-15 Thread ydary
Yaniv Dary has uploaded a new change for review. Change subject: history: fixed post upgrade enum change issue .. history: fixed post upgrade enum change issue Change-Id: I9e2139f436a42135ded32e14d02687c1057e8cd3 Signed-off-by:

[Engine-patches] Change in ovirt-engine[master]: gluster: release lock in canDoActionOnly()

2013-01-15 Thread sanjal
Shireesh Anjal has abandoned this change. Change subject: gluster: release lock in canDoActionOnly() .. Patch Set 2: Abandoned Taken care with a different approach in http://gerrit.ovirt.org/#/c/10812 -- To view, visit http://

[Engine-patches] Change in ovirt-engine[master]: gluster: GlusterMultipleActionRunner

2013-01-15 Thread sanjal
Shireesh Anjal has submitted this change and it was merged. Change subject: gluster: GlusterMultipleActionRunner .. gluster: GlusterMultipleActionRunner Gluster commands always acquire a lock on the cluster. This means that if

[Engine-patches] Change in ovirt-engine[master]: gluster: GlusterMultipleActionRunner

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: gluster: GlusterMultipleActionRunner .. Patch Set 4: Looks good to me, approved Merging based on Michael's +2 -- To view, visit http://gerrit.ovirt.org/10812 T

[Engine-patches] Change in ovirt-engine[master]: gluster: GlusterMultipleActionRunner

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: gluster: GlusterMultipleActionRunner .. Patch Set 4: Verified -- To view, visit http://gerrit.ovirt.org/10812 To unsubscribe, visit http://gerrit.ovirt.org/sett

[Engine-patches] Change in ovirt-engine[master]: api: Add support for GlusterFS in api.

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: api: Add support for GlusterFS in api. .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8844 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: gluster: GlusterMultipleActionRunner

2013-01-15 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: gluster: GlusterMultipleActionRunner .. Patch Set 3: Looks good to me, approved The problem is actually the design of MultipleActionRunner, so we will need to i

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread emesika
Eli Mesika has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: Looks good to me, but someone else must approve DB stuff OK -- To view, visit http://gerrit.ovirt.org/11041

[Engine-patches] Change in ovirt-engine[master]: webadmin: improve warning on SPICE Native usb migration supp...

2013-01-15 Thread rgolan
Roy Golan has posted comments on this change. Change subject: webadmin: improve warning on SPICE Native usb migration support .. Patch Set 2: (1 inline comment) File fronten

[Engine-patches] Change in ovirt-engine[master]: packaging: Copy postgresql-jdbc.jar instead of link

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Copy postgresql-jdbc.jar instead of link .. Patch Set 1: The new thing is that we actually use this after installation, while we try to avoid doing th

[Engine-patches] Change in ovirt-engine[master]: core: Adding locks in case of gluster hosts

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: core: Adding locks in case of gluster hosts .. Patch Set 2: (2 inline comments) new patch-set to follow. F

[Engine-patches] Change in ovirt-engine[master]: packaging: Copy postgresql-jdbc.jar instead of link

2013-01-15 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: packaging: Copy postgresql-jdbc.jar instead of link .. Patch Set 1: I can think of several operating systems that don't provide that package. Our build system

[Engine-patches] Change in ovirt-engine[master]: gluster: GlusterMultipleActionRunner

2013-01-15 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: gluster: GlusterMultipleActionRunner .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/10812 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: packaging: Copy postgresql-jdbc.jar instead of link

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Copy postgresql-jdbc.jar instead of link .. Patch Set 1: When does it happen? Which 'system' does not have postgres jdbc package? I really don't like

[Engine-patches] Change in ovirt-engine[master]: packaging: Copy postgresql-jdbc.jar instead of link

2013-01-15 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: packaging: Copy postgresql-jdbc.jar instead of link .. Patch Set 1: Verified Verified building/installing/configuring new packages. -- To view, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: core: Adding locks in case of gluster hosts

2013-01-15 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: Adding locks in case of gluster hosts .. Patch Set 2: (2 inline comments) File backend/manager/modules

[Engine-patches] Change in ovirt-engine[master]: packaging: Copy postgresql-jdbc.jar instead of link

2013-01-15 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: packaging: Copy postgresql-jdbc.jar instead of link .. packaging: Copy postgresql-jdbc.jar instead of link Currently we create a symbolic link to the postgresq

[Engine-patches] Change in ovirt-engine[master]: core: Add GlusterFS support to AsyncDataProvider

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: core: Add GlusterFS support to AsyncDataProvider .. Patch Set 4: I can see the method IsVersionMatchStorageType() in AsyncDataProvider. Am I looking at the wro

[Engine-patches] Change in ovirt-engine[master]: userportal, webadmin: console error message translation ignor...

2013-01-15 Thread tjelinek
Tomas Jelinek has uploaded a new change for review. Change subject: userportal,webadmin: console error message translation ignored .. userportal,webadmin: console error message translation ignored When the SPICE or RDP console r

[Engine-patches] Change in ovirt-engine[master]: core: Adding locks in case of gluster hosts

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: core: Adding locks in case of gluster hosts .. Patch Set 1: (1 inline comment) new patch-set to follow. Fi

[Engine-patches] Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...

2013-01-15 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: Verify vdsm cluster level support before send getHardwareInfo request .. Patch Set 2: (2 inline comments) Fi

[Engine-patches] Change in ovirt-engine[master]: core: Adding locks in case of gluster hosts

2013-01-15 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Adding locks in case of gluster hosts .. Patch Set 1: (1 inline comment) File backend/manager/module

[Engine-patches] Change in ovirt-engine[master]: engine: Removing using of latch all around a code

2013-01-15 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine: Removing using of latch all around a code .. Patch Set 1: (2 inline comments) File backend/manager

[Engine-patches] Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...

2013-01-15 Thread iheim
Itamar Heim has posted comments on this change. Change subject: Verify vdsm cluster level support before send getHardwareInfo request .. Patch Set 2: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: engine: Removing using of latch all around a code

2013-01-15 Thread rnori
Ravi Nori has posted comments on this change. Change subject: engine: Removing using of latch all around a code .. Patch Set 1: (2 inline comments) File backend/manager/modu

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) +1 for frontend .

[Engine-patches] Change in ovirt-engine[master]: webadmin: Select english when browser is other locale

2013-01-15 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: Select english when browser is other locale .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/11047 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: Verify vdsm cluster level support before send getHardwareInf...

2013-01-15 Thread ybronhei
Yaniv Bronhaim has posted comments on this change. Change subject: Verify vdsm cluster level support before send getHardwareInfo request .. Patch Set 2: Verified missed the string list. now it works properly, Might be nicer w

[Engine-patches] Change in ovirt-host-deploy[master]: build: add missing .gitignores to dist

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: add missing .gitignores to dist .. build: add missing .gitignores to dist Change-Id: Ie7e72995a00ee6a68e7ed631fa498bf0463b35ae Signed-off-by: Alon

[Engine-patches] Change in ovirt-host-deploy[master]: build: add missing .gitignores to dist

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: add missing .gitignores to dist .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11050 To unsubscribe, visit

[Engine-patches] Change in otopi[master]: build: add missing .gitignores to dist

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: add missing .gitignores to dist .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11049 To unsubscribe, visit

[Engine-patches] Change in otopi[master]: build: add missing .gitignores to dist

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: add missing .gitignores to dist .. build: add missing .gitignores to dist Change-Id: Icc3f8c1f90193a252be6af045c58a2b552a06826 Signed-off-by: Alon

[Engine-patches] Change in ovirt-host-deploy[master]: build: add missing .gitignores to dist

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: add missing .gitignores to dist .. build: add missing .gitignores to dist Change-Id: Ie7e72995a00ee6a68e7ed631fa498bf0463b35ae Signed-off-by: Alon Bar-Lev

[Engine-patches] Change in otopi[master]: build: add missing .gitignores to dist

2013-01-15 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: add missing .gitignores to dist .. build: add missing .gitignores to dist Change-Id: Icc3f8c1f90193a252be6af045c58a2b552a06826 Signed-off-by: Alon Bar-Lev

[Engine-patches] Change in ovirt-engine-sdk-java[master]: sdk: replace mvn-release plugin with local

2013-01-15 Thread mpastern
Michael Pasternak has submitted this change and it was merged. Change subject: sdk: replace mvn-release plugin with local .. sdk: replace mvn-release plugin with local Change-Id: I4ee2aa4edb306eed465eb14451a3837e39351800 Signed

[Engine-patches] Change in ovirt-engine-sdk-java[master]: sdk: replace mvn-release plugin with local

2013-01-15 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: sdk: replace mvn-release plugin with local .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/11048 To unsubscrib

[Engine-patches] Change in ovirt-engine-sdk-java[master]: sdk: replace mvn-release plugin with local

2013-01-15 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: sdk: replace mvn-release plugin with local .. sdk: replace mvn-release plugin with local Change-Id: I4ee2aa4edb306eed465eb14451a3837e39351800 Signed-off-by:

[Engine-patches] Change in ovirt-engine[master]: webadmin: improve behavior of bookmark list (2/2)

2013-01-15 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: improve behavior of bookmark list (2/2) .. webadmin: improve behavior of bookmark list (2/2) This patch series fixes some problems with the be

[Engine-patches] Change in ovirt-engine[master]: webadmin: improve behavior of bookmark list (2/2)

2013-01-15 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: improve behavior of bookmark list (2/2) .. Patch Set 3: Verified; Looks good to me, approved Managed to reproduce it in oVirt (some-kind of a race) (th

[Engine-patches] Change in ovirt-engine[master]: packaging: Application Mode option in installer

2013-01-15 Thread alourie
Alex Lourie has posted comments on this change. Change subject: packaging: Application Mode option in installer .. Patch Set 5: Looks good to me, but someone else must approve (1 inline comment) ...

[Engine-patches] Change in ovirt-engine[master]: webadmin: Application continue polling after logout

2013-01-15 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: Application continue polling after logout .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment)

[Engine-patches] Change in ovirt-engine[master]: engine: use vds query instead of search

2013-01-15 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: engine: use vds query instead of search .. Patch Set 1: (1 inline comment) File frontend/webadmin/modules/u

[Engine-patches] Change in ovirt-engine[master]: build: use release java packages of otopi and ovirt-host-dep...

2013-01-15 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: build: use release java packages of otopi and ovirt-host-deploy .. build: use release java packages of otopi and ovirt-host-deploy Change-Id: I5a06b720

[Engine-patches] Change in ovirt-engine[master]: build: use release java packages of otopi and ovirt-host-dep...

2013-01-15 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: build: use release java packages of otopi and ovirt-host-deploy .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1

[Engine-patches] Change in ovirt-engine[master]: webadmin: Select english when browser is other locale

2013-01-15 Thread awels
Alexander Wels has uploaded a new change for review. Change subject: webadmin: Select english when browser is other locale .. webadmin: Select english when browser is other locale - Fixed ability to select English (which was def

[Engine-patches] Change in ovirt-engine[master]: core: Introduce AddGlusterFsStorageDomainCommand.

2013-01-15 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: core: Introduce AddGlusterFsStorageDomainCommand. .. Patch Set 3: (1 inline comment) File backend/manager/

  1   2   3   >