| opendevreview | Merged openstack/project-config master: Retire puppet-ec2api: Remove Project from Infrastructure System https://review.opendev.org/c/openstack/project-config/+/912713 | 07:01 |
|---|---|---|
| opendevreview | Merged openstack/project-config master: Retire puppet-murano: Remove Project from Infrastructure System https://review.opendev.org/c/openstack/project-config/+/913296 | 07:16 |
| opendevreview | Merged openstack/project-config master: Retire puppet-sahara: Remove Project from Infrastructure System https://review.opendev.org/c/openstack/project-config/+/910455 | 07:42 |
| frickler | infra-root: I'm trying to find out why zuul is still (in its logs) complaining about a missing project-template in https://review.opendev.org/c/openstack/openstack-ansible/+/913418 even though that patch removes the reference to it | 12:43 |
| frickler | I even added a noop template to that patch, the error in the zuul log stays the same regardless. note that there are existing config-errors that this patch tries to resolve https://zuul.opendev.org/t/openstack/config-errors?project=openstack%2Fopenstack-ansible&skip=0 | 12:44 |
| fungi | reminder: we have a ptg session running 14:00-16:00 utc today (about 75 minutes from now) in https://meetpad.opendev.org/apr2024-ptg-opendev | 12:45 |
| frickler | unrelated, I saw we still (or again) have a periodic pipeline job for the sandbox repo queued | 12:45 |
| fungi | frickler: did you grep all branches of openstack-ansible for references to that project template? | 12:48 |
| frickler | fungi: yes, it is both being defined and used there (at least in master, didn't check others) | 12:50 |
| fungi | oh, you're not trying to remove it, zuul's complaining about a project template that isn't being removed? | 12:53 |
| fungi | complaining it's missing i mean | 12:53 |
| fungi | do you have a paste of the error from the log? | 12:55 |
| frickler | https://paste.opendev.org/show/bH6yuByjDbT9E4xQLE38/ | 12:56 |
| fungi | okay, so it's saying the openstack-ansible-deploy-distro_ceph-jobs template is missing (it wasn't clear from your earlier description which project template you were talking about) | 12:59 |
| frickler | ah, sorry, that's the one that is mentioned as zuul config error, so I assumed that to be clear | 12:59 |
| frickler | note in PS6 I even re-add that template, but that doesn't seem to help, either. so somehow zuul not using the speculative state as I'd expect it to do | 13:00 |
| fungi | frickler: looks like it was removed from master almost 4 years ago in https://review.opendev.org/746881 so probably the last stable branch prior to that got deleted recently | 13:02 |
| fungi | i'm not sure if zuul might be blocked by the existing broken configuration, preventing it from testing the proposed fix which adds the missing template back | 13:03 |
| frickler | oh right, that's a different template name in master, I didn't look at that closely enough. so anyway if the change cannot be made self-testing (which I'd consider to be a bug in zuul?) we can only try to force-merge it and iterate on that if needed? | 13:10 |
| noonedeadpunk | I'd be fine with force-merging it | 13:20 |
| noonedeadpunk | given that it's needed to drop the branch after all | 13:20 |
| frickler | noonedeadpunk: so it actually wouldn't matter in the end whether zuul was working or not, right? should I revert to PS4, then? | 13:27 |
| noonedeadpunk | well, we want it to work on unmaintained brach afterwards:) | 13:31 |
| noonedeadpunk | so removing obvious errors is good | 13:31 |
| *** dhill is now known as Guest836 | 13:32 | |
| frickler | ah, right, that's for eom only, not eol. anyway I'll go ahead with that, then | 13:35 |
| frickler | #status log force-merged 913418,6 to work around zuul config error issue | 13:45 |
| frickler | and the errors are gone | 13:45 |
| opendevstatus | frickler: finished logging | 13:45 |
| clarkb | frickler: the revert of the sandbox change would prevent new enqueues, but not clera out the existing buildset. My change to avoid the underlying problem landed though so maybe it will clear out on its own after the next restart | 13:54 |
| corvus | that error that frickler pasted for openstack ansible template was in the release-approval pipeline; that probably wasn't causing it to to fail to enqueue in check (though perhaps it had the same error there) | 13:57 |
| frickler | ah, I was somehow thinking you had dequeued the existing things | 13:57 |
| frickler | corvus: yes, that was the same error afaict | 13:57 |
| frickler | btw. there are also still errors about the sandbox issue in the current logs, shouldn't those be gone now? or do they persist as long as the item is in the queue? | 13:59 |
| corvus | the latter | 14:02 |
| clarkb | https://github.com/ether/etherpad-lite/issues/6320 has been filed | 18:27 |
| fungi | thanks! | 18:47 |
| clarkb | fungi: I know the openstack security sig stuff is about to start but when you have time can you double check the infra root keys cleanup changes. You mentioend having looked at that stuff in the past so probably have the context for reviewing it | 20:48 |
| fungi | yeah, i already had a look at the first change, went ahead and left my +2 since it can go in before the updated keys | 20:50 |
| fungi | seeing the last time i updated an ssh key there was approximately 9 years ago... that's embarrassing | 20:52 |
| clarkb | and ya I think we can land that first change whenever and check if the cloud updater tooling is happy. Then land the second change after existing roots have a chance to change their key or nack the opportunity to do so | 20:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!