Yaniv Bronhaim has posted comments on this change.

Change subject: restapi: Foreman host provider
......................................................................


Patch Set 4: Code-Review+1

Thanks!! I will test it as well!

Although I still see that GET discovery_hosts list API is missing here, which 
should direct to the list comes from api/discovered_hosts , and we should add 
also the POST for provision the host. (or should it be part of the addHost api 
? )

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2144125f00822263dc68da881eb3180c4cd6b237
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to