Allon Mureinik has posted comments on this change.

Change subject: [wip] providers: add the openstack image support
......................................................................


Patch Set 4:

@Alon:

> well, it is not good that we have dependency in maven that have no own 
> package.
> but this is not new, someone already merged a patch that uses openstack jars 
> without attending to this.
Very bad.
Are you on it, or do you want me to take it?

> apart of new dependency, is there file structure change / requirement at 
> engine machine?
Some new classes, obviously, but nothing far-fetching

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I424998de6f6514d65938484fffa5405e35857e43
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Alissa Bonas <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Liron Ar <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to