Saturday, 2023-11-18

-@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/+/90134500:43
@weebull:matrix.orgDoes 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.orgit's also been long enough it may need rebasing to address merge conflicts with the current state of any files it's modifying13:24
@weebull:matrix.orgWould 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.orggitea 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 design13:25
@fungicide:matrix.orgwe 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 flag13:26
@fungicide:matrix.orgbut generally if you see a change that has clear need for improvement, it's okay to update it13:27
@weebull:matrix.org> lick cookies13:27
Took me a while to get what you meant :D
@fungicide:matrix.orgoh, sorry, that was very idiomatic of me13:28
@fungicide:matrix.orgwhat 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 done13:29
@fungicide:matrix.organd to ask questions if you're not sure13:29
@weebull:matrix.orgMight 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.orgPaul, fungi: I am still interested in getting it finally merged after years of waiting. 14:04
@gtema:matrix.orgBut feel free also to rebate it whenever necessary 14:05
@gtema:matrix.orgThis change is running in production in OTC, so it is clearly a working ine14:05
@gtema:matrix.orgOne14: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.orgNope, it was ready, but I wasn't checking since Gita 1.19 new features and API changes14:09
@gtema:matrix.orgI 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.orgis 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.orgnever mind! a forced refresh got it to show up20:22
@fungicide:matrix.orgah, though that was for the artifacts view not the task summary20:23
@fungicide:matrix.orgjob-output.json is nearly a gigabyte in size, that's probably why20:46

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!