[Fuego] [PATCH] linaro: add option to skip installing dependencies
Tim.Bird at sony.com
Tim.Bird at sony.com
Wed Mar 13 19:13:54 UTC 2019
Looks good. Applied to the fuegotest 'next' branch.
Thanks,
-- Tim
> -----Original Message-----
> From: fuego-bounces at lists.linuxfoundation.org [mailto:fuego-
> bounces at lists.linuxfoundation.org] On Behalf Of Daniel Sangorrin
> Sent: Monday, March 04, 2019 9:39 PM
> To: fuego at lists.linuxfoundation.org
> Subject: [Fuego] [PATCH] linaro: add option to skip installing dependencies
>
> Signed-off-by: Daniel Sangorrin <daniel.sangorrin at toshiba.co.jp>
> ---
> tests/Functional.linaro/fuego_test.sh | 14 ++++++++++----
> tests/Functional.linaro/test.yaml | 5 +++++
> 2 files changed, 15 insertions(+), 4 deletions(-)
>
> diff --git a/tests/Functional.linaro/fuego_test.sh
> b/tests/Functional.linaro/fuego_test.sh
> index 3959256..86d3c43 100755
> --- a/tests/Functional.linaro/fuego_test.sh
> +++ b/tests/Functional.linaro/fuego_test.sh
> @@ -1,14 +1,13 @@
> gitrepo="https://github.com/Linaro/test-definitions.git"
>
> # Root permissions required for
> -# - installing dependencies on the target (debian/centos) when -s is not
> specified
> +# - installing dependencies on the target (debian/centos) (unless
> SKIPINSTALL=1)
> # - executing some of the tests
> # FIXTHIS: don't force root permissions for tests that do not require them
> NEED_ROOT=1
>
> function test_pre_check {
> # linaro parser dependencies
> - # FIXTHIS: use dependencies specified in the test definition yaml
> assert_has_program sed
> assert_has_program awk
> assert_has_program egrep
> @@ -50,8 +49,15 @@ function test_run {
> PARAMS=""
> fi
>
> - # FIXTHIS: don't use -s for targets with debian/centos
> - test-runner -o ${LOGDIR} $test_or_plan_flag ${REPO_PATH}/$yaml_file
> $PARAMS -g $LOGIN@$IPADDR -s -e
> + # Note: linaro already detects if the OS supports installing dependencies
> + SKIPINSTALL=${FUNCTIONAL_LINARO_SKIPINSTALL:-0}
> + if [ "$SKIPINSTALL" -eq 1 ]; then
> + SKIPFLAG="-s"
> + else
> + SKIPFLAG=""
> + fi
> +
> + test-runner -o ${LOGDIR} $test_or_plan_flag ${REPO_PATH}/$yaml_file
> $PARAMS -g $LOGIN@$IPADDR $SKIPFLAG -e
> }
>
> # FIXTHIS: the log directory is populated with a copy of the whole repository,
> clean unnecessary files
> diff --git a/tests/Functional.linaro/test.yaml
> b/tests/Functional.linaro/test.yaml
> index a2efee8..c31d8f4 100644
> --- a/tests/Functional.linaro/test.yaml
> +++ b/tests/Functional.linaro/test.yaml
> @@ -19,6 +19,11 @@ params:
> description: List of params for the test PARAM1=VALUE1
> [PARAM2=VALUE2]
> example: "TESTS='pwd'"
> optional: yes
> + - SKIPINSTALL:
> + description: Skip install dependencies on the target
> + example: 1 (skip) or 0 (don't skip them)
> + default: 0 (don't skip them)
> + optional: yes
> data_files:
> - chart_config.json
> - fuego_test.sh
> --
> 2.7.4
>
> _______________________________________________
> Fuego mailing list
> Fuego at lists.linuxfoundation.org
> https://lists.linuxfoundation.org/mailman/listinfo/fuego
More information about the Fuego
mailing list