On 5/25/21 11:40 AM, Philippe Mathieu-Daudé wrote:
> On 5/20/21 7:42 PM, Alex Bennée wrote:
>> The hexagon container is always manually built but of course not
>> everyone will be building it themselves and pushing to their
>> registries. We still need to create a "local" registry copy for the
>> actual gitlab tests to run. We don't build it in this case, just pull
>> it across from the upstream registry. We disable this rule from
>> running on the qemu-project itself so it doesn't accidentally wipe out
>> our master copy.
>>
>> Fixes: 910c40ee94 ("gitlab: add build-user-hexagon test")
>> Signed-off-by: Alex Bennée <alex.ben...@linaro.org>
>> Tested-by: Cornelia Huck <coh...@redhat.com>
>> Reviewed-by: Willian Rampazzo <willi...@redhat.com>
>> Cc: Cornelia Huck <coh...@redhat.com>
>>
>> ---
>> v2
>> - fix silly typo
>> ---
>> .gitlab-ci.d/containers.yml | 27 +++++++++++++++++++++++++++
>> .gitlab-ci.yml | 2 ++
>> 2 files changed, 29 insertions(+)
>>
>> diff --git a/.gitlab-ci.d/containers.yml b/.gitlab-ci.d/containers.yml
>> index 3fb3c14f06..088c7e68c3 100644
>> --- a/.gitlab-ci.d/containers.yml
>> +++ b/.gitlab-ci.d/containers.yml
>> @@ -101,6 +101,33 @@ armhf-debian-cross-container:
>> variables:
>> NAME: debian-armhf-cross
>>
>> +# We never want to build hexagon in the CI system and by default we
>> +# always want to refer to the master registry where it lives.
>> +hexagon-cross-container:
>> + image: docker:stable
>> + stage: containers
>> + except:
>> + variables:
>> + - $CI_PROJECT_NAMESPACE == 'qemu-project'
>
> FYI Daniel said we should be consistent and use the 'rules:' syntax:
> https://lists.gnu.org/archive/html/qemu-devel/2021-05/msg07308.html
Since our rule set default to 'always', this should be:
rules:
- if: '$CI_PROJECT_NAMESPACE == "qemu-project"'
when: always
- when: never
