]> 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:33 +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 ee9ee32f8b5c1cc97f4229a369bb4a23ceb5b264..0325c79c3ccd444af19ca2204e13cb645257b31c 100644 (file)
@@ -1049,7 +1049,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"