From 948029405b3bf6d16b46424ff669ca7959a42a82 Mon Sep 17 00:00:00 2001 From: Craig Comstock Date: Tue, 18 Nov 2025 15:31:24 -0600 Subject: [PATCH] Fixed collection of masterfiles acceptance test results Was guarded by an "if nova" condition which is incorrect. We test masterfiles in community builds so need them always. Ticket: ENT-13515 Changelog: none --- build-scripts/test-on-testmachine | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/build-scripts/test-on-testmachine b/build-scripts/test-on-testmachine index 172fef7ce..d1e37643a 100755 --- a/build-scripts/test-on-testmachine +++ b/build-scripts/test-on-testmachine @@ -110,6 +110,15 @@ rsync -rv $INCLUDES --exclude="*" \ "$BASEDIR"/core/tests/acceptance/ \ >>/tmp/rsync.log +log_debug "Collecting test results from masterfiles repo..." +# shellcheck disable=SC2086 +# > Double quote to prevent globbing and word splitting. +# We want word splitting +rsync -rv $INCLUDES --exclude="*" \ + "$TESTMACHINE_URI$BASEDIR"/masterfiles/tests/acceptance/ \ + "$BASEDIR"/masterfiles/tests/acceptance/ \ + >>/tmp/rsync.log + if [ "$PROJECT" = nova ]; then log_debug "Collecting test results from enterprise repo..." # shellcheck disable=SC2086 @@ -119,14 +128,6 @@ if [ "$PROJECT" = nova ]; then "$TESTMACHINE_URI$BASEDIR"/enterprise/tests/acceptance/ \ "$BASEDIR"/enterprise/tests/acceptance/ \ >>/tmp/rsync.log - log_debug "Collecting test results from masterfiles repo..." - # shellcheck disable=SC2086 - # > Double quote to prevent globbing and word splitting. - # We want word splitting - rsync -rv $INCLUDES --exclude="*" \ - "$TESTMACHINE_URI$BASEDIR"/masterfiles/tests/acceptance/ \ - "$BASEDIR"/masterfiles/tests/acceptance/ \ - >>/tmp/rsync.log fi if [ $return_code -ne 0 ]; then