If you wanted one compatible with both you could have a line like this. session = ctx.session if isinstance(ctx, neutron.context.Context) else ctx
On Tue, Jan 3, 2017 at 2:52 PM, Ian Wells <ijw.ubu...@cack.org.uk> wrote: > I see this changes a function's argument types without changing the > function's name - for instance, in the proposed networking-cisco change, > https://review.openstack.org/#/c/409045/ . This makes it hard to detect > that there's been a change and react accordingly. What's the recommended > way to write a mechanism driver that is compatible with both pre- and > post-change Neutron versions? > -- > Ian. > > On 27 December 2016 at 02:29, Anna Taraday <akamyshnik...@mirantis.com> > wrote: > >> Hello everyone! >> >> Please, note that all changes to Neutron merged. >> >> Changes that needs to be merged for external repos: >> segments db refactor - https://review.openstack.org/# >> /q/status:open+branch:master+topic:segmentsdb >> ml2 db refactor - https://review.openstack.org >> /#/q/status:open+branch:master+topic:refactor_ml2db >> >> Happy holidays for everyone! >> >> >> On Thu, Dec 22, 2016 at 7:36 AM Russell Bryant <rbry...@redhat.com> >> wrote: >> >>> >>> On Wed, Dec 21, 2016 at 10:50 AM, Anna Taraday < >>> akamyshnik...@mirantis.com> wrote: >>> >>> Hello everyone! >>> >>> I've got two changes with refactor of TypeDriver [1] and segments db [2] >>> which is needed for implementation new engine facade [3]. >>> >>> Reviewers of networking-cisco, networking-arista, networking-nec >>> <https://review.openstack.org/#/projects/openstack/networking-nec,dashboards/default> >>> , networking-midonet >>> <https://review.openstack.org/#/projects/openstack/networking-midonet,dashboards/default> >>> , networking-edge-vpn, networking-bagpipe, >>> tricircle, group-based-policy - pay attention for [4]. >>> >>> Also recently merged refactor of ml2/db.py [5]. Fixes >>> for networking-cisco, networking-cisco, networking-cisco - are on >>> review [6] >>> >>> [1] - https://review.openstack.org/#/c/398873/ >>> [2] - https://review.openstack.org/#/c/406275/ >>> [3] - https://blueprints.launchpad.net/neutron/+spec/enginefacade-switch >>> [4] - https://review.openstack.org/#/q/topic:segmentsdb >>> [5] - https://review.openstack.org/#/c/404714/ >>> [6] - https://review.openstack.org/#/q/status:open++branch:maste >>> r+topic:refactor_ml2db >>> >>> >>> ​Thanks a lot for looking out for the various networking-* projects when >>> working on changes like this. It's really great to see. >>> >>> -- >>> Russell Bryant >>> ____________________________________________________________ >>> ______________ >>> OpenStack Development Mailing List (not for usage questions) >>> Unsubscribe: openstack-dev-requ...@lists.op >>> enstack.org?subject:unsubscribe >>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >>> >> -- >> Regards, >> Ann Taraday >> >> ____________________________________________________________ >> ______________ >> OpenStack Development Mailing List (not for usage questions) >> Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscrib >> e >> 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 > >
__________________________________________________________________________ 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
