Tuesday, 2017-01-03

*** jamielennox is now known as jamielennox|away01:05
*** jamielennox|away is now known as jamielennox02:19
*** timrc_ has quit IRC04:59
*** timrchavez_ has joined #zuul05:05
*** timrchavez_ has quit IRC05:06
*** timrchavez_ has joined #zuul05:07
*** timrchavez_ has quit IRC05:16
*** timrchavez_ has joined #zuul05:20
*** timrchavez_ has quit IRC05:21
*** timrchavez_ has joined #zuul05:21
*** timrchavez_ has quit IRC05:25
*** timrchavez_ has joined #zuul05:27
*** jamielennox is now known as jamielennox|away05:52
*** jamielennox|away is now known as jamielennox06:00
*** abregman_ has joined #zuul06:53
*** bhavik1 has joined #zuul07:56
*** saneax-_-|AFK is now known as saneax08:12
*** hashar has joined #zuul09:11
*** Cibo_ has joined #zuul09:34
*** bhavik1 has quit IRC09:48
*** Cibo has joined #zuul12:07
Cibohello folks, can anybody please give a clue: I have zuul stoping every 12 hours with DEBUG zuul.Scheduler: Run handler sleeping12:09
Ciboand never starting again...12:09
Cibowhat should I check?12:09
*** TheJulia has quit IRC12:19
*** TheJulia has joined #zuul12:25
*** saneax is now known as saneax-_-|AFK12:49
*** threebean1 is now known as threebean13:30
*** threebean has quit IRC13:31
*** threebean has joined #zuul13:31
*** gundalow has quit IRC13:47
*** gundalow has joined #zuul13:47
openstackgerritJan Hruban proposed openstack-infra/zuul: Merge pull requests from github reporter  https://review.openstack.org/24325014:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Fix job hierarchy bug.  https://review.openstack.org/19245714:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Encapsulate determining the event purpose  https://review.openstack.org/24748714:14
openstackgerritJan Hruban proposed openstack-infra/zuul: support github pull reqeust labels  https://review.openstack.org/24742114:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Better merge message for GitHub pull reqeusts  https://review.openstack.org/28066714:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Add 'push' and 'tag' github webhook events.  https://review.openstack.org/19120714:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Add 'pr-comment' github webhook event  https://review.openstack.org/23920314:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Support for dependent pipelines with github  https://review.openstack.org/24750014:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Configurable SSH access to GitHub  https://review.openstack.org/23913814:14
openstackgerritJan Hruban proposed openstack-infra/zuul: GitHub file matching support  https://review.openstack.org/29237614:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Allow github trigger to match on branches/refs  https://review.openstack.org/25844814:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Log GitHub API rate limit  https://review.openstack.org/29237714:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Set filter according to PR/Change in URL  https://review.openstack.org/32530014:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Fix the file matcher for github  https://review.openstack.org/39767914:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Query changes only via relevant sources  https://review.openstack.org/39767814:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Do not share the gerrit queue dbs in tests  https://review.openstack.org/39767714:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Add basic Github Zuul Reporter.  https://review.openstack.org/19131214:14
openstackgerritJan Hruban proposed openstack-infra/zuul: Support for github commit status  https://review.openstack.org/23930314:14
*** abregman_ has quit IRC14:58
*** timrchavez_ is now known as timrc_16:10
*** herlo_ is now known as herlo17:51
*** herlo has quit IRC17:51
*** herlo has joined #zuul17:51
Shrewsi believe i have forgotten all things zuul+nodepool18:49
Shrewsjeblair: Just to clear up confusion, the zuulv3 spec states: "The component of Nodepool which will process these requests is known as a “launcher”. A Nodepool system may consiste of multiple launchers (for instance, one launcher for each cloud provider)." Since there already exists a NodePoolLauncher class representing a thread creating a node, does the statement from the spec describe a "launcher" that is19:00
Shrewsnodepoold itself (with separate configs forr other nodepoold instances for different providers)? Or do you envision new threads within a single nodepoold handling different providers?19:00
Shrewswow, that's a lot of words. too many for post-vacation questions  :(19:01
jeblairShrews: i was not thinking of the nodepoollauncher class when i wrote that -- so your first thing is correct -- "nodepoold itself".  so a single nodepoold may handle one or more providers.19:04
jeblairand yeah, i'm trying to remember what i was working on too :)19:04
Shrewsjeblair: ok. that's what i thought (and hoped). thanks19:04
clarkband we decided to punt on overlapping providers in multiple nodepoolds for now19:19
*** fungi has quit IRC19:35
*** TheJulia has quit IRC19:35
*** rmoe has quit IRC19:35
*** jeblair has quit IRC19:35
*** jeblair has joined #zuul19:35
*** rmoe has joined #zuul19:36
*** fungi has joined #zuul19:36
*** harlowja has joined #zuul19:38
*** Cibo_ has quit IRC19:40
*** TheJulia has joined #zuul19:43
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool: Register launcher name with ZooKeeper  https://review.openstack.org/41633519:53
Shrewsand so it begins19:55
openstackgerritK Jonathan Harker proposed openstack-infra/zuul: Add github reporter status_url config option  https://review.openstack.org/41575020:01
openstackgerritK Jonathan Harker proposed openstack-infra/zuul: Add support for github enterprise  https://review.openstack.org/40767520:01
openstackgerritK Jonathan Harker proposed openstack-infra/zuul: Use integration_id with github  https://review.openstack.org/41154420:01
*** openstackgerrit has quit IRC20:03
*** openstackgerrit has joined #zuul20:23
openstackgerritK Jonathan Harker proposed openstack-infra/zuul: Add github reporter status_url config option  https://review.openstack.org/41575020:23
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool: Register launcher name with ZooKeeper  https://review.openstack.org/41633520:56
openstackgerritJames E. Blair proposed openstack-infra/zuul: Re-submit node requests on ZooKeeper disconnect  https://review.openstack.org/41635721:45
Shrewsjeblair: so 416357... you are now entering a problem area (one that I fought against the first go round)21:49
jeblairShrews: yeah, i figured tackling a hard problem would get me up to speed quickly :)21:49
jeblairShrews: (also, i wanted to write that so a had a better handle on what would be going on before i looked at the DataWatch stuff)21:50
Shrewsjeblair: the connection listener happens in a totally separate thread of execution. also, it must NOT do anything intensive during the callback (documented in the kazoo docs somewhere)21:51
jeblairShrews: yeah i know the connection callback stuff can be tricky (thanks for the docstrings!); what i've done there is plugged into zuul's normal async processing21:51
Shrewss/intensive/potentially blocking/21:51
jeblairShrews: so basically, it just throws an event onto one of zuul's queues21:52
jeblairand zuul gets around to handling that in its main thread later21:52
Shrews*nod*  just wanted to throw those dragons out there for ya  ;)21:52
jeblairya...21:52
jeblairShrews: we *could* just have zuul poll the status each time through the loop.  that's a little more in-line with the current zk.py api21:53
jeblairShrews: but the whole "kazoo callback -> zuul event queue" thing seemed tailor-made for this architecture, so i wanted to give it a shot21:54
Shrewswell, if you get it working as-is, great. let's do that21:54
Shrewsit didn't fit the model well for the current api21:54
Shrewsthough i think the DataWatcher api could have worked now21:55
jeblairyeah, i'll look at it next21:56
jheskethMorning21:58
*** openstack has joined #zuul23:17
*** saneax-_-|AFK is now known as saneax23:29
*** hashar has quit IRC23:34

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!