| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 08:36 | |
| - [zuul/nodepool] 849582: Speed up node listing https://review.opendev.org/c/zuul/nodepool/+/849582 | ||
| - [zuul/nodepool] 848171: Fix ignore_provider_quota in statemachine drivers https://review.opendev.org/c/zuul/nodepool/+/848171 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Tobias Henkel: [zuul/nodepool] 771961: Use libyaml parsing when available https://review.opendev.org/c/zuul/nodepool/+/771961 | 09:34 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849728: Implement gitea refs https://review.opendev.org/c/zuul/zuul/+/849728 | 09:43 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850008: Implement gitea merge reporter https://review.opendev.org/c/zuul/zuul/+/850008 | 09:48 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850413: Implement gitea requirement https://review.opendev.org/c/zuul/zuul/+/850413 | 10:07 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850413: Implement gitea requirement https://review.opendev.org/c/zuul/zuul/+/850413 | 10:34 | |
| @weebull:matrix.org | Back again.... So we currently track our cross-project dependencies using git submodules (for better or worse). Theory being we can pin different versions in different projects. | 12:30 |
|---|---|---|
| I guess if we introduce Zuul that all works fine , but it undermines it's cross project testing and atomic merges. Have I understood correctly? | ||
| @fungicide:matrix.org | > <@weebull:matrix.org> Back again.... So we currently track our cross-project dependencies using git submodules (for better or worse). Theory being we can pin different versions in different projects. | 12:51 |
| > | ||
| > I guess if we introduce Zuul that all works fine , but it undermines it's cross project testing and atomic merges. Have I understood correctly? | ||
| i think there's some support for making sure submodules are properly checked out in jobs, but for the most part yes zuul's earliest strengths arise from its use in a massive multi-repo ecosystem which avoided submodules, and zuul existed in great part to solve the same sorts of problems in better and more flexible ways than submodules could | ||
| @weebull:matrix.org | Makes sense. Pinning versions is about avoiding changes to one project breaking another, and Zuul stops that through testing. | 12:53 |
| @fungicide:matrix.org | some users in here do use submoduels pretty extensively though, i think, so maybe they have some tips for getting them to coexist and maybe letting zuul override things where it's more useful to do so | 12:54 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850883: Implement gitea review trigger https://review.opendev.org/c/zuul/zuul/+/850883 | 12:58 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849033: Initial implementation of the gitea driver https://review.opendev.org/c/zuul/zuul/+/849033 | 13:04 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849670: Improve gitea trigger testing https://review.opendev.org/c/zuul/zuul/+/849670 | 13:04 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849680: Implement gitea reporter interface https://review.opendev.org/c/zuul/zuul/+/849680 | 13:04 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849719: Implement gitea commit status reporter https://review.opendev.org/c/zuul/zuul/+/849719 | 13:04 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849720: Implement gitea comment trigger https://review.opendev.org/c/zuul/zuul/+/849720 | 13:05 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849727: Add gitea pr dyn_reconf test https://review.opendev.org/c/zuul/zuul/+/849727 | 13:05 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849728: Implement gitea refs https://review.opendev.org/c/zuul/zuul/+/849728 | 13:05 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850008: Implement gitea merge reporter https://review.opendev.org/c/zuul/zuul/+/850008 | 13:05 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850413: Implement gitea requirement https://review.opendev.org/c/zuul/zuul/+/850413 | 13:05 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850883: Implement gitea review trigger https://review.opendev.org/c/zuul/zuul/+/850883 | 13:05 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849728: Implement gitea refs https://review.opendev.org/c/zuul/zuul/+/849728 | 13:09 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850008: Implement gitea merge reporter https://review.opendev.org/c/zuul/zuul/+/850008 | 13:09 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850413: Implement gitea requirement https://review.opendev.org/c/zuul/zuul/+/850413 | 13:09 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850883: Implement gitea review trigger https://review.opendev.org/c/zuul/zuul/+/850883 | 13:10 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849680: Implement gitea reporter interface https://review.opendev.org/c/zuul/zuul/+/849680 | 14:07 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849719: Implement gitea commit status reporter https://review.opendev.org/c/zuul/zuul/+/849719 | 14:08 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849720: Implement gitea comment trigger https://review.opendev.org/c/zuul/zuul/+/849720 | 14:14 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849727: Add gitea pr dyn_reconf test https://review.opendev.org/c/zuul/zuul/+/849727 | 14:16 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 849728: Implement gitea refs https://review.opendev.org/c/zuul/zuul/+/849728 | 14:16 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850008: Implement gitea merge reporter https://review.opendev.org/c/zuul/zuul/+/850008 | 14:16 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850413: Implement gitea requirement https://review.opendev.org/c/zuul/zuul/+/850413 | 14:16 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850883: Implement gitea review trigger https://review.opendev.org/c/zuul/zuul/+/850883 | 14:16 | |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850898: Implement gitea label_updated event trigger https://review.opendev.org/c/zuul/zuul/+/850898 | 14:25 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 848172: AWS multi quota support https://review.opendev.org/c/zuul/nodepool/+/848172 | 15:23 | |
| @clarkb:matrix.org | corvus: on ianw's stack starting at https://review.opendev.org/c/zuul/zuul/+/850269 to update console streaming you +2'd but didn't approve many of the changes. Were you hoping for additional reviews or just wanting to make sure that ianw and I saw your responses to comments? | 15:38 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 850694: Fix deduplicated build references https://review.opendev.org/c/zuul/zuul/+/850694 | 15:41 | |
| @jim:acmegating.com | Clark: i particularly wanted to make sure you and ianw saw my comment on 850269, since the stack as currently written solves the problem 90% and i made a suggestion of how to get to 100%. | 15:46 |
| @clarkb:matrix.org | Got it. The add tmpreaper functionality bit | 15:47 |
| @jim:acmegating.com | a built-in tmpreaper is capable of deleting files in all cases but after a delay. the finalizer can delete files in the typical case (but not all cases), but does so immediately. | 15:47 |
| @jim:acmegating.com | the tmpreaper alone provides 100% coverage if the delay is acceptable | 15:48 |
| @clarkb:matrix.org | I'll hold off on approving anything until ianw starts and can chime in | 15:48 |
| @clarkb:matrix.org | corvus: talking out loud here: if opendev continues to be happy with current zuul should we make a new release? Looks like we are about 50 commits ahead of the last release | 17:38 |
| @jim:acmegating.com | Clark: i'd like to wait for https://review.opendev.org/850694 | 17:40 |
| @clarkb:matrix.org | ok. I'll see if I can add that stack to the revie wlist today | 17:41 |
| @jim:acmegating.com | Clark: thanks. it has 2 parents and also needs a re-review from swest but i'm fairly confident i addressed his comments, so i think the stack is probably ready for more eyes at this point. | 17:42 |
| @avass:vassast.org | > <@fungicide:matrix.org> some users in here do use submoduels pretty extensively though, i think, so maybe they have some tips for getting them to coexist and maybe letting zuul override things where it's more useful to do so | 17:55 |
| Paul: we use submodules without any speculative state from zuul by configuring required-projects and then set up the correct workspace state with the local copy of the repo (so the job doesn't need any auth for Gerrit). We're thinking about extending that with Gerrits super project subscriptions to allow cross project speculative testing :) | ||
| @vlotorev:matrix.org | We are also using Gerrit super project subscriptions. Dirty script that substitutes projects cloned by zuul (by required-projects or Depends-On or Gerrit topic) for repos listed in .gitmodules can be found here https://github.com/vlotorev/zuul-tools | 18:24 |
| @vlotorev:matrix.org | * We are also using Gerrit super project subscriptions. Dirty script that substitutes projects cloned by zuul (by required-projects or Depends-On or Gerrit topic) for repos listed in .gitmodules can be found here https://github.com/vlotorev/zuul-tools | 18:25 |
| This requires secret on nodes though. | ||
| @vlotorev:matrix.org | * We are also using Gerrit super project subscription. Dirty script that substitutes projects cloned by zuul (by required-projects or Depends-On or Gerrit topic) for repos listed in .gitmodules can be found here https://github.com/vlotorev/zuul-tools | 18:25 |
| This requires secret on nodes though. | ||
| @clarkb:matrix.org | Only if your repos require auth to clone though right? | 18:27 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul] 850941: Fix gitea PR sync https://review.opendev.org/c/zuul/zuul/+/850941 | 18:34 | |
| @clarkb:matrix.org | corvus: question on https://review.opendev.org/c/zuul/zuul/+/850692 but feel free to approve if that was intentional or if you just want to do a followup to cleanup | 18:40 |
| @vlotorev:matrix.org | > <@clarkb:matrix.org> Only if your repos require auth to clone though right? | 18:51 |
| Clark: right | ||
| @jim:acmegating.com | Clark: yeah we can drop the second project. the oversight was leaving it in the fixture, not that the test needs it, so i think it's safe to approve and maybe cleanup in a followup. i'll push one up in a bit. | 18:54 |
| @clarkb:matrix.org | corvus: a suggestion on https://review.opendev.org/c/zuul/zuul/+/850693 as well but it is super minor. Will let you approve if you don't want to change it | 18:56 |
| @jim:acmegating.com | Clark: in that case, i can update both of those changes and i think we can carry over swest's +2 on those. how does that sound? | 18:58 |
| @clarkb:matrix.org | corvus: couple of questions on the last change in that stack | 20:01 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 850944: Fix race with two builders deleting images https://review.opendev.org/c/zuul/nodepool/+/850944 | 20:13 | |
| @jim:acmegating.com | Clark: i think the answer to both of those is we don't need _path and it was leftover from an earlier iteration; i'll fix that up too | 20:16 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 20:23 | |
| - [zuul/zuul] 850692: Do not deduplicate the noop job https://review.opendev.org/c/zuul/zuul/+/850692 | ||
| - [zuul/zuul] 850693: Fix duplicate setResult calls in deduplicated builds https://review.opendev.org/c/zuul/zuul/+/850693 | ||
| - [zuul/zuul] 850694: Fix deduplicated build references https://review.opendev.org/c/zuul/zuul/+/850694 | ||
| @jim:acmegating.com | Clark: ^ all comments should be addressed, thanks! | 20:23 |
| @clarkb:matrix.org | all three lgtm. I'll let you +A on the first two to carry the votes over | 20:31 |
| @jim:acmegating.com | done; hopefully swest can give the 3rd a quick re-review tomorrow. | 20:37 |
| @clarkb:matrix.org | corvus: some questions and comments on https://review.opendev.org/c/zuul/nodepool/+/848172 | 21:23 |
| @jim:acmegating.com | Clark: thanks; also i think https://review.opendev.org/850119 should go in the next release | 21:25 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 848172: AWS multi quota support https://review.opendev.org/c/zuul/nodepool/+/848172 | 21:41 | |
| @jim:acmegating.com | Clark: ^ updated | 21:41 |
| @clarkb:matrix.org | thanks lgtm | 21:46 |
| -@gerrit:opendev.org- Miguel Lavalle proposed: [zuul/zuul-jobs] 850917: Revert "Revert "Install venv for all platforms in ensure-pip"" https://review.opendev.org/c/zuul/zuul-jobs/+/850917 | 22:57 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 850955: [wip] allow ensure-pip to install python3.8 as well https://review.opendev.org/c/zuul/zuul-jobs/+/850955 | 23:45 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 850955: [wip] allow ensure-pip to install python3.8 as well https://review.opendev.org/c/zuul/zuul-jobs/+/850955 | 23:48 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul-jobs] 850955: [wip] allow ensure-pip to install python3.8 as well https://review.opendev.org/c/zuul/zuul-jobs/+/850955 | 23:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!