[PATCH 0/6] tests/docker: Update Fedora containers

John Snow posted 6 patches 4 years, 3 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211103144844.1285634-1-jsnow@redhat.com
Maintainers: Wainer dos Santos Moschetta <wainersm@redhat.com>, Willian Rampazzo <willianr@redhat.com>, "Philippe Mathieu-Daudé" <f4bug@amsat.org>, "Alex Bennée" <alex.bennee@linaro.org>, Thomas Huth <thuth@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>
include/ui/qemu-spice.h                       |   6 +
hw/display/qxl.c                              |  14 ++-
ui/clipboard.c                                |   3 +-
ui/spice-display.c                            |  11 ++
.gitlab-ci.d/buildtest.yml                    |  16 +++
.gitlab-ci.d/container-core.yml               |   5 +
.../dockerfiles/fedora-cris-cross.docker      |   2 +-
.../dockerfiles/fedora-win32-cross.docker     |   2 +-
.../dockerfiles/fedora-win64-cross.docker     |   2 +-
tests/docker/dockerfiles/fedora.docker        |   2 +-
tests/docker/dockerfiles/fedora34.docker      | 117 ++++++++++++++++++
tests/docker/dockerfiles/python.docker        |   2 +-
12 files changed, 175 insertions(+), 7 deletions(-)
create mode 100644 tests/docker/dockerfiles/fedora34.docker
[PATCH 0/6] tests/docker: Update Fedora containers
Posted by John Snow 4 years, 3 months ago
Fedora 33 will be EOL before 6.2 ships; Fedora 35 was just
released. Update our various containers to test on F34 and F35.

Fix a minor code warning issue that surfaces in a new version of Clang,
and fix a small deprecation issue for the latest version of spice.

In testing, I found that oss-fuzz was more likely to time out on GitLab;
it's unclear if this is a performance regression or just getting unlucky
with when I test.

Summary:
  fedora.docker updated to F34 and then F35
  fedora34.docker copied from fedora.docker, new build test added
  cross-compile targets bumped from F33 to F34

John Snow (6):
  spice: Update QXLInterface for spice >= 0.15.0
  ui/clipboard: Don't use g_autoptr just to free a variable
  docker: update fedora container to Fedora 34
  docker: update Fedora-based cross-compiler containers to Fedora 34
  docker: update 'python' dockerfile to use Fedora registry
  docker: Add Fedora 35 container

 include/ui/qemu-spice.h                       |   6 +
 hw/display/qxl.c                              |  14 ++-
 ui/clipboard.c                                |   3 +-
 ui/spice-display.c                            |  11 ++
 .gitlab-ci.d/buildtest.yml                    |  16 +++
 .gitlab-ci.d/container-core.yml               |   5 +
 .../dockerfiles/fedora-cris-cross.docker      |   2 +-
 .../dockerfiles/fedora-win32-cross.docker     |   2 +-
 .../dockerfiles/fedora-win64-cross.docker     |   2 +-
 tests/docker/dockerfiles/fedora.docker        |   2 +-
 tests/docker/dockerfiles/fedora34.docker      | 117 ++++++++++++++++++
 tests/docker/dockerfiles/python.docker        |   2 +-
 12 files changed, 175 insertions(+), 7 deletions(-)
 create mode 100644 tests/docker/dockerfiles/fedora34.docker

-- 
2.31.1



Re: [PATCH 0/6] tests/docker: Update Fedora containers
Posted by Daniel P. Berrangé 4 years, 2 months ago
On Wed, Nov 03, 2021 at 10:48:38AM -0400, John Snow wrote:
> Fedora 33 will be EOL before 6.2 ships; Fedora 35 was just
> released. Update our various containers to test on F34 and F35.
> 
> Fix a minor code warning issue that surfaces in a new version of Clang,
> and fix a small deprecation issue for the latest version of spice.
> 
> In testing, I found that oss-fuzz was more likely to time out on GitLab;
> it's unclear if this is a performance regression or just getting unlucky
> with when I test.

For me it times out every time on Fedora 35. In fact if I run it
locally, it doesnn't even finish after 24 hours ! There's a real
bug in there somewhere.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|


Re: [PATCH 0/6] tests/docker: Update Fedora containers
Posted by Philippe Mathieu-Daudé 4 years, 2 months ago
Cc'ing Alex.

On 11/24/21 13:19, Daniel P. Berrangé wrote:
> On Wed, Nov 03, 2021 at 10:48:38AM -0400, John Snow wrote:
>> Fedora 33 will be EOL before 6.2 ships; Fedora 35 was just
>> released. Update our various containers to test on F34 and F35.
>>
>> Fix a minor code warning issue that surfaces in a new version of Clang,
>> and fix a small deprecation issue for the latest version of spice.
>>
>> In testing, I found that oss-fuzz was more likely to time out on GitLab;
>> it's unclear if this is a performance regression or just getting unlucky
>> with when I test.
> 
> For me it times out every time on Fedora 35. In fact if I run it
> locally, it doesnn't even finish after 24 hours ! There's a real
> bug in there somewhere.
> 
> 
> Regards,
> Daniel
> 


Re: [PATCH 0/6] tests/docker: Update Fedora containers
Posted by Alex Bennée 4 years, 2 months ago
Philippe Mathieu-Daudé <philmd@redhat.com> writes:

> Cc'ing Alex.
>
> On 11/24/21 13:19, Daniel P. Berrangé wrote:
>> On Wed, Nov 03, 2021 at 10:48:38AM -0400, John Snow wrote:
>>> Fedora 33 will be EOL before 6.2 ships; Fedora 35 was just
>>> released. Update our various containers to test on F34 and F35.
>>>
>>> Fix a minor code warning issue that surfaces in a new version of Clang,
>>> and fix a small deprecation issue for the latest version of spice.
>>>
>>> In testing, I found that oss-fuzz was more likely to time out on GitLab;
>>> it's unclear if this is a performance regression or just getting unlucky
>>> with when I test.
>> 
>> For me it times out every time on Fedora 35. In fact if I run it
>> locally, it doesnn't even finish after 24 hours ! There's a real
>> bug in there somewhere.

Is there an other spin of this series coming? I saw there were some
comments to address so I didn't look in detail myself.

-- 
Alex Bennée

Re: [PATCH 0/6] tests/docker: Update Fedora containers
Posted by Philippe Mathieu-Daudé 4 years, 2 months ago
On 11/24/21 13:51, Alex Bennée wrote:
> Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> 
>> Cc'ing Alex.
>>
>> On 11/24/21 13:19, Daniel P. Berrangé wrote:
>>> On Wed, Nov 03, 2021 at 10:48:38AM -0400, John Snow wrote:
>>>> Fedora 33 will be EOL before 6.2 ships; Fedora 35 was just
>>>> released. Update our various containers to test on F34 and F35.
>>>>
>>>> Fix a minor code warning issue that surfaces in a new version of Clang,
>>>> and fix a small deprecation issue for the latest version of spice.
>>>>
>>>> In testing, I found that oss-fuzz was more likely to time out on GitLab;
>>>> it's unclear if this is a performance regression or just getting unlucky
>>>> with when I test.
>>>
>>> For me it times out every time on Fedora 35. In fact if I run it
>>> locally, it doesnn't even finish after 24 hours ! There's a real
>>> bug in there somewhere.
> 
> Is there an other spin of this series coming? I saw there were some
> comments to address so I didn't look in detail myself.

I meant Alexander Bulekov for oss-fuzz :)