[Engine-patches] Change in ovirt-engine[master]: restapi: allow client to define the HTTP session TTL #921013

2013-03-14 Thread mpastern
Michael Pasternak has submitted this change and it was merged. Change subject: restapi: allow client to define the HTTP session TTL #921013 .. restapi: allow client to define the HTTP session TTL #921013 Session TTL can be expl

[Engine-patches] Change in ovirt-engine[master]: restapi: allow client to define the HTTP session TTL #921013

2013-03-14 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: allow client to define the HTTP session TTL #921013 .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1

[Engine-patches] Change in ovirt-engine[master]: restapi: allow client to define the HTTP session TTL #921013

2013-03-14 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: allow client to define the HTTP session TTL #921013 .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1

[Engine-patches] Change in ovirt-engine[master]: gluster: added cluster level check

2013-03-14 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: gluster: added cluster level check .. Patch Set 5: (1 inline comment) Response to Omer's comment in-line.

[Engine-patches] Change in ovirt-engine[master]: gluster: Added ports in gluster iptables config

2013-03-14 Thread sabose
Sahina Bose has posted comments on this change. Change subject: gluster: Added ports in gluster iptables config .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/12974 To

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: spec: gettext-devel is not required if gettext is emb...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: spec: gettext-devel is not required if gettext is embedded .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: spec: gettext-devel is not required if gettext is emb...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: spec: gettext-devel is not required if gettext is embedded .. build: spec: gettext-devel is not required if gettext is embedded Change-Id: I8df67

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: spec: gettext-devel is not required if gettext is emb...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: spec: gettext-devel is not required if gettext is embedded .. build: spec: gettext-devel is not required if gettext is embedded Change-Id: I8df67f285dc15

[Engine-patches] Change in ovirt-log-collector[master]: cleanup: update .gitignore

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: cleanup: update .gitignore .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13083 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit

[Engine-patches] Change in ovirt-log-collector[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use embedded gettext .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13082 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerri

[Engine-patches] Change in ovirt-log-collector[master]: cleanup: update .gitignore

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: cleanup: update .gitignore .. cleanup: update .gitignore Change-Id: If42ce8d6e2106c529576aa6171a0f57364e14869 Signed-off-by: Alon Bar-Lev --- M .gitignore 1 fil

[Engine-patches] Change in ovirt-log-collector[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use embedded gettext .. build: use embedded gettext there are too many variants of version within different distributions for us to support it externally

[Engine-patches] Change in ovirt-image-uploader[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use embedded gettext .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13081 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerri

[Engine-patches] Change in ovirt-image-uploader[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use embedded gettext .. build: use embedded gettext there are too many variants of version within different distributions for us to support it externally

[Engine-patches] Change in ovirt-engine[master]: engine: unit test for VdsDeploy.getEngineSSHPublicKey

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: unit test for VdsDeploy.getEngineSSHPublicKey .. Patch Set 4: http://gerrit.ovirt.org/#/c/13080/ -- To view, visit http://gerrit.ovirt.org/12719 To unsub

[Engine-patches] Change in ovirt-engine[master]: Partial Revert "engine: unit test for VdsDeploy.getEngineSSH...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: Partial Revert "engine: unit test for VdsDeploy.getEngineSSHPublicKey" .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13080 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: VdsDeploy cleanup 1 .. Patch Set 3: http://gerrit.ovirt.org/#/c/13079/ -- To view, visit http://gerrit.ovirt.org/12972 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: Revert "engine: VdsDeploy cleanup 1"

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: Revert "engine: VdsDeploy cleanup 1" .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13079 To unsubscribe, visit http://gerrit.ovirt.org/settin

[Engine-patches] Change in ovirt-engine[master]: Revert "engine: VdsDeploy cleanup 1"

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: Revert "engine: VdsDeploy cleanup 1" .. Revert "engine: VdsDeploy cleanup 1" This reverts commit ee531b788e880e60cae87d82c3553239f2eb7778. Change-Id: I1f964a047

[Engine-patches] Change in ovirt-engine[master]: Partial Revert "engine: unit test for VdsDeploy.getEngineSSH...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: Partial Revert "engine: unit test for VdsDeploy.getEngineSSHPublicKey" .. Partial Revert "engine: unit test for VdsDeploy.getEngineSSHPublicKey" This reverts co

[Engine-patches] Change in ovirt-iso-uploader[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use embedded gettext .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/13078 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerri

[Engine-patches] Change in ovirt-iso-uploader[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use embedded gettext .. build: use embedded gettext there are too many variants of version within different distributions for us to support it externally

[Engine-patches] Change in ovirt-engine[master]: engine: unit test for VdsDeploy.getEngineSSHPublicKey

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: unit test for VdsDeploy.getEngineSSHPublicKey .. Patch Set 4: guys, how come you don't CC module owner? You cannot just go ahead and commit stuff for som

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: VdsDeploy cleanup 1 .. Patch Set 3: guys, how come you don't CC module owner? You cannot just go ahead and commit stuff for something you don't own. PLE

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 2

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: VdsDeploy cleanup 2 .. Patch Set 3: guys, how come you don't CC module owner? You cannot just go ahead and commit stuff for something you don't own. --

[Engine-patches] Change in otopi[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use embedded gettext .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13077 To unsubscribe, visit http://gerr

[Engine-patches] Change in otopi[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: use embedded gettext .. build: use embedded gettext Change-Id: I54f5a254a3a4669ab7f8b363fa81dbe0266f5cc7 Signed-off-by: Alon Bar-Lev --- M .gitig

[Engine-patches] Change in otopi[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use embedded gettext .. build: use embedded gettext Change-Id: I54f5a254a3a4669ab7f8b363fa81dbe0266f5cc7 Signed-off-by: Alon Bar-Lev --- M .gitignore M M

[Engine-patches] Change in ovirt-host-deploy[master]: build: spec: gettext-devel is not required if gettext is emb...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: spec: gettext-devel is not required if gettext is embedded .. build: spec: gettext-devel is not required if gettext is embedded Change-Id: I8df67

[Engine-patches] Change in ovirt-host-deploy[master]: build: spec: gettext-devel is not required if gettext is emb...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: spec: gettext-devel is not required if gettext is embedded .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-host-deploy[master]: build: spec: gettext-devel is not required if gettext is emb...

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: spec: gettext-devel is not required if gettext is embedded .. build: spec: gettext-devel is not required if gettext is embedded Change-Id: I8df67f285dc15

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove hide popup feature.

2013-03-14 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: remove hide popup feature. .. Patch Set 3: Verified Rebase to latest patches. -- To view, visit http://gerrit.ovirt.org/12023 To unsubscribe, visit h

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post ovirt-host-deploy-1.0.1 .. packaging: post ovirt-host-deploy-1.0.1 Change-Id: I6c859e739bacdc7a8cb54f7cf057c576dfdc8734 Signed-off-by: Al

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: post ovirt-host-deploy-1.0.1 .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13075 To unsubscribe, visit

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: ovirt-host-deploy-1.0.1 .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13074 To unsubscribe, visit http

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: ovirt-host-deploy-1.0.1 .. packaging: ovirt-host-deploy-1.0.1 Change-Id: I22b1df5505bfa6d3075bc3d52def7bb8cf2dc79a Signed-off-by: Alon Bar-Lev

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: use embedded gettext .. build: use embedded gettext The difference between environments is so large that best to provide gettext within tarball.

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use embedded gettext .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13073 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: post ovirt-host-deploy-1.0.1 .. packaging: post ovirt-host-deploy-1.0.1 Change-Id: I6c859e739bacdc7a8cb54f7cf057c576dfdc8734 Signed-off-by: Alon Bar-L

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use embedded gettext .. build: use embedded gettext The difference between environments is so large that best to provide gettext within tarball. Change-I

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: ovirt-host-deploy-1.0.1 .. packaging: ovirt-host-deploy-1.0.1 Change-Id: I22b1df5505bfa6d3075bc3d52def7bb8cf2dc79a Signed-off-by: Alon Bar-Lev --- M

[Engine-patches] Change in ovirt-engine[master]: webadmin: grids jump to last selected entity

2013-03-14 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: grids jump to last selected entity .. Patch Set 2: Verified Addressed the concerns of the previous patch. -- To view, visit http://gerrit.ovirt.org/1

[Engine-patches] Change in ovirt-engine[master]: webadmin: remove hide popup feature.

2013-03-14 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: webadmin: remove hide popup feature. .. Patch Set 2: Alex - please rebase (to eliminate outdated dependency) Vojtech - any reason for not giving this patch "+2"?

[Engine-patches] Change in ovirt-engine[master]: webadmin: grids jump to last selected entity

2013-03-14 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: grids jump to last selected entity .. Patch Set 1: (2 inline comments) File frontend/webadmin/mo

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

2013-03-14 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: core: provided cleared frontend logging .. Patch Set 5: Yair - patch is pending only on your re-review. thanks. -- To view, visit http://gerrit.ovirt.org/12559 To

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

2013-03-14 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: Application continue polling after logout .. Patch Set 8: Vojtech, no that statement actually was never relevant. I mistook the entities that were ma

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

2013-03-14 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: Application continue polling after logout .. Patch Set 7: (2 inline comments) File frontend/weba

[Engine-patches] Change in ovirt-host-deploy[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: use embedded gettext .. build: use embedded gettext The difference between environments is so large that best to provide gettext within tarball.

[Engine-patches] Change in ovirt-host-deploy[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use embedded gettext .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13072 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-host-deploy[master]: build: use embedded gettext

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use embedded gettext .. build: use embedded gettext The difference between environments is so large that best to provide gettext within tarball. Change-I

[Engine-patches] Change in ovirt-engine[master]: webadmin: enable gzip compression

2013-03-14 Thread awels
Alexander Wels has posted comments on this change. Change subject: webadmin: enable gzip compression .. Patch Set 9: Verified Fixed merge conflict. -- To view, visit http://gerrit.ovirt.org/10911 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: webadmin: load image refreshing data

2013-03-14 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: webadmin: load image refreshing data .. Patch Set 2: Vojtech/Alex - what do you think? Derez has raised an interesting point here: The patches seem to do pretty m

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: post ovirt-host-deploy-1.0.1 .. packaging: post ovirt-host-deploy-1.0.1 Change-Id: I044a8741f29e4fadd3f79e4552b61e74fab2b6ad Signed-off-by: Al

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: post ovirt-host-deploy-1.0.1 .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13071 To unsubscribe, visit

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: ovirt-host-deploy-1.0.1 .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13070 To unsubscribe, visit http

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: ovirt-host-deploy-1.0.1 .. packaging: ovirt-host-deploy-1.0.1 Change-Id: I7b50425f76087c880f364779f538b05e93a45bd1 Signed-off-by: Alon Bar-Lev

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: ovirt-host-deploy-1.0.1 .. packaging: ovirt-host-deploy-1.0.1 Change-Id: I7b50425f76087c880f364779f538b05e93a45bd1 Signed-off-by: Alon Bar-Lev --- M

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: packaging: post ovirt-host-deploy-1.0.1

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: post ovirt-host-deploy-1.0.1 .. packaging: post ovirt-host-deploy-1.0.1 Change-Id: I044a8741f29e4fadd3f79e4552b61e74fab2b6ad Signed-off-by: Alon Bar-L

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

2013-03-14 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: webadmin: Select english when browser is other locale .. Patch Set 4: Thanks Vojtech. Alona - would be great if you can re-review the updated patch. Thanks. --

[Engine-patches] Change in ovirt-engine[master]: webadmin: enable gzip compression

2013-03-14 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: webadmin: enable gzip compression .. Patch Set 8: Alex: Due to some changes introduced by http://gerrit.ovirt.org/#/c/12411/, the patch is not rebasing smoothly (

[Engine-patches] Change in ovirt-engine[master]: core: unify the GetAllImagesList queries

2013-03-14 Thread fsimonce
Federico Simoncelli has posted comments on this change. Change subject: core: unify the GetAllImagesList queries .. Patch Set 1: > Should we call it GetImagesList instead of GetAllImagesList? Yes, I'll take care of it. -- To

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: doc: pki: update environment documentation

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: doc: pki: update environment documentation .. doc: pki: update environment documentation Change-Id: Id7c8fde4165ec85940d2f05ea53570f63813f011 Signed-off-

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: doc: pki: update environment documentation

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: doc: pki: update environment documentation .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13069 To unsubscribe, vi

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: doc: pki: update environment documentation

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: doc: pki: update environment documentation .. doc: pki: update environment documentation Change-Id: Id7c8fde4165ec85940d2f05ea53570f63813f011 Signed-off-by: Alon

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: use older gettext-0.17

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use older gettext-0.17 .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13068 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: use older gettext-0.17

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: use older gettext-0.17 .. build: use older gettext-0.17 For old rhel/centos hosts Change-Id: I0c90c34450961e4388b760982ddc002cca040359 Signed-off

[Engine-patches] Change in ovirt-host-deploy[ovirt-host-deploy-1.0]: build: use older gettext-0.17

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use older gettext-0.17 .. build: use older gettext-0.17 For old rhel/centos hosts Change-Id: I0c90c34450961e4388b760982ddc002cca040359 Signed-off-by: Alo

[Engine-patches] Change in ovirt-engine[master]: core: unify the GetAllImagesList queries

2013-03-14 Thread rnori
Ravi Nori has posted comments on this change. Change subject: core: unify the GetAllImagesList queries .. Patch Set 1: No score Should we call it GetImagesList instead of GetAllImagesList? -- To view, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: core: rename FileTypeExtension to ImageType

2013-03-14 Thread rnori
Ravi Nori has posted comments on this change. Change subject: core: rename FileTypeExtension to ImageType .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13067 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: unify the GetAllImagesList queries

2013-03-14 Thread rnori
Ravi Nori has posted comments on this change. Change subject: core: unify the GetAllImagesList queries .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13066 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: core: rename FileTypeExtension to ImageType

2013-03-14 Thread fsimonce
Federico Simoncelli has uploaded a new change for review. Change subject: core: rename FileTypeExtension to ImageType .. core: rename FileTypeExtension to ImageType Moving into the direction of supporting image storage domain th

[Engine-patches] Change in ovirt-engine[master]: core: unify the GetAllImagesList queries

2013-03-14 Thread fsimonce
Federico Simoncelli has uploaded a new change for review. Change subject: core: unify the GetAllImagesList queries .. core: unify the GetAllImagesList queries This patch unifies and consolidates the queries to retrieve the list

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

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Added checking DB state before upgrade .. Patch Set 5: I would prefer that you didn't submit this To make it clear, I put here -1. The dba should mana

[Engine-patches] Change in ovirt-engine[master]: core: add ability edit NFS path in webadmin

2013-03-14 Thread laravot
Liron Ar has posted comments on this change. Change subject: core: add ability edit NFS path in webadmin .. Patch Set 6: (1 inline comment) File backend/manager/modules/bll/

[Engine-patches] Change in ovirt-host-deploy[master]: build: use older gettext-0.17

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has submitted this change and it was merged. Change subject: build: use older gettext-0.17 .. build: use older gettext-0.17 For old rhel/centos hosts Change-Id: I0c90c34450961e4388b760982ddc002cca040359 Signed-off

[Engine-patches] Change in ovirt-host-deploy[master]: build: use older gettext-0.17

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: build: use older gettext-0.17 .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13065 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-host-deploy[master]: build: use older gettext-0.17

2013-03-14 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: build: use older gettext-0.17 .. build: use older gettext-0.17 For old rhel/centos hosts Change-Id: I0c90c34450961e4388b760982ddc002cca040359 Signed-off-by: Alo

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 2

2013-03-14 Thread lhornyak
Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: VdsDeploy cleanup 2 .. engine: VdsDeploy cleanup 2 - Added type parameter to generic types to get rid of warning. Change-Id: Ibfb00c823d42d3a7

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 2

2013-03-14 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: engine: VdsDeploy cleanup 2 .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12973 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 1

2013-03-14 Thread lhornyak
Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: VdsDeploy cleanup 1 .. engine: VdsDeploy cleanup 1 - removed hunngarian notation from method names Change-Id: I8aedeef3aa6a614881ad3a9b7dc46b9

[Engine-patches] Change in ovirt-engine[master]: engine: VdsDeploy cleanup 1

2013-03-14 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: engine: VdsDeploy cleanup 1 .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12972 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: engine: VmDynamic cleanup 1

2013-03-14 Thread lhornyak
Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: VmDynamic cleanup 1 .. engine: VmDynamic cleanup 1 - cleanup in fields, removed hungarian notation, and underscores Change-Id: I9558be538ad585

[Engine-patches] Change in ovirt-engine[master]: engine: VmDynamic cleanup 1

2013-03-14 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: engine: VmDynamic cleanup 1 .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12975 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: engine: Extending migrate verb with an additional parameter-...

2013-03-14 Thread alkaplan
Alona Kaplan has uploaded a new change for review. Change subject: engine: Extending migrate verb with an additional parameter- "dstqemo" .. engine: Extending migrate verb with an additional parameter- "dstqemo" The dstqemo wil

[Engine-patches] Change in ovirt-engine[master]: core: add the image block alignment scan

2013-03-14 Thread laravot
Liron Ar has posted comments on this change. Change subject: core: add the image block alignment scan .. Patch Set 6: (5 inline comments) File backend/manager/modules/bll/sr

[Engine-patches] Change in ovirt-iso-uploader[master]: Fixing format_ssh_command

2013-03-14 Thread kroberts
Keith Robertson has submitted this change and it was merged. Change subject: Fixing format_ssh_command .. Fixing format_ssh_command format_ssh_command failed setting the Identity file causing an error like no such file or direc

[Engine-patches] Change in ovirt-iso-uploader[master]: Fixing format_ssh_command

2013-03-14 Thread kroberts
Keith Robertson has posted comments on this change. Change subject: Fixing format_ssh_command .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13055 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: tools: Setting ClientConsoleModeDefault

2013-03-14 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: tools: Setting ClientConsoleModeDefault .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13002 To unsubs

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

2013-03-14 Thread kroberts
Keith Robertson has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 5: Verify it and I'll push it. -- To view, visit http://gerrit.ovirt.org/10815 To unsubscrib

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

2013-03-14 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: Fixed creating objects in an insecure way .. Patch Set 5: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: engine: Added ValidationResult matchers

2013-03-14 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Added ValidationResult matchers .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/13061 To unsubscr

[Engine-patches] Change in ovirt-engine[master]: core: add ability edit NFS path in webadmin

2013-03-14 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: add ability edit NFS path in webadmin .. Patch Set 6: (16 inline comments) File backend/manager/module

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

2013-03-14 Thread sbonazzo
Sandro Bonazzola has posted comments on this change. Change subject: packaging: Added checking DB state before upgrade .. Patch Set 5: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/12

[Engine-patches] Change in ovirt-engine[master]: webadmin: Show migrating VM on destination host

2013-03-14 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: webadmin: Show migrating VM on destination host .. Patch Set 2: (2 inline comments) File frontend/webadmin/

[Engine-patches] Change in ovirt-engine[master]: core: add ability edit NFS path in webadmin

2013-03-14 Thread laravot
Liron Ar has posted comments on this change. Change subject: core: add ability edit NFS path in webadmin .. Patch Set 6: (2 inline comments) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-engine[master]: core: Added generic message that name is used

2013-03-14 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: core: Added generic message that name is used .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/12361 To u

[Engine-patches] Change in ovirt-engine[master]: engine: unit test for VdsDeploy.getEngineSSHPublicKey

2013-03-14 Thread lhornyak
Laszlo Hornyak has submitted this change and it was merged. Change subject: engine: unit test for VdsDeploy.getEngineSSHPublicKey .. engine: unit test for VdsDeploy.getEngineSSHPublicKey - removed the _ prefix from method name

[Engine-patches] Change in ovirt-engine[master]: engine: unit test for VdsDeploy.getEngineSSHPublicKey

2013-03-14 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: engine: unit test for VdsDeploy.getEngineSSHPublicKey .. Patch Set 4: Verified; Looks good to me, approved thx -- To view, visit http://gerrit.ovirt.org/12719

[Engine-patches] Change in ovirt-engine[master]: engine: Added ValidationResult matchers

2013-03-14 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: engine: Added ValidationResult matchers .. engine: Added ValidationResult matchers The matchers can be used for testing that validation result complies with a g

[Engine-patches] Change in ovirt-engine[master]: engine: Network validator tests use assertThat

2013-03-14 Thread mkolesni
Mike Kolesnik has uploaded a new change for review. Change subject: engine: Network validator tests use assertThat .. engine: Network validator tests use assertThat Refactored the network validator's tests to use assertThat synt

  1   2   3   4   >