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

[Kevin Benton]

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  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 
> 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 
>> 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
>>> 
>>> , networking-midonet
>>> 
>>> , 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

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

[Ian Wells]

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 
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  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
>> 
>> , networking-midonet
>> 
>> , 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:
>> master+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.openstack.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: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

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

[Anna Taraday]

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  wrote:

>
> On Wed, Dec 21, 2016 at 10:50 AM, Anna Taraday  > 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
> 
> , networking-midonet
> ,
>  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:master+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.openstack.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:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

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

[Russell Bryant]

On Wed, Dec 21, 2016 at 10:50 AM, Anna Taraday 
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
> 
> , networking-midonet
> 
> , 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:
> master+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.openstack.org?subject:unsubscribe
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev

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

[Anna Taraday]

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

, networking-midonet
,
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:master+topic:refactor_ml2db
-- 
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