| -@gerrit:opendev.org- Zuul merged on behalf of Benjamin Schanzel: [zuul/zuul-jobs] 944878: mirror-container-images: use skopeo to mirror multiarch images https://review.opendev.org/c/zuul/zuul-jobs/+/944878 | 00:00 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Benjamin Schanzel: [zuul/zuul] 939969: web: Add an option to sort pipelines https://review.opendev.org/c/zuul/zuul/+/939969 | 00:16 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Benjamin Schanzel: | 00:20 | |
| - [zuul/zuul] 940439: web: Give root element whole viewport height https://review.opendev.org/c/zuul/zuul/+/940439 | ||
| - [zuul/nodepool] 945211: Replace assertDictContainsSubset in tests https://review.opendev.org/c/zuul/nodepool/+/945211 | ||
| @aureliojargas:matrix.org | Hi Clark, I have addressed your comments and all the tests succeeded. It's ready for re-review: https://review.opendev.org/c/zuul/zuul-jobs/+/941490 | 06:27 |
|---|---|---|
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/zuul] 945423: WIP: Changes needed for run tests in container https://review.opendev.org/c/zuul/zuul/+/945423 | 07:00 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 940500: web: Use a select filter for pipelines and queues on status page https://review.opendev.org/c/zuul/zuul/+/940500 | 09:47 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 940379: web: Upgrade nodejs to latest v23 https://review.opendev.org/c/zuul/zuul/+/940379 | 09:51 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 944328: Avoid reconfig for quick create/delete of branches https://review.opendev.org/c/zuul/zuul/+/944328 | 10:13 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 943385: Remove deprecated status_url config and format https://review.opendev.org/c/zuul/zuul/+/943385 | 11:58 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 943385: Remove deprecated status_url config and format https://review.opendev.org/c/zuul/zuul/+/943385 | 12:03 | |
| -@gerrit:opendev.org- Dong Zhang proposed wip: [zuul/nodepool] 921425: WIP: Run tests in container https://review.opendev.org/c/zuul/nodepool/+/921425 | 12:58 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 945457: web: Pass auth header to log endpoint if serving from the same origin https://review.opendev.org/c/zuul/zuul/+/945457 | 13:43 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/nodepool] 943389: Clean up empty zk image build nodes https://review.opendev.org/c/zuul/nodepool/+/943389 | 13:54 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/nodepool] 943389: Clean up empty zk image build nodes https://review.opendev.org/c/zuul/nodepool/+/943389 | 13:58 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed: [zuul/zuul] 940379: web: Upgrade nodejs to latest v23 https://review.opendev.org/c/zuul/zuul/+/940379 | 14:17 | |
| -@gerrit:opendev.org- Dong Zhang proposed: [zuul/nodepool] 943389: Clean up empty zk image build nodes https://review.opendev.org/c/zuul/nodepool/+/943389 | 14:21 | |
| -@gerrit:opendev.org- Simon Westphahl proposed: [zuul/zuul] 943385: Remove deprecated status_url config and format https://review.opendev.org/c/zuul/zuul/+/943385 | 14:23 | |
| @mordred:waterwanders.com | Clark: https://jira.mariadb.org/browse/MDEV-35663 | 16:13 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 945481: Fix sql query timeout https://review.opendev.org/c/zuul/zuul/+/945481 | 16:15 | |
| @mordred:waterwanders.com | Clark: Fixed in 10.5.28, 10.6.21, 10.11.11, 11.4.5, 11.7.2 | 16:15 |
| @mordred:waterwanders.com | I mean - our fix is still better - those calls were not needed - but we were also right, there was a bug | 16:15 |
| @clarkb:matrix.org | nice. I guess its good they've already fixed it even if we won't get the fix for a bit. Any idea where the actual code fixes are? I'm having a hard time finding them in jira. Maybe I have to map those numbers next to releases to github PRs or something | 16:17 |
| @jim:acmegating.com | https://review.opendev.org/945481 should fix the opendev timeout issue | 16:18 |
| @clarkb:matrix.org | no the naive copy number into github and go idea didn't work | 16:18 |
| @clarkb:matrix.org | corvus: did you want to rebase my mariadb stack on top of that or the other way around? Or is this trivial enough we're happy with it independently? | 16:19 |
| @jim:acmegating.com | independent is fine i think | 16:19 |
| @clarkb:matrix.org | I guess the postgres path would cover taht too +2 from me | 16:20 |
| @clarkb:matrix.org | as a heads up the two mariadb changes have enough +2s now to get approved but I've got meetings this morning then the weather here is great today so I'm popping out this afternoon. I'll plan to approve those changes tomorrow when I can better monitor for unexpected fallout | 16:22 |
| @mordred:waterwanders.com | Clark: https://github.com/MariaDB/server/commit/0031f4a74fcf7c61493409af5da5ab2c90905113 | 16:26 |
| @mordred:waterwanders.com | Clark: turns out you have to be logged in to jira for the git sidebar to work - but once it does it'll show links to the commit and whatnot | 16:27 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul-jobs] 944813: Add upload-image-s3 role https://review.opendev.org/c/zuul/zuul-jobs/+/944813 | 16:28 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 945419: Add logging of kubectl process group id to executor https://review.opendev.org/c/zuul/zuul/+/945419 | 16:29 | |
| @clarkb:matrix.org | thanks | 16:32 |
| @clarkb:matrix.org | zuulians https://review.opendev.org/c/zuul/zuul/+/945418 is a safe change that only drops the multi scheduler job from check and adds it to experimental. This should speed up the round trip time on getting check results. It is failing ci on unrelated errors but I think can be safeyl approved if that sounds like a good idea to anyone else | 17:02 |
| @jim:acmegating.com | i've previously discussed that with felixedel and i think that's fine | 17:35 |
| @clarkb:matrix.org | Aurelio Jargas: all of those changes lgtm | 17:38 |
| @jim:acmegating.com | small comment on the first one | 17:52 |
| @clarkb:matrix.org | good catch | 17:55 |
| -@gerrit:opendev.org- Clark Boylan proposed: [zuul/zuul] 945521: Add python3.13 unittest job https://review.opendev.org/c/zuul/zuul/+/945521 | 19:42 | |
| @clarkb:matrix.org | corvus: ^ it was quick to push that up and see if we are even compatible with python3.13 so I went ahead and did it | 19:42 |
| @clarkb:matrix.org | but as mentioned I'm not sure that will give us good performance feedback | 19:43 |
| -@gerrit:opendev.org- Aurelio Jargas proposed: [zuul/zuul-jobs] 941490: Add role: `ensure-python-command`, refactor similar roles https://review.opendev.org/c/zuul/zuul-jobs/+/941490 | 21:06 | |
| @aureliojargas:matrix.org | BTW, when migrating a project to Python 3.13 at work, instead of relying in that version being installed in the CI disk image, we switched to use `uv` to install Python on-the-fly in the CI (downloading from an on-premise GitHub mirror), and the speed was crazy. `uv sync` takes 3 seconds to download and install Python **and** install all the project dependencies, were previously `pip install` took 10 seconds to only install the dependencies. 🤯 | 21:29 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 945481: Fix sql query timeout https://review.opendev.org/c/zuul/zuul/+/945481 | 21:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!