summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Maw <richard.maw@codethink.co.uk>2015-03-09 16:18:32 +0000
committerRichard Maw <richard.maw@codethink.co.uk>2015-03-20 16:06:23 +0000
commitc749c3bcc6b8c232dadfdd441360765990f1fd1a (patch)
treee692437ed492f1f3fbb6fa3657b268aade7f52bc
parent58e9d80287efb0ac17766fddded1977d5b77185a (diff)
downloaddefinitions-c749c3bcc6b8c232dadfdd441360765990f1fd1a.tar.gz
Only start neutron-openvswitch-plugin after cleanup has finished
ovs-cleanup is responsible for reconciling the state in openvswitch's database and neutron's configuration. This can fail if other services are also changing ovs configuration though, and the missing dependency resulted in neutron removing the interface while ovs-cleanup was about to do so.
-rw-r--r--openstack/etc/systemd/system/openstack-neutron-ovs-cleanup.service6
1 files changed, 4 insertions, 2 deletions
diff --git a/openstack/etc/systemd/system/openstack-neutron-ovs-cleanup.service b/openstack/etc/systemd/system/openstack-neutron-ovs-cleanup.service
index 5eaf8a30..083cf701 100644
--- a/openstack/etc/systemd/system/openstack-neutron-ovs-cleanup.service
+++ b/openstack/etc/systemd/system/openstack-neutron-ovs-cleanup.service
@@ -1,10 +1,12 @@
[Unit]
Description=Neutron OVS cleanup
-After=network-online.target openstack-neutron-setup.service
+After=network-online.target openstack-neutron-setup.service openvswitch.service
+Before=openstack-neutron-plugin-openvswitch-agent.service
ConditionFileIsExecutable=/usr/bin/neutron-ovs-cleanup
[Service]
-Type=simple
+Type=oneshot
+RemainAfterExit=yes
User=neutron
ExecStart=/usr/bin/neutron-ovs-cleanup \
--log-file /var/log/neutron/ovs-cleanup.log \