From 961621fedccdcd45062c9ec04b60637fe21e76e0da4e6d7007815b0921f2578b Mon Sep 17 00:00:00 2001 From: Marcus Meissner Date: Tue, 22 Nov 2016 10:46:01 +0000 Subject: [PATCH 1/2] Accepting request 441194 from home:markoschandras:network - Do not restart the openvswitch service after a package update. Restarting the systemd service may break connectivity so let the user decide when it is the best time for such an action. OBS-URL: https://build.opensuse.org/request/show/441194 OBS-URL: https://build.opensuse.org/package/show/network/openvswitch?expand=0&rev=108 --- openvswitch-dpdk.changes | 7 +++++++ openvswitch-dpdk.spec | 8 ++++++++ openvswitch.changes | 7 +++++++ openvswitch.spec | 8 ++++++++ 4 files changed, 30 insertions(+) diff --git a/openvswitch-dpdk.changes b/openvswitch-dpdk.changes index 4611a16..dae9376 100644 --- a/openvswitch-dpdk.changes +++ b/openvswitch-dpdk.changes @@ -1,3 +1,10 @@ +------------------------------------------------------------------- +Mon Nov 21 11:53:00 UTC 2016 - mchandras@suse.de + +- Do not restart the openvswitch service after a package update. + Restarting the systemd service may break connectivity so let the + user decide when it is the best time for such an action. + ------------------------------------------------------------------- Thu Nov 3 10:48:32 UTC 2016 - mchandras@suse.de diff --git a/openvswitch-dpdk.spec b/openvswitch-dpdk.spec index de40c71..7936271 100644 --- a/openvswitch-dpdk.spec +++ b/openvswitch-dpdk.spec @@ -375,6 +375,14 @@ rm -rf %{buildroot}%{py_sitedir} %postun switch %service_del_postun ovsdb-server.service %service_del_postun ovs-vswitchd.service +# Do not restart the openvswitch service on package updates. +# Restarting the service may break the existing network state. +# For example, openflow rules are not automatically re-installed +# after an OvS update if no SDN controller is used. Moreover, restaring +# the OvS can break remote administration during the update so let the +# admin decide when it's the best time for an OvS restart. +# 5771f476573445710834234a6a9f7bd999a027e7 ("fedora: do not restart the service on a pkg upgrade") +export DISABLE_RESTART_ON_UPDATE=yes %service_del_postun openvswitch.service %pre ovn diff --git a/openvswitch.changes b/openvswitch.changes index 4611a16..dae9376 100644 --- a/openvswitch.changes +++ b/openvswitch.changes @@ -1,3 +1,10 @@ +------------------------------------------------------------------- +Mon Nov 21 11:53:00 UTC 2016 - mchandras@suse.de + +- Do not restart the openvswitch service after a package update. + Restarting the systemd service may break connectivity so let the + user decide when it is the best time for such an action. + ------------------------------------------------------------------- Thu Nov 3 10:48:32 UTC 2016 - mchandras@suse.de diff --git a/openvswitch.spec b/openvswitch.spec index 5b22018..89a1e27 100644 --- a/openvswitch.spec +++ b/openvswitch.spec @@ -373,6 +373,14 @@ rm -rf %{buildroot}%{py_sitedir} %postun switch %service_del_postun ovsdb-server.service %service_del_postun ovs-vswitchd.service +# Do not restart the openvswitch service on package updates. +# Restarting the service may break the existing network state. +# For example, openflow rules are not automatically re-installed +# after an OvS update if no SDN controller is used. Moreover, restaring +# the OvS can break remote administration during the update so let the +# admin decide when it's the best time for an OvS restart. +# 5771f476573445710834234a6a9f7bd999a027e7 ("fedora: do not restart the service on a pkg upgrade") +export DISABLE_RESTART_ON_UPDATE=yes %service_del_postun openvswitch.service %pre ovn From 22ee90756e8a389f14ba1d725c04041d8d04b307d174884f95d133099f26cc33 Mon Sep 17 00:00:00 2001 From: Markos Chandras Date: Mon, 28 Nov 2016 11:39:20 +0000 Subject: [PATCH 2/2] Accepting request 442335 from home:markoschandras:network - Relax the DPDK dependency a bit so we can support stable and possibly new minor releases as well. OBS-URL: https://build.opensuse.org/request/show/442335 OBS-URL: https://build.opensuse.org/package/show/network/openvswitch?expand=0&rev=109 --- openvswitch-dpdk.changes | 6 ++++++ openvswitch-dpdk.spec | 5 +++-- openvswitch.changes | 6 ++++++ openvswitch.spec | 5 +++-- 4 files changed, 18 insertions(+), 4 deletions(-) diff --git a/openvswitch-dpdk.changes b/openvswitch-dpdk.changes index dae9376..fb76aed 100644 --- a/openvswitch-dpdk.changes +++ b/openvswitch-dpdk.changes @@ -1,3 +1,9 @@ +------------------------------------------------------------------- +Fri Nov 25 16:36:40 UTC 2016 - mchandras@suse.de + +- Relax the DPDK dependency a bit so we can support stable and + possibly new minor releases as well. + ------------------------------------------------------------------- Mon Nov 21 11:53:00 UTC 2016 - mchandras@suse.de diff --git a/openvswitch-dpdk.spec b/openvswitch-dpdk.spec index 7936271..f75825b 100644 --- a/openvswitch-dpdk.spec +++ b/openvswitch-dpdk.spec @@ -70,8 +70,9 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-build %if %{with dpdk} # We need to be a bit strict with the dpdk version since # it's very possible for DPDK to change it's API between -# releases. This version currently requires 16.07. -BuildRequires: dpdk-devel = 16.07 +# releases. This version currently requires >=16.07 but not +# 17.XX +BuildRequires: dpdk-devel >= 16.07 BuildRequires: libnuma-devel BuildRequires: libpcap-devel # We can't have openvswitch and openvswitch-dpdk in parallel diff --git a/openvswitch.changes b/openvswitch.changes index dae9376..fb76aed 100644 --- a/openvswitch.changes +++ b/openvswitch.changes @@ -1,3 +1,9 @@ +------------------------------------------------------------------- +Fri Nov 25 16:36:40 UTC 2016 - mchandras@suse.de + +- Relax the DPDK dependency a bit so we can support stable and + possibly new minor releases as well. + ------------------------------------------------------------------- Mon Nov 21 11:53:00 UTC 2016 - mchandras@suse.de diff --git a/openvswitch.spec b/openvswitch.spec index 89a1e27..9ad0ac7 100644 --- a/openvswitch.spec +++ b/openvswitch.spec @@ -68,8 +68,9 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-build %if %{with dpdk} # We need to be a bit strict with the dpdk version since # it's very possible for DPDK to change it's API between -# releases. This version currently requires 16.07. -BuildRequires: dpdk-devel = 16.07 +# releases. This version currently requires >=16.07 but not +# 17.XX +BuildRequires: dpdk-devel >= 16.07 BuildRequires: libnuma-devel BuildRequires: libpcap-devel # We can't have openvswitch and openvswitch-dpdk in parallel