| *** zenkuro has quit IRC | 00:41 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: PF4 minor rework of log viewer page https://review.opendev.org/751140 | 01:11 |
|---|---|---|
| *** Goneri has quit IRC | 01:42 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-kubernetes: workaround slow account creation https://review.opendev.org/752184 | 03:09 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Retire Fedora 31 for 32 https://review.opendev.org/751975 | 03:10 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: only run docker-setup.yaml when installed https://review.opendev.org/747062 | 03:10 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: update-json-file: add role to combine values into a .json https://review.opendev.org/746834 | 03:10 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: Linaro MTU workaround https://review.opendev.org/747063 | 03:10 |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: PF4 minor rework of log viewer page https://review.opendev.org/751140 | 04:36 |
| *** vishalmanchanda has joined #zuul | 04:45 | |
| *** saneax has joined #zuul | 05:39 | |
| *** reiterative has quit IRC | 05:42 | |
| *** reiterative has joined #zuul | 05:42 | |
| openstackgerrit | Merged zuul/zuul master: Ignore subdirs with .zuul.ignore on config read https://review.opendev.org/744811 | 05:52 |
| *** mach1na has joined #zuul | 05:57 | |
| *** CraigR has joined #zuul | 06:09 | |
| *** mach1na has quit IRC | 06:12 | |
| openstackgerrit | Merged zuul/zuul-jobs master: ensure-kubernetes: workaround slow account creation https://review.opendev.org/752184 | 06:39 |
| *** jpena|off is now known as jpena | 06:43 | |
| *** hashar has joined #zuul | 06:54 | |
| *** jcapitao has joined #zuul | 07:02 | |
| *** armstrongs has joined #zuul | 07:03 | |
| *** CraigR has quit IRC | 07:07 | |
| *** armstrongs has quit IRC | 07:12 | |
| *** mach1na has joined #zuul | 07:24 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data https://review.opendev.org/742746 | 07:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/742747 | 07:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/742748 | 07:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/739198 | 07:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/742749 | 07:25 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/739078 | 07:25 |
| Pilou | tobiash: can you take a look at https://review.opendev.org/#/c/744814/, please ? | 07:28 |
| openstackgerrit | Pierre-Louis Bonicoli proposed zuul/zuul-jobs master: Use ansible_distribution* facts instead of ansible_lsb https://review.opendev.org/742310 | 07:31 |
| *** mach1na has quit IRC | 07:39 | |
| *** mach1na has joined #zuul | 07:40 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: consistent font sizes on console page and PF4 for task summary https://review.opendev.org/752383 | 07:46 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: consistent font sizes on console page and PF4 for task summary https://review.opendev.org/752383 | 07:47 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: update-json-file: add role to combine values into a .json https://review.opendev.org/746834 | 07:53 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: Linaro MTU workaround https://review.opendev.org/747063 | 07:53 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Retire Fedora 31 for 32 https://review.opendev.org/751975 | 07:57 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: only run docker-setup.yaml when installed https://review.opendev.org/747062 | 07:57 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: update-json-file: add role to combine values into a .json https://review.opendev.org/746834 | 07:57 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-docker: Linaro MTU workaround https://review.opendev.org/747063 | 07:57 |
| ianw | tristanC: ^ could you look in on https://review.opendev.org/751975 which drops the copr repo and tries to use the packaged fedora haskell, which is what i understand we're supposed to do from the upstream bug? | 07:58 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: [DNM] Add zuul-client to requirements https://review.opendev.org/752385 | 07:59 |
| * zbr is grateful for ianw for the fedora 32 change | 08:25 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add mode to packer https://review.opendev.org/752213 | 08:31 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add managed jobs to periodic-weekly https://review.opendev.org/748682 | 08:33 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality https://review.opendev.org/709735 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Separate connection registries in tests https://review.opendev.org/712958 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler https://review.opendev.org/717269 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests https://review.opendev.org/721254 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Driver event ingestion https://review.opendev.org/717299 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper https://review.opendev.org/716221 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper https://review.opendev.org/716875 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 09:03 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: WIP: Switch to using zookeeper instead of gearman for jobs (keep gearman for mergers) https://review.opendev.org/744416 | 09:03 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/691253 | 09:09 |
| *** nils has joined #zuul | 09:15 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Support per branch change queues https://review.opendev.org/718531 | 09:16 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Move queue from pipeline to project https://review.opendev.org/720182 | 09:16 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/685354 | 09:16 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Check cycle items are mergeable before reporting https://review.opendev.org/743450 | 09:16 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/691253 | 09:16 |
| *** zenkuro has joined #zuul | 09:18 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: bindep: perform serial install if needed https://review.opendev.org/693637 | 09:37 |
| *** holser has quit IRC | 09:38 | |
| *** holser has joined #zuul | 10:03 | |
| *** jcapitao is now known as jcapitao_lunch | 10:05 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: bindep: perform serial install if needed https://review.opendev.org/693637 | 10:06 |
| *** tosky has joined #zuul | 10:19 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: [DNM] Add zuul-client to requirements https://review.opendev.org/752385 | 10:28 |
| *** vishalmanchanda has quit IRC | 10:55 | |
| *** wuchunyang has joined #zuul | 11:05 | |
| *** mach1na has quit IRC | 11:16 | |
| *** jpena is now known as jpena|lunch | 11:37 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Remove an unneeded api call when creating check_runs https://review.opendev.org/752056 | 11:44 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Save superfluous api requests in check run reporting https://review.opendev.org/752079 | 11:44 |
| *** jcapitao_lunch is now known as jcapitao | 11:48 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Save superfluous api requests in check run reporting https://review.opendev.org/752079 | 11:59 |
| *** mach1na has joined #zuul | 12:00 | |
| *** mach1na has quit IRC | 12:03 | |
| *** wuchunyang has quit IRC | 12:04 | |
| *** mach1na has joined #zuul | 12:04 | |
| *** rlandy has joined #zuul | 12:09 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings https://review.opendev.org/752443 | 12:09 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: [DNM] Add zuul-client to requirements https://review.opendev.org/752385 | 12:11 |
| tobiash | Pilou: thanks, lgtm | 12:13 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings https://review.opendev.org/752443 | 12:17 |
| zbr | avass: tobiash: can you do https://review.opendev.org/#/c/751975/ ? -- have at least 4-5 patches blocked by it. | 12:18 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Save superfluous api requests in check run reporting https://review.opendev.org/752079 | 12:22 |
| *** holser has quit IRC | 12:22 | |
| *** rfolco has joined #zuul | 12:23 | |
| *** rfolco is now known as rfolco|ruck | 12:24 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings https://review.opendev.org/752443 | 12:25 |
| *** jpena|lunch is now known as jpena | 12:30 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: [DNM] Add zuul-client to requirements https://review.opendev.org/752385 | 12:34 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings https://review.opendev.org/752443 | 12:44 |
| *** holser has joined #zuul | 12:45 | |
| *** Goneri has joined #zuul | 12:49 | |
| *** iurygregory has quit IRC | 12:56 | |
| *** iurygregory has joined #zuul | 12:58 | |
| *** holser has quit IRC | 13:21 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: [DNM] Add zuul-client to requirements https://review.opendev.org/752385 | 13:29 |
| openstackgerrit | Merged zuul/zuul-jobs master: Retire Fedora 31 for 32 https://review.opendev.org/751975 | 13:32 |
| *** holser has joined #zuul | 13:38 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings https://review.opendev.org/752443 | 13:56 |
| *** hashar has quit IRC | 14:10 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Add logging and monitoring of reporting timings https://review.opendev.org/752443 | 14:25 |
| mhu | fungi, corvus: with this patch chain zuul-client is ready for a corrective release: https://review.opendev.org/#/q/status:open+topic:zuul-client_0.0.2 | 14:33 |
| mhu | the last patch of the lot, https://review.opendev.org/752385 requires a new tag for zuul-client, but the tests results show that integration with zuul is working | 14:34 |
| *** pabelanger has joined #zuul | 14:56 | |
| pabelanger | morning, we might have a need to do multi node testing across providers, I know this has been discussed before and we have reasons of single region today (less hops). Has anyone started work to support this with a flag | 14:58 |
| tobiash | pabelanger: this needs a redesign of the nodepool protocol since we currently have one provider, one node request | 15:00 |
| tobiash | multi node across providers would either require partial node request processing on nodepool side of the ability to have more than one nodeset on a job to split the requests on zuul's side | 15:02 |
| tobiash | I guess both options can be rather complicated | 15:02 |
| pabelanger | Hmm, okay | 15:02 |
| pabelanger | thank is what I suspected | 15:03 |
| fungi | i wonder if your use case could be solved by some mechanism for a merged provider? you're trying to mix devices which use different drivers, right? | 15:03 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: WIP: E208 work https://review.opendev.org/748606 | 15:04 |
| fungi | no idea if that would be any easier to implement than cross-provider multinode | 15:04 |
| pabelanger | fungi:yah, that is right. We have some resources in aws, and the rest in openstack | 15:04 |
| pabelanger | we have to use aws to get rhel8 | 15:04 |
| pabelanger | but do not want to move all our network appliances into aws | 15:04 |
| pabelanger | we may have too however | 15:04 |
| tobiash | I could imagine another ugly hack involving job pause | 15:05 |
| fungi | hrm... or a meta-provider? basically a logical provider layer which serves as a request proxy to get nodes from one or more real providers | 15:06 |
| fungi | that might get really crazy though | 15:06 |
| tobiash | you could have the network appliance started and paused in a parent job, then the other ones in a child job using add host to also use the network applience | 15:06 |
| tobiash | yeah, really crazy | 15:07 |
| tobiash | but that could work today | 15:07 |
| tobiash | I guess I'd prefer the meta-provider | 15:09 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Change Pipeline.getAllItems to generator https://review.opendev.org/752493 | 15:35 |
| *** mach1na has quit IRC | 15:36 | |
| pabelanger | tobiash: yah, was thinking of that too | 15:40 |
| pabelanger | okay, thanks for now | 15:41 |
| *** hashar has joined #zuul | 15:44 | |
| clarkb | dib made a new release recently to fix some suse build problems. Would be good to have a nodepool change land so that we'll pick that up in the docker images. https://review.opendev.org/#/c/747277/ is one of mine that is possibly ready to land. As an alternative I can push up a noop type change that is easy to land | 15:50 |
| clarkb | if others have an opinion on the preferable choice there let me know and I'll help move that along | 15:51 |
| *** holser has quit IRC | 15:56 | |
| *** jpena is now known as jpena|off | 15:57 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add mode to packer https://review.opendev.org/752213 | 16:21 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: Add file mode to fetch-sphinx-tarball https://review.opendev.org/748606 | 16:29 |
| *** jcapitao has quit IRC | 16:31 | |
| *** holser has joined #zuul | 16:43 | |
| *** hashar has quit IRC | 16:55 | |
| *** tosky has quit IRC | 17:00 | |
| *** nils has quit IRC | 17:09 | |
| *** pabelanger has left #zuul | 17:09 | |
| clarkb | fungi: any thoughts on a nodepool change to get a dib update in the nodepool docker images? https://review.opendev.org/#/c/747277/ has a +2 from tobiash and I'm also happy to push up a more noopy change | 17:21 |
| fungi | i'll take a look | 17:21 |
| fungi | is the iterate_timeout suggestion something worth adding in a followup? | 17:22 |
| clarkb | ya I'll push that as a followup now | 17:23 |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Followup test improvement to use iterate_timeout https://review.opendev.org/752539 | 17:35 |
| clarkb | fungi: tobiash ^ followup pushed | 17:35 |
| fungi | thanks! | 17:42 |
| fungi | running https://zuul.opendev.org/t/openstack/stream/92e8f3352fa84ba7b21167d22962970c?logfile=console.log | 17:43 |
| fungi | er, wrong channel, sorry | 17:43 |
| *** irclogbot_2 has quit IRC | 17:58 | |
| *** hamalq has joined #zuul | 17:59 | |
| *** saneax has quit IRC | 18:00 | |
| *** irclogbot_3 has joined #zuul | 18:02 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Add mode to packer https://review.opendev.org/752213 | 18:52 |
| corvus | mhu, fungi: reviewed https://review.opendev.org/751264 | 19:08 |
| mhu | corvus, thanks, will look tomorrow! | 19:11 |
| fungi | i concur. i think the original intent was making sure the package could be installed from master (because the released one can't, or at least doesn't install any modules) | 19:12 |
| fungi | and also mhu was saying something about wanting to use zuul-client in a job for zuul, so i'm guessing it's just not fleshed out yet | 19:12 |
| fungi | ahh, yeah, in the cli tests | 19:18 |
| *** mattd01 has joined #zuul | 19:20 | |
| *** openstackgerrit has quit IRC | 19:21 | |
| *** hashar has joined #zuul | 19:27 | |
| *** openstackgerrit has joined #zuul | 19:29 | |
| openstackgerrit | Clark Boylan proposed zuul/nodepool master: Use rax.dfw mirror for arm64 wheels https://review.opendev.org/752553 | 19:29 |
| clarkb | fungi: I've reapproved the changes after double checking the linaro mirror seems up and ^ should make things more reliable until we figure out why that linaro mirror keeps shutting down | 19:30 |
| fungi | ahh, okay, so it was that they hadn't merged yet. got it | 19:30 |
| clarkb | ya they failed to merge due to the arm64 mirror problem | 19:31 |
| clarkb | https://review.opendev.org/752553 addresses that and we should be good for the reappoval builds short term | 19:31 |
| fungi | when you mentioned enqueuing them again i thought you meant manually | 19:31 |
| clarkb | ah | 19:31 |
| clarkb | are there any security concerns with https://review.opendev.org/#/c/744814/2 ? Like possibly passing in directory/file input to a command that would allow a previously assumed safe command to do unsafe things? I think the answer is no because we don't filter shell/command like that we only filter other tasks that properly operate on files | 19:42 |
| fungi | digging into the history of our callback fork there, it's not obvious to me that setting the command's stdin to the supplied data is necessarily under any more direct control than setting the command to be run, but i'm not entirely sure i grok all the code in there | 20:01 |
| clarkb | this is incredibly weird, now it seems to be building a wheel for pynacl. It grabbed version 1.4.0's sdist which we have a wheel for :/ | 20:33 |
| *** tosky has joined #zuul | 20:34 | |
| clarkb | I wonder if there was a network error to the wheel mirror making it not see that wheel | 20:35 |
| clarkb | https://review.opendev.org/752553's check job seems to be happier | 20:35 |
| clarkb | yes finally managed to confirm it grabbed the wheel in ^ | 20:39 |
| clarkb | so maybe we need to land that first then the other two can happen | 20:39 |
| *** mattd01 has quit IRC | 20:45 | |
| clarkb | so ya I think https://review.opendev.org/752553 may need to happen first | 20:47 |
| *** hashar has quit IRC | 20:55 | |
| *** holser has quit IRC | 21:15 | |
| *** jhesketh has quit IRC | 21:55 | |
| *** rlandy has quit IRC | 22:03 | |
| *** tosky has quit IRC | 22:19 | |
| openstackgerrit | Clark Boylan proposed zuul/zuul master: Clean up stale git index.lock files on merger startup https://review.opendev.org/751370 | 22:19 |
| clarkb | tobiash: ^ now with testing :) figuring out how to test that was a bit tricky since we need a running merger to bootstrap tseting | 22:19 |
| clarkb | anyways I eventually realized I could just modify the state once the test was happy and go from there | 22:19 |
| clarkb | I had tried to start with disabling the executor's merger but if you do that you just hang the test | 22:20 |
| clarkb | ianw: if you have time for https://review.opendev.org/752553 that would be good as the linaro flakyness has made the nodepool jobs unhappy | 22:33 |
| clarkb | I think if we land ^ then we can land https://review.opendev.org/#/c/747277/ and its child. That will then get us new dib in the docker images | 22:33 |
| ianw | clarkb: the mirror is back though right? or are there other issues than it turning off? | 22:34 |
| ianw | i've realised i got distracted and didn't setup the netconsole :/ | 22:34 |
| clarkb | yes, I reran the jobs for those other changes and it didnt find the pynacl wheel on that mirror | 22:35 |
| clarkb | the job for thr rax url update did | 22:35 |
| clarkb | I assume the fetch against linaro mirror errored but unfortunately it didnt log that | 22:35 |
| clarkb | I did verify I could reach the linaro mirror from home before that recheck so its a weird one | 22:37 |
| *** mgagne has joined #zuul | 22:39 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: consistent font sizes on console page and PF4 for task summary https://review.opendev.org/752383 | 23:07 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: consistent font sizes on console page and PF4 for task summary https://review.opendev.org/752383 | 23:08 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: PF4 minor rework of log viewer page https://review.opendev.org/751140 | 23:11 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: consistent font sizes on console page and PF4 for task summary https://review.opendev.org/752383 | 23:11 |
| ianw | the linter enforcing no ";" in the jsx code is to me, an old C programmer, quite annoying | 23:12 |
| ianw | it seems like we should have made it "don't rely on javascript's magic ; interpolation" rather than "you must rely on it" | 23:13 |
| ianw | anyway, /rant | 23:13 |
| ianw | sorry, syntax error </Rant> | 23:14 |
| fungi | "it works without the ; so think of all the puppies^Wbytes you just saved!" (which the uglifier would have put back anyway when it strung the lines together into one huge unreadable mess) | 23:25 |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: consistent font sizes on console page and PF4 for task summary https://review.opendev.org/752383 | 23:26 |
| *** jhesketh has joined #zuul | 23:31 | |
| *** saneax has joined #zuul | 23:34 | |
| *** rfolco|ruck has quit IRC | 23:42 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul master: web: Don't use filled tabs for results page https://review.opendev.org/752583 | 23:44 |
| *** Goneri has quit IRC | 23:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!