| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 937983: Use FilterIcon for filter dropdown on QueueItem https://review.opendev.org/c/zuul/zuul/+/937983 | 13:03 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 937876: Incorporate tenants stats in Toolbar section https://review.opendev.org/c/zuul/zuul/+/937876 | 13:17 | |
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 937876: Incorporate tenants stats in Toolbar section https://review.opendev.org/c/zuul/zuul/+/937876 | 13:23 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 937895: Add "autohold_if_failure" var in job inventory https://review.opendev.org/c/zuul/zuul/+/937895 | 14:16 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 937895: Add "autohold_if_failure" var in job inventory https://review.opendev.org/c/zuul/zuul/+/937895 | 15:49 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 937895: Add "autohold_if_failure" var in job inventory https://review.opendev.org/c/zuul/zuul/+/937895 | 15:51 | |
| @harbott.osism.tech:regio.chat | could someone take a look at https://review.opendev.org/c/zuul/nodepool/+/930205? it would be helpful for our local deployment, but I'm not sure how to avoid leaking resources | 16:15 |
|---|---|---|
| @clarkb:matrix.org | Jens Harbott: I think your assumption about deleting the volume automatically is correct. Basically that only works if you have nova create the volume for you | 16:18 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: sqlreporter: expand some columns, enforce length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 18:42 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 937895: Add "autohold_if_failure" var in job inventory https://review.opendev.org/c/zuul/zuul/+/937895 | 18:44 | |
| @mhuin:matrix.org | context for this one: we have a very long root url for our log server, and our users like to define jobs with really long names. We noticed that whenever the resulting log_url would be over 255 characters, the sql reporter would fail silently to update the database, which led to incorrect results being displayed in the zuul web UI (ie jobs being reported as incomplete/in error despite completing successfully) | 18:47 |
| @jim:acmegating.com | mhu: see discussion on https://review.opendev.org/c/zuul/zuul/+/926803 and https://review.opendev.org/c/zuul/zuul-jobs/+/927582 | 19:20 |
| @mhuin:matrix.org | Ok, so I guess the first step is to get 927582 merged. Unfortunately Tristan needs to be OoO for a while, so I'll try to take over | 19:33 |
| @mhuin:matrix.org | as for the rest, I still think we shouldn't assume urls to be under 255 characters | 19:33 |
| @mhuin:matrix.org | but I can remove that part of my patch and keep at least the length verification since the sql database enforces hard limits for tenant, project, pipelines... names to be under 255 chars | 19:34 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: sqlreporter: expand some columns, enforce length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 19:42 | |
| @jim:acmegating.com | mhu: yeah, i agree in theory that the column defs should be better; i'd just like to avoid an avoidable massive data migration just for that. so if we can work around it (limits+zuul-jobs change) for now and bundle that up for the next time we have an unavoidable massive data migration, that'd be great. :) | 19:51 |
| @mhuin:matrix.org | ok... at least removing the job's name from the equation is going to be helpful in my use case, it should greatly reduce the size of the log url. This issue makes the build result page useless when we hit the length limit | 19:56 |
| @clarkb:matrix.org | One thing we (opendev) seem to struggle with is getting people to provide/use the zuul dashboard log links. Anecdotally it seems like I either get a link to a gerrit change or a link to the backend log files 90% of the time | 20:00 |
| @clarkb:matrix.org | anyway I think reducing urls down to uuid/stuff/uuid will make situations like that even more painful | 20:00 |
| @clarkb:matrix.org | it might be a good idea to encode (if we don't already I can't remember off the top of my head) the info that was in the urls inside of the job log itself so you can work backwards if necessary from a raw log to the zuul dashboard or code review system in those cases | 20:01 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 938011: sqlreporter: expand some columns, enforce length check in config https://review.opendev.org/c/zuul/zuul/+/938011 | 20:10 | |
| @jim:acmegating.com | Clark: i think the proposed new log url will be easier to backtrack because you can just search for the uuid (current backtracking involves triangulating the project+job+build). | 20:53 |
| @clarkb:matrix.org | that is a good point | 20:55 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:56 | |
| - [zuul/zuul] 937947: Add web API image delete endpoints https://review.opendev.org/c/zuul/zuul/+/937947 | ||
| - [zuul/zuul] 938022: Allow deleting images through web UI https://review.opendev.org/c/zuul/zuul/+/938022 | ||
| - [zuul/zuul] 938023: Add REST API method to trigger image build https://review.opendev.org/c/zuul/zuul/+/938023 | ||
| - [zuul/zuul] 938024: Add a web UI button to build an image https://review.opendev.org/c/zuul/zuul/+/938024 | ||
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!