Change in ovirt-engine[master]: utils: Encapsulte OVF build algo into OvfBuilder

rgolan at redhat.com rgolan at redhat.com
Mon Oct 31 10:16:51 UTC 2016


Roy Golan has submitted this change and it was merged.

Change subject: utils: Encapsulte OVF build algo into OvfBuilder
......................................................................


utils: Encapsulte OVF build algo into OvfBuilder

The steps to build the OVF where called outside by the OvfManager. Its
the responsiblity of the Object to build its internal structure and so
it is encapsulated into it, making the API safer, easier and cleaner.

Change-Id: I427aa4e9ada86c59e4d7cf999dbdb23aaaa44581
Bug-Url: https://bugzilla.redhat.com/1358383
Signed-off-by: Roy Golan <rgolan at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/IOvfBuilder.java
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfManager.java
2 files changed, 12 insertions(+), 17 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Roy Golan: Verified
  Liron Aravot: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I427aa4e9ada86c59e4d7cf999dbdb23aaaa44581
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list