opendevreview | yatin proposed openstack/project-config master: Use 2024.2 constraints in master translation jobs https://review.opendev.org/c/openstack/project-config/+/934402 | 08:10 |
---|---|---|
ykarel | frickler, fungi previous patch worked fine for master, need ^ for get it working for stable too | 08:12 |
frickler | ykarel: I looked at the job history and I see that job only running on master, do you have a pointer to a failure on stable branches? | 08:27 |
ykarel | frickler, https://zuul.openstack.org/builds?job_name=propose-translation-update&skip=0 | 08:37 |
ykarel | https://zuul.openstack.org/builds?job_name=upstream-translation-update&skip=0 | 08:37 |
frickler | ah, right, I had only looked at the latter, not at the derived jobs. +2, then | 08:41 |
ykarel | thx | 08:52 |
fungi | ykarel: frickler: isn't that going to break as soon as stable/2025.1 branches? | 14:13 |
fungi | or is the hope that we'll be done moving off zanata by then? | 14:13 |
ykarel | yes stable/2025.1 have enough time so considering ^ | 14:14 |
frickler | should be doable with an update of the regexes | 14:18 |
fungi | the increasing complexity there obligates me to reiterate clarkb's earlier question: why not just drop constraints from those jobs entirely? it seems like the things it needs to run shouldn't need the projects installed | 14:18 |
ykarel | hmm seems that might work too, can try | 14:51 |
ykarel | whatever you prefer | 14:51 |
*** haleyb is now known as haleyb|out | 16:17 | |
JayF | this is extremely strange; I have a patch at "Ready to Submit" here: https://review.opendev.org/c/openstack/ironic-ui/+/906784/1 | 18:46 |
JayF | I've literally never seen that before in gerrit and I'm wondering what's wrong | 18:46 |
JayF | cc cardoe | 18:46 |
fungi | JayF: not sure how it ended up in that state, but it needs a rebase since its parent changed out from under it | 18:50 |
fungi | it's parented on patchset 1 of 906783 but patchset 2 is what ended up merging | 18:50 |
JayF | ack; rebased | 18:52 |
JayF | thank you for the pointer, rare to see something that new | 18:52 |
clarkb | it was ready to submit because it had all the necessary votes | 19:40 |
clarkb | I suspect this happened because it gated but couldn't merge due to the old parent | 19:41 |
clarkb | usually when verified +2 happens it is also merged but not in this case so it was in the ready to submit state | 19:41 |
fungi | i thought normally when zuul tried to submit the change after voting verified +2 and received an api-level rejection from gerrit, it updated its verified vote to -2 | 19:44 |
fungi | but there might have been an incident occurring at that point in time | 19:44 |
JayF | ah, that makes sense, so it's usually a transitory state but it was left here because for [????] reason that V-2 vote wasn't posted | 20:02 |
fungi | yes, that's my best guess, without spelunking in 10-month-old irc logs to look at what might have been going on back then | 20:03 |
fungi | or 7 months, whatever it was, been long enough i've wiped my local meat-based storage | 20:04 |
JayF | yeah, doesn't really matter | 20:09 |
JayF | if I see this again I'll just rebase :D | 20:09 |
fungi | well, specifically, if you see an outdated parent then rebase | 20:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!