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