Thursday, 2023-07-20

-@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/+/88901409: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/+/88901410:44
@flaper87:matrix.orgcorvus tristanC  I just noticed this in the latest patchset Radoslaw uploaded: https://review.opendev.org/c/zuul/zuul/+/885455/11/web/src/api.js#4112:39
@flaper87:matrix.orgI'm afraid that is something we don't want to change as it may result in a backwards incompatible change12:39
@flaper87:matrix.orgAh, nvm, I just re-read the docs and it does resolve the empty-string to `/` 12:44
@yoctozepto:matrix.orghey, trust me more ;p I may be rudely hijacking changes but I am not delivering obviously bad code (usually :D )12:59
@flaper87:matrix.orgHas 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.orgyoctozepto: oh, you Radoslaw ? Nice, I didn't know you were in here :D 13:00
@yoctozepto:matrix.orgyeah, it's me13:01
@yoctozepto:matrix.orgI remind ppl my nick on the mailing list13:01
@flaper87:matrix.orgyoctozepto:  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 review13:02
@yoctozepto:matrix.orgno problem, and sorry again for the bad hijack, /me used to other practices in the very same interface13:02
@flaper87:matrix.orgyou 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.orgno, I just gave you the harsh treatment of the deeper wilderness of openinfra 🤣13:16
@yoctozepto:matrix.orgbut I'm glad we are good13: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/+/88910414: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/+/88545521: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/+/88914122: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/!