Friday, 2025-08-01

opendevreviewSimon Westphahl proposed zuul/zuul-jobs master: Add default for build diskimage image name  https://review.opendev.org/c/zuul/zuul-jobs/+/95621907:00
opendevreviewBartosz Bezak proposed opendev/system-config master: Mirror docker registry v3  https://review.opendev.org/c/opendev/system-config/+/95635411:24
corvushttps://zuul.opendev.org/t/openstack/build/81c430709fb1414598ef9766362db918/log/job-output.txt#647413:43
corvusfungi: thanks for the recheck.  that was a really weird timeout ^13:43
corvusit looks like that was in the middle of some tasks that just "echo text > file"13:43
corvusmaybe one of the remote nodes became unresponsive in some way that didn't cause an unreachable error13:44
opendevreviewJames E. Blair proposed opendev/zuul-providers master: Remove build_diskimage_image_name variable  https://review.opendev.org/c/opendev/zuul-providers/+/95637313:56
stephenfinclarkb: fungi: Would you be able to take a look through the remaining pbr patches early next week? I'd really like to cut a release asap just to catch any issues I may have introduced14:20
* stephenfin is concerned about upcoming deadlines14:20
fungistephenfin: i can try to take a look today even14:21
stephenfinsweet, thanks. though I do think you've reviewed nearly all of them at this point 14:22
stephenfinin fact you have, so clarkb is up14:22
fungii switched to running gertty on a different machine and redid my subscription list, just realized i forgot to include pbr14:22
fungiso this was a good reminder14:22
stephenfinI was hoping to get the proper pyproject.toml support done in case they plan to remove support for distutils keywords also but it seems unlikely in the two weeks remaining14:24
stephenfin(as in support for a '[tool.pbr]' section, whose mere presence would enable pbr magic14:24
stephenfin)14:24
Clark[m]I thought the deadline was October 31? I'm out Monday but can try to take a look today14:25
fungiah, so it could contain toggles for different features in the future too, i guess14:25
stephenfinyeah, exactly. We already have that in setup.cfg so it's more of a continuation of that functionality14:27
opendevreviewBartosz Bezak proposed opendev/system-config master: Add Registry v3 (3.0.0) to OpenDev mirror  https://review.opendev.org/c/opendev/system-config/+/95635414:29
opendevreviewBartosz Bezak proposed opendev/system-config master: Add Registry v3 to OpenDev mirror  https://review.opendev.org/c/opendev/system-config/+/95635414:30
opendevreviewMerged opendev/system-config master: Update zuul-launcher logging config  https://review.opendev.org/c/opendev/system-config/+/95631114:36
opendevreviewMerged opendev/zone-opendev.org master: Add eavesdrop02 records  https://review.opendev.org/c/opendev/zone-opendev.org/+/95612114:36
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Update upload-image-swift  https://review.opendev.org/c/zuul/zuul-jobs/+/95637714:37
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Update upload-image-swift  https://review.opendev.org/c/zuul/zuul-jobs/+/95637714:39
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Synchronize upload-image-s3 to upload-image-swift  https://review.opendev.org/c/zuul/zuul-jobs/+/95637814:45
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Update upload-image-swift  https://review.opendev.org/c/zuul/zuul-jobs/+/95637716:04
opendevreviewJames E. Blair proposed zuul/zuul-jobs master: Synchronize upload-image-s3 to upload-image-swift  https://review.opendev.org/c/zuul/zuul-jobs/+/95637816:04
clarkbstephenfin: I've just approved https://review.opendev.org/c/openstack/pbr/+/954049 but left some inline commenst where my diffing found differences between the source and final setup post refactor. One of which I think may be worth a followup change.16:34
clarkbI don't think you should push a new patchset. A followup would e fine16:34
opendevreviewCorey Leong proposed openstack/project-config master: New Project Cascade  https://review.opendev.org/c/openstack/project-config/+/95638616:36
clarkbstephenfin: fungi: I approved everything up to https://review.opendev.org/c/openstack/pbr/+/949055/ I +2'd that change but didn't approve it as I'm slightly worried the documentation update there may paint pbr in a bad light when it comes to setuptools and setup.cfg and compatibility between the two16:48
clarkbIf others disagree with me its fine to approve (this is why I +2'd anyway) but I left a suggestion on how we might rephrase that and make it less problematic for pbr (in my opinion anyway)16:49
stephenfinNo, I agree. I wasn't happy with my wording of that either16:49
fungisgtm16:50
stephenfinIt's the end of my day/week, so I am happy for you to reword as you see fit and re-approve, if that works for you16:50
stephenfinOtherwise I can do a follow-up next week, whatever works16:50
clarkbsure I can push an update to that change. fungi did you want to check what I had written to see if it makes sense?16:51
fungii did already, read it in the e-mail notification gerrit sent me16:51
fungihence my rather terse "sgtm" in here ;)16:51
fungihappy to +2/approve it once pushed16:51
clarkback I should have that up in a few minutes16:52
fungithe subsequent changes will require rebasing, unless you want to throw it into a new change at the end16:52
fungii'm good either way16:52
clarkbI'll just do a change at the end and I can also remove that redundant line in that one test that snuck in during the refactor16:53
clarkb(as two separate changes)16:53
clarkbcurrent problem: copying text out of gerrit comment literal blocks is not easy in the brwoser16:53
clarkbhittingthe quote button helps16:54
fungiwfm16:57
fungithe plan i mean16:58
fungii agree the gerrit webui gets in the way of a lot of things, not just hughlighting/copy-paste but also things like conflicting with my browser's keybindings16:58
clarkbok both changes are pushed. I'll go ahead and approve the one I held off on and defer review/approval on the two new changes to others16:59
fungiall problems i don't have with gertty, hence my avoidance of the webui16:59
clarkbfrickler: I've updated my zuul python3.13 change to run on trixie but then it immediately fails as we don't have a mirror for it yet. Is there some easy way to disable that that jobs are using elsewhere already?17:26
clarkbI'll revert for now back to the ubuntu-noble compile python 3.13 version17:29
fungiclarkb: looks like it checks a zj_repo.condition flag? https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/configure-mirrors/tasks/mirror/Debian.yaml#L1817:35
fungioh, though that's only used directly for configure_mirrors_extra_repos17:35
fungiyeah i'm not seeing any obvious toggle17:35
opendevreviewCorey Leong proposed openstack/project-config master: New Project Cascade  https://review.opendev.org/c/openstack/project-config/+/95640118:39
fungilooks like the first pbr change in that series failed in the gate18:58
profcoreyWhat did I do wrong this time19:05
profcoreyUsed hard tabs and replaced the inheritance from openstack19:06
fungiprofcorey: for starters, it looks like you pushed a new change rather than revising the existing change https://docs.opendev.org/opendev/infra-manual/latest/developers.html#updating-a-change19:07
fungiprofcorey: but also it looks like some of the lines may still have spaces in their indentation or mixed spaces+tabs, and you added a tab inside the [access "refs/heads/*"] label too19:09
fungihttps://zuul.opendev.org/t/openstack/build/e604bbf1cc93432d80775328a7d30c0e/console19:10
fungiyou can use `tox -e gerrit` to run the same checks locally too19:10
profcoreyI thought I saw in the docs to push a new change rather then the existing19:10
profcoreyGosh ok I'll start from scratch again19:11
profcoreyrun a check locally first19:12
profcoreythanks!19:12
fungiprofcorey: if you just make sure "Change-Id: I95e59c64a2f7a9766176acae0f3ee21519d0a125" is at the end of your commit message (a line by itself in the last paragraph, what git docs refer to as a "trailer"), it'll be a revision to the existing change automatically19:13
profcoreyok will do19:13
fungibasically gerrit uses that id in combination with the project name and target branch name to identify a commit as a revision to a specific change19:14
profcoreygotcha19:16
fungithere is also an acl normalizer command you can run to justy fix it up rather than trying to edit it into shape, lemme see if i can remember the magic incantation19:18
profcoreyok19:18
fungisometimes that's easier if your editor doesn't do a good job of differentiating spaces from tabs visually, or you have trouble telling whether things are sorted into alpha order19:19
profcoreyI'm using vi to edit my files19:20
fungiprofcorey: `python3 tools/normalize_acl.py x gerrit/acls/x/cascade.config apply`19:23
profcoreysuper, thank you!19:24
profcoreyI'll give that a try when I can19:24
fungithere's a ton of code comments at the top of the normalize_acl.py script too, for the curious19:25
profcoreyok19:25
profcoreythank you!19:26
fungiprofcorey: tip from a fellow vim user, i set "listchars=tab:»·" and "trail:·" so that i see "»·" in place of a hard tab and "·" for spaces at the end of lines, but there are a lot of alternative solutions out there too19:28
profcorey[REDACTED]19:28
fungi(you might want to change that)19:28
profcoreysorry about that19:28
profcoreywill do19:28
fungii can redact it from the web logs for the channel19:28
profcoreyThank you!19:29
profcoreyIt's changejd19:29

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