jgraham, gsnedders : If either of you can take a look at web-platform-tests/wpt #27175 soonish (next couple days), that'd be great :).

I'm reasonably confident that the Pool thing is what is causing the 'wpt manifest' hangs in Chromium CI with Py3, but I'm not super confident that I'm calling the right APIs now. (Do I have to close() it too? join() it?)

wpt.live seems to be down.

PiersW: Thanks for the report, looking!

smcgruer_[EST]: Will look this afternoon (i.e. soon)

ty!

smcgruer_[EST]: r+

You could have done some wrapping to turn it into a context manager for Py2, but this is defintiely a smaller patch

(and I appreciate the detailed commit message, thanks)

PiersW: wpt.live should be back up I think, lmk if you see any more problems

Looks good, thanks!

jgraham: Thanks!

jgraham: The context manager thing was a little tricky as well, because we use `imap`; so the `with ...` statement would have to wrap all the way until we loop the results. Would still be doable with some refactoring of the code, but yeah given that I was lazy and just terminated it manually :)

Ah, Python where RAII except when you can't quite be bothered ;)

(not a complaint about the patch :)

Heh, I feel like we could shorten it to... 'Python, when you can't quite be bothered'

:)

I know I've asked this before, but... are there live per-PR deploys I can run in my browser? I can't figure out which of the checks in web-platform-tests/wpt #27178 might have it...

smcgruer_[EST]: ^

I know wpt.live is supposed to support this, but I also know there were problems and I don't recall if they ever got resolved

Domenic: So Chromium exports are not deployed by default. You can add the label 'safe for preview' to cause them to be deployed.

Once deployed, they show up as a github deployment, i.e. this box should change to 'one deployment' usercontent.irccloud-cdn.com/file/fXvFAf0v/image.png

And indeed, it looks like it fell over when I tried to deploy that PR.

Yeah, sigh. wpt.live / wptpr.live honestly is unlikely to keep living the way it is. It's unloved and the implementation has turned out to be very fragile. (Especially the wptpr.live part)

Oh god, wptpr.live is out of space again isnt it...

Aw. We love it. Very convenient! (wpt.live, anyway.)

I think wpt.live could easily be made much more manageable

wptpr.live is another beast (mirroring PRs is hard)

Ok, I've cleaned up about 14 or so bad PRs that shouldnt have been mirrored, so we're down to 'only' 70 mirrored PRs

Domenic: Well, I dont know why its still marked *pending* rather than available, but hey there's a deployment for that PR now usercontent.irccloud-cdn.com/file/trOAw121/image.png

(the 'view deployment' button)

Hooray, thank you!