update comment for a potential bug

sonic-net · Jan 17, 2020 · 4602b14 · 4602b14
1 parent 4376b7f
commit 4602b14
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/files/build_templates/docker_image_ctl.j2 b/files/build_templates/docker_image_ctl.j2
@@ -113,6 +113,9 @@ function preStartAction()
     fi
 
     echo -n "" > /tmp/snmpagentaddr.yml
+    # we should avoid using 'keys' operation via redis-cli
+    # there would be an issue when KEY in database contains space or '\n'
+    # for loop on the non-tty 'keys' output will take the space or `\n` as seperator when parsing the element
     keys=`sonic-db-cli CONFIG_DB keys "SNMP_AGENT_ADDRESS_CONFIG|*"`
     count=1
     for key in $keys;do