[openstack-dev] [Neutron][networking-*] Attention for upcoming refactoring

Kevin Benton kevin at benton.pub
Wed Jan 4 08:18:42 UTC 2017


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.ubuntu at 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 <akamyshnikova at 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 <rbryant at redhat.com>
>> wrote:
>>
>>>
>>> On Wed, Dec 21, 2016 at 10:50 AM, Anna Taraday <
>>> akamyshnikova at 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-request at 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-request at 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-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20170104/12437151/attachment.html>


More information about the OpenStack-dev mailing list