| *** vishalmanchanda has quit IRC | 00:09 | |
| *** tosky has quit IRC | 00:14 | |
| *** openstackgerrit has joined #zuul | 00:46 | |
| openstackgerrit | Merged zuul/zuul master: Report max 50 file comments to github https://review.opendev.org/c/zuul/zuul/+/762869 | 00:46 |
|---|---|---|
| *** piotrowskim has quit IRC | 01:04 | |
| *** rlandy has quit IRC | 01:31 | |
| *** hamalq has quit IRC | 01:39 | |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: implement support for diskimages https://review.opendev.org/c/zuul/nodepool/+/781187 | 03:30 |
| *** evrardjp has quit IRC | 03:33 | |
| *** evrardjp has joined #zuul | 03:33 | |
| *** ianychoi has quit IRC | 03:37 | |
| *** vishalmanchanda has joined #zuul | 04:16 | |
| *** ykarel has joined #zuul | 04:59 | |
| *** jfoufas1 has joined #zuul | 05:28 | |
| *** ykarel has quit IRC | 07:00 | |
| *** ykarel has joined #zuul | 07:02 | |
| *** hashar has joined #zuul | 07:30 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Improve test output by using named queues https://review.opendev.org/c/zuul/zuul/+/775620 | 08:21 |
| *** rpittau|afk is now known as rpittau | 08:24 | |
| *** ajitha has joined #zuul | 08:28 | |
| *** tosky has joined #zuul | 08:54 | |
| *** holser has joined #zuul | 09:00 | |
| *** holser has quit IRC | 09:02 | |
| *** holser has joined #zuul | 09:03 | |
| *** ykarel has quit IRC | 09:11 | |
| *** newbie2020 has joined #zuul | 09:23 | |
| newbie2020 | Hi guys, | 09:26 |
| newbie2020 | we have a setup with Zuul 3.18.0 + GitHub. | 09:26 |
| newbie2020 | Is it legit to have two PRs on the same repo/project and explicitly adding a 'Depends-On: from one pull request to the other? | 09:26 |
| newbie2020 | I do not see why not... | 09:26 |
| newbie2020 | But we are getting an "Unknown configuration message" and a colleague is insisting it is not possible to have Depends-On between PRs on the same repo :) | 09:26 |
| avass | newbie2020: it's working for gerrit so I don't see why you couldn't do that for github unless there's any limitation I'm not aware of | 09:29 |
| avass | I'd expect it to work at least | 09:30 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Test zuul-client builds subcommand https://review.opendev.org/c/zuul/zuul/+/758141 | 09:46 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: test zuul-client build-info https://review.opendev.org/c/zuul/zuul/+/758783 | 09:47 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-client: Test buildsets, buildset-info subcommands https://review.opendev.org/c/zuul/zuul/+/758985 | 09:48 |
| *** ricolin has quit IRC | 10:01 | |
| *** nils has joined #zuul | 10:13 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Test zuul-client builds subcommand https://review.opendev.org/c/zuul/zuul/+/758141 | 10:18 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: test zuul-client build-info https://review.opendev.org/c/zuul/zuul/+/758783 | 10:24 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-client: Test buildsets, buildset-info subcommands https://review.opendev.org/c/zuul/zuul/+/758985 | 10:26 |
| zbr | avass: any chance to +W https://review.opendev.org/c/zuul/zuul-jobs/+/773245 ? i am worried it will soon need a rebase if we do not. | 10:34 |
| *** hashar has quit IRC | 10:37 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-client: test change-status command https://review.opendev.org/c/zuul/zuul/+/759840 | 10:37 |
| *** harrymichal has joined #zuul | 10:40 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add api/tenant/{tenant}/jobs/running endpoint https://review.opendev.org/c/zuul/zuul/+/771648 | 10:41 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add show running-jobs subcommand https://review.opendev.org/c/zuul/zuul-client/+/771853 | 10:44 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add show running-jobs subcommand https://review.opendev.org/c/zuul/zuul-client/+/771853 | 10:45 |
| *** ykarel|lunch has joined #zuul | 10:45 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-client: test "show running-jobs" command https://review.opendev.org/c/zuul/zuul/+/771962 | 10:45 |
| newbie2020 | Thanks avass | 10:48 |
| *** ykarel|lunch is now known as ykarel | 11:06 | |
| avass | zbr: done | 11:16 |
| *** harrymichal has quit IRC | 11:17 | |
| zbr | thanks | 11:22 |
| *** harrymichal has joined #zuul | 11:35 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Upgrade ansible-lint to 5.0 https://review.opendev.org/c/zuul/zuul-jobs/+/773245 | 11:37 |
| *** rlandy has joined #zuul | 11:37 | |
| *** sassyn has joined #zuul | 12:00 | |
| *** ykarel has quit IRC | 12:06 | |
| *** ykarel has joined #zuul | 12:07 | |
| *** zbr is now known as zbr|rover | 12:30 | |
| *** ykarel has quit IRC | 12:35 | |
| *** ykarel has joined #zuul | 12:35 | |
| openstackgerrit | Benedikt Löffler proposed zuul/nodepool master: Add zookeeper-timeout connection config https://review.opendev.org/c/zuul/nodepool/+/752022 | 12:40 |
| *** ajitha has quit IRC | 13:09 | |
| *** jfoufas1 has quit IRC | 13:33 | |
| *** irclogbot_1 has quit IRC | 13:33 | |
| *** openstackgerrit has quit IRC | 13:33 | |
| *** irclogbot_1 has joined #zuul | 13:37 | |
| *** saneax has quit IRC | 13:40 | |
| *** hashar has joined #zuul | 14:06 | |
| corvus | nodepool: commit 9fb7c9bb19301b0eacfe30983ce5c0f8436948a2 (HEAD -> master, tag: 4.1.0, origin/master, gerrit/master) | 14:21 |
| corvus | zuul: commit 4bb45bf2a0223c1c624dbd8f44efff207e6b4097 (tag: 4.1.0, refs/changes/69/781169/1) | 14:21 |
| corvus | zuul-maint: ^ i would like to tag and release both of those today | 14:22 |
| corvus | (i think we need both released because there's a matching nodepool/zuul change with shell-type) | 14:22 |
| corvus | the zuul commit is what i restarted opendev on on wednesday; there are some changes since then, but they aren't critical | 14:23 |
| corvus | opendev has > 24h of zk performance data collected: https://grafana.opendev.org/d/5Imot6EMk/zuul-status?orgId=1 | 14:33 |
| corvus | so i think we can start merging the initial zk scheduler changes | 14:34 |
| corvus | we can restart opendev once those land, and if things go poorly, we'll be able to switch back to 4.1.0 easily | 14:34 |
| mordred | ++ | 14:40 |
| fungi | also opendev is still fairly busy (though not as busy as last week when everyone in openstack was scrambling to make the release feature freeze deadline) | 14:45 |
| corvus | fungi: yeah -- i think it's just the right amount of busy :) busy enough to have decent data, and not so busy that restarting later would be hugely disruptive. | 14:47 |
| corvus | anyone want to double check those shas? :) | 14:48 |
| fungi | looking | 14:49 |
| fungi | for some reason i have 6de1694f077cf56046997203593ed95313b8380e for zuul's origin/master | 14:51 |
| avass | same | 14:52 |
| fungi | oh, right, stuff has merged since what opendev is running | 14:52 |
| fungi | yeah, three changes in master newer than that point, but since it's what we've seen run in production 4bb45bf2a0223c1c624dbd8f44efff207e6b4097 should be good for 4.1.0 | 14:53 |
| *** ykarel has quit IRC | 14:53 | |
| fungi | and 9fb7c9bb19301b0eacfe30983ce5c0f8436948a2 looks right for nodepool | 14:54 |
| fungi | corvus: lgtm | 14:54 |
| corvus | cool, i'll push those now | 14:55 |
| tristanC | corvus: would it be possible to merge the zuul-runner change first? | 14:58 |
| *** ykarel has joined #zuul | 15:08 | |
| *** rpittau is now known as rpittau|afk | 15:08 | |
| *** chkumar|ruck is now known as raukadah | 15:24 | |
| *** openstackgerrit has joined #zuul | 15:36 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add build-info subcommand https://review.opendev.org/c/zuul/zuul-client/+/751070 | 15:36 |
| fungi | tristanC: which zuul-runner change, and first before what? | 15:37 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/759838 | 15:42 |
| *** holser has quit IRC | 16:05 | |
| *** ykarel has quit IRC | 16:16 | |
| *** ykarel has joined #zuul | 16:17 | |
| *** holser has joined #zuul | 16:18 | |
| tristanC | fungi: the changes that mostly refactor the executor code, e.g. 607078, 607079, 648642 and 648643 | 16:20 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Resync github permissions docs https://review.opendev.org/c/zuul/zuul/+/781793 | 16:22 |
| fungi | tristanC: i think we wanted to run whatever we're going to tag in production on opendev before tagging, so if you're asking to delay 4.1.0 to include those changes, it probably means another zuul restart in opendev. we could always add them in a 4.1.1 or 4.2.0 in a few days though, right? now that we're over the 4.0.0 hump we can go back to releasing as often as is convenient | 16:28 |
| *** ykarel is now known as ykarel|away | 16:29 | |
| tristanC | fungi: oh i meant before other core changes, the 4.1.0 tag looks good to me | 16:30 |
| fungi | tristanC: aha, thanks. i was lacking context on "first before what" | 16:31 |
| *** hashar has quit IRC | 16:33 | |
| avass | so how much of the sos stack is planned to be merged now? | 16:48 |
| corvus | i think we're ready for 761163 through 761738 | 16:50 |
| corvus | that moves trigger and management events into zk; we'll be able to observe how that affects performance | 16:50 |
| corvus | (things will probably get a little bit worse before they're able to get better by having more than one scheduler, so a big question is how much worse) | 16:51 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API, Web UI: add pipelines' manager, triggers data in status https://review.opendev.org/c/zuul/zuul/+/736968 | 16:55 |
| avass | I suppose the java gc could be a performance hit there for big deployments | 16:58 |
| mordred | corvus: https://review.opendev.org/c/zuul/zuul/+/761164/29 has a failed job | 17:05 |
| mordred | corvus: it looks unhappy - but I think it was just a test oops and is not related | 17:07 |
| corvus | mordred: yeah, though i don't think it's related to that change | 17:08 |
| mordred | agree | 17:08 |
| corvus | could be that tests.unit.test_scheduler.TestSchedulerZoneFallback.test_jobs_executed is flakey, caused a timeout, then a cascading failure | 17:10 |
| corvus | i have re-enqueued the zuul release because the docker build failed (npm mirror issues) | 17:10 |
| clarkb | note if the failure was installing nodeenv it bypasses the npm mirrors iirc | 17:12 |
| corvus | "https://registry.yarnpkg.com/@patternfly/react-tokens/-/react-tokens-4.9.12.tgz: ESOCKETTIMEDOUT". | 17:13 |
| corvus | https://zuul.opendev.org/t/zuul/build/17448085f19a46208aa650c204107037 | 17:13 |
| corvus | so yeah, maybe not the opendev mirrors but the upstream repo | 17:14 |
| clarkb | that looks to have bypassed the mirror too? | 17:14 |
| corvus | clarkb: probably because it's inside the docker build? | 17:18 |
| corvus | i can't remember where we got to on mirrors-in-docker-builds | 17:18 |
| clarkb | ya that could be, but also the js tools have bad habits of ignoring mirror settings ime | 17:19 |
| clarkb | the nodeenv stuff doesn't even check if they exist iirc | 17:19 |
| clarkb | they just hardcode urls in their toolign :( | 17:19 |
| fungi | it's the javascript way | 17:20 |
| fungi | probably they expect you to vendor your own fork of their tools if you need a different url | 17:21 |
| *** ykarel|away has quit IRC | 17:31 | |
| mordred | fungi: or fork their vendored fork | 17:34 |
| fungi | oh, yep that | 17:39 |
| *** newbie2020 has quit IRC | 17:40 | |
| *** hamalq has joined #zuul | 17:41 | |
| openstackgerrit | Merged zuul/zuul master: Move management and result events to model https://review.opendev.org/c/zuul/zuul/+/761163 | 18:10 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add a state machine driver framework https://review.opendev.org/c/zuul/nodepool/+/779071 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add an example state machine driver https://review.opendev.org/c/zuul/nodepool/+/779072 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Improve state machine keyscan handling https://review.opendev.org/c/zuul/nodepool/+/779379 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Support threadless deletes https://review.opendev.org/c/zuul/nodepool/+/779637 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Add azure state machine driver https://review.opendev.org/c/zuul/nodepool/+/779420 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: delete disk based on VM reference https://review.opendev.org/c/zuul/nodepool/+/779897 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: reconcile config objects https://review.opendev.org/c/zuul/nodepool/+/779898 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: Handle IPv6 https://review.opendev.org/c/zuul/nodepool/+/780400 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: don't require full subnet id https://review.opendev.org/c/zuul/nodepool/+/780402 | 18:12 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: add quota support https://review.opendev.org/c/zuul/nodepool/+/780439 | 18:13 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: implement launch retries https://review.opendev.org/c/zuul/nodepool/+/780682 | 18:13 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: implement support for diskimages https://review.opendev.org/c/zuul/nodepool/+/781187 | 18:13 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: handle leaked image upload resources https://review.opendev.org/c/zuul/nodepool/+/781855 | 18:13 |
| openstackgerrit | James E. Blair proposed zuul/nodepool master: Azure: use rate limiting https://review.opendev.org/c/zuul/nodepool/+/781856 | 18:13 |
| corvus | another zuul release failure: 1mE: Failed to fetch https://deb.nodesource.com/node_10.x/pool/main/n/nodejs/nodejs_10.24.0-1nodesource1_amd64.deb Could not connect to deb.nodesource.com:443 (104.123.50.113), connection timed out Could not connect to deb.nodesource.com:443 (104.123.50.89), connection timed out | 18:13 |
| corvus | re-enqueing | 18:13 |
| mordred | corvus: jeez | 18:16 |
| mordred | corvus: that's the nodeenv stuff clarkb was mentioning before probably right? | 18:19 |
| openstackgerrit | Merged zuul/zuul master: Allow (de-)serialization of management events https://review.opendev.org/c/zuul/zuul/+/761164 | 18:21 |
| openstackgerrit | Merged zuul/zuul master: Add and fix fields in driver trigger event models https://review.opendev.org/c/zuul/zuul/+/761166 | 18:22 |
| openstackgerrit | Merged zuul/zuul master: Allow (de-)serialization of trigger events https://review.opendev.org/c/zuul/zuul/+/761167 | 18:23 |
| mordred | corvus: did you re-enqueue it before it reported? I was looking for the failure in zuul and don't see it | 18:24 |
| openstackgerrit | Merged zuul/zuul master: Interface to get a driver's trigger event class https://review.opendev.org/c/zuul/zuul/+/761168 | 18:24 |
| corvus | mordred: this one https://zuul.opendev.org/t/zuul/buildset/637952fb323b4350bef44f1b30aaa29a | 18:25 |
| mordred | corvus: ahhh - thanks | 18:26 |
| clarkb | mordred: ya I think that is how it manifests | 18:26 |
| mordred | well - fwiw, this isn't even nodeenv or other javascript hardcoding - this is us hardcoding the nodesource deb repo in install-js-tools | 18:27 |
| mordred | I don't know that we'd get it right in the Dockerfile if we improved install-js-tools - that would be a whole other layer of work | 18:27 |
| mordred | that said ... https://review.opendev.org/c/zuul/zuul/+/728690 would potentially make this part a bit more resilient | 18:30 |
| clarkb | https://github.com/ekalinin/nodeenv/blob/master/nodeenv.py#L1042-L1053 looks like nodeenv did add a flag for setting a mirror | 18:32 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: allow a privileged user to promote a change https://review.opendev.org/c/zuul/zuul/+/781858 | 18:32 |
| mordred | clarkb: yah - but in this case we're not using nodeenv | 18:33 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | 18:34 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Use node image to build javascript https://review.opendev.org/c/zuul/zuul/+/728690 | 18:34 |
| openstackgerrit | Monty Taylor proposed zuul/zuul master: Use node image to build javascript https://review.opendev.org/c/zuul/zuul/+/728690 | 18:35 |
| *** hamalq has quit IRC | 18:36 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | 18:36 |
| *** hamalq has joined #zuul | 18:36 | |
| mordred | clarkb: I thnik three different mirrors we'd need in the general case - one for npm - but also one for the node deb repo and one for the yarn deb repo | 18:37 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | 18:39 |
| mhu | so I've got a baby pretty much about to pop in the gate pipeline, most likely next week ... the admin web ui patch chain at https://review.opendev.org/q/topic:%22fffaff%22 should be in a reviewable state; the docker compose has everything except for the autohold(s) page | 18:43 |
| mordred | mhu: so when you say baby here, you mean a small human not a patch series you've been working on | 18:44 |
| mhu | if someone wants to take over while I am knee deep in diapers feel free to | 18:44 |
| mhu | mordred, is there a difference? | 18:44 |
| mordred | hahaha. I believe one produces more poop - otherwise, no? | 18:44 |
| mordred | congratulations in advance! | 18:44 |
| mhu | but yeah, my gf has been working hard on releasing mhu 2.0 | 18:44 |
| mordred | I'm sure she'll appreciate being called a gate pipeline ;) | 18:45 |
| mhu | she moderately appreciated the zuul sticker on her belly! | 18:46 |
| mordred | mhu: how long before 2.0 starts submitting patches - we can use more hands around here | 18:46 |
| mordred | hahahaha! nice | 18:46 |
| mhu | mordred: pretty much on day 1, if you're not picky about code quality | 18:47 |
| corvus | mordred: wait, which one makes poop again? :) | 18:47 |
| mordred | corvus: yes | 18:47 |
| corvus | mhu: that's great news! :) | 18:47 |
| clarkb | the day they take care of poop themselves is a glorious day | 18:48 |
| mhu | corvus, thanks! with less sleeping time, I can code more! | 18:48 |
| corvus | that's really the whole idea of test-driven development, right? "clean up your own poop" | 18:49 |
| clarkb | corvus: yes, but it comes with emergent behaviors | 18:51 |
| *** rlandy is now known as rlandy|training | 18:57 | |
| *** harrymichal has quit IRC | 19:02 | |
| *** harrymichal has joined #zuul | 19:22 | |
| *** vishalmanchanda has quit IRC | 19:35 | |
| corvus | okay, the docker 410 job finally succeeded, so both codebases are now released. i'll send out notices shortly. | 19:59 |
| *** rlandy|training is now known as rlandy | 20:04 | |
| openstackgerrit | Merged zuul/zuul master: Implementation of Zookeeper backed event queues https://review.opendev.org/c/zuul/zuul/+/761170 | 20:59 |
| *** rlandy has quit IRC | 21:15 | |
| *** openstackgerrit has quit IRC | 21:22 | |
| *** harrymichal has quit IRC | 21:59 | |
| *** dpawlik has quit IRC | 22:11 | |
| *** dpawlik9 has joined #zuul | 22:12 | |
| *** dpawlik9 is now known as dpawlik | 22:13 | |
| *** kgz has quit IRC | 22:13 | |
| *** tobiash has quit IRC | 22:13 | |
| *** tobiash has joined #zuul | 22:14 | |
| *** mgoddard has quit IRC | 22:16 | |
| *** jpena|off has quit IRC | 22:16 | |
| *** kgz has joined #zuul | 22:17 | |
| *** jpena|off has joined #zuul | 22:18 | |
| *** mgoddard has joined #zuul | 22:19 | |
| *** nils has quit IRC | 22:39 | |
| *** sshnaidm|off has quit IRC | 22:54 | |
| *** hamalq has quit IRC | 23:05 | |
| *** holser has quit IRC | 23:49 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!