| === guiverc2 is now known as guiverc | ||
| === mfo_ is now known as mfo | ||
| andersson123 | bluca: I just read your messages from yesterday, sorry, looking into it now. | 14:14 |
|---|---|---|
| bluca | thanks! | 14:14 |
| bluca | if you need any info (like the hook's config) let me know | 14:15 |
| andersson123 | please yea that'd be good | 14:16 |
| andersson123 | as I understand, you had some temporary issues whilst some of launchpad was borked, then actual errors after right? | 14:18 |
| bluca | yes | 14:18 |
| bluca | the http get is: | 14:18 |
| bluca | https://autopkgtest.ubuntu.com/request.cgi?release=jammy&arch=amd64&package=systemd-upstream&build-git=https%3A%2F%2Fsalsa.debian.org%2Fsystemd-team%2Fsystemd.git%23upstream-ci&ppa=upstream-systemd-ci/systemd-ci&env=CFLAGS=-O0%3BDEB_BUILD_PROFILES=noudeb%3BTEST_UPSTREAM=1%3BCONFFLAGS_UPSTREAM=--werror%20-Dslow-tests%3Dtrue | 14:18 |
| bluca | we get back error 400, with text: "You submitted an invalid request: Missing field in JSON data: 'ppa'" | 14:19 |
| bluca | but ppa= is there as you can see, I had a look at https://git.launchpad.net/autopkgtest-cloud/tree/charms/focal/autopkgtest-web/webcontrol/request/submit.py#n144 and it might not like the ppa, but dunno why, it is here: https://launchpad.net/~upstream-systemd-ci/+archive/ubuntu/systemd-ci | 14:19 |
| bluca | that's pretty much it, the github webhook is configured with json and ssl, and to run on PR, there were no changes there | 14:22 |
| andersson123 | Hmmm this is weird because the changes I introduced yesterday don't have anything to do with checking keyword arguments | 14:22 |
| andersson123 | let me do a bit of digging. | 14:23 |
| bluca | thanks, let me know if you need anything else | 14:23 |
| andersson123 | bluca: Could you please resubmit the request? | 14:48 |
| andersson123 | It's not an issue on your end, I know what the problem is but I need a traceback to investigate further, recently introduced an "all exception handler" in the app for tidyness but it masks the actual traceback now :melt: | 14:52 |
| bluca | a new PR was opened 1 minute ago and the jobs are showing up | 14:55 |
| bluca | do you see that one? or do you need another trigger? | 14:56 |
| bluca | it triggered at 14:52:50 UTC | 14:56 |
| andersson123 | And the request failed or no? | 14:59 |
| andersson123 | I don't see any new failures in the logs... | 15:00 |
| bluca | it succeeded | 15:01 |
| bluca | while until ~30 minutes ago they still all failed, so whatever you changed, it fixed it :) | 15:01 |
| bluca | ah, new PR two minutes ago, and this one failed | 15:02 |
| bluca | again with 400 and ppa | 15:02 |
| bluca | can you check if you see this? | 15:02 |
| bluca | triggered at 15:00:43 | 15:03 |
| andersson123 | Yeah I see, that's helpful, thanks | 15:05 |
| bluca | it's weird that one went through before this one? | 15:06 |
| bluca | I haven't touched anything in the webhooks | 15:06 |
| andersson123 | The app checks running.json. There's an error parsing the file - right now when there are no systemd-upstream requests (or any github requests) running, the check for the file is fine. When there are systemd tests already running, the function checks for a field in the file which is incorrectly parsing the file. So in summary your webhooks will work as long as there's no systemd-upstream requests currently running :) | 15:08 |
| andersson123 | I will disable the check to unblock you. When I have a hotfix, I'll implement it and let you know :) | 15:08 |
| bluca | ah I see, that sounds fun eheh | 15:09 |
| bluca | thank you very much for your help | 15:10 |
| andersson123 | np | 15:10 |
| bluca | let me know if I need to change the webhook config in any way | 15:10 |
| bluca | I don't need to keep compat if there's some new option needed, I can just update them | 15:10 |
| andersson123 | you don't need to change anything on your end dw, but thanks. I've removed the check now. | 15:15 |
| andersson123 | bluca: I've added an actual hotfix now, let me know if you have any issues from now on (as of 2 mins ago) | 16:12 |
| bluca | got another "Missing field in JSON data" at 16:12:17, was that before or after the change? | 16:20 |
| andersson123 | Thanks for letting me know, I'll disable it and try another fix | 16:23 |
| ahasenack | @pilot in | 16:33 |
| === ChanServ changed the topic of #ubuntu-devel to: Archive: Mantic post-Beta Freeze | Devel of Ubuntu (not support) | Build failures: http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of Bionic-Lunar | Patch Pilots: ahasenack | ||
| ahasenack | bdmurray: hi, do you know if the DEP8 needs-internet restriction is observed in the ubuntu dep8 infrastructure? | 19:56 |
| === liushuyu1 is now known as liushuyu | ||
| ahasenack | @pilot off | 21:01 |
| ubottu | (pilot <in|out>) -- Set yourself as in or out of patch pilot. | 21:01 |
| ahasenack | @pilot out | 21:01 |
| === ChanServ changed the topic of #ubuntu-devel to: Archive: Mantic post-Beta Freeze | Devel of Ubuntu (not support) | Build failures: http://qa.ubuntuwire.com/ftbfs/ | #ubuntu for support and discussion of Bionic-Lunar | Patch Pilots: N/A | ||
| === justache is now known as justHaunting | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!