Change in ovirt-engine[master]: packaging: setup: copy virtio files if they exist

oschreib at redhat.com oschreib at redhat.com
Mon Aug 26 11:47:44 UTC 2013


Ofer Schreiber has submitted this change and it was merged.

Change subject: packaging: setup: copy virtio files if they exist
......................................................................


packaging: setup: copy virtio files if they exist

Current implementation didn't load the image_upload
plugin correctly. Additionally, the plugin itself
checked its own relevance before NFS export was created.

New implementation loads the plugin correctly and moves
NFS check to the validation stage, where NFS creation/export
configuration step already have passed.

Change-Id: Ie35c5a3ad38b2f7bdae88c1f8d0790d1a8c988c2
Signed-off-by: Alex Lourie <alourie at redhat.com>
---
M packaging/setup/plugins/ovirt-engine-setup/distro-rpm/__init__.py
M packaging/setup/plugins/ovirt-engine-setup/distro-rpm/image_upload.py
2 files changed, 4 insertions(+), 2 deletions(-)

Approvals:
  Ofer Schreiber: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/18504
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie35c5a3ad38b2f7bdae88c1f8d0790d1a8c988c2
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>



More information about the Engine-commits mailing list