Hi Aaron, On 07/20/2016 10:21 PM, Aaron Conole wrote: > This commit builds upon some of the recent ovs-ctl changes to build a > more integrated systemd setup. A new service (ovs-vswitchd) is > added to track the ovs-vswitchd, and ovsdb-server service is reserved > for the ovsdb-server daemon. The systemd scripts still use ovs-ctl to > actually initialize the daemons. >
> > diff --git a/rhel/usr_lib_systemd_system_ovs-vswitchd.service > b/rhel/usr_lib_systemd_system_ovs-vswitchd.service > new file mode 100644 > index 0000000..d3d020a > --- /dev/null > +++ b/rhel/usr_lib_systemd_system_ovs-vswitchd.service > @@ -0,0 +1,18 @@ > +[Unit] > +Description=Open vSwitch Forwarding Unit > +After=ovsdb-server.service > +Requires=ovsdb-server.service [...] > diff --git a/rhel/usr_lib_systemd_system_ovsdb-server.service > b/rhel/usr_lib_systemd_system_ovsdb-server.service > index e4c2a66..847948e 100644 > --- a/rhel/usr_lib_systemd_system_ovsdb-server.service > +++ b/rhel/usr_lib_systemd_system_ovsdb-server.service > @@ -1,15 +1,18 @@ > [Unit] > -Description=Open vSwitch Internal Unit > +Description=Open vSwitch Database Unit > After=syslog.target > +ReloadPropagatedFrom=openvswitch.service > +Before=ovs-vswitchd.service Is the "Before=" here really necessary? You already have "After=ovsdb-server.service" in the ovs-vswitchd.service. Overall the change looks good to me so fwiw Reviewed-by: Markos Chandras <mchand...@suse.de> -- markos SUSE LINUX GmbH | GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Maxfeldstr. 5, D-90409, Nürnberg _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
