| *** darmach5 is now known as darmach | 06:52 | |
| opendevreview | Lukas Kranz proposed openstack/diskimage-builder master: Fix setting apt mirror for noble https://review.opendev.org/c/openstack/diskimage-builder/+/920466 | 07:07 |
|---|---|---|
| opendevreview | Lukas Kranz proposed openstack/diskimage-builder master: Fix setting apt mirror for noble. currently setting the apt mirror for noble is broken as the format and the location of the sources file has changed. https://review.opendev.org/c/openstack/diskimage-builder/+/920466 | 07:08 |
| opendevreview | Lukas Kranz proposed openstack/diskimage-builder master: Fix setting apt mirror for noble https://review.opendev.org/c/openstack/diskimage-builder/+/920466 | 08:23 |
| frickler | hmm, if I click on the "This reverts commit 0ec71d87cd67c25790f56143a73bd65d3cbd6d36." link on https://review.opendev.org/c/openstack/kolla-ansible/+/920359 and then on the origin commit, I get an error popup: "Error 400 (Bad Request): Invalid change format" although the link itself looks pretty normal | 10:43 |
| frickler | seems to be specific to that review, I don't get the same for e.g. https://review.opendev.org/c/nebulous/activemq/+/919869 | 10:47 |
| frickler | I can also reproduce this in a different browser (chromium instead of ff) without being logged in | 10:49 |
| opendevreview | Sven Kieske proposed openstack/project-config master: restrict enforced backport voting to master branch https://review.opendev.org/c/openstack/project-config/+/920472 | 11:50 |
| SvenKieske | seems we only need backportvoting enforced in master branch, which makes sense, see discussion currently in #openstack-kolla | 11:53 |
| opendevreview | Sven Kieske proposed openstack/project-config master: restrict enforced backport voting to master branch https://review.opendev.org/c/openstack/project-config/+/920472 | 12:10 |
| fungi | frickler: the link loads for me, maybe gerrit's search index was lagging behind? | 12:23 |
| frickler | fungi: I can still reproduce, even directly opening https://review.opendev.org/c/openstack/kolla-ansible/+/912734 | 12:31 |
| fungi | oh, i think what changed is that the commit link now returns a page listing two changes. so only one of them is broken i guess | 12:32 |
| fungi | no mention in error_log of the change number, change id or commit id for that either | 12:37 |
| frickler | yes, I tried to call it "origin commit", maybe that wasn't explicit enough. the error is only with "not-the-revert" one. but it also didn't occur to me earlier that the error happens when directly opening that review regardless from which source | 12:37 |
| Clark[m] | I'm not quite awake yet but it you open browser dev tools maybe you can see which component of the web UI is making that request and failing. It seems the change mostly loads so it's probably some less critical sub component of the page | 14:25 |
| frickler | hmm, I see a successful query for https://review.opendev.org/changes/?q=change:I33d62e910ff6151eb03c814ee7b8984c6f4e5308 and directly afterwards the failing query for https://review.opendev.org/changes/?q=change:II33d62e910ff6151eb03c814ee7b8984c6f4e5308 (note the double "I") | 14:44 |
| Clark[m] | Maybe one of our regex rules is bad and over matching some text? | 14:46 |
| Clark[m] | For the URL links on comments | 14:46 |
| frickler | initiator is gr-app.js:24203 for this and some other queries, but looking at the source this is some monster line https://review.opendev.org/elements/gr-app.js | 14:50 |
| frickler | oh, wait, there's actually two "created a revert" comments (the two last ones), the second has that "II" in it | 14:51 |
| clarkb | accidentally turning on caps lock when you are using vimdiif to resolve gitea template changes between releases leads to a wild time | 16:23 |
| fungi | certainly sounds exciting | 16:26 |
| clarkb | annoyingly it was never going to be a clean diff so it is quite a pain to see if I fixed thinsg up properly | 16:29 |
| clarkb | I try again then compare results | 16:29 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update Gitea to version 1.22 https://review.opendev.org/c/opendev/system-config/+/920580 | 17:05 |
| clarkb | looks like gitea has an undocumented breaking change requiring golang 1.22 or newer for gitea 1.22 | 17:18 |
| opendevreview | Clark Boylan proposed opendev/system-config master: Update Gitea to version 1.22 https://review.opendev.org/c/opendev/system-config/+/920580 | 17:19 |
| fungi | ouch | 17:26 |
| clarkb | I just made some small updates to the meeting agenda. I'll get that sent out closer to the end of my day. Get your stuff in or let me know what I should add/edit/remove | 17:40 |
| clarkb | I'm also likely to pop out after lunch to spend time with the kids after it warms up (its a holiday and they are out of school) | 17:44 |
| fungi | have fun! | 17:45 |
| clarkb | #status log Set Storpool CI's Gerrit account (15670) back to active at their request after they indicated changes have been made to the CI system to address prior concerns | 22:00 |
| opendevstatus | clarkb: finished logging | 22:00 |
| clarkb | meeting agenda has been sent | 22:04 |
| opendevreview | Clark Boylan proposed opendev/system-config master: DNM intentional gitea failure to hold a node https://review.opendev.org/c/opendev/system-config/+/848181 | 22:08 |
| clarkb | I have requested an autohold for ^. Will use that to check some of the breaking change concerns like the doctor command execution | 22:09 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!