Change in ovirt-engine[master]: core: AddLocalStorageDomainCommand DbFacade

amureini at redhat.com amureini at redhat.com
Mon Jun 13 10:01:05 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: core: AddLocalStorageDomainCommand DbFacade
......................................................................


core: AddLocalStorageDomainCommand DbFacade

Use the available methods to get the relevant DAOs instead of using
DbFacade.getInstance() directly.

This is done both to clean up the code and to make unit testing easier.

Change-Id: Ibdb076a421d14f0ce71bb1926bafc7fb013e24ee
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/AddLocalStorageDomainCommand.java
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Allon Mureinik: Verified; Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/59029
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibdb076a421d14f0ce71bb1926bafc7fb013e24ee
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list