Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-03 Thread Stefano Stabellini
On Wed, 2 Oct 2024, Stefano Stabellini wrote: > On Thu, 3 Oct 2024, Marek Marczykowski-Górecki wrote: > > On Wed, Oct 02, 2024 at 04:30:25PM -0700, Stefano Stabellini wrote: > > > On Thu, 3 Oct 2024, Marek Marczykowski-Górecki wrote: > > > > The problem is this doesn't work. The group-level variabl

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Stefano Stabellini
On Thu, 3 Oct 2024, Marek Marczykowski-Górecki wrote: > On Wed, Oct 02, 2024 at 04:30:25PM -0700, Stefano Stabellini wrote: > > On Thu, 3 Oct 2024, Marek Marczykowski-Górecki wrote: > > > The problem is this doesn't work. The group-level variable overrides the > > > one in yaml. See the commit mess

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Marek Marczykowski-Górecki
On Wed, Oct 02, 2024 at 04:30:25PM -0700, Stefano Stabellini wrote: > On Thu, 3 Oct 2024, Marek Marczykowski-Górecki wrote: > > The problem is this doesn't work. The group-level variable overrides the > > one in yaml. See the commit message and the link there... > > Now I understand the problem, w

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Stefano Stabellini
On Thu, 3 Oct 2024, Marek Marczykowski-Górecki wrote: > On Wed, Oct 02, 2024 at 03:22:59PM -0700, Stefano Stabellini wrote: > > I forgot to reply to one important part below > > > > > > On Wed, 2 Oct 2024, Stefano Stabellini wrote: > > > On Wed, 2 Oct 2024, Marek Marczykowski-Górecki wrote: > > >

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Andrew Cooper
On 02/10/2024 11:55 pm, Marek Marczykowski-Górecki wrote: > On Wed, Oct 02, 2024 at 03:22:59PM -0700, Stefano Stabellini wrote: >> I forgot to reply to one important part below >> >> >> On Wed, 2 Oct 2024, Stefano Stabellini wrote: >>> On Wed, 2 Oct 2024, Marek Marczykowski-Górecki wrote: Chec

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Marek Marczykowski-Górecki
On Wed, Oct 02, 2024 at 03:22:59PM -0700, Stefano Stabellini wrote: > I forgot to reply to one important part below > > > On Wed, 2 Oct 2024, Stefano Stabellini wrote: > > On Wed, 2 Oct 2024, Marek Marczykowski-Górecki wrote: > > > Check if xen.efi is bootable with an XTF dom0. > > > > > > The T

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Stefano Stabellini
I forgot to reply to one important part below On Wed, 2 Oct 2024, Stefano Stabellini wrote: > On Wed, 2 Oct 2024, Marek Marczykowski-Górecki wrote: > > Check if xen.efi is bootable with an XTF dom0. > > > > The TEST_TIMEOUT is set in the script to override project-global value. > > Setting it in

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Stefano Stabellini
On Wed, 2 Oct 2024, Marek Marczykowski-Górecki wrote: > Check if xen.efi is bootable with an XTF dom0. > > The TEST_TIMEOUT is set in the script to override project-global value. > Setting it in the gitlab yaml file doesn't work, as it's too low > priority > (https://docs.gitlab.com/ee/ci/variable

Re: [PATCH 2/2] automation: add a smoke test for xen.efi on X86

2024-10-02 Thread Andrew Cooper
On 02/10/2024 1:42 pm, Marek Marczykowski-Górecki wrote: > Check if xen.efi is bootable with an XTF dom0. > > The TEST_TIMEOUT is set in the script to override project-global value. > Setting it in the gitlab yaml file doesn't work, as it's too low > priority > (https://docs.gitlab.com/ee/ci/variab