| *** Goneri has quit IRC | 00:09 | |
| *** Goneri has joined #zuul | 00:15 | |
| *** armstrongs has quit IRC | 00:23 | |
| *** Goneri has quit IRC | 00:29 | |
| *** freenzyfriday has joined #zuul | 00:35 | |
| *** yolanda has quit IRC | 00:38 | |
| *** yolanda has joined #zuul | 00:39 | |
| *** freenzyfriday has quit IRC | 00:39 | |
| *** rfolco|ruck has quit IRC | 01:14 | |
| *** masterpe has quit IRC | 01:23 | |
| *** Eighth_Doctor has quit IRC | 01:24 | |
| *** masterpe has joined #zuul | 01:28 | |
| *** Eighth_Doctor has joined #zuul | 01:29 | |
| *** yolanda has quit IRC | 02:06 | |
| *** yolanda has joined #zuul | 02:13 | |
| *** iamweswilson has quit IRC | 02:14 | |
| *** ericsysmin has quit IRC | 02:14 | |
| *** stevthedev has quit IRC | 02:14 | |
| *** gundalow has quit IRC | 02:14 | |
| *** piotrowskim has quit IRC | 02:14 | |
| *** ttx has quit IRC | 02:14 | |
| *** freefood has quit IRC | 02:14 | |
| *** stevthedev has joined #zuul | 02:15 | |
| *** piotrowskim has joined #zuul | 02:15 | |
| *** iamweswilson has joined #zuul | 02:15 | |
| *** ericsysmin has joined #zuul | 02:15 | |
| *** gundalow has joined #zuul | 02:15 | |
| *** freefood has joined #zuul | 02:20 | |
| *** bhavikdbavishi has joined #zuul | 03:46 | |
| *** bhavikdbavishi has quit IRC | 03:48 | |
| *** bhavikdbavishi1 has joined #zuul | 03:48 | |
| *** bhavikdbavishi has joined #zuul | 03:52 | |
| *** bhavikdbavishi1 has quit IRC | 03:54 | |
| *** freenzyfriday has joined #zuul | 04:13 | |
| *** freenzyfriday has quit IRC | 04:17 | |
| *** bhavikdbavishi has quit IRC | 04:27 | |
| *** bhavikdbavishi has joined #zuul | 04:27 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** saneax has joined #zuul | 04:58 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Recover from cat job failures during startup https://review.opendev.org/754636 | 05:06 |
|---|---|---|
| *** freenzyfriday has joined #zuul | 05:07 | |
| *** freenzyfriday has quit IRC | 05:16 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Optimize GitHub requests on PR merge https://review.opendev.org/752886 | 05:25 |
| swest | clarkb: ianw: sorry, was distracted with something else. I removed the dead code in the FakePull ^ | 05:29 |
| *** jfoufas1 has joined #zuul | 05:45 | |
| *** bhavikdbavishi1 has joined #zuul | 06:02 | |
| *** bhavikdbavishi has quit IRC | 06:04 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 06:04 | |
| *** yolanda has quit IRC | 06:10 | |
| *** smyers has quit IRC | 06:15 | |
| *** mach1na has joined #zuul | 06:20 | |
| *** freenzyfriday has joined #zuul | 06:30 | |
| *** mach1na has quit IRC | 06:34 | |
| *** mach1na has joined #zuul | 06:50 | |
| *** hashar has joined #zuul | 06:57 | |
| *** yolanda has joined #zuul | 07:05 | |
| *** jcapitao has joined #zuul | 07:06 | |
| *** smyers has joined #zuul | 07:07 | |
| *** ttx has joined #zuul | 07:17 | |
| *** tosky has joined #zuul | 07:30 | |
| *** jpena|off is now known as jpena | 07:32 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Optimize stats reporting per node request https://review.opendev.org/754967 | 07:35 |
| *** bhavikdbavishi has quit IRC | 07:48 | |
| *** jfoufas1 has quit IRC | 08:04 | |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Warn user when dynamic layout ignores zuul config https://review.opendev.org/720249 | 08:06 |
| *** mach1na has quit IRC | 08:13 | |
| *** bhavikdbavishi has joined #zuul | 08:25 | |
| *** mach1na has joined #zuul | 08:26 | |
| *** bhavikdbavishi1 has joined #zuul | 08:27 | |
| *** bhavikdbavishi has quit IRC | 08:30 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 08:30 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Separate connection registries in tests https://review.opendev.org/712958 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler https://review.opendev.org/717269 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests https://review.opendev.org/721254 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Driver event ingestion https://review.opendev.org/717299 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper https://review.opendev.org/716221 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper https://review.opendev.org/716875 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Improve typings in context of 744416 https://review.opendev.org/753578 | 08:43 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Merge Zookeeper connection methods and specialize exceptions https://review.opendev.org/754360 | 08:43 |
| tobiash | corvus, clarkb: since the scheduler pause patch is only needed later in the ha scheduler stack we've removed it from that stack so we can iterate on it independently ^ | 08:44 |
| tobiash | corvus, clarkb: so we don't have to block the whole stack on it | 08:44 |
| bolg | corvus, clarb, tistanC: I rebased the rest of the stack so the pause/resume pacht is separate, I was kind of waiting for corvus to decide which patch he wants to use 709735 or 588610 but forget to post the result of our private conv. in the ticket. my bad. I am finishing something on the 744416 and then I will take a look and reply on 709735 | 08:52 |
| bolg | tobiash: ^ | 08:52 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Annotate logger with event ID when (un)locking node set https://review.opendev.org/748122 | 08:54 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 09:23 |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 09:40 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Switch to using zookeeper instead of gearman for jobs https://review.opendev.org/744416 | 09:46 |
| *** hashar has quit IRC | 09:57 | |
| *** bhavikdbavishi has quit IRC | 10:01 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 10:34 |
| *** jcapitao is now known as jcapitao_afk | 10:38 | |
| *** hashar has joined #zuul | 11:13 | |
| *** mach1na has quit IRC | 11:14 | |
| *** jpena is now known as jpena|lunch | 11:28 | |
| *** wxy has quit IRC | 11:30 | |
| *** bhavikdbavishi has joined #zuul | 11:34 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 11:48 |
| *** mach1na has joined #zuul | 11:51 | |
| *** rfolco has joined #zuul | 11:53 | |
| *** rfolco is now known as rfolco|ruck|bbl | 11:53 | |
| *** mach1na has quit IRC | 11:56 | |
| *** rlandy has joined #zuul | 11:58 | |
| *** jcapitao_afk is now known as jcapitao | 12:08 | |
| *** armstrongs has joined #zuul | 12:16 | |
| *** Goneri has joined #zuul | 12:17 | |
| *** mach1na has joined #zuul | 12:22 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 12:34 |
| *** jpena|lunch is now known as jpena | 12:36 | |
| *** bhavikdbavishi1 has joined #zuul | 12:51 | |
| *** bhavikdbavishi has quit IRC | 12:52 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 12:52 | |
| *** bhavikdbavishi has quit IRC | 12:55 | |
| *** bhavikdbavishi has joined #zuul | 12:55 | |
| *** rlandy is now known as rlandy|rover | 13:22 | |
| *** rlandy|rover is now known as rlandy|rover|mtg | 13:31 | |
| *** bhavikdbavishi has quit IRC | 13:33 | |
| *** lyr has joined #zuul | 13:35 | |
| AJaeger | infra-root, zuul-jobs maintainer, do we want to merge now https://review.opendev.org/#/c/742732/ - "Merge upload logs modules into common role". It has two +2s, waiting for +A for a good time to merge . | 13:48 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add object models and support for formatters https://review.opendev.org/755292 | 13:53 |
| corvus | bolg, clarkb, tobiash: sorry i forgot i was supposed to reply to bolg earlier. anyway, it seems like the main difference is whether we empty the queues before pausing. is that necessary? i like 2 things about 588610: it's simpler, and it pauses ASAP. | 14:06 |
| *** rfolco|ruck|bbl is now known as rfolco | 14:15 | |
| mhu | clarkb, corvus any blockers left on the zuul-client functional testing chain? https://review.opendev.org/#/c/752039/, https://review.opendev.org/#/c/751264/ and https://review.opendev.org/#/c/753096/ | 14:16 |
| *** rlandy|rover|mtg is now known as rlandy|rover | 14:17 | |
| *** bhagyashri|rover is now known as bhagyashri | 14:20 | |
| bolg | corvus, clarkb, tristanC, tobiash: on a second look the emptying queues will not work on 709735 as it is implemented now. It needs different entrance to the queues (when multiple schedulers are there), e.g. zookeeper | 14:29 |
| bolg | I will explain in the change | 14:29 |
| bolg | I commented on https://review.opendev.org/c/709735 let's abadon that one and use the 588610 one | 14:38 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Scheduler's pause/resume functionality https://review.opendev.org/709735 | 14:42 |
| corvus | bolg: thanks | 14:45 |
| corvus | mhu: i'll look after breakfast :) | 14:46 |
| mhu | thx! | 14:46 |
| *** freenzyfriday has quit IRC | 14:49 | |
| *** rlandy|rover is now known as rlandy|rover|mtg | 14:50 | |
| *** AshBullock has joined #zuul | 15:08 | |
| corvus | mhu: the last change has a comment from clarkb and needs a rebase | 15:20 |
| corvus | mhu: +3 on the first 2 | 15:20 |
| mhu | corvus, shall I make the job voting then? I based it on the nodepool-zuul-functional job | 15:22 |
| mhu | (re: clarkb's comment) | 15:22 |
| corvus | mhu: i think we can make this voting | 15:22 |
| mhu | ok, new patch incoming then | 15:22 |
| corvus | mhu: nodepool functional is non-voting because of instability related to devstack (but i think that's really stable now, and actually i think we could even make that one voting) | 15:23 |
| *** armstrongs has quit IRC | 15:23 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add zuul-client-zuul-functional job (non voting) https://review.opendev.org/753096 | 15:24 |
| clarkb | tobiash: tristanC AJaeger question for https://review.opendev.org/#/c/742732/8. Specifically the ansible docs https://docs.ansible.com/ansible/latest/user_guide/playbooks_reuse_roles.html#using-role-dependencies say that role dependencies are not executed when importing or including a role. We (opendev) use include_role to pull in upload-logs-swift. In this case we still have the tasks/main.yaml in the swift | 15:25 |
| clarkb | role running the library code in the base role. What isn't clear to me is if that base role's library will be available to the swift role if the role "isn't executed" as a dependency | 15:25 |
| clarkb | I expect that it will work and that execution here means tasks/main.yaml in the base role (which does not exist so we don't care) | 15:25 |
| mhu | corvus, clarkb shall I add it to the gate pipeline as well? | 15:25 |
| corvus | mhu: ++ | 15:25 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Add zuul-client-zuul-functional job https://review.opendev.org/753096 | 15:27 |
| mhu | should be good now | 15:27 |
| *** mach1na has quit IRC | 15:31 | |
| openstackgerrit | Merged zuul/zuul-jobs master: shake-build: add shake_target variable https://review.opendev.org/744990 | 15:41 |
| *** rlandy|rover|mtg is now known as rlandy|rover | 15:44 | |
| *** openstackgerrit has quit IRC | 15:46 | |
| *** mach1na has joined #zuul | 15:47 | |
| *** mach1na has quit IRC | 15:52 | |
| *** openstackgerrit has joined #zuul | 16:02 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Separate connection registries in tests https://review.opendev.org/712958 | 16:02 |
| *** tosky has quit IRC | 16:19 | |
| openstackgerrit | zbr proposed zuul/zuul-jobs master: tox: allow tox to be upgraded https://review.opendev.org/690057 | 16:24 |
| *** jcapitao has quit IRC | 16:30 | |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Prepare Zookeeper for scale-out scheduler https://review.opendev.org/717269 | 16:35 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Mandatory Zookeeper connection for ZuulWeb in tests https://review.opendev.org/721254 | 16:36 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Driver event ingestion https://review.opendev.org/717299 | 16:36 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect merger to Zookeeper https://review.opendev.org/716221 | 16:39 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect fingergw to Zookeeper https://review.opendev.org/716875 | 16:46 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Connect executor to Zookeeper https://review.opendev.org/716262 | 16:46 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Improve typings in context of 744416 https://review.opendev.org/753578 | 16:46 |
| openstackgerrit | Jan Kubovy proposed zuul/zuul master: Merge Zookeeper connection methods and specialize exceptions https://review.opendev.org/754360 | 16:46 |
| *** jpena is now known as jpena|off | 16:57 | |
| openstackgerrit | Merged zuul/zuul master: Add zuul-client testing https://review.opendev.org/752039 | 17:31 |
| *** AshBullock has quit IRC | 17:37 | |
| *** hamalq has joined #zuul | 17:52 | |
| fungi | fbo: masterpe: aside from https://review.opendev.org/728681 is there any other work in progress for the gitlab driver? | 18:04 |
| fungi | also wondering if the admonition at the top of https://zuul-ci.org/docs/zuul/reference/drivers/gitlab.html about lack of feature parity with other drivers is still accurate, and what the missing features are at this point | 18:05 |
| fungi | git diff | 18:07 |
| fungi | heh, this is not my shell prompt ;) | 18:07 |
| *** hamalq_ has joined #zuul | 18:29 | |
| *** hamalq has quit IRC | 18:31 | |
| *** avass has quit IRC | 18:56 | |
| *** tosky has joined #zuul | 19:32 | |
| clarkb | ianw: heads up I think there is one additional condition we need to check in https://review.opendev.org/#/c/753222/1 to make find safe but then I agree we can do that | 19:32 |
| clarkb | I noted it in the change | 19:33 |
| openstackgerrit | Merged zuul/zuul master: action: Remove unused uri handler https://review.opendev.org/753220 | 20:48 |
| openstackgerrit | Merged zuul/zuul master: Add test for fileglob behaviour https://review.opendev.org/753221 | 20:48 |
| *** hashar has quit IRC | 20:55 | |
| clarkb | corvus: bolg maybe we should go ahead and land https://review.opendev.org/#/c/588610/2 then? to avoid confusion around the plan there? | 20:57 |
| *** hashar has joined #zuul | 21:05 | |
| *** _erlon_ has joined #zuul | 21:13 | |
| *** hashar has quit IRC | 21:23 | |
| *** rfolco has quit IRC | 22:18 | |
| ianw | clarkb: thanks, responded, i believe the path checking function does that | 22:25 |
| clarkb | ianw: wow we've basically implemented find in the checker then :) | 22:46 |
| clarkb | just need to add on a regex matcher :) | 22:46 |
| clarkb | butya I agree it does an os.walk and checks things | 22:46 |
| clarkb | ianw: I've approved it, sorry for not looking closer at _fail_if_unsafe | 22:47 |
| *** tosky has quit IRC | 22:54 | |
| *** armstrongs has joined #zuul | 23:23 | |
| *** armstrongs has quit IRC | 23:33 | |
| *** rlandy|rover has quit IRC | 23:38 | |
| openstackgerrit | Merged zuul/zuul master: Allow find on the executor https://review.opendev.org/753222 | 23:50 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!