| opendevreview | Clark Boylan proposed opendev/system-config master: Update to Gitea 1.20 https://review.opendev.org/c/opendev/system-config/+/886993 | 00:12 |
|---|---|---|
| clarkb | that patchset is saddening. We're now setting multiple options for features we explicitly disable because without those options gitea side effects the system in ways that don't work | 00:12 |
| opendevreview | Merged opendev/system-config master: Force rebuild of python base images https://review.opendev.org/c/opendev/system-config/+/888633 | 00:29 |
| ianw | you mean like the jwt tokens that we don't use but have to set? | 01:07 |
| tonyb | When (Add python bookworm images) 888102 merged the 'promote python-builder-3.* images' didn't run / wasn't triggered. I'm not sure where to look for why | 03:50 |
| Clark[m] | ianw: exactly. We don't do oauth2 (we explicitly disable it) but have to add all the jwt infrastructure for it | 03:59 |
| Clark[m] | tonyb: scroll back covers that a bit. I believe it is due to file matchers. The uwsgi jobs ran because we updated files they match but we didnt do so for the normal images | 03:59 |
| tonyb | Clark[m]: Ahh thanks. So should we update zuul.d/docker-images/python-3.*-{bullseye,bookworm}.yaml to reference itself in the files section so that we match in the future? | 04:20 |
| tonyb | Clark[m]: will that work? | 04:21 |
| Clark[m] | I'm not sure. It may be worth a try | 04:22 |
| tonyb | ... obviously you've fixed the immediate problem. | 04:22 |
| Clark[m] | I think it would work because the zuul files show up in the list of modified files like everything else. I don't think zuul will do any special filtering of those files | 04:22 |
| tonyb | Okay. I'll follow-up with that. | 04:25 |
| tonyb | Why does https://opendev.org/opendev/system-config/src/branch/master/zuul.d/docker-images/python-3.9-bookworm.yaml#L111 include the trailing '.*' as opposed to https://opendev.org/opendev/system-config/src/branch/master/zuul.d/docker-images/python-3.9-bookworm.yaml#L23 ? | 04:32 |
| ianw | i don't think there's any technical reason; both should match the same. the .* is probably clearer as to the intent i guess | 04:36 |
| tonyb | ianw: thanks. I was hoping that was it | 04:42 |
| frickler | ianw: Clark[m]: while you're around, https://review.opendev.org/c/opendev/system-config/+/888065 and https://review.opendev.org/c/opendev/system-config/+/886259 would be two nice little things | 05:19 |
| ianw | doh, thanks for finding that one on the logrotate file | 05:20 |
| frickler | ianw: one day those regular cron mails got my attention, like, after some weeks ;) | 05:22 |
| Clark[m] | tonyb: ianw: correct a trailing .* Is unnecessary | 05:32 |
| Clark[m] | frickler: I'll take a look when I get the laptop back out. Probably after dinner at this point | 05:33 |
| *** elodilles_pto is now known as elodilles | 06:16 | |
| opendevreview | Tony Breeds proposed opendev/system-config master: Update to latest tag for Limnoria https://review.opendev.org/c/opendev/system-config/+/888110 | 07:41 |
| opendevreview | Tony Breeds proposed opendev/system-config master: Update ircbot and matrix-eavesdrop to python-3.11 https://review.opendev.org/c/opendev/system-config/+/888111 | 07:41 |
| opendevreview | Tony Breeds proposed opendev/system-config master: Add $self to the list of files for python images. https://review.opendev.org/c/opendev/system-config/+/888663 | 07:41 |
| opendevreview | Tony Breeds proposed opendev/system-config master: Update docker-images/python* to remove '.*' https://review.opendev.org/c/opendev/system-config/+/888664 | 07:41 |
| clarkb | tonyb: ha we collided on meeting agenda updates | 08:24 |
| clarkb | I went ahead and sent out that agenda | 08:33 |
| tonyb | Thanks | 08:33 |
| opendevreview | Merged opendev/system-config master: Add download.cirros-cloud.net to certcheck https://review.opendev.org/c/opendev/system-config/+/886259 | 08:40 |
| clarkb | frickler: I felt confident approvinv ^ thta one but not the other this close to bedtime | 08:42 |
| clarkb | feel free to approve the other change when you can keep an eye on it or I'll try to do it when I've got more time | 08:42 |
| frickler | clarkb: ack, thx, approved and will watch | 08:43 |
| opendevreview | Merged opendev/system-config master: Update accessbot,ircboot and matrix-eavesdrop to bookworm container https://review.opendev.org/c/opendev/system-config/+/888106 | 09:49 |
| *** dhill is now known as Guest6230 | 11:16 | |
| opendevreview | Merged opendev/system-config master: logrotate: fix removal of old config files https://review.opendev.org/c/opendev/system-config/+/888065 | 11:49 |
| *** iurygregory_ is now known as iurygregory | 12:08 | |
| *** amoralej is now known as amoralej|lunch | 12:27 | |
| *** amoralej|lunch is now known as amoralej | 13:15 | |
| *** iurygregory_ is now known as iurygregory | 14:59 | |
| opendevreview | Merged openstack/diskimage-builder master: Add nm-dhcp-ib-interfaces element https://review.opendev.org/c/openstack/diskimage-builder/+/882507 | 15:15 |
| opendevreview | Merged openstack/diskimage-builder master: Adds passwd to rocky-container os packages https://review.opendev.org/c/openstack/diskimage-builder/+/855530 | 15:15 |
| *** amoralej is now known as amoralej|off | 16:34 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!