Re: [Libosinfo] [libosinfo PATCH] tree: Add "os" property

2019-07-17 Thread Fabiano Fidêncio
ping! On Fri, Jul 5, 2019 at 9:31 AM Fabiano Fidêncio wrote: > > Similarly to what we have for media, let's add the "os" property to the > trees. > > osinfo-detect code is already relying on having an "os" property and, > since commit 722e7d1d4, it errors out due to the missing property: > > (osi

[Libosinfo] [libosinfo PATCH] tools, detect: Check for NULL variants

2019-07-17 Thread Fabiano Fidêncio
Both trees & medias may return NULL in case no os-variant was found. Let's check for this and avoid a segmentation fault on calling osinfo_list_get_length() with a NULL argument. Signed-off-by: Fabiano Fidêncio --- tools/osinfo-detect.c | 12 1 file changed, 8 insertions(+), 4 delet

[Libosinfo] [libosinfo PATCH] tree: Return after calling g_task_return_error()

2019-07-17 Thread Fabiano Fidêncio
Otherwise we'll end up reaching: g_task_return_error: assertion 'task->ever_returned' failed. Signed-off-by: Fabiano Fidêncio --- This patch fixes the CI breakage with virt-manager and will be pushed soon. --- osinfo/osinfo_tree.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --g

Re: [Libosinfo] [libosinfo PATCH] tree: Return after calling g_task_return_error()

2019-07-17 Thread Pino Toscano
On Wednesday, 17 July 2019 12:18:26 CEST Fabiano Fidêncio wrote: > Otherwise we'll end up reaching: > g_task_return_error: assertion 'task->ever_returned' failed. > > Signed-off-by: Fabiano Fidêncio > --- > This patch fixes the CI breakage with virt-manager and will be pushed > soon. > --- > osi

[Libosinfo] [libosinfo PATCH v2 0/3] CI breakage fixes

2019-07-17 Thread Fabiano Fidêncio
Those patches (partially) the breakage caused on virt-manager's tests caught by libvirt-jenkins-ci. There's still an issue happening, which I'm investigating, that I'm getting a SIGTERM when running testCLI0450virt_convert It also seems to be related to the latest tree libsoup related patches on

[Libosinfo] [libosinfo PATCH v2 3/3] tree: cleanup on any error & return on _create_from_location_async_helper()

2019-07-17 Thread Fabiano Fidêncio
When calling a g_task_return_error(), we should also cleanup the CreateFromLocationAsyncData and return immediately after that. By doing so we avoid errors as: g_task_return_error: assertion 'task->ever_returned' failed. Signed-off-by: Fabiano Fidêncio --- osinfo/osinfo_tree.c | 2 ++ 1 file ch

[Libosinfo] [libosinfo PATCH v2 2/3] tree: goto cleanup on any error, on on_content_read()

2019-07-17 Thread Fabiano Fidêncio
Instead of just free'ing the CreateFromLocationAsyncData, let's just go to cleanup, which will call create_from_location_async_data_free() and return after that. By doing so we avoid errors as: g_task_return_error: assertion 'task->ever_returned' failed. Signed-off-by: Fabiano Fidêncio --- osin

[Libosinfo] [libosinfo PATCH v2 1/3] tree: Use g_clear_object in create_from_location_async_data_free()

2019-07-17 Thread Fabiano Fidêncio
We should check whether the objects are NULL or not before calling g_object_unref(). Instead of doing so, let's use g_clear_object(), which doesn't require the check. Signed-off-by: Fabiano Fidêncio --- osinfo/osinfo_tree.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [Libosinfo] [libosinfo PATCH v2 1/3] tree: Use g_clear_object in create_from_location_async_data_free()

2019-07-17 Thread Daniel P . Berrangé
On Wed, Jul 17, 2019 at 02:24:48PM +0200, Fabiano Fidêncio wrote: > We should check whether the objects are NULL or not before calling > g_object_unref(). Instead of doing so, let's use g_clear_object(), which > doesn't require the check. > > Signed-off-by: Fabiano Fidêncio > --- > osinfo/osinfo

Re: [Libosinfo] [libosinfo PATCH v2 2/3] tree: goto cleanup on any error, on on_content_read()

2019-07-17 Thread Daniel P . Berrangé
On Wed, Jul 17, 2019 at 02:24:49PM +0200, Fabiano Fidêncio wrote: > Instead of just free'ing the CreateFromLocationAsyncData, let's just go > to cleanup, which will call create_from_location_async_data_free() and > return after that. > > By doing so we avoid errors as: > g_task_return_error: asser

Re: [Libosinfo] [libosinfo PATCH v2 3/3] tree: cleanup on any error & return on _create_from_location_async_helper()

2019-07-17 Thread Daniel P . Berrangé
On Wed, Jul 17, 2019 at 02:24:50PM +0200, Fabiano Fidêncio wrote: > When calling a g_task_return_error(), we should also cleanup the > CreateFromLocationAsyncData and return immediately after that. > > By doing so we avoid errors as: > g_task_return_error: assertion 'task->ever_returned' failed. >

Re: [Libosinfo] [libosinfo PATCH v2 0/3] CI breakage fixes

2019-07-17 Thread Daniel P . Berrangé
On Wed, Jul 17, 2019 at 02:24:47PM +0200, Fabiano Fidêncio wrote: > Those patches (partially) the breakage caused on virt-manager's tests > caught by libvirt-jenkins-ci. Wonder if there's anything we can add to libosinfo tests tat would catchthis sooner > > There's still an issue happening, whi

[Libosinfo] [PATCH osinfo-db 1/2] Add database support for SLE15-SP1

2019-07-17 Thread Charles Arnold
--- data/os/suse.com/sle-15.1.xml.in | 65 1 file changed, 65 insertions(+) create mode 100644 data/os/suse.com/sle-15.1.xml.in diff --git a/data/os/suse.com/sle-15.1.xml.in b/data/os/suse.com/sle-15.1.xml.in new file mode 100644 index 000..649f2bf --

[Libosinfo] [PATCH osinfo-db 0/2] Add support for SLE15-SP1

2019-07-17 Thread Charles Arnold
Charles Arnold (2): Add database support for SLE15-SP1 Add SLE15-SP1 ISO Information data/os/suse.com/sle-15.1.xml.in | 65 ++ ...LE-15-SP1-Installer-DVD-aarch64-GM-DVD1.iso.txt | 29 ++ ...LE-15-SP1-Installer-DVD-ppc64le-GM-DVD1.iso.txt | 16 ++

[Libosinfo] [PATCH osinfo-db 2/2] Add SLE15-SP1 ISO Information

2019-07-17 Thread Charles Arnold
--- ...LE-15-SP1-Installer-DVD-aarch64-GM-DVD1.iso.txt | 29 ++ ...LE-15-SP1-Installer-DVD-ppc64le-GM-DVD1.iso.txt | 16 .../SLE-15-SP1-Installer-DVD-s390x-GM-DVD1.iso.txt | 29 ++ ...SLE-15-SP1-Installer-DVD-x86_64-GM-DVD1.iso.txt | 29

Re: [Libosinfo] [libosinfo PATCH v2 0/3] CI breakage fixes

2019-07-17 Thread Cole Robinson
On 7/17/19 8:24 AM, Fabiano Fidêncio wrote: > Those patches (partially) the breakage caused on virt-manager's tests > caught by libvirt-jenkins-ci. > > There's still an issue happening, which I'm investigating, that I'm > getting a SIGTERM when running testCLI0450virt_convert > > It also seems to

Re: [Libosinfo] [libosinfo PATCH] tree: Add "os" property

2019-07-17 Thread Cole Robinson
On 7/17/19 3:00 AM, Fabiano Fidêncio wrote: > ping! > > On Fri, Jul 5, 2019 at 9:31 AM Fabiano Fidêncio wrote: >> >> Similarly to what we have for media, let's add the "os" property to the >> trees. >> >> osinfo-detect code is already relying on having an "os" property and, >> since commit 722e7d

Re: [Libosinfo] [libosinfo PATCH] tools, detect: Check for NULL variants

2019-07-17 Thread Cole Robinson
On 7/17/19 6:01 AM, Fabiano Fidêncio wrote: > Both trees & medias may return NULL in case no os-variant was found. > Let's check for this and avoid a segmentation fault on calling > osinfo_list_get_length() with a NULL argument. > > Signed-off-by: Fabiano Fidêncio Reviewed-by: Cole Robinson -