]> git.puffer.fish Git - matthieu/frr.git/commitdiff
pbrd: fix show pbr map detail json
authorSindhu Parvathi Gopinathan <sgopinathan@nvidia.com>
Mon, 23 Oct 2023 05:59:06 +0000 (22:59 -0700)
committerMergify <37929162+mergify[bot]@users.noreply.github.com>
Sun, 29 Oct 2023 08:42:34 +0000 (08:42 +0000)
'detail' and 'josn' keyword is given as an optional parameter
for cli arguments. Hence 'detail' keyword was consider as a
pbr 'name' for "show pbr map detail json" command.

Before Fix:

```
cumulus#
cumulus# show pbr map detail json
[
]
cumulus#
```

After Fix:
```
cumulus# show pbr map detail json
[
  {
    "name":"MAP1",
    "valid":false,
    "policies":[
      {
        "id":1,
        "sequenceNumber":10,
        "ruleNumber":309,
        "vrfUnchanged":false,
        "installed":false,
        "installedReason":"Invalid Src or Dst",
        "vrfName":"default"
      }
    ]
  }
]
cumulus#
```

Ticket:#3638600

Issue:3638600

Testing: UT done

Signed-off-by: Sindhu Parvathi Gopinathan's <sgopinathan@nvidia.com>
(cherry picked from commit d621c365a65f825adfbc00128360390d0b2bd5be)

pbrd/pbr_vty.c

index abd997e9c08b083359e6996ebe6e67e710708fb3..af83993c9eaa553dbddaca2d1f653c29162bc0cb 100644 (file)
@@ -1062,7 +1062,7 @@ static void vty_json_pbr_map(json_object *j, struct vty *vty,
 
 DEFPY (show_pbr_map,
        show_pbr_map_cmd,
-       "show pbr map [NAME$name] [detail$detail|json$json]",
+       "show pbr map [NAME$name] [detail$detail] [json$json]",
        SHOW_STR
        PBR_STR
        "PBR Map\n"