From 46b3f230ee8a4c10d69b8425e30fb6032e1be8ac Mon Sep 17 00:00:00 2001 From: Obed N Munoz Date: Tue, 19 Nov 2019 14:58:37 -0600 Subject: [PATCH] scaling: Remove tty parameter in report's generation cmd This is in order to avoid tty-related issues in our CI systems which by default is not supporting tty. With this change we'll avoid the following faling report's generation `docker run` command. ``` the input device is not a TTY ``` Signed-off-by: Obed N Munoz --- metrics/report/makereport.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/metrics/report/makereport.sh b/metrics/report/makereport.sh index 962be76..0b49afd 100755 --- a/metrics/report/makereport.sh +++ b/metrics/report/makereport.sh @@ -100,7 +100,7 @@ setup() { } run() { - docker run -ti --rm -v ${HOSTINPUTDIR}:${GUESTINPUTDIR} -v ${HOSTOUTPUTDIR}:${GUESTOUTPUTDIR} ${extra_volumes} ${IMAGE} ${extra_command} + docker run ${extra_opts} --rm -v ${HOSTINPUTDIR}:${GUESTINPUTDIR} -v ${HOSTOUTPUTDIR}:${GUESTOUTPUTDIR} ${extra_volumes} ${IMAGE} ${extra_command} ls -la ${HOSTOUTPUTDIR}/* } @@ -113,6 +113,7 @@ main() { # In debug mode, run a shell instead of the default report generation extra_command="bash" extra_volumes="-v ${HOSTSCRIPTDIR}:${GUESTSCRIPTDIR}" + extra_opts="-ti" ;; esac done