[Engine-patches] Change in ovirt-engine[master]: core: Added internal param to create snapshot params

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Added internal param to create snapshot params .. core: Added internal param to create snapshot params Snapshot type is needed for internal use,

[Engine-patches] Change in ovirt-engine[master]: core: Added internal param to create snapshot params

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Added internal param to create snapshot params .. Patch Set 3: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2047 To unsu

[Engine-patches] Change in ovirt-engine[master]: engine-core: [autorecovery] add column to db

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: engine-core: [autorecovery] add column to db .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/2010 To un

[Engine-patches] Change in ovirt-engine[master]: core: Exctract VM network management code

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Exctract VM network management code .. core: Exctract VM network management code Extracted code that is used for add/remove of VmNetworkInterface

[Engine-patches] Change in ovirt-engine[master]: core: Exctract VM network management code

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Exctract VM network management code .. Patch Set 6: Verified; Looks good to me, approved Rebased, already acked by yzaslavs -- To view, visit http://gerri

[Engine-patches] Change in ovirt-engine[master]: engine: Add consume quota permission to add Tempalte

2012-02-20 Thread lpeer
Livnat Peer has posted comments on this change. Change subject: engine: Add consume quota permission to add Tempalte .. Patch Set 6: (1 inline comment) File backend/manager/

[Engine-patches] Change in ovirt-engine[master]: engine: Add permission support for Quota consume

2012-02-20 Thread lpeer
Livnat Peer has posted comments on this change. Change subject: engine: Add permission support for Quota consume .. Patch Set 6: (3 inline comments) >From this patch it is not obvious why you need addQuotaPermissionSubject both

[Engine-patches] Change in ovirt-engine[master]: core: Added internal param to create snapshot params

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Added internal param to create snapshot params .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2047 To unsu

[Engine-patches] Change in ovirt-engine[master]: core: Exctract VM network management code

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Exctract VM network management code .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1969 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: engine-core: Fixing AddVmCommand (Review Only)

2012-02-20 Thread jchoate
Jonathan Choate has posted comments on this change. Change subject: engine-core: Fixing AddVmCommand (Review Only) .. Patch Set 1: (3 inline comments) File backend/manager/m

[Engine-patches] Change in ovirt-engine[master]: engine-core: Fixing AddVmTemplateCommand for multiple storag...

2012-02-20 Thread jchoate
Jonathan Choate has posted comments on this change. Change subject: engine-core: Fixing AddVmTemplateCommand for multiple storage domains .. Patch Set 2: (2 inline comments)

[Engine-patches] Change in ovirt-engine[master]: engine: Add Quota canDoActionErrors

2012-02-20 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: engine: Add Quota canDoActionErrors .. Patch Set 6: rebase -- To view, visit http://gerrit.ovirt.org/2080 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: engine: Add default quota indication field

2012-02-20 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: engine: Add default quota indication field .. Patch Set 4: Adding is_default_quota to update sp -- To view, visit http://gerrit.ovirt.org/2075 To unsubscribe, vi

[Engine-patches] Change in ovirt-engine[master]: core: dal: Removing DiskImageTemplateDAO from DAL

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: dal: Removing DiskImageTemplateDAO from DAL .. core: dal: Removing DiskImageTemplateDAO from DAL Change-Id: I97d362095ec9af12041ea1ff3d4a33bbc5

[Engine-patches] Change in ovirt-engine[master]: core: dal: Removing DiskImageTemplateDAO from DAL

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: dal: Removing DiskImageTemplateDAO from DAL .. Patch Set 21: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1572 To unsub

[Engine-patches] Change in ovirt-engine[master]: core: common: Removing DiskImageTemplate entity

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: common: Removing DiskImageTemplate entity .. core: common: Removing DiskImageTemplate entity Change-Id: I31d5ba8badd3510ac71b7de5581673d9582a13d7

[Engine-patches] Change in ovirt-engine[master]: core: common: Removing DiskImageTemplate entity

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: common: Removing DiskImageTemplate entity .. Patch Set 21: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1571 To unsubscr

[Engine-patches] Change in ovirt-engine[master]: core: Using DiskImage instead of DiskImage template

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Using DiskImage instead of DiskImage template .. core: Using DiskImage instead of DiskImage template The changes in general are as follows: 1. Us

[Engine-patches] Change in ovirt-engine[master]: core: Using DiskImage instead of DiskImage template

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Using DiskImage instead of DiskImage template .. Patch Set 21: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1570 To unsu

[Engine-patches] Change in ovirt-engine[master]: core: dao removing of DiskImageTemplate

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: dao removing of DiskImageTemplate .. core: dao removing of DiskImageTemplate This patch removes DiskImageTemplate usage from DAL Change-Id: I977

[Engine-patches] Change in ovirt-engine[master]: core: dao removing of DiskImageTemplate

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: dao removing of DiskImageTemplate .. Patch Set 17: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1568 To unsubscribe, vis

[Engine-patches] Change in ovirt-engine[master]: core: Fixing DB conf (and conf test) files

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: Fixing DB conf (and conf test) files .. core: Fixing DB conf (and conf test) files Fixing DB engine-daos.properties and fixtures.xml files to ref

[Engine-patches] Change in ovirt-engine[master]: core: Fixing DB conf (and conf test) files

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: Fixing DB conf (and conf test) files .. Patch Set 17: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1567 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: common: Fixing entities to reflect removal of DiskImageTempl...

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: common: Fixing entities to reflect removal of DiskImageTemplate .. -- To view, visit http://gerrit.ovirt.org/1569 To unsubscribe, visit http://gerrit.ovirt.org

[Engine-patches] Change in ovirt-engine[master]: common: Fixing entities to reflect removal of DiskImageTempl...

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: common: Fixing entities to reflect removal of DiskImageTemplate .. common: Fixing entities to reflect removal of DiskImageTemplate VmTemplate holds now

[Engine-patches] Change in ovirt-engine[master]: common: Fixing entities to reflect removal of DiskImageTempl...

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: common: Fixing entities to reflect removal of DiskImageTemplate .. Patch Set 17: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/

[Engine-patches] Change in ovirt-engine[master]: common: introducing VmEntityType

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: common: introducing VmEntityType .. Patch Set 12: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1882 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: common: introducing VmEntityType

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: common: introducing VmEntityType .. common: introducing VmEntityType This enum will help destinguish between entities if they are associated with VM or

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

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: webadmin: Changes in webadmin .. Patch Set 17: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1566 To unsubscribe, visit http://

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

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: webadmin: Changes in webadmin .. webadmin: Changes in webadmin This patch removes refence to DiskImageTeplated from SharedGwt.gwt.xl and introduces the

[Engine-patches] Change in ovirt-engine[master]: core: db scripts for removing disk image template

2012-02-20 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: core: db scripts for removing disk image template .. core: db scripts for removing disk image template Patch in order to remove template_images (which

[Engine-patches] Change in ovirt-engine[master]: core: db scripts for removing disk image template

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: db scripts for removing disk image template .. Patch Set 16: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1565 To unsubs

[Engine-patches] Change in ovirt-engine[master]: engine: Add default quota indication field

2012-02-20 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine: Add default quota indication field .. Patch Set 3: missing change to update quota stored procedure -- To view, visit http://gerrit.ovirt.org/2075 To unsu

[Engine-patches] Change in ovirt-engine[master]: webadmin: integrating reports to webadmin (commnets fixes)

2012-02-20 Thread danny . ran
Danny Rankevich has submitted this change and it was merged. Change subject: webadmin: integrating reports to webadmin (commnets fixes) .. webadmin: integrating reports to webadmin (commnets fixes) no bz - Getting Reports.xml

[Engine-patches] Change in ovirt-engine[master]: webadmin: integrating reports to webadmin (commnets fixes)

2012-02-20 Thread danny . ran
Danny Rankevich has posted comments on this change. Change subject: webadmin: integrating reports to webadmin (commnets fixes) .. Patch Set 6: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1952

[Engine-patches] Change in ovirt-engine[master]: engine: Add permission support for Quota consume

2012-02-20 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine: Add permission support for Quota consume .. Patch Set 5: (2 inline comments) File backend/manager/mo

[Engine-patches] Change in ovirt-engine[master]: webadmin: Tab UI reuse concept

2012-02-20 Thread gchaplik
Gilad Chaplik has posted comments on this change. Change subject: webadmin: Tab UI reuse concept .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/2120 To unsubscribe, vis

[Engine-patches] Change in ovirt-reports[master]: reports: added reports directory xml

2012-02-20 Thread alkaplan
Alona Kaplan has posted comments on this change. Change subject: reports: added reports directory xml .. Patch Set 3: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/2125 To u

[Engine-patches] Change in ovirt-engine[master]: engine: Add permission support for Quota consume

2012-02-20 Thread mlipchuk
Maor Lipchuk has posted comments on this change. Change subject: engine: Add permission support for Quota consume .. Patch Set 5: changed action to actionGroup Removed the use of quota consume and add a helper method in the quo

[Engine-patches] Change in ovirt-engine[master]: core: Filtering for StorageDoaminDAO.get

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Filtering for StorageDoaminDAO.get .. core: Filtering for StorageDoaminDAO.get Added optional permission filtering to the get(Guid) method and the

[Engine-patches] Change in ovirt-engine[master]: packaging: add controller class (plugins step 3/3)

2012-02-20 Thread oschreib
Ofer Schreiber has posted comments on this change. Change subject: packaging: add controller class (plugins step 3/3) .. Patch Set 7: Verified -- To view, visit http://gerrit.ovirt.org/1867 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Filtering for StorageDoaminDAO.get

2012-02-20 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Filtering for StorageDoaminDAO.get .. Patch Set 1: Verified -- To view, visit http://gerrit.ovirt.org/2128 To unsubscribe, visit http://gerrit.ovirt.org/s

[Engine-patches] Change in ovirt-engine[master]: core: Filtering for StorageDoaminDAO.get

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Filtering for StorageDoaminDAO.get .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2128 To unsubscribe, visit http://g

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetStorageDomainByIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup GetStorageDomainByIdQuery .. core: Cleanup GetStorageDomainByIdQuery Cleaned up * imports using Eclipse IDE. Change-Id: I38a3ea417b7671e2

[Engine-patches] Change in ovirt-engine[master]: core: Filter GetStorageDomainsByStoragePoolIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Filter GetStorageDomainsByStoragePoolIdQuery .. core: Filter GetStorageDomainsByStoragePoolIdQuery Added optional permission filtering to GetStora

[Engine-patches] Change in ovirt-engine[master]: core: Filter GetStorageDomainsByStoragePoolIdQuery

2012-02-20 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Filter GetStorageDomainsByStoragePoolIdQuery .. Patch Set 3: Verified -- To view, visit http://gerrit.ovirt.org/2117 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetStorageDomainByIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup GetStorageDomainByIdQuery .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2118 To unsubscribe, visit http://ge

[Engine-patches] Change in ovirt-engine[master]: core: Filter GetStorageDomainsByStoragePoolIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Filter GetStorageDomainsByStoragePoolIdQuery .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2117 To unsubscribe, visi

[Engine-patches] Change in ovirt-engine[master]: core: Ffiltering for StorageDomainDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Ffiltering for StorageDomainDAO .. core: Ffiltering for StorageDomainDAO Added optional permission filtering for StorageDomainDAO.getAllForStorage

[Engine-patches] Change in ovirt-engine[master]: core: Ffiltering for StorageDomainDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Ffiltering for StorageDomainDAO .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2116 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup StorageDoaminDAO's implementations

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup StorageDoaminDAO's implementations .. core: Cleanup StorageDoaminDAO's implementations Cleaned up warnings (mostly missplaced @SuppressWar

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup StorageDoaminDAO's implementations

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup StorageDoaminDAO's implementations .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2062 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Clean GetStorageDomainsByStoragePoolIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Clean GetStorageDomainsByStoragePoolIdQuery .. core: Clean GetStorageDomainsByStoragePoolIdQuery Replace * imports with specific imports with Ecli

[Engine-patches] Change in ovirt-engine[master]: core: Clean GetStorageDomainsByStoragePoolIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Clean GetStorageDomainsByStoragePoolIdQuery .. Patch Set 4: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2061 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Added filtering to GetStoragePoolByIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Added filtering to GetStoragePoolByIdQuery .. core: Added filtering to GetStoragePoolByIdQuery Added optional filtering according to user permissi

[Engine-patches] Change in ovirt-engine[master]: core: Added filtering to GetStoragePoolByIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Added filtering to GetStoragePoolByIdQuery .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2046 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Add filtering to StoragePoolDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add filtering to StoragePoolDAO .. core: Add filtering to StoragePoolDAO Added optional permissions filtering to StoragePoolDAO and undelying stor

[Engine-patches] Change in ovirt-engine[master]: core: Add filtering to StoragePoolDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add filtering to StoragePoolDAO .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2045 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup StoragePoolDAO implementations

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup StoragePoolDAO implementations .. core: Cleanup StoragePoolDAO implementations Cleanup warnings, dead code pieces Change-Id: I92c157aec04

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup StoragePoolDAO implementations

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup StoragePoolDAO implementations .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2044 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetStoragePoolByIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup GetStoragePoolByIdQuery .. core: Cleanup GetStoragePoolByIdQuery Remove wildcard imports. Change-Id: Iddf1f8ac84ee4c3e6c8596d17c8110cf400

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetStoragePoolByIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup GetStoragePoolByIdQuery .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2043 To unsubscribe, visit http://gerr

[Engine-patches] Change in ovirt-engine[master]: core: Filtering GetAllNetworksByClusterIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Filtering GetAllNetworksByClusterIdQuery .. core: Filtering GetAllNetworksByClusterIdQuery Refactored GetAllNetworksByClusterIdQuery to use new AP

[Engine-patches] Change in ovirt-engine[master]: core: Filtering GetAllNetworksByClusterIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Filtering GetAllNetworksByClusterIdQuery .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2042 To unsubscribe, visit ht

[Engine-patches] Change in ovirt-engine[master]: core: Add optional filtering to NetworkDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add optional filtering to NetworkDAO .. core: Add optional filtering to NetworkDAO Filtering was added to the DAO's API and the stored procedure t

[Engine-patches] Change in ovirt-engine[master]: core: Add optional filtering to NetworkDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add optional filtering to NetworkDAO .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2041 To unsubscribe, visit http:/

[Engine-patches] Change in ovirt-engine[master]: core: Fix typos in NetworkDAOTest

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Fix typos in NetworkDAOTest .. core: Fix typos in NetworkDAOTest Change-Id: Ib1f4f3210d3c7be71832375feeb1b7188a848e22 --- M backend/manager/modul

[Engine-patches] Change in ovirt-engine[master]: core: Fix typos in NetworkDAOTest

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Fix typos in NetworkDAOTest .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2040 To unsubscribe, visit http://gerrit.o

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetAllNetworkByClusterId

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup GetAllNetworkByClusterId .. core: Cleanup GetAllNetworkByClusterId Changed query to select all the fields of network_cluster instead of th

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetAllNetworkByClusterId

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup GetAllNetworkByClusterId .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2039 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup warnings on NetworkDAODbFacadeImpl

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup warnings on NetworkDAODbFacadeImpl .. core: Cleanup warnings on NetworkDAODbFacadeImpl Removed useless @SupressWarning("unchecked"), empty

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup warnings on NetworkDAODbFacadeImpl

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup warnings on NetworkDAODbFacadeImpl .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2038 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Imports for GetAllNetworksByClusterIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Imports for GetAllNetworksByClusterIdQuery .. core: Imports for GetAllNetworksByClusterIdQuery Cleaned up the * improts using Eclipse IDE's automa

[Engine-patches] Change in ovirt-engine[master]: core: Imports for GetAllNetworksByClusterIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Imports for GetAllNetworksByClusterIdQuery .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2037 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Move user ID constants to BaseDAOTestCase

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Move user ID constants to BaseDAOTestCase .. core: Move user ID constants to BaseDAOTestCase Constants were pulled up so they could be used by tes

[Engine-patches] Change in ovirt-engine[master]: core: Move user ID constants to BaseDAOTestCase

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Move user ID constants to BaseDAOTestCase .. Patch Set 5: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2036 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: Filtering for GetAllVmSnapshotsByDriveQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Filtering for GetAllVmSnapshotsByDriveQuery .. core: Filtering for GetAllVmSnapshotsByDriveQuery Permission filtering according to the user who is

[Engine-patches] Change in ovirt-engine[master]: core: Filtering for GetAllVmSnapshotsByDriveQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Filtering for GetAllVmSnapshotsByDriveQuery .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1993 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Refactoring GetAllVmSnapshotsByDriveQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Refactoring GetAllVmSnapshotsByDriveQuery .. core: Refactoring GetAllVmSnapshotsByDriveQuery Call DiskImageDAO directly and then feed the results

[Engine-patches] Change in ovirt-engine[master]: core: Refactoring GetAllVmSnapshotsByDriveQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Refactoring GetAllVmSnapshotsByDriveQuery .. Patch Set 7: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1992 To unsubscribe, visit h

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetAllVmSnapshotsByDriveQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Cleanup GetAllVmSnapshotsByDriveQuery .. core: Cleanup GetAllVmSnapshotsByDriveQuery removed useless logger, suppressed warnings that we're prepar

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetAllVmSnapshotsByDriveQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Cleanup GetAllVmSnapshotsByDriveQuery .. Patch Set 10: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1910 To unsubscribe, visit http

[Engine-patches] Change in ovirt-engine[master]: core: Permissions for GetAllDisksByVmIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Permissions for GetAllDisksByVmIdQuery .. core: Permissions for GetAllDisksByVmIdQuery Use the permission filtering API in GetAllDisksByVmIdQuery.

[Engine-patches] Change in ovirt-engine[master]: core: Permissions for GetAllDisksByVmIdQuery

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Permissions for GetAllDisksByVmIdQuery .. Patch Set 14: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1872 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-reports[master]: reports: added reports directory xml

2012-02-20 Thread ydary
Yaniv Dary has posted comments on this change. Change subject: reports: added reports directory xml .. Patch Set 3: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/2125 To uns

[Engine-patches] Change in ovirt-reports[master]: reports: changed dates order to desc

2012-02-20 Thread ydary
Yaniv Dary has posted comments on this change. Change subject: reports: changed dates order to desc .. Patch Set 2: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/2126 To uns

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT dal: Removing DiskImageTemplateDAO fro...

2012-02-20 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: DO NOT SUBMIT dal: Removing DiskImageTemplateDAO from DAL .. Patch Set 19: Mike - following your last comment - it was ineeded removed from engine-dao

[Engine-patches] Change in ovirt-engine[master]: core: Add filtering to VmDeviceDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Add filtering to VmDeviceDAO .. core: Add filtering to VmDeviceDAO Added overloaded versions of required methods with filtering parameters, and th

[Engine-patches] Change in ovirt-engine[master]: core: Add filtering to VmDeviceDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Add filtering to VmDeviceDAO .. Patch Set 16: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1845 To unsubscribe, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: core: Added filtering option to DiskImageDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has submitted this change and it was merged. Change subject: core: Added filtering option to DiskImageDAO .. core: Added filtering option to DiskImageDAO Overloaded DiskImageDAO.getAllForVM with filtering paramete

[Engine-patches] Change in ovirt-engine[master]: core: Added filtering option to DiskImageDAO

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: Added filtering option to DiskImageDAO .. Patch Set 17: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1841 To unsubscribe, visit htt

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT dal: Removing DiskImageTemplateDAO fro...

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: DO NOT SUBMIT dal: Removing DiskImageTemplateDAO from DAL .. Patch Set 19: Don't you need to remove it from the DAOs properties file also? -- To view,

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT common: Removing DiskImageTemplate entit...

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: DO NOT SUBMIT common: Removing DiskImageTemplate entity .. Patch Set 19: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1571 To unsub

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT BLL: Using DiskImage instead of DiskImag...

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: DO NOT SUBMIT BLL: Using DiskImage instead of DiskImage template .. Patch Set 19: (1 inline comment)

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT DAL: removing of DiskImageTemplate

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: DO NOT SUBMIT DAL: removing of DiskImageTemplate .. Patch Set 15: (1 inline comment) File backend/man

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT Fixing DB conf (and conf test) files

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: DO NOT SUBMIT Fixing DB conf (and conf test) files .. Patch Set 15: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1567 To unsubscrib

[Engine-patches] Change in ovirt-engine[master]: common: DO NOT SUBMIT Fixing entities to reflect removal of ...

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: common: DO NOT SUBMIT Fixing entities to reflect removal of DiskImageTemplate .. Patch Set 15: Looks good to me, approved -- To view, visit http://gerrit.ovirt.

[Engine-patches] Change in ovirt-engine[master]: common: DO NOT SUBMIT common introducing VmEntityType

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: common: DO NOT SUBMIT common introducing VmEntityType .. Patch Set 10: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1882 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: DO NOT SUBMIT db scripts for removing disk image templ...

2012-02-20 Thread mkolesni
Mike Kolesnik has posted comments on this change. Change subject: core: DO NOT SUBMIT db scripts for removing disk image template .. Patch Set 15: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1565 To uns

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup GetStorageDomainByIdQuery

2012-02-20 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Cleanup GetStorageDomainByIdQuery .. Patch Set 3: Verified no changes since mkolesni's approval - just a rebase. -- To view, visit http://gerrit.ovirt.or

  1   2   3   >