[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v1 3/8] gitlab: add special rule for the hexagon container
From: |
Alex Bennée |
Subject: |
Re: [PATCH v1 3/8] gitlab: add special rule for the hexagon container |
Date: |
Tue, 25 May 2021 16:06:13 +0100 |
User-agent: |
mu4e 1.5.13; emacs 28.0.50 |
Philippe Mathieu-Daudé <f4bug@amsat.org> writes:
> 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.bennee@linaro.org>
>>> Tested-by: Cornelia Huck <cohuck@redhat.com>
>>> Reviewed-by: Willian Rampazzo <willianr@redhat.com>
>>> Cc: Cornelia Huck <cohuck@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
Other way around.
--
Alex Bennée
- [PATCH v1 0/8] various misc fixes (gitlab, gdbstub, plugins), Alex Bennée, 2021/05/20
- [PATCH v1 4/8] gdbstub: Constify GdbCmdParseEntry, Alex Bennée, 2021/05/20
- [PATCH v1 6/8] hmp-commands: expand type of icount to "l" in replay commands, Alex Bennée, 2021/05/20
- [PATCH v1 5/8] gdbstub: Replace GdbCmdContext with plain g_array(), Alex Bennée, 2021/05/20
- [PATCH v1 8/8] plugins/syscall: Added a table-like summary output, Alex Bennée, 2021/05/20
- [PATCH v1 7/8] gdbstub: tidy away reverse debugging check into function, Alex Bennée, 2021/05/20
- Re: [PATCH v1 0/8] various misc fixes (gitlab, gdbstub, plugins), Alex Bennée, 2021/05/24