[Engine-patches] Change in ovirt-engine[master]: engine-core: add lun to the log when can't connect

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine-core: add lun to the log when can't connect .. Patch Set 1: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1650 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add JobRepository

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add JobRepository .. Patch Set 3: Next patch will change the package of JobRepository from org.ovirt.engine.core.dao.job to

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add Tables

2012-02-07 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: Task Manager Add Tables .. Patch Set 8: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1083 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Fixing VM Static DAO test.

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Fixing VM Static DAO test. .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1645 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Fixing VM Static DAO test.

2012-02-07 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: core: Fixing VM Static DAO test. .. core: Fixing VM Static DAO test. Now that VmStatic does not have originType automatically set this fix is required as

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add stored-procedures

2012-02-07 Thread emesika
Eli Mesika has posted comments on this change. Change subject: core: Task Manager Add stored-procedures .. Patch Set 9: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1084 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add Tables

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add Tables .. Patch Set 8: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1083 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add Tables

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add Tables .. Patch Set 9: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1083 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add Tables

2012-02-07 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: core: Task Manager Add Tables .. core: Task Manager Add Tables The patch adds 3 tables for the Task Manager: Job table - stores the job entity, describes

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add stored-procedures

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add stored-procedures .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1084 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add stored-procedures

2012-02-07 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: core: Task Manager Add stored-procedures .. core: Task Manager Add stored-procedures The patch adds stored-procedures for the Task Manager feature. The

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add business entities

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add business entities .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1085 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add ExecutionContext

2012-02-07 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: core: Task Manager Add ExecutionContext .. core: Task Manager Add ExecutionContext The patch introduces the ExecutionContext class which acts as the

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add DAO objects

2012-02-07 Thread masayag
Moti Asayag has submitted this change and it was merged. Change subject: core: Task Manager Add DAO objects .. core: Task Manager Add DAO objects The patch adds the DAO interfaces and implementation for the Job, Step and

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add DAO objects

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add DAO objects .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1086 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: engine-core: Copy Disk Command

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine-core: Copy Disk Command .. Patch Set 2: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: engine-core: Move disk command

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine-core: Move disk command .. Patch Set 3: (1 inline comment) Except small remark, everything is ok for me.( Also adding someone from gwt team for error

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: packaging - Remove release field from tarball

2012-02-07 Thread oschreib
Ofer Schreiber has uploaded a new change for review. Change subject: sdk: packaging - Remove release field from tarball .. sdk: packaging - Remove release field from tarball The source tarball shouldn't containt the release

[Engine-patches] [oVirt Jenkins] ovirt_engine_dao_unit_tests - Build # 104 - Fixed!

2012-02-07 Thread Jenkins oVirt Server
ovirt_engine_dao_unit_tests - Build # 104 - Fixed: Check console output at http://jenkins.ovirt.org/job/ovirt_engine_dao_unit_tests/104/console to view the results. ___ Engine-patches mailing list Engine-patches@ovirt.org

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - vdsbroker changes

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine : Hotplug - vdsbroker changes .. Patch Set 2: (3 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: core: Views for handling query permissions

2012-02-07 Thread amureini
Allon Mureinik has uploaded a new change for review. Change subject: core: Views for handling query permissions .. core: Views for handling query permissions Each view gives a flat set of user IDs and the objects of that type

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: packaging - Remove release field from tarball

2012-02-07 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: sdk: packaging - Remove release field from tarball .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1698 To

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: packaging - Remove release field from tarball

2012-02-07 Thread mpastern
Michael Pasternak has submitted this change and it was merged. Change subject: sdk: packaging - Remove release field from tarball .. sdk: packaging - Remove release field from tarball The source tarball shouldn't containt the

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - vdsbroker changes

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine : Hotplug - vdsbroker changes .. Patch Set 2: (3 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add JobRepositoryFactory

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add JobRepositoryFactory .. Patch Set 7: moved JobRepository* from org.ovirt.engine.core.dao.job. to org.ovirt.engine.core.dal.job.. -- To

[Engine-patches] Change in ovirt-engine[master]: core: Task Manager Add Jobs handling on server startup

2012-02-07 Thread masayag
Moti Asayag has posted comments on this change. Change subject: core: Task Manager Add Jobs handling on server startup .. Patch Set 6: moved JobRepository* from org.ovirt.engine.core.dao.job. to

[Engine-patches] Change in ovirt-engine[master]: Engine : Hotplug feature - error messages

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: Engine : Hotplug feature - error messages .. Patch Set 2: (2 inline comments) File

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: rebase on 1.3 upstream

2012-02-07 Thread mpastern
Michael Pasternak has posted comments on this change. Change subject: sdk: rebase on 1.3 upstream .. Patch Set 1: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1700 To unsubscribe, visit

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

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Cleanup warnings in CommandBase .. Patch Set 9: Looks good to me, but someone else must approve reviewed and approved by mkublin in previous patch - was

[Engine-patches] Change in ovirt-engine[master]: core: addUserTheadContext in SessionDataContainer

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: addUserTheadContext in SessionDataContainer .. Patch Set 10: Looks good to me, but someone else must approve reviewed and approved by mkublin in previous

[Engine-patches] Change in ovirt-engine[master]: core: Clean up warnings on Backend

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Clean up warnings on Backend .. Patch Set 9: Looks good to me, but someone else must approve reviewed and approved by mkublin in previous patch - was

[Engine-patches] Change in ovirt-engine[master]: core: Introduce notion of internal queries

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Introduce notion of internal queries .. Patch Set 8: Looks good to me, but someone else must approve reviewed and approved by mkublin in previous patch -

[Engine-patches] Change in ovirt-engine[master]: webadmin: Suppress Etag for GWT selector scripts (#784379)

2012-02-07 Thread derez
Daniel Erez has posted comments on this change. Change subject: webadmin: Suppress Etag for GWT selector scripts (#784379) .. Patch Set 2: Verified; Looks good to me, but someone else must approve -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: packaging: classify params and groups (plugins step 1/3)

2012-02-07 Thread eedri
Eyal Edri has posted comments on this change. Change subject: packaging: classify params and groups (plugins step 1/3) .. Patch Set 4: (2 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - a new command added, main logic

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine : Hotplug - a new command added, main logic .. Patch Set 2: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: packaging: Fixed using ovirt-engine instead of 'rhevm' (#...

2012-02-07 Thread oschreib
Ofer Schreiber has posted comments on this change. Change subject: packaging: Fixed using ovirt-engine instead of 'rhevm' (#784850) .. Patch Set 4: Verified; Looks good to me, approved -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: packaging: Fixed using ovirt-engine instead of 'rhevm' (#...

2012-02-07 Thread oschreib
Ofer Schreiber has posted comments on this change. Change subject: packaging: Fixed using ovirt-engine instead of 'rhevm' (#784850) .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: packaging: Fixed using ovirt-engine instead of 'rhevm' (#...

2012-02-07 Thread oschreib
Ofer Schreiber has submitted this change and it was merged. Change subject: packaging: Fixed using ovirt-engine instead of 'rhevm' (#784850) .. packaging: Fixed using ovirt-engine instead of 'rhevm' (#784850)

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - a new command added, main logic

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine : Hotplug - a new command added, main logic .. Patch Set 2: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: core: Test suite for SessionDataContainer

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Test suite for SessionDataContainer .. Patch Set 11: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1315 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: engine-core: host with same name/address (#786104)

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine-core: host with same name/address (#786104) .. Patch Set 1: that's good for data integrity, i wonder if we have a nice way to fail this in canDoAction

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup to basic query classes

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Cleanup to basic query classes .. Patch Set 10: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1293 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Introduce notion of internal queries

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Introduce notion of internal queries .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1386 To unsubscribe, visit

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

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Cleanup warnings in CommandBase .. Patch Set 9: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1316 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: addUserTheadContext in SessionDataContainer

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: addUserTheadContext in SessionDataContainer .. Patch Set 10: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1317 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Clean up warnings on Backend

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Clean up warnings on Backend .. Patch Set 9: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1375 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - a new command added, main logic

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine : Hotplug - a new command added, main logic .. Patch Set 2: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug feature - db scripts

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine : Hotplug feature - db scripts .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1631 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: VdsInstaller: drop netconsole code

2012-02-07 Thread dfediuck
Doron Fediuck has posted comments on this change. Change subject: VdsInstaller: drop netconsole code .. Patch Set 1: I would prefer that you didn't submit this (1 inline comment) Hi Dan, thanks. This is still not ready to be

[Engine-patches] Change in ovirt-engine[master]: Engine : Hotplug feature - error messages

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: Engine : Hotplug feature - error messages .. Patch Set 3: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1632 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Take 2: Prepration for query permissions

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Take 2: Prepration for query permissions .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1272 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Checking permissions when executing queries

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Checking permissions when executing queries .. Patch Set 8: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1585 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: engine-core: host with same name/address (#786104)

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: engine-core: host with same name/address (#786104) .. Patch Set 1: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/1648

[Engine-patches] Change in ovirt-engine[master]: restapi: Vm Start Time Displayed When Vm is Down

2012-02-07 Thread oliel
ol...@redhat.com has uploaded a new change for review. Change subject: restapi: Vm Start Time Displayed When Vm is Down .. restapi: Vm Start Time Displayed When Vm is Down Change-Id: Ide3aed65480d2cbfa2096d37a070c190ba637a94

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - a new command added, main logic

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine : Hotplug - a new command added, main logic .. Patch Set 4: Verified; Looks good to me, approved already acked, rebase -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - vdsbroker changes

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine : Hotplug - vdsbroker changes .. Patch Set 3: Verified; Looks good to me, approved already acked, rebase -- To view, visit http://gerrit.ovirt.org/1634

[Engine-patches] Change in ovirt-engine[master]: Enginne : Hotplug feature - changes at api

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: Enginne : Hotplug feature - changes at api .. Patch Set 3: Verified; Looks good to me, approved already acked, rebase -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: Engine : Hotplug feature - error messages

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: Engine : Hotplug feature - error messages .. Patch Set 3: Verified; Looks good to me, approved already acked, rebase -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - a new command added, main logic

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: engine : Hotplug - a new command added, main logic .. engine : Hotplug - a new command added, main logic The following patch caontains a new added

[Engine-patches] Change in ovirt-engine[master]: engine : Hotplug - vdsbroker changes

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: engine : Hotplug - vdsbroker changes .. engine : Hotplug - vdsbroker changes This change is dine on vdsBroker level in order to allow an rpc call with

[Engine-patches] Change in ovirt-engine[master]: Engine : Hotplug feature - error messages

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: Engine : Hotplug feature - error messages .. Engine : Hotplug feature - error messages The following patch contains a newally added error messages and

[Engine-patches] Change in ovirt-engine[master]: Enginne : Hotplug feature - changes at api

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: Enginne : Hotplug feature - changes at api .. Enginne : Hotplug feature - changes at api The following changes are done in API, because of a new field

[Engine-patches] Change in ovirt-engine[master]: [DO NOT SUBMIT!] core: Rename MergeSnapshot to RemoveSnapsho...

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: [DO NOT SUBMIT!] core: Rename MergeSnapshot to RemoveSnapshot .. Patch Set 2: (2 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: [DO NOT SUBMIT!] core: Cleanup of parameters - remove the JA...

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: [DO NOT SUBMIT!] core: Cleanup of parameters - remove the JAXB annotations .. Patch Set 2: Looks good to me, approved -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: engine-core: host with same name/address (#786104)

2012-02-07 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: engine-core: host with same name/address (#786104) .. Patch Set 2: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1648 To

[Engine-patches] Change in ovirt-engine[master]: engine-core: host with same name/address (#786104)

2012-02-07 Thread yzaslavs
Yair Zaslavsky has submitted this change and it was merged. Change subject: engine-core: host with same name/address (#786104) .. engine-core: host with same name/address (#786104)

[Engine-patches] Change in ovirt-engine[master]: [DO NOT SUBMIT!] core: Simplify RemoveSnapshot command to ge...

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: [DO NOT SUBMIT!] core: Simplify RemoveSnapshot command to get id .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1130 To

[Engine-patches] Change in ovirt-engine[master]: core: Take 2: Prepration for query permissions

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Take 2: Prepration for query permissions .. -- To view, visit http://gerrit.ovirt.org/1272 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup to basic query classes

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Cleanup to basic query classes .. Patch Set 10: Verified -- To view, visit http://gerrit.ovirt.org/1293 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Test suite for SessionDataContainer

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Test suite for SessionDataContainer .. Patch Set 11: Verified -- To view, visit http://gerrit.ovirt.org/1315 To unsubscribe, visit

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

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Cleanup warnings in CommandBase .. Patch Set 9: Verified -- To view, visit http://gerrit.ovirt.org/1316 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: addUserTheadContext in SessionDataContainer

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: addUserTheadContext in SessionDataContainer .. Patch Set 10: Verified -- To view, visit http://gerrit.ovirt.org/1317 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Checking permissions when executing queries

2012-02-07 Thread amureini
Allon Mureinik has posted comments on this change. Change subject: core: Checking permissions when executing queries .. Patch Set 8: Verified -- To view, visit http://gerrit.ovirt.org/1585 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Take 2: Prepration for query permissions

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Take 2: Prepration for query permissions .. -- To view, visit http://gerrit.ovirt.org/1272 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: core: Checking permissions when executing queries

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Checking permissions when executing queries .. Patch Set 8: Verified -- To view, visit http://gerrit.ovirt.org/1585 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Introduce notion of internal queries

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Introduce notion of internal queries .. -- To view, visit http://gerrit.ovirt.org/1386 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: core: Clean up warnings on Backend

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Clean up warnings on Backend .. Patch Set 9: Verified -- To view, visit http://gerrit.ovirt.org/1375 To unsubscribe, visit

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

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Cleanup warnings in CommandBase .. -- To view, visit http://gerrit.ovirt.org/1316 To unsubscribe, visit http://gerrit.ovirt.org/settings

[Engine-patches] Change in ovirt-engine[master]: core: Test suite for SessionDataContainer

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Test suite for SessionDataContainer .. Patch Set 12: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1315 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Introduce notion of internal queries

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Introduce notion of internal queries .. Patch Set 9: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1386 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Clean up warnings on Backend

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Clean up warnings on Backend .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1375 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Cleanup to basic query classes

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Cleanup to basic query classes .. Patch Set 11: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1293 To unsubscribe, visit

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

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Cleanup warnings in CommandBase .. Patch Set 10: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1316 To unsubscribe,

[Engine-patches] Change in ovirt-engine[master]: core: Take 2: Prepration for query permissions

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: core: Take 2: Prepration for query permissions .. Patch Set 9: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1272 To

[Engine-patches] Change in ovirt-engine[master]: core: Introduce notion of internal queries

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: core: Introduce notion of internal queries .. core: Introduce notion of internal queries currently, this is only a property (which can be set and

[Engine-patches] Change in ovirt-engine[master]: core: addUserTheadContext in SessionDataContainer

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: core: addUserTheadContext in SessionDataContainer .. core: addUserTheadContext in SessionDataContainer Move addUserToThreadContext from CommandBase to

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

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: core: Cleanup warnings in CommandBase .. core: Cleanup warnings in CommandBase (Already approved by mkublin - resent due to merging issues, no code

[Engine-patches] Change in ovirt-engine[master]: packaging: classify params and groups (plugins step 1/3)

2012-02-07 Thread bazulay
Barak Azulay has posted comments on this change. Change subject: packaging: classify params and groups (plugins step 1/3) .. Patch Set 4: (3 inline comments) please fix comments

[Engine-patches] [oVirt Jenkins] ovirt_engine_dao_unit_tests - Build # 108 - Still Failing!

2012-02-07 Thread Jenkins oVirt Server
ovirt_engine_dao_unit_tests - Build # 108 - Still Failing: Check console output at http://jenkins.ovirt.org/job/ovirt_engine_dao_unit_tests/108/console to view the results. ___ Engine-patches mailing list Engine-patches@ovirt.org

[Engine-patches] Change in ovirt-engine[master]: core : Fix default values of is_plugged property for devices

2012-02-07 Thread mkublin
Michael Kublin has submitted this change and it was merged. Change subject: core : Fix default values of is_plugged property for devices .. core : Fix default values of is_plugged property for devices Disk Interface should

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

2012-02-07 Thread ofrenkel
Omer Frenkel has posted comments on this change. Change subject: core: DO NOT SUBMIT DAL: removing of DiskImageTemplate .. Patch Set 3: Looks good to me, but someone else must approve -- To view, visit

[Engine-patches] Change in ovirt-engine[master]: engine-core: Move disk command

2012-02-07 Thread derez
Daniel Erez has posted comments on this change. Change subject: engine-core: Move disk command .. Patch Set 4: Verified; Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/1608 To

[Engine-patches] [oVirt Jenkins] ovirt_engine_dao_unit_tests - Build # 109 - Still Failing!

2012-02-07 Thread Jenkins oVirt Server
ovirt_engine_dao_unit_tests - Build # 109 - Still Failing: Check console output at http://jenkins.ovirt.org/job/ovirt_engine_dao_unit_tests/109/console to view the results. ___ Engine-patches mailing list Engine-patches@ovirt.org

[Engine-patches] Change in ovirt-engine[master]: core: Modify commands invocation with compansation context

2012-02-07 Thread masayag
Moti Asayag has uploaded a new change for review. Change subject: core: Modify commands invocation with compansation context .. core: Modify commands invocation with compansation context The patch modifies invocation of

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

2012-02-07 Thread yzaslavs
Yair Zaslavsky has posted comments on this change. Change subject: core: DO NOT SUBMIT common: Fixing entities to reflect removal of DiskImageTemplate .. Patch Set 3: (1 inline comment)

[Engine-patches] Change in ovirt-engine[master]: engine-core: Uniting 2 Commands (Review Only)

2012-02-07 Thread msalem
Muli Salem has uploaded a new change for review. Change subject: engine-core: Uniting 2 Commands (Review Only) .. engine-core: Uniting 2 Commands (Review Only) This patch unites the AttachUserToVmCommand with the

[Engine-patches] Change in ovirt-engine-sdk[master]: sdk: Add HTTP request and response hooks.

2012-02-07 Thread jhenner
Jaroslav Henner has posted comments on this change. Change subject: sdk: Add HTTP request and response hooks. .. Patch Set 3: (1 inline comment) File

[Engine-patches] Change in ovirt-engine[master]: engine-core: Uniting 2 Commands (Review Only)

2012-02-07 Thread mkublin
Michael Kublin has posted comments on this change. Change subject: engine-core: Uniting 2 Commands (Review Only) .. Patch Set 2: (7 inline comments) File

[Engine-patches] Change in ovirt-engine[master]: core: Minor clean up of VDSStatus

2012-02-07 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: Minor clean up of VDSStatus .. Patch Set 2: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/1617 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Minor clean up of VDSStatus

2012-02-07 Thread rgolan
Roy Golan has posted comments on this change. Change subject: core: Minor clean up of VDSStatus .. -- To view, visit http://gerrit.ovirt.org/1617 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType:

  1   2   >