Don't create a pty to run vtysh inside of the docker container (sonic…

…-net#1792)
justin-oliver · Jun 14, 2018 · fae346f · fae346f
1 parent 1942183
commit fae346f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 16 deletions.
diff --git a/dockers/docker-fpm-frr/base_image_files/vtysh b/dockers/docker-fpm-frr/base_image_files/vtysh
@@ -1,10 +1,3 @@
 #!/bin/bash
 
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS bgp vtysh "$@"
+docker exec -i bgp vtysh "$@"
diff --git a/dockers/docker-fpm-quagga/base_image_files/vtysh b/dockers/docker-fpm-quagga/base_image_files/vtysh
@@ -1,10 +1,3 @@
 #!/bin/bash
 
-DOCKER_EXEC_FLAGS="i"
-
-# Determine whether stdout is on a terminal
-if [ -t 1 ] ; then
-    DOCKER_EXEC_FLAGS+="t"
-fi
-
-docker exec -$DOCKER_EXEC_FLAGS bgp vtysh "$@"
+docker exec -i bgp vtysh "$@"