Re: [OSSTEST PATCH 2/2] Testing cpupools: recipe for it and job definition

[Dario Faggioli <dario.faggioli@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 2260 bytes --]

On Thu, 2014-12-18 at 16:10 +0100, Juergen Gross wrote:
> On 12/18/2014 04:04 PM, Wei Liu wrote:
> > On Thu, Dec 18, 2014 at 04:00:12PM +0100, Juergen Gross wrote:
> >> On 12/18/2014 02:39 PM, Dario Faggioli wrote:
> >>> Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
> >>> ---
> >>>   make-flight |   11 +++++++++++
> >>>   sg-run-job  |    6 ++++++
> >>>   2 files changed, 17 insertions(+)
> >>>
> >>> diff --git a/make-flight b/make-flight
> >>> index a91f256..ce02c89 100755
> >>> --- a/make-flight
> >>> +++ b/make-flight
> >>> @@ -266,6 +266,16 @@ do_credit2_tests () {
> >>>               $debian_runvars all_hostflags=$most_hostflags
> >>>   }
> >>>
> >>> +do_cpupools_tests () {
> >>> +  if [ $xenarch != amd64 -o $dom0arch != amd64 ]; then
> >>> +    return
> >>> +  fi
> >>
> >> Why can't you test cpupools on x86_64?
> >>
> >
> > Don't worry. That's the name used for 64 bit (be it amd or intel) in
> > OSSTest. :-)
> 
Exactly, it comes from the Debian arches names.

> Aargh. Some automatic filter in my brain translated this to arm64 :-)
> 
HeHe :-)

> So: Why can't you test cpupools on ARM?
> 
We certainly can. ARM testing via OSSTest is a still experimental,
AFAIUI.

I don't even know if we have the hardware, yet, as I think the "rack" of
ARM dev board IanC was working on will be setup in the new testing COLO,
rather than in current OSSTest pool of hardware (someone correct me if
I'm wrong).

However, that seems to be taken into account by the fact that, in
make-flight, in test_matrix_do_one(), after only 2 jobs are created (the
basic debian one and a libvirt one) we have this:

  # No further arm tests at the moment
  if [ $dom0arch = armhf ]; then
      return
  fi

So, yes, I guess I can get rid of such filters in my new function, so
that, when ARM maintainers  will disable the safety catch above, a job
will actually be generated.

I'll give it a shot for v2.

Thanks and Regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel