On 07.01.22 14:36, Gylstorff Quirin wrote:
> 
> 
> On 1/7/22 13:00, Jan Kiszka wrote:
>> From: Jan Kiszka <[email protected]>
>>
>> This makes the inclusion structure more regular and reduces duplications
>> by re-using the kernel_<version>.yml files for 3.1 and next. The trick
>> is that we pull common variables to additional top-level 'variables:'
>> blocks and only set what differes in the job-specific 'variables:'.
>>
>> DEPLOY_DIR_EXTENSION is now identical to BUILD_IDENTIFIER, and the
>> latter is derived from the XENOMAI_VERSION and KERNEL_VERSION. Only
>> those version variables are set at xenomai_<version>.yml and kernel-
>> specific job level, respectively.
>>
>> Signed-off-by: Jan Kiszka <[email protected]>
>> ---
>>   ci/gitlab-ci-base.yml                         |  3 +-
>>   ..._4_19_xenomai_next.yml => kernel_4_19.yml} | 32 +++----
>>   ..._5_10_xenomai_next.yml => kernel_5_10.yml} | 32 +++----
>>   ...el_5_4_xenomai_next.yml => kernel_5_4.yml} | 32 +++----
>>   ci/xenomai_3_0_x.yml                          | 23 ++---
>>   ci/xenomai_3_1_x.yml                          | 91 ++-----------------
>>   ci/xenomai_next.yml                           | 13 ++-
>>   7 files changed, 64 insertions(+), 162 deletions(-)
>>   rename ci/{kernel_4_19_xenomai_next.yml => kernel_4_19.yml} (71%)
>>   rename ci/{kernel_5_10_xenomai_next.yml => kernel_5_10.yml} (71%)
>>   rename ci/{kernel_5_4_xenomai_next.yml => kernel_5_4.yml} (71%)
>>
>> diff --git a/ci/gitlab-ci-base.yml b/ci/gitlab-ci-base.yml
>> index 367085d..27271ce 100644
>> --- a/ci/gitlab-ci-base.yml
>> +++ b/ci/gitlab-ci-base.yml
>> @@ -18,10 +18,11 @@ variables:
>>     https_proxy: "$HTTPS_PROXY"
>>     ftp_proxy: "$FTP_PROXY"
>>     no_proxy: "$NO_PROXY"
>> -  XENOMAI_BUILD_OPTION: ":opt-xenomai-next.yml"
>>     ISAR_IMAGE: demo-image
>>     ISAR_DISTRIBUTION: xenomai-demo
>>     LAVA_TESTS_ENABLED: "true"
>> +  DEPLOY_DIR_EXTENSION: "${BUILD_IDENTIFIER}"
>> +  BUILD_IDENTIFIER:
>> "xenomai-${XENOMAI_VERSION}_kernel-${KERNEL_VERSION}"
> 
> The variable DEPLOY_DIR_EXTENSIONS is no longer necessary. We could
> replace it in the scripts `deploy_to_aws.sh` and `run-lava-tests.sh`
> with BUILD_IDENTIFIER.
> 

DEPLOY_DIR_EXTENSION was mentioned somewhere in the README, so I didn't
dare to touch it. How should tests/README.md adjusted best then?

Jan

-- 
Siemens AG, Technology
Competence Center Embedded Linux

Reply via email to