-@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 901345: Fix some API endpoints with special chars https://review.opendev.org/c/zuul/zuul/+/901345 | 00:43 | |
@weebull:matrix.org | Does anybody know the status of the Gitea driver that was proposed (here)[https://review.opendev.org/c/zuul/zuul/+/859939]? It looks pretty complete with just a couple of minor lint issues, but quiet for a couple of months. | 08:50 |
---|---|---|
@weebull:matrix.org | * Does anybody know the status of the Gitea driver that was proposed (here)[https://review.opendev.org/c/zuul/zuul/+/859939\]? It looks pretty complete with just a couple of minor lint issues, but quiet for a couple of months. | 08:51 |
@weebull:matrix.org | * Does anybody know the status of the Gitea driver that was proposed [here](https://review.opendev.org/c/zuul/zuul/+/859939)? It looks pretty complete with just a couple of minor lint issues, but quiet for a couple of months. | 08:52 |
@fungicide:matrix.org | > <@weebull:matrix.org> Does anybody know the status of the Gitea driver that was proposed [here](https://review.opendev.org/c/zuul/zuul/+/859939)? It looks pretty complete with just a couple of minor lint issues, but quiet for a couple of months. | 13:23 |
possible the original author lost interest and it's waiting for someone else with interest to renew the effort. if it were me i'd probably start by getting fresh test results and trying to address those, as well as earlier reviewer comments still outstanding (if any) | ||
@fungicide:matrix.org | it's also been long enough it may need rebasing to address merge conflicts with the current state of any files it's modifying | 13:24 |
@weebull:matrix.org | Would it be seen as bad etiquette if I tried to move it forward? It rebases cleanly as it has minimal overlap with other code. | 13:24 |
@fungicide:matrix.org | gitea itself has also made new releases recently, so might be good to revisit their api to see if anything's changed that could impact the design | 13:25 |
@fungicide:matrix.org | we don't lick cookies here ;) maybe if gtema happens to be paying attention in this room he can say for sure if he's planning to resume work on that or has concerns with someone picking up the flag | 13:26 |
@fungicide:matrix.org | but generally if you see a change that has clear need for improvement, it's okay to update it | 13:27 |
@weebull:matrix.org | > lick cookies | 13:27 |
Took me a while to get what you meant :D | ||
@fungicide:matrix.org | oh, sorry, that was very idiomatic of me | 13:28 |
@fungicide:matrix.org | what i meant to say is that it's fine to pitch in and help as long as you have a clear picture of what needs to be done | 13:29 |
@fungicide:matrix.org | and to ask questions if you're not sure | 13:29 |
@weebull:matrix.org | Might need a little guidance on how you do gerrit, as it's not my domain, but happy to chip in otherwise. | 13:30 |
@weebull:matrix.org | (Hence the interest in a gitea driver) | 13:31 |
@gtema:matrix.org | Paul, fungi: I am still interested in getting it finally merged after years of waiting. | 14:04 |
@gtema:matrix.org | But feel free also to rebate it whenever necessary | 14:05 |
@gtema:matrix.org | This change is running in production in OTC, so it is clearly a working ine | 14:05 |
@gtema:matrix.org | One | 14:05 |
@weebull:matrix.org | @gtema That's great. Is there anything outstanding on the change as far as you're concerned? | 14:07 |
@weebull:matrix.org | * gtema: That's great. Is there anything outstanding on the change as far as you're concerned? | 14:08 |
@gtema:matrix.org | Nope, it was ready, but I wasn't checking since Gita 1.19 new features and API changes | 14:09 |
@gtema:matrix.org | I would really propose to get this finally merged and then add new features instead of trying to get the change improved now, it took too much time already | 14:10 |
@fungicide:matrix.org | is anyone else able to get https://zuul.opendev.org/t/zuul/build/872a0b1fb13b49ef8a200415906d6e87 to load? it's just *Fetching info...* forever for me (okay technically not forever, but i waited about half an hour before i gave up) | 20:22 |
@fungicide:matrix.org | never mind! a forced refresh got it to show up | 20:22 |
@fungicide:matrix.org | ah, though that was for the artifacts view not the task summary | 20:23 |
@fungicide:matrix.org | job-output.json is nearly a gigabyte in size, that's probably why | 20:46 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!