Re: [Libosinfo] [PATCH] tests: add one test case per OS tested

2019-02-04 Thread Fabiano Fidêncio
On Mon, Feb 4, 2019 at 12:18 PM Daniel P. Berrangé wrote: > > The image, media & tree URIs test suites just register one test case for > all the work they do. As a result when they run they just sit there > printing no progress information for a long time. > > This changes then to register one

Re: [Libosinfo] [libosinfo PATCH 1/2] tests: Avoid re-using the curl handler on URIs tests

2019-02-04 Thread Daniel P . Berrangé
On Mon, Feb 04, 2019 at 02:44:25PM +0100, Fabiano Fidêncio wrote: > On CentOS, we've hit an issue that when a curl handler is re-used we end > up receiving malfunctioning responses from NetBSD CDN. > > Dan investigated the issue and suggested that the problem may be a > timing issue happening

Re: [Libosinfo] [libosinfo PATCH 2/2] tests: abort() in case curl misbehaves

2019-02-04 Thread Daniel P . Berrangé
On Mon, Feb 04, 2019 at 02:44:26PM +0100, Fabiano Fidêncio wrote: > Although the issue with curl should be fixed by the previous commit, > let's abort in case it misbehaves, avoiding then to have all the data > printed on stdout. > > Signed-off-by: Fabiano Fidêncio > --- >

[Libosinfo] [osinfo-db PATCH] debian9: Update URL links to 9.7.0

2019-02-04 Thread Fabiano Fidêncio
Signed-off-by: Fabiano Fidêncio --- data/os/debian.org/debian-9.xml.in | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/data/os/debian.org/debian-9.xml.in b/data/os/debian.org/debian-9.xml.in index 5ee944b..2c02a69 100644 ---

[Libosinfo] [libosinfo PATCH 1/2] tests: Avoid re-using the curl handler on URIs tests

2019-02-04 Thread Fabiano Fidêncio
On CentOS, we've hit an issue that when a curl handler is re-used we end up receiving malfunctioning responses from NetBSD CDN. Dan investigated the issue and suggested that the problem may be a timing issue happening only in certain versions of libcurl (as it only hit us when testing on el7).

[Libosinfo] [libosinfo PATCH 2/2] tests: abort() in case curl misbehaves

2019-02-04 Thread Fabiano Fidêncio
Although the issue with curl should be fixed by the previous commit, let's abort in case it misbehaves, avoiding then to have all the data printed on stdout. Signed-off-by: Fabiano Fidêncio --- tests/test-imageuris.c | 9 + tests/test-mediauris.c | 9 + tests/test-treeuris.c |

[Libosinfo] [libosinfo PATCH] tests: Reduce code duplication in URI tests

2019-02-04 Thread Fabiano Fidêncio
From: Fabiano Fidêncio Let's introduce a common file that does the check of the URI for all the URI related tests. By doing this we can avoid code duplication while also avoiding to have all the tests merged together into a single one, which is not so good as limits the developer to run and

[Libosinfo] [PATCH] tests: add one test case per OS tested

2019-02-04 Thread Daniel P . Berrangé
The image, media & tree URIs test suites just register one test case for all the work they do. As a result when they run they just sit there printing no progress information for a long time. This changes then to register one test case per OS in the database. As a result we get progress

Re: [Libosinfo] [PATCH osinfo-db 0/5] Misc os data updates

2019-02-04 Thread Fabiano Fidêncio
On Sun, Feb 3, 2019 at 11:52 PM Cole Robinson wrote: > > Upstream virt-install is now using libosinfo to lookup > kernel/initrd paths foReviewed-by: Fabiano Fidêncio r > iso media, alongside our own legacy > detection logic. In testing this my local test suite exposed > some missing data in

Re: [Libosinfo] [PATCH libosinfo] ubuntu: Add 19.04 ppc64el test data

2019-02-04 Thread Fabiano Fidêncio
On Sun, Feb 3, 2019 at 11:52 PM Cole Robinson wrote: > > Signed-off-by: Cole Robinson Reviewed-by: Fabiano Fidêncio [snip] ___ Libosinfo mailing list Libosinfo@redhat.com https://www.redhat.com/mailman/listinfo/libosinfo

Re: [Libosinfo] [osinfo-db PATCH] debian9: Update URL links to 9.7.0

2019-02-04 Thread Fabiano Fidêncio
On Mon, Feb 4, 2019 at 1:01 PM Fabiano Fidêncio wrote: > > Signed-off-by: Fabiano Fidêncio This one has been pushed under test breakage rule. ___ Libosinfo mailing list Libosinfo@redhat.com https://www.redhat.com/mailman/listinfo/libosinfo

Re: [Libosinfo] [osinfo-db PATCH] debian9: Update URL links to 9.7.0

2019-02-04 Thread Cole Robinson
On 2/4/19 7:00 AM, Fabiano Fidêncio wrote: Signed-off-by: Fabiano Fidêncio --- data/os/debian.org/debian-9.xml.in | 16 1 file changed, 8 insertions(+), 8 deletions(-) Reviewed-by: Cole Robinson - Cole ___ Libosinfo mailing