From f4a9f7edaac11a9985fc99870bb68d34e386d122 Mon Sep 17 00:00:00 2001 From: Stepan Blyshchak <38952541+stepanblyschak@users.noreply.github.com> Date: Sun, 19 Jun 2022 14:14:01 +0300 Subject: [PATCH] [sonic-feature.yang] fix check_up_status field type (#10986) - Why I did it An issue is encountered when a value "False" is written for a feature in "check_up_status" field, which does not pass YANG validation. - How I did it We usually use stypes::boolean_type for such fields, even in this YANG model. This custom type, supports "False" value. - How to verify it Write "False" in "check_up_status" field and see if YANG validation passes. Signed-off-by: Stepan Blyschak --- src/sonic-yang-models/yang-models/sonic-feature.yang | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/sonic-yang-models/yang-models/sonic-feature.yang b/src/sonic-yang-models/yang-models/sonic-feature.yang index 54133ef106..f27411788e 100644 --- a/src/sonic-yang-models/yang-models/sonic-feature.yang +++ b/src/sonic-yang-models/yang-models/sonic-feature.yang @@ -90,8 +90,8 @@ module sonic-feature{ leaf check_up_status { description "This configuration controls the system ready tool to check the app ready/up status"; - type boolean; - default false; + type stypes:boolean_type; + default "false"; } } }