On Wed, Sep 28, 2016, at 07:15 AM, Sean Dague wrote: > On 09/27/2016 06:19 PM, Andrew Laski wrote: > <snip> > > > > I totally understand where you're coming from. I just see it > > differently. > > > > The way it was done did not affect any other projects, and the plumbing > > used is something that could have easily be left in. And as luck would > > have it there's another patch up to use that same plumbing so it's > > probably going in anyways. > > > > I completely agree that before expanding this to be in any way cross > > project it's worth figuring out a better way to do it. But at this point > > I don't feel comfortable enough with a long term vision to tackle that. > > I would much prefer to experiment in a small way before moving forward. > > Ok, so what if we go forward with your existing patch, but agree only to > replace the post_test_hook on Nova specific jobs, like the placement > one. (The change proposed here - > https://review.openstack.org/#/c/376537/6/jenkins/jobs/nova.yaml). That > should give enough data and experimentation, and it show up for every > nova patch. > > Before migrating this kind of approach to integrated gate jobs, we > revisit a way to make sure that multiple projects can plumb content like > this. Be it, expanding the test hook model, or a common post hook > project that could have core members from multiple teams, or some > generic yaml thing (though I agree, that's a lot harder to wrap my head > around).
Deal. > > -Sean > > -- > Sean Dague > http://dague.net > > __________________________________________________________________________ > OpenStack Development Mailing List (not for usage questions) > Unsubscribe: > openstack-dev-requ...@lists.openstack.org?subject:unsubscribe > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev __________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
