| opendevreview | Tony Breeds proposed zuul/zuul-jobs master: ensure-devstack: Add the ability is specify a custom cpu model https://review.opendev.org/c/zuul/zuul-jobs/+/949547 | 00:58 |
|---|---|---|
| opendevreview | Tony Breeds proposed openstack/diskimage-builder master: WIP: Add support for CentOS Stream 10 https://review.opendev.org/c/openstack/diskimage-builder/+/934045 | 01:03 |
| opendevreview | Tony Breeds proposed opendev/glean master: Add support for CentOS 10 keyfiles https://review.opendev.org/c/opendev/glean/+/941672 | 01:15 |
| opendevreview | Qihui Zhao proposed openstack/project-config master: to create a new repo under cfn for a new-initiated sub-group heteregeneous distributed training framework https://review.opendev.org/c/openstack/project-config/+/949549 | 01:39 |
| opendevreview | Qihui Zhao proposed openstack/project-config master: [200~to create a new repo under cfn for a new-initiated sub-group heteregeneous distributed training framework https://review.opendev.org/c/openstack/project-config/+/949552 | 01:57 |
| opendevreview | Qihui Zhao proposed openstack/project-config master: to create a new repo for a cfn new launched sub-group heterogeneous distributed training framework https://review.opendev.org/c/openstack/project-config/+/949555 | 03:31 |
| opendevreview | Qihui Zhao proposed openstack/project-config master: to create a new repo for a cfn new launched sub-group heterogeneous distributed training framework https://review.opendev.org/c/openstack/project-config/+/949555 | 03:33 |
| opendevreview | Qihui Zhao proposed openstack/project-config master: to create a new repo for a cfn new launched sub-group heterogeneous distributed training framework https://review.opendev.org/c/openstack/project-config/+/949555 | 03:42 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add image builds and provider config for Debian Bookworm https://review.opendev.org/c/opendev/zuul-providers/+/949127 | 06:39 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add bookworm/arm64 image builds https://review.opendev.org/c/opendev/zuul-providers/+/949482 | 06:39 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add bookworm and bullseye arm64 image builds https://review.opendev.org/c/opendev/zuul-providers/+/949482 | 06:44 |
| mnasiadka | Ok, that should be good now. | 06:45 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add bookworm and bullseye arm64 image builds https://review.opendev.org/c/opendev/zuul-providers/+/949482 | 06:51 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-stream-9 image definitions https://review.opendev.org/c/opendev/zuul-providers/+/949567 | 07:18 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 07:58 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-stream-9 image definitions https://review.opendev.org/c/opendev/zuul-providers/+/949567 | 08:00 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 08:00 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 08:00 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 08:44 |
| frickler | infra-root: missed to add this to the agenda (and likely won't be at the meering, either) but maybe we should discuss a strategy for dealing with the pbr CI issues? https://review.opendev.org/q/project:openstack/pbr+status:open or should this rather be discussed in the oslo team context? | 09:06 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 09:50 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: WIP: Use bindep to install DIB dependencies https://review.opendev.org/c/zuul/zuul-jobs/+/949589 | 10:04 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 10:05 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 10:12 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: Use bindep to install DIB dependencies https://review.opendev.org/c/zuul/zuul-jobs/+/949589 | 10:16 |
| opendevreview | Michal Nasiadka proposed openstack/diskimage-builder master: bindep: Add dnf and dnf-plugins-core https://review.opendev.org/c/openstack/diskimage-builder/+/949590 | 10:22 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 10:23 |
| *** tosky_ is now known as tosky | 12:27 | |
| opendevreview | Michal Nasiadka proposed openstack/diskimage-builder master: bindep: Add dnf and dnf-plugins-core https://review.opendev.org/c/openstack/diskimage-builder/+/949590 | 13:28 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: Use bindep to install DIB dependencies https://review.opendev.org/c/zuul/zuul-jobs/+/949589 | 13:34 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: Use bindep to install DIB dependencies https://review.opendev.org/c/zuul/zuul-jobs/+/949589 | 13:41 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 13:49 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 13:50 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 13:58 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 14:04 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 14:05 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 14:05 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 14:09 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 14:20 |
| mnasiadka | clarkb, corvus : Seems niz-debian is sorted out - so if you can have a look in https://review.opendev.org/q/topic:%22niz-debian%22 I'd be grateful | 14:23 |
| mnasiadka | Also merging https://review.opendev.org/c/opendev/zuul-providers/+/949567/2 would unblock my CentOS work | 14:23 |
| clarkb | frickler: I think pbr should be discussed within the context of oslo. There are interested parties that would not be invovled if we tried to drive it on our side. We are obviously impacted though and should participate in those central discussions should they happen | 14:23 |
| fungi | i concur | 14:24 |
| clarkb | my main feedback at this point (if I don't make it to that central discussion) is that I think an "ideal" state (the one least likely to produce future problems for us) is one where pbr prefers pyproject.toml for configuration and uses "accepted" setuptools hook points (like those used by setuptools scm). | 14:28 |
| clarkb | I thik if we had all the time in the world we could also consider what it would take to drop pbr in favor of setuptools scm (this would require us to improve setuptools scm version handling or drop openstack's relianceo n pbr's future version generation stuff). And/or use some other package tool that can meet these needs. And then if we had all the time and were also bored we could | 14:29 |
| clarkb | make pbr its own package tool that doesn't rely on external tools | 14:29 |
| corvus | mnasiadka: +2 but please don't add any more normal labels... there was originally only one and it was deprecated | 14:32 |
| mnasiadka | yes, once it merges I will delete all normal labels and flavors | 14:33 |
| mnasiadka | if that's ok | 14:33 |
| clarkb | https://review.opendev.org/c/opendev/system-config/+/949544 gitea 1.23.8 appears to look good in screenshots | 14:33 |
| corvus | mnasiadka: ++ yeah i tried to write that patch but it merge conflicts and i don't want to force a rebase so that plan sgtm | 14:33 |
| mnasiadka | sure, will handle it | 14:37 |
| clarkb | re gitea I've got meetings all morning and then supposedly today is the only day this week that won't rain so I'm probably going to try and pop outside for a bit this afternoon. I don't expect gitea issues (but reviews very much welcome) if we just want to send it sometime today. Or I can get to it first thing tomorrow morning | 14:43 |
| clarkb | mnasiadka: I'm not sure i understand why https://review.opendev.org/c/openstack/diskimage-builder/+/949493 makes things better? | 14:46 |
| clarkb | shouldn't that be a noop as it doesn't affect the return code of the type built in? | 14:46 |
| mnasiadka | clarkb: Initially I thought the grub2-mkconfig erroring out is the problem, but yeah - I can remove that from Depends-On | 14:46 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add bookworm and bullseye arm64 image builds https://review.opendev.org/c/opendev/zuul-providers/+/949482 | 14:47 |
| fungi | i'll try to take a look at the gitea upgrade change during meetings | 14:47 |
| clarkb | mnasiadka: re https://review.opendev.org/c/opendev/zuul-providers/+/949127/8..9/zuul.d/image-build-jobs.yaml I think I remember what is going on there now. That specifies mbr partitions which don't work for arm64. Removign them allows dib to be smarter about image partitions based on the arch I think | 14:49 |
| clarkb | mnasiadka: and I want to say that mbr stuff was added to build images that exercised dib's partitioning abilities in a low risk way. I would say that isn't necessary anymore and the cleanup is appropriate for niz | 14:49 |
| mnasiadka | clarkb: yes, that's the case - although nodepool images have block-device-efi enabled by default - https://opendev.org/openstack/project-config/src/commit/d709e82f709f83f21beb2ab851797357048cd2cf/nodepool/nb07.opendev.org.yaml#L103 | 14:50 |
| mnasiadka | Not sure if we want to do that in niz as well | 14:50 |
| clarkb | mnasiadka: if the arm64 images boot as is then I think its fine with out that. If they do not boot then we should add that back in | 14:51 |
| clarkb | I went ahead and approved https://review.opendev.org/c/opendev/zuul-providers/+/949482/ carrying over corvus' previous +2 since the commit message update was minimal | 14:51 |
| clarkb | mnasiadka: I think there is some logic in dib to determine what sort of block device to use. Its possible we were just being explicit about something that was the default anyway | 14:52 |
| clarkb | mnasiadka: if [[ "arm64 aarch64" =~ $ARCH ]] ; then elements="$elements block-device-efi" from diskimage_builder/lib/common-functions | 14:53 |
| clarkb | so ya that is probably redundant | 14:53 |
| clarkb | (I'm not positive that function gets called but the implication is that dib knows it should do this) | 14:54 |
| mnasiadka | clarkb: they do boot, but I think we end up with -mbr for x86 and -efi for arm64, doubt it's a problem - just noting the difference | 14:57 |
| clarkb | mnasiadka: yup that is expected | 14:57 |
| clarkb | currently not all of the x86 clouds can boot gpt/efi iirc. But they can all do mbr. And for arm64 they can only do gpt/efi | 14:58 |
| clarkb | depending on your block sizes and partition tables I think you can produce images that are both mbr and gpt. But that is dependent on the sizes of entries iirc so not a generic solution | 14:58 |
| opendevreview | Merged zuul/zuul-jobs master: Add support for using depends-on in ensure-dib role https://review.opendev.org/c/zuul/zuul-jobs/+/949514 | 14:58 |
| mnasiadka | clarkb: if you could also merge https://review.opendev.org/c/opendev/zuul-providers/+/949567 - I could continue with CentOS | 15:11 |
| clarkb | done | 15:13 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 15:16 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 15:16 |
| mnasiadka | thanks | 15:16 |
| *** ralonsoh is now known as ralonsoh_out | 16:03 | |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 16:05 |
| opendevreview | Michal Nasiadka proposed zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 16:21 |
| opendevreview | Merged opendev/zuul-providers master: Add image builds and provider config for Debian Bookworm https://review.opendev.org/c/opendev/zuul-providers/+/949127 | 16:37 |
| opendevreview | Merged opendev/zuul-providers master: Add bookworm and bullseye arm64 image builds https://review.opendev.org/c/opendev/zuul-providers/+/949482 | 16:37 |
| opendevreview | Merged opendev/zuul-providers master: Add centos-stream-9 image definitions https://review.opendev.org/c/opendev/zuul-providers/+/949567 | 16:37 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 16:37 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 16:38 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 16:49 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 17:37 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds, labels and provider config https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 17:38 |
| mnasiadka | corvus, clarkb : niz centos 9 stream builds look good now - I had to add some dnf-plugins-core handling to ensure-dib role (just like dib functests do) - if you can have a look I'd be grateful: https://review.opendev.org/q/topic:%22niz-centos%22 | 17:40 |
| corvus | mnasiadka: lgtm! thanks! | 17:43 |
| clarkb | mnasiadka: I think there is a small but important detail that needs updating on https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 17:45 |
| clarkb | (I posted ac omment) | 17:45 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add centos-9-stream builds, labels and provider config https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 17:45 |
| mnasiadka | clarkb: right, not that small - updated :) | 17:46 |
| clarkb | +2 thanks | 17:47 |
| mnasiadka | the ensure-dib patch has 2 +2s but no +w ;-) | 17:47 |
| clarkb | I think corvus and I raced. I'll fix it | 17:48 |
| mnasiadka | Ok, I'm only planning to add Rocky - openEuler or Gentoo are outside my interest - if that's ok | 17:48 |
| clarkb | yes I think thats fine. You've helped a ton already. Really appreciate it | 17:49 |
| mnasiadka | Do we need Rocky 8? I would assume not really | 17:49 |
| clarkb | is it still supported or is 8 eol? | 17:49 |
| clarkb | if it is still supported then ideally we'd continue to provide that image to jobs that use it | 17:50 |
| clarkb | support ends march 2029 | 17:50 |
| clarkb | *may 2029 | 17:50 |
| fungi | openeuler and gentoo image builds were in bad shape for a long time before this, so i wouldn't be too concerned if lack of interest leads to them being dropped | 17:50 |
| mnasiadka | I can add that, shouldn't be a lot of additional work | 17:51 |
| opendevreview | Michal Nasiadka proposed opendev/zuul-providers master: Add rockylinux-8/9 image definitions https://review.opendev.org/c/opendev/zuul-providers/+/949662 | 17:53 |
| mnasiadka | clarkb: there is no rockylinux-8-arm64 in openstack/project-config - so I omitted that | 17:54 |
| clarkb | that should be fine. | 17:54 |
| clarkb | the main concern is supporting jobs that already exist. IF there isn't any arm64 yet then we don't have that problem | 17:54 |
| opendevreview | Merged zuul/zuul-jobs master: ensure-dib: Add download of dnf-plugins-core https://review.opendev.org/c/zuul/zuul-jobs/+/949632 | 18:00 |
| mnasiadka | Ok then, enough for today | 18:09 |
| mnasiadka | clarkb: if there's an option https://review.opendev.org/c/opendev/zuul-providers/+/949662 could be merged before my morning - I could work on the Rocky stuff tomorrow in my daytime | 18:10 |
| clarkb | +2 if no one else reviews it I'll try to remember to approve later today | 18:12 |
| corvus | done | 18:43 |
| opendevreview | Merged opendev/zuul-providers master: Add centos-9-stream builds, labels and provider config https://review.opendev.org/c/opendev/zuul-providers/+/949571 | 19:59 |
| opendevreview | Merged opendev/zuul-providers master: Add rockylinux-8/9 image definitions https://review.opendev.org/c/opendev/zuul-providers/+/949662 | 19:59 |
| clarkb | infra-root my intention is to approve the gitea 1.23.8 change as my day starts tomorrow | 23:16 |
| tonyb | Sounds good. If you care about a second pair of eyes I'll be around | 23:42 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!