diff options
| author | Chirag Shah <chirag@nvidia.com> | 2023-10-19 22:07:15 -0700 |
|---|---|---|
| committer | Chirag Shah <chirag@nvidia.com> | 2023-12-05 11:30:50 -0800 |
| commit | 64745d413ccf0b87df540bcf9e8a2b1dcbb893e2 (patch) | |
| tree | 2f3b1fbd1d8d6277328dbd989d09b6e5de0423af | |
| parent | 03e3b34b4df8d6ca67c2a33f16c4d1b4501a72b1 (diff) | |
pbrd: fix map seq installed flag in json
Ticket:#3638598
Testing:
Post fix:
tor-2# show pbr map json
[
{
"name":"global-vrf-PBR-map",
"valid":true,
"policies":[
{
"id":1,
"sequenceNumber":10,
"ruleNumber":309,
"vrfUnchanged":false,
"installed":true, <<<< now display correct value
"installedReason":"Valid",
"vrfName":"sym_1",
"matchSrc":"10.1.200.0\/24",
"matchDst":"10.6.200.0\/24"
}
]
}
Signed-off-by: Chirag Shah <chirag@nvidia.com>
| -rw-r--r-- | pbrd/pbr_vty.c | 3 | ||||
| -rw-r--r-- | tests/topotests/pbr_topo1/r1/pbr-map.json | 6 |
2 files changed, 4 insertions, 5 deletions
diff --git a/pbrd/pbr_vty.c b/pbrd/pbr_vty.c index 9589e5be23..64d88847c8 100644 --- a/pbrd/pbr_vty.c +++ b/pbrd/pbr_vty.c @@ -1667,8 +1667,7 @@ static void vty_json_pbrms(json_object *j, struct vty *vty, json_object_int_add(jpbrm, "sequenceNumber", pbrms->seqno); json_object_int_add(jpbrm, "ruleNumber", pbrms->ruleno); json_object_boolean_add(jpbrm, "vrfUnchanged", pbrms->vrf_unchanged); - json_object_boolean_add(jpbrm, "installed", - pbr_nht_get_installed(nhg_name)); + json_object_boolean_add(jpbrm, "installed", pbrms->installed); json_object_string_add(jpbrm, "installedReason", pbrms->reason ? rbuf : "Valid"); diff --git a/tests/topotests/pbr_topo1/r1/pbr-map.json b/tests/topotests/pbr_topo1/r1/pbr-map.json index bfa0ecb849..aaf2b5e8dc 100644 --- a/tests/topotests/pbr_topo1/r1/pbr-map.json +++ b/tests/topotests/pbr_topo1/r1/pbr-map.json @@ -18,7 +18,7 @@ { "sequenceNumber":10, "vrfUnchanged":false, - "installed":true, + "installed":false, "installedReason":"Invalid Src or Dst", "nexthopGroup":{ "name":"C", @@ -98,7 +98,7 @@ { "sequenceNumber":5, "vrfUnchanged":false, - "installed":false, + "installed":true, "installedReason":"Invalid NH-group", "nexthopGroup":{ "name":"B", @@ -111,7 +111,7 @@ { "sequenceNumber":10, "vrfUnchanged":true, - "installed":false, + "installed":true, "installedReason":"Valid", "matchSrc":"1.2.0.0\/16", "matchDst":"3.4.5.0\/24" |