Add phase messages to fuego scripts

Commit Message

Bird, Timothy June 25, 2016, 12:08 a.m. UTC

Add some extra echo statements in various fuego scripts.
These show up in the console log, and are very handy for debugging.
They are useful to find the commands for a particular phase more
easily in the log.
---
 engine/scripts/benchmark.sh  | 5 +++++
 engine/scripts/functional.sh | 6 ++++++
 engine/scripts/functions.sh  | 2 ++
 3 files changed, 13 insertions(+)

Comments

Dmitry Cherkasov July 5, 2016, 7:10 p.m. UTC | #1

This is convenient.

Thanks for the patch.

On Sat, Jun 25, 2016 at 3:08 AM, Bird, Timothy <[e-mail]>
wrote:

> Add some extra echo statements in various fuego scripts.
> These show up in the console log, and are very handy for debugging.
> They are useful to find the commands for a particular phase more
> easily in the log.
> ---
>  engine/scripts/benchmark.sh  | 5 +++++
>  engine/scripts/functional.sh | 6 ++++++
>  engine/scripts/functions.sh  | 2 ++
>  3 files changed, 13 insertions(+)
>
> diff --git a/engine/scripts/benchmark.sh b/engine/scripts/benchmark.sh
> index 3ffd51b..cbdd10e 100644
> --- a/engine/scripts/benchmark.sh
> +++ b/engine/scripts/benchmark.sh
> @@ -28,16 +28,21 @@ source $FUEGO_SCRIPTS_PATH/functions.sh
>
>  source $FUEGO_SCRIPTS_PATH/reports.sh
>
> +echo "##### doing fuego phase: pre_test ########"
>  pre_test $TESTDIR
>
> +echo "##### doing fuego phase: build ########"
>  if $Rebuild; then
>      build
>  fi
>
> +echo "##### doing fuego phase: deploy ########"
>  deploy
>
> +echo "##### doing fuego phase: run ########"
>  test_run
>
> +echo "##### doing fuego phases: get_testlog AND processing ########"
>  set_testres_file
>  bench_processing
>
> diff --git a/engine/scripts/functional.sh b/engine/scripts/functional.sh
> index 8f91c74..b93bafc 100644
> --- a/engine/scripts/functional.sh
> +++ b/engine/scripts/functional.sh
> @@ -27,16 +27,22 @@ set_overlay_vars
>  source $FUEGO_SCRIPTS_PATH/reports.sh
>  source $FUEGO_SCRIPTS_PATH/functions.sh
>
> +echo "##### doing fuego phase: pre_test ########"
>  pre_test $TESTDIR
>
> +echo "##### doing fuego phase: build ########"
>  if $Rebuild; then
>      build
>  fi
>
> +echo "##### doing fuego phase: deploy ########"
>  deploy
>
> +echo "##### doing fuego phase: run ########"
>  test_run
>
> +echo "##### doing fuego phase: get_testlog ########"
>  get_testlog $TESTDIR
>
> +echo "##### doing fuego phase: processing ########"
>  test_processing
> diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
> index 2d23bc0..abb8d78 100755
> --- a/engine/scripts/functions.sh
> +++ b/engine/scripts/functions.sh
> @@ -351,6 +351,8 @@ function fail_check_cases () {
>  }
>
>  function post_test {
> +  echo "##### doing fuego phase: post_test ########"
> +
>    # source generated prolog.sh file since post_test is called separately
>    source $FUEGO_ENGINE_PATH/work/${NODE_NAME}_prolog.sh
>    export SSHPASS=$PASSWORD
> --
> 2.1.4
>
>

Patch

diff --git a/engine/scripts/benchmark.sh b/engine/scripts/benchmark.sh
index 3ffd51b..cbdd10e 100644
--- a/engine/scripts/benchmark.sh
+++ b/engine/scripts/benchmark.sh
@@ -28,16 +28,21 @@  source $FUEGO_SCRIPTS_PATH/functions.sh
 
 source $FUEGO_SCRIPTS_PATH/reports.sh
 
+echo "##### doing fuego phase: pre_test ########"
 pre_test $TESTDIR
 
+echo "##### doing fuego phase: build ########"
 if $Rebuild; then
     build
 fi
 
+echo "##### doing fuego phase: deploy ########"
 deploy
 
+echo "##### doing fuego phase: run ########"
 test_run
 
+echo "##### doing fuego phases: get_testlog AND processing ########"
 set_testres_file
 bench_processing
 
diff --git a/engine/scripts/functional.sh b/engine/scripts/functional.sh
index 8f91c74..b93bafc 100644
--- a/engine/scripts/functional.sh
+++ b/engine/scripts/functional.sh
@@ -27,16 +27,22 @@  set_overlay_vars
 source $FUEGO_SCRIPTS_PATH/reports.sh
 source $FUEGO_SCRIPTS_PATH/functions.sh
 
+echo "##### doing fuego phase: pre_test ########"
 pre_test $TESTDIR
 
+echo "##### doing fuego phase: build ########"
 if $Rebuild; then
     build
 fi
 
+echo "##### doing fuego phase: deploy ########"
 deploy
 
+echo "##### doing fuego phase: run ########"
 test_run
 
+echo "##### doing fuego phase: get_testlog ########"
 get_testlog $TESTDIR
 
+echo "##### doing fuego phase: processing ########"
 test_processing
diff --git a/engine/scripts/functions.sh b/engine/scripts/functions.sh
index 2d23bc0..abb8d78 100755
--- a/engine/scripts/functions.sh
+++ b/engine/scripts/functions.sh
@@ -351,6 +351,8 @@  function fail_check_cases () {
 }
 
 function post_test {
+  echo "##### doing fuego phase: post_test ########"
+
   # source generated prolog.sh file since post_test is called separately
   source $FUEGO_ENGINE_PATH/work/${NODE_NAME}_prolog.sh
   export SSHPASS=$PASSWORD