[Engine-patches] Change in ovirt-engine[master]: webadmin: Network builders

2013-04-28 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: webadmin: Network builders .. Patch Set 4: Because the "new VM dialog" will contain a part where you will be able to select a NIC and assign it to a logical net

[Engine-patches] Change in ovirt-engine[master]: [WIP] core: Introducing findbugs filter to bll module

2013-04-28 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: [WIP] core: Introducing findbugs filter to bll module .. Patch Set 3: Laszlo, Care to respond, so we can advance with this? -- To view, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: tools: enable unit tests

2013-04-28 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: tools: enable unit tests .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14271 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: inactive domain can be selected as master

2013-04-28 Thread laravot
Liron Ar has posted comments on this change. Change subject: core: inactive domain can be selected as master .. Patch Set 5: Michael, thanks for your review. can you please elaborate on few raised issues - "The patch will ca

[Engine-patches] Change in ovirt-engine[master]: userportal, webadmin: branding support.

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: userportal, webadmin: branding support. .. Patch Set 11: The exception I got was due to I37156061cbdd. -- To view, visit http://gerrit.ovirt.org/13181 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: core: re-support broken apache proxy

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: re-support broken apache proxy .. Patch Set 1: Verified Hi, In future, please test changes that may effect setup using production product setup aka instal

[Engine-patches] Change in ovirt-engine[master]: core: re-support broken apache proxy

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: core: re-support broken apache proxy .. core: re-support broken apache proxy apache proxy support was broken in: I37156061cbdd, 'core: i18n splash screen.' The

[Engine-patches] Change in ovirt-engine[master]: core: rhevm-upgrade creates log/helping files...

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: core: rhevm-upgrade creates log/helping files... .. Patch Set 1: Looks good to me, but someone else must approve Thanks! -- To view, visit http://gerrit.ovirt.or

[Engine-patches] Change in ovirt-engine[master]: gluster: Override CommandBase.getJobMessageProperties

2013-04-28 Thread masayag
Moti Asayag has posted comments on this change. Change subject: gluster: Override CommandBase.getJobMessageProperties .. Patch Set 4: Shireesh, may I ask you to verify the patch as I don't have gluster installed on my environm

[Engine-patches] Change in ovirt-engine[master]: webadmin: Making VM related substabs resizable

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Making VM related substabs resizable .. Patch Set 1: (2 inline comments) File frontend/webadmin/mod

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

2013-04-28 Thread Jenkins ci oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/ Build: http://jenkins.ovirt.org/job/ovirt-mom_create_rpms/49/ Build Number: 49 Build Status: Fixed Triggered By: Started by an SCM change - Changes Since Last Success: ---

[Engine-patches] Change in ovirt-engine[master]: findbugs: Method with Boolean return explicit null

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: findbugs: Method with Boolean return explicit null .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14227

[Engine-patches] Change in ovirt-engine[master]: webadmin: Host Tab - "Primary" & "Secondary" are not localiz...

2013-04-28 Thread alkaplan
Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: Host Tab - "Primary" & "Secondary" are not localized .. webadmin: Host Tab - "Primary" & "Secondary" are not localized These strings were hard

[Engine-patches] Change in ovirt-engine[master]: webadmin: Host Tab - "Primary" & "Secondary" are not localiz...

2013-04-28 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: Host Tab - "Primary" & "Secondary" are not localized .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1429

[Engine-patches] Change in ovirt-engine[master]: webadmin: Host Tab - "Primary" & "Secondary" are not localiz...

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Host Tab - "Primary" & "Secondary" are not localized .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14295 To unsubs

[Engine-patches] Change in ovirt-engine[master]: packaging: move all packaging related to packaging

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: move all packaging related to packaging .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14302 To unsubscribe, visit http://gerrit.ov

[Engine-patches] Change in ovirt-engine[master]: packaging: move all packaging related to packaging

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has uploaded a new change for review. Change subject: packaging: move all packaging related to packaging .. packaging: move all packaging related to packaging currently we manage packaging scripts all over the tree,

[Engine-patches] Change in ovirt-engine-cli[master]: cli: do not use preserved "type" as param name

2013-04-28 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: cli: do not use preserved "type" as param name .. cli: do not use preserved "type" as param name Change-Id: If709df3d9c63ae77c19f2b52cad27e5490b46481 Signed

[Engine-patches] Change in ovirt-engine-cli[master]: cli: remove pexpect dependency

2013-04-28 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: cli: remove pexpect dependency .. cli: remove pexpect dependency GetKeyCommand is a deprecated POC that no longer maintained/supported, since pexpect was fo

[Engine-patches] Change in ovirt-engine-cli[master]: cli: normalize sdk errors

2013-04-28 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: cli: normalize sdk errors .. cli: normalize sdk errors Change-Id: Ia4fef4767f7c007ae882c3734b592ab63b32deb5 Signed-off-by: Michael pasternak --- M src/cli/

[Engine-patches] Change in ovirt-engine[master]: webadmin: JP - truncation in interface column

2013-04-28 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: JP - truncation in interface column .. webadmin: JP - truncation in interface column Attach Disk: fix columns headers truncation Change-Id: Iee

[Engine-patches] Change in ovirt-engine[master]: webadmin: JP - truncation in interface column

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: JP - truncation in interface column .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14287 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: webadmin: Network builders

2013-04-28 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: Network builders .. Patch Set 4: Hi Tomas, I don't really understand why do you need builders here. Why can't you use the current models for the instan

[Engine-patches] Change in ovirt-engine[master]: core: rhevm-upgrade creates log/helping files...

2013-04-28 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: rhevm-upgrade creates log/helping files... .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14298 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: rhevm-upgrade creates log/helping files...

2013-04-28 Thread emesika
Eli Mesika has uploaded a new change for review. Change subject: core: rhevm-upgrade creates log/helping files... .. core: rhevm-upgrade creates log/helping files... rhevm-upgrade creates log/helping files in /usr/share/ovirt-en

[Engine-patches] Change in ovirt-engine[master]: webadmin: Host Tab - "Primary" & "Secondary" are not localiz...

2013-04-28 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: Host Tab - "Primary" & "Secondary" are not localized .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/14295 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine-cli[master]: cli: recognize AmbiguousQueryError as COMMAND_ERROR

2013-04-28 Thread mpastern
Michael Pasternak has uploaded a new change for review. Change subject: cli: recognize AmbiguousQueryError as COMMAND_ERROR .. cli: recognize AmbiguousQueryError as COMMAND_ERROR Change-Id: I61665beffe059b3ebe2296ebb4916bfe49f1f

[Engine-patches] Change in ovirt-engine[master]: findbugs: String comparison is not using equals method

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: findbugs: String comparison is not using equals method .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: tools: enable unit tests

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: tools: enable unit tests .. Patch Set 1: @Alon - unfortunately, not :-( -- To view, visit http://gerrit.ovirt.org/14271 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: webadmin: Remove VM comparators in Common.gwt.xml

2013-04-28 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: webadmin: Remove VM comparators in Common.gwt.xml .. webadmin: Remove VM comparators in Common.gwt.xml Removed VmsComparer's and VmsComparerByDiskSize

[Engine-patches] Change in ovirt-engine[master]: core: unify and move Disk Comparators

2013-04-28 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: unify and move Disk Comparators .. core: unify and move Disk Comparators Organized the various Disk and DiskImage comparators in a sensible manne

[Engine-patches] Change in ovirt-engine[master]: core: Introducing the comparators package

2013-04-28 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: Introducing the comparators package .. core: Introducing the comparators package org.ovirt.engine.core.common.businessentities includes several c

[Engine-patches] Change in ovirt-engine[master]: webadmin: Remove VM comparators in Common.gwt.xml

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin: Remove VM comparators in Common.gwt.xml .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14285 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: Introducing the comparators package

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14192 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: core: unify and move Disk Comparators

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13849 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: findbugs: equals between different types

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: findbugs: equals between different types .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14296 To unsu

[Engine-patches] Change in ovirt-engine[master]: findbugs: equals between different types

2013-04-28 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: findbugs: equals between different types .. findbugs: equals between different types Change-Id: I125e5df8def7a223f9469decc7aa43dab61f4a47 Signed-off-by: Shahar

[Engine-patches] Change in ovirt-engine[master]: webadmin: Host Tab - "Primary" & "Secondary" are not localiz...

2013-04-28 Thread alkaplan
Alona Kaplan has uploaded a new change for review. Change subject: webadmin: Host Tab - "Primary" & "Secondary" are not localized .. webadmin: Host Tab - "Primary" & "Secondary" are not localized These strings were hard coded in

[Engine-patches] Change in ovirt-engine[master]: findbugs: self comparison of user to itself

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: findbugs: self comparison of user to itself .. Patch Set 1: Looks good to me, but someone else must approve Thanks! I looked several times on the findbugs, and I

[Engine-patches] Change in ovirt-engine[master]: webadmin: Remove VM comparators in Common.gwt.xml

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: webadmin: Remove VM comparators in Common.gwt.xml .. Patch Set 2: Verified; Looks good to me, approved Rebased, no code changes. +2 by Tal. -- To view, visit h

[Engine-patches] Change in ovirt-engine[master]: core: Introducing the comparators package

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 4: Verified; Looks good to me, approved Rebased, no code changes. +2 by Yair. -- To view, visit http://g

[Engine-patches] Change in ovirt-engine[master]: core: unify and move Disk Comparators

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 9: Verified; Looks good to me, approved Rebased, no code changes. +2 by Maor. -- To view, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: findbugs: toString may return null

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: findbugs: toString may return null .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14294 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: findbugs: Method with Boolean return explicit null

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: findbugs: Method with Boolean return explicit null .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/142

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disk dialog - capitalize labels

2013-04-28 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Disk dialog - capitalize labels .. webadmin: Disk dialog - capitalize labels New/Edit disk dialog: s/Is bootable/Is Bootable s/Is shareable/Is S

[Engine-patches] Change in ovirt-engine[master]: findbugs: self comparison of user to itself

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: findbugs: self comparison of user to itself .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14293 To u

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disk dialog - capitalize labels

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Disk dialog - capitalize labels .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14281 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disks VMs Templates sub-tabs resizable columns

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Disks VMs Templates sub-tabs resizable columns .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14280 To un

[Engine-patches] Change in ovirt-engine[master]: findbugs: toString may return null

2013-04-28 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: findbugs: toString may return null .. findbugs: toString may return null Change-Id: I7540e6cfadb76f742f5b9959418b5e0a6cbf4fed Signed-off-by: Shahar Havivi ---

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disks VMs Templates sub-tabs resizable columns

2013-04-28 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Disks VMs Templates sub-tabs resizable columns .. webadmin: Disks VMs Templates sub-tabs resizable columns VMs/Templates sub-tabs under Disks ma

[Engine-patches] Change in ovirt-engine[master]: webadmin: ISO/Export sub-tabs resizable columns

2013-04-28 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: ISO/Export sub-tabs resizable columns .. webadmin: ISO/Export sub-tabs resizable columns ISO/Export sub-tabs under Storage main-tab now support

[Engine-patches] Change in ovirt-engine[master]: findbugs: self comparison of user to itself

2013-04-28 Thread shavivi
Shahar Havivi has uploaded a new change for review. Change subject: findbugs: self comparison of user to itself .. findbugs: self comparison of user to itself Change-Id: Ifa88e752da82043857f27d334ba86ca1c5587bcd Signed-off-by: S

[Engine-patches] Change in ovirt-engine[master]: webadmin: ISO/Export sub-tabs resizable columns

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: ISO/Export sub-tabs resizable columns .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14279 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: restapi: disabling POST & DELETE implementations for quota

2013-04-28 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: disabling POST & DELETE implementations for quota .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14046 To unsu

[Engine-patches] Change in ovirt-engine[master]: restapi: disabling POST & DELETE implementations for quota

2013-04-28 Thread dfediuck
Doron Fediuck has posted comments on this change. Change subject: restapi: disabling POST & DELETE implementations for quota .. 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: Introducing the comparators package

2013-04-28 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14192 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: core: Introducing the comparators package

2013-04-28 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 3: Well done. -- To view, visit http://gerrit.ovirt.org/14192 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: core: Introducing the comparators package

2013-04-28 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14192 To unsubscribe, visit http://

[Engine-patches] [oVirt Jenkins] ovirt_engine_update_db_multiple_os - Build # 748 - Fixed!

2013-04-28 Thread Jenkins ci oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine_update_db_multiple_os/ Build: http://jenkins.ovirt.org/job/ovirt_engine_update_db_multiple_os/748/ Build Number: 748 Build Status: Fixed Triggered By: Started by an SCM change - Changes Since Last Success: ---

[Engine-patches] Change in ovirt-engine[master]: webadmin: Remove VM comparators in Common.gwt.xml

2013-04-28 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: webadmin: Remove VM comparators in Common.gwt.xml .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14285

[Engine-patches] Change in ovirt-engine[master]: core: unify and move Disk Comparators

2013-04-28 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13849 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: Revert "engine: Use isBlockDomain() where possible"

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: Revert "engine: Use isBlockDomain() where possible" .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14292 To unsubscribe, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: Revert "engine: Use isBlockDomain() where possible"

2013-04-28 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: Revert "engine: Use isBlockDomain() where possible" .. Revert "engine: Use isBlockDomain() where possible" This is a partial revert of the rest parts of commit

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] ram snapshots related changes in bll project

2013-04-28 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: core: [WIP] ram snapshots related changes in bll project .. core: [WIP] ram snapshots related changes in bll project Change-Id: Ifbb28efda1b63e506233a88399488a256e

[Engine-patches] Change in ovirt-engine[master]: core: [WIP] extract general code from HibernateVmCommand

2013-04-28 Thread ahadas
Arik Hadas has uploaded a new change for review. Change subject: core: [WIP] extract general code from HibernateVmCommand .. core: [WIP] extract general code from HibernateVmCommand This patch extract code which is not specific

[Engine-patches] Change in ovirt-engine[master]: webadmin: new pool - update disks list on template change

2013-04-28 Thread mlipchuk
Maor Lipchuk has submitted this change and it was merged. Change subject: webadmin: new pool - update disks list on template change .. webadmin: new pool - update disks list on template change Ensure that IsDiskAvailable event

[Engine-patches] Change in ovirt-engine[master]: webadmin: new pool - update disks list on template change

2013-04-28 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: webadmin: new pool - update disks list on template change .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14286 To

[Engine-patches] Change in ovirt-engine[master]: webadmin: new pool - update disks list on template change

2013-04-28 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: webadmin: new pool - update disks list on template change .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14286 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: build: Explicit versions definitions for maven plugins.

2013-04-28 Thread asaf
Asaf Shakarchi has uploaded a new change for review. Change subject: build: Explicit versions definitions for maven plugins. .. build: Explicit versions definitions for maven plugins. Maven 3.x deprecates definition of plugins a

[Engine-patches] Change in ovirt-engine[master]: findbugs: Field masks field in superclass

2013-04-28 Thread shavivi
Shahar Havivi has submitted this change and it was merged. Change subject: findbugs: Field masks field in superclass .. findbugs: Field masks field in superclass Change-Id: I544d5e4e602040c70020fa9b2c43a49ad0bbcb9d Signed-off-b

[Engine-patches] Change in ovirt-engine[master]: findbugs: Field masks field in superclass

2013-04-28 Thread shavivi
Shahar Havivi has posted comments on this change. Change subject: findbugs: Field masks field in superclass .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14197 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: Adding parameters to delay fence based on host's load

2013-04-28 Thread ybronhei
Yaniv Bronhaim has uploaded a new change for review. Change subject: Adding parameters to delay fence based on host's load .. Adding parameters to delay fence based on host's load Before performing fencing to host, we wait Timeo

[Engine-patches] Change in ovirt-engine[master]: restapi: #920483 - Group Wrongly Displayed as User (within p...

2013-04-28 Thread oliel
Ori Liel has submitted this change and it was merged. Change subject: restapi: #920483 - Group Wrongly Displayed as User (within permission) .. restapi: #920483 - Group Wrongly Displayed as User (within permission) Change-Id:

[Engine-patches] Change in ovirt-engine[master]: restapi: #920483 - Group Wrongly Displayed as User (within p...

2013-04-28 Thread oliel
Ori Liel has posted comments on this change. Change subject: restapi: #920483 - Group Wrongly Displayed as User (within permission) .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: webadmin: new pool - update disks list on template change

2013-04-28 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: new pool - update disks list on template change .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14286 To unsubscribe

[Engine-patches] Change in ovirt-engine[master]: webadmin: JP - truncation in interface column

2013-04-28 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: JP - truncation in interface column .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14287 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: webadmin: Suggest an action when network is out of sync

2013-04-28 Thread masayag
Moti Asayag has posted comments on this change. Change subject: webadmin: Suggest an action when network is out of sync .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14282 To uns

[Engine-patches] Change in ovirt-engine[master]: webadmin: Suggest an action when network is out of sync

2013-04-28 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: webadmin: Suggest an action when network is out of sync .. webadmin: Suggest an action when network is out of sync Change-Id: I21bccd87bf1311f6d2d87683f1f

[Engine-patches] Change in ovirt-engine[master]: core: Mount options not sent when adding POSIX domain

2013-04-28 Thread abaron
Ayal Baron has posted comments on this change. Change subject: core: Mount options not sent when adding POSIX domain .. Patch Set 1: Could the underlying issue simply be an upgrade issue? i.e. AdvancedNFSOptions was not set pr

[Engine-patches] [oVirt Jenkins] ovirt_engine_update_db_multiple_os - Build # 743 - Fixed!

2013-04-28 Thread Jenkins ci oVirt Server
Project: http://jenkins.ovirt.org/job/ovirt_engine_update_db_multiple_os/ Build: http://jenkins.ovirt.org/job/ovirt_engine_update_db_multiple_os/743/ Build Number: 743 Build Status: Fixed Triggered By: Started by an SCM change - Changes Since Last Success: ---

[Engine-patches] Change in ovirt-engine[master]: webadmin: Remove VM comparators in Common.gwt.xml

2013-04-28 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: webadmin: Remove VM comparators in Common.gwt.xml .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14285 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disk dialog - capitalize labels

2013-04-28 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: webadmin: Disk dialog - capitalize labels .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14281 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: webadmin: ISO/Export sub-tabs resizable columns

2013-04-28 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: webadmin: ISO/Export sub-tabs resizable columns .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14279 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: webadmin: Disks VMs Templates sub-tabs resizable columns

2013-04-28 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: webadmin: Disks VMs Templates sub-tabs resizable columns .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14280 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: webadmin: JP - truncation in interface column

2013-04-28 Thread derez
Daniel Erez has uploaded a new change for review. Change subject: webadmin: JP - truncation in interface column .. webadmin: JP - truncation in interface column Attach Disk: fix columns headers truncation Change-Id: Ieee042d7ef

[Engine-patches] Change in ovirt-engine[master]: restapi: make cpu-mode enumerable values lower-case #915042

2013-04-28 Thread mpastern
Michael Pasternak has submitted this change and it was merged. Change subject: restapi: make cpu-mode enumerable values lower-case #915042 .. restapi: make cpu-mode enumerable values lower-case #915042 Change-Id: I50623ebbc1ec7

[Engine-patches] Change in ovirt-engine[master]: restapi: make cpu-mode enumerable values lower-case #915042

2013-04-28 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: restapi: make cpu-mode enumerable values lower-case #915042 .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14

[Engine-patches] Change in ovirt-engine[master]: webadmin: quota - subtabs resizable columns

2013-04-28 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: quota - subtabs resizable columns .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14278 To unsubscribe, visit http:

[Engine-patches] Change in ovirt-engine[master]: webadmin: new pool - update disks list on template change

2013-04-28 Thread mlipchuk
Maor Lipchuk has uploaded a new change for review. Change subject: webadmin: new pool - update disks list on template change .. webadmin: new pool - update disks list on template change Ensure that IsDiskAvailable event sent on

[Engine-patches] Change in ovirt-engine[master]: webadmin: Suggest an action when network is out of sync

2013-04-28 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: Suggest an action when network is out of sync .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14282 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: wip: initVdsOnUp- remove reconstruct and host status.

2013-04-28 Thread laravot
Liron Ar has posted comments on this change. Change subject: core: wip: initVdsOnUp- remove reconstruct and host status. .. Patch Set 9: (1 inline comment) File backend/mana

[Engine-patches] Change in ovirt-engine[master]: engine: fetch gluster host uuid during InitVdsOnUp

2013-04-28 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine: fetch gluster host uuid during InitVdsOnUp .. Patch Set 3: (2 inline comments) File backend/manager/

[Engine-patches] Change in ovirt-engine[master]: core: InitVdsOnUp- proceed regardless of ConnectHostToStorag...

2013-04-28 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: InitVdsOnUp- proceed regardless of ConnectHostToStoragePoolServers .. Patch Set 2: I would prefer that you didn't submit this (2 inline comments) Bad au

[Engine-patches] Change in ovirt-engine[master]: core: wip: initVdsOnUp- remove reconstruct and host status.

2013-04-28 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: wip: initVdsOnUp- remove reconstruct and host status. .. Patch Set 9: I would prefer that you didn't submit this (1 inline comment) .

[Engine-patches] Change in ovirt-engine[master]: webadmin: Correct timezone default in New/Edit VM dialog

2013-04-28 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: webadmin: Correct timezone default in New/Edit VM dialog .. Patch Set 1: (1 inline comment) also, if we want the users to be able to modify the new configuration

[Engine-patches] Change in ovirt-engine[master]: packaging: log rotation - do not execute ls on non-existing ...

2013-04-28 Thread oschreib
Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: log rotation - do not execute ls on non-existing files .. packaging: log rotation - do not execute ls on non-existing files Due to the usag

[Engine-patches] Change in ovirt-engine[master]: packaging: log rotation - do not execute ls on non-existing ...

2013-04-28 Thread oschreib
Ofer Schreiber has posted comments on this change. Change subject: packaging: log rotation - do not execute ls on non-existing files .. Patch Set 4: Verified; Looks good to me, approved Based on own verification and Sandro's +

[Engine-patches] Change in ovirt-engine[master]: packaging: log rotation - do not execute ls on non-existing ...

2013-04-28 Thread oschreib
Ofer Schreiber has posted comments on this change. Change subject: packaging: log rotation - do not execute ls on non-existing files .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14097 To

[Engine-patches] Change in ovirt-engine[master]: core: unify and move Disk Comparators

2013-04-28 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 8: Verified Changes in previous patch in the series, no changes to this one. -- To view, visit http://gerrit

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

2013-04-28 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: refactored firewalld support .. Patch Set 11: Looks good to me, but someone else must approve Sorry for the noise. Rebase patch#9, nothing changed. --

  1   2   >