| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 831895: functional-test : ensure glean started https://review.opendev.org/c/zuul/nodepool/+/831895 | 00:43 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 831895: functional-test : ensure glean started https://review.opendev.org/c/zuul/nodepool/+/831895 | 03:54 | |
| @iwienand:matrix.org | Clark: ^ feeling like that is the kiss approach that would have correctly failed with the latest rocky work and the missing glean matches. interested in alternative ideas | 05:02 |
|---|---|---|
| -@gerrit:opendev.org- Albin Vass proposed: [zuul/zuul] 830840: Make it possible to configure job retries with zuul_return https://review.opendev.org/c/zuul/zuul/+/830840 | 08:33 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 832258: web: add skip_stats param to api/tenants https://review.opendev.org/c/zuul/zuul/+/832258 | 15:03 | |
| @mhuin:matrix.org | corvus: regarding your comment on https://review.opendev.org/c/zuul/zuul/+/832258 I proposed it following tobiash 's comment on https://review.opendev.org/c/zuul/zuul/+/810955 | 15:49 |
| @mhuin:matrix.org | I can do a full-on cache if preferred, or if needed at all | 15:49 |
| @mhuin:matrix.org | oh, wait - do you mean we could drop the queue_size entirely? | 15:50 |
| @tobias.henkel:matrix.org | the queue size is part of the tenant list page which atm sums the queue lengths of all pipelines of the tenant | 15:51 |
| @mhuin:matrix.org | from a quick search in the code, it doesn't seem to be used anywhere, react app included | 15:51 |
| @mhuin:matrix.org | ah my bad, wrong string search | 15:52 |
| @jim:acmegating.com | yeah, i'm wondering if we can just stop reporting the queue size -- i haven't looked into the code history to find out why we added it. was it important? or was it just "nice to have" and "free"? if it was not that important, then now that it's expensive, maybe we can just remove it. or if it is semi-important, maybe we can default it to off? | 15:53 |
| @tobias.henkel:matrix.org | well I personally look quite often at the tenant list to get q quick glance how crowded a tenant is atm | 15:55 |
| @tobias.henkel:matrix.org | so I like this column | 15:55 |
| @jim:acmegating.com | okay, that sounds good enough to me to keep it. :) | 15:58 |
| @mhuin:matrix.org | works for me too, so what do you prefer? the skip_stats param or a cache? | 15:58 |
| @tobias.henkel:matrix.org | we still could either cache it (preferred by me to also speed up the tenant page) or add skip_stats for the pure tenant drop down | 15:59 |
| @jim:acmegating.com | cache seems like win-win | 15:59 |
| @mhuin:matrix.org | okay I'll leave the skip_stats change open and work on the cache then | 16:00 |
| @mhuin:matrix.org | thks | 16:00 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 832293: REST API: cache tenants list https://review.opendev.org/c/zuul/zuul/+/832293 | 17:33 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 832293: REST API: cache tenants list https://review.opendev.org/c/zuul/zuul/+/832293 | 19:39 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:38 | |
| - [zuul/zuul] 832361: Support submitWholeTopic in Gerrit https://review.opendev.org/c/zuul/zuul/+/832361 | ||
| - [zuul/zuul] 832362: Report per-branch cyclic-dependency conflicts https://review.opendev.org/c/zuul/zuul/+/832362 | ||
| - [zuul/zuul] 832363: Add queue.dependencies-by-topic https://review.opendev.org/c/zuul/zuul/+/832363 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 832384: Add Felix to Zuul Maintainers https://review.opendev.org/c/zuul/zuul/+/832384 | 22:50 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 832388: Add Simon to Zuul Maintainers https://review.opendev.org/c/zuul/zuul/+/832388 | 22:51 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 832391: Remove inactive maintainers https://review.opendev.org/c/zuul/zuul/+/832391 | 22:53 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 832391: Remove inactive maintainers https://review.opendev.org/c/zuul/zuul/+/832391 | 22:54 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!