| *** JasonCL has quit IRC | 00:56 | |
| *** haint has joined #zuul | 02:09 | |
| *** EmilienM_ has joined #zuul | 02:17 | |
| *** _ari__ has joined #zuul | 02:17 | |
| *** cinerama` has joined #zuul | 02:20 | |
| *** haint_ has quit IRC | 02:21 | |
| *** myoung has joined #zuul | 02:21 | |
| *** EmilienM has quit IRC | 02:22 | |
| *** _ari_ has quit IRC | 02:22 | |
| *** myoung|ruck has quit IRC | 02:22 | |
| *** zigo has quit IRC | 02:22 | |
| *** cinerama has quit IRC | 02:22 | |
| *** EmilienM_ is now known as EmilienM | 02:22 | |
| *** EmilienM has quit IRC | 02:22 | |
| *** EmilienM has joined #zuul | 02:22 | |
| *** myoung is now known as myoung|ruck | 02:23 | |
| *** patriciadomin has quit IRC | 02:23 | |
| *** pbrobinson has quit IRC | 02:23 | |
| *** toabctl has quit IRC | 02:25 | |
| *** patriciadomin has joined #zuul | 02:26 | |
| *** pbrobinson has joined #zuul | 02:26 | |
| *** zigo has joined #zuul | 02:27 | |
| *** toabctl has joined #zuul | 02:27 | |
| *** zigo is now known as Guest13268 | 02:29 | |
| *** JasonCL has joined #zuul | 02:45 | |
| *** dkranz has quit IRC | 03:18 | |
| *** dkranz has joined #zuul | 03:20 | |
| *** JasonCL has quit IRC | 03:24 | |
| *** JasonCL has joined #zuul | 03:56 | |
| *** JasonCL has quit IRC | 04:01 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul-jobs master: Convert back to zuul.projects https://review.openstack.org/524460 | 04:02 |
|---|---|---|
| *** threestrands_ has joined #zuul | 04:24 | |
| *** threestrands_ has quit IRC | 04:24 | |
| *** threestrands_ has joined #zuul | 04:24 | |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul feature/zuulv3: Convert zuul.projects to a dict https://review.openstack.org/514119 | 04:24 |
| openstackgerrit | Ian Wienand proposed openstack-infra/zuul feature/zuulv3: Remove zuul._projects https://review.openstack.org/524463 | 04:24 |
| *** vivsoni_ has joined #zuul | 04:26 | |
| *** threestrands has quit IRC | 04:26 | |
| vivsoni_ | Hi Team, please help me with on how to execute zuul checks (i.e. all jobs) on my local openstack machine | 04:31 |
| *** bhavik1 has joined #zuul | 04:47 | |
| *** bhavik1 has quit IRC | 04:55 | |
| tobiash | jeblair: I've a comment on 524410 | 06:16 |
| tobiash | vivsoni_: you want to execute zuul jobs outside of zuul? | 06:17 |
| *** threestrands_ has quit IRC | 06:24 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Remove nodesets from builds canceled during reconfiguration https://review.openstack.org/524409 | 06:38 |
| vivsoni_ | tobiash: yes.. is it possible | 06:50 |
| openstackgerrit | Rui Chen proposed openstack-infra/nodepool feature/zuulv3: Fix nodepool alien-list issue https://review.openstack.org/522495 | 07:45 |
| tobiash | vivsoni_: currently only with manual steps, I think mordred and some others have local scripts ti simplify that, but there is no tooling upstream yet to accomplish that | 08:14 |
| vivsoni_ | ok | 08:15 |
| *** hashar has joined #zuul | 08:38 | |
| openstackgerrit | Krzysztof Klimonda proposed openstack-infra/zuul feature/zuulv3: Support autoholding nodes for specific changes/refs https://review.openstack.org/515169 | 09:09 |
| *** electrofelix has joined #zuul | 09:46 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: Move github webhook from webapp to zuul-web https://review.openstack.org/504267 | 10:15 |
| tristanC | jlk: hope you don't mind i rebase 504267... what do you think it's missing? | 10:23 |
| tristanC | (asking because it's in the zuul roadmap for the 3.0 release) | 10:30 |
| *** openstackgerrit has quit IRC | 10:33 | |
| *** openstackgerrit has joined #zuul | 10:34 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: Serve keys from canonical project name https://review.openstack.org/504807 | 10:34 |
| tristanC | jeblair: i rebased 504807 and fixed a comment from tobiash | 10:34 |
| tristanC | Shrews: would you mind having a look at https://review.openstack.org/#/c/523640/, it seems like this improve the nodepool behavior when the zk connection is flaky | 10:37 |
| *** jesusaur has quit IRC | 10:42 | |
| *** jesusaur has joined #zuul | 10:45 | |
| *** JasonCL has joined #zuul | 11:00 | |
| *** openstackgerrit has quit IRC | 11:18 | |
| *** jesusaur has quit IRC | 11:31 | |
| *** jesusaur has joined #zuul | 11:32 | |
| *** JasonCL has quit IRC | 11:36 | |
| *** haint has quit IRC | 11:50 | |
| *** JasonCL has joined #zuul | 12:00 | |
| *** nguyentrihai has joined #zuul | 12:02 | |
| Shrews | tristanC: left a comment. you get a default Retry for free | 13:10 |
| Shrews | tristanC: oh, you know what, i might be wrong about that | 13:11 |
| Shrews | code says one thing, but their docs say another | 13:11 |
| Shrews | so now i'm confused | 13:12 |
| Shrews | https://kazoo.readthedocs.io/en/latest/_modules/kazoo/client.html#KazooClient seems to clearly show a KazooRetry being created for you if command_retry is not specified | 13:14 |
| Shrews | but https://kazoo.readthedocs.io/en/latest/basic_usage.html#retrying-commands says another thing | 13:15 |
| Shrews | harlowja: is that right??? ^^^^ | 13:15 |
| * Shrews feels like this may be something he's looked at long ago | 13:19 | |
| tristanC | Shrews: yes, I set max_tries to 3 so that it still fails after a second. I wasn't sure we want to make nodepool wait infinitely | 13:20 |
| Shrews | tristanC: oh, that's a good point | 13:21 |
| tristanC | in our deployment, zookeeper is on a dedicated openstack instance, and for some reason i often see kazoo.exceptions.ConnectionLoss being raised by the nodepool-launcher service. Using that 1 second retry is enough to fix those exception | 13:22 |
| Shrews | tristanC: wait, i'm confused. i thought you were seeing nodepool wait forever? getting exceptions seems to indicate the opposite | 13:24 |
| tristanC | Shrews: when not using KazooRetry(max_tries=3), then the client seems to retry infinitely | 13:26 |
| tristanC | that would fix the exception being raised, but if zookeeper is turned off, then nodepool would be stuck retrying i guess | 13:27 |
| Shrews | trying to make breakfast atm.... gimme a few minutes | 13:35 |
| tristanC | Shrews: no huries, i left a clearer comment on the review | 13:39 |
| *** openstackgerrit has joined #zuul | 14:10 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/nodepool feature/zuulv3: Add a plugin interface for drivers https://review.openstack.org/524620 | 14:10 |
| *** hashar is now known as hasharAway | 14:22 | |
| *** hasharAway has quit IRC | 14:28 | |
| *** jkilpatr has quit IRC | 14:35 | |
| jeblair | tobiash: the window increase factor has a minimum value of 1 | 14:58 |
| tobiash | jeblair: I meant factor of 1 in exponential case ;) | 14:58 |
| jeblair | oh that's a mean trick | 14:59 |
| tobiash | that gives a constant window | 14:59 |
| tobiash | we used this trick to simulate semaphores before I added them | 14:59 |
| jeblair | tobiash: the only thing i can think to do is to specially handle that one case. if new_type==exponential and new_increase==1, just use the new value. otherwise do what i wrote in 524410. is that good? any other ideas? | 15:03 |
| tobiash | jeblair: sounds good, I would do it the same | 15:03 |
| jeblair | okay, i'll rework that. i think it's going to need a new test. | 15:04 |
| dmsimard | jeblair: was https://review.openstack.org/#/c/524225/ what you meant for the END OF STREAM thing ? | 15:15 |
| *** jkilpatr has joined #zuul | 15:16 | |
| jeblair | dmsimard: yep | 15:18 |
| jeblair | dmsimard: though -- i wonder if that should have a timestamp on it... | 15:20 |
| dmsimard | jeblair: good point, let me fix that | 15:20 |
| jeblair | dmsimard: there's some code in that file that does that, search for: job_output.write("{now} | 15:21 |
| dmsimard | yup | 15:26 |
| Shrews | gah, system update has broken my docker (and thus my zookeeper container i use for testing) | 15:33 |
| * Shrews gets stabby | 15:33 | |
| *** ianw has quit IRC | 15:34 | |
| *** JasonCL has quit IRC | 15:38 | |
| *** JasonCL has joined #zuul | 15:45 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Fix double centos in URL for CentOS extras https://review.openstack.org/524653 | 15:50 |
| *** JasonCL has quit IRC | 15:51 | |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Fix double centos in URL for CentOS Plus infrastructure mirror https://review.openstack.org/524653 | 15:55 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul feature/zuulv3: Print a message when we start the Zuul console https://review.openstack.org/524225 | 15:59 |
| jlk | tristanC: please run with it. I lost track over the last few weeks where I've been unable to focus on Zuul | 16:27 |
| openstackgerrit | David Moreau Simard proposed openstack-infra/zuul-jobs master: Remove the CentOS Plus mirror from the configured mirrors https://review.openstack.org/524653 | 16:28 |
| clarkb | Shrews: fwiw zk totally runs fine as normal user out of its tarball | 16:30 |
| clarkb | Shrews: if you want a quick fix to docker not working | 16:30 |
| clarkb | tobiash: hacking the window resizing, nice | 16:32 |
| jeblair | tobiash: do you have a second to +3 https://review.openstack.org/523929 ? | 16:32 |
| Shrews | clarkb: yup, already have it downloaded | 16:33 |
| tobiash | jeblair: looking | 16:35 |
| *** JasonCL has joined #zuul | 16:38 | |
| tobiash | jeblair: lgtm | 16:38 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Print a message when we start the Zuul console https://review.openstack.org/524225 | 16:45 |
| *** fbo_ has joined #zuul | 16:50 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix branch checkout order https://review.openstack.org/523929 | 16:52 |
| *** weshay is now known as weshay_mtg | 16:57 | |
| *** JasonCL has quit IRC | 17:36 | |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Don't shrink windows on reconfiguration https://review.openstack.org/524410 | 17:42 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Don't set job var override_checkout if null https://review.openstack.org/524414 | 17:42 |
| jeblair | tobiash, clarkb: ^ now with more special casing and another test | 17:42 |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Remove the CentOS Plus mirror from the configured mirrors https://review.openstack.org/524653 | 17:43 |
| tobiash | jeblair: I like your todo comment ;) | 17:48 |
| jlk | tristanC: I had a rebase done here, but I hadn't tested it yet. | 17:54 |
| *** JasonCL has joined #zuul | 17:55 | |
| harlowja | Shrews that seems right ; i'd trust the code docs vs the basic usage one :-P | 18:00 |
| *** weshay_mtg is now known as weshay_bbiab | 18:05 | |
| *** myoung|ruck is now known as myoung|ruck|lunc | 18:11 | |
| *** myoung|ruck|lunc is now known as myoung|ruck|food | 18:12 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Don't shrink windows on reconfiguration https://review.openstack.org/524410 | 18:18 |
| harlowja | Shrews though let me know, i can investigate more | 18:18 |
| Shrews | harlowja: nah, just wanted to make sure I was reading the code correctly (and if so, point out the doc disagreement) | 18:19 |
| harlowja | kk | 18:19 |
| Shrews | harlowja: though even the code docs are in disagreement with actual code (mentions using -1, but None seems to actually mean "forever") | 18:19 |
| harlowja | fun fun | 18:20 |
| harlowja | do u want to make a PR :) | 18:20 |
| harlowja | i will merge it, ha | 18:20 |
| Shrews | i assume a signed int is being used for the counter, so eventually it would become -1 :) | 18:20 |
| harlowja | lol | 18:20 |
| harlowja | not with python bigints :-P | 18:20 |
| Shrews | forever is just further away | 18:21 |
| harlowja | :-P | 18:22 |
| *** electrofelix has quit IRC | 18:31 | |
| *** myoung|ruck|food is now known as myoung|ruck | 18:49 | |
| *** jkilpatr has quit IRC | 19:33 | |
| openstackgerrit | Monty Taylor proposed openstack-infra/zuul-jobs master: Ensure ChangeLog exists in install-if-python https://review.openstack.org/524712 | 19:37 |
| openstackgerrit | James E. Blair proposed openstack-infra/zuul feature/zuulv3: Fix scheduler reconfiguration handler https://review.openstack.org/524714 | 19:48 |
| jeblair | clarkb, fungi: ^ can we speedy merge that? | 19:48 |
| fungi | on it | 19:49 |
| * clarkb looks at it speedily | 19:49 | |
| fungi | ooh, ouch | 19:49 |
| clarkb | approved | 19:50 |
| jeblair | it's not impacting ongoing operations, but would be nice to restart with that fix so we can add new projects again | 19:50 |
| *** weshay_bbiab is now known as weshay | 20:06 | |
| *** weshay is now known as weshay_interview | 20:12 | |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Don't set job var override_checkout if null https://review.openstack.org/524414 | 20:16 |
| openstackgerrit | Merged openstack-infra/zuul feature/zuulv3: Fix scheduler reconfiguration handler https://review.openstack.org/524714 | 20:17 |
| *** JasonCL has quit IRC | 20:29 | |
| *** JasonCL has joined #zuul | 20:29 | |
| openstackgerrit | Merged openstack-infra/zuul-jobs master: Ensure ChangeLog exists in install-if-python https://review.openstack.org/524712 | 20:44 |
| *** jkilpatr has joined #zuul | 20:51 | |
| *** JasonCL has quit IRC | 20:53 | |
| *** JasonCL has joined #zuul | 20:53 | |
| *** ianw has joined #zuul | 21:09 | |
| *** weshay_interview is now known as weshay | 21:19 | |
| *** JasonCL has quit IRC | 21:38 | |
| *** JasonCL has joined #zuul | 22:06 | |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul master: tox: remove validate-layout https://review.openstack.org/524753 | 23:02 |
| openstackgerrit | Tristan Cacqueray proposed openstack-infra/zuul feature/zuulv3: tox: remove validate-layout https://review.openstack.org/524757 | 23:09 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!