ylqjgm
/
canvas-lms
mirror of https://github.com/instructure/canvas-lms.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

remove unwanted output from setup script in jenkins 

We the addition of the spinner jenkins is showing a lot of output
coming from the spinner library. We don't need the spinner in local
dev smoker test. We add the ability to disable the spinner if is
running in jenkins.

refs DE-623
flag=none

test-plan:
1- Local dev smoker test passes and does not show unwanted output
2- setup script works locally

Change-Id: I4e476137e72ba44db69284130a1a4a7e6531dca7
Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/262831
Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com>
QA-Review: Andrea Cirulli <andrea.cirulli@instructure.com>
Reviewed-by: James Butters <jbutters@instructure.com>
Product-Review: Andrea Cirulli <andrea.cirulli@instructure.com>
This commit is contained in:
Andrea Cirulli 2021-04-13 13:58:30 -05:00
parent a1454de972
commit 9e02838107
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
script/common/utils/spinner.sh
View File

@ -71,16 +71,20 @@ function _inst_spinner() {
}
function start_spinner {
_inst_spinner "start" "$1" &
export _sp_pid=$!
disown
if ! is_running_on_jenkins; then
_inst_spinner "start" "$1" &
export _sp_pid=$!
disown
fi
}
function stop_spinner {
exit_status=$?
if [[ ! -z ${1-} ]]; then
exit_status=$1
if ! is_running_on_jenkins; then
exit_status=$?
if [[ ! -z ${1-} ]]; then
exit_status=$1
fi
_inst_spinner "stop" $exit_status ${_sp_pid:-}
unset _sp_pid
fi
_inst_spinner "stop" $exit_status ${_sp_pid:-}
unset _sp_pid
}