Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bgpd : dynamic modification in lcomm-list is not taking effect. #4266

Merged
merged 1 commit into from
May 7, 2019

Conversation

vishaldhingra
Copy link
Contributor

lcomm-list is configured and attached to route-map via match clause.
If you modify the lcomm-list then it is not taking into effect via routemap.

Signed-off-by: vishaldhingra vdhingra@vmware.com

lcomm-list is configured and attached to route-map via match clause.
If you modify the lcomm-list then it is not taking into effect via routemap.

Signed-off-by: vishaldhingra <vdhingra@vmware.com>
@polychaeta polychaeta added the bgp label May 6, 2019
@vishaldhingra
Copy link
Contributor Author

c t
bgp large-community-list 1 permit 100:100:100
end

c t
route-map LRM permit 1
match large-community 1
end

Now apply this route-map to the neighbor with direction IN.
Suppose the route X is coming with lcomm val 200:200:200

Now the BGP fib has no entry for X route.

Modify the large-community-list
c t
bgp large-community-list 1 permit 200:200:200
end

with this change it should install the route in BGP RIB

This PR will send the trigger to route-map dependency framework to calculate the routemaps
dependent on lcomm list 1

@NetDEF-CI
Copy link
Collaborator

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-FRRPULLREQ-7531/

This is a comment from an automated CI system.
For questions and feedback in regards to this CI system, please feel free to email
Martin Winter - mwinter (at) opensourcerouting.org.


CLANG Static Analyzer Summary

  • Github Pull Request 4266, comparing to Git base SHA 9a968a1

No Changes in Static Analysis warnings compared to base

12 Static Analyzer issues remaining.

See details at
https://ci1.netdef.org/browse/FRR-FRRPULLREQ-7531/artifact/shared/static_analysis/index.html

@LabN-CI
Copy link
Collaborator

LabN-CI commented May 6, 2019

💚 Basic BGPD CI results: SUCCESS, 0 tests failed

Results table
_ _
Result SUCCESS git merge/4266 35f6f85
Date 05/06/2019
Start 10:05:24
Finish 10:29:25
Run-Time 24:01
Total 1816
Pass 1816
Fail 0
Valgrind-Errors 0
Valgrind-Loss 0
Details vncregress-2019-05-06-10:05:24.txt
Log autoscript-2019-05-06-10:06:10.log.bz2
Memory 441 443 376

For details, please contact louberger

@donaldsharp donaldsharp merged commit b0a973b into FRRouting:master May 7, 2019
@vishaldhingra vishaldhingra deleted the lcomm_dyn branch December 10, 2019 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants