On Fri, Oct 7, 2016 at 1:48 PM, Flavio Leitner <f...@sysclose.org> wrote:

> 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 <f...@sysclose.org>
>

I'm fine with these backports.  I'll wait until we have the follow-up patch
applied to master, which is pending your (Flavio's) Signed-off-by.

-- 
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to