| -@gerrit:opendev.org- Michael Kelly proposed on behalf of Tristan Cacqueray https://matrix.to/#/@tristanc_:matrix.org: [zuul/zuul-operator] 852855: Update zookeeper image to 3.8.3 https://review.opendev.org/c/zuul/zuul-operator/+/852855 | 00:05 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 910556: Publish container images to quay.io https://review.opendev.org/c/zuul/zuul-operator/+/910556 | 00:19 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 00:25 | |
| - [zuul/zuul] 902180: Add a filter icon to config error values https://review.opendev.org/c/zuul/zuul/+/902180 | ||
| - [zuul/zuul] 902185: Try left alignment of zoomy icons https://review.opendev.org/c/zuul/zuul/+/902185 | ||
| - [zuul/zuul] 910558: Add filter actions menu to error table https://review.opendev.org/c/zuul/zuul/+/910558 | ||
| @jim:acmegating.com | Clark: fungi ^ you reviewed the first two changes there; i think i like the third one that i just wrote for actual merging. they are alternatives, and if we pick one we like, i can squash as appropriate. | 00:32 |
|---|---|---|
| @clarkb:matrix.org | I'll have to pull that up in the site preview as I'm not quite sure I get what the third one does from the commit message | 00:35 |
| @jim:acmegating.com | https://e02da289fa5cd71d2848-a802bb880ba142924be00bfc16ee185a.ssl.cf5.rackcdn.com/910558/1/check/zuul-build-dashboard-opendev/97ea07e/npm/html/ | 00:36 |
| @jim:acmegating.com | huh that doesn't look right | 00:37 |
| @jim:acmegating.com | oh wow, i think that's actually a difference between the different ways of building js used in local dev vs the preview build. neat. | 00:40 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 910558: Add filter actions menu to error table https://review.opendev.org/c/zuul/zuul/+/910558 | 00:47 | |
| @jim:acmegating.com | let's try that | 00:47 |
| @jim:acmegating.com | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_2d5/910558/2/check/zuul-build-dashboard-opendev/2d5aab8/npm/html/ | 00:57 |
| @jim:acmegating.com | that one looks better ^ | 00:57 |
| @fungicide:matrix.org | cool, just started to check on it myself | 00:57 |
| @fungicide:matrix.org | neat! yep, that works great as far as i can tell | 00:59 |
| @fungicide:matrix.org | maybe Clark has time to check them too, he groks js better than i do, but lgtm | 01:00 |
| @jim:acmegating.com | do you prefer that to the earlier versions? | 01:01 |
| @fungicide:matrix.org | i do think i have a slight preference for it over the other | 01:01 |
| @fungicide:matrix.org | cleaner | 01:02 |
| @clarkb:matrix.org | I haven't reviewed the code yet but I think number 3 works | 01:09 |
| @clarkb:matrix.org | its a bit less cluttered too so I'm happy with it | 01:09 |
| @fungicide:matrix.org | yeah, i'm a fan of minimalism where applicable | 01:09 |
| @jim:acmegating.com | cool, i'll go ahead and squash it and abandon the others | 01:11 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 910558: Add filter actions menu to error table https://review.opendev.org/c/zuul/zuul/+/910558 | 01:13 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 910556: Publish container images to quay.io https://review.opendev.org/c/zuul/zuul-operator/+/910556 | 02:51 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 910556: Publish container images to quay.io https://review.opendev.org/c/zuul/zuul-operator/+/910556 | 03:10 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 910556: Publish container images to quay.io https://review.opendev.org/c/zuul/zuul-operator/+/910556 | 03:16 | |
| -@gerrit:opendev.org- Michael Kelly proposed: [zuul/zuul-operator] 910556: Publish container images to quay.io https://review.opendev.org/c/zuul/zuul-operator/+/910556 | 03:40 | |
| -@gerrit:opendev.org- Lukas Kranz marked as active: [zuul/zuul-jobs] 887917: prepare-workspace-git: Add ability to define synced pojects https://review.opendev.org/c/zuul/zuul-jobs/+/887917 | 08:40 | |
| -@gerrit:opendev.org- Lukas Kranz proposed: [zuul/zuul-jobs] 910582: Make prepare-workspace-git fail faster. https://review.opendev.org/c/zuul/zuul-jobs/+/910582 | 08:49 | |
| @clarkb:matrix.org | corvus: if nodepool 10.0.0 hasn't been tagged yet. Did we merge the hostname override removal change yet? This is not backward compatible so should go on 10.0.0 | 15:26 |
| @jim:acmegating.com | Clark: we did and it has (i agree and i was looking out for that) | 15:28 |
| @jim:acmegating.com | i'm going to send out the email shortly | 15:29 |
| @clarkb:matrix.org | corvus: oh good. I woke up and for whatever reason this occurred to me | 16:15 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 910550: Emit per-branch queue stats separately https://review.opendev.org/c/zuul/zuul/+/910550 | 17:03 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 910558: Add filter actions menu to error table https://review.opendev.org/c/zuul/zuul/+/910558 | 17:34 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!