| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/zuul] 885426: job: Allow dict in file matchers https://review.opendev.org/c/zuul/zuul/+/885426 | 02:57 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 885299: Document the source of the afs 0x40084301 ioctl magic number https://review.opendev.org/c/zuul/zuul/+/885299 | 04:18 | |
| -@gerrit:opendev.org- Flavio Percoco Premoli proposed: [zuul/zuul] 885455: Use built-in URL data type instead of custom parse https://review.opendev.org/c/zuul/zuul/+/885455 | 09:31 | |
| @flaper87:matrix.org | > <@gerrit:opendev.org> Flavio Percoco Premoli proposed: [zuul/zuul] 885455: Use built-in URL data type instead of custom parse https://review.opendev.org/c/zuul/zuul/+/885455 | 09:32 |
|---|---|---|
| Alright, I have updated this patch according to the feedback. Switched to using URL and added test cases for the known URLs. I don't think that list is exahustive so please, let me know if there's any other URL we should add/test there. | ||
| /cc tristanC | ||
| -@gerrit:opendev.org- Flavio Percoco Premoli proposed: [zuul/zuul] 885455: Use built-in URL data type instead of custom parse https://review.opendev.org/c/zuul/zuul/+/885455 | 10:46 | |
| @rancher:matrix.org | > <@fungicide:matrix.org> Rancher: can you see if the non-canonical http://192.168.33.78:9000/t/tenant/project/root/zuul-config returns content? | 11:49 |
| It doesn't. There are some new errors: https://privatebin.net/?a04111cce0928d64#FGPStmv4aQujATJy2oqZokXL1Tx2mm8RwAxRCBfHjX9L | ||
| @tristanc_:matrix.org | flaper87: I'm presently travelling until the end of the month, so I can't really test the change. Though looking at the code, I left a comment on the review. Thanks! | 11:51 |
| @flaper87:matrix.org | tristanC: enjoy your travels. I've run some tests myself on Keycoack, Google Auth, and the general navigation for zuul. Lemme look at your review, thanks a lot | 11:52 |
| @fungicide:matrix.org | > <@rancher:matrix.org> It doesn't. There are some new errors: https://privatebin.net/?a04111cce0928d64#FGPStmv4aQujATJy2oqZokXL1Tx2mm8RwAxRCBfHjX9L | 13:49 |
| so i guess the question is why the root/zuul-config project from your gitlab connection has no metadata in the loaded tenant config. the scheduler log you pasted has tracebacks indicating those projects aren't found in the min_ltimes dict during configuration loading, i have no idea if that's related. the only thing i can think to suggest is maybe something is confused in the persisted ephemeral state in zk, so you could try https://zuul-ci.org/docs/zuul/latest/client.html#delete-state and see if you get any different results afterward. i can't rule out the possibility that you've run into a bug specific to the gitlab driver (i don't have any systems which use that one), so maybe someone who does use it will have more ideas | ||
| -@gerrit:opendev.org- Flavio Percoco Premoli proposed: [zuul/zuul] 885455: Use built-in URL data type instead of custom parse https://review.opendev.org/c/zuul/zuul/+/885455 | 14:48 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 885101: Fix setting autoholds through API with change supplied https://review.opendev.org/c/zuul/zuul/+/885101 | 22:46 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!