| corvus | i think it can be a reasonable thing to add crds in a multi-tenant cluster, but in specific circumstances: like if you know that none of the tenants will have any version conflicts (if you know that no other tenants will be using the zuul operator, or, if all of the tenants using the zuul operater can use the same version). | 00:00 |
|---|---|---|
| corvus | so it's mostly a negotiation of local needs/policy + available technology. there's certainly no right answer. | 00:00 |
| *** josefwells_ has joined #zuul | 00:08 | |
| *** josefwells has quit IRC | 00:12 | |
| *** jamesmcarthur has quit IRC | 00:36 | |
| *** jamesmcarthur has joined #zuul | 00:36 | |
| *** jamesmcarthur has quit IRC | 00:48 | |
| *** jamesmcarthur has joined #zuul | 00:49 | |
| *** jamesmcarthur has quit IRC | 00:54 | |
| *** jamesmcarthur has joined #zuul | 00:55 | |
| *** jamesmcarthur has quit IRC | 01:02 | |
| *** jamesmcarthur has joined #zuul | 01:02 | |
| *** josefwells_ has quit IRC | 01:32 | |
| *** jamesmcarthur has quit IRC | 01:50 | |
| *** jamesmcarthur has joined #zuul | 01:51 | |
| *** ikhan has joined #zuul | 01:51 | |
| *** jamesmcarthur has quit IRC | 01:52 | |
| *** jamesmcarthur has joined #zuul | 01:52 | |
| *** ikhan has quit IRC | 02:28 | |
| *** ricolin has quit IRC | 02:33 | |
| *** bhavikdbavishi has joined #zuul | 02:49 | |
| *** sanjayu_ has quit IRC | 02:56 | |
| *** bhavikdbavishi has quit IRC | 02:58 | |
| *** josefwells has joined #zuul | 03:00 | |
| *** ikhan has joined #zuul | 03:12 | |
| *** ikhan has quit IRC | 03:14 | |
| *** bhavikdbavishi has joined #zuul | 03:40 | |
| *** bhavikdbavishi1 has joined #zuul | 03:42 | |
| *** ykarel|away has joined #zuul | 03:43 | |
| *** bhavikdbavishi has quit IRC | 03:44 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:44 | |
| *** jamesmcarthur has quit IRC | 04:02 | |
| *** jamesmcarthur has joined #zuul | 04:04 | |
| *** jamesmcarthur has quit IRC | 04:08 | |
| *** josefwells has quit IRC | 04:14 | |
| *** jfoufas1 has joined #zuul | 04:16 | |
| *** jamesmcarthur has joined #zuul | 04:33 | |
| *** rlandy has quit IRC | 04:52 | |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Add nodepool-build-image-builder-arm64 job https://review.opendev.org/c/zuul/nodepool/+/792052 | 05:03 |
| openstackgerrit | Ian Wienand proposed zuul/nodepool master: Add nodepool-build-image-builder-arm64 job https://review.opendev.org/c/zuul/nodepool/+/792052 | 05:20 |
| *** jamesmcarthur has quit IRC | 05:36 | |
| *** ykarel|away is now known as ykarel | 05:40 | |
| *** bhavikdbavishi has quit IRC | 05:44 | |
| *** bhavikdbavishi has joined #zuul | 05:44 | |
| *** jamesmcarthur has joined #zuul | 05:51 | |
| *** bhavikdbavishi has quit IRC | 05:53 | |
| *** jamesmcarthur has quit IRC | 05:58 | |
| *** saneax has joined #zuul | 06:03 | |
| *** ykarel has quit IRC | 06:09 | |
| *** jamesmcarthur has joined #zuul | 06:10 | |
| *** jamesmcarthur has quit IRC | 06:16 | |
| *** ykarel has joined #zuul | 06:27 | |
| avass | johanssone, corvus: Do the operator really need crds to function or is it just a nice way to configure the operator instead of using configmaps? Because that seems like a reason to not use crds | 06:48 |
| avass | tristanC: ^ ? | 06:49 |
| *** jcapitao has joined #zuul | 06:51 | |
| *** saneax has quit IRC | 06:51 | |
| *** ricolin has joined #zuul | 06:54 | |
| zbr | avass: corvus: current versions of ansible-lint does report both line and column, with the remark that column may be "1" in many cases | 07:03 |
| zbr | few days ago i fixed a bug where the linter was wrongly reporting line or column 0, which is invalid value. fix was already released. | 07:04 |
| *** saneax has joined #zuul | 07:25 | |
| *** reiterative has left #zuul | 07:27 | |
| *** hashar has joined #zuul | 07:41 | |
| *** rpittau|afk is now known as rpittau | 07:43 | |
| *** jpena|off is now known as jpena | 07:53 | |
| *** tosky has joined #zuul | 07:59 | |
| *** ykarel is now known as ykarel|lunch | 08:21 | |
| *** nils has joined #zuul | 08:23 | |
| *** mgoddard has quit IRC | 08:54 | |
| *** mgoddard has joined #zuul | 08:54 | |
| *** saneax has quit IRC | 09:09 | |
| *** ykarel|lunch is now known as ykarel | 09:12 | |
| *** saneax has joined #zuul | 09:52 | |
| avass | zbr: right, I think you've mentioned that before | 09:59 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add build-info subcommand https://review.opendev.org/c/zuul/zuul-client/+/751070 | 10:47 |
| *** jcapitao is now known as jcapitao_lunch | 11:03 | |
| tristanC | avass: that's a good point, i guess the current design could work with a single config map instead of a crd. | 11:07 |
| tristanC | corvus: heh, i'll rebase the ansible-lint change now | 11:17 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul-jobs master: Add ansible-lint job https://review.opendev.org/c/zuul/zuul-jobs/+/532083 | 11:20 |
| tristanC | avass: corvus:unless we are going to add extra crd such as ZuulBackup/ZuulRestore (or perhaps ZuulTenant), then maybe we could make the operator work in both context? (e.g. watching a CRD or driven by CM) | 11:22 |
| avass | tristanC: I think one benefit of using crds is quick feedback if your config is faulty | 11:27 |
| openstackgerrit | daniel.pawlik proposed zuul/zuul-registry master: Do not overwrite the kept_manifests variable when pruning https://review.opendev.org/c/zuul/zuul-registry/+/693410 | 11:29 |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** ikhan has joined #zuul | 11:34 | |
| *** bhavikdbavishi has joined #zuul | 11:43 | |
| *** rlandy has joined #zuul | 11:47 | |
| *** jamesmcarthur has joined #zuul | 11:55 | |
| *** jamesmcarthur has quit IRC | 12:07 | |
| *** jamesmcarthur has joined #zuul | 12:07 | |
| *** jcapitao_lunch is now known as jcapitao | 12:09 | |
| *** jamesmcarthur has quit IRC | 12:10 | |
| *** jamesmcarthur has joined #zuul | 12:10 | |
| *** holser_ has quit IRC | 12:26 | |
| *** jpena|lunch is now known as jpena | 12:29 | |
| *** Shrews has joined #zuul | 12:34 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Remove layout attribute from queue items https://review.opendev.org/c/zuul/zuul/+/792622 | 13:06 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Optimize layout re-calculation after re-enqueue https://review.opendev.org/c/zuul/zuul/+/792623 | 13:06 |
| *** bhavikdbavishi has quit IRC | 13:10 | |
| *** bhavikdbavishi has joined #zuul | 13:10 | |
| *** bhavikdbavishi has quit IRC | 13:15 | |
| *** jamesmcarthur has quit IRC | 13:17 | |
| *** jamesmcarthur has joined #zuul | 13:17 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add buildsets, buildset-info to subcommands https://review.opendev.org/c/zuul/zuul-client/+/752909 | 13:23 |
| *** jamesmcarthur has quit IRC | 13:24 | |
| avass | this could need a bit more documentation: https://zuul-ci.org/docs/nodepool/azure.html#attr-providers.[azure].auth-path :) | 13:29 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 13:41 |
| *** josefwells has joined #zuul | 13:47 | |
| *** jamesmcarthur has joined #zuul | 13:48 | |
| *** sshnaidm|pto has quit IRC | 13:49 | |
| *** jamesmcarthur has quit IRC | 13:53 | |
| corvus | avass: it has more docs in the new code | 13:58 |
| *** Shrews has quit IRC | 14:04 | |
| *** Shrews has joined #zuul | 14:05 | |
| corvus | avass: see https://review.opendev.org/781926 | 14:07 |
| *** jamesmcarthur has joined #zuul | 14:08 | |
| *** jfoufas1 has quit IRC | 14:12 | |
| avass | corvus: Oh, I thought it had been merged. | 14:14 |
| *** jamesmcarthur has quit IRC | 14:14 | |
| *** jangutter_ has joined #zuul | 14:15 | |
| *** saneax has quit IRC | 14:17 | |
| *** jangutter has quit IRC | 14:18 | |
| *** jangutter has joined #zuul | 14:19 | |
| *** jangutter_ has quit IRC | 14:23 | |
| *** rpittau is now known as rpittau|afk | 14:27 | |
| *** jamesmcarthur has joined #zuul | 14:27 | |
| *** jamesmcarthur has quit IRC | 14:38 | |
| *** jamesmcarthur has joined #zuul | 14:46 | |
| *** sshnaidm has joined #zuul | 14:50 | |
| *** jamesmcarthur has quit IRC | 14:50 | |
| *** sshnaidm is now known as sshnaidm|pto | 14:52 | |
| *** Shrews has quit IRC | 15:03 | |
| *** jamesmcarthur has joined #zuul | 15:04 | |
| *** ykarel is now known as ykarel|away | 15:05 | |
| *** jamesmcarthur has quit IRC | 15:16 | |
| *** jamesmcarthur has joined #zuul | 15:16 | |
| *** ykarel|away has quit IRC | 15:17 | |
| *** jangutter has quit IRC | 15:33 | |
| *** jangutter has joined #zuul | 15:33 | |
| *** jangutter_ has joined #zuul | 15:36 | |
| *** jangutter has quit IRC | 15:40 | |
| *** bhavikdbavishi has joined #zuul | 15:40 | |
| *** josefwells_ has joined #zuul | 16:07 | |
| *** ykarel|away has joined #zuul | 16:09 | |
| *** josefwells has quit IRC | 16:10 | |
| *** gmann is now known as gmann_afk | 16:11 | |
| *** jcapitao has quit IRC | 16:27 | |
| *** ykarel|away has quit IRC | 16:28 | |
| *** ykarel|away has joined #zuul | 16:38 | |
| *** ykarel|away has quit IRC | 16:43 | |
| *** maxamillion has quit IRC | 16:48 | |
| openstackgerrit | Merged zuul/zuul master: Stream API for sharded Zookeeper data https://review.opendev.org/c/zuul/zuul/+/771454 | 16:52 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/759838 | 16:54 |
| *** jamesmcarthur has quit IRC | 17:00 | |
| *** jpena is now known as jpena|off | 17:03 | |
| *** bhavikdbavishi has quit IRC | 17:07 | |
| *** jamesmcarthur has joined #zuul | 17:21 | |
| tobiash | corvus: the test failure on 789576 looks like a race in a shutdown procedure: http://paste.openstack.org/show/805566/ | 17:31 |
| tobiash | swest: ^ | 17:33 |
| corvus | tobiash: oh hrm, do you think it's related to that change, or just something that snuck in recently? | 17:35 |
| tobiash | I don't think it's related, to me it looks more component registry related | 17:36 |
| tobiash | it crashed while setting the component status to stopped | 17:36 |
| tobiash | and all the other test failures in that build are likely due to a leaked thread due to this failure | 17:37 |
| *** hashar has quit IRC | 17:54 | |
| *** Shrews has joined #zuul | 18:03 | |
| *** Shrews has quit IRC | 18:09 | |
| tobiash | corvus: I've added some thoughts to 779420 (state machine) | 18:22 |
| *** ikhan has quit IRC | 18:24 | |
| *** jamesmcarthur has quit IRC | 19:09 | |
| clarkb | zuulians https://review.opendev.org/c/zuul/zuul/+/792310 is an easy job fixup | 19:10 |
| *** jamesmcarthur has joined #zuul | 19:10 | |
| *** jamesmcarthur has quit IRC | 19:13 | |
| *** jamesmcarthur has joined #zuul | 19:20 | |
| *** jamesmcarthur has quit IRC | 19:20 | |
| openstackgerrit | Merged zuul/zuul master: Allow monitoring length of connection event queues https://review.opendev.org/c/zuul/zuul/+/790213 | 19:44 |
| openstackgerrit | Merged zuul/zuul master: Add a logical timestamp to management events https://review.opendev.org/c/zuul/zuul/+/789576 | 19:45 |
| *** sduthil has quit IRC | 20:14 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add buildsets, buildset-info to subcommands https://review.opendev.org/c/zuul/zuul-client/+/752909 | 20:46 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add console-stream subcommand https://review.opendev.org/c/zuul/zuul-client/+/751238 | 20:54 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul-client master: Add change-status subcommand https://review.opendev.org/c/zuul/zuul-client/+/759838 | 20:55 |
| *** rlandy has quit IRC | 21:05 | |
| corvus | tobiash: thx, replied | 21:06 |
| openstackgerrit | Merged zuul/zuul master: Add pipeline event queue lengths to status JSON https://review.opendev.org/c/zuul/zuul/+/790038 | 21:06 |
| openstackgerrit | Merged zuul/zuul master: Add metrics for pipeline event queues https://review.opendev.org/c/zuul/zuul/+/790039 | 21:06 |
| *** tosky has quit IRC | 21:15 | |
| *** guillaumec has quit IRC | 21:15 | |
| *** tosky has joined #zuul | 21:16 | |
| *** guillaumec has joined #zuul | 21:16 | |
| *** jamesmcarthur has joined #zuul | 21:39 | |
| *** webknjaz has quit IRC | 21:53 | |
| *** jamesmcarthur has quit IRC | 21:56 | |
| *** jamesmcarthur has joined #zuul | 21:58 | |
| *** webknjaz has joined #zuul | 21:59 | |
| *** jamesmcarthur has quit IRC | 22:04 | |
| *** jamesmcarthur has joined #zuul | 22:23 | |
| openstackgerrit | Merged zuul/zuul master: Implement API for caching raw config files in ZK https://review.opendev.org/c/zuul/zuul/+/789577 | 22:29 |
| *** nils has quit IRC | 22:58 | |
| *** tosky has quit IRC | 23:03 | |
| *** jamesmcarthur has quit IRC | 23:12 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!