Thursday, 2024-07-11

tonybclarkb, fungi: I can do that (force merging the cs8 ozj changes) today if you'd like?00:33
Clark[m]Up to you. There will almost certainly be fallout and I'm happy to take the heat :)00:34
Clark[m]922649 is the change number00:35
tonybI'm also happy to take any heat00:41
tonybwhat can go wrong from a technical POV?00:41
Clark[m]Projects like glance will have zuul configuration errors and may stop running the expected set of zuul jobs00:43
Clark[m]The fix is straightforward they need to use the centos 9 fips jobs instead of the centos 8 fips jobs00:43
tonybOkay.  That's what I thought00:44
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] Deb Package  https://review.opendev.org/c/opendev/system-config/+/92369001:59
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] Deb Package and PyPi Package  https://review.opendev.org/c/opendev/system-config/+/92369101:59
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] PyPi Package Only  https://review.opendev.org/c/opendev/system-config/+/92369201:59
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] Deb Package  https://review.opendev.org/c/opendev/system-config/+/92369003:09
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] Deb Package and PyPi Package  https://review.opendev.org/c/opendev/system-config/+/92369103:09
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] PyPi Package Only  https://review.opendev.org/c/opendev/system-config/+/92369203:09
opendevreviewTony Breeds proposed opendev/system-config master: [DNM] Hack to fix job  https://review.opendev.org/c/opendev/system-config/+/92391403:09
tonybHmm it looks like my manual intervention didn't fix meetpad long term: https://zuul.opendev.org/t/openstack/builds?job_name=infra-prod-service-meetpad+&pipeline=periodic&skip=0&limit=1003:32
tonybmeetpad service seems to be functional03:33
tonybianw: Please reply with your thoughts on https://review.opendev.org/c/opendev/system-config/+/923684/comment/ef1b7314_319aa07b/ when you get some time05:49
opendevreviewJens Harbott proposed openstack/diskimage-builder master: Update functests  https://review.opendev.org/c/openstack/diskimage-builder/+/92389606:52
fricklerclarkb: fungi: I take it the foundation folks are aware of www.openstack.org expiring in 12 days?12:21
opendevreviewDmitriy Rabotyagov proposed openstack/diskimage-builder master: Fix regexp for Fedora element SUBRELEASE parsing  https://review.opendev.org/c/openstack/diskimage-builder/+/92210312:31
opendevreviewDmitriy Rabotyagov proposed openstack/diskimage-builder master: Allow to override Fedora subrelease regexp  https://review.opendev.org/c/openstack/diskimage-builder/+/92210412:31
opendevreviewDmitriy Rabotyagov proposed openstack/diskimage-builder master: Add support for building Fedora 40  https://review.opendev.org/c/openstack/diskimage-builder/+/92210912:31
fungifrickler: yeah, it's apparently one of a cluster of nginx servers that didn't get the memo to restart after its cert was replaced13:01
fungii'll ping the webdev contract folks about it later today13:02
fungiwe've mentioned it to them a few times already13:02
opendevreviewMerged opendev/system-config master: Fix task name to reflect correct distro  https://review.opendev.org/c/opendev/system-config/+/92368314:06
fungirunning some quick errands, but should return shortly14:31
clarkbbit of a slow start today, but mostly around at this point. mnaser I don't see any email yet about x/ospurge (that is the correct project right?) if you confirm you haven't sent anything I'll write an email today indicating a handover on July 18 if we don't hear otherwise. That seem reasonable?15:58
clarkbinfra-root https://review.opendev.org/c/opendev/system-config/+/920580 is the gitea 1.22.1 change and https://104.130.219.4:3081/opendev/system-config is the held node if you have time to review that15:58
clarkbalso I think we can start considering if we want to remove centos 8 stream nodesets entirely now that the ozj cleanups have merged: https://review.opendev.org/c/opendev/base-jobs/+/92265315:59
opendevreviewMichael Johnson proposed openstack/diskimage-builder master: Add podman to bindep  https://review.opendev.org/c/openstack/diskimage-builder/+/92381517:27
clarkbyoctozepto: fyi I left a comment and a -1 on the nebulous cleanup change. I do not think we should add submit rules to repos that are effectively retiring.17:39
clarkbfungi: ^ you +2'd the change, maybe you know why that is being done?17:39
fungiclarkb: i presumed it was in order to be able to update the readme on them if needed without re-adding the tenant, but i agree it seems unlikely and superfluous17:42
clarkbit's also a dangerous cargo cultable configuration that I think we should avoid if possible17:43
opendevreviewRadosÅ‚aw Piliszek proposed openstack/project-config master: Drop NebulOuS Zuul tenant  https://review.opendev.org/c/openstack/project-config/+/92172518:00
yoctozeptofungi, clarkb: hi! I agree this is no longer needed and cargo-culting something like that could be troublesome, I concur - I "fixed" it now ^18:00
yoctozeptothe readme is in place18:00
yoctozeptolinks updated18:00
yoctozeptonews sent18:01
yoctozeptoetc.18:01
yoctozeptowe should be fine ;-)18:01
clarkbcool18:01
yoctozeptodo I need something to get that "read-only" status on the repos? or will you handle this yourselves? (you have my blessing)18:01
clarkbits straightforward ish to add a tenant back in and noop jobs to merge stuff if necessary later. I just want to avoid leaving something like that in lpace forever as a honeypot18:01
clarkbyoctozepto: if you follow project retirement steps one of them is to retire the projects in the gerrit project config which will do that. This can happen in a followup change18:02
clarkbopenstack/project-config/projects.yaml updates in particular handle that18:02
yoctozeptoclarkb: you mean do this? https://docs.opendev.org/opendev/infra-manual/latest/drivers.html#step-3-remove-project-from-infrastructure-systems18:04
clarkbyoctozepto: yup though I notice that it says to use the openstack retired.config which isn't the case anymore since that gives special powers to openstack people18:05
clarkbyoctozepto: you want to use opendev/retired.config as the new config file18:05
yoctozeptoclarkb: precisely why my mind skipped it :D18:05
yoctozeptook18:05
* clarkb fixes the docs18:05
yoctozeptothanks18:05
yoctozeptowill do my part tomorrow18:05
yoctozeptohave a nice rest of the day18:05
opendevreviewClark Boylan proposed opendev/infra-manual master: Correct retirement acl config file path in docs  https://review.opendev.org/c/opendev/infra-manual/+/92398418:08
clarkbyoctozepto: thanks and you as well ^ thats the doc fixup18:08
fungii'll push a corresponding clarification to https://docs.openstack.org/project-team-guide/repository.html#steps-6-remove-project-from-infrastructure-systems for that18:10
opendevreviewJay Faulkner proposed openstack/diskimage-builder master: WIP: [gentoo] Support latest profile 23.0; use by default  https://review.opendev.org/c/openstack/diskimage-builder/+/92398518:19
opendevreviewMerged opendev/infra-manual master: Correct retirement acl config file path in docs  https://review.opendev.org/c/opendev/infra-manual/+/92398418:22
opendevreviewTony Breeds proposed opendev/system-config master: Also include tzdata when installing ARA  https://review.opendev.org/c/opendev/system-config/+/92368420:33
opendevreviewTony Breeds proposed opendev/system-config master: [create-venv] Add the ability to create a venv with specific python  https://review.opendev.org/c/opendev/system-config/+/92368520:33
opendevreviewTony Breeds proposed opendev/system-config master: Allow ansible to install/run with a specific version of python  https://review.opendev.org/c/opendev/system-config/+/92368620:33
opendevreviewTony Breeds proposed opendev/system-config master: Update ansible-devel job to address job failures  https://review.opendev.org/c/opendev/system-config/+/92368720:33
opendevreviewTony Breeds proposed opendev/system-config master: Add noble testing for ansible-devel  https://review.opendev.org/c/opendev/system-config/+/92368820:33
opendevreviewTony Breeds proposed opendev/system-config master: Install ARA master in the ansible-devel job  https://review.opendev.org/c/opendev/system-config/+/92401220:33
tonybI'm looking again at the proposal generation/git review process.  My aim is to largely decouple the code/config/jobs for generating the updates from the code to run git review to publish it21:16
tonybI'm just not sure of the "best" way to communicate this between jobs.  I'm currently thinking that a the generation job will zuul_return a dict where the key is a repo+SHA combo and the content is a git diff to apply.  The job to publish would then clone+checkout the repo+SHA, write the diff to a file and apply it.  Then run git review.21:19
clarkbtonyb: could also possibly have a parent job that runs the git review process in post-run and the child job pass to parents some variables like the repo to run git review in?21:20
tonybI think that'd mean the proposal job itself is the only part that'd need to live in project-config and everything else could live in a project repo.  Eg requirements21:20
clarkbI think that would also be the case with a parent/child job setup similar to hwo th docker image stuff works21:21
tonybclarkb: Oh that could also work and be kinda neat21:21
clarkbI think ^ is probably the main reason to consider this method: it keeps the process similar to other things that also do similar21:21
clarkbotherwise I think your proposal would also work fine21:22
clarkbI haven't heard from mnaser but I also haven't seen anything indicating someone else is coordinating ospurge handover stuff so I'm just going to go ahead with the plan I proposed above with a handover in a week if no one says anything21:23
tonybclarkb: Sounds good to me21:24
tonybboth the proposal and ospurge21:25
clarkbemail sent21:40
fungitonyb: one thing to think about is multiple jobs that each generate some piece of output which the depending job then concatenates in some way before generating a single commit to push for review21:52
fungie.g. generate-constraints runs on multiple platforms which then feed into a single update in gerrit21:53
opendevreviewJay Faulkner proposed openstack/diskimage-builder master: [gentoo] Make package removals less likely to fail  https://review.opendev.org/c/openstack/diskimage-builder/+/92401422:08
opendevreviewClark Boylan proposed openstack/diskimage-builder master: Run functest jobs on bookworm  https://review.opendev.org/c/openstack/diskimage-builder/+/92401922:53
opendevreviewJay Faulkner proposed openstack/diskimage-builder master: WIP: [gentoo] Support latest profile 23.0; use by default  https://review.opendev.org/c/openstack/diskimage-builder/+/92398523:38

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!