matthew/sonic-buildimage
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Fix telemetry.sh passing in null as log level value (#14303)

Browse Source 
#### Why I did it

Bug in script that was passing in null as log level value if missing from config_db

#### How I did it

Added more robust conditional statement

#### How to verify it

1) Remove log_level from config db
2) config reload -y
3) telemetry should not crash
This commit is contained in:
Zain Budhwani 2023-03-20 16:22:11 -07:00 committed by GitHub
parent 4429bdd091
commit 881b925d19
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
dockers/docker-sonic-telemetry/telemetry.sh
View File

@ -63,7 +63,7 @@ if [ -z $CLIENT_AUTH ] || [ $CLIENT_AUTH == "false" ]; then
fi fi
LOG_LEVEL=$(echo $GNMI | jq -r '.log_level') LOG_LEVEL=$(echo $GNMI | jq -r '.log_level')
if [ ! -z $LOG_LEVEL ]; then if [[ $LOG_LEVEL =~ ^[0-9]+$ ]]; then
TELEMETRY_ARGS+=" -v=$LOG_LEVEL" TELEMETRY_ARGS+=" -v=$LOG_LEVEL"
else else
TELEMETRY_ARGS+=" -v=2" TELEMETRY_ARGS+=" -v=2"