| @sjal:matrix.org | hey guys, wondering about the best approach for situation where check and gate are the exact same jobs but I would want to skip executing the jobs if they were already checked for the same patchset | 13:51 |
|---|---|---|
| @sjal:matrix.org | am I forced to do some API-based self-check if Gerrit had a +1 already? or there's a smarter way | 13:52 |
| @jim:acmegating.com | sjal: zuul isn't really designed for that, it's designed to catch problems that happen because other patches have merged since the last check, or external dependencies have changed, etc. | 13:54 |
| @haitaobz:matrix.org | I used old version of ZUUL, my way is saved the hash value of check status, if gate hash value is same, my job will pass | 13:56 |
| @fungicide:matrix.org | that's still risky in situations where you have integration testing across multiple projects (one of zuul's main reasons for existing), and also if you have communities that may drag their heels on reviewing changes (think a change which was uploaded and tested, then 6 months later someone approves it and the whole world has changed in the meantime) | 14:43 |
| @bridgefan:matrix.org | Hi! I discovered ansible-lint recently and was wondering if there are any zuul specific linters? I found this (https://github.com/pycontribs/zuul-lint) for config checking but wasn't sure if anything was being used by Zuul devs. | 17:52 |
| @mnaser:matrix.org | I think this isn’t maintained much anymore. I think it’s moved into schemastore as per the issues. | 17:54 |
| You can have a look at the linters job in zuul/zuul-jobs for some inspiration | ||
| @bridgefan:matrix.org | ok I will check into that. Thanks! | 17:55 |
| @bridgefan:matrix.org | Also, for I just found zuul-lint may have a new fork that seems to be ongoing called zuulcilint https://github.com/codesquadnest/zuulcilint | 18:14 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 914562: Add an upgrade test https://review.opendev.org/c/zuul/zuul/+/914562 | 20:18 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 915091: Add --disable-pipelines option https://review.opendev.org/c/zuul/zuul/+/915091 | 22:19 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:43 | |
| - [zuul/zuul] 914562: Add an upgrade test https://review.opendev.org/c/zuul/zuul/+/914562 | ||
| - [zuul/zuul] 915095: Run the upgrade test job https://review.opendev.org/c/zuul/zuul/+/915095 | ||
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!