Re: [openstack-dev] [neutron][oslo.db] db retry madness

2017-09-10 Thread Michel Peterson
Thanks for this most interesting and informative exchange. On Fri, Sep 8, 2017 at 5:20 PM, Michael Bayer wrote: > On Fri, Sep 8, 2017 at 8:53 AM, Kevin Benton wrote: > > Since the goal of that patch is to deal with deadlocks, the retry can't > > happen down at that level, it will need to be some

Re: [openstack-dev] [neutron][oslo.db] db retry madness

2017-09-08 Thread Michael Bayer
On Fri, Sep 8, 2017 at 8:53 AM, Kevin Benton wrote: > Thanks for asking this. It's good to get this documented on the mailing list > since most of it is spread across bug reports and commits over the last > couple of years. > >>1. since the retry_if_session_inactive decorator assumes functions tha

Re: [openstack-dev] [neutron][oslo.db] db retry madness

2017-09-08 Thread Kevin Benton
Thanks for asking this. It's good to get this documented on the mailing list since most of it is spread across bug reports and commits over the last couple of years. >1. since the retry_if_session_inactive decorator assumes functions that receive a "context", what does it mean that networking_odl

[openstack-dev] [neutron][oslo.db] db retry madness

2017-09-07 Thread Michael Bayer
I'm trying to get a handle on neutron's retry decorators.I'm assisting a developer in the networking_odl project who wishes to make use of neutron's retry_if_session_inactive decorator. The developer has run into several problems, including that networking_odl methods here seem to accept a "se