mbox series

[ovs-dev,0/2] ovs-ctl: Permit to specify additional options

Message ID cover.1549911128.git.tredaelli@redhat.com
Headers show
Series ovs-ctl: Permit to specify additional options | expand

Message

Timothy Redaelli Feb. 11, 2019, 6:55 p.m. UTC
Currently using ovs-ctl is not possible to specify additional options
for ovs-vswitchd and ovsdb-server (for example to specify a different
loglevel during daemon startup).

This series adds --ovs-vswitchd-options and --ovsdb-server-options
options to ovs-ctl in order to specify the additional options.

Due to word splitting it may not be possible to specify an option that
includes whitespaces.

The series also adds an example to
rhel/usr_share_openvswitch_scripts_systemd_sysconfig.template.

Timothy Redaelli (2):
  ovs-ctl: Permit to specify additional options
  rhel: Add an example to specify custom options

 rhel/usr_share_openvswitch_scripts_systemd_sysconfig.template | 4 ++++
 utilities/ovs-ctl.in                                          | 4 ++++
 2 files changed, 8 insertions(+)

Comments

Ben Pfaff Feb. 12, 2019, 1:54 a.m. UTC | #1
On Mon, Feb 11, 2019 at 07:55:51PM +0100, Timothy Redaelli wrote:
> Currently using ovs-ctl is not possible to specify additional options
> for ovs-vswitchd and ovsdb-server (for example to specify a different
> loglevel during daemon startup).
> 
> This series adds --ovs-vswitchd-options and --ovsdb-server-options
> options to ovs-ctl in order to specify the additional options.
> 
> Due to word splitting it may not be possible to specify an option that
> includes whitespaces.
> 
> The series also adds an example to
> rhel/usr_share_openvswitch_scripts_systemd_sysconfig.template.
> 
> Timothy Redaelli (2):
>   ovs-ctl: Permit to specify additional options
>   rhel: Add an example to specify custom options
> 
>  rhel/usr_share_openvswitch_scripts_systemd_sysconfig.template | 4 ++++
>  utilities/ovs-ctl.in                                          | 4 ++++
>  2 files changed, 8 insertions(+)

Thank you for the patches.

Please document the new options in utilities/ovs-ctl.8.

I'd be open to folding these together, unless you prefer to keep them
apart.