| corvus | #status log restart zuul-launchers with node locality fix | 00:36 |
|---|---|---|
| corvus | er, "restarted", but you get the idea | 00:36 |
| opendevstatus | corvus: finished logging | 00:38 |
| opendevreview | Merged opendev/system-config master: Fix zuul scheduler container name in docker exec commands https://review.opendev.org/c/opendev/system-config/+/954051 | 01:01 |
| corvus | i think the most helpful changes to merge for the niz effort would be these two: | 02:06 |
| corvus | https://review.opendev.org/c/openstack/diskimage-builder/+/721581 | 02:06 |
| corvus | https://review.opendev.org/c/opendev/zuul-providers/+/953269 | 02:06 |
| corvus | mnasiadka: ^ do you want to take 721581 out of WIP? | 02:09 |
| *** ykarel__ is now known as ykarel | 04:10 | |
| mnasiadka | corvus: I can’t, I’m not the owner :) | 05:11 |
| mnasiadka | frickler: can you take 721581 out of WIP pretty please? :-) | 05:14 |
| frickler | seems I need to get out my superpowers to do so, will have a closer look soonish | 07:05 |
| mnasiadka | thanks :) | 07:20 |
| mnasiadka | And I assume we'll need a DIB release so zuul-providers jobs can consume it | 07:25 |
| mnasiadka | But that can wait until US folks wake up :) | 07:25 |
| mnasiadka | ok, gating | 07:41 |
| frickler | I haven't found a way to do this via gerrit CLI, so needed to make myself Administrator and use the UI. for a release I would try to get https://review.opendev.org/c/openstack/diskimage-builder/+/951469 updated and merged, too | 07:44 |
| opendevreview | Merged openstack/diskimage-builder master: Retry git clone/fetch on errors https://review.opendev.org/c/openstack/diskimage-builder/+/721581 | 08:56 |
| mnasiadka | frickler: looking at Debian bug report we’re not getting s-p soon - so we need that for trixie as well? | 09:23 |
| frickler | mnasiadka: yes, that's the change I discussed with clarkb the other day and that still needs to be made in the code, but I didn't get to that yet due to downstream things. if you want to amend the change I'd be grateful | 09:44 |
| mnasiadka | I’ll have some time in an hour - will do that | 09:55 |
| opendevreview | Michal Nasiadka proposed openstack/diskimage-builder master: debian: software-properties-common isn't in testing https://review.opendev.org/c/openstack/diskimage-builder/+/951469 | 12:52 |
| mnasiadka | frickler: ^^ hopefully I did understand what needs to be done :) | 12:55 |
| frickler | mnasiadka: looks reasonable at a glance, not sure about unstable, but I think it doesn't hurt, either. let's wait for CI results | 13:02 |
| mnasiadka | I have no clue if we should care about unstable/testing builds, but it won't hurt to make it this way | 13:02 |
| frickler | well testing is currently needed because trixie isn't released yet, so I build the image under the testing name for which there is some other repo workaround in dib | 13:06 |
| fungi | yes, this is a temporary situation for another month or two | 13:06 |
| fungi | like adding ubuntu 2026.4 images when it's in beta before it officially releases | 13:07 |
| frickler | well I like your optimism, but yeah ;) we've usually refused to do this with nodepool, but I'd think with zuul-launcher things are easy enough to update now that this is no longer a blocker. and getting native py313 testing is IMO worth a bit of effort | 13:10 |
| fungi | in the past we've added ubuntu lts images while they were still in beta, in order to have them already available at the time they officially released rather than weeks or months after | 13:12 |
| fungi | and i think we did it with one or two debian releases as well, though not as often | 13:12 |
| fungi | mostly it was because openstack's pti for the release cycle starting around the time of a new ubuntu lts may want to reference that version, so being able to use it right away helps | 13:14 |
| mnasiadka | frickler: if there is another workaround in place for testing, then it makes sense to add that one as well | 13:14 |
| mnasiadka | so let's see if all jobs pass | 13:16 |
| opendevreview | James E. Blair proposed opendev/zuul-providers master: Run DIB from git instead of release https://review.opendev.org/c/opendev/zuul-providers/+/954126 | 13:42 |
| corvus | we can merge that and we don't have to make a dib release | 13:43 |
| corvus | clarkb isn't around today, but we talked about it in the meeting, so i know he's in favor of that. | 13:51 |
| mnasiadka | that makes a lot of sense | 13:58 |
| opendevreview | James E. Blair proposed opendev/zuul-providers master: Fix zuul_success detection https://review.opendev.org/c/opendev/zuul-providers/+/954129 | 14:00 |
| opendevreview | Cyril Roelandt proposed openstack/project-config master: Add centos-10-stream nodepool image https://review.opendev.org/c/openstack/project-config/+/954136 | 14:15 |
| corvus | i left a review on that redirecting them to zuul-providers | 14:23 |
| corvus | i don't think we're quite ready to remove the nodepool config, but at this point, i don't think it's worth adding anything new. just unfortunate timing that someone showed up with a new image during this window. | 14:24 |
| mnasiadka | corvus: should we add new images like stream10 or rocky10 without the niz- prefix? | 14:53 |
| mnasiadka | corvus: seems 954129 will not get in (git timeouts) without 954126 being merged first | 15:12 |
| corvus | yeah, i think we can omit niz- from new labels | 15:15 |
| corvus | and i agree, https://review.opendev.org/954126 is probably the highest priority niz change now | 15:17 |
| opendevreview | James E. Blair proposed opendev/zuul-providers master: Fix zuul_success detection https://review.opendev.org/c/opendev/zuul-providers/+/954129 | 15:18 |
| corvus | (i just tweaked the commit msg on that) | 15:18 |
| opendevreview | Merged openstack/diskimage-builder master: debian: software-properties-common isn't in testing https://review.opendev.org/c/openstack/diskimage-builder/+/951469 | 18:21 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!