| -@gerrit:opendev.org- Lukas Kranz proposed: [zuul/zuul-jobs] 921637: Remove get_md5 parameter from stat module. https://review.opendev.org/c/zuul/zuul-jobs/+/921637 | 06:02 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 06:47 | |
| - [zuul/zuul] 921625: Avoid double read in configloader https://review.opendev.org/c/zuul/zuul/+/921625 | ||
| - [zuul/zuul] 921626: Start yappi last and stop it first https://review.opendev.org/c/zuul/zuul/+/921626 | ||
| @sylvass:albinvass.se | fungi: I guess it answers the question "what resources does zuul need to run?", it can run on a raspberry pi :) (though it's the beefiest one, but I don't think zuul needs much tbh) | 09:45 |
|---|---|---|
| @sylvass:albinvass.se | I haven't used it yet, but I expect mergers/executors to be slow unless I give them something better than a microsd or cifs mount to write to, but I'm working on that. | 09:47 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 921682: Stop logging refs set https://review.opendev.org/c/zuul/zuul/+/921682 | 15:03 | |
| -@gerrit:opendev.org- Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org proposed: [zuul/zuul] 921689: web: encodeURIComponent for zuul-manifest path https://review.opendev.org/c/zuul/zuul/+/921689 | 16:02 | |
| @mordred:waterwanders.com | tristanC: when does the manifest path have a %? seems like a weird thing to be in obj.name in the first place :) | 16:13 |
| @tristanc_:matrix.org | mordred: when a log file has one, like in this build: https://centos.softwarefactory-project.io/zuul/t/centos/build/4d6bc5624b22471e93fdc50398788949 | 16:14 |
| @mordred:waterwanders.com | AH - nod | 16:15 |
| @tristanc_:matrix.org | I see, the commit message is not exactly clear. Also, while this change fixes the immediate error (e.g. the build page crashing), clicking on the offending file in the tree view results in a "This logfile could not be found", though the raw link works as expected. I can't figure out what is the code path that needs to be adjusted in that case... | 16:20 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 921694: Urlencode git url in prepare-workspace-git https://review.opendev.org/c/zuul/zuul-jobs/+/921694 | 17:09 | |
| @jim:acmegating.com | note that if we're going to say that "name" is not url-encoded (which is reasonable), then we'll need to make sure to keep that in mind when we construct urls from it. | 17:24 |
| @sylvass:albinvass.se | corvus: heh I got an old change for that: https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 17:38 |
| @jim:acmegating.com | Albin Vass: happy to merge it if you remove wip and update :) | 17:43 |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul-jobs] 839225: mirror-workspace-git: urlencode src_dir https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 18:20 | |
| @sylvass:albinvass.se | should mirror-workspace-git-repos be removed since prepare-workspace-git integrated that functionality with the sync-projects.yaml task list? | 18:23 |
| @sylvass:albinvass.se | and do we need the test-prepare-workspace-git and test-mirror-workspace-git roles since we can run tests in zuul-jobs for them now | 18:26 |
| -@gerrit:opendev.org- Zuul merged on behalf of Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org: [zuul/zuul] 921689: web: encodeURIComponent for zuul-manifest path https://review.opendev.org/c/zuul/zuul/+/921689 | 18:47 | |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul-jobs] 839225: mirror-workspace-git: urlencode src_dir https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 18:57 | |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul-jobs] 839225: mirror-workspace-git: urlencode src_dir https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 19:32 | |
| @sylvass:albinvass.se | we should remove the fetch-zuul-cloner role and test playbooks at least. it's failing on workspace-scheme=unique and it being deprecated for 5 years is long enough :) | 19:54 |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul-jobs] 839225: mirror-workspace-git: urlencode src_dir https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 19:54 | |
| @sylvass:albinvass.se | eh, should this be "deprecated roles and test roles" or is it _really_ "deprecated roles, and test roles": https://zuul-ci.org/docs/zuul-jobs/latest/deprecated-roles.html ? | 19:59 |
| @jim:acmegating.com | at least in english as i understand it, all 3 of those are the same thing :) so if there's a nuance you want to convey, i'm missing it. but regardless, it is the resting place for two types of roles most people don't need to look at. | 20:03 |
| @sylvass:albinvass.se | the file is called `deprecated-roles.rst` | 20:03 |
| @jim:acmegating.com | ah; i don't think it's worth a rename :) | 20:04 |
| @jim:acmegating.com | (that breaks links unless we do a lot of work to redirect) | 20:04 |
| @jim:acmegating.com | i think we can safely stop testing fetch-zuul-cloner; we could probably remove it, but that'll need an announcement. | 20:05 |
| @sylvass:albinvass.se | I removed the test for it at least | 20:05 |
| -@gerrit:opendev.org- Albin Vass proposed wip: [zuul/zuul-jobs] 839225: prepare-workspace-git: urlencode src_dir https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 20:15 | |
| @sylvass:albinvass.se | ^ updated the commit message but that seems to work, ubuntu-focal failed but I think it was intermittent | 20:15 |
| @sylvass:albinvass.se | I'll be back in an hour | 20:16 |
| @sylvass:albinvass.se | weird, any idea why this only fails on focal? https://zuul.opendev.org/t/zuul/build/d4ca763e5557452489a6605ba7a47d15/console#2/12/16/ubuntu-focal3 | 20:35 |
| -@gerrit:opendev.org- Albin Vass marked as active: [zuul/zuul-jobs] 839225: prepare-workspace-git: urlencode src_dir https://review.opendev.org/c/zuul/zuul-jobs/+/839225 | 20:36 | |
| @jpew:matrix.org | I'm trying to use the promote-container-image roles from zuul-jobs to promote from the intermediate registry, but it doesn't appear that it logs into the intermediate registry before hand, so it's failing with an authentication error. Anyone else seen this or know how to work around it | 21:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!