Skip to content

doc: bgp vpn-vrf leaking CLI#1937

Merged
qlyoung merged 1 commit intoFRRouting:masterfrom
LabNConsulting:working/master/bgp-vpn-leak-doc
Mar 20, 2018
Merged

doc: bgp vpn-vrf leaking CLI#1937
qlyoung merged 1 commit intoFRRouting:masterfrom
LabNConsulting:working/master/bgp-vpn-leak-doc

Conversation

@gpziemba
Copy link
Copy Markdown
Contributor

@gpziemba gpziemba commented Mar 20, 2018 •
edited
Loading

Documentation for bgpd vpn-vrf leaking commands of #1913.

@gpziemba
doc: bgp vpn-vrf leaking CLI
b572f82 
Signed-off-by: G. Paul Ziemba <paulz@labn.net>
@LabN-CI
Copy link
Copy Markdown
Collaborator

LabN-CI commented Mar 20, 2018

💚 Basic BGPD CI results: SUCCESS, 0 tests failed

Results table
_ _
Result SUCCESS git merge/1937 b572f82
Date 03/20/2018
Start 00:35:19
Finish 00:58:32
Run-Time 23:13
Total 1816
Pass 1816
Fail 0
Valgrind-Errors 0
Valgrind-Loss 0
Details vncregress-2018-03-20-00:35:19.txt
Log autoscript-2018-03-20-00:36:14.log.bz2

For details, please contact louberger

@gpziemba gpziemba mentioned this pull request Mar 20, 2018
Merged
gpziemba added a commit to LabNConsulting/frr that referenced this pull request Mar 20, 2018
@gpziemba
bgpd: new vpn-policy CLI
b9c7bc5 
PR FRRouting#1739 added code to leak routes between (default VRF) VPN safi and unicast RIBs in any VRF. That set of changes included temporary CLI including vpn-policy blocks to specify RD/RT/label/&c. After considerable discussion, we arrived at a consensus CLI shown below.

The code of this PR implements the vpn-specific parts of this syntax:

router bgp <as> [vrf <FOO>]
    address-family <afi> unicast
        rd (vpn|evpn) export (AS:NN | IP:nn)
        label (vpn|evpn) export (0..1048575)
        rt (vpn|evpn) (import|export|both) RTLIST...
        nexthop vpn (import|export) (A.B.C.D | X:X::X:X)
        route-map (vpn|evpn|vrf NAME) (import|export) MAP

        [no] import|export [vpn|evpn|evpn8]
        [no] import|export vrf NAME

User documentation of the vpn-specific parts of the above syntax is in PR FRRouting#1937

Signed-off-by: G. Paul Ziemba <paulz@labn.net>
@NetDEF-CI
Copy link
Copy Markdown
Collaborator

NetDEF-CI commented Mar 20, 2018

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-2994/

This is a comment from an EXPERIMENTAL 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 1937, comparing to Git base SHA 08a4253

No Changes in Static Analysis warnings compared to base

19 Static Analyzer issues remaining.

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

@NetDEF-CI
Copy link
Copy Markdown
Collaborator

NetDEF-CI commented Mar 20, 2018

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-2995/

This is a comment from an EXPERIMENTAL 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 1937, comparing to Git base SHA 08a4253

No Changes in Static Analysis warnings compared to base

19 Static Analyzer issues remaining.

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

qlyoung
qlyoung approved these changes Mar 20, 2018
View reviewed changes
Copy link
Copy Markdown
Member

@qlyoung qlyoung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, though I will come back through and rewrap some of these paragraphs to 80 characters in a followup pr

@qlyoung qlyoung merged commit 0cef942 into FRRouting:master Mar 20, 2018
qlyoung pushed a commit to qlyoung/frr that referenced this pull request Mar 22, 2018
@gpziemba @donaldsharp
bgpd: new vpn-policy CLI
bf37133 
PR FRRouting#1739 added code to leak routes between (default VRF) VPN safi and unicast RIBs in any VRF. That set of changes included temporary CLI including vpn-policy blocks to specify RD/RT/label/&c. After considerable discussion, we arrived at a consensus CLI shown below.

The code of this PR implements the vpn-specific parts of this syntax:

router bgp <as> [vrf <FOO>]
    address-family <afi> unicast
        rd (vpn|evpn) export (AS:NN | IP:nn)
        label (vpn|evpn) export (0..1048575)
        rt (vpn|evpn) (import|export|both) RTLIST...
        nexthop vpn (import|export) (A.B.C.D | X:X::X:X)
        route-map (vpn|evpn|vrf NAME) (import|export) MAP

        [no] import|export [vpn|evpn|evpn8]
        [no] import|export vrf NAME

User documentation of the vpn-specific parts of the above syntax is in PR FRRouting#1937

Signed-off-by: G. Paul Ziemba <paulz@labn.net>
@louberger louberger deleted the working/master/bgp-vpn-leak-doc branch March 17, 2020 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@qlyoung qlyoung qlyoung approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@gpziemba @LabN-CI @NetDEF-CI @qlyoung