Hi Seyeong, I think your patch makes sense. I wonder if we also need to add an After statement to ensure openvswitch-switch starts before neutron-ovs-cleanup:
Note that the LSB init info in the beginning of /etc/init.d/neutron-ovs-cleanup does have a "Required-Start: ... openvswitch-switch", but that is apparently not enough, or perhaps needs to be replaced by your patch.
I chatted with dosaboy a bit about this and it seems like we should also review nova-compute, neutron-l3-agent, and neutron-dhcp-agent as part of this fix. They need to start after neutron-ovs-cleanup is finished and they use "Should-Start: ... neutron-ovs-cleanup" in their LSB init info.
Hi Seyeong, I think your patch makes sense. I wonder if we also need to add an After statement to ensure openvswitch-switch starts before neutron- ovs-cleanup:
+Requires= openvswitch- switch. service openvswitch- switch. service
+After=
Note that the LSB init info in the beginning of /etc/init. d/neutron- ovs-cleanup does have a "Required-Start: ... openvswitch- switch" , but that is apparently not enough, or perhaps needs to be replaced by your patch.
I chatted with dosaboy a bit about this and it seems like we should also review nova-compute, neutron-l3-agent, and neutron-dhcp-agent as part of this fix. They need to start after neutron-ovs-cleanup is finished and they use "Should-Start: ... neutron- ovs-cleanup" in their LSB init info.