Juan Hernandez has posted comments on this change.

Change subject: tools: (WIP, review don't submit) Adding Reload Configs 
Implementation
......................................................................


Patch Set 4: Verified

I built a new set of RPMs after applying this patch on top of the master 
branch. The result builds and installs correctly. The --reload option of 
engine-config works with SSL after changing the PROTOCOL parameter in 
engine-config.conf to https.

The notification service also can talk SSL with the engine after configuring it 
properly (there is a classpath problem, but not related to this change).

I would like to avoid additional parameters in engine-config.conf, and use 
/etc/sysconfig/ovirt-engine instead, but that is part of a different change 
(see 5362) that can bee rebased/adjusted on top of this one.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I4b9e790d7a8386359edfd62172df1078da033e91
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Muli Salem <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Muli Salem <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to