Skip to content

Commit 20d3bd8

Browse files
committed
yang: policer should not define cbs default value of 0
It looks like there is a behavior difference between libyang1 and libyang3 where a default value of 0 was actually treated the same as not set, so now the `POLICER_PIR_WITHOUT_CIR` test case is failing with: ``` ERROR YANG-TEST:test_yang_model.py:303 POLICER_PIR_WITHOUT_CIR: Mismatch ["pir can't be configured without cir."] and failed to parse data tree: cbs can't be configured without cir.: Data path: /sonic-policer:sonic-policer/POLICER/POLICER_LIST[name='e3policer']/cbs ``` Get rid of this bogus default.
1 parent c73bca1 commit 20d3bd8

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/sonic-yang-models/yang-templates/sonic-policer.yang.j2

-1
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ module sonic-policer {
6666
error-message "cbs must be greater than or equal to cir";
6767
}
6868
type uint64;
69-
default 0;
7069
description
7170
"Committed burst size for the dual-rate token bucket
7271
policer. This value represents the depth of the token

0 commit comments

Comments
 (0)