Skip to content
This repository was archived by the owner on Aug 7, 2025. It is now read-only.

Commit 46b3f23

Browse files
Obed N Munozgrahamwhaley
authored andcommitted
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 <obed.n.munoz@intel.com>
1 parent b6c7cf1 commit 46b3f23

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

metrics/report/makereport.sh

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ setup() {
100100
}
101101

102102
run() {
103-
docker run -ti --rm -v ${HOSTINPUTDIR}:${GUESTINPUTDIR} -v ${HOSTOUTPUTDIR}:${GUESTOUTPUTDIR} ${extra_volumes} ${IMAGE} ${extra_command}
103+
docker run ${extra_opts} --rm -v ${HOSTINPUTDIR}:${GUESTINPUTDIR} -v ${HOSTOUTPUTDIR}:${GUESTOUTPUTDIR} ${extra_volumes} ${IMAGE} ${extra_command}
104104
ls -la ${HOSTOUTPUTDIR}/*
105105
}
106106

@@ -113,6 +113,7 @@ main() {
113113
# In debug mode, run a shell instead of the default report generation
114114
extra_command="bash"
115115
extra_volumes="-v ${HOSTSCRIPTDIR}:${GUESTSCRIPTDIR}"
116+
extra_opts="-ti"
116117
;;
117118
esac
118119
done

0 commit comments

Comments
 (0)