| *** hamalq has quit IRC | 00:12 | |
| *** hamalq has joined #zuul | 00:12 | |
| *** rlandy|bbl has quit IRC | 02:28 | |
| *** hamalq has quit IRC | 02:45 | |
| *** rlandy has joined #zuul | 03:31 | |
| *** rlandy has quit IRC | 03:35 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: UI: Remove stretchable link from build and buildset table https://review.opendev.org/c/zuul/zuul/+/761621 | 03:40 |
|---|---|---|
| *** bhavikdbavishi has joined #zuul | 03:56 | |
| *** bhavikdbavishi1 has joined #zuul | 04:07 | |
| *** bhavikdbavishi has quit IRC | 04:09 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 04:09 | |
| *** ykarel has joined #zuul | 04:49 | |
| *** saneax has joined #zuul | 05:09 | |
| *** ykarel has quit IRC | 05:17 | |
| *** ykarel has joined #zuul | 05:18 | |
| *** vishalmanchanda has joined #zuul | 05:19 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| *** ykarel_ has joined #zuul | 05:51 | |
| *** ykarel has quit IRC | 05:53 | |
| *** ykarel_ is now known as ykarel | 06:00 | |
| felixedel | ianw: +1 on the squashed | 06:18 |
| felixedel | version of the "Remove stretchable link" change | 06:19 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Store tenant layout state in Zookeeper https://review.opendev.org/c/zuul/zuul/+/771461 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Configure unique command socket path per scheduler https://review.opendev.org/c/zuul/zuul/+/771462 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Support cross scheduler config loading https://review.opendev.org/c/zuul/zuul/+/771463 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Add UUID for queue items https://review.opendev.org/c/zuul/zuul/+/772512 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Store semaphore state in Zookeeper https://review.opendev.org/c/zuul/zuul/+/772513 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Fix test_gerrit.TestPolling.test_config_update https://review.opendev.org/c/zuul/zuul/+/773023 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Move serialization helper methods to ZooKeeperBase class https://review.opendev.org/c/zuul/zuul/+/773024 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Implement ZooKeeper backed merge jobs https://review.opendev.org/c/zuul/zuul/+/773025 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Switch to ZooKeeper backed merge jobs https://review.opendev.org/c/zuul/zuul/+/773026 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Collect statsd information from ZooKeeper rather than gearman https://review.opendev.org/c/zuul/zuul/+/773027 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Remove remaining gearman parts from merger https://review.opendev.org/c/zuul/zuul/+/773028 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Provide zk_client to merger client rather than the whole scheduler https://review.opendev.org/c/zuul/zuul/+/773029 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Remove history from RecordingMergeClient https://review.opendev.org/c/zuul/zuul/+/773030 | 06:47 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Align ZooKeeper builds and merger API https://review.opendev.org/c/zuul/zuul/+/773031 | 06:47 |
| *** reiterative has quit IRC | 06:48 | |
| *** reiterative has joined #zuul | 06:48 | |
| *** jcapitao has joined #zuul | 07:04 | |
| *** bhavikdbavishi has quit IRC | 07:14 | |
| *** rpittau|afk is now known as rpittau | 07:39 | |
| *** jpena|off is now known as jpena | 07:51 | |
| *** hashar has joined #zuul | 08:18 | |
| *** bhavikdbavishi has joined #zuul | 08:32 | |
| *** tosky has joined #zuul | 08:46 | |
| *** bhavikdbavishi1 has joined #zuul | 08:51 | |
| *** bhavikdbavishi has quit IRC | 08:53 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 08:53 | |
| *** fbo|off is now known as fbo | 08:55 | |
| *** ykarel is now known as ykarel|lunch | 09:15 | |
| *** nilsph has joined #zuul | 09:23 | |
| *** icey has quit IRC | 09:46 | |
| *** icey has joined #zuul | 09:50 | |
| *** ykarel|lunch is now known as ykarel | 10:16 | |
| *** hashar is now known as hasharLunch | 10:50 | |
| openstackgerrit | Andy Ladjadj proposed zuul/nodepool master: [driver][aws] propagate the instance_id in the node host_id field https://review.opendev.org/c/zuul/nodepool/+/758393 | 11:05 |
| *** sshnaidm|afk is now known as sshnaidm | 11:53 | |
| *** hasharLunch has quit IRC | 11:58 | |
| *** jcapitao is now known as jcapitao_lunch | 12:09 | |
| *** jpena is now known as jpena|lunch | 12:30 | |
| *** rlandy has joined #zuul | 12:34 | |
| *** bhavikdbavishi has quit IRC | 12:44 | |
| *** hasharLunch has joined #zuul | 12:46 | |
| *** bhavikdbavishi has joined #zuul | 12:47 | |
| *** jcapitao_lunch is now known as jcapitao | 13:14 | |
| *** rlandy is now known as rlandy|meeting | 13:15 | |
| *** hasharLunch is now known as hashar | 13:19 | |
| *** jpena|lunch is now known as jpena | 13:33 | |
| zbr | avass: are you ok with https://github.com/ansible-community/ansible-lint/pull/1303/files ? | 13:58 |
| *** bhavikdbavishi1 has joined #zuul | 14:01 | |
| *** ikhan has joined #zuul | 14:01 | |
| *** bhavikdbavishi has quit IRC | 14:02 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 14:02 | |
| *** bhavikdbavishi has quit IRC | 14:32 | |
| *** bhavikdbavishi has joined #zuul | 14:33 | |
| *** bhavikdbavishi has quit IRC | 14:50 | |
| avass | anyone wanna review nodepool builder for aws? :) https://review.opendev.org/c/zuul/nodepool/+/735217 | 14:54 |
| avass | zbr: I'll look more in a bit, but I think it's ok | 14:54 |
| *** rlandy|meeting is now known as rlandy | 15:07 | |
| *** hashar has quit IRC | 15:19 | |
| *** wuchunyang has joined #zuul | 15:31 | |
| *** wuchunyang has quit IRC | 15:36 | |
| *** sshnaidm is now known as sshnaidm|afk | 15:48 | |
| *** nilsph is now known as nils | 16:21 | |
| *** ykarel has quit IRC | 16:33 | |
| *** ykarel has joined #zuul | 16:33 | |
| avass | zbr: yeah that looks fine | 16:35 |
| zbr | thanks | 16:44 |
| *** wuchunyang has joined #zuul | 16:46 | |
| *** wuchunyang has quit IRC | 16:50 | |
| *** bhavikdbavishi has joined #zuul | 16:56 | |
| *** jpena is now known as jpena|off | 17:01 | |
| *** bhavikdbavishi1 has joined #zuul | 17:05 | |
| *** bhavikdbavishi has quit IRC | 17:08 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 17:08 | |
| clarkb | avass: done, left some questions/comments | 17:17 |
| clarkb | I think my biggest concern is the confusion around diskimage-import and vmimport in the docs and provider schema | 17:18 |
| clarkb | seems a bit inconsistent | 17:18 |
| corvus | i'd like to look at that too, but probably won't have a chance until tomorrow. | 17:19 |
| clarkb | corvus: I didn't +2 fwiw I expect it will still be there for you tomorrow | 17:21 |
| clarkb | well and now that you've asked to review it too I'm sure it will be :) | 17:21 |
| *** openstack has joined #zuul | 17:25 | |
| *** ChanServ sets mode: +o openstack | 17:25 | |
| *** ykarel has quit IRC | 17:28 | |
| *** jcapitao is now known as jcapitao_off | 17:29 | |
| pabelanger | we are looking at bringing online RHEL8 nodes in nodepool, for the point of discussion we'll only be able to launch 16 across all our providers in nodepool. Given we can use a semaphone on zuul side, for jobs. I am wondering if it also makes sense to have something like that on nodepool side? like, a global semaphore, where only x number of labels can be launched | 17:29 |
| clarkb | pabelanger: you could put them in their own pool and set a max-servers value? | 17:30 |
| pabelanger | that won't work across providers | 17:31 |
| pabelanger | unless we have per accounts for each region | 17:32 |
| clarkb | or divide your 16 by number of providers or something | 17:32 |
| pabelanger | yah, that is something we are looking at now, but the idea of a semaphore in nodepool was something I started to think about | 17:32 |
| clarkb | just thinking out loud here: maybe instead of a generic semaphore it would be sufficient to set label max-servers (we already set min-ready on labels) | 17:35 |
| *** vishalmanchanda has quit IRC | 17:38 | |
| tristanC | clarkb: label max-servers sounds like a great solution | 18:02 |
| tristanC | corvus: is https://review.opendev.org/c/zuul/zuul/+/630472 the last missing piece for zuul-v4, and when do we want to release it? | 18:04 |
| corvus | tristanC: that and some zk changes which are on my list; i'm going to look into those next, then i'll send a last-call email to the list to make sure we haven't missed anything we planned to deprecate. | 18:07 |
| corvus | tristanC: (i think you have already reviewed the zk changes) | 18:07 |
| *** hashar has joined #zuul | 18:07 | |
| tristanC | if not, i'd be happy to do that now | 18:07 |
| corvus | i think https://review.opendev.org/716262 is the cutoff | 18:08 |
| corvus | i'm expecting "very soon" for the release :) | 18:08 |
| tristanC | ok great, thank you for the feedback | 18:09 |
| *** rpittau is now known as rpittau|afk | 18:10 | |
| pabelanger | clarkb: okay, I can hack a little on that | 18:10 |
| pabelanger | see how difficult to do | 18:10 |
| *** hamalq has joined #zuul | 18:14 | |
| clarkb | corvus: https://review.opendev.org/c/zuul/zuul-jobs/+/764152 that might be a good one for you to review since I think you wrote the original workspace git mirroring. I feel like there is a lot of context in that process that I quite have paged in | 18:41 |
| corvus | clarkb: ack; starred for later review | 18:42 |
| tobiash | I've commented on that with a thought ^ | 19:15 |
| *** bhavikdbavishi has quit IRC | 19:33 | |
| *** ikhan has quit IRC | 19:37 | |
| masterpe | No talks about zuul this year on fosdem? | 19:47 |
| fungi | i've honestly been too busy with other things to look at the virtual fosdem lineup this year | 19:50 |
| clarkb | tobiash: related to your comment, I wonder if providing ansible with some timing data around real world perofrmance of things like loops and task execution would be beneficial | 19:54 |
| mordred | just tell Shrews - he can fix that for you now | 19:54 |
| clarkb | I think even just normal task execution has a fairly high overhead now | 19:55 |
| mordred | yeah - that file already does a bundling of tasks into a single multi-line shell later in the file | 19:56 |
| Shrews | sure, i'll "fix the glitch" | 19:56 |
| mordred | Shrews: just revert the patch that migrated to multiprocessing. easy | 19:58 |
| * Shrews fires mordred. there... fixed | 19:58 | |
| * mordred goes to work on his boat | 19:58 | |
| Shrews | ooh, i should check up on your channel. thx for the reminder | 19:58 |
| mordred | Shrews: nothing new yet | 19:59 |
| mordred | Shrews: I'll ping you when there is - backlogged on editting currently | 19:59 |
| * Shrews can't wait for some high seas drama | 19:59 | |
| *** akrpan-pure has joined #zuul | 20:00 | |
| akrpan-pure | What's the current way with gerrit in zuul to mark a response as "autogenerated" or "CI" so that the "Only comments" toggle hides them? I tried with `tag: autogenerated:ci` (with and without quotes) and Zuul just started spitting errors and not responding at all | 20:01 |
| clarkb | akrpan-pure: I think zuul automatically does the annotation if the gerrit is new enough | 20:04 |
| *** ikhan has joined #zuul | 20:04 | |
| fungi | yes, zuul added that roughly a year ago | 20:09 |
| akrpan-pure | "Zuul version: 3.19.1-1.el7" seem new enough? | 20:33 |
| akrpan-pure | I suppose that release is July 23rd 2020 so that's within the last year | 20:34 |
| akrpan-pure | I guess then my question now becomes "why aren't my CI responses getting filtered by that toggle"? | 20:34 |
| tristanC | akrpan-pure: is the issue on review.opendev.org? could you share a link where it happens? | 20:38 |
| akrpan-pure | tristanC: yeah, I've been working on ci-sandbox. https://review.opendev.org/c/opendev/ci-sandbox/+/768733 | 20:38 |
| corvus | akrpan-pure: are you using http to report or ssh? | 20:40 |
| corvus | akrpan-pure: that will only work with http authentication configured: https://zuul-ci.org/docs/zuul/reference/drivers/gerrit.html#attr-%3Cgerrit%20connection%3E.password | 20:41 |
| akrpan-pure | Ahhhh, looks to be SSH | 20:41 |
| akrpan-pure | Set up using software factory | 20:42 |
| tristanC | akrpan-pure: in sfconfig.yaml, you can enable http by setting password and auth_type in the gerrit connection dictionary of opendev | 20:43 |
| akrpan-pure | tristanC: Awesome, thanks! I'll give that a try once this test run is done | 20:43 |
| tristanC | actually, just setting a password should make zuul use http report | 20:44 |
| *** hashar has quit IRC | 21:07 | |
| *** nils has quit IRC | 21:27 | |
| *** jcapitao_off has quit IRC | 21:53 | |
| fungi | agreed, setting an auth password is what causes zuul to decide to use the gerrit rest api | 22:11 |
| *** ikhan has quit IRC | 22:36 | |
| *** rlandy has quit IRC | 23:15 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!