| *** openstack has joined #zuul | 20:01 | |
| *** SotK has quit IRC | 20:01 | |
| *** mmedvede has joined #zuul | 20:04 | |
| auggy | looks like jeblair 's talk from summit is uploaded! https://www.youtube.com/watch?v=R4EmE1QEvNU | 20:07 |
|---|---|---|
| cinerama | oh great | 20:08 |
| pabelanger | remote: https://review.openstack.org/392953 Re-enable test_can_merge unit test | 20:10 |
| *** Zara has quit IRC | 20:11 | |
| pabelanger | jeblair: I think ^ is the less painful way to enable test_can_merge. | 20:11 |
| pabelanger | jeblair: I suspect there is some issues with my TriggerEvent I created, maybe I should mock it out better | 20:12 |
| *** hashar has joined #zuul | 20:12 | |
| jeblair | pabelanger: as long as it works, it should be good enough for now, and we can think about better testing of that later as we develop the source/trigger API more | 20:14 |
| pabelanger | jeblair: okay, that works for me | 20:15 |
| *** openstackgerrit has joined #zuul | 20:19 | |
| *** SotK has joined #zuul | 20:29 | |
| *** ianw has joined #zuul | 21:14 | |
| *** ChanServ changes topic to "Discussion of the project gating system Zuul | Docs: http://docs.openstack.org/infra/zuul/ | Source: https://git.openstack.org/cgit/openstack-infra/zuul/ | V3 development: http://git.openstack.org/cgit/openstack-infra/zuul/tree/README.rst?h=feature/zuulv3 | Channel logs: http://eavesdrop.openstack.org/irclogs/%23zuul/" | 21:18 | |
| -jeblair- This channel is now logged at http://eavesdrop.openstack.org/irclogs/%23zuul/ | 21:20 | |
| *** herlo has quit IRC | 21:29 | |
| adam_g | o/ jeblair and everyone. i was poking at enabling test_new_patchset_dequeues_old_without_dependents as it seems like a pretty straight forward test to start with | 21:41 |
| adam_g | ive found the IndependentPipelineManager calling a non-existent method @ http://git.openstack.org/cgit/openstack-infra/zuul/tree/zuul/manager/independent.py?h=feature/zuulv3#n32 | 21:41 |
| adam_g | ... which was removed at https://review.openstack.org/#/c/358009/ | 21:42 |
| adam_g | should the test be using a new config fixture to declare the projects explictly, and the stale code cleaned out? | 21:43 |
| adam_g | or should the IPM be configuring projects dynamically using some new things? | 21:43 |
| jeblair | adam_g: ah, i think pabelanger and i may have worked through this with in https://review.openstack.org/386769 | 21:46 |
| adam_g | oh, cool. /me looks | 21:46 |
| auggy | jeblair: do you know if there is currently an item in storyboard for fixing the zuul v3 tests? | 21:58 |
| auggy | i wanted to add something about grouping them by logical path/feature | 21:59 |
| auggy | i can only find 2 zuulv3 things in storyboard, so that doesn't seem right to me | 21:59 |
| auggy | also how are we currently tracking who is working on what tests? | 22:00 |
| *** harlowja has quit IRC | 22:02 | |
| *** hashar has quit IRC | 22:07 | |
| jeblair | auggy: nope not yet -- SpamapS is starting work on storyboarding that RSN. current anti-collision mechanism for fixing tests is https://etherpad.openstack.org/p/zuulv3 | 22:07 |
| jeblair | adam_g, pabelanger: you might want to add some entries to that etherpad ^ | 22:07 |
| auggy | jeblair: maybe for now, if it's something people check, i could create an interim storyboard ticket with that link? or is there somewhere else you think we should put it that people are more likely to see it? | 22:08 |
| jeblair | cinerama: ^ see note about etherpad | 22:09 |
| adam_g | jeblair: sure | 22:09 |
| jeblair | auggy: we're not really using storyboard properly yet, so i'd prefer to just keep using the etherpad for now, then make a clean break once we start using it for real | 22:10 |
| cinerama | auggy: that's kind of a good question. when i looked at tests last it seems like a lot of the tests that want to be reenabled depend on having the new layout stuff in place | 22:10 |
| cinerama | (please correct me if i'm wrong here) | 22:10 |
| auggy | ah yeah that's probably good stuff to know, what tests have other dependencies | 22:12 |
| jeblair | cinerama: yeah, i think your change (388266) is one of those things that opened up a potential new series of tasks. there's a bunch of configuration file stuff that has changed and not a clear path for what the layoutvalidator (or it's successor in interest) should actually do. i've put it on my list of things that we should make a story about. | 22:13 |
| cinerama | jeblair: yup that was my next question - "so layouts are different, validation is different - what now?" | 22:14 |
| cinerama | and updateConfigLayout has to be different too | 22:15 |
| auggy | jeblair: sounds like we might need a story-a-thon hehe | 22:16 |
| jeblair | cinerama: exactly. and i don't have the answer right now, but i'm excited about discovering the question. :) | 22:16 |
| jeblair | auggy: yes, i think next week may be beginning to look like that :) | 22:17 |
| auggy | jeblair: well if there's anything you'd like folks to help with, at a minimum we could just create stub tickets with a link to the gerrit reviews in question or something | 22:19 |
| auggy | and also update the gerrit reviews with a comment linking to the storyboard ticket if that's a thing that might be necessary | 22:19 |
| auggy | but sounds like something that given an etherpad a few hands could at least start and then some clean up could happen later or some such | 22:20 |
| cinerama | do we have any other etherpads we should link up to that one? | 22:21 |
| jeblair | auggy: i don't want to step on SpamapS's toes, so let's see how he wants to coordinate the storyboardification | 22:22 |
| jeblair | cinerama: that's it | 22:22 |
| cinerama | good | 22:24 |
| jeblair | yeah, i mean, one etherpad is probably already too many. :) | 22:25 |
| pabelanger | jeblair: adam_g: sure, I'll add my reviews | 22:34 |
| SpamapS | I plan to start boarding tomorrow morning | 22:41 |
| SpamapS | Hopefully will get them all in one shot | 22:41 |
| pabelanger | okay, etherpad updated with current patchsets | 22:42 |
| openstackgerrit | Paul Belanger proposed openstack-infra/zuul: Re-enable test_needed_changes_enqueue testing https://review.openstack.org/393001 | 23:06 |
| pabelanger | yay | 23:07 |
| pabelanger | openstackgerrit: o/ | 23:07 |
| pabelanger | and now, I rest for the night | 23:07 |
| *** harlowja has joined #zuul | 23:08 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!