Friday, 2023-05-26

« Thursday, 2023-05-25 Index Monday, 2023-05-29 »
-@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 883985: Add error information to config-errors API endpoint https://review.opendev.org/c/zuul/zuul/+/88398506:47
@rancher:matrix.orgInspired by the other user, I tried to load a local config project, but it says the following:09:44
```
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: 1 errors detected during tenant tenant configuration loading
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: Zuul encountered an error while accessing the repo zuul-config. The
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: error was:
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader:
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: Cmd('git') failed due to: exit code(128)
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: cmdline: git ls-remote --heads --tags file:///srv/git/zuul-config
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: stderr: 'fatal: '/srv/git/zuul-config' does not appear to be a git repository
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: fatal: Could not read from remote repository.
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader:
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: Please make sure you have the correct access rights
scheduler_1 | 2023-05-26 09:33:24,939 WARNING zuul.ConfigLoader: and the repository exists.'
```
I cloned the repository in `/srv/git/zuul-config` and successfully executed `git status` as well as `git ls-remote --heads --tags file:///srv/git/zuul-config`.
zuul.conf:
```
[connection "local"]
driver=git
baseurl=file:///srv/git
```
main.yaml:
```
source:
local:
config-projects:
- zuul-config
```
@rancher:matrix.orgDocker-compose is ran with `sudo` (`sudo -E docker-compose -p zuul-tutorial up`).09:47
@woju:invisiblethingslab.comI didn't run docker-compose, I've `pip install`ed zuul into a virtualenv09:57
@rancher:matrix.orgI see, but this should work nevertheless, right?10:04
@woju:invisiblethingslab.comIDK how this docker does volumes10:05
@woju:invisiblethingslab.comdidn't you put those repo outside of docker? it might be that scheduler tries to read inside of the docker and there's no /srv/git there without `--volume`10:06
@woju:invisiblethingslab.com * didn't you put that repo outside of docker? it might be that scheduler tries to read inside of the docker and there's no /srv/git there without `--volume`10:06
@woju:invisiblethingslab.com * didn't you put that repo outside of docker? it might be that scheduler tries to read inside of the docker and there's no `/srv/git` there without `--volume`10:06
@rancher:matrix.orgOh, that's probably it. I'm new to dockers, not sure how to access them and change files.10:07
@woju:invisiblethingslab.comhttps://docs.docker.com/storage/volumes/#use-a-volume-with-docker-compose10:07
@rancher:matrix.orgThanks. i'll take a look.10:08
@woju:invisiblethingslab.comor just use `--volume` argument when you start the container10:09
@rancher:matrix.orgCool.10:09
@jpew:matrix.orgI've got a bunch of zuul-jobs changes that haven't been reviewed; is there something I need to to do move them forward?15:08
@clarkb:matrix.orgfor me personally I've been swamped elsewhere so mostly help elsewhere so I have more time for zuul? I just finished up a big thing yesterday though I'll see if I can take a look today15:23
@avass:matrix.vassast.org> <@jpew:matrix.org> I've got a bunch of zuul-jobs changes that haven't been reviewed; is there something I need to to do move them forward?15:29
I think i can take a look in a sec
@yoctozepto:matrix.orgAlbin Vass: I would love it if you could also take a look at this little docs' oldie of mine: https://review.opendev.org/c/zuul/zuul/+/87683415:49
@avass:matrix.vassast.org> <@yoctozepto:matrix.org> Albin Vass: I would love it if you could also take a look at this little docs' oldie of mine: https://review.opendev.org/c/zuul/zuul/+/87683415:55
sadly I'm only a maintainer of zuul-jobs and not zuul :)
@yoctozepto:matrix.orgsadly indeed!15:56
@jim:acmegating.comyoctozepto: typically when a maintainer -1s a change we like for them to re-review updates to see if that addresses their concerns, so you might consider starting by asking corvus to take a look since he previously reviewed it.15:59
@yoctozepto:matrix.orgcorvus: well, I did try pinging you 2 months ago :D 15:59
@jim:acmegating.comsorry if i missed that, though if it was only via gerrit, i don't get those notifications16:01
@yoctozepto:matrix.orgah, could have been, at least the last one16:02
@yoctozepto:matrix.orggood to know16:02
@jim:acmegating.comyoctozepto: i like it when brevity in documentation also removes ambiguity.  :)16:11
@yoctozepto:matrix.orgme too :-)16:11
@avass:matrix.vassast.orgjpew: I don't get this change, Ian mentions a readme and I can't find any mentioning `docker_mirror`. am I missing it? https://review.opendev.org/c/zuul/zuul-jobs/+/86976116:16
@avass:matrix.vassast.orgoh! gitea doesn't render the rst correctly: https://opendev.org/zuul/zuul-jobs/src/commit/739d7b51188fb4c64f886e1c34c1f5609f1d7012/roles/use-docker-mirror/README.rst16:17
@avass:matrix.vassast.orgvs https://opendev.org/zuul/zuul-jobs/raw/commit/739d7b51188fb4c64f886e1c34c1f5609f1d7012/roles/use-docker-mirror/README.rst16:17
@yoctozepto:matrix.orgyeah, it strips the contents unfortunately16:18
@yoctozepto:matrix.orgI read https://zuul-ci.org/docs/zuul-jobs/latest/roles.html16:18
@avass:matrix.vassast.orgtoo many clicks :)16:19
@yoctozepto:matrix.org:D16:19
@avass:matrix.vassast.orgjpew: 871849 could need a test :)16:23
@fungicide:matrix.org> <@avass:matrix.vassast.org> oh! gitea doesn't render the rst correctly: https://opendev.org/zuul/zuul-jobs/src/commit/739d7b51188fb4c64f886e1c34c1f5609f1d7012/roles/use-docker-mirror/README.rst16:28
gitea's readme renderer struggles, particularly with sphinx directives. i advocated for plaintext readmes by default when we deployed it, but rendered readme files are a big hit with people who like the repository browser to double as a sort of project homepage
@fungicide:matrix.orgpersonally, i feel like rst rendering belongs in the documentation publishing pipeline, not the code browser, but that's just my opinion16:28
@jim:acmegating.comit uses pandoc, but that could be changed to something that understood rst better if someone were so inclined.16:31
@jim:acmegating.comi wonder if it could be made a per-repo option16:31
@clarkb:matrix.orgI think the issue is the non standard directives16:31
@clarkb:matrix.orgwhatever renderer is chosen would need to understand the custom directives16:32
@jim:acmegating.comor understand that deleting unknown directives is not the best choice16:32
@fungicide:matrix.orgunderstand or have an inclusive fallback16:32
@fungicide:matrix.orglike just print things if you don't know what else to do with them16:32
@fungicide:matrix.orgmaybe it screws up formatting, but poorly-formatted content is (again in my opinion anyway) superior to silently omitting potentially important information16:33
@yoctozepto:matrix.org++16:34
-@gerrit:opendev.org- Zuul merged on behalf of Radosław Piliszek https://matrix.to/#/@yoctozepto:matrix.org: [zuul/zuul] 876834: [doc] Fix the description of the "shadow" attr https://review.opendev.org/c/zuul/zuul/+/87683417:29
@yoctozepto:matrix.org<3 17:48
-@gerrit:opendev.org- Zuul merged on behalf of Joshua Watt: [zuul/zuul-jobs] 869761: use-docker-mirror: Also run task if docker_mirror is defined https://review.opendev.org/c/zuul/zuul-jobs/+/86976117:50
« Thursday, 2023-05-25 Index Monday, 2023-05-29 »

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!