Change in ovirt-engine[master]: core: Change modifier of constructors to public

mlipchuk at redhat.com mlipchuk at redhat.com
Tue Jan 5 16:51:23 UTC 2016


Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Change modifier of constructors to public
......................................................................


core: Change modifier of constructors to public

Since there was a refactor of the classes packages,
some commands which were used to be called as part of
the infrastructure flow fails to be called now,
since their constructors are protected and they were
moved to a different package then the CommandBase.
One example is removeImageCommand which use as part
of revert task of addVMTemaplate.

Change-Id: If1930ba0fccfa7f65234bf3d1e4bc953c73e5041
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/RemoveImageCommand.java
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Maor Lipchuk: Verified
  Liron Aravot: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If1930ba0fccfa7f65234bf3d1e4bc953c73e5041
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby 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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list