| -@gerrit:opendev.org- Lukas Kranz proposed: [zuul/zuul-jobs] 945795: Add limit-log-file-size role https://review.opendev.org/c/zuul/zuul-jobs/+/945795 | 07:18 | |
| -@gerrit:opendev.org- Lukas Kranz proposed: [zuul/zuul-jobs] 946033: mirror-workspace-git-repos: Allow deleting current branch https://review.opendev.org/c/zuul/zuul-jobs/+/946033 | 07:27 | |
| -@gerrit:opendev.org- Lukas Kranz proposed: [zuul/zuul-jobs] 946035: emit-job-header: add region information https://review.opendev.org/c/zuul/zuul-jobs/+/946035 | 07:45 | |
| -@gerrit:opendev.org- Lukas Kranz proposed: [zuul/zuul-jobs] 946033: mirror-workspace-git-repos: Allow deleting current branch https://review.opendev.org/c/zuul/zuul-jobs/+/946033 | 07:48 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 947805: Improve job result colors on status page https://review.opendev.org/c/zuul/zuul/+/947805 | 09:32 | |
| @blart_versenwald_3:matrix.org | Clark: Hi again! | 11:52 |
|---|---|---|
| I think I have found out how this works now. I cannot use a tag in **job.required-project.override-checkout** to select what version (tag) I want my parent to originate from. If I do this I will get "master" branch (default over here) on the parent, but the checked out version of the project in my workspace will honor the tag value. | ||
| In my case this is troublesome, I just don't want changes in the parent to fly by unnoticed. I want those changes to go threw the same CI as anything else. That is, if there is an update for the parent job I want to integrate them by a change of the tag. | ||
| You can argue that if there are that tight dependencies the parent should maybe be defined in the same repo. I am starting to wonder if that would not actually make most sense. | ||
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 947817: Silently dequeue changes when handling promote events https://review.opendev.org/c/zuul/zuul/+/947817 | 11:53 | |
| @mhuin:matrix.org | Hello zuul-maint, this small fix to the jobs endpoint of the REST API just needs a second +2 https://review.opendev.org/c/zuul/zuul/+/946748 | 12:32 |
| -@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 946748: zuul-web: fix traceback in "job" endpoint if a job doesn't exist https://review.opendev.org/c/zuul/zuul/+/946748 | 14:22 | |
| @f2ked:matrix.org | do you guys have a "best practice" for getting submodules loaded before `prepare-workspace` in `pre.yaml` | 15:01 |
| @fungicide:matrix.org | i'm not personally familiar with any projects that use git submodules (some of us have have negative experiences with submodules in the past and avoid them). zuul is, to a great extent, designed to ease development of projects consisting of multiple separate git repositories without needing to rely on submodules, so it has features that are somewhat in conflict with how other projects often use submodules anyway | 15:11 |
| @fungicide:matrix.org | the only documentation i find mentioning submodules in zuul is this spec talking about ways to potentially support them better: https://zuul-ci.org/docs/zuul/latest/developer/specs/configure-projects.html | 15:11 |
| @fungicide:matrix.org | i think i remember some zuul users saying that they add the repositories that would be submodules as just normal projects in their zuul configuration, and then stitch them into the correct location in the checkout early in their jobs in order to simulate a submodule tree without breaking zuul's speculative checkout state in each of the not-quite-submodule repositories | 15:14 |
| @f2ked:matrix.org | That sounds like our current strategy, to just populate the "work tree" in the executor and rsync it around. | 15:17 |
| @f2ked:matrix.org | This leads me to the next question as to how to best have a "trusted" role that grabs a set a submodules in a list defined by an untrusted job? | 15:18 |
| @f2ked:matrix.org | * This leads me to the next question as to how to best have a "trusted" role that grabs a set of submodules from a list defined by an untrusted job? | 15:18 |
| @fungicide:matrix.org | yeah, the main problem is if you want to do things like depends-on changes in your submodules, then the pinned commit ids in the submodule list tends to conflict with zuul's speculative states | 15:18 |
| @f2ked:matrix.org | the SHAs for the submodules are in the main repo, so the "depends-on" relationship is handled by git | 15:19 |
| @fungicide:matrix.org | right, which is what i mean by being in conflict with how zuul handles dependencies between commits/changes between repositories | 15:20 |
| @fungicide:matrix.org | zuul generally expects repositories needed in a job to be listed as part of the job (via required-projects), so is a job-level concept rather than being at the repository level like a submodule list | 15:23 |
| @fungicide:matrix.org | there is some regression testing in zuul wrt git submodule handling, but i don't find anything explicitly documented about it: https://opendev.org/zuul/zuul/src/commit/b70717e/tests/unit/test_merger_repo.py#L77-L119 | 15:31 |
| @clarkb:matrix.org | fungi: f2ked opendev's gerrit image builds rely on the fact that gerrit's submodules are defined as being relative to the gerrit repo itself. This means that git will look for those modules locally when checked out and there is no origin remote: https://opendev.org/opendev/system-config/src/branch/master/playbooks/zuul/gerrit/repos.yaml#L46 | 15:46 |
| @clarkb:matrix.org | I think in general if you use relative paths for submodules this makes things easy. Of course if your submodules are not hosted on the same server then this may not work | 15:47 |
| @fungicide:matrix.org | though presumably zuul doesn't see the repositories in the submodules list as implicit required-projects, so they still need to be declared to jobs in order to get the right checkouts for depends-on between changes in different submodules and/or the superproject? | 15:56 |
| @clarkb:matrix.org | yes | 16:02 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 947848: DNM: test podman image build https://review.opendev.org/c/zuul/zuul/+/947848 | 16:13 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 947649: Add newrev to timer-triggered items https://review.opendev.org/c/zuul/zuul/+/947649 | 16:25 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 947867: Add periodic stats to launcher https://review.opendev.org/c/zuul/zuul/+/947867 | 18:23 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!