[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread oourfali
Oved Ourfali has submitted this change and it was merged. Change subject: userportal: SearchResultsLimit race condition (#853904) .. userportal: SearchResultsLimit race condition (#853904) https://bugzilla.redhat.com/853904 Se

[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: userportal: SearchResultsLimit race condition (#853904) .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8181 To uns

[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: userportal: SearchResultsLimit race condition (#853904) .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8181 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Extracted VmValidator

2012-09-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: Extracted VmValidator .. Patch Set 4: Looks good to me, approved look ok - except that this ValidationResult is a useless wrapper to VdcBllMessage. we can act

[Engine-patches] Change in ovirt-engine[master]: webadmin: Using editable combo for volume option key

2012-09-24 Thread kmayilsa
Kanagaraj M has uploaded a new change for review. Change subject: webadmin: Using editable combo for volume option key .. webadmin: Using editable combo for volume option key In the Volume option dialog, the keys will be shown i

[Engine-patches] Change in ovirt-engine[master]: tools: Handling add first/removing last domain (#858769)

2012-09-24 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: tools: Handling add first/removing last domain (#858769) .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8178 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageTargetToLunList FindBugs

2012-09-24 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: SanStorageTargetToLunList FindBugs .. webadmin: SanStorageTargetToLunList FindBugs Solved FindBugs warning RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageTargetToLunList FindBugs

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: SanStorageTargetToLunList FindBugs .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8177 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageLunToTargetList FindBugs

2012-09-24 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: SanStorageLunToTargetList FindBugs .. webadmin: SanStorageLunToTargetList FindBugs Solved FindBugs warning RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BE

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageLunToTargetList FindBugs

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: SanStorageLunToTargetList FindBugs .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8176 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread derez
Daniel Erez has uploaded a new change for review. Change subject: userportal: SearchResultsLimit race condition (#853904) .. userportal: SearchResultsLimit race condition (#853904) https://bugzilla.redhat.com/853904 SearchableL

[Engine-patches] Change in ovirt-engine[master]: Revert "userportal: SearchResultsLimit race condition (#8539...

2012-09-24 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: Revert "userportal: SearchResultsLimit race condition (#853904)" .. Revert "userportal: SearchResultsLimit race condition (#853904)" This reverts commit 0

[Engine-patches] Change in ovirt-engine[master]: Revert "userportal: SearchResultsLimit race condition (#8539...

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: Revert "userportal: SearchResultsLimit race condition (#853904)" .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/818

[Engine-patches] Change in ovirt-engine[master]: Revert "userportal: SearchResultsLimit race condition (#8539...

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: Revert "userportal: SearchResultsLimit race condition (#853904)" .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/818

[Engine-patches] Change in ovirt-engine[master]: Revert "userportal: SearchResultsLimit race condition (#8539...

2012-09-24 Thread derez
Daniel Erez has reverted this change. Change subject: Revert "userportal: SearchResultsLimit race condition (#853904)" .. Patch Set 2: Reverted This patchset was reverted in change: Iee499669af30b4b620b9fa9bd86b775b4f9e1eee --

[Engine-patches] Change in ovirt-engine[master]: tools: Handling add first/removing last domain (#858769)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has uploaded a new change for review. Change subject: tools: Handling add first/removing last domain (#858769) .. tools: Handling add first/removing last domain (#858769) https://bugzilla.redhat.com/858769 This p

[Engine-patches] Change in ovirt-image-uploader[master]: tools: Don't print help on error (BZ858968)

2012-09-24 Thread mgoldboi
Moran Goldboim has posted comments on this change. Change subject: tools: Don't print help on error (BZ858968) .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8090 To un

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-09-24 Thread abaron
Ayal Baron has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 3: (1 inline comment) File backend/manager/modules/bl

[Engine-patches] Change in ovirt-engine[master]: packaging: add a method to work with parsed sql output

2012-09-24 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: add a method to work with parsed sql output .. Patch Set 4: Verified OK, a new dependency is added "cvs". Now we parse correctly most of the outputs,

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageTargetToLunList FindBugs

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin: SanStorageTargetToLunList FindBugs .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8177 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageLunToTargetList FindBugs

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin: SanStorageLunToTargetList FindBugs .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8176 To unsubscribe, visit http://gerrit.ovirt.o

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageTargetToLunList FindBugs

2012-09-24 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: webadmin: SanStorageTargetToLunList FindBugs .. webadmin: SanStorageTargetToLunList FindBugs Solved FindBugs warning RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_

[Engine-patches] Change in ovirt-engine[master]: webadmin: SanStorageLunToTargetList FindBugs

2012-09-24 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: webadmin: SanStorageLunToTargetList FindBugs .. webadmin: SanStorageLunToTargetList FindBugs Solved FindBugs warning RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_

[Engine-patches] Change in ovirt-engine[master]: packaging: cleanup: remove unused isDBUp, execSqlCommand

2012-09-24 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: cleanup: remove unused isDBUp, execSqlCommand .. Patch Set 1: Verified Well, verified as in tested no exceptions, and greping usage... -- To view, vis

[Engine-patches] Change in ovirt-engine[master]: engine:Validate maximum number of hosts in DC.(#771699)

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: engine:Validate maximum number of hosts in DC.(#771699) .. Patch Set 12: Looks good to me, but someone else must approve (1 inline comment) ...

[Engine-patches] Change in ovirt-engine[master]: core: throttle running of VMs (#843058)

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: throttle running of VMs (#843058) .. Patch Set 9: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/7204 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: Forbid attaching a locked disk to a VM (#856137)

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Forbid attaching a locked disk to a VM (#856137) .. Patch Set 1: I would prefer that you didn't submit this (2 inline comments) see inline, minor issues

[Engine-patches] Change in ovirt-engine[master]: Use the real parent command for getParametersForTask

2012-09-24 Thread fsimonce
Federico Simoncelli has abandoned this change. Change subject: Use the real parent command for getParametersForTask .. Patch Set 1: Abandoned -- To view, visit http://gerrit.ovirt.org/8025 To unsubscribe, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: core: replace Date with long for time variable

2012-09-24 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: replace Date with long for time variable .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7245 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: fix in error message (#853704)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: fix in error message (#853704) .. core: fix in error message (#853704) Previous patch changed the message, but there was a typo there. Fixing the

[Engine-patches] Change in ovirt-engine[master]: core: fix in error message (#853704)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: fix in error message (#853704) .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8173 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: fix in error message (#853704)

2012-09-24 Thread oourfali
Oved Ourfali has uploaded a new change for review. Change subject: core: fix in error message (#853704) .. core: fix in error message (#853704) Previous patch changed the message, but there was a typo there. Fixing the message.

[Engine-patches] Change in ovirt-engine[master]: packaging: Updated names of etl and notifier services

2012-09-24 Thread alourie
Alex Lourie has uploaded a new change for review. Change subject: packaging: Updated names of etl and notifier services .. packaging: Updated names of etl and notifier services Change-Id: Icea5953c7d233b068056b75955f440b6dd6df32

[Engine-patches] Change in ovirt-engine[master]: core: VmMigrateDiskoComplete VDS command

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: VmMigrateDiskoComplete VDS command .. Patch Set 2: (1 inline comment) Commit Message Line 3: AuthorDa

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-09-24 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 5: Fails (1 inline comment) Frankly, this looks like a considerably less elegant solution to me, but a

[Engine-patches] Change in ovirt-engine[master]: core: using pg_dump --column-inserts ...(#858643)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: using pg_dump --column-inserts ...(#858643) .. core: using pg_dump --column-inserts ...(#858643) using pg_dump --column-inserts slows down the re

[Engine-patches] Change in ovirt-engine[master]: core: using pg_dump --column-inserts ...(#858643)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: using pg_dump --column-inserts ...(#858643) .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/8134 To unsubscribe, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: core: using pg_dump --column-inserts ...(#858643)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: using pg_dump --column-inserts ...(#858643) .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8134 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: core: using pg_dump --column-inserts ...(#858643)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: using pg_dump --column-inserts ...(#858643) .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8134 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: core: task manager cache is not cleared..(#854308)

2012-09-24 Thread emesika
Eli Mesika has submitted this change and it was merged. Change subject: core: task manager cache is not cleared..(#854308) .. core: task manager cache is not cleared..(#854308) task manager cache is not cleared after getting er

[Engine-patches] Change in ovirt-engine[master]: core: task manager cache is not cleared..(#854308)

2012-09-24 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: task manager cache is not cleared..(#854308) .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8161 To unsubscrib

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: defend against disconnected mode when no proxy cached

2012-09-24 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: sdk: defend against disconnected mode when no proxy cached .. sdk: defend against disconnected mode when no proxy cached Change-Id: Id2349669dcbd26b32d877d9

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: remove unused filter_header argument

2012-09-24 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: sdk: remove unused filter_header argument .. sdk: remove unused filter_header argument Change-Id: I8f99f1acbb434ee05c26a9e50c563b9e113fd923 Signed-off-by: M

[Engine-patches] Change in ovirt-engine[master]: core: WIP :prevent simultaneous ReconstructsMasterDomain on ...

2012-09-24 Thread laravot
Liron Aravot has posted comments on this change. Change subject: core: WIP :prevent simultaneous ReconstructsMasterDomain on the same pool(#845838) .. Patch Set 2: (1 inline comment) ...

[Engine-patches] Change in ovirt-engine[master]: core: WIP :prevent simultaneous ReconstructsMasterDomain on ...

2012-09-24 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: WIP :prevent simultaneous ReconstructsMasterDomain on the same pool(#845838) .. Patch Set 2: (1 inline comment) .

[Engine-patches] Change in ovirt-engine[master]: core: throttle running of VMs (#843058)

2012-09-24 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: throttle running of VMs (#843058) .. Patch Set 8: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/7204 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: packaging: updated handling of pgpass file between versions

2012-09-24 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: packaging: updated handling of pgpass file between versions .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovir

[Engine-patches] Change in ovirt-engine[master]: userportal, webadmin: Stop->Power Off (#857327)

2012-09-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: userportal,webadmin: Stop->Power Off (#857327) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8043 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: webadmin: Make HA consistant in VM and template tabs

2012-09-24 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Make HA consistant in VM and template tabs .. webadmin: Make HA consistant in VM and template tabs Problems: * Inconsistent display of the hig

[Engine-patches] Change in ovirt-engine[master]: webadmin: Make HA consistant in VM and template tabs

2012-09-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: Make HA consistant in VM and template tabs .. Patch Set 2: Verified; Looks good to me, approved rebased & verified -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: core: spm and master domain maintenance actions cause wrong ...

2012-09-24 Thread oourfali
Oved Ourfali has submitted this change and it was merged. Change subject: core: spm and master domain maintenance actions cause wrong error (#853704) .. core: spm and master domain maintenance actions cause wrong error (#853704

[Engine-patches] Change in ovirt-engine[master]: core: spm and master domain maintenance actions cause wrong ...

2012-09-24 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: core: spm and master domain maintenance actions cause wrong error (#853704) .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.

[Engine-patches] Change in ovirt-engine[master]: core: spm and master domain maintenance actions cause wrong ...

2012-09-24 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: core: spm and master domain maintenance actions cause wrong error (#853704) .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/8166 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: core: Forbid attaching a locked disk to a VM (#856137)

2012-09-24 Thread asaf
Asaf Shakarchi has uploaded a new change for review. Change subject: core: Forbid attaching a locked disk to a VM (#856137) .. core: Forbid attaching a locked disk to a VM (#856137) - Forbid attaching a locked disk to a VM. - Re

[Engine-patches] Change in ovirt-engine[master]: core: spm and master domain maintenance actions cause wrong ...

2012-09-24 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: spm and master domain maintenance actions cause wrong error (#853704) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: core: Renamed Refresh to refreshModel

2012-09-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: core: Renamed Refresh to refreshModel .. Patch Set 4: Verified; Looks good to me, approved rebased & verified -- To view, visit http://gerrit.ovirt.org/8111 To

[Engine-patches] Change in ovirt-engine[master]: core: Renamed Refresh to refreshModel

2012-09-24 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: core: Renamed Refresh to refreshModel .. core: Renamed Refresh to refreshModel Fixes FindBugs issues - 1. VERY confusing to have methods org.ovirt.eng

[Engine-patches] Change in ovirt-engine[master]: core: spm and master domain maintenance actions cause wrong ...

2012-09-24 Thread oourfali
Oved Ourfali has uploaded a new change for review. Change subject: core: spm and master domain maintenance actions cause wrong error (#853704) .. core: spm and master domain maintenance actions cause wrong error (#853704) When

[Engine-patches] Change in ovirt-engine[master]: webadmin: pass HA value to template from vm

2012-09-24 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: pass HA value to template from vm .. webadmin: pass HA value to template from vm Problems: * When creating tamplate from vm the high availabil

[Engine-patches] Change in ovirt-engine[master]: webadmin: pass HA value to template from vm

2012-09-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: pass HA value to template from vm .. Patch Set 5: Verified; Looks good to me, approved reviewed -- To view, visit http://gerrit.ovirt.org/8154 To unsu

[Engine-patches] Change in ovirt-engine[master]: webadmin: Allow attaching v2 domains for v3 pool (#856166)

2012-09-24 Thread asaf
Asaf Shakarchi has submitted this change and it was merged. Change subject: webadmin: Allow attaching v2 domains for v3 pool (#856166) .. webadmin: Allow attaching v2 domains for v3 pool (#856166) https://bugzilla.redhat.com/85

[Engine-patches] Change in ovirt-engine[master]: webadmin: Allow attaching v2 domains for v3 pool (#856166)

2012-09-24 Thread asaf
Asaf Shakarchi has posted comments on this change. Change subject: webadmin: Allow attaching v2 domains for v3 pool (#856166) .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8091 T

[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread oourfali
Oved Ourfali has submitted this change and it was merged. Change subject: userportal: SearchResultsLimit race condition (#853904) .. userportal: SearchResultsLimit race condition (#853904) https://bugzilla.redhat.com/853904 Se

[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread oourfali
Oved Ourfali has posted comments on this change. Change subject: userportal: SearchResultsLimit race condition (#853904) .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8132 To uns

[Engine-patches] Change in ovirt-engine[master]: core: Proccessing Vdsm Error Code to API (#858719)

2012-09-24 Thread msalem
Muli Salem has submitted this change and it was merged. Change subject: core: Proccessing Vdsm Error Code to API (#858719) .. core: Proccessing Vdsm Error Code to API (#858719) https://bugzilla.redhat.com/858719 Propagating er

[Engine-patches] Change in ovirt-engine[master]: core: Proccessing Vdsm Error Code to API (#858719)

2012-09-24 Thread msalem
Muli Salem has posted comments on this change. Change subject: core: Proccessing Vdsm Error Code to API (#858719) .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8151 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: engine: Allow empty quota in audit mode (#855633)

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: engine: Allow empty quota in audit mode (#855633) .. Patch Set 2: Looks good to me, but someone else must approve +1 on frontend -- To view, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Cannot edit properties of running vms.

2012-09-24 Thread emesika
Eli Mesika has uploaded a new change for review. Change subject: core: Cannot edit properties of running vms. .. core: Cannot edit properties of running vms. Cannot edit properties (that does not require guest reboot to take eff

[Engine-patches] Change in ovirt-engine[master]: core: AuditLog for Activate/Deactivate VmNic (#858719)

2012-09-24 Thread msalem
Muli Salem has submitted this change and it was merged. Change subject: core: AuditLog for Activate/Deactivate VmNic (#858719) .. core: AuditLog for Activate/Deactivate VmNic (#858719) https://bugzilla.redhat.com/858719 Adding

[Engine-patches] Change in ovirt-engine[master]: core: AuditLog for Activate/Deactivate VmNic (#858719)

2012-09-24 Thread msalem
Muli Salem has posted comments on this change. Change subject: core: AuditLog for Activate/Deactivate VmNic (#858719) .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8152 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: Error Code Handling for AddVmNic (#858719)

2012-09-24 Thread msalem
Muli Salem has submitted this change and it was merged. Change subject: core: Error Code Handling for AddVmNic (#858719) .. core: Error Code Handling for AddVmNic (#858719) https://bugzilla.redhat.com/858719 This patch makes s

[Engine-patches] Change in ovirt-engine[master]: core: Error Code Handling for AddVmNic (#858719)

2012-09-24 Thread msalem
Muli Salem has posted comments on this change. Change subject: core: Error Code Handling for AddVmNic (#858719) .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8150 To unsubscribe,

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: allow persistent_auth via localhost

2012-09-24 Thread oliel
Ori Liel has posted comments on this change. Change subject: sdk: allow persistent_auth via localhost .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/8163 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: webadmin: import: validate quota in enforce mode (#855635)

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: import: validate quota in enforce mode (#855635) .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8060 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: webadmin: Make HA consistant in VM and template tabs

2012-09-24 Thread nslomian
Noam Slomianko has posted comments on this change. Change subject: webadmin: Make HA consistant in VM and template tabs .. Patch Set 1: Verified Seems to be the desired behavior -- To view, visit http://gerrit.ovirt.org/8164 T

[Engine-patches] Change in ovirt-engine[master]: webadmin: import: storage drop down is empty (#856976)

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: import: storage drop down is empty (#856976) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8156 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: webadmin: pass HA value to template from vm

2012-09-24 Thread nslomian
Noam Slomianko has posted comments on this change. Change subject: webadmin: pass HA value to template from vm .. Patch Set 4: Verified Split commit as suggested -- To view, visit http://gerrit.ovirt.org/8154 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: webadmin: Make HA consistant in VM and template tabs

2012-09-24 Thread nslomian
Noam Slomianko has uploaded a new change for review. Change subject: webadmin: Make HA consistant in VM and template tabs .. webadmin: Make HA consistant in VM and template tabs Problems: * Inconsistent display of the high avail

[Engine-patches] Change in ovirt-engine[master]: webadmin: Allow attaching v2 domains for v3 pool (#856166)

2012-09-24 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Allow attaching v2 domains for v3 pool (#856166) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8091 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: engine: Allow empty quota in audit mode (#855633)

2012-09-24 Thread dfediuck
Doron Fediuck has posted comments on this change. Change subject: engine: Allow empty quota in audit mode (#855633) .. Patch Set 2: Unitests? -- To view, visit http://gerrit.ovirt.org/8059 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: AuditLog for Activate/Deactivate VmNic (#858719)

2012-09-24 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: AuditLog for Activate/Deactivate VmNic (#858719) .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8152 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: core: VmMigrateDiskoComplete VDS command

2012-09-24 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: core: VmMigrateDiskoComplete VDS command .. Patch Set 2: (1 inline comment) Commit Message Line 3: AuthorDat

[Engine-patches] Change in ovirt-engine[master]: core: Error Code Handling for AddVmNic (#858719)

2012-09-24 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Error Code Handling for AddVmNic (#858719) .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8150 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: allow persistent_auth via localhost

2012-09-24 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: sdk: allow persistent_auth via localhost .. sdk: allow persistent_auth via localhost Change-Id: If3df5b766cfcea48dcc5a9cfb266c55570e55234 Signed-off-by: Mic

[Engine-patches] Change in ovirt-engine[master]: [WIP] tools: Refactor of EngineConfigLogic (#858769)

2012-09-24 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: [WIP] tools: Refactor of EngineConfigLogic (#858769) .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/81

[Engine-patches] Change in ovirt-engine[master]: core: Proccessing Vdsm Error Code to API (#858719)

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Proccessing Vdsm Error Code to API (#858719) .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8151 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: userportal: SearchResultsLimit race condition (#853904)

2012-09-24 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: userportal: SearchResultsLimit race condition (#853904) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8132 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] introducing SPMAsyncTaskHandler

2012-09-24 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: [WIP] introducing SPMAsyncTaskHandler .. Patch Set 5: I would prefer that you didn't submit this (1 inline comment) Now, you in correct direction, after

[Engine-patches] Change in ovirt-engine[master]: core: task manager cache is not cleared..(#854308)

2012-09-24 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: task manager cache is not cleared..(#854308) .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8161 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: restapi: Schema API element - add min/max occurences

2012-09-24 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: Schema API element - add min/max occurences .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8160 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: core: task manager cache is not cleared..(#854308)

2012-09-24 Thread emesika
Eli Mesika has uploaded a new change for review. Change subject: core: task manager cache is not cleared..(#854308) .. core: task manager cache is not cleared..(#854308) task manager cache is not cleared after getting error that

[Engine-patches] Change in ovirt-engine[master]: restapi: Schema API element - add min/max occurences

2012-09-24 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: Schema API element - add min/max occurences .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8160 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: webadmin: pass HA value to template from vm

2012-09-24 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: pass HA value to template from vm .. Patch Set 3: I would first fix the bug (at VmListModel.java), and the other fixes can be pushed in a separate pa

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

2012-09-24 Thread sanjal
Shireesh Anjal has posted comments on this change. Change subject: engine: Refresh gluster data periodically .. Patch Set 11: (1 inline comment) File frontend/webadmin/modul

[Engine-patches] Change in ovirt-engine[master]: restapi: schema add missing minOccurs, maxOccurs

2012-09-24 Thread oliel
Ori Liel has abandoned this change. Change subject: restapi: schema add missing minOccurs, maxOccurs .. Patch Set 1: Abandoned -- To view, visit http://gerrit.ovirt.org/8155 To unsubscribe, visit http://gerrit.ovirt.org/setting

[Engine-patches] Change in ovirt-engine[master]: restapi: Schema API element - add min/max occurences

2012-09-24 Thread oliel
Ori Liel has uploaded a new change for review. Change subject: restapi: Schema API element - add min/max occurences .. restapi: Schema API element - add min/max occurences Change-Id: I1208d0ba3355fbde74cd43cac4fd444058943f1a Sig

[Engine-patches] [oVirt Jenkins] ovirt_engine - Build # 3102 - Fixed!

2012-09-24 Thread Jenkins oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine/ Build: http://jenkins.ovirt.org/job/ovirt_engine/3102/ Build Number: 3102 Build Status: Fixed Triggered By: Started by an SCM change - Changes Since Last Success: - Changes

[Engine-patches] Change in ovirt-engine[master]: core: fix checkstyle in tests

2012-09-24 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: fix checkstyle in tests .. core: fix checkstyle in tests Adds NON-NLS to the tests Change-Id: I91f931fd176d8c96b11a4ac2ec8c1ec0dbe0dcf5 Signed-o

[Engine-patches] Change in ovirt-engine[master]: core: fix checkstyle in tests

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: fix checkstyle in tests .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8159 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: core: fix checkstyle in tests

2012-09-24 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: fix checkstyle in tests .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/8159 To unsubscribe, visit http://g

  1   2   >