| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Include project name in gerrit branch cache https://review.opendev.org/c/zuul/zuul/+/811488 | 00:21 | |
| @jim:acmegating.com | Clark: ^ with more test stabilization | 00:22 |
|---|---|---|
| @clarkb:matrix.org | I went ahead and reapproved it | 00:23 |
| @clarkb:matrix.org | I can check in after dinner if we still want to do a restart at that point | 00:23 |
| @jim:acmegating.com | anyone have any good emoji names for zuul? i was thinking maybe 🐕️👻 | 00:31 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] Include project name in gerrit branch cache https://review.opendev.org/c/zuul/zuul/+/811488 | 01:49 | |
| @clarkb:matrix.org | corvus: ^ I'm still around if you want to do that restart | 01:51 |
| @clarkb:matrix.org | though I need to load ssh keys on this machine | 01:51 |
| @jim:acmegating.com | Clark: yep; let's move to #opendev | 01:52 |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] wip: Items ahead/behind wrong after re-enqueue https://review.opendev.org/c/zuul/zuul/+/798691 | 11:24 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] wip: Items ahead/behind wrong after re-enqueue https://review.opendev.org/c/zuul/zuul/+/798691 | 11:33 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] wip: Items ahead/behind wrong after re-enqueue https://review.opendev.org/c/zuul/zuul/+/798691 | 11:39 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] Fix items ahead/behind wrong after re-enqueue https://review.opendev.org/c/zuul/zuul/+/798691 | 12:12 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] Fix items ahead/behind wrong after re-enqueue https://review.opendev.org/c/zuul/zuul/+/798691 | 12:13 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] Fix items ahead/behind wrong after re-enqueue https://review.opendev.org/c/zuul/zuul/+/798691 | 12:14 | |
| @clarkb:matrix.org | corvus: after debugging the event throughput issues any sense for whether https://review.opendev.org/c/zuul/zuul/+/810467 would be helpful? Maybe just the message for election winning? | 14:52 |
| @clarkb:matrix.org | swest: that item ahead issue was preexisting? Looking at the change I don't think that was related to the recent sos work? Just want to make sure it isn't a new regression I need to watch out for :) | 15:01 |
| @fungicide:matrix.org | we said https://review.opendev.org/807702 needed a two-week warning to the ml, it's been three weeks now since that message was sent with no objections there or on the change itself, any reason i shouldn't approve it at this point? | 15:02 |
| @clarkb:matrix.org | fungi: no objection from me | 15:03 |
| @westphahl:matrix.org | Clark: no, that's been a long-standing issue that I just stumbled upon. I just resurrected the change and update the test. I also took a stab at fixing it but it seems that makes another test fail. Need to check if my fix is wrong/incomplete or the test is buggy | 15:03 |
| @clarkb:matrix.org | swest: thanks for confirming | 15:03 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: | 16:37 | |
| - [zuul/zuul] Web UI: add duration info on buildset page https://review.opendev.org/c/zuul/zuul/+/811752 | ||
| - [zuul/zuul] Web UI: add builds timeline on buildset page https://review.opendev.org/c/zuul/zuul/+/811753 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Dong Zhang: [zuul/zuul] Disable aliases in inventory.yaml for better readibility https://review.opendev.org/c/zuul/zuul/+/802674 | 16:41 | |
| @fungicide:matrix.org | okay, merging 807702 now | 17:19 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Fix deserializing change refs from cache https://review.opendev.org/c/zuul/zuul/+/811759 | 18:19 | |
| @jim:acmegating.com | Clark, fungi: ^ that should take care of the issue i saw yesterday when re-enqueing changes.... | 18:20 |
| @jim:acmegating.com | Clark, fungi, swest, tobiash, felixedel: ^ i think you're going to like the way i chose to add a test for that. :) | 18:21 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Fix deserializing change refs from cache https://review.opendev.org/c/zuul/zuul/+/811759 | 18:26 | |
| @jim:acmegating.com | that just tidies up the test a bit | 18:27 |
| @ologinov:synapse.sardinasystems.com | Hi all! Can Zuul work without gerrit, but instead with gitlab? | 18:29 |
| @ologinov:synapse.sardinasystems.com | I saw that Zuul supports gitlab, but I didn't find any information that it can work without gerrit | 18:32 |
| @jim:acmegating.com | Oleg Loginov: yes! | 18:32 |
| @ologinov:synapse.sardinasystems.com | @corvus good news! | 18:33 |
| @jim:acmegating.com | from zuul's pov, gerrit, gitlab, github, and pagure are all nearly equivalent, and it can work with anywhere from 0 to all 4 of them at the same time. | 18:33 |
| @fungicide:matrix.org | i want to say gtema in here is using it with gitlab? | 18:33 |
| @fungicide:matrix.org | so might have tips | 18:34 |
| @gtema:matrix.org | Yes, we run zuul simultaneously with GitHub and gitlab | 18:34 |
| @ologinov:synapse.sardinasystems.com | I tried docker-compose quick installation for it, exclude gerrit, include gitlab..but I can't found addtitional info for debug (It looks like is work but local Zuul web-site is down). | 18:38 |
| @ologinov:synapse.sardinasystems.com | with gerrit works all ok | 18:38 |
| @gtema:matrix.org | What exactly do you mean? | 18:39 |
| @ologinov:synapse.sardinasystems.com | I customized quick-installation (cut gerrit,include gitlab settings ) but http://host:9000 port is down | 18:40 |
| @ologinov:synapse.sardinasystems.com | all containers up and work at this moment | 18:41 |
| @gtema:matrix.org | 9000 is what? I can't recall now | 18:41 |
| @ologinov:synapse.sardinasystems.com | You can see zuul tenants/jobs/etc status there | 18:42 |
| @gtema:matrix.org | Ah, zuul web | 18:42 |
| @ologinov:synapse.sardinasystems.com | this port binding for `web` container | 18:42 |
| @ologinov:synapse.sardinasystems.com | * yep,this port binding for `web` container | 18:43 |
| @gtema:matrix.org | Well, you should really look into this container logs | 18:43 |
| @ologinov:synapse.sardinasystems.com | do you mean watch `zuul-web -d` for debug ? | 18:43 |
| @gtema:matrix.org | Yes | 18:44 |
| @fungicide:matrix.org | yeah, that doesn't seem to be a gitlab-related problem. zuul-web should start and bnid to the port normally so either its configuration was changed or something's preventing it from starting | 18:44 |
| @ologinov:synapse.sardinasystems.com | gitlab has a long start before work(it also included into docker-compose.yaml) | 18:46 |
| @gtema:matrix.org | You included gitlab container or what? | 18:46 |
| @ologinov:synapse.sardinasystems.com | yes | 18:46 |
| @gtema:matrix.org | Uhm | 18:46 |
| @gtema:matrix.org | Then this can be really a problem if it takes too long, cause zuul tries to connect to it | 18:47 |
| @gtema:matrix.org | You might separate them from each other | 18:48 |
| @ologinov:synapse.sardinasystems.com | indeed, I should check it | 18:48 |
| @gtema:matrix.org | I have no experience running gitlab itself, in our case other people take care of that | 18:51 |
| @ologinov:synapse.sardinasystems.com | gtema: if it can work together, it will work. It's really good news | 18:52 |
| @gtema:matrix.org | They can really work together, but I would definitely separate installation from each other | 18:53 |
| @gtema:matrix.org | Potentially you will not be that happy of free gitlab version, but that is a different story. It is still usable | 18:53 |
| @tobias.henkel:matrix.org | > <@gerrit:opendev.org> Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: | 19:19 |
| > - [zuul/zuul] Web UI: add duration info on buildset page https://review.opendev.org/c/zuul/zuul/+/811752 | ||
| > - [zuul/zuul] Web UI: add builds timeline on buildset page https://review.opendev.org/c/zuul/zuul/+/811753 | ||
| that is awesome :) | ||
| @mhuin:matrix.org | > <@tobias.henkel:matrix.org> that is awesome :) | 19:28 |
| thanks! That was suggested by Fabien Boucher [fbo] | ||
| @avass:vassast.org | mhu: nice :) | 19:30 |
| @morucci:matrix.org | that nice mhu ! thanks ! do you think we can display that info (start and end time) on the buildsets page as well ? | 19:38 |
| @mhuin:matrix.org | > <@morucci:matrix.org> that nice mhu ! thanks ! do you think we can display that info (start and end time) on the buildsets page as well ? | 19:40 |
| Not without modifying zuul-web, the buildsets endpoint doesn't provide time info, see for example https://zuul.opendev.org/api/tenant/zuul/buildsets?result=SUCCESS&limit=1 | ||
| @morucci:matrix.org | ok got it, but that might be a good addition to zuul-web + ui | 19:42 |
| @mhuin:matrix.org | or we could display a tooltip on hover and fetch the buildset's info as needed | 19:44 |
| @tobias.henkel:matrix.org | corvus, fungi this has +13 now, shall we merge it? https://review.opendev.org/c/zuul/zuul-jobs/+/809899 | 19:50 |
| @jim:acmegating.com | tobiash: ++ | 19:50 |
| @avass:vassast.org | tobiash: +15 ;) | 19:52 |
| @clarkb:matrix.org | All the ++s | 19:57 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] Remove time database https://review.opendev.org/c/zuul/zuul/+/808841 | 20:35 | |
| @jpew:matrix.org | I have some projects that depend on other upstream projects, and I'd like to trigger builds in my project when the upstream branch changes; is this possible and whats the best way? | 20:36 |
| @jpew:matrix.org | (upstream projects are not managed by zuul) | 20:36 |
| @clarkb:matrix.org | I think you can subscribe to ref updated events on the git driver? | 20:37 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] Fix deserializing change refs from cache https://review.opendev.org/c/zuul/zuul/+/811759 | 20:41 | |
| @jpew:matrix.org | Ya, so do I need a specific pipeline for that upstream check (e.g. `cool-upstream-check`) ? | 20:41 |
| @clarkb:matrix.org | jpew: I think you can add it as a separate trigger into your existing check pipeline. OpenDev set up a separate github check pipelin though | 20:43 |
| -@gerrit:opendev.org- Zuul merged on behalf of Dong Zhang: [zuul/zuul] Show emoji to highlight failed jobs in build result in Github https://review.opendev.org/c/zuul/zuul/+/803547 | 21:12 | |
| @jim:acmegating.com | > <@gerrit:opendev.org> Zuul merged on behalf of Dong Zhang: [zuul/zuul] Show emoji to highlight failed jobs in build result in Github https://review.opendev.org/c/zuul/zuul/+/803547 | 21:18 |
| 🐕️👻✅ | ||
| @jim:acmegating.com | mhu: 811752+3 with note; suggestion on 811753 but i love it. | 21:38 |
| @jim:acmegating.com | mhu, Fabien Boucher [fbo] adding buildset duration to the zuul-web buildset response should be easy and efficient; i think better to do that than the async fetch. | 21:39 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Don't add node resources to nonexistent tenant https://review.opendev.org/c/zuul/zuul/+/811800 | 22:07 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] Don't add node resources to nonexistent tenant https://review.opendev.org/c/zuul/zuul/+/811800 | 22:07 | |
| @jim:acmegating.com | Clark, fungi: ^ that's the last traceback i observed during our recent restarts. it's the least severe (it's effectively logspam that eventually disappears), but we should avoid it so we can find the actual important tracebacks :) | 22:08 |
| @jim:acmegating.com | i'd love to merge that, do one more restart, then seriously consider a release | 22:09 |
| @clarkb:matrix.org | corvus: I've got a few zuul things to review. Sorry today has been a bunch of "learn me an ssl/tls and pem" day. I think I have time to do zuul reviews shortly though | 22:21 |
| @jpew:matrix.org | I removed a project from my tenant configuration, and it appears to have broken my scheduler; it's still requesting all the branches when the scheduler starts up ; this takes a *very* long time | 22:26 |
| @clarkb:matrix.org | corvus: https://review.opendev.org/c/zuul/zuul/+/811800 lgtm I'll let you decide if you want to single +2 merge that to keep things moving along here | 22:27 |
| @clarkb:matrix.org | jpew: is it a github project? | 22:27 |
| @jpew:matrix.org | No, git driver | 22:28 |
| @clarkb:matrix.org | ok, the github driver will ask the github api for all installed applications then do some pre cache priming work on that stuff. Thought maybe that could be related | 22:28 |
| @clarkb:matrix.org | I wonder if the git driver does similar now though | 22:28 |
| @jpew:matrix.org | Ya, I decided I didn't want this project (because its huge) so I removed it and restarted the scheduler, but it's just broken worse now :/ | 22:29 |
| @jim:acmegating.com | jpew: try stop everything; "zuul delete-state"; then start | 22:31 |
| @fungicide:matrix.org | jpew: i wonder if it could be a regression in state persistence within zk. have you tried clearing the zk content? | 22:31 |
| @jpew:matrix.org | I'll try that | 22:31 |
| @jim:acmegating.com | fungi and i said similar things with different words | 22:31 |
| @fungicide:matrix.org | jpew: also what version of zuul are you running? recent master branch or a tagged release? | 22:32 |
| @jpew:matrix.org | farily recent master (few days maybe?) | 22:32 |
| @jpew:matrix.org | master as of 9/27 | 22:32 |
| @jim:acmegating.com | jpew: i recommend moving to latest master when you stop | 22:32 |
| @jim:acmegating.com | many bugs fixed, some of which may need a 'zuul delete-state' anyway, so two birds, one stone | 22:33 |
| @jpew:matrix.org | corvus: K will do. Would be a little easier if someone could look at my patches so I can stop carrying them locally (hint, hint) ;) | 22:34 |
| @clarkb:matrix.org | Nova has discovered that an empty commit is not CI able: https://paste.opendev.org/show/809680/ | 22:48 |
| @jim:acmegating.com | https://review.opendev.org/811204 does look like an easy +3 | 22:48 |
| @clarkb:matrix.org | I'm not necessarily sure this is a use case zuul needs to support | 22:48 |
| @clarkb:matrix.org | But calling it out here just in case since a user hit it in the wild | 22:48 |
| @clarkb:matrix.org | Might also want to report a better error message in that case | 22:49 |
| @clarkb:matrix.org | > <@jim:acmegating.com> https://review.opendev.org/811204 does look like an easy +3 | 22:51 |
| +3'd just have to rtfs the requests adapter code real quick | ||
| @clarkb:matrix.org | > <@jim:acmegating.com> https://review.opendev.org/811204 does look like an easy +3 | 22:51 |
| * +3'd just had to rtfs the requests adapter code real quick | ||
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/nodepool] kubernetes: Add startup-timeout https://review.opendev.org/c/zuul/nodepool/+/811234 | 23:43 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/nodepool] kubernetes, openshift: Add kvm resource https://review.opendev.org/c/zuul/nodepool/+/811232 | 23:44 | |
| -@gerrit:opendev.org- Joshua Watt proposed: [zuul/nodepool] kubernetes, openshift: Add volume mount support https://review.opendev.org/c/zuul/nodepool/+/810322 | 23:46 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!