[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [OSSTEST PATCH v7 3/3] Create a flight to test OpenStack with xen-unstable and libvirt



On Fri, Nov 18, 2016 at 06:57:52PM +0000, Ian Jackson wrote:
> Anthony PERARD writes ("[Xen-devel] [OSSTEST PATCH v7 3/3] Create a flight to 
> test OpenStack with xen-unstable and libvirt"):
> > This patch should create a flight "openstack-nova", with those jobs:
> 
> Did I already ask for a diff of the output of
>    standalone-generate-dump-flight-runvars
> before and after ?

You asked that on my patch set :-)

Perhaps it should be part of the README.devel?

> 
> You probably want
>   AP_FETCH_PLACEHOLDERS=y eatmydata ./standalone-generate-dump-flight-runvars
> 
> Your new shell code is mostly good.  I have a style comment:
> 
> > +define_openstack_trees() {
> > +    local openstack_trees=(cinder glance keystone nova requirements 
> > tempest)
> 
> The use of an array variable here is not necessary since none of these
> contain spaces.  I don't object, except that:
> 
> > +    for tree in ${openstack_trees[@]}; do
> 
> This sets off my style alarm because of the lack of "" around a
> ${something[@]} (which off course doesn't matter here.
> 
> Can you either add the " ", or switch to a plain (space-separated)
> variable.
> 
> > +    for tree in ${openstack_trees[@]}; do
> 
> Here too.
> 
> Aside from that, and assuming the runvars diff is good, this is
> looking good.

I concur. I used his patchset to grok the OSSTest.
> 
> Ian.
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> https://lists.xen.org/xen-devel

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.