Thanks, Paul.

For me both have their strength and weaknesses:

* If you want to add a new tox job for an existing target like pep8, I
like how simple that is with
https://review.openstack.org/#/c/438281/25/.zuul.yaml

* On the other hand, if you add a new tox job - for a completely new
target, then https://review.openstack.org/#/c/442180/3/.zuul.yaml allows
me to add the job in a single place instead of adding another file like
https://review.openstack.org/#/c/438281/25/playbooks/tox-cover.yaml.

Knowing that most projects do not add their own non-standard jobs, the
more compact setting like 438281 wins for me - and the other way is not
too difficult either.

So, I'm fine with either solution and have a slight preference to 438281.

I fear only time will tell, whether there's really a significant
difference ;)

Andreas
-- 
 Andreas Jaeger aj@{suse.com,opensuse.org} Twitter: jaegerandi
  SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
   GF: Felix Imendörffer, Jane Smithard, Graham Norton,
       HRB 21284 (AG Nürnberg)
    GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126


_______________________________________________
OpenStack-Infra mailing list
[email protected]
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-infra

Reply via email to