| *** rlandy|rover has quit IRC | 00:01 | |
| *** jamesmcarthur has quit IRC | 00:03 | |
| *** jamesmcarthur has joined #zuul | 00:04 | |
| *** jamesmcarthur has quit IRC | 00:08 | |
| *** jamesmcarthur has joined #zuul | 00:09 | |
| *** jamesmcarthur has quit IRC | 00:26 | |
| *** jamesmcarthur has joined #zuul | 00:29 | |
| *** jamesmcarthur has quit IRC | 00:33 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Add waiting status to status page https://review.opendev.org/c/zuul/zuul/+/788376 | 00:49 |
|---|---|---|
| *** jamesmcarthur has joined #zuul | 00:53 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Support overlapping repos and a flat workspace scheme https://review.opendev.org/c/zuul/zuul/+/787451 | 00:56 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Emit stats for semaphore holders https://review.opendev.org/c/zuul/zuul/+/788378 | 00:58 |
| *** jamesmcarthur has quit IRC | 01:02 | |
| *** jamesmcarthur has joined #zuul | 01:08 | |
| *** jamesmcarthur has quit IRC | 01:12 | |
| *** jamesmcarthur has joined #zuul | 01:18 | |
| *** josefwells has quit IRC | 01:22 | |
| *** hamalq has quit IRC | 01:30 | |
| *** jamesmcarthur has quit IRC | 01:43 | |
| *** jamesmcarthur has joined #zuul | 01:50 | |
| *** jamesmcarthur has quit IRC | 01:54 | |
| openstackgerrit | Merged zuul/zuul master: Support overlapping repos and a flat workspace scheme https://review.opendev.org/c/zuul/zuul/+/787451 | 02:06 |
| *** evrardjp has quit IRC | 02:33 | |
| *** evrardjp has joined #zuul | 02:33 | |
| *** bhavikdbavishi has joined #zuul | 03:03 | |
| *** bhavikdbavishi1 has joined #zuul | 03:08 | |
| *** bhavikdbavishi has quit IRC | 03:10 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:10 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: [wip] switch to using devstack's clouds.yaml https://review.opendev.org/c/zuul/nodepool/+/788684 | 03:17 |
| *** bhavikdbavishi has quit IRC | 03:23 | |
| *** bhavikdbavishi has joined #zuul | 03:25 | |
| *** sam_wan has joined #zuul | 03:30 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Remove nodepool-functional-openstack https://review.opendev.org/c/zuul/nodepool/+/788406 | 03:36 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Switch to using openstackclient https://review.opendev.org/c/zuul/nodepool/+/788684 | 03:36 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Switch functional test to Focal nodeset https://review.opendev.org/c/zuul/nodepool/+/788876 | 03:36 |
| *** bhavikdbavishi has quit IRC | 04:00 | |
| *** bhavikdbavishi has joined #zuul | 04:20 | |
| *** sam_wan has quit IRC | 04:21 | |
| *** bhavikdbavishi has quit IRC | 04:24 | |
| *** bhavikdbavishi has joined #zuul | 04:32 | |
| *** jamesmcarthur has joined #zuul | 04:37 | |
| *** jamesmcarthur has quit IRC | 04:37 | |
| *** ajitha has joined #zuul | 04:50 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Switch functional test to Focal nodeset https://review.opendev.org/c/zuul/nodepool/+/788876 | 05:24 |
| *** jfoufas1 has joined #zuul | 05:24 | |
| *** saneax has joined #zuul | 05:43 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Fix broken jobs overriding checkout to an unprotected branch https://review.opendev.org/c/zuul/zuul/+/788694 | 05:58 |
| *** jcapitao has joined #zuul | 06:31 | |
| *** bhavikdbavishi has quit IRC | 06:36 | |
| *** bhavikdbavishi has joined #zuul | 07:16 | |
| *** hashar has joined #zuul | 07:17 | |
| *** ricolin has quit IRC | 07:18 | |
| *** bhavikdbavishi1 has joined #zuul | 07:19 | |
| *** bhavikdbavishi has quit IRC | 07:21 | |
| *** sam_wan has joined #zuul | 07:21 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 07:21 | |
| *** rpittau|afk is now known as rpittau | 07:24 | |
| *** nils has joined #zuul | 07:26 | |
| *** jpena has joined #zuul | 07:35 | |
| *** tosky has joined #zuul | 08:21 | |
| *** hashar has quit IRC | 08:47 | |
| *** saneax has quit IRC | 08:59 | |
| *** bhavikdbavishi has quit IRC | 09:20 | |
| *** bhavikdbavishi has joined #zuul | 09:21 | |
| *** sam_wan has quit IRC | 09:33 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add search filters when relevant https://review.opendev.org/c/zuul/zuul-client/+/788847 | 09:33 |
| lyr | Hi there. What does this error means ? "Unable to freeze job graph: Pre-review pipeline experimental does not allow post-review job yarn-build-test". Zuul was ok with that until I added the missing secret | 10:25 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Provide statsd client to Nodepool and make scheduler optional https://review.opendev.org/c/zuul/zuul/+/774605 | 10:30 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Make NodeSet fully serializable https://review.opendev.org/c/zuul/zuul/+/774608 | 10:30 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Lock/unlock nodes on executor server https://review.opendev.org/c/zuul/zuul/+/774610 | 10:30 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Calculate build start and end time on executor server https://review.opendev.org/c/zuul/zuul/+/788915 | 10:30 |
| *** jcapitao has quit IRC | 10:32 | |
| *** bhavikdbavishi has quit IRC | 10:34 | |
| *** bhavikdbavishi has joined #zuul | 10:34 | |
| *** bhavikdbavishi has quit IRC | 10:35 | |
| *** bhavikdbavishi1 has joined #zuul | 10:35 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 10:37 | |
| *** bhavikdbavishi has quit IRC | 10:42 | |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Switch to ZooKeeper backed merge result events https://review.opendev.org/c/zuul/zuul/+/784195 | 10:49 |
| fungi | lyr: it means the yarn-build-test job has been set so it can only run in post-review pipelines (usually for safety so as not to expose credentials, does that job use any secrets?), but its been requested to run in the experimental pipeline which happens pre-review | 10:51 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Improve component registry https://review.opendev.org/c/zuul/zuul/+/787684 | 10:53 |
| openstackgerrit | Felix Edel proposed zuul/zuul master: Calculate statsd metrics via ComponentRegistry https://review.opendev.org/c/zuul/zuul/+/787685 | 10:53 |
| *** PrinzElvis has quit IRC | 11:01 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Bump dib to 3.11.0 https://review.opendev.org/c/zuul/nodepool/+/788937 | 11:25 |
| *** jpena is now known as jpena|lunch | 11:30 | |
| *** rlandy has joined #zuul | 11:42 | |
| *** sshnaidm is now known as sshnaidm|off | 11:59 | |
| lyr | fungi: yarn-build-test got run & post-run & secrets, and was indeed asked to run in experimental. Can live without experimental, but is it okay as periodic ? The secret usage I mean | 12:05 |
| avass | lyr: yep the periodic pipeline only run on branch heads (commits that are already merged) and should be post-review | 12:14 |
| lyr | thanks | 12:14 |
| fungi | i concur | 12:17 |
| *** hashar has joined #zuul | 12:31 | |
| *** bhavikdbavishi has joined #zuul | 12:33 | |
| *** jpena|lunch is now known as jpena | 12:34 | |
| *** bhavikdbavishi1 has joined #zuul | 13:28 | |
| *** bhavikdbavishi has quit IRC | 13:29 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 13:29 | |
| *** bhavikdbavishi has quit IRC | 13:44 | |
| *** josefwells has joined #zuul | 13:51 | |
| openstackgerrit | Merged zuul/nodepool master: Bump dib to 3.11.0 https://review.opendev.org/c/zuul/nodepool/+/788937 | 14:01 |
| *** rpittau is now known as rpittau|afk | 14:10 | |
| *** jamesmcarthur has joined #zuul | 14:13 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Store tenants in unparsed abide as dict https://review.opendev.org/c/zuul/zuul/+/771456 | 14:23 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Always get all configured project config files https://review.opendev.org/c/zuul/zuul/+/788962 | 14:23 |
| *** avass has quit IRC | 14:27 | |
| *** avass has joined #zuul | 14:28 | |
| *** bhavikdbavishi has joined #zuul | 14:35 | |
| *** bhavikdbavishi1 has joined #zuul | 14:38 | |
| *** bhavikdbavishi has quit IRC | 14:40 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 14:40 | |
| *** ssmashnuk has joined #zuul | 14:42 | |
| *** jamesmcarthur has quit IRC | 14:44 | |
| *** jamesmcarthur has joined #zuul | 14:46 | |
| *** jamesmcarthur has quit IRC | 14:53 | |
| *** jamesmcarthur has joined #zuul | 14:55 | |
| *** bhavikdbavishi has quit IRC | 15:02 | |
| *** bhavikdbavishi has joined #zuul | 15:02 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/759838 | 15:04 |
| *** jamesmcarthur has quit IRC | 15:05 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/788969 | 15:10 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 15:12 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/759838 | 15:12 |
| *** ssmashnuk has quit IRC | 15:12 | |
| *** jamesmcarthur has joined #zuul | 15:13 | |
| avass | mhu: ooh nice :) | 15:13 |
| mhu | avass: got a few goodies in the pipe to make zuul-client a usable alternative to the GUI | 15:14 |
| mhu | I think this one is pretty neat: display a job's console stream in your shell https://review.opendev.org/c/zuul/zuul-client/+/751238 | 15:15 |
| avass | mhu: yeah I saw both of those. I'm looking forward to being able to keep my entire workflow in the terminal :) | 15:17 |
| *** jamesmcarthur has quit IRC | 15:23 | |
| *** jamesmcarthur has joined #zuul | 15:24 | |
| *** jamesmcarthur has quit IRC | 15:27 | |
| *** jamesmcarthur has joined #zuul | 15:27 | |
| avass | what version of zuul is opendev on at the moment? | 15:30 |
| mordred | 4.2.1.dev27 5f9f597a | 15:31 |
| corvus | we're not running the latest zk patch yet; i'd like to restart with it today | 15:32 |
| clarkb | the last restart was tuesdayish | 15:33 |
| avass | mordred: uh, obviously I could have checked that myself :) | 15:33 |
| mordred | avass: :) | 15:33 |
| avass | corvus: you mean 782939? | 15:34 |
| corvus | avass: correct | 15:35 |
| *** hashar has quit IRC | 15:36 | |
| *** Shrews has joined #zuul | 15:41 | |
| *** jamesmcarthur has quit IRC | 15:43 | |
| *** jamesmcarthur has joined #zuul | 15:43 | |
| *** jamesmcarthur has quit IRC | 15:43 | |
| *** jamesmcarthur has joined #zuul | 15:47 | |
| *** bhavikdbavishi has quit IRC | 15:54 | |
| *** josefwells has quit IRC | 16:24 | |
| *** jfoufas1 has quit IRC | 16:27 | |
| *** jpena is now known as jpena|off | 16:38 | |
| mordred | mhu: reading your change-status subcommand - it made me wonder if there would be a way for the zuul-client to be able to use the state of a local git repo to figure out missing parameters. like, instead of having to provide --change - a local repo would at least know the sha - and maybe it would be possible for the client to get from that to the change ... just thinking out loud | 16:51 |
| avass | mordred: the github cli works something like that and isn't it possible to get to a change by searching with a commit in gerrit? | 16:53 |
| fungi | if your local repo included gerrit's refs/notes/review it could potentially figure it out that way, but would be a very gerrit-centric solution. i agree some commit id lookup through the zuul api could provide interesting glue options | 16:53 |
| avass | yeah at least you get to my change by entering: 834c2d001c19d97cd2cd6096f0f66c9f331c4a8e in the search field | 16:54 |
| fungi | there was a similar request earlier in the week to be able to dereference changes to commit ids, i think, for other reasons | 16:54 |
| fungi | or maybe it was in the other direction, i don't recall now | 16:54 |
| *** rlandy is now known as rlandy|training | 16:55 | |
| avass | mordred: you can do curl https://review.opendev.org/changes/834c2d001c19d97cd2cd6096f0f66c9f331c4a8e | 16:58 |
| avass | mhu: ^ | 16:59 |
| fungi | yeah, but that requires something to tell zuulclient where to find your gerrit | 16:59 |
| fungi | would be better if zuulclient could just ask zuul for the status of a commit id and get back status for a change/pull request if it corresponded to one | 17:00 |
| avass | fungi: right | 17:00 |
| fungi | and then automatically infer that commit from the current directory if it's in a git repository | 17:01 |
| avass | I was gonna say that you could try the "gerrit" or "origin" remote but that would require the api to be served on that and not just be used for ssh/git | 17:01 |
| fungi | it would also mean the client connecting to multiple apis | 17:02 |
| mhu | avass: what's that? the gerrit change id? or the commit hash? we'd want something that works regardless of the connection used (gerrit, git, gitlab, etc) | 17:03 |
| avass | mhu: commit hash | 17:03 |
| mhu | ok so that'd work for everything, neat | 17:03 |
| corvus | do people really leave their git repos sitting around in the state of a change they pushed up for review? i push up a change and then move on to something else | 17:03 |
| avass | only when I feel like slacking :) | 17:03 |
| fungi | i leave my git repos in multiple states i call "branches" ;) | 17:04 |
| mhu | we should actually do that for enqueue, dequeue etc as well, i'm never sure how to do it for a github change FWIW | 17:04 |
| fungi | but yes, point taken, inferring the item from the commit of the current repository state is likely of limited utility, more generally being able to look up queue items by commit id might still have other benefits | 17:05 |
| corvus | i just want us to spend our limited development and review bandwidth on things that will be used :) | 17:06 |
| mhu | I'll look into that later, the little one is calling - his periodic "sleep" job might have ended in a POOP_FAILURE | 17:07 |
| fungi | good luck inspecting that log | 17:07 |
| clarkb | mhu: I think the little ones call that SUCCESS | 17:07 |
| corvus | certainly worth keeping in mind for reasons why we might want to index the commit id in the db, but since we don't have it now, it's a few steps removed | 17:07 |
| avass | mhu: maybe he just had a jobs: [- poop] in his pipeline? | 17:08 |
| fungi | corvus: that was mostly my point. to do what mordred was suggesting we'd really need an additional index exposed via the api | 17:08 |
| fungi | (or at least used by the api for aliasing/lookups) | 17:09 |
| corvus | i guess incorporating that into an ide could be useful; or a shell prompt if you could do it in a non-laggy way | 17:11 |
| avass | it could be useful if you have fast jobs, however most jobs I see are at least >10min (usually up to an hour) | 17:13 |
| avass | oh but I was thinking about the console now, not just change-status | 17:14 |
| corvus | ah, that's potentially more useful as i think you're more likely to still be on a branch if you're insterested in live-streaming a log | 17:15 |
| avass | true | 17:15 |
| fungi | maybe also useful for zuul-runner integration? | 17:17 |
| *** hamalq has joined #zuul | 17:19 | |
| *** hamalq has quit IRC | 17:20 | |
| *** hamalq has joined #zuul | 17:21 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: WIP: Add ExecutorApi https://review.opendev.org/c/zuul/zuul/+/787832 | 17:25 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: WIP change zone handling in ExecutorApi https://review.opendev.org/c/zuul/zuul/+/787833 | 17:25 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: WIP Add some executor API tests https://review.opendev.org/c/zuul/zuul/+/788624 | 17:25 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: WIP Execute builds via ZooKeeper https://review.opendev.org/c/zuul/zuul/+/788988 | 17:25 |
| mordred | corvus, fungi, avass yeah - I was more thinking of console (good point about IDE). of course, having a file in the repo like the .gitreview file that could indicate zuul server, tenant and repo name would also help cut down on typing. BUT - as corvus said, we have limited dev resources | 17:53 |
| avass | can't the zuul-client also be configured with a zuul.conf? | 17:55 |
| *** rlandy|training is now known as rlandy | 18:01 | |
| *** fdegir has quit IRC | 18:04 | |
| *** icey has quit IRC | 18:04 | |
| *** shanemcd has quit IRC | 18:04 | |
| *** fdegir has joined #zuul | 18:07 | |
| *** icey has joined #zuul | 18:07 | |
| *** shanemcd has joined #zuul | 18:07 | |
| *** gouthamr has quit IRC | 18:08 | |
| *** gouthamr has joined #zuul | 18:09 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Add 'uuid' to 'src_dir' to allow parallel jobs for static nodes https://review.opendev.org/c/zuul/zuul/+/735981 | 18:16 |
| *** jamesmcarthur has quit IRC | 18:33 | |
| *** jamesmcarthur has joined #zuul | 18:34 | |
| *** jamesmcarthur has quit IRC | 18:49 | |
| *** jamesmcarthur has joined #zuul | 18:49 | |
| *** jamesmcarthur has quit IRC | 18:50 | |
| *** jamesmcarthur has joined #zuul | 18:50 | |
| *** Shrews has quit IRC | 19:02 | |
| mhu | avass: yes | 19:08 |
| mhu | you can specify the api url and tenant in the conf | 19:08 |
| mhu | and you can have as many subsections as "zuuls" you interact with | 19:08 |
| *** nils has quit IRC | 19:15 | |
| *** nils has joined #zuul | 19:28 | |
| *** nils has quit IRC | 19:29 | |
| *** nils has joined #zuul | 19:31 | |
| *** jamesmcarthur has quit IRC | 19:35 | |
| *** jamesmcarthur has joined #zuul | 19:35 | |
| openstackgerrit | James E. Blair proposed zuul/zuul master: WIP Execute builds via ZooKeeper https://review.opendev.org/c/zuul/zuul/+/788988 | 19:36 |
| *** jamesmcarthur has quit IRC | 19:40 | |
| mhu | looks like zuul.opendev.org is kaputt? | 19:48 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-client: test change-status command https://review.opendev.org/c/zuul/zuul/+/759840 | 19:50 |
| fungi | mhu: it's being restarted | 19:51 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: REST API: add api/tenant/{tenant}/jobs/running endpoint https://review.opendev.org/c/zuul/zuul/+/771648 | 19:53 |
| corvus | re-enqueing | 20:04 |
| corvus | wrong channel sorry | 20:04 |
| *** jamesmcarthur has joined #zuul | 20:06 | |
| *** jamesmcarthur has quit IRC | 20:20 | |
| *** jamesmcarthur has joined #zuul | 20:33 | |
| *** jamesmcarthur has quit IRC | 20:37 | |
| *** jamesmcarthur has joined #zuul | 20:50 | |
| *** jamesmcarthur has quit IRC | 20:56 | |
| *** ajitha has quit IRC | 20:59 | |
| *** jamesmcarthur has joined #zuul | 21:07 | |
| *** jamesmcarthur has quit IRC | 21:14 | |
| *** jamesmcarthur has joined #zuul | 21:26 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/759838 | 21:31 |
| *** jamesmcarthur has quit IRC | 21:32 | |
| *** erbarr has quit IRC | 21:37 | |
| *** erbarr has joined #zuul | 21:40 | |
| *** nils has quit IRC | 21:42 | |
| *** jamesmcarthur has joined #zuul | 21:44 | |
| *** rlandy has quit IRC | 21:47 | |
| *** jamesmcarthur has quit IRC | 21:52 | |
| *** jamesmcarthur has joined #zuul | 22:06 | |
| clarkb | corvus: tobiash I'm looking at https://review.opendev.org/c/zuul/zuul/+/788694/3/zuul/manager/__init__.py and noticing it feels odd that we add branches to a list without any tie back to their projects? I guess what that means is we'll update a branch in all projects if it is overridden in one of them? | 22:08 |
| *** jamesmcarthur has quit IRC | 22:14 | |
| corvus | clarkb: my understanding is basically: if any project specifies that branch, we better have a state for it for all the projects because we might end up checking it out | 22:14 |
| clarkb | ya as I tpyed it out I sort of realized that this is likely desirable | 22:15 |
| corvus | and i think the more the merrier; we just don't want all 10,000 branches of a branch-per-pr repo | 22:15 |
| corvus | looks like that's gtg so i +3d | 22:16 |
| *** jamesmcarthur has joined #zuul | 22:25 | |
| clarkb | https://review.opendev.org/c/zuul/zuul/+/788681 is an interesting one too | 22:29 |
| clarkb | and related fallout from the repo state changes | 22:29 |
| corvus | (i wouldn't consider that negative fallout; more of a bonus) | 22:29 |
| clarkb | ya not negative. Like shaking a tree and getting an apple | 22:30 |
| corvus | it's nice we can speed things up a bit right now, since we're probably about to slow them down by putting things in zk, before they speed up again by adding paralellism | 22:31 |
| corvus | gives us a little more headroom | 22:31 |
| clarkb | corvus: https://review.opendev.org/c/zuul/zuul/+/787751/2 you've approved this one but it can't land beacuse its parent is unreviewed. I'm reviewing the parent now if you also want to take a look | 22:32 |
| clarkb | oh wait you wrote the parent | 22:32 |
| corvus | lgtm! :) | 22:32 |
| corvus | that might be a good one to ask tobiash to sanity check? | 22:33 |
| clarkb | ya as a windows user that is probably a good idea | 22:33 |
| *** jamesmcarthur has quit IRC | 22:33 | |
| clarkb | tobiash: https://review.opendev.org/c/zuul/zuul/+/787743/2 specifically is the change to double check. It adds pywinrm to the zuul ansible installs by default | 22:35 |
| clarkb | pywinrm is MIT licensed and you only need optional build deps if using features like kerberos | 22:35 |
| avass | at least the ansible docs mention pywinrm: https://docs.ansible.com/ansible/latest/user_guide/windows_winrm.html#what-is-winrm we've been using it since we started using zuul and there's a testcase in zuul that mentions it: https://opendev.org/zuul/zuul/src/branch/master/tests/unit/test_executor.py#L934 | 22:36 |
| avass | :) | 22:37 |
| clarkb | corvus: any idea with what happened to this one https://review.opendev.org/c/zuul/zuul/+/764498 ? | 22:38 |
| corvus | clarkb: no idea | 22:39 |
| mhu | hey there zuul-maint, a few quick reviews for your consideration on zuul-client: | 22:46 |
| mhu | setuptools dependency https://review.opendev.org/c/zuul/zuul-client/+/783970 | 22:46 |
| mhu | better logging and CLI output https://review.opendev.org/c/zuul/zuul-client/+/788340 | 22:46 |
| mhu | output formatters https://review.opendev.org/c/zuul/zuul-client/+/788832 | 22:47 |
| mhu | I'd like these merged before the rest if possible | 22:47 |
| *** jamesmcarthur has joined #zuul | 22:47 | |
| clarkb | mhu: to be clear pbr does imply setuptools (it won't work without it); however, if you install a wheel then you may not have setuptools or pbr | 22:48 |
| clarkb | (or a distro apckage I suppose) | 22:48 |
| mhu | clarkb, yeah I did a pip install pbr and setuptools wasn't installed | 22:49 |
| clarkb | mhu: ya it is because setuptools installs pbr (via setup_requires) | 22:49 |
| mhu | I guess the test envs have setuptools installed by default | 22:49 |
| *** jamesmcarthur has quit IRC | 22:53 | |
| *** jamesmcarthur has joined #zuul | 23:07 | |
| clarkb | mhu: I think you can use https://docs.python.org/3/library/datetime.html#datetime.datetime.fromisoformat instead of dateutil.isoparse and avoid adding unnecessary deps (I left that in a comment on the change too) | 23:19 |
| clarkb | at least I'm assuming that the other direction is emitted by something like datetime.isoformat() and so we can reverse it this way | 23:19 |
| clarkb | (seems likely if we're only just now adding the dateutil dep) | 23:20 |
| openstackgerrit | Merged zuul/zuul master: Fix broken jobs overriding checkout to an unprotected branch https://review.opendev.org/c/zuul/zuul/+/788694 | 23:20 |
| *** jamesmcarthur has quit IRC | 23:21 | |
| *** tosky has quit IRC | 23:31 | |
| *** jamesmcarthur has joined #zuul | 23:36 | |
| *** jamesmcarthur has quit IRC | 23:41 | |
| openstackgerrit | Merged zuul/zuul master: [reporter][elasticsearch] fix the timestamp when the system has a different timezone by forcing the UTC timezone https://review.opendev.org/c/zuul/zuul/+/786444 | 23:53 |
| *** jamesmcarthur has joined #zuul | 23:55 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!