| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848576: Fix flaky zuul-web tests using cached tenant list https://review.opendev.org/c/zuul/zuul/+/848576 | 07:53 | |
| @westphahl:matrix.org | zuul-maint: ^ 848576 fixes the flaky zuul-web tests. tests were unreliable due to cache timings | 07:56 |
|---|---|---|
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848234: Fix read-only branches error in zuul-web https://review.opendev.org/c/zuul/zuul/+/848234 | 09:35 | |
| -@gerrit:opendev.org- Simon Westphahl proposed on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 848524: Add branch cache fetch failure tests https://review.opendev.org/c/zuul/zuul/+/848524 | 09:35 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848632: Only update commit dependencies when necessary https://review.opendev.org/c/zuul/zuul/+/848632 | 12:25 | |
| @westphahl:matrix.org | corvus: sorry, just noticed that you already had a fix in 848522 for one of flaky tests. Should we go with your`cache_expiry = 0` or the `iterate_timeout()` that I used? | 13:27 |
| @jim:acmegating.com | swest: cache_expiry=0 is better i think | 13:41 |
| @jim:acmegating.com | usually any time we can get the test running reliably in one pass rather than iterating+waiting is best for the whole test suite i think | 13:42 |
| @westphahl:matrix.org | corvus: k, I'll rebase my change on 848522 and use the `cache_expiry=0` | 13:43 |
| @jim:acmegating.com | mhu tristanC felixedel see https://review.opendev.org/848522 please (re ^) | 13:45 |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848576: Fix flaky zuul-web tests using cached tenant list https://review.opendev.org/c/zuul/zuul/+/848576 | 13:49 | |
| @westphahl:matrix.org | corvus: ^ using cache expiry now | 13:50 |
| @westphahl:matrix.org | hm , I think test_empty_config_startup needs a different fix. probably need to wait for the local layout update of zuul-web to complete | 13:53 |
| @jim:acmegating.com | swest: i don't think that test hits the tenant list api endpoint | 13:56 |
| @westphahl:matrix.org | yeah, it's a different raise. we need to wait for the layout state of zuul-web to be == scheduler layout | 13:57 |
| @westphahl:matrix.org | * yeah, it's a different race. we need to wait for the layout state of zuul-web to be == scheduler layout | 13:57 |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848576: Fix flaky zuul-web tests using cached tenant list https://review.opendev.org/c/zuul/zuul/+/848576 | 13:59 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 848576: Fix flaky zuul-web tests using cached tenant list https://review.opendev.org/c/zuul/zuul/+/848576 | 13:59 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848644: [WIP] web UI: improve API error handling https://review.opendev.org/c/zuul/zuul/+/848644 | 14:52 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848132: zuul-web: return errors as JSON https://review.opendev.org/c/zuul/zuul/+/848132 | 15:02 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 848644: [WIP] web UI: improve API error handling https://review.opendev.org/c/zuul/zuul/+/848644 | 15:07 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 848522: Fix race is test_tenant_add_remove https://review.opendev.org/c/zuul/zuul/+/848522 | 15:29 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!