Re: [openstack-dev] [heat][neutron] Extraroute support

2018-06-05 Thread Lajos Katona 

Thanks for the answer.

On 2018-06-01 18:04, Kevin Benton wrote:
The neutron API now supports compare and swap updates with an If-Match 
header so the race condition can be avoided.

https://bugs.launchpad.net/neutron/+bug/1703234



On Fri, Jun 1, 2018, 04:57 Rabi Mishra > wrote:



On Fri, Jun 1, 2018 at 3:57 PM, Lajos Katona
mailto:lajos.kat...@ericsson.com>> wrote:

Hi,

Could somebody help me out with Neutron's Extraroute support
in Hot templates.
The support status of the Extraroute is support.UNSUPPORTED in
heat, and only create and delete are the supported operations.
see:

https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/neutron/extraroute.py#LC35

As I see the unsupported tag was added when the feature was
moved from the contrib folder to in-tree
(https://review.openstack.org/186608)
Perhaps you can help me out why only create and delete are
supported and update not.


I think most of the resources when moved from contrib to in-tree
are marked as unsupported. Adding routes to an existing router by
multiple stacks can be racy and is probably the reason use of this
resource is not encouraged and hence it's not supported. You can
see the discussion in the original patch that proposed this
resource https://review.openstack.org/#/c/41044/

Not sure if things have changed on neutron side for us to revisit
the concerns.

Also it does not have any update_allowed properties, hence no
handle_update(). It would be replaced if you change any property.

Hope it helps.

Thanks in advance for  the help.

Regards
Lajos



__
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,

Rabi Mishra

__
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


__
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] [heat][neutron] Extraroute support

2018-06-01 Thread Kevin Benton 
The neutron API now supports compare and swap updates with an If-Match
header so the race condition can be avoided.
https://bugs.launchpad.net/neutron/+bug/1703234



On Fri, Jun 1, 2018, 04:57 Rabi Mishra  wrote:

>
> On Fri, Jun 1, 2018 at 3:57 PM, Lajos Katona 
> wrote:
>
>> Hi,
>>
>> Could somebody help me out with Neutron's Extraroute support in Hot
>> templates.
>> The support status of the Extraroute is support.UNSUPPORTED in heat, and
>> only create and delete are the supported operations.
>> see:
>> https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/neutron/extraroute.py#LC35
>>
>>
> As I see the unsupported tag was added when the feature was moved from the
>> contrib folder to in-tree (https://review.openstack.org/186608)
>> Perhaps you can help me out why only create and delete are supported and
>> update not.
>>
>>
> I think most of the resources when moved from contrib to in-tree are
> marked as unsupported. Adding routes to an existing router by multiple
> stacks can be racy and is probably the reason use of this resource is not
> encouraged and hence it's not supported. You can see the discussion in the
> original patch that proposed this resource
> https://review.openstack.org/#/c/41044/
>
> Not sure if things have changed on neutron side for us to revisit the
> concerns.
>
> Also it does not have any update_allowed properties, hence no
> handle_update(). It would be replaced if you change any property.
>
> Hope it helps.
>
>
>
>> Thanks in advance for  the help.
>>
>> Regards
>> Lajos
>>
>>
>> __
>> 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,
> Rabi Mishra
>
> __
> 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] [heat][neutron] Extraroute support

2018-06-01 Thread Rabi Mishra 
On Fri, Jun 1, 2018 at 3:57 PM, Lajos Katona 
wrote:

> Hi,
>
> Could somebody help me out with Neutron's Extraroute support in Hot
> templates.
> The support status of the Extraroute is support.UNSUPPORTED in heat, and
> only create and delete are the supported operations.
> see: https://github.com/openstack/heat/blob/master/heat/engine/re
> sources/openstack/neutron/extraroute.py#LC35
>
>
As I see the unsupported tag was added when the feature was moved from the
> contrib folder to in-tree (https://review.openstack.org/186608)
> Perhaps you can help me out why only create and delete are supported and
> update not.
>
>
I think most of the resources when moved from contrib to in-tree are marked
as unsupported. Adding routes to an existing router by multiple stacks can
be racy and is probably the reason use of this resource is not encouraged
and hence it's not supported. You can see the discussion in the original
patch that proposed this resource https://review.openstack.org/#/c/41044/

Not sure if things have changed on neutron side for us to revisit the
concerns.

Also it does not have any update_allowed properties, hence no
handle_update(). It would be replaced if you change any property.

Hope it helps.



> Thanks in advance for  the help.
>
> Regards
> Lajos
>
>
> __
> 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,
Rabi Mishra
__
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] [heat][neutron] Extraroute support

2018-06-01 Thread Lajos Katona 

Hi,

Could somebody help me out with Neutron's Extraroute support in Hot 
templates.
The support status of the Extraroute is support.UNSUPPORTED in heat, and 
only create and delete are the supported operations.
see: 
https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/neutron/extraroute.py#LC35


As I see the unsupported tag was added when the feature was moved from 
the contrib folder to in-tree (https://review.openstack.org/186608)
Perhaps you can help me out why only create and delete are supported and 
update not.


Thanks in advance for  the help.

Regards
Lajos


__
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