Skip to content

Conversation

@chiragshah6
Copy link
Member

For auto configured value RD value comes as NULL,
switching back to original change will ensure to cover for both auto and user configured RD value in JSON.

tor-11# show bgp vrf blue ipv4 unicast route-leak json {
  "vrf":"blue",
  "afiSafi":"ipv4Unicast",
  "importFromVrfs":[
    "purple"
  ],
  "importRts":"10.10.3.11:6",
  "exportToVrfs":[
    "purple"
  ],
  "routeDistinguisher":"(null)", <<<<<
  "exportRts":"10.10.3.11:10"
}

Signed-off-by: Chirag Shah chirag@nvidia.com

For auto configured value RD value comes as NULL,
switching back to original change will ensure to cover
for both auto and user configured RD value in JSON.

tor-11# show bgp vrf blue ipv4 unicast route-leak json
{
  "vrf":"blue",
  "afiSafi":"ipv4Unicast",
  "importFromVrfs":[
    "purple"
  ],
  "importRts":"10.10.3.11:6",
  "exportToVrfs":[
    "purple"
  ],
  "routeDistinguisher":"(null)", <<<<<
  "exportRts":"10.10.3.11:10"
}

Signed-off-by: Chirag Shah <chirag@nvidia.com>
@ton31337
Copy link
Member

ton31337 commented Feb 4, 2025

@Mergifyio backport stable/10.2 stable/10.1 stable/10.0

@mergify
Copy link

mergify bot commented Feb 4, 2025

backport stable/10.2 stable/10.1 stable/10.0

✅ Backports have been created

Details

Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

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

looks good

@riw777 riw777 merged commit f74fa95 into FRRouting:master Feb 4, 2025
18 checks passed
Jafaral added a commit that referenced this pull request Feb 4, 2025
bgpd: fix route-distinguisher in vrf leak json cmd (backport #17992)
Jafaral added a commit that referenced this pull request Feb 4, 2025
bgpd: fix route-distinguisher in vrf leak json cmd (backport #17992)
Jafaral added a commit that referenced this pull request Feb 4, 2025
bgpd: fix route-distinguisher in vrf leak json cmd (backport #17992)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants