[Engine-patches] Change in ovirt-engine[master]: core: Compatibility of db scripts with MacOS.

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: Compatibility of db scripts with MacOS. .. Patch Set 2: (1 inline comment) Although I agree that shell script should be indent using tabs... there is no re

[Engine-patches] Change in ovirt-engine[engine_3.2]: packaging: reimplemented engine_firewalld

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: reimplemented engine_firewalld .. Patch Set 1: I thought that[1] fixed it. But I guess there is more the API does not handle. I am fine with dropping

[Engine-patches] Change in ovirt-engine[engine_3.2]: tools: engine-setup - Fixed memory requirements

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: tools: engine-setup - Fixed memory requirements .. Patch Set 1: Looks good to me, but someone else must approve I am not sure this worth for z stream. -- To view

[Engine-patches] Change in ovirt-engine[engine_3.2]: tools: engine-setup - fixed installation messages

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: tools: engine-setup - fixed installation messages .. Patch Set 2: Besides: I am not sure that display the example files for all services (iptables, firewalld) is

[Engine-patches] Change in ovirt-engine[engine_3.2]: tools: engine-setup - fixed installation messages

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: tools: engine-setup - fixed installation messages .. Patch Set 2: I don't think this should go into z of 3.2 -- To view, visit http://gerrit.ovirt.org/13321 To u

[Engine-patches] Change in ovirt-engine[master]: packaging: Added generic checking DB state before upgrade

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Added generic checking DB state before upgrade .. Patch Set 8: (2 inline comments) File packaging/

[Engine-patches] Change in ovirt-engine[engine_3.2]: packaging: reimplemented engine_firewalld

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: reimplemented engine_firewalld .. Patch Set 1: Any reason why we need it at 3.2? -- To view, visit http://gerrit.ovirt.org/13609 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: packaging: refactored firewalld support

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: refactored firewalld support .. Patch Set 1: (2 inline comments) File packaging/fedora/setup/http.

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: fixup systemd packaging

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has abandoned this change. Change subject: packaging: engine-service: fixup systemd packaging .. Patch Set 1: Abandoned Split into two other -- To view, visit http://gerrit.ovirt.org/13644 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 3: Verified Rebase minor fixups. -- To view, visit http://gerrit.ovirt.org/13628 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: fix systemd packaging per fedora ...

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: fix systemd packaging per fedora requirements .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13647 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: move service to backend subpackag...

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: move service to backend subpackage .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13646 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: remove unused tmpfiles config

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: remove unused tmpfiles config .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13645 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: fix systemd packaging per fedora ...

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: engine-service: fix systemd packaging per fedora requirements .. packaging: engine-service: fix systemd packaging per fedora requirements Change-Id:

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: move service to backend subpackag...

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: engine-service: move service to backend subpackage .. packaging: engine-service: move service to backend subpackage Change-Id: I3df9c0f0d221d883c7a49b

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: remove unused tmpfiles config

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: engine-service: remove unused tmpfiles config .. packaging: engine-service: remove unused tmpfiles config Change-Id: I81175c21f018be180157ffcaadef1dd2

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/13628 To unsubscribe, visit http://gerrit.ovirt.org/se

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: fixup systemd packaging

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: fixup systemd packaging .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13644 To unsubscribe, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: do not clean environment

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: do not clean environment .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13643 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: do not clean environment

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: engine-service: do not clean environment .. packaging: engine-service: do not clean environment the init system may put important variables in environ

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: fixup systemd packaging

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: engine-service: fixup systemd packaging .. packaging: engine-service: fixup systemd packaging Change-Id: Iad20fb41030644f32673922e85234b540b68f14f Sig

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-serv

[Engine-patches] Change in ovirt-engine[master]: webadmin, core: WIP edit Posix storage connection

2013-04-05 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: webadmin,core: WIP edit Posix storage connection .. Patch Set 1: Fails fails on vdsm side due to known issue with posix connect. -- To view, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: webadmin, core: WIP edit Posix storage connection

2013-04-05 Thread abonas
Alissa Bonas has uploaded a new change for review. Change subject: webadmin,core: WIP edit Posix storage connection .. webadmin,core: WIP edit Posix storage connection please don't review yet Change-Id: I8904ffab80e9d6e6bfdee4b

[Engine-patches] Change in ovirt-engine[master]: restapi: Apply template on VmPool creation using REST API

2013-04-05 Thread mbetak
Martin Beták has posted comments on this change. Change subject: restapi: Apply template on VmPool creation using REST API .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13639 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: restapi: Apply template on VmPool creation using REST API

2013-04-05 Thread mbetak
Martin Beták has uploaded a new change for review. Change subject: restapi: Apply template on VmPool creation using REST API .. restapi: Apply template on VmPool creation using REST API The BackendVmPoolsResource.mapToVM method

[Engine-patches] Change in ovirt-engine[master]: packaging: refactored firewalld support

2013-04-05 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: refactored firewalld support .. Patch Set 1: Verified Verified in a f18 VM, without enabling AIO. Appreciate any review and testing. -- To view, v

[Engine-patches] Change in ovirt-engine[master]: packaging: refactored firewalld support

2013-04-05 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: refactored firewalld support .. Patch Set 1: Just verified that the package build. Needs more testing. -- To view, visit http://gerrit.ovirt.org/1

[Engine-patches] Change in ovirt-engine[master]: packaging: refactored firewalld support

2013-04-05 Thread sbonazzo
Sandro Bonazzola has uploaded a new change for review. Change subject: packaging: refactored firewalld support .. packaging: refactored firewalld support Added one service configuration template file for each service that should

[Engine-patches] Change in ovirt-engine[master]: restapi : don't set jsessionid cookie when authentication fa...

2013-04-05 Thread rnori
Ravi Nori has posted comments on this change. Change subject: restapi : don't set jsessionid cookie when authentication fails(#927140) .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/13371 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: engine, webadmin: Run and pause action enabled

2013-04-05 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: engine,webadmin: Run and pause action enabled .. Patch Set 2: (2 inline comments) If I understand correctly, the isRunAndPauseEditor is visible in new/edit pool,

[Engine-patches] Change in ovirt-engine[master]: engine : ovirt-node enter installing status when the install...

2013-04-05 Thread rnori
Ravi Nori has posted comments on this change. Change subject: engine : ovirt-node enter installing status when the install fails .. Patch Set 5: Verified -- To view, visit http://gerrit.ovirt.org/13622 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: restapi: ProductInfo use ProductRPMVersion

2013-04-05 Thread rnori
Ravi Nori has posted comments on this change. Change subject: restapi: ProductInfo use ProductRPMVersion .. Patch Set 4: I would prefer that you didn't submit this (1 inline comment) ...

[Engine-patches] Change in ovirt-engine[master]: restapi: ProductInfo use ProductRPMVersion

2013-04-05 Thread rnori
Ravi Nori has posted comments on this change. Change subject: restapi: ProductInfo use ProductRPMVersion .. Patch Set 3: (1 inline comment) File backend/manager/modules/rest

[Engine-patches] Change in ovirt-engine[master]: userportal, webadmin: i18n refresh 'sec'

2013-04-05 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: userportal, webadmin: i18n refresh 'sec' .. userportal, webadmin: i18n refresh 'sec' - Fixed 'sec' not being i18ned on the refresh panel drop down. - Rena

[Engine-patches] Change in ovirt-engine[master]: userportal, webadmin: i18n refresh 'sec'

2013-04-05 Thread derez
Daniel Erez has posted comments on this change. Change subject: userportal, webadmin: i18n refresh 'sec' .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13525 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: provided cleared frontend logging

2013-04-05 Thread ecohen
Einav Cohen has submitted this change and it was merged. Change subject: core: provided cleared frontend logging .. core: provided cleared frontend logging - Added code to display better debug messages for -- RunQuery -- RunPub

[Engine-patches] Change in ovirt-engine[master]: core: provided cleared frontend logging

2013-04-05 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: core: provided cleared frontend logging .. Patch Set 7: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/12559 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: core: Move scheduler to modules

2013-04-05 Thread juan . hernandez
Juan Hernandez has submitted this change and it was merged. Change subject: core: Move scheduler to modules .. core: Move scheduler to modules The scheduler component used to be into a "beans" folder, but it is not different fr

[Engine-patches] Change in ovirt-engine[master]: core: Move scheduler to modules

2013-04-05 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Move scheduler to modules .. Patch Set 3: Verified; Looks good to me, approved Merging based on previous approval. -- To view, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: WIP: Add support for backup/restore API for ISVs

2013-04-05 Thread deepakcs
Deepak C Shetty has posted comments on this change. Change subject: core: WIP: Add support for backup/restore API for ISVs .. Patch Set 1: (2 inline comments) my 2 cents :)

[Engine-patches] Change in ovirt-engine[master]: packaging: build: support subst more than once in line

2013-04-05 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: build: support subst more than once in line .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: packaging: build: support subst more than once in line

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: build: support subst more than once in line .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13630 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: packaging: build: support subst more than once in line

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: build: support subst more than once in line .. packaging: build: support subst more than once in line Change-Id: Ie91ea6bb2765825c022ba699ddf1d7660298

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-serv

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-serv

[Engine-patches] Change in ovirt-engine[master]: restapi: Refactored BackendGlusterBrickResource

2013-04-05 Thread sanjal
Shireesh Anjal has submitted this change and it was merged. Change subject: restapi: Refactored BackendGlusterBrickResource .. restapi: Refactored BackendGlusterBrickResource Refactored BackendGlusterBrickResource to use parent

[Engine-patches] Change in ovirt-engine[master]: restapi: Refactored BackendGlusterBrickResource

2013-04-05 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: restapi: Refactored BackendGlusterBrickResource .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13120 To unsubscr

[Engine-patches] Change in ovirt-guest-agent[master]: agent: Ignore usbfs filesystems'

2013-04-05 Thread vfeenstr
Vinzenz Feenstra has posted comments on this change. Change subject: agent: Ignore usbfs filesystems' .. Patch Set 2: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13629 To

[Engine-patches] Change in ovirt-engine[master]: packaging: setup plugins should not be executable

2013-04-05 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: setup plugins should not be executable .. Patch Set 1: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ov

[Engine-patches] Change in ovirt-guest-agent[master]: agent: Ignore usbfs filesystems'

2013-04-05 Thread vfeenstr
Vinzenz Feenstra has uploaded a new change for review. Change subject: agent: Ignore usbfs filesystems' .. agent: Ignore usbfs filesystems' Change-Id: Iee799b05f3c700e8e3912305e2984dad6e049f7d Bug-Url: https://bugzilla.redhat.co

[Engine-patches] Change in ovirt-engine[master]: webadmin: Console connect button availability fix

2013-04-05 Thread fkobzik
Frank Kobzik has posted comments on this change. Change subject: webadmin: Console connect button availability fix .. Patch Set 2: No score -- To view, visit http://gerrit.ovirt.org/13343 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-se

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-serv

[Engine-patches] Change in ovirt-engine[master]: packaging: engine-service: major cleanup

2013-04-05 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: engine-service: major cleanup .. Patch Set 1: (1 inline comment) File packaging/fedora/engine-