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(a)redhat.com&gt; --- 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(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com&gt; Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;