[Engine-patches] Change in ovirt-engine[master]: userportal:Spice Option:Pass Ctrl-Alt-Del to VM is grayed ou...

2012-10-29 Thread fkobzik
Frank Kobzik has posted comments on this change. Change subject: userportal:Spice Option:Pass Ctrl-Alt-Del to VM is grayed out on windows client .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/8906 To unsubs

[Engine-patches] Change in ovirt-engine[master]: engine: User Portal - auto-connect selected only works once ...

2012-10-29 Thread fkobzik
Frank Kobzik has posted comments on this change. Change subject: engine: User Portal - auto-connect selected only works once per browser session .. Patch Set 6: Verified -- To view, visit http://gerrit.ovirt.org/8851 To unsubs

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

2012-10-29 Thread dgopal
Dhandapani Gopal has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 20: (2 inline comments) Incorporated review comments.

[Engine-patches] Change in ovirt-engine[master]: core: Dead store to tempVar in GetTagsByUserIdQuery.executeQ...

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Dead store to tempVar in GetTagsByUserIdQuery.executeQueryCommand() .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit htt

[Engine-patches] Change in ovirt-engine[master]: engine: Refresh gluster data periodically

2012-10-29 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 20: (1 inline comment) New patch-set to follow, incorporating Allon's comments.

[Engine-patches] Change in ovirt-log-collector[master]: tools: Fix locale issue (BZ858918)

2012-10-29 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: tools: Fix locale issue (BZ858918) .. Patch Set 1: Why not? -- To view, visit http://gerrit.ovirt.org/8919 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-log-collector[master]: tools: Fix locale issue (BZ858918)

2012-10-29 Thread kroberts
Keith Robertson has uploaded a new change for review. Change subject: tools: Fix locale issue (BZ858918) .. tools: Fix locale issue (BZ858918) Removed md5sum check as it isn't really necessary. Change-Id: Iaa74a561bcc2e585e73d0

[Engine-patches] Change in ovirt-engine[master]: webadmin: Fit New Storage Domain dialog for French locale

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Fit New Storage Domain dialog for French locale .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8785 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: better log when LUN in problem state

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: core: better log when LUN in problem state .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8880 To unsubs

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout - Fit Cluster Networking for JP Locale - V2

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Layout - Fit Cluster Networking for JP Locale - V2 .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment) Changes on AbstractVali

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: fit Pools general sub-tab for JP locale

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Layout: fit Pools general sub-tab for JP locale .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8891 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: fit Manage-Events for JP Locale

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Layout: fit Manage-Events for JP Locale .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8902 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout - make template dialog overlap with JP loca...

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Layout - make template dialog overlap with JP locale .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8886

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add checkbox to override NFS mount options

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Add checkbox to override NFS mount options .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8798 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: core, restapi: Convert NFS version to enum

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: core, restapi: Convert NFS version to enum .. Patch Set 8: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8797 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core, webadmin: Use NFS v3 as default

2012-10-29 Thread derez
Daniel Erez has posted comments on this change. Change subject: core, webadmin: Use NFS v3 as default .. Patch Set 12: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8242 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: packaging: Updated VDC utilities to accept alternative binar...

2012-10-29 Thread alourie
Alex Lourie has posted comments on this change. Change subject: packaging: Updated VDC utilities to accept alternative binary .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/8913 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-engine[master]: core: Dead store to tempVar in GetTagsByUserIdQuery.executeQ...

2012-10-29 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: Dead store to tempVar in GetTagsByUserIdQuery.executeQueryCommand() .. Patch Set 2: Verified +2 for patchset 1. No code changes in patchset 2. -- To view

[Engine-patches] Change in ovirt-engine[master]: core, restapi: Convert NFS version to enum

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core, restapi: Convert NFS version to enum .. Patch Set 8: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8797 To uns

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add checkbox to override NFS mount options

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin: Add checkbox to override NFS mount options .. Patch Set 8: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8

[Engine-patches] Change in ovirt-engine[master]: core, webadmin: Use NFS v3 as default

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core, webadmin: Use NFS v3 as default .. Patch Set 12: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8242 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: packaging: Updated VDC utilities to accept alternative binar...

2012-10-29 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Updated VDC utilities to accept alternative binary .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: packaging: Updated VDC utilities to accept alternative binar...

2012-10-29 Thread alourie
Alex Lourie has posted comments on this change. Change subject: packaging: Updated VDC utilities to accept alternative binary .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8913 To unsubscribe, visit http://

[Engine-patches] Change in ovirt-engine[master]: core: Dead store to tempVar in GetTagsByUserIdQuery.executeQ...

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Dead store to tempVar in GetTagsByUserIdQuery.executeQueryCommand() .. Patch Set 1: Looks good to me, but someone else must approve (1 inline comment) s

[Engine-patches] Change in ovirt-engine[master]: engine: Refresh gluster data periodically

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 21: patchset 21: +1 on adressing my comments from previous patches. I'm not giving +1 since I'm not sure

[Engine-patches] Change in ovirt-engine[master]: engine: Refresh gluster data periodically

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 20: (2 inline comments) patchset 20: answered questions inline .

[Engine-patches] Change in ovirt-engine[master]: core: Renamed POSIXFSStorageHelper.java to PosixFsStorageHel...

2012-10-29 Thread Sharad Mishra
Sharad Mishra has abandoned this change. Change subject: core: Renamed POSIXFSStorageHelper.java to PosixFsStorageHelper.java .. Patch Set 1: Abandoned classs cannot be renamed as it is a created using reflection based on Sto

[Engine-patches] Change in ovirt-engine[master]: engine: Refresh gluster data periodically

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 21: (1 inline comment) answered inline File backend

[Engine-patches] Change in ovirt-engine[master]: core: Alert for deprecated NFS mount options in vdsm.conf

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Alert for deprecated NFS mount options in vdsm.conf .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: Fix run VM with locked disk msg

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Fix run VM with locked disk msg .. Patch Set 5: I would prefer that you didn't submit this (4 inline comments) ..

[Engine-patches] Change in ovirt-engine[master]: core: Fix err msg - disk locked on MoveOrCopyDisk

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Fix err msg - disk locked on MoveOrCopyDisk .. Patch Set 6: Looks good to me, but someone else must approve (1 inline comment) Minor style issue, see in

[Engine-patches] Change in ovirt-engine[master]: core: WIP: verify iscsi lun connection when adding lun disk

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: WIP: verify iscsi lun connection when adding lun disk .. Patch Set 4: (2 inline comments) File backe

[Engine-patches] Change in ovirt-engine[master]: core: better log when LUN in problem state

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: better log when LUN in problem state .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8880 To uns

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add checkbox to override NFS mount options

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin: Add checkbox to override NFS mount options .. Patch Set 7: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8

[Engine-patches] Change in ovirt-engine[master]: core, webadmin: Use NFS v3 as default

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core, webadmin: Use NFS v3 as default .. Patch Set 11: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8242 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: core, restapi: Convert NFS version to enum

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core, restapi: Convert NFS version to enum .. Patch Set 7: Looks good to me, but someone else must approve (1 inline comment) See minor inline comment regardin

[Engine-patches] Change in ovirt-engine[master]: core: Use template0 instead of template1

2012-10-29 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: core: Use template0 instead of template1 .. core: Use template0 instead of template1 This change was already initiated in commit 9538658ab884fc1d2e5a78fdd63a3e

[Engine-patches] Change in ovirt-engine[master]: core: Add scan domain query (revised)

2012-10-29 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Add scan domain query (revised) .. Patch Set 2: Looks good to me, but someone else must approve (1 inline comment) Look OK to me - just remove the redund

[Engine-patches] Change in ovirt-engine[master]: webadin: layout: fix english new/edit pool layout

2012-10-29 Thread oourfali
Oved Ourfali has submitted this change and it was merged. Change subject: webadin: layout: fix english new/edit pool layout .. webadin: layout: fix english new/edit pool layout revert patch 8a0c9f4b0d8b753a80a9c38ff04680c78db83

[Engine-patches] Change in ovirt-engine[master]: webadin: layout: fix english new/edit pool layout

2012-10-29 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: webadin: layout: fix english new/edit pool layout .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8915 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: webadin: layout: fix english new/edit pool layout

2012-10-29 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: webadin: layout: fix english new/edit pool layout .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8915 To unsubscri

[Engine-patches] Change in ovirt-engine[master]: webadin: layout: fix english new/edit pool layout

2012-10-29 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: webadin: layout: fix english new/edit pool layout .. webadin: layout: fix english new/edit pool layout revert patch 8a0c9f4b0d8b753a80a9c38ff04680c78db83b0c Ch

[Engine-patches] Change in ovirt-engine[master]: webadmin: Fix localization issue in edit quota

2012-10-29 Thread omasad
ofri masad has posted comments on this change. Change subject: webadmin: Fix localization issue in edit quota .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8914 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: webadmin: Fix localization issue in edit quota

2012-10-29 Thread omasad
ofri masad has uploaded a new change for review. Change subject: webadmin: Fix localization issue in edit quota .. webadmin: Fix localization issue in edit quota Edit Quota popup view had some hard coded strings which are imposi

[Engine-patches] Change in ovirt-engine[master]: packaging: Updated VDC utilities to accept alternative binar...

2012-10-29 Thread alourie
Alex Lourie has uploaded a new change for review. Change subject: packaging: Updated VDC utilities to accept alternative binary .. packaging: Updated VDC utilities to accept alternative binary * Added the optional parameter for

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has abandoned this change. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. Patch Set 5: Abandoned abandoned for the moment. -- To view, visit http://gerrit.ovirt.org/8911 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has restored this change. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. Patch Set 4: Restored -- To view, visit http://gerrit.ovirt.org/8911 To unsubscribe, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has abandoned this change. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. Patch Set 4: Abandoned -- To view, visit http://gerrit.ovirt.org/8911 To unsubscribe, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: restapi: RFE: add smartcard support

2012-10-29 Thread tjelinek
Tomas Jelinek has submitted this change and it was merged. Change subject: restapi: RFE: add smartcard support .. restapi: RFE: add smartcard support Added boolean flag "smartcard_enabled" to VM and template Change-Id: Ibddf2c

[Engine-patches] Change in ovirt-engine[master]: restapi: RFE: add smartcard support

2012-10-29 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: restapi: RFE: add smartcard support .. Patch Set 5: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/8512 To unsubscribe,

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 12 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/12/ Build Number: 12 Build Status: Still Failing Triggered By: Started by user Eyal Edri - Changes Since Last Success: --

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 11 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/11/ Build Number: 11 Build Status: Still Failing Triggered By: Started by user Eyal Edri - Changes Since Last Success: --

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 10 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/10/ Build Number: 10 Build Status: Still Failing Triggered By: Started by user Eyal Edri - Changes Since Last Success: --

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: Fix JP Locale date string format

2012-10-29 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: Layout: Fix JP Locale date string format .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8912 To unsubscribe, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: Fix JP Locale date string format

2012-10-29 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: webadmin: Layout: Fix JP Locale date string format .. webadmin: Layout: Fix JP Locale date string format Change-Id: I2374ec9cd97747646c3a94ad926b73e12f0746bd Bu

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. Patch Set 14: this patch is abandoned as a new solution was sent upstream which has too many cha

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has uploaded a new change for review. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. core: WIP: prevent simultaneous reconstructs on the same pool ReconsturctMasterDomain command ca

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has abandoned this change. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. Patch Set 14: Abandoned -- To view, visit http://gerrit.ovirt.org/7982 To unsubscribe, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has abandoned this change. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. Patch Set 2: Abandoned -- To view, visit http://gerrit.ovirt.org/8910 To unsubscribe, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: core: add toString method to VM and VDS

2012-10-29 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: add toString method to VM and VDS .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7640 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: engine: Integrate smartcard support

2012-10-29 Thread tjelinek
Tomas Jelinek has submitted this change and it was merged. Change subject: engine: Integrate smartcard support .. engine: Integrate smartcard support This patch integrates the smartcard support into the frontend and engine. Th

[Engine-patches] Change in ovirt-engine[master]: engine: Integrate smartcard support

2012-10-29 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: engine: Integrate smartcard support .. Patch Set 12: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8484 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: engine: Integrate smartcard support

2012-10-29 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: engine: Integrate smartcard support .. Patch Set 11: Verified; Looks good to me, approved rebased -- To view, visit http://gerrit.ovirt.org/8484 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: WIP: prevent simultaneous reconstructs on the same poo...

2012-10-29 Thread laravot
Liron Aravot has uploaded a new change for review. Change subject: core: WIP: prevent simultaneous reconstructs on the same pool .. core: WIP: prevent simultaneous reconstructs on the same pool ReconsturctMasterDomain command ca

[Engine-patches] Change in ovirt-engine[master]: webadmin: Add checkbox to override NFS mount options

2012-10-29 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: webadmin: Add checkbox to override NFS mount options .. Patch Set 7: Verified Rebased, no other changes -- To view, visit http://gerrit.ovirt.org/8798 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: core, webadmin: Use NFS v3 as default

2012-10-29 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: core, webadmin: Use NFS v3 as default .. Patch Set 11: Verified Rebased, no other changes -- To view, visit http://gerrit.ovirt.org/8242 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: core, restapi: Convert NFS version to enum

2012-10-29 Thread gpadgett
Greg Padgett has posted comments on this change. Change subject: core, restapi: Convert NFS version to enum .. Patch Set 7: Verified Ori, thanks, updated with your input. -- To view, visit http://gerrit.ovirt.org/8797 To unsub

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 9 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/9/ Build Number: 9 Build Status: Still Failing Triggered By: Started by an SCM change - Changes Since Last Success: -

[Engine-patches] Change in ovirt-engine[master]: core: removing unused static class

2012-10-29 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: removing unused static class .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8877 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: backup.sh script should allow (#866963)

2012-10-29 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: backup.sh script should allow (#866963) .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/ To u

[Engine-patches] Change in ovirt-engine[master]: core: add marks for key-value in sysprep handler log (#87010...

2012-10-29 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: add marks for key-value in sysprep handler log (#870100) .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: userportal:Spice Option:Pass Ctrl-Alt-Del to VM is grayed ou...

2012-10-29 Thread fkobzik
Frank Kobzik has uploaded a new change for review. Change subject: userportal:Spice Option:Pass Ctrl-Alt-Del to VM is grayed out on windows client .. userportal:Spice Option:Pass Ctrl-Alt-Del to VM is grayed out on windows clien

[Engine-patches] Change in ovirt-engine[master]: engine: Type Safety GetVmTemplatesByNetworkIdQuery

2012-10-29 Thread msalem
Muli Salem has submitted this change and it was merged. Change subject: engine: Type Safety GetVmTemplatesByNetworkIdQuery .. engine: Type Safety GetVmTemplatesByNetworkIdQuery This patch adds type safety for GetVmTemplatesAndN

[Engine-patches] Change in ovirt-engine[master]: engine: Type Safety GetVmTemplatesByNetworkIdQuery

2012-10-29 Thread msalem
Muli Salem has posted comments on this change. Change subject: engine: Type Safety GetVmTemplatesByNetworkIdQuery .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8903 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: core: Quota refactor - QuotaManager step 4

2012-10-29 Thread omasad
ofri masad has posted comments on this change. Change subject: core: Quota refactor - QuotaManager step 4 .. Patch Set 5: (1 inline comment) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-engine[master]: engine: Type Safety GetVmTemplatesByNetworkIdQuery

2012-10-29 Thread masayag
Moti Asayag has posted comments on this change. Change subject: engine: Type Safety GetVmTemplatesByNetworkIdQuery .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8903 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Quota refactor - QuotaManager step 2

2012-10-29 Thread omasad
ofri masad has posted comments on this change. Change subject: core: Quota refactor - QuotaManager step 2 .. Patch Set 5: (6 inline comments) File backend/manager/modules/bl

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: fit Manage-Events for JP Locale

2012-10-29 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: webadmin: Layout: fit Manage-Events for JP Locale .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8902 T

[Engine-patches] Change in ovirt-engine[master]: core: WIP: verify iscsi lun connection when adding lun disk

2012-10-29 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: WIP: verify iscsi lun connection when adding lun disk .. Patch Set 4: (1 inline comment) File backend/

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: fit Manage-Events for JP Locale

2012-10-29 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: webadmin: Layout: fit Manage-Events for JP Locale .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8902 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: engine: Type Safety GetVmTemplatesByNetworkIdQuery

2012-10-29 Thread msalem
Muli Salem has uploaded a new change for review. Change subject: engine: Type Safety GetVmTemplatesByNetworkIdQuery .. engine: Type Safety GetVmTemplatesByNetworkIdQuery This patch adds type safety for GetVmTemplatesAndNetworkIn

[Engine-patches] Change in ovirt-engine[master]: core: WIP: verify iscsi lun connection when adding lun disk

2012-10-29 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: WIP: verify iscsi lun connection when adding lun disk .. Patch Set 4: (1 inline comment) File backend/

[Engine-patches] Change in ovirt-engine[master]: webadmin: Layout: fit Manage-Events for JP Locale

2012-10-29 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: webadmin: Layout: fit Manage-Events for JP Locale .. webadmin: Layout: fit Manage-Events for JP Locale Change-Id: I82d22a02149750a92fe1fa0c92d15153a9ff9526 Bug-

[Engine-patches] Change in ovirt-engine[master]: core: Quota refactor - QuotaManager step 1

2012-10-29 Thread omasad
ofri masad has posted comments on this change. Change subject: core: Quota refactor - QuotaManager step 1 .. Patch Set 5: (2 inline comments) File backend/manager/modules/bl

[Engine-patches] Change in ovirt-engine[master]: core: Quota refactor - QuotaManager step 3

2012-10-29 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Quota refactor - QuotaManager step 3 .. Patch Set 5: Locking explained. I wonder if synchronized isn't an overkill on the values of the map (perhaps to i

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

2012-10-29 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: engine: Get Volume Advanced Details Query .. Patch Set 20: (2 inline comments) File backend/manager/module

[Engine-patches] Change in ovirt-engine[master]: core: Add scan domain query (revised)

2012-10-29 Thread ricky . hopper
Ricky Hopper has abandoned this change. Change subject: core: Add scan domain query (revised) .. Patch Set 1: Abandoned Code has been merged into the last incarnation of the patch. Thanks for the help, Allon! -- To view, visi

[Engine-patches] Change in ovirt-reports[master]: reports: fixed date display on br13 (#870720)

2012-10-29 Thread ydary
Yaniv Dary has posted comments on this change. Change subject: reports: fixed date display on br13 (#870720) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8895 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-reports[master]: reports: fixed date display on br13 (#870720)

2012-10-29 Thread ydary
Yaniv Dary has submitted this change and it was merged. Change subject: reports: fixed date display on br13 (#870720) .. reports: fixed date display on br13 (#870720) Had missing field in date display that caused error. Changed

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 8 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
+ rm -rf /home/jenkins/workspace/ovirt-mom_create_rpms/rpms/ + rm -rf /home/jenkins/workspace/ovirt-mom_create_rpms/tarball ++ git log -1 --pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile ++ sed '

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 7 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
/workspace/ovirt-mom_create_rpms/rpms/ + rm -rf /home/jenkins/workspace/ovirt-mom_create_rpms/tarball ++ git log -1 --pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile egrep: Makefile: No such file or direct

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 6 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
/jenkins/workspace/ovirt-mom_create_rpms/tarball ++ git log -1 --pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile egrep: Makefile: No such file or directory ++ sed 's/^.*=//' + CURRENT_RELEASE= +

[Engine-patches] Change in ovirt-engine[master]: core: WIP: verify iscsi lun connection when adding lun disk

2012-10-29 Thread laravot
Liron Aravot has uploaded a new change for review. Change subject: core: WIP: verify iscsi lun connection when adding lun disk .. core: WIP: verify iscsi lun connection when adding lun disk When adding a lun disk there's no vali

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 5 - Failure!

2012-10-29 Thread Jenkins oVirt Server
/home/jenkins/workspace/ovirt-mom_create_rpms/tarball ++ git log -1 --pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile egrep: Makefile: No such file or directory ++ sed 's/^.*=//' + CURRENT_RELEASE= +

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 4 - Fixed!

2012-10-29 Thread Jenkins oVirt Server
/home/jenkins/workspace/ovirt-mom_create_rpms/rpms/ + rm -rf /home/jenkins/workspace/ovirt-mom_create_rpms/tarball ++ git log -1 --pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile ++ sed 's/^.*=//'

[Engine-patches] Change in ovirt-engine[master]: core: Quota refactor - QuotaManager step 4

2012-10-29 Thread omasad
ofri masad has posted comments on this change. Change subject: core: Quota refactor - QuotaManager step 4 .. Patch Set 5: (1 inline comment) File backend/manager/modules/bll

[Engine-patches] Change in ovirt-engine[master]: core: Quota refactor - QuotaManager step 3

2012-10-29 Thread omasad
ofri masad has posted comments on this change. Change subject: core: Quota refactor - QuotaManager step 3 .. Patch Set 5: The locking actually remained the same as in the previous version. but never the less, the locks in this

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 3 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
/workspace/ovirt-mom_create_rpms/dist ++ git log -1 --pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile ++ sed 's/^.*=//' egrep: Makefile: No such file or directory + CURRENT_RELEASE= + RPM_RELEA

[Engine-patches] Change in ovirt-engine[master]: engine: Refresh gluster data periodically

2012-10-29 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 20: (10 inline comments) Thanks Allon for catching some good ones! Responses to comments inline. I'm sen

[Engine-patches] [oVirt Jenkins] ovirt-mom_create_rpms - Build # 2 - Still Failing!

2012-10-29 Thread Jenkins oVirt Server
--pretty=format:%h + git_head=f9f2c5e ++ date --utc +%Y%m%d + GIT_RELEASE=20121029.gitf9f2c5e ++ egrep '^RPM_RELEASE_VERSION' Makefile ++ sed 's/^.*=//' egrep: Makefile: No such file or directory + CURRENT_RELEASE= + RPM_RELEASE_VERSION=.20121029.gitf9f2c5e + mkdir -p /home/jen

  1   2   >