Folks, note that this change basically changes ml2 driver API that was supposed to be stable (now drivers receive context objects instead of sqlalchemy sessions). I would like folks to chime in the review. I would also suggest to announce the change during the next team meetings.
> On 22 Nov 2016, at 18:09, Anna Taraday <akamyshnik...@mirantis.com> wrote: > > Hello everyone! > I'm working on implementation new engine facade in Neutron. [1] > > > We have a number of functions that expect to get session as one the > arguments. Passing session is not correct and this prevents of usage new > enginefacade which expect context to be passed and session to be injected in > current context. And also when we use new enginefacade the concept of a > "subtransaction" is completely different. So, I made refactor patch for > Neutron [2]. > As these changes affect other projects that use these functions, I propose > patches to fix them [3]. > > Kindly, please be aware of these changes. > > > [1] - > https://blueprints.launchpad.net/neutron/+spec/enginefacade-switch > [2] - https://review.openstack.org/#/c/398873/ > [4] - > https://review.openstack.org/#/q/message:%22Change+passing+session+to+context+in+segments+db+functions%22 > -- > Regards, > Ann Taraday > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
