Skip to content

Commit

Permalink
Expose missing --peer-ca-cert and SSL options in usage and manpages.
Browse files Browse the repository at this point in the history
Signed-off-by: Dan Williams <[email protected]>
Signed-off-by: Ben Pfaff <[email protected]>
  • Loading branch information
dcbw authored and blp committed May 9, 2018
1 parent d45d996 commit c33fa58
Show file tree
Hide file tree
Showing 9 changed files with 39 additions and 3 deletions.
1 change: 1 addition & 0 deletions lib/automake.mk
Original file line number Diff line number Diff line change
Expand Up @@ -476,6 +476,7 @@ EXTRA_DIST += \
lib/db-ctl-base.xml \
lib/ssl.xml \
lib/ssl-bootstrap.xml \
lib/ssl-peer-ca-cert.xml \
lib/table.xml \
lib/vlog.xml \
lib/unixctl.xml
Expand Down
22 changes: 22 additions & 0 deletions lib/ssl-peer-ca-cert.xml
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
<?xml version="1.0" encoding="utf-8"?>
<dl>
<dt><code>--peer-ca-cert=</code><var>peer-cacert.pem</var></dt>
<dd>
<p>
Specifies a PEM file that contains one or more additional certificates
to send to SSL peers. <var>peer-cacert.pem</var> should be the CA
certificate used to sign the program's own certificate, that is, the
certificate specified on <code>-c</code> or <code>--certificate</code>.
If the program's certificate is self-signed, then
<code>--certificate</code> and <code>--peer-ca-cert</code> should specify
the same file.
</p>
<p>
This option is not useful in normal operation, because the SSL peer
must already have the CA certificate for the peer to have any
confidence in the program's identity. However, this offers a way for
a new installation to bootstrap the CA certificate on its first SSL
connection.
</p>
</dd>
</dl>
1 change: 1 addition & 0 deletions manpages.mk
Original file line number Diff line number Diff line change
Expand Up @@ -256,6 +256,7 @@ vswitchd/ovs-vswitchd.8: \
lib/netdev-dpdk-unixctl.man \
lib/service.man \
lib/ssl-bootstrap.man \
lib/ssl-peer-ca-cert.man \
lib/ssl.man \
lib/unixctl.man \
lib/vlog-unixctl.man \
Expand Down
9 changes: 9 additions & 0 deletions ovn/controller-vtep/ovn-controller-vtep.8.xml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,15 @@
database (see <code>vtep</code>(5)) over the OVSDB protocol.
</p>

<h2>PKI Options</h2>
<p>
PKI configuration is required in order to use SSL for the connections to
the VTEP and Southbound databases.
</p>
<xi:include href="lib/ssl.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/>
<xi:include href="lib/ssl-bootstrap.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/>
<xi:include href="lib/ssl-peer-ca-cert.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/>

<h1>Configuration</h1>
<p>
<code>ovn-controller-vtep</code> retrieves its configuration
Expand Down
2 changes: 1 addition & 1 deletion ovn/controller-vtep/ovn-controller-vtep.c
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ Options:\n\
-o, --options list available options\n\
-V, --version display version information\n\
", program_name, program_name, default_db(), default_db());
stream_usage("database", true, false, false);
stream_usage("database", true, false, true);
daemon_usage();
vlog_usage();
exit(EXIT_SUCCESS);
Expand Down
2 changes: 2 additions & 0 deletions ovn/controller/ovn-controller.8.xml
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,8 @@
the Northbound and Southbound databases.
</p>
<xi:include href="lib/ssl.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/>
<xi:include href="lib/ssl-bootstrap.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/>
<xi:include href="lib/ssl-peer-ca-cert.xml" xmlns:xi="http://www.w3.org/2003/XInclude"/>

<h2>Other Options</h2>

Expand Down
2 changes: 1 addition & 1 deletion ovn/controller/ovn-controller.c
Original file line number Diff line number Diff line change
Expand Up @@ -971,7 +971,7 @@ usage(void)
"usage %s [OPTIONS] [OVS-DATABASE]\n"
"where OVS-DATABASE is a socket on which the OVS OVSDB server is listening.\n",
program_name, program_name);
stream_usage("OVS-DATABASE", true, false, false);
stream_usage("OVS-DATABASE", true, false, true);
daemon_usage();
vlog_usage();
printf("\nOther options:\n"
Expand Down
2 changes: 1 addition & 1 deletion utilities/ovs-vsctl.c
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,7 @@ Options:\n\
vlog_usage();
printf("\
--no-syslog equivalent to --verbose=vsctl:syslog:warn\n");
stream_usage("database", true, true, false);
stream_usage("database", true, true, true);
printf("\n\
Other options:\n\
-h, --help display this help message\n\
Expand Down
1 change: 1 addition & 0 deletions vswitchd/ovs-vswitchd.8.in
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ configuration.
.SS "Public Key Infrastructure Options"
.so lib/ssl.man
.so lib/ssl-bootstrap.man
.so lib/ssl-peer-ca-cert.man
.SS "Logging Options"
.so lib/vlog.man
.SS "Other Options"
Expand Down

0 comments on commit c33fa58

Please sign in to comment.