Wednesday, 2023-08-30

opendevreviewDale Smith proposed openstack/project-config master: Add magnum-capi-helm-charts to Magnum project  https://review.opendev.org/c/openstack/project-config/+/89311700:47
opendevreviewDale Smith proposed openstack/project-config master: Add magnum-capi-helm-charts to Magnum project  https://review.opendev.org/c/openstack/project-config/+/89311700:52
opendevreviewDale Smith proposed openstack/project-config master: Add magnum-capi-helm-charts to Magnum project  https://review.opendev.org/c/openstack/project-config/+/89311701:53
noonedeadpunkclarkb: well... I know they are not. We had quite some discussions about starting sharing queues but ended up to keep the behaviour and just be careful when setting +w until dependencies are merged07:52
noonedeadpunkThough the thing is that we usually have quite some patches to land, so we want to avoid gate failure when one of the patches somewhere has failed intermittently07:53
noonedeadpunkwhich happens due to OOMs from time to time07:54
noonedeadpunkSo 1 of 10 patches will fail intermittently, and it's not rarely when we can have 10 patches to different repos in gate jobs07:57
opendevreviewDr. Jens Harbott proposed openstack/project-config master: Unpause image uploads for rax-iad part 2  https://review.opendev.org/c/openstack/project-config/+/89314510:36
opendevreviewMerged openstack/project-config master: Add magnum-capi-helm-charts to Magnum project  https://review.opendev.org/c/openstack/project-config/+/89311711:56
*** d34dh0r5- is now known as d34dh0r5312:14
*** ralonsoh is now known as ralonsoh_ooo16:20
mmalchukfolks, please review: https://review.opendev.org/q/hashtag:ironic-week-prio+(status:open+OR+status:merged)+topic:extend-dib-checksum16:45
clarkbmmalchuk: you probably want to ask the ironic folks for reviews on those changes16:46
mmalchukops...wrong channel16:46
mmalchuksorry16:46
fungicame up in a debian-project ml thread, but https://docs.github.com/en/communities/moderating-comments-and-conversations/limiting-interactions-in-your-organization might provide a means to indefinitely block github pull request submissions18:00
opendevreviewDr. Jens Harbott proposed openstack/project-config master: Unpause image uploads for rax-iad part 2  https://review.opendev.org/c/openstack/project-config/+/89314519:08
JayFWhere would I find cores for PBR? Looking for a review for https://review.opendev.org/c/openstack/pbr/+/884789 to ensure pbr can run on 3.12 once released20:22
JayFalready has a +2 from clark, needs another20:22
fricklerJayF: stephenfin maybe? fungi seems to be a bit distracted by external events. not sure how much oslo ppl still are involved https://review.opendev.org/admin/groups/a0113ae80e46db8e16ae2d16d49b42c24f27f6f4,members20:28
JayFfrickler: only admin folks can see that fyi20:28
JayFwell, I can see the group20:28
JayFI cannot see the group has access from the ui20:28
JayFah so it is oslo too, cool, I'll post it over there20:28
frickleroh, I thought everyone could see those members, sorry. the list is essentially stephenfin, lifeless, infra-core and oslo-core20:29
JayFI can see members20:29
JayFbut from PBR I can't see pbr-core has access20:29
JayF(even if it's borderline-obvious now that I know that group exists lol)20:30
fricklerJayF: ah, you could see that in project-config if needed, but that pattern to have a -core group for a project should be pretty generic in our deployment20:31
JayFwell, the place that often breaks down is shared pieces, so admittedly this time I didn't look like I normally would :D 20:31
fricklerhttps://opendev.org/openstack/project-config/src/branch/master/gerrit/acls/openstack/pbr.config20:32
fricklerthe ACL not being visible in the gerrit UI was for some security concern I think, but I don't remember the details20:33
JayFThere's a lot of "we do X for a reason that is lost to time" around :( /me just ran into another one an hour ago20:34
JayFthanks for the help though :D20:34
JayFbtw, was happy to see you're running for TC, good luck20:34
fungiyeah, gerrit upstream considers being able to see who has access to do things you don't as an information leak/security risk, classic "security by obscurity" if you ask me, but whatever20:43
fungii'll take a look at the change, but you're right pbr is an oslo deliverable and #openstack-oslo is the best place to seek reviews20:44
opendevreviewMerged openstack/project-config master: Unpause image uploads for rax-iad part 2  https://review.opendev.org/c/openstack/project-config/+/89314520:54
opendevreviewClark Boylan proposed openstack/project-config master: Switch OpenStack's Zuul tenant to Ansible 8 by default  https://review.opendev.org/c/openstack/project-config/+/89329021:21
JayFIf someone has a second, can you help me make sure I'm testing that properly? https://review.opendev.org/c/openstack/bifrost/+/893291/1/zuul.d/bifrost-jobs.yaml21:25
clarkbthat looks right we can confirm in the job log too21:27
clarkbI'll find an example of the string to look for21:27
JayFack; thanks21:27
JayFgoing to push one for ironic to be safe, but bifrost is more where I'm worried :)21:27
clarkbhttps://zuul.opendev.org/t/openstack/build/4f7fbf3cf6c643029d4d9d1c5fdca56a/log/job-output.txt#35 <- Ansible 8 uses ansible core 2.15. https://zuul.opendev.org/t/openstack/build/d0965f0c8b784cefadee86068e3a1d1e/log/job-output.txt#35 <- Ansible 6 uses ansible core 2.1321:29
JayFYep, confirmed on my output21:29
JayFthanks :)21:29
clarkbyou're welcome21:31
opendevreviewMerged openstack/pbr master: Replace imp with importlib.machinery  https://review.opendev.org/c/openstack/pbr/+/88478922:33
*** benj_0 is now known as benj_23:20

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