May 2012 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Gluster Volumes Collection implementation
by mpastern＠redhat.com 22 May '12

22 May '12

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: Gluster Volumes Collection implementation ...................................................................... restapi: Gluster Volumes Collection implementation Implementation of BackendGlusterVolumesResource - list - add - remove and the JUnit tests for the same Change-Id: I578f5b7ea0328057d2e29d04a31393856b021f08 Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com> --- M backend/manager/modules/restapi/interface/common/jaxrs/src/main/java/org/ovirt/engine/api/common/util/QueryHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClusterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumesResource.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumesResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/GlusterVolumeMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/GlusterVolumeMapperTest.java 6 files changed, 305 insertions(+), 12 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Shireesh Anjal: Verified -- To view, visit http://gerrit.ovirt.org/3932 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I578f5b7ea0328057d2e29d04a31393856b021f08 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Eoghan Glynn <eoghan.glynn(a)gmail.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: engine: Add ID fields to gluster brick and option
by ofrenkel＠redhat.com 22 May '12

22 May '12

Omer Frenkel has submitted this change and it was merged. Change subject: engine: Add ID fields to gluster brick and option ...................................................................... engine: Add ID fields to gluster brick and option - Added id column to gluster_bricks and gluster_options - New Dao classes for these tables - JUnit tests for the new Dao classes - Moved methods related to DB operations on bricks and options from GlusterVolumeDao to the new Dao classes - Changed existing method signatures to perform DB operations based on id instead of other unique keys - Changed existing BLL commands to use the new signatures Change-Id: I2698e1686b18a03111f94145d0eb961be85d0131 Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com> --- M backend/manager/dbscripts/gluster_volumes_sp.sql A backend/manager/dbscripts/upgrade/03_01_1090_gluster_brick_option_id.sql M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterVolumeCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterVolumeRemoveBricksCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ReplaceGlusterVolumeBrickCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/ResetGlusterVolumeOptionsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/SetGlusterVolumeOptionCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterBrickEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeEntity.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeOptionEntity.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/validation/group/gluster/AddBrick.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDaoDbFacadeImpl.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterOptionDao.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterOptionDaoDbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoDbFacadeImpl.java M backend/manager/modules/dal/src/main/jdbc-resources/engine-daos.properties M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterBrickDaoTest.java A backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterOptionDaoTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/gluster/GlusterVolumeDaoTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml 25 files changed, 583 insertions(+), 394 deletions(-) Approvals: Shireesh Anjal: Verified Omer Frenkel: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4431 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2698e1686b18a03111f94145d0eb961be85d0131 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: RSDL metadata for gluster related REST api
by mpastern＠redhat.com 22 May '12

22 May '12

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: RSDL metadata for gluster related REST api ...................................................................... restapi: RSDL metadata for gluster related REST api Added RSDL parameter metadata for gluster related REST api. Change-Id: I2883e452b8f6bc4b51bd2199fbd712f56dd96ab5 Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata_v-3.1.yaml 1 file changed, 100 insertions(+), 0 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Shireesh Anjal: Verified -- To view, visit http://gerrit.ovirt.org/3918 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2883e452b8f6bc4b51bd2199fbd712f56dd96ab5 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Eoghan Glynn <eoghan.glynn(a)gmail.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: restapi: Gluster Resources Implementation classes
by mpastern＠redhat.com 22 May '12

22 May '12

Michael Pasternak has submitted this change and it was merged. Change subject: restapi: Gluster Resources Implementation classes ...................................................................... restapi: Gluster Resources Implementation classes All gluster related resource implementation classes. These classes do not contain any functionality yet. All resource methods throw NotImplementedException for now, except the ones that fetch child resources. Subsequent patches will add proper implementation to the resource methods one by one. This patch also modifies the ClusterResource and introduces GlusterVolumesResource as it's child resource. Change-Id: I18afc04f2501e4bbca272a9b5e18cf33d5ac9187 Signed-off-by: Shireesh Anjal <sanjal(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/ClusterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClusterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClustersResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBrickResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterBricksResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumeResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/gluster/BackendGlusterVolumesResource.java 7 files changed, 227 insertions(+), 3 deletions(-) Approvals: Michael Pasternak: Verified; Looks good to me, approved Shireesh Anjal: Verified -- To view, visit http://gerrit.ovirt.org/3905 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18afc04f2501e4bbca272a9b5e18cf33d5ac9187 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Eoghan Glynn <eoghan.glynn(a)gmail.com> Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Support for mode specific Tabs and Sub Tabs
by gchaplik＠redhat.com 22 May '12

22 May '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Support for mode specific Tabs and Sub Tabs ...................................................................... webadmin: Support for mode specific Tabs and Sub Tabs This patch is part of Mode Specific UI implementation in Ovirt. Currently supported modes are 1. VirtOnly(1) -- binary equivalent 0000 0001 2. GlusterOnly(2) -- binary equivalent 0000 0010 3. VirtGluster(3) -- binary equivalent 0000 0011 4. AllModes(255) -- binary equivalent 1111 1111 The value for the UiMode is read at the time of loading and stored in ApplicationModeHelper. UiMode class is moved from webadmin to uicommonweb A new field(availableInModes) is introduced in Model class to specify in what modes the model should be available. The value for this field is set to AllModes(255) by default. So that any model become available in all modes by default. Appropriate values can be set in the respective subclasses. getIsAvailable() method of Model class is intercepted to check for the mode as well. GlusterModeEnum and its references are removed. Change-Id: I55da9510a1480634ce30cf77b9bd9ecb263186bc Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Configurator.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/ApplicationModeHelper.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/Model.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/disks/DiskListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/ConfigureLocalStorageModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/pools/PoolListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/templates/TemplateListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/users/UserQuotaListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/uimode/UiMode.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/system/ApplicationInit.java D frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/uimode/UiMode.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/uimode/UiModeData.java 20 files changed, 126 insertions(+), 69 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4450 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55da9510a1480634ce30cf77b9bd9ecb263186bc Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Gluster Brick sub tab - removing columns
by gchaplik＠redhat.com 22 May '12

22 May '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Gluster Brick sub tab - removing columns ...................................................................... webadmin: Gluster Brick sub tab - removing columns As the moutpoint/storage device listing not supported currently, the columns which shows free space, total space in brick table is removed. Change-Id: Ib6abcf21fedc8cfcc22b1bc4b552864a84450ebc Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/gluster/SubTabVolumeBrickView.java 1 file changed, 0 insertions(+), 16 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4534 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6abcf21fedc8cfcc22b1bc4b552864a84450ebc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Gluster Volume Options - populating from help xml
by gchaplik＠redhat.com 22 May '12

22 May '12

Gilad Chaplik has submitted this change and it was merged. Change subject: webadmin: Gluster Volume Options - populating from help xml ...................................................................... webadmin: Gluster Volume Options - populating from help xml In the Volume Option dialog, the option list is populated from the help xml provided by the gluster. The description of the particular option will be shown on selecting the option in the list. Change-Id: Idaacbd89c1c3a72ff6e85d7f18ae0da510174581 Signed-off-by: Kanagaraj M <kmayilsa(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeParameterListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeParameterModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumeParameterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumeParameterPopupView.ui.xml 6 files changed, 113 insertions(+), 35 deletions(-) Approvals: Gilad Chaplik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4486 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idaacbd89c1c3a72ff6e85d7f18ae0da510174581 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com> Gerrit-Reviewer: Shireesh Anjal <sanjal(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: UnPowerMock AttachNetworkToVdsGroupCommandTest
by mkolesni＠redhat.com 22 May '12

22 May '12

Mike Kolesnik has submitted this change and it was merged. Change subject: core: UnPowerMock AttachNetworkToVdsGroupCommandTest ...................................................................... core: UnPowerMock AttachNetworkToVdsGroupCommandTest Removed PowerMock dependency in AttachNetworkToVdsGroupCommandTest by annonymusly extending it and mocking the relevant getXXXDAO() methods. In order to acheive this, getNetworkDAO() was added to AttachNetworkToVdsGroupCommand and a minor cleanup was required in both it and VdsGroupCommandBase. Change-Id: I11f007ddfbb81dc85198dc2953b52e841fe081e4 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AttachNetworkToVdsGroupCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsGroupCommandBase.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AttachNetworkToVdsGroupCommandTest.java 3 files changed, 55 insertions(+), 46 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Allon Mureinik: Verified -- To view, visit http://gerrit.ovirt.org/4634 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I11f007ddfbb81dc85198dc2953b52e841fe081e4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: UnPowerMock AddQuotaCommandTest
by mkolesni＠redhat.com 22 May '12

22 May '12

Mike Kolesnik has submitted this change and it was merged. Change subject: core: UnPowerMock AddQuotaCommandTest ...................................................................... core: UnPowerMock AddQuotaCommandTest Removed the PowerMock dependecy from the test. In order to acheive that a getQuotaHelper() method was added to the command, in a similar fashion to the getXXDAO() methods. Change-Id: Ib6ffe7a98bc9cafb286b3b9a54249144ad914915 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddQuotaCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AddQuotaCommandTest.java 2 files changed, 16 insertions(+), 28 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Allon Mureinik: Verified -- To view, visit http://gerrit.ovirt.org/4629 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib6ffe7a98bc9cafb286b3b9a54249144ad914915 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: UnPowerMock ActivateStorageDomainCommandTest
by mkolesni＠redhat.com 22 May '12

22 May '12

Mike Kolesnik has submitted this change and it was merged. Change subject: core: UnPowerMock ActivateStorageDomainCommandTest ...................................................................... core: UnPowerMock ActivateStorageDomainCommandTest PowerMock was used to mock DbFacade - replaced with regular mockito spying of the commands getXXDAO() methods. Change-Id: Ifbf347b6f56e77e49049a9c2e1c5b9467b086bd6 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommandTest.java 1 file changed, 14 insertions(+), 26 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Allon Mureinik: Verified -- To view, visit http://gerrit.ovirt.org/4628 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifbf347b6f56e77e49049a9c2e1c5b9467b086bd6 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0