| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Lock nodes when nodepool request is fulfilled https://review.openstack.org/416700 | 00:09 |
|---|---|---|
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Return nodes after use https://review.openstack.org/416739 | 00:09 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Mark nodes as 'in-use' before launching jobs https://review.openstack.org/416737 | 00:09 |
| jeblair | okay, that stack is green now :) | 00:19 |
| *** jamielennox is now known as jamielennox|away | 01:02 | |
| *** jamielennox|away is now known as jamielennox | 01:21 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Cancel/return nodepool requests on job cancel https://review.openstack.org/416795 | 01:30 |
| *** harlowja has quit IRC | 03:05 | |
| *** yolanda has quit IRC | 03:05 | |
| *** jkt has quit IRC | 03:05 | |
| *** gothicmindfood has quit IRC | 03:05 | |
| *** nibalizer has quit IRC | 03:05 | |
| *** jkt has joined #zuul | 03:05 | |
| *** gothicmindfood has joined #zuul | 03:05 | |
| *** yolanda has joined #zuul | 03:05 | |
| *** nibalizer has joined #zuul | 03:05 | |
| *** bhavik1 has joined #zuul | 04:32 | |
| *** jamielennox is now known as jamielennox|away | 05:23 | |
| *** jamielennox|away is now known as jamielennox | 05:30 | |
| *** Cibo_ has joined #zuul | 05:41 | |
| *** openstackgerrit has quit IRC | 07:50 | |
| *** bhavik1 has quit IRC | 07:57 | |
| *** bhavik1 has joined #zuul | 08:42 | |
| *** saneax-_-|AFK is now known as saneax | 08:46 | |
| *** saneax is now known as saneax-_-|AFK | 08:47 | |
| *** saneax-_-|AFK is now known as saneax | 08:50 | |
| *** hashar has joined #zuul | 09:27 | |
| *** saneax is now known as saneax-_-|AFK | 09:43 | |
| *** saneax-_-|AFK is now known as saneax | 09:47 | |
| *** openstack has joined #zuul | 10:17 | |
| *** bhavik1 has quit IRC | 11:20 | |
| *** nigelb has quit IRC | 11:55 | |
| *** ajafo has quit IRC | 11:55 | |
| *** Shrews has quit IRC | 11:55 | |
| *** phschwartz has quit IRC | 11:55 | |
| *** timrc has quit IRC | 11:55 | |
| *** jasondotstar has quit IRC | 11:55 | |
| *** jasondotstar has joined #zuul | 11:55 | |
| *** timrc has joined #zuul | 11:55 | |
| *** phschwartz has joined #zuul | 11:56 | |
| *** nigelb has joined #zuul | 11:56 | |
| *** nigelb has quit IRC | 11:56 | |
| *** nigelb has joined #zuul | 11:56 | |
| *** Shrews has joined #zuul | 11:56 | |
| *** hashar is now known as hasharLunch | 12:10 | |
| *** saneax is now known as saneax-_-|AFK | 12:11 | |
| *** hasharLunch is now known as hashar | 13:09 | |
| *** bhavik1 has joined #zuul | 14:44 | |
| *** bhavik1 has quit IRC | 14:50 | |
| *** hashar has quit IRC | 15:15 | |
| *** hashar has joined #zuul | 15:16 | |
| *** Cibo_ has quit IRC | 15:28 | |
| *** bhavik1 has joined #zuul | 16:06 | |
| *** bhavik1 has quit IRC | 16:08 | |
| Shrews | jeblair: i think you need to set NodeRequest.state='requested' somewhere in your patchset | 16:11 |
| Shrews | i'm writing code right now that looks for that | 16:11 |
| *** saneax-_-|AFK is now known as saneax | 16:49 | |
| jeblair | Shrews: yeah -- that's the default in model.py. i figured that would be okay since zuul always creates node requests (it never reads existing ones that it did not create). but we could set the default to something else and set it explicitly if that seems better. | 16:53 |
| *** openstackgerrit has joined #zuul | 16:55 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Remove unused clasess from zk.py https://review.openstack.org/417091 | 16:55 |
| Shrews | jeblair: oh, i was looking at zk.py. I didn't look at model.py but now I'm confused as to why there are two NodeRequest... | 16:55 |
| Shrews | ah... :) | 16:55 |
| jeblair | which reminds me ^ | 16:55 |
| jeblair | yeah, sorry, i wasn't sure which way i was going to go with that, and then figured it out but forgot to clean up. | 16:55 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool: Disable tests for nodepoold changes https://review.openstack.org/417108 | 17:22 |
| pabelanger | let the fun time roll | 17:28 |
| *** saneax is now known as saneax-_-|AFK | 17:53 | |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool: Remove Gearman and ZMQ https://review.openstack.org/417128 | 18:13 |
| Shrews | ^^^ that one's a bit large, but mostly removes a bunch of stuff | 18:13 |
| *** hashar has quit IRC | 18:26 | |
| pabelanger | nice | 18:49 |
| *** harlowja has joined #zuul | 19:16 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Verify nodes and requests are not leaked https://review.openstack.org/417136 | 19:19 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Verify nodes and requests are not leaked https://review.openstack.org/417136 | 19:43 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool: Minor code cleanup https://review.openstack.org/417199 | 21:36 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Verify nodes and requests are not leaked https://review.openstack.org/417136 | 21:38 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul: Remove excess printing from stats test https://review.openstack.org/417200 | 21:38 |
| jeblair | Shrews: i pointed out two small sections of code which may still be relevant in your big removal change | 21:45 |
| * Shrews looks | 21:46 | |
| jeblair | Shrews: (i'm fine approving that and pulling them back later if needed; either way) | 21:46 |
| jeblair | (they certainly aren't relevant in their current form and would need significant morphing -- mostly they're an important part of the node lifecycle) | 21:47 |
| Shrews | jeblair: let me add that class back in, just so i won't forget to look at it again | 21:49 |
| jeblair | Shrews: sounds good; i thought that might be part of your process | 21:51 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool: Remove Gearman and ZMQ https://review.openstack.org/417128 | 21:56 |
| openstackgerrit | David Shrewsbury proposed openstack-infra/nodepool: Minor code cleanup https://review.openstack.org/417199 | 21:56 |
| Shrews | jeblair: added it back. thx for the heads up | 21:56 |
| jeblair | np | 21:56 |
| *** saneax-_-|AFK is now known as saneax | 22:09 | |
| Shrews | jeblair: left some things to think about in 416700 for you | 22:14 |
| jeblair | Shrews: thanks, responded | 22:22 |
| *** saneax is now known as saneax-_-|AFK | 22:26 | |
| *** saneax-_-|AFK is now known as saneax | 22:34 | |
| *** saneax is now known as saneax-_-|AFK | 22:54 | |
| *** saneax-_-|AFK is now known as saneax | 23:05 | |
| *** saneax is now known as saneax-_-|AFK | 23:16 | |
| *** saneax-_-|AFK is now known as saneax | 23:26 | |
| *** saneax is now known as saneax-_-|AFK | 23:43 | |
| *** saneax-_-|AFK is now known as saneax | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!