opendevreview | Dale Smith proposed openstack/project-config master: Add magnum-capi-helm-charts to Magnum project https://review.opendev.org/c/openstack/project-config/+/893117 | 00:47 |
---|---|---|
opendevreview | Dale Smith proposed openstack/project-config master: Add magnum-capi-helm-charts to Magnum project https://review.opendev.org/c/openstack/project-config/+/893117 | 00:52 |
opendevreview | Dale Smith proposed openstack/project-config master: Add magnum-capi-helm-charts to Magnum project https://review.opendev.org/c/openstack/project-config/+/893117 | 01:53 |
noonedeadpunk | clarkb: 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 merged | 07:52 |
noonedeadpunk | Though 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 intermittently | 07:53 |
noonedeadpunk | which happens due to OOMs from time to time | 07:54 |
noonedeadpunk | So 1 of 10 patches will fail intermittently, and it's not rarely when we can have 10 patches to different repos in gate jobs | 07:57 |
opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Unpause image uploads for rax-iad part 2 https://review.opendev.org/c/openstack/project-config/+/893145 | 10:36 |
opendevreview | Merged openstack/project-config master: Add magnum-capi-helm-charts to Magnum project https://review.opendev.org/c/openstack/project-config/+/893117 | 11:56 |
*** d34dh0r5- is now known as d34dh0r53 | 12:14 | |
*** ralonsoh is now known as ralonsoh_ooo | 16:20 | |
mmalchuk | folks, please review: https://review.opendev.org/q/hashtag:ironic-week-prio+(status:open+OR+status:merged)+topic:extend-dib-checksum | 16:45 |
clarkb | mmalchuk: you probably want to ask the ironic folks for reviews on those changes | 16:46 |
mmalchuk | ops...wrong channel | 16:46 |
mmalchuk | sorry | 16:46 |
fungi | came 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 submissions | 18:00 |
opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Unpause image uploads for rax-iad part 2 https://review.opendev.org/c/openstack/project-config/+/893145 | 19:08 |
JayF | Where 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 released | 20:22 |
JayF | already has a +2 from clark, needs another | 20:22 |
frickler | JayF: 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,members | 20:28 |
JayF | frickler: only admin folks can see that fyi | 20:28 |
JayF | well, I can see the group | 20:28 |
JayF | I cannot see the group has access from the ui | 20:28 |
JayF | ah so it is oslo too, cool, I'll post it over there | 20:28 |
frickler | oh, I thought everyone could see those members, sorry. the list is essentially stephenfin, lifeless, infra-core and oslo-core | 20:29 |
JayF | I can see members | 20:29 |
JayF | but from PBR I can't see pbr-core has access | 20:29 |
JayF | (even if it's borderline-obvious now that I know that group exists lol) | 20:30 |
frickler | JayF: 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 deployment | 20:31 |
JayF | well, the place that often breaks down is shared pieces, so admittedly this time I didn't look like I normally would :D | 20:31 |
frickler | https://opendev.org/openstack/project-config/src/branch/master/gerrit/acls/openstack/pbr.config | 20:32 |
frickler | the ACL not being visible in the gerrit UI was for some security concern I think, but I don't remember the details | 20:33 |
JayF | There's a lot of "we do X for a reason that is lost to time" around :( /me just ran into another one an hour ago | 20:34 |
JayF | thanks for the help though :D | 20:34 |
JayF | btw, was happy to see you're running for TC, good luck | 20:34 |
fungi | yeah, 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 whatever | 20:43 |
fungi | i'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 reviews | 20:44 |
opendevreview | Merged openstack/project-config master: Unpause image uploads for rax-iad part 2 https://review.opendev.org/c/openstack/project-config/+/893145 | 20:54 |
opendevreview | Clark Boylan proposed openstack/project-config master: Switch OpenStack's Zuul tenant to Ansible 8 by default https://review.opendev.org/c/openstack/project-config/+/893290 | 21:21 |
JayF | If 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.yaml | 21:25 |
clarkb | that looks right we can confirm in the job log too | 21:27 |
clarkb | I'll find an example of the string to look for | 21:27 |
JayF | ack; thanks | 21:27 |
JayF | going to push one for ironic to be safe, but bifrost is more where I'm worried :) | 21:27 |
clarkb | https://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.13 | 21:29 |
JayF | Yep, confirmed on my output | 21:29 |
JayF | thanks :) | 21:29 |
clarkb | you're welcome | 21:31 |
opendevreview | Merged openstack/pbr master: Replace imp with importlib.machinery https://review.opendev.org/c/openstack/pbr/+/884789 | 22: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/!