[ovs-dev] [Backport Request] For branch-2.5, backport rhel-systemd integration

Flavio Leitner fbl at sysclose.org
Fri Oct 7 17:48:54 UTC 2016


On Thu, Sep 08, 2016 at 07:58:21PM +0100, Markos Chandras wrote:
> On 09/08/2016 05:50 PM, Aaron Conole wrote:
> > 
> >> It sounds like new feature territory, but you do make a case for it being considered a set of fixes ...
> > 
> > I agree - it straddles a line.  I was hesitant to even ask, but in RHEL
> > we probably need to backport these anyway, so I made an assumption
> > (maybe poor) that other systemd distros might use the rhel scripts and
> > run into this class of issues also.  Good net-izen, and all that :)
> > 
> > -Aaron
> 
> Hi Aaron,
> 
> Thanks for the backport request. I am also interested in these fixes and
> it's something I could use in the SUSE package as well. For my point of
> view I see no blockers for those updating their 2.5.0 installations with
> these fixes in but I will do some testing on my end as well.


Most of the changes are transparent and should not cause any issues.

I say most of the changes because the problem is if someone is relying
on the openvswitch-nonetwork service state.  In that case, the service
doesn't exist anymore after the patchset, so it might break something.

I am not sure if we care about that because that service should be
considered internal to OVS (used only by ifcfg scripts).  The front end
is the openvswitch.service which remains available.

Regarding to be a new feature or bugfix, I think this is a bugfix for
two things at least.

1) The real service state is represented by openvswitch service.
Before the patchset there were situations where the OVS threads were
dead but the systemd service was up & running.  The patchset intends
to fix this issue.

2) The shutdown ordering. Before the patchset we could have OVS shutting
down before other networking services that depends on network.  When OVS
terminates, it breaks networking connectivity causing issues. The patchset
intends to fix the issue as well, though it needs to include the follow up
fix  http://openvswitch.org/pipermail/dev/2016-October/080426.html

In another words,
Acked-by: Flavio Leitner <fbl at sysclose.org>

-- 
fbl




More information about the dev mailing list