smartbgp / yabgp

:bowtie:Yet Another BGP Python Implementation
http://yabgp.readthedocs.org
Apache License 2.0
238 stars 70 forks source link

ENLP #103

Closed vishchan85 closed 5 years ago

vishchan85 commented 5 years ago

I have the following configuration for my BGP SR-TE udpate - {'attr': {'1': 0, '2': [[2, [100, 2]]], '14': {'nlri': {'color': 250, 'endpoint': '206.1.0', 'distinguisher': 6001}, 'afi_safi': [1, 73], 'nexthop': '47.8.1.1'}, '23': {'0': 'new', '13': 965540, '12': 4200, '128': [{'9': 10, '1': [{'1': {'label': 999980}}, {'1': {'label': 900003}}, {'1': {'label': 900002}}]}]}, '8': ['NO_ADVERTISE']}}

I was looking to add ENLP functionality described in https://tools.ietf.org/html/draft-ietf-idr-segment-routing-te-policy-07#section-2.4.4 What is the option to be used. The documentation at https://yabgp.readthedocs.io/en/latest/msg_format.html#ipv4-sr-policy doesn't mention about it.

meidli commented 5 years ago

@vishchan85 Hi, this parameter is temporarily not supported, we will support it as soon as possible, thanks.

meidli commented 5 years ago

@zlpqingmei please help to complete it, here are some reference documents.

https://tools.ietf.org/html/draft-ietf-idr-segment-routing-te-policy-07#section-2.4.4 https://tools.ietf.org/html/draft-ietf-idr-segment-routing-te-policy-07#section-2.4.5 https://tools.ietf.org/html/draft-ietf-idr-segment-routing-te-policy-07#section-2.4.6 https://tools.ietf.org/html/draft-ietf-idr-tunnel-encaps-12#section-3.1 https://www.iana.org/assignments/bgp-parameters/bgp-parameters.xhtml

xiaopeng163 commented 5 years ago

@vishchan85 could you please try the latest master branch, we did a fix for this issue in master branch.

vishchan85 commented 5 years ago

thanks works now