[Engine-patches] Change in ovirt-engine[master]: webadmin: Making DC/cluster substab resizable

2013-04-25 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: webadmin: Making DC/cluster substab resizable .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14258 To unsubscribe, visit http://gerrit.ovirt.

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

2013-04-25 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: webadmin: Making VM related substabs resizable .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14253 To unsubscribe, visit http://gerrit.ovirt

[Engine-patches] Change in ovirt-engine[master]: webadmin: Making DC/cluster substab resizable

2013-04-25 Thread tjelinek
Tomas Jelinek has uploaded a new change for review. Change subject: webadmin: Making DC/cluster substab resizable .. webadmin: Making DC/cluster substab resizable Change-Id: I707c028cd034b1aea5a62b44831150d2111e326c Signed-off-b

[Engine-patches] Change in ovirt-engine[master]: userportal: Fix exception on browser resizing

2013-04-25 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: userportal: Fix exception on browser resizing .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14240 To unsubscribe, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: userportal: closing RDP crashes IE

2013-04-25 Thread tjelinek
Tomas Jelinek has posted comments on this change. Change subject: userportal: closing RDP crashes IE .. Patch Set 1: (1 inline comment) File frontend/webadmin/modules/gwt-co

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

2013-04-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: [WIP] core: Introducing findbugs filter to bll module .. Patch Set 3: (1 inline comment) File backend/manag

[Engine-patches] Change in ovirt-engine[engine_3.1]: restapi: if filter: header is set, use query instead of sear...

2013-04-25 Thread rnori
Ravi Nori has posted comments on this change. Change subject: restapi: if filter: header is set, use query instead of search (#882946) .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/14257 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: restapi: WIP: Created restapi interface for backup.

2013-04-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: restapi: WIP: Created restapi interface for backup. .. Patch Set 1: (1 inline comment) File backend/manager

[Engine-patches] Change in ovirt-engine[master]: restapi: WIP: Add support for backup APIs.

2013-04-25 Thread Sharad Mishra
Sharad Mishra has posted comments on this change. Change subject: restapi: WIP: Add support for backup APIs. .. Patch Set 2: In patch set 2, I have removed Actions like preview, undo, commit etc. And also removed nic, disk, c

[Engine-patches] Change in ovirt-engine[engine_3.1]: restapi: if filter: header is set, use query instead of sear...

2013-04-25 Thread rnori
Ravi Nori has posted comments on this change. Change subject: restapi: if filter: header is set, use query instead of search (#869334) .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14257 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[engine_3.1]: restapi: if filter: header is set, use query instead of sear...

2013-04-25 Thread rnori
Ravi Nori has uploaded a new change for review. Change subject: restapi: if filter: header is set, use query instead of search (#869334) .. restapi: if filter: header is set, use query instead of search (#869334) When the filte

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

2013-04-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: userportal, webadmin: branding support. .. Patch Set 11: Hello, Just for the [public] record. When designing pluggable infrastructure, the ultimate test is if t

[Engine-patches] Change in ovirt-engine[master]: core: Remove ImageStatus.INVALID

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Remove ImageStatus.INVALID .. Patch Set 3: Looks good to me, approved Nice catch Alissa -- To view, visit http://gerrit.ovirt.org/14194 To unsubscribe, visit

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

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: findbugs: String compression is not using equals method .. Patch Set 1: (5 inline comments) Some comments that are not relevant to the scope of the change directly b

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

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: findbugs: Field masks field in superclass .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14197 To unsubscribe, visit http://ger

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

2013-04-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: userportal, webadmin: branding support. .. Patch Set 11: > Alon, the error message you are seeing is due to a XSRF token being used. if > you close the browser t

[Engine-patches] Change in ovirt-engine[master]: engine: Allow engine to configure management network

2013-04-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: engine: Allow engine to configure management network .. Patch Set 2: This was the whole point of the original feature, to automate the addition of the management

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

2013-04-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: packaging: Added generic DB validations before the upgrade .. Patch Set 14: Looks good to me, but someone else must approve (2 inline comments) Looks good!

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

2013-04-25 Thread awels
Alexander Wels has posted comments on this change. Change subject: userportal, webadmin: branding support. .. Patch Set 11: Verified Alon, the error message you are seeing is due to a XSRF token being used. if you close the br

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

2013-04-25 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/723/ Build Number: 723 Build Status: Fixed Triggered By: Started by an SCM change - Changes Since Last Success: ---

[Engine-patches] Change in ovirt-engine[master]: core: Add IDs to ImageStatus

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Add IDs to ImageStatus .. Patch Set 3: Verified Fixed long lines in the commit message, no code changes -- To view, visit http://gerrit.ovirt.org/14193 T

[Engine-patches] Change in ovirt-engine[master]: core: Add IDs to ImageStatus

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Add IDs to ImageStatus .. Patch Set 3: Looks good to me, approved Fixed long lines in the commit message, no code changes -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: core, restapi: Don't store domain suffix in user name

2013-04-25 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core, restapi: Don't store domain suffix in user name .. Patch Set 1: Verified This is not urgent, but it would simplify some of the changes that I am doing re

[Engine-patches] Change in ovirt-engine[master]: core, restapi: Don't store domain suffix in user name

2013-04-25 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: core, restapi: Don't store domain suffix in user name .. core, restapi: Don't store domain suffix in user name Currently when we save the information about an

[Engine-patches] Change in ovirt-engine[master]: frontend: Change executed method name in frontend callbacks

2013-04-25 Thread tnisan
Tal Nisan has submitted this change and it was merged. Change subject: frontend: Change executed method name in frontend callbacks .. frontend: Change executed method name in frontend callbacks Changed the IFrontendMultipleActi

[Engine-patches] Change in ovirt-engine[master]: frontend: Change executed method name in frontend callbacks

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: frontend: Change executed method name in frontend callbacks .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14254 To u

[Engine-patches] Change in ovirt-engine[master]: frontend: Change executed method name in frontend callbacks

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: frontend: Change executed method name in frontend callbacks .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14254 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: frontend: Change executed method name in frontend callbacks

2013-04-25 Thread asaf
Asaf Shakarchi has posted comments on this change. Change subject: frontend: Change executed method name in frontend callbacks .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14254 To unsubs

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

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

[Engine-patches] Change in ovirt-engine[master]: frontend: Change executed method name in frontend callbacks

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: frontend: Change executed method name in frontend callbacks .. 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]: core: restrict comp version of edit storage conn

2013-04-25 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: restrict comp version of edit storage conn .. core: restrict comp version of edit storage conn Added upgrade script that restricts the compatibil

[Engine-patches] Change in ovirt-engine[master]: core: restrict comp version of edit storage conn

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: restrict comp version of edit storage conn .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14249 To unsubsc

[Engine-patches] Change in ovirt-engine[master]: core: Remove get image info call before images commands

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Remove get image info call before images commands .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.or

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

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: findbugs: Field masks field in superclass .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14197 To uns

[Engine-patches] Change in ovirt-engine[master]: restapi: WIP: Created restapi interface for backup.

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: restapi: WIP: Created restapi interface for backup. .. Patch Set 1: (1 inline comment) see my main question inline. Can we get a version with mock-uped actions

[Engine-patches] Change in ovirt-engine[master]: restapi: WIP: Add support for backup APIs.

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: restapi: WIP: Add support for backup APIs. .. Patch Set 1: (2 inline comments) File backend/manager/module

[Engine-patches] Change in ovirt-engine[master]: core: restrict comp version of edit storage conn

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: restrict comp version of edit storage conn .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14249 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: core: remove unneeded static method declarations

2013-04-25 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: core: remove unneeded static method declarations .. core: remove unneeded static method declarations In AttachUserToVmFromPoolAndRunCommand class. and a

[Engine-patches] Change in ovirt-engine[master]: core: remove unneeded static method declarations

2013-04-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: core: remove unneeded static method declarations .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14245 To unsubscr

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

2013-04-25 Thread tjelinek
Tomas Jelinek has uploaded a new change for review. Change subject: webadmin: Making VM related substabs resizable .. webadmin: Making VM related substabs resizable VM, template and host related subtabs Change-Id: Ibb8a7edcf437

[Engine-patches] Change in ovirt-engine[master]: engine: Allow engine to configure management network

2013-04-25 Thread xpavli44
Martin Pavlik has posted comments on this change. Change subject: engine: Allow engine to configure management network .. Patch Set 2: Verified with this patch host is added without bridge, however it leads to host non-operati

[Engine-patches] Change in ovirt-engine[master]: frontend: Fix findbug errors in Linq.java

2013-04-25 Thread tnisan
Tal Nisan has submitted this change and it was merged. Change subject: frontend: Fix findbug errors in Linq.java .. frontend: Fix findbug errors in Linq.java Change-Id: I03d340ffe8286180f907785b998d6bbf3c0e09bd Signed-off-by: T

[Engine-patches] Change in ovirt-engine[master]: frontend: Fix findbug errors in Linq.java

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: frontend: Fix findbug errors in Linq.java .. Patch Set 3: Verified; Looks good to me, approved Rebased -- To view, visit http://gerrit.ovirt.org/14212 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: core: remove unneeded static method declarations

2013-04-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: remove unneeded static method declarations .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14245 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: userportal: closing RDP crashes IE

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: userportal: closing RDP crashes IE .. Patch Set 1: (1 inline comment) File frontend/webadmin/modules/gwt-comm

[Engine-patches] Change in ovirt-engine[master]: webadmin: In some locales alerts and Events Icons overlappin...

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: In some locales alerts and Events Icons overlapping on text .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14250 T

[Engine-patches] Change in ovirt-engine[master]: frontend: Fix findbug errors in Linq.java

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: frontend: Fix findbug errors in Linq.java .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14212 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: webadmin: capitalize Spice

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: capitalize Spice .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14231 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: webadmin: In some locales alerts and Events Icons overlappin...

2013-04-25 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: webadmin: In some locales alerts and Events Icons overlapping on text .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14250 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: core: remove unneeded static method declarations

2013-04-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: core: remove unneeded static method declarations .. Patch Set 2: Verified -- To view, visit http://gerrit.ovirt.org/14245 To unsubscribe, visit http://gerrit.ovi

[Engine-patches] Change in ovirt-engine[master]: core: Remove get image info call before images commands

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Remove get image info call before images commands .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13994 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: userportal: Fix legend localization

2013-04-25 Thread gchaplik
Gilad Chaplik has submitted this change and it was merged. Change subject: userportal: Fix legend localization .. userportal: Fix legend localization Legend panel in the three info boxes of the resource view was too narrow to h

[Engine-patches] Change in ovirt-engine[master]: userportal: Fix legend localization

2013-04-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: userportal: Fix legend localization .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14222 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: userportal: Fix legend localization

2013-04-25 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: userportal: Fix legend localization .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14222 To unsubscribe, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: webadmin: expand Up/Down buttons for JP locale

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: expand Up/Down buttons for JP locale .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14246 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: webadmin: fix truncated lined in "New Template" dialog

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: fix truncated lined in "New Template" dialog .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14238 To unsubscribe, v

[Engine-patches] Change in ovirt-engine[master]: restapi: Added usages option to CLI add cluster network

2013-04-25 Thread lvernia
Lior Vernia has posted comments on this change. Change subject: restapi: Added usages option to CLI add cluster network .. Patch Set 1: Verified Michael, please submit as I cannot. -- To view, visit http://gerrit.ovirt.org/138

[Engine-patches] Change in ovirt-engine[master]: engine: custom properties: allow '=' in regex

2013-04-25 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine: custom properties: allow '=' in regex .. Patch Set 3: Looks good to me, but someone else must approve Engine part - looks good. -- To view, visit http:

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

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 6: Please remove ImagesComparerByName as well -- To view, visit http://gerrit.ovirt.org/13849 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: userportal: Fix legend localization

2013-04-25 Thread omasad
ofri masad has posted comments on this change. Change subject: userportal: Fix legend localization .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14222 To unsubscribe, visit http://gerrit.ovirt.org/settings

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

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 6: (1 inline comment) Commit Message Line 10: manner. Line

[Engine-patches] Change in ovirt-engine[master]: core: GetImageInfoVDSCommand - remove apparentsize

2013-04-25 Thread amureini
Allon Mureinik has submitted this change and it was merged. Change subject: core: GetImageInfoVDSCommand - remove apparentsize .. core: GetImageInfoVDSCommand - remove apparentsize In GetImageInfoVDSCommand appertsize is read f

[Engine-patches] Change in ovirt-engine[master]: core: GetImageInfoVDSCommand - remove apparentsize

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: GetImageInfoVDSCommand - remove apparentsize .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14176 To unsub

[Engine-patches] Change in ovirt-engine[master]: core: GetImageInfoVDSCommand - remove apparentsize

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: GetImageInfoVDSCommand - remove apparentsize .. Patch Set 2: Verified; Looks good to me, approved Rebased, no code changes. -- To view, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: Rename storage_pool entity to StoragePool

2013-04-25 Thread tnisan
Tal Nisan has submitted this change and it was merged. Change subject: core: Rename storage_pool entity to StoragePool .. core: Rename storage_pool entity to StoragePool Renamed the storage_pool entity to StoragePool to fit Jav

[Engine-patches] Change in ovirt-engine[master]: core: Rename storage_pool entity to StoragePool

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Rename storage_pool entity to StoragePool .. Patch Set 4: Verified; Looks good to me, approved Rebased -- To view, visit http://gerrit.ovirt.org/14208 To unsu

[Engine-patches] Change in ovirt-engine[master]: core: Rename storage_pool entity to StoragePool

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Rename storage_pool entity to StoragePool .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14208 To unsubscribe,

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

2013-04-25 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Disks and Snapshots sub-tabs resizable columns .. webadmin: Disks and Snapshots sub-tabs resizable columns Disks and Snapshots sub-tabs now supp

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

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

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

2013-04-25 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Disks main-tab: sub-tabs resizable columns .. webadmin: Disks main-tab: sub-tabs resizable columns Sub-tabs under Disks main-tab now support res

[Engine-patches] Change in ovirt-engine[master]: core: Remove GetVdsHooksById2Query

2013-04-25 Thread lhornyak
Laszlo Hornyak has posted comments on this change. Change subject: core: Remove GetVdsHooksById2Query .. Patch Set 4: (1 inline comment) File backend/manager/modules/utils/s

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

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Disks main-tab: sub-tabs resizable columns .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14206 To unsubs

[Engine-patches] Change in ovirt-engine[master]: webadmin: Storage main-tab: sub-tabs resizable columns

2013-04-25 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: Storage main-tab: sub-tabs resizable columns .. webadmin: Storage main-tab: sub-tabs resizable columns Sub-tabs under Storage main-tab now suppo

[Engine-patches] Change in ovirt-engine[master]: webadmin: Storage main-tab: sub-tabs resizable columns

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Storage main-tab: sub-tabs resizable columns .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14205 To unsu

[Engine-patches] Change in ovirt-engine[master]: webadmin: DataCenters main-tab: sub-tabs resizable columns

2013-04-25 Thread derez
Daniel Erez has submitted this change and it was merged. Change subject: webadmin: DataCenters main-tab: sub-tabs resizable columns .. webadmin: DataCenters main-tab: sub-tabs resizable columns Sub-tabs under DataCenters main-t

[Engine-patches] Change in ovirt-engine[master]: webadmin: DataCenters main-tab: sub-tabs resizable columns

2013-04-25 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: DataCenters main-tab: sub-tabs resizable columns .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14105 To

[Engine-patches] Change in ovirt-engine[master]: core: GetImageInfoVDSCommand - remove apparentsize

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: GetImageInfoVDSCommand - remove apparentsize .. Patch Set 1: > A side question - when you have a VM with 2 disks, each disc > on a different domain - is

[Engine-patches] Change in ovirt-engine[master]: core: Rename storage_pool entity to StoragePool

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Rename storage_pool entity to StoragePool .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14208 To unsubscribe, visit h

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

2013-04-25 Thread tnisan
Tal Nisan has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14192 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: userportal, webadmin: Serve static content with web server

2013-04-25 Thread Alon Bar-Lev
Alon Bar-Lev has posted comments on this change. Change subject: userportal, webadmin: Serve static content with web server .. Patch Set 3: Hi, While at it, won't it be nicer to split dynamic content from static content in tw

[Engine-patches] Change in ovirt-engine[master]: webadmin: In some locales alerts and Events Icons overlappin...

2013-04-25 Thread alkaplan
Alona Kaplan has uploaded a new change for review. Change subject: webadmin: In some locales alerts and Events Icons overlapping on text .. webadmin: In some locales alerts and Events Icons overlapping on text On the bottom Ale

[Engine-patches] Change in ovirt-engine[master]: core: Rename storage_pool entity to StoragePool

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Rename storage_pool entity to StoragePool .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14208 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Use static memeber to get ISO regex pattern.

2013-04-25 Thread mlipchuk
Maor Lipchuk has submitted this change and it was merged. Change subject: core: Use static memeber to get ISO regex pattern. .. core: Use static memeber to get ISO regex pattern. Use final static member instead recalculating th

[Engine-patches] Change in ovirt-engine[master]: core: Use name groups for ISO tools

2013-04-25 Thread mlipchuk
Maor Lipchuk has submitted this change and it was merged. Change subject: core: Use name groups for ISO tools .. core: Use name groups for ISO tools Use java7 feature of name groups when matching cluster and tool version for IS

[Engine-patches] Change in ovirt-engine[master]: core: Use name groups for ISO tools

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Use name groups for ISO tools .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13899 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: core: Use static memeber to get ISO regex pattern.

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Use static memeber to get ISO regex pattern. .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14027 To unsubsc

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

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: unify and move Disk Comparators .. Patch Set 6: Verified Fixed spelling mistake in package name -- To view, visit http://gerrit.ovirt.org/13849 To unsubs

[Engine-patches] Change in ovirt-engine[master]: core: Use name groups for ISO tools

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Use name groups for ISO tools .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/13899 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: core: Use static memeber to get ISO regex pattern.

2013-04-25 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: core: Use static memeber to get ISO regex pattern. .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14027 To unsubsc

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

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Introducing the comparators package .. Patch Set 2: Verified Fixed spelling mistake in package name -- To view, visit http://gerrit.ovirt.org/14192 To un

[Engine-patches] Change in ovirt-engine[master]: webadmin: capitalize Spice

2013-04-25 Thread michal . skrivanek
Michal Skrivanek has posted comments on this change. Change subject: webadmin: capitalize Spice .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment)

[Engine-patches] Change in ovirt-engine[master]: frontend: Fix findbug errors in Linq.java

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: frontend: Fix findbug errors in Linq.java .. Patch Set 2: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14212 To uns

[Engine-patches] Change in ovirt-engine[master]: core: restrict comp version of edit storage conn

2013-04-25 Thread abonas
Alissa Bonas has posted comments on this change. Change subject: core: restrict comp version of edit storage conn .. Patch Set 1: Verified the compatibility version error message is displayed correctly in webadmin when trying

[Engine-patches] Change in ovirt-engine[master]: core: remove unneeded static method declarations

2013-04-25 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: remove unneeded static method declarations .. Patch Set 1: If you're going to do that, please also replace bFacade.getInstance().getVmPoolDao() with getV

[Engine-patches] Change in ovirt-engine[master]: core: change the order of checks in canDoAction of migrate v...

2013-04-25 Thread ofrenkel
Omer Frenkel has submitted this change and it was merged. Change subject: core: change the order of checks in canDoAction of migrate vm .. core: change the order of checks in canDoAction of migrate vm Change the order in which

[Engine-patches] Change in ovirt-engine[master]: core: restrict comp version of edit storage conn

2013-04-25 Thread abonas
Alissa Bonas has uploaded a new change for review. Change subject: core: restrict comp version of edit storage conn .. core: restrict comp version of edit storage conn Added upgrade script that restricts the compatibility versio

[Engine-patches] Change in ovirt-engine[master]: core: change the order of checks in canDoAction of migrate v...

2013-04-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: change the order of checks in canDoAction of migrate vm .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14247

[Engine-patches] Change in ovirt-engine[master]: core: [cleanup] split execute method of CreateAllSnapshots

2013-04-25 Thread ofrenkel
Omer Frenkel has submitted this change and it was merged. Change subject: core: [cleanup] split execute method of CreateAllSnapshots .. core: [cleanup] split execute method of CreateAllSnapshots CreateAllSnapshotsFromVmCommand#

[Engine-patches] Change in ovirt-engine[master]: core: [cleanup] split execute method of CreateAllSnapshots

2013-04-25 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: [cleanup] split execute method of CreateAllSnapshots .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/14179 To

[Engine-patches] Change in ovirt-engine[master]: core: change the order of checks in canDoAction of migrate v...

2013-04-25 Thread ahadas
Arik Hadas has posted comments on this change. Change subject: core: change the order of checks in canDoAction of migrate vm .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/14247 To unsubscribe, visit http://

  1   2   3   >