| Message ID | 20201108204535.2319870-16-philmd@redhat.com |
|---|---|
| State | New |
| Headers | show |
| Series | [PATCH-for-5.2,v4,01/17] configure: Check vhost-user is available to build vhost-user-blk-server | expand |
On Sun, Nov 08, 2020 at 09:45:33PM +0100, Philippe Mathieu-Daudé wrote: > Xen packages are available in CentOS 7, but have been > removed from CentOS 8. Use the CentOS 7 container. Technically Xen has never been in CentOS 8, I'm working on it, slowly. > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > --- > Cc: Eduardo Habkost <ehabkost@redhat.com> > Cc: Stefano Stabellini <sstabellini@kernel.org> > Cc: Anthony Perard <anthony.perard@citrix.com> > Cc: Paul Durrant <paul@xen.org> > Cc: xen-devel@lists.xenproject.org > --- > .gitlab-ci.yml | 21 +++++++++++++++++++++ > 1 file changed, 21 insertions(+) > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > index 2f0da7b3dc1..8e15266c277 100644 > --- a/.gitlab-ci.yml > +++ b/.gitlab-ci.yml > @@ -557,6 +557,27 @@ check-crypto-only-gnutls: > IMAGE: centos7 > MAKE_CHECK_ARGS: check > > +build-xen-centos: > + <<: *native_build_job_definition > + variables: > + IMAGE: centos7 > + TARGETS: i386-softmmu x86_64-softmmu > + CONFIGURE_ARGS: --enable-xen > + MAKE_CHECK_ARGS: check-build > + artifacts: > + paths: > + - build > + > +check-xen-centos: > + <<: *native_test_job_definition > + needs: > + - job: build-xen-centos > + artifacts: true > + variables: > + IMAGE: centos7 > + MAKE_CHECK_ARGS: check Is `make check` going to do something useful with the Xen support? Or is it going to need more work in order to test the Xen support of QEMU? (Like starting an actual Xen guest.) Cheers, -- Anthony PERARD
On Thu, Nov 26, 2020 at 05:38:24PM +0000, Anthony PERARD wrote: > On Sun, Nov 08, 2020 at 09:45:33PM +0100, Philippe Mathieu-Daudé wrote: > > Xen packages are available in CentOS 7, but have been > > removed from CentOS 8. Use the CentOS 7 container. > > Technically Xen has never been in CentOS 8, I'm working on it, slowly. > > > Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> > > --- > > Cc: Eduardo Habkost <ehabkost@redhat.com> > > Cc: Stefano Stabellini <sstabellini@kernel.org> > > Cc: Anthony Perard <anthony.perard@citrix.com> > > Cc: Paul Durrant <paul@xen.org> > > Cc: xen-devel@lists.xenproject.org > > --- > > .gitlab-ci.yml | 21 +++++++++++++++++++++ > > 1 file changed, 21 insertions(+) > > > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml > > index 2f0da7b3dc1..8e15266c277 100644 > > --- a/.gitlab-ci.yml > > +++ b/.gitlab-ci.yml > > @@ -557,6 +557,27 @@ check-crypto-only-gnutls: > > IMAGE: centos7 > > MAKE_CHECK_ARGS: check > > > > +build-xen-centos: > > + <<: *native_build_job_definition > > + variables: > > + IMAGE: centos7 > > + TARGETS: i386-softmmu x86_64-softmmu > > + CONFIGURE_ARGS: --enable-xen > > + MAKE_CHECK_ARGS: check-build > > + artifacts: > > + paths: > > + - build > > + > > +check-xen-centos: > > + <<: *native_test_job_definition > > + needs: > > + - job: build-xen-centos > > + artifacts: true > > + variables: > > + IMAGE: centos7 > > + MAKE_CHECK_ARGS: check > > Is `make check` going to do something useful with the Xen support? Or is > it going to need more work in order to test the Xen support of QEMU? > (Like starting an actual Xen guest.) I don't think it will test Xen support, but we still want to at least check if --enable-xen doesn't break anything else. Is there any public CI system anywhere where Xen support is tested today? -- Eduardo
On Thu, Nov 26, 2020 at 12:45:59PM -0500, Eduardo Habkost wrote: > On Thu, Nov 26, 2020 at 05:38:24PM +0000, Anthony PERARD wrote: > > Is `make check` going to do something useful with the Xen support? Or is > > it going to need more work in order to test the Xen support of QEMU? > > (Like starting an actual Xen guest.) > > I don't think it will test Xen support, but we still want to at > least check if --enable-xen doesn't break anything else. That sound good. > Is there any public CI system anywhere where Xen support is > tested today? Yes, we have osstest which regularly test Xen with QEMU from upstream. Result are sent to xen-devel. But that might not be very useful for qemu-devel. We also have a GitLab CI which does some Xen tests, but I don't think QEMU is tested there. https://xenbits.xen.org/gitweb/?p=xen.git;a=blob;f=automation/gitlab-ci/test.yaml;hb=HEAD https://gitlab.com/xen-project/xen/ Cheers, -- Anthony PERARD
On Fri, 27 Nov 2020, Anthony PERARD wrote: > On Thu, Nov 26, 2020 at 12:45:59PM -0500, Eduardo Habkost wrote: > > On Thu, Nov 26, 2020 at 05:38:24PM +0000, Anthony PERARD wrote: > > > Is `make check` going to do something useful with the Xen support? Or is > > > it going to need more work in order to test the Xen support of QEMU? > > > (Like starting an actual Xen guest.) > > > > I don't think it will test Xen support, but we still want to at > > least check if --enable-xen doesn't break anything else. > > That sound good. > > > Is there any public CI system anywhere where Xen support is > > tested today? > > Yes, we have osstest which regularly test Xen with QEMU from upstream. > Result are sent to xen-devel. But that might not be very useful for > qemu-devel. > > We also have a GitLab CI which does some Xen tests, but I don't think > QEMU is tested there. > https://xenbits.xen.org/gitweb/?p=xen.git;a=blob;f=automation/gitlab-ci/test.yaml;hb=HEAD > https://gitlab.com/xen-project/xen/ QEMU (the version of QEMU picked by the Xen tools) is built but not used in the Xen Project CI-loop yet. I am extending the CI-loop with more tests [1], and I would like to have at least one QEMU test at some point soon. Probably something based on Xen 9pfs. [1] https://marc.info/?l=xen-devel&m=160627845825763
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index 2f0da7b3dc1..8e15266c277 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -557,6 +557,27 @@ check-crypto-only-gnutls: IMAGE: centos7 MAKE_CHECK_ARGS: check +build-xen-centos: + <<: *native_build_job_definition + variables: + IMAGE: centos7 + TARGETS: i386-softmmu x86_64-softmmu + CONFIGURE_ARGS: --enable-xen + MAKE_CHECK_ARGS: check-build + artifacts: + paths: + - build + +check-xen-centos: + <<: *native_test_job_definition + needs: + - job: build-xen-centos + artifacts: true + variables: + IMAGE: centos7 + MAKE_CHECK_ARGS: check + + # We don't need to exercise every backend with every front-end build-trace-multi-user: <<: *native_build_job_definition
Xen packages are available in CentOS 7, but have been removed from CentOS 8. Use the CentOS 7 container. Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com> --- Cc: Eduardo Habkost <ehabkost@redhat.com> Cc: Stefano Stabellini <sstabellini@kernel.org> Cc: Anthony Perard <anthony.perard@citrix.com> Cc: Paul Durrant <paul@xen.org> Cc: xen-devel@lists.xenproject.org --- .gitlab-ci.yml | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+)