[PATCH 0/1] Acceptance Tests: bump Avocado version requirement to 87.0

[PATCH 0/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Cleber Rosa]

This is being proposed as a separate single patch simply to show
that no known regressions have been introduced as far as the
acceptance tests/jobs are related.  CI job:

   https://gitlab.com/cleber.gnu/qemu/-/pipelines/286347312

This version (and 86.0) contain improvements that address specific
QEMU use cases, including:

 * Fix to the error message given when downloading assets

 * Asset listing/purging capabilities

Cleber Rosa (1):
  Acceptance Tests: bump Avocado version requirement to 87.0

 tests/requirements.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.25.4

[PATCH 1/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Cleber Rosa]

This version (and 86.0) contain improvements that address specific
QEMU use cases, including:

* Fix to the error message given when downloading assets

* Asset listing/purging capabilities

Signed-off-by: Cleber Rosa <crosa@redhat.com>
---
 tests/requirements.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/requirements.txt b/tests/requirements.txt
index 91f3a343b9..f57e895c4a 100644
--- a/tests/requirements.txt
+++ b/tests/requirements.txt
@@ -1,5 +1,5 @@
 # Add Python module requirements, one per line, to be installed
 # in the tests/venv Python virtual environment. For more info,
 # refer to: https://pip.pypa.io/en/stable/user_guide/#id1
-avocado-framework==85.0
+avocado-framework==87.0
 pycdlib==1.11.0
-- 
2.25.4

Re: [PATCH 1/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Wainer dos Santos Moschetta]

On 4/14/21 1:11 PM, Cleber Rosa wrote:
> This version (and 86.0) contain improvements that address specific
> QEMU use cases, including:
>
> * Fix to the error message given when downloading assets
>
> * Asset listing/purging capabilities
>
> Signed-off-by: Cleber Rosa <crosa@redhat.com>
> ---
>   tests/requirements.txt | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)


Acked-by: Wainer dos Santos Moschetta <wainersm@redhat.com>


>
> diff --git a/tests/requirements.txt b/tests/requirements.txt
> index 91f3a343b9..f57e895c4a 100644
> --- a/tests/requirements.txt
> +++ b/tests/requirements.txt
> @@ -1,5 +1,5 @@
>   # Add Python module requirements, one per line, to be installed
>   # in the tests/venv Python virtual environment. For more info,
>   # refer to: https://pip.pypa.io/en/stable/user_guide/#id1
> -avocado-framework==85.0
> +avocado-framework==87.0
>   pycdlib==1.11.0

Re: [PATCH 1/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Willian Rampazzo]

On Wed, Apr 14, 2021 at 1:12 PM Cleber Rosa <crosa@redhat.com> wrote:
>
> This version (and 86.0) contain improvements that address specific
> QEMU use cases, including:
>
> * Fix to the error message given when downloading assets
>
> * Asset listing/purging capabilities
>
> Signed-off-by: Cleber Rosa <crosa@redhat.com>
> ---
>  tests/requirements.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>

The release is at the door so, soon it will be good to do this change:

Reviewed-by: Willian Rampazzo <willianr@redhat.com>

> diff --git a/tests/requirements.txt b/tests/requirements.txt
> index 91f3a343b9..f57e895c4a 100644
> --- a/tests/requirements.txt
> +++ b/tests/requirements.txt
> @@ -1,5 +1,5 @@
>  # Add Python module requirements, one per line, to be installed
>  # in the tests/venv Python virtual environment. For more info,
>  # refer to: https://pip.pypa.io/en/stable/user_guide/#id1
> -avocado-framework==85.0
> +avocado-framework==87.0
>  pycdlib==1.11.0
> --
> 2.25.4
>

Re: [PATCH 0/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Philippe Mathieu-Daudé]

On 4/14/21 6:11 PM, Cleber Rosa wrote:
> This is being proposed as a separate single patch simply to show
> that no known regressions have been introduced as far as the
> acceptance tests/jobs are related.  CI job:
> 
>    https://gitlab.com/cleber.gnu/qemu/-/pipelines/286347312
> 
> This version (and 86.0) contain improvements that address specific
> QEMU use cases, including:
> 
>  * Fix to the error message given when downloading assets
> 
>  * Asset listing/purging capabilities
> 
> Cleber Rosa (1):
>   Acceptance Tests: bump Avocado version requirement to 87.0
> 
>  tests/requirements.txt | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Ping? This patch should fix the mainstream pipeline error:
https://gitlab.com/qemu-project/qemu/-/jobs/1229752162

(see
https://www.mail-archive.com/qemu-devel@nongnu.org/msg794416.html)

Re: [PATCH 0/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Cleber Rosa]

----- Original Message -----
> From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
> To: "Cleber Rosa" <crosa@redhat.com>, qemu-devel@nongnu.org
> Cc: "Thomas Huth" <thuth@redhat.com>, "Beraldo Leal" <bleal@redhat.com>, "Wainer dos Santos Moschetta"
> <wainersm@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, "Willian Rampazzo" <wrampazz@redhat.com>, "Eduardo
> Habkost" <ehabkost@redhat.com>
> Sent: Sunday, May 2, 2021 11:24:44 AM
> Subject: Re: [PATCH 0/1] Acceptance Tests: bump Avocado version requirement to 87.0
> 
> On 4/14/21 6:11 PM, Cleber Rosa wrote:
> > This is being proposed as a separate single patch simply to show
> > that no known regressions have been introduced as far as the
> > acceptance tests/jobs are related.  CI job:
> > 
> >    https://gitlab.com/cleber.gnu/qemu/-/pipelines/286347312
> > 
> > This version (and 86.0) contain improvements that address specific
> > QEMU use cases, including:
> > 
> >  * Fix to the error message given when downloading assets
> > 
> >  * Asset listing/purging capabilities
> > 
> > Cleber Rosa (1):
> >   Acceptance Tests: bump Avocado version requirement to 87.0
> > 
> >  tests/requirements.txt | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> 
> Ping? This patch should fix the mainstream pipeline error:
> https://gitlab.com/qemu-project/qemu/-/jobs/1229752162
> 

Hi Phil,

I'll send with my python-next (and other) patches to be queued in a day
or two... but the bad news is: I don't think Avocado 87.0 has the fix for
the pipeline issue above.  I looked at the test log, and the kernel boot
hangs right after:

11:43:53 DEBUG| Freeing unused kernel memory: 1176K
11:43:53 DEBUG| This architecture does not have kernel memory protection.

And then gets interrupted by Avocado some 90 seconds after that.  I'll try
to reproduce it and debug.

> (see
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg794416.html)
> 
> 

Avocado 87.0 does fix the issue you referenced here though.

Regards,
- Cleber.

Re: [PATCH 0/1] Acceptance Tests: bump Avocado version requirement to 87.0

[Philippe Mathieu-Daudé]

On 5/3/21 3:50 AM, Cleber Rosa wrote:
> ----- Original Message -----
>> From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
>> On 4/14/21 6:11 PM, Cleber Rosa wrote:
>>> This is being proposed as a separate single patch simply to show
>>> that no known regressions have been introduced as far as the
>>> acceptance tests/jobs are related.  CI job:
>>>
>>>    https://gitlab.com/cleber.gnu/qemu/-/pipelines/286347312
>>>
>>> This version (and 86.0) contain improvements that address specific
>>> QEMU use cases, including:
>>>
>>>  * Fix to the error message given when downloading assets
>>>
>>>  * Asset listing/purging capabilities
>>>
>>> Cleber Rosa (1):
>>>   Acceptance Tests: bump Avocado version requirement to 87.0
>>>
>>>  tests/requirements.txt | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>
>> Ping? This patch should fix the mainstream pipeline error:
>> https://gitlab.com/qemu-project/qemu/-/jobs/1229752162
>>
> 
> Hi Phil,
> 
> I'll send with my python-next (and other) patches to be queued in a day
> or two... but the bad news is: I don't think Avocado 87.0 has the fix for
> the pipeline issue above.  I looked at the test log, and the kernel boot
> hangs right after:
> 
> 11:43:53 DEBUG| Freeing unused kernel memory: 1176K
> 11:43:53 DEBUG| This architecture does not have kernel memory protection.
> 
> And then gets interrupted by Avocado some 90 seconds after that.  I'll try
> to reproduce it and debug.

Oh, there was 2 errors... I just looked at the first one. Thanks for
being cautious with the test logs :)

Thomas / Alex, what do you suggest we do with the test_or1k_sim
timeouting? This is not the first time:
https://www.mail-archive.com/qemu-devel@nongnu.org/msg781695.html

> 
>> (see
>> https://www.mail-archive.com/qemu-devel@nongnu.org/msg794416.html)
>>
>>
> 
> Avocado 87.0 does fix the issue you referenced here though.
> 
> Regards,
> - Cleber.
>