-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 889014: Don't cancel Github check-run during re-enqueue https://review.opendev.org/c/zuul/zuul/+/889014 | 09:03 | |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 889014: Don't cancel Github check-run during re-enqueue https://review.opendev.org/c/zuul/zuul/+/889014 | 10:44 | |
@flaper87:matrix.org | corvus tristanC I just noticed this in the latest patchset Radoslaw uploaded: https://review.opendev.org/c/zuul/zuul/+/885455/11/web/src/api.js#41 | 12:39 |
---|---|---|
@flaper87:matrix.org | I'm afraid that is something we don't want to change as it may result in a backwards incompatible change | 12:39 |
@flaper87:matrix.org | Ah, nvm, I just re-read the docs and it does resolve the empty-string to `/` | 12:44 |
@yoctozepto:matrix.org | hey, trust me more ;p I may be rudely hijacking changes but I am not delivering obviously bad code (usually :D ) | 12:59 |
@flaper87:matrix.org | Has anyone encountered this error message when using Zuul with Github: | 13:00 |
> Failed to update check run zuul-infra/check: 422 Client Error: Unprocessable Entity for url: https://api.github.com/repos/ORGANIZATION/REPOSITORY/check-runs/15202207821 | ||
It doesn't always happen, just for some PRs and I can't point my finger at the actual problem | ||
@flaper87:matrix.org | yoctozepto: oh, you Radoslaw ? Nice, I didn't know you were in here :D | 13:00 |
@yoctozepto:matrix.org | yeah, it's me | 13:01 |
@yoctozepto:matrix.org | I remind ppl my nick on the mailing list | 13:01 |
@flaper87:matrix.org | yoctozepto: cool cool! yeah, I'm not in the ML. Enough emails for me but nice to know you are here. Thanks again for the feedback in the review | 13:02 |
@yoctozepto:matrix.org | no problem, and sorry again for the bad hijack, /me used to other practices in the very same interface | 13:02 |
@flaper87:matrix.org | you are good! I guess some things have changed in some parts of the Open Infra workflows after my departure and I was caught off-guard 🤣 | 13:06 |
@yoctozepto:matrix.org | 🤣 | 13:16 |
@yoctozepto:matrix.org | no, I just gave you the harsh treatment of the deeper wilderness of openinfra 🤣 | 13:16 |
@yoctozepto:matrix.org | but I'm glad we are good | 13:17 |
-@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 889104: Don't schedule repo state for items w/o jobs https://review.opendev.org/c/zuul/zuul/+/889104 | 14:14 | |
-@gerrit:opendev.org- Zuul merged on behalf of Flavio Percoco Premoli: [zuul/zuul] 885455: Use built-in URL data type instead of custom parse https://review.opendev.org/c/zuul/zuul/+/885455 | 21:40 | |
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/nodepool] 889141: DNM: Add extra debugging to AWS leaked uploads https://review.opendev.org/c/zuul/nodepool/+/889141 | 22:27 | |
-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 23:20 | |
- [zuul/zuul] 885380: Add manager/reporter support for config warnings https://review.opendev.org/c/zuul/zuul/+/885380 | ||
- [zuul/zuul] 885415: Attach loading_errors to the ParseContext https://review.opendev.org/c/zuul/zuul/+/885415 | ||
- [zuul/zuul] 885416: Add configuration warnings https://review.opendev.org/c/zuul/zuul/+/885416 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!