| @jim:acmegating.com | yep exactly | 00:00 |
|---|---|---|
| @jim:acmegating.com | our gate/promote pipeline does that | 00:01 |
| @clarkb:matrix.org | 104.130.13.12 is the held etherpad and I've got a clarkb-test on that instance seems to be working | 00:01 |
| @clarkb:matrix.org | then ya I think this is fine. The only other case where I can think of automated rebuilds coming into play is when we delete an image due to some fundamental problem with the image itself (bad kernel package etc). And in those cases we usually don't want to immediately rebuild things anyway | 00:02 |
| @jim:acmegating.com | yep, manual deletion followed by a manual rebuild request seems like a good fit for that sort of thing | 00:05 |
| @clarkb:matrix.org | ++ | 00:05 |
| @mnasiadka:matrix.org | Anyone willing for a second +2 on https://review.opendev.org/c/opendev/system-config/+/956354? | 10:46 |
| @janosin:matrix.org | Hello, please check this change https://review.opendev.org/c/opendev/glean/+/974346 Thank you | 10:55 |
| -@gerrit:opendev.org- Dmitriy Rabotyagov proposed: [openstack/project-config] 974994: Add access to Freezer Unmaintained for freezer-stable-maint https://review.opendev.org/c/openstack/project-config/+/974994 | 11:50 | |
| -@gerrit:opendev.org- László Jánosi proposed: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 14:16 | |
| -@gerrit:opendev.org- Jeremy Stanley https://matrix.to/#/@fungicide:matrix.org proposed wip: [opendev/system-config] 974942: Inject docs honeypot URL into body comment https://review.opendev.org/c/opendev/system-config/+/974942 | 15:16 | |
| -@gerrit:opendev.org- László Jánosi proposed: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 16:10 | |
| @fungicide:matrix.org | gonna go run a bunch of errands during lunch today, but i'll be back in a while and can look at the etherpad upgrade then | 16:10 |
| @clarkb:matrix.org | fungi: thanks. I'm reviewing that glean change and the registry v3 mirroring change then going to try and debug our own registry pruning | 16:12 |
| @clarkb:matrix.org | janosin: I left a few qusetions on patchset 3 (sorry I was writing them as patchset 4 showed up) | 16:17 |
| -@gerrit:opendev.org- Zuul merged on behalf of Bartosz Bezak: [opendev/system-config] 956354: Add Registry v3 to OpenDev mirror https://review.opendev.org/c/opendev/system-config/+/956354 | 16:29 | |
| -@gerrit:opendev.org- László Jánosi proposed: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 16:34 | |
| @clarkb:matrix.org | I am able to reproduce the registry pruning problem with yesterday's etherpad 2.6.1 image build. `curl -X GET 'https://insecure-ci-registry.opendev.org:5000/auth/token?scope=repository%3Aopendevorg%2Fetherpad%3Apull'` to get a token then `curl -I -H 'Authorization: Bearer TOKENHERE' https://insecure-ci-registry.opendev.org:5000/v2/opendevorg/etherpad/manifests/6d91a73d19c345d28cb0ced93459bee4_v2.6.1` will get you a 404 as expected | 16:36 |
| @clarkb:matrix.org | I'm going to recheck that change now so that I get a new image that I can inspect the details of to cross check against the pruning process | 16:36 |
| @clarkb:matrix.org | I also see in the pruning log that the first time that 6d91a... hash shows up it says it will keep that manifest then it lists the layers, then there is a second log entry indicating it is an unknown manifest | 16:37 |
| @clarkb:matrix.org | I suspect that points at a likely bug in the pruning process (why does the amnifest go from a keep to unknown?) but I want to better understand the manifest content too | 16:38 |
| @janosin:matrix.org | > <@clarkb:matrix.org> janosin: I left a few qusetions on patchset 3 (sorry I was writing them as patchset 4 showed up) | 16:53 |
| Thank you. I addressed those in patchset 5. | ||
| -@gerrit:opendev.org- László Jánosi proposed: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 18:44 | |
| @clarkb:matrix.org | https://review.opendev.org/c/zuul/zuul-registry/+/975088 is my proposed fix for registry pruning. TL;DR is that we were deleting a small amount of extra backend data that we need to preserve | 19:22 |
| @janosin:matrix.org | > <@gerrit:opendev.org> László Jánosi proposed: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 20:06 |
| @fungi | ||
| @janosin:matrix.org | > <@gerrit:opendev.org> László Jánosi proposed: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 20:08 |
| @fungi Thank you for the approval. | ||
| @Clark @mnasiadka unfortunately, I had to push one more patch due to a "too long line" CI error. If you are still OK with the patch, please vote again. Thank you. | ||
| -@gerrit:opendev.org- Zuul merged on behalf of László Jánosi: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 21:23 | |
| @janosin:matrix.org | > <@gerrit:opendev.org> Zuul merged on behalf of László Jánosi: [opendev/glean] 974346: Fix vlan and bond config generation https://review.opendev.org/c/opendev/glean/+/974346 | 21:31 |
| Thank you all for the help. | ||
| -@gerrit:opendev.org- Maksim Malchuk proposed: [openstack/diskimage-builder] 975100: Add devuser to the specified group(s) https://review.opendev.org/c/openstack/diskimage-builder/+/975100 | 21:41 | |
| @clarkb:matrix.org | janosin: you're welcome. I think there may be one or two other glean updates that we should look at before making a release: https://review.opendev.org/c/opendev/glean/+/924068 https://review.opendev.org/c/opendev/glean/+/971350 and https://review.opendev.org/c/opendev/glean/+/963010. I need to review the last two (I have reviewed the first) | 21:45 |
| @clarkb:matrix.org | I have -1'd https://review.opendev.org/c/opendev/glean/+/963010 beacuse I don't think it is backward compatible on all platforms | 21:52 |
| @fungicide:matrix.org | i discovered the ubuntu-ports mirror hasn't updated since november 21, presenting with a stale reprepro lockfile. clearing that, its checksums.db seems to be corrupted. at the moment i have that rebuilding from a root screen session on the mirror-update server, but expect it will take longer today than i'll be awake, so i plan to check in on it tomorrow and continue trying to get the mirror back in sync | 22:04 |
| @clarkb:matrix.org | I also -1'd https://review.opendev.org/c/opendev/glean/+/971350 because I don't think this fixes the underlying issue. I also have a concern about backward compatibility | 22:07 |
| @clarkb:matrix.org | fungi: you reviewed both of those changes. If you get a chance can you check my reviews and weigh in on whether or not those concerns are valid? | 22:08 |
| @fungicide:matrix.org | done, thanks | 22:33 |
| @clarkb:matrix.org | great thanks | 22:36 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!