[openstack-dev] [placement] [infra] [qa] tuning some zuul jobs from "it works" to "proper"

Luigi Toscano ltoscano at redhat.com
Thu Sep 20 08:55:31 UTC 2018


On Thursday, 20 September 2018 04:47:20 CEST Matthew Treinish wrote:
> On Thu, Sep 20, 2018 at 11:11:12AM +0900, Ghanshyam Mann wrote:
> >  ---- On Wed, 19 Sep 2018 23:29:46 +0900 Monty Taylor
> >  <mordred at inaugust.com> wrote ---->  
> >  > On 09/19/2018 09:23 AM, Monty Taylor wrote:
> >  > > On 09/19/2018 08:25 AM, Chris Dent wrote:
> >  > >> I have a patch in progress to add some simple integration tests to
> >  > >> 
> >  > >> placement:
> >  > >>      https://review.openstack.org/#/c/601614/
> >  > >> 
> >  > >> They use https://github.com/cdent/gabbi-tempest . The idea is that
> >  > >> the method for adding more tests is to simply add more yaml in
> >  > >> gate/gabbits, without needing to worry about adding to or think
> >  > >> about tempest.
> >  > >> 
> >  > >> What I have at that patch works; there are two yaml files, one of
> >  > >> which goes through the process of confirming the existence of a
> >  > >> resource provider and inventory, booting a server, seeing a change
> >  > >> in allocations, resizing the server, seeing a change in allocations.
> >  > >> 
> >  > >> But this is kludgy in a variety of ways and I'm hoping to get some
> >  > >> help or pointers to the right way. I'm posting here instead of
> >  > >> asking in IRC as I assume other people confront these same
> >  > >> confusions. The issues:
> >  > >> 
> >  > >> * The associated playbooks are cargo-culted from stuff labelled
> >  > >> 
> >  > >>    "legacy" that I was able to find in nova's jobs. I get the
> >  > >>    impression that these are more verbose and duplicative than they
> >  > >>    need to be and are not aligned with modern zuul v3 coolness.
> >  > > 
> >  > > Yes. Your life will be much better if you do not make more legacy
> >  > > jobs.
> >  > > They are brittle and hard to work with.
> >  > > 
> >  > > New jobs should either use the devstack base job, the
> >  > > devstack-tempest
> >  > > base job or the devstack-tox-functional base job - depending on what
> >  > > things are intended.
> > 
> > +1. All the base job from Tempest and Devstack (except grenade which is in
> > progress) are available to use as base for legacy jobs. Using
> > devstack-temepst in your patch is right things. In addition, you need to
> > mention the tox_envlist as all-plugins to make tempest_test_regex work. I
> > commented on review.
> No, all-plugins is incorrect and should never be used. It's only there for
> legacy support, it is deprecated and I thought we pushed a patch to
> indicating that (but I can't find it).

This one?
https://review.openstack.org/#/c/543974/

Ciao
-- 
Luigi





More information about the OpenStack-dev mailing list