| *** sam_wan has joined #zuul | 02:09 | |
| *** evrardjp has quit IRC | 02:33 | |
| *** evrardjp has joined #zuul | 02:33 | |
| *** saneax has joined #zuul | 04:19 | |
| *** vishalmanchanda has joined #zuul | 04:52 | |
| *** sam_wan has quit IRC | 04:59 | |
| *** ykarel has joined #zuul | 05:29 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Log raw connection events on submit https://review.opendev.org/c/zuul/zuul/+/787913 | 05:43 |
|---|---|---|
| *** ykarel_ has joined #zuul | 05:52 | |
| *** ykarel has quit IRC | 05:54 | |
| *** jcapitao has joined #zuul | 05:56 | |
| *** jfoufas1 has joined #zuul | 05:56 | |
| *** sam_wan has joined #zuul | 06:13 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Switch to ZooKeeper backed build result events https://review.opendev.org/c/zuul/zuul/+/787915 | 06:28 |
| *** ykarel_ is now known as ykarel | 06:50 | |
| *** iurygregory has joined #zuul | 06:58 | |
| *** rpittau|afk is now known as rpittau | 07:12 | |
| *** tosky has joined #zuul | 07:42 | |
| *** nils has joined #zuul | 07:43 | |
| *** openstack has joined #zuul | 07:53 | |
| *** ChanServ sets mode: +o openstack | 07:53 | |
| *** jpena|off is now known as jpena | 07:54 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Improve component registry https://review.opendev.org/c/zuul/zuul/+/787684 | 08:02 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Calculate statsd metrics via ComponentRegistry https://review.opendev.org/c/zuul/zuul/+/787685 | 08:02 |
| *** openstackgerrit has quit IRC | 08:11 | |
| *** webknjaz has quit IRC | 08:24 | |
| *** dcastellani has quit IRC | 08:24 | |
| *** PrinzElvis has quit IRC | 08:25 | |
| *** webknjaz has joined #zuul | 08:30 | |
| *** dcastellani has joined #zuul | 08:30 | |
| *** PrinzElvis has joined #zuul | 08:30 | |
| *** hamalq has joined #zuul | 08:33 | |
| *** hamalq has quit IRC | 08:38 | |
| *** openstackgerrit has joined #zuul | 08:53 | |
| openstackgerrit | Ashley Bullock proposed zuul/zuul master: Add initial bitbucket cloud driver using webhooks https://review.opendev.org/c/zuul/zuul/+/759003 | 08:53 |
| *** parallax has joined #zuul | 08:57 | |
| *** ykarel is now known as ykarel|lunch | 09:16 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Improve component registry https://review.opendev.org/c/zuul/zuul/+/787684 | 09:56 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Calculate statsd metrics via ComponentRegistry https://review.opendev.org/c/zuul/zuul/+/787685 | 09:56 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Switch to ZooKeeper backed merge result events https://review.opendev.org/c/zuul/zuul/+/784195 | 10:06 |
| *** ykarel|lunch is now known as ykarel | 10:29 | |
| *** hamalq has joined #zuul | 10:34 | |
| *** hamalq has quit IRC | 10:39 | |
| *** sam_wan has quit IRC | 10:45 | |
| *** jcapitao is now known as jcapitao_lunch | 10:58 | |
| lyr | Hi there. Is there a prometheus exporter for zuul & nodepool & co ? Or it's only zuul & co into statd into prometheus ? | 11:04 |
| tristanC | lyr: the prometheus exporter is still under review, you have to setup a statsd_exporter to use zuul with prometheus | 11:07 |
| lyr | ah, there's one coming out somewhen then ? | 11:08 |
| tristanC | lyr: the review is https://review.opendev.org/c/zuul/zuul/+/599209 | 11:09 |
| lyr | thanks | 11:15 |
| *** jpena is now known as jpena|lunch | 11:26 | |
| tristanC | perhaps another solution would be the one described at https://github.com/nolar/kopf/issues/289 | 11:27 |
| *** rlandy has joined #zuul | 11:41 | |
| *** rlandy is now known as rlandy|rover | 11:42 | |
| *** jcapitao_lunch is now known as jcapitao | 12:02 | |
| *** jpena|lunch is now known as jpena | 12:28 | |
| *** hamalq has joined #zuul | 12:35 | |
| *** sduthil has joined #zuul | 12:36 | |
| *** hamalq has quit IRC | 12:39 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: prometheus: add options to start the server and process collector https://review.opendev.org/c/zuul/zuul/+/599209 | 12:53 |
| *** sshnaidm|afk is now known as sshnaidm | 13:10 | |
| *** vishalmanchanda has quit IRC | 13:41 | |
| *** vishalmanchanda has joined #zuul | 14:22 | |
| *** openstackgerrit has quit IRC | 14:23 | |
| *** josefwells has joined #zuul | 14:32 | |
| *** openstackgerrit has joined #zuul | 14:34 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: prometheus: add options to start the server and process collector https://review.opendev.org/c/zuul/zuul/+/599209 | 14:34 |
| *** hamalq has joined #zuul | 14:36 | |
| corvus | tristanC: https://github.com/prometheus/client_python/issues/132 is your understanding that once 3.8.0 is our minimum, that should work? | 14:37 |
| *** hamalq has quit IRC | 14:40 | |
| tristanC | corvus: python-3.8 seems to be the requirements for http server on v6 yes | 14:41 |
| corvus | tristanC: ok. seems like that's probably something we can live with; maybe add a comment to the code so we don't forget why that's set that way? | 14:44 |
| corvus | otherwise lgtm | 14:45 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: prometheus: add options to start the server and process collector https://review.opendev.org/c/zuul/zuul/+/599209 | 14:47 |
| tristanC | corvus: nice, thanks! | 14:47 |
| *** nils has quit IRC | 14:49 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: statsd: decrement resources gauge for held node https://review.opendev.org/c/zuul/zuul/+/781092 | 14:56 |
| *** nils has joined #zuul | 15:00 | |
| openstackgerrit | Merged zuul/zuul master: Log raw connection events on submit https://review.opendev.org/c/zuul/zuul/+/787913 | 15:03 |
| *** nils has quit IRC | 15:09 | |
| *** nils has joined #zuul | 15:11 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Capture the openstack logs in the functional job https://review.opendev.org/c/zuul/nodepool/+/788028 | 15:18 |
| clarkb | I don't know that we want to merge ^ but trying to debug some failures with the functional openstack jobs | 15:18 |
| *** ykarel has quit IRC | 15:21 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add build-info subcommand https://review.opendev.org/c/zuul/zuul-client/+/751070 | 15:36 |
| *** mnaser has joined #zuul | 15:54 | |
| *** rpittau is now known as rpittau|afk | 15:55 | |
| *** ykarel has joined #zuul | 15:58 | |
| *** hamalq has joined #zuul | 15:58 | |
| *** hamalq has quit IRC | 16:03 | |
| *** ykarel has quit IRC | 16:03 | |
| *** hamalq has joined #zuul | 16:04 | |
| *** jcapitao has quit IRC | 16:28 | |
| *** saneax has quit IRC | 16:33 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Capture the openstack logs in the functional job https://review.opendev.org/c/zuul/nodepool/+/788028 | 16:43 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Roles to create, cleanup and promote snapshots in ec2 https://review.opendev.org/c/zuul/zuul-jobs/+/787677 | 16:47 |
| *** jpena is now known as jpena|off | 17:10 | |
| *** sshnaidm is now known as sshnaidm|afk | 17:12 | |
| *** josefwells has quit IRC | 17:16 | |
| *** jfoufas1 has quit IRC | 17:30 | |
| sean-k-mooney | hi has any discussion of zuul and ansible execution enviornments taken place | 17:56 |
| sean-k-mooney | https://www.ansible.com/blog/introduction-to-ansible-builder | 17:56 |
| sean-k-mooney | or integrating with https://ansible-runner.readthedocs.io/en/latest/install.html | 18:00 |
| sean-k-mooney | ansible-runner | 18:00 |
| sean-k-mooney | sorry i should have linked https://ansible-runner.readthedocs.io/en/latest/intro.html | 18:02 |
| SpamapS | corvus: I just read that hackernews comment. Pretty cool. Did you see this mentioned there? https://blog.mergify.io/announcing-speculative-merge-queues | 18:03 |
| SpamapS | The diagram there looks AWFULLY familiar. ;) | 18:04 |
| SpamapS | "Faster & Safer code merge" | 18:05 |
| SpamapS | Oh I see.. those are our people. :-D | 18:05 |
| sean-k-mooney | it does not mention zuul but ya it look familar alright | 18:07 |
| sean-k-mooney | i dont know if they are using zuul underneath or not | 18:09 |
| pabelanger | sean-k-mooney: we'd have to decided to use podman or docker on the zuul-executors over bwrap | 18:11 |
| sean-k-mooney | ya but alsom maybe on the target nodes | 18:12 |
| pabelanger | as you said, the interface is ansible-runner | 18:12 |
| sean-k-mooney | its not clear if you can or should have the execution env on the nodes being managed or not | 18:13 |
| corvus | if a-r existed when we wrote zuulv3, we probably would have used it. as it stands now though, we're pretty good at running ansible, so i don't think it's worth refactoring | 18:13 |
| pabelanger | no, it is just needed on the controller side | 18:13 |
| pabelanger | (also I am building EEs for downstream Redhat as my day job) :) | 18:13 |
| sean-k-mooney | corvus: well the concern is that installing ansible from non source might get more challaning going forward | 18:13 |
| sean-k-mooney | pabelanger: we should talk so | 18:14 |
| pabelanger | so far, ansible community is still rolling an ansible package on pypi. But for productization, it is EEs moving forward | 18:14 |
| pabelanger | which is why ansible blog is talking about them now | 18:14 |
| pabelanger | It would be worth seeing, moving to ansible-core and selecting which collections to install into zuul-executor. while still using bwrap | 18:20 |
| corvus | ++ we've always wanted to support (first galaxy, now collections) in zuul | 18:20 |
| *** vishalmanchanda has quit IRC | 18:21 | |
| sean-k-mooney | cool i was not sure if that was public so didnt want to say that here | 18:35 |
| sean-k-mooney | but also wanted to use zuul more downstream and wanted to make sure we still could in the future | 18:36 |
| *** gouthamr has joined #zuul | 19:01 | |
| *** BaJ has joined #zuul | 20:04 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add buildsets, buildset-info to subcommands https://review.opendev.org/c/zuul/zuul-client/+/752909 | 20:23 |
| *** jpena|off has quit IRC | 20:32 | |
| *** fbo has quit IRC | 20:33 | |
| fungi | SpamapS: yeah, for some reason i thought jd had added speculative states to mergify years ago (like around when you were working on bonnyci), but maybe i misremembered, or maybe he's improved it or is just reannouncing it | 20:56 |
| *** nils has quit IRC | 21:34 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Switch to ZooKeeper backed build result events https://review.opendev.org/c/zuul/zuul/+/782939 | 21:36 |
| clarkb | corvus: in the unique repo checkouts change the reason the playbook checkouts aren't a problem is they are prefixed with project_X right? | 21:37 |
| corvus | clarkb: exactly, they're basically one repo per dir | 21:38 |
| corvus | the repo dirs are called project_X and then it doesn't matter what's underneath that | 21:38 |
| clarkb | yup, just double checking my understanding of that | 21:39 |
| clarkb | corvus: left a question on https://review.opendev.org/c/zuul/zuul/+/787451 but overall it lgtm | 21:53 |
| corvus | clarkb: replied | 21:58 |
| clarkb | corvus: I see and we only need to supply the scheme in that one case because we are overriding the default scheme | 22:06 |
| corvus | clarkb: yep | 22:08 |
| clarkb | corvus: I wrote that question down more formally in the code review if you want to double check I've got it right still | 22:09 |
| clarkb | we override the default scheme in a couple of other locations and wonder if they should be explicit about the cache scheme too | 22:09 |
| corvus | clarkb: replied; the merger cache always uses the unique scheme, and i figured that's typical enough to just leave it as a default... | 22:24 |
| corvus | it wouldn't hurt to be explicit; i guess in my mind i just figured that was the usual case, and we only need to be explicit when we're moving out into the jobdirs | 22:26 |
| clarkb | got it, it is root dependent. That makes sense | 22:33 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 22:35 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Test zuul-client builds subcommand https://review.opendev.org/c/zuul/zuul/+/758141 | 22:39 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: test zuul-client build-info https://review.opendev.org/c/zuul/zuul/+/758783 | 22:39 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-client: Test buildsets, buildset-info subcommands https://review.opendev.org/c/zuul/zuul/+/758985 | 22:39 |
| mhu | before I rebase or abandon https://review.opendev.org/c/zuul/zuul-client/+/755292 , are object models and annotations a hard no? I remember tristanc being interested in that for zuul-client | 22:44 |
| mhu | as a consumer of the API it would bring security in handling data | 22:45 |
| mhu | (if we don't go the object model route I'll modify the change to just cover output formatters) | 22:46 |
| fungi | mhu: i don't thing the ml thread has reached a conclusion, but it was pointed out that there's no "halfway" in annotating the codebase in a reasonable manner, and trying to shoehorn type hints into the entire codebase is unlikely to be a great use of someone's time | 22:47 |
| fungi | plus type hinting in python is still in an immature enough state to be painful to deal with | 22:48 |
| mhu | fungi: this change is for zuul-client specifically, so it's isolated | 22:49 |
| corvus | i think we're pretty close to no on annotations; | 22:49 |
| mhu | ok, just wanted to make sure to avoid working on something that'd get the boot | 22:51 |
| mhu | thanks! | 22:51 |
| corvus | mhu: thanks :) | 22:52 |
| fungi | most recent post on that thread was more than a month ago: ansible localhost -i localhost, -m setup | 22:52 |
| fungi | er, http://lists.zuul-ci.org/pipermail/zuul-discuss/2021-March/001538.html | 22:52 |
| fungi | if anyone has any more suggestions on how to make it not a pain, that's probably the best place to resume the discussion | 22:53 |
| corvus | yeah, should probably add something to dev docs about it | 22:53 |
| mhu | by the way, anything I should check on the ML for April? I just recently emerged from diaper duty and I must say I didn't think of checking the ML | 22:54 |
| mhu | (baby poop has this effect on you) | 22:54 |
| fungi | there were 5 threads: http://lists.zuul-ci.org/pipermail/zuul-discuss/2021-April/thread.html | 22:54 |
| fungi | ideally you can tell from their subjects if they're anything you care about | 22:55 |
| fungi | (otherwise they're not well-composed subject lines) | 22:55 |
| clarkb | corvus: felixedel I left some comments/questions on https://review.opendev.org/c/zuul/zuul/+/782939 | 22:55 |
| clarkb | Looks like it needs an update for some linting problems (that seem valid), and then I've got a few questions too | 22:55 |
| corvus | mhu: the change ref'd by 'proposed behavior change' has merged; seems like no one figured that was a showstopper; the other one is in review; we made a couple releases, mostly checkpointing the scale-out-scheduler stuff | 22:56 |
| corvus | clarkb: oof, i just copied felix's suggestion; i'll fix that. | 22:57 |
| mhu | corvus, thanks for the summary | 22:57 |
| mhu | do you think there's some bandwidth to move forward on the GUI authentication and admin actions? | 22:58 |
| corvus | mhu: i'm hoping to look at it within the next week or two | 22:59 |
| mhu | cool! I think the underlying mechanic is sound, it should be mainly cosmetic stuff to tinker with | 23:01 |
| mhu | hopefully | 23:01 |
| *** tosky has quit IRC | 23:01 | |
| corvus | clarkb: i replied to some of your comments; will leave felixedel to reply to others | 23:09 |
| clarkb | thanks! | 23:10 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Switch to ZooKeeper backed build result events https://review.opendev.org/c/zuul/zuul/+/782939 | 23:12 |
| corvus | that should take care of the copypasta | 23:13 |
| *** smyers_ has joined #zuul | 23:22 | |
| *** smyers has quit IRC | 23:22 | |
| *** smyers_ is now known as smyers | 23:22 | |
| *** rlandy|rover has quit IRC | 23:47 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!