| stub | Having the test suite rewind the database is best done by not using the system PostgreSQL service, but having it run the postgres executable on a DATADIR itself. It is easy then to kill it and reset the DATADIR. | 05:15 |
|---|---|---|
| stub | We didn't do it *at the time* because it was slower than keeping the process running a creating a new database from a template. | 05:16 |
| stub | But those timings change every release. | 05:16 |
| wgrant | It's still slower | 05:17 |
| wgrant | Well, it was ~2y ago when I last tested | 05:17 |
| wgrant | Startup isn't quick. | 05:17 |
| wgrant | IMO the most likely way to optimise further is to have another thread/process keeping a pool of fresh DBs available that can just be renamed across. | 05:18 |
| wgrant | There are also possibilities around causing the librarian to run in the same snapshot as the main testrunner (and testrunner-appserver, where applicable), so the commit after librarian adds can be optimised out in the test suite. | 05:21 |
| SpecialK|Canon | cjwatson: nice! | 09:22 |
| cjwatson | ilasc: go ahead and top-approve https://code.launchpad.net/~ilasc/launchpad/+git/launchpad/+merge/377875 when you're ready | 11:47 |
| ilasc | cjwatson thanks! done | 11:48 |
| cjwatson | Could I have reviews of https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/378126 and https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/378131 ? Simple-ish dependency upgrades | 11:50 |
| ilasc | on it now | 11:51 |
| cjwatson | Thanks | 12:00 |
| cjwatson | More py3 reviews wanted: https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/378430 (dep upgrade), https://code.launchpad.net/~cjwatson/lazr.restful/wsgiref-py2/+merge/378432 (setup.py tweak), https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/378459 (dependency adjustments), https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/378460 (large, ... | 14:21 |
| cjwatson | ... bulk conversion to six.moves.urllib.*) | 14:21 |
| cjwatson | Other than the above, FWIW, all it takes to get the virtualenv building on py3 is: (1) PYTHON := python3 (2) remove bzr, mailman, meliae, Paste*, subvertpy (3) upgrade wsgi-intercept and FormEncode | 14:25 |
| cjwatson | subvertpy needs non-trivial work to unblock removing bzr properly, as previously discussed, but everything else is in progress | 14:27 |
| cjwatson | Then there's a lot to do before the rest of the build will work, because building API documentation tends to import basically everything. And running very much of the test suite before lazr.restful at least sort of works is going to be hard. | 14:29 |
| SpecialK|Canon | That's awesome | 14:32 |
| SpecialK|Canon | Nice work | 14:32 |
| cjwatson | https://github.com/cdent/wsgi-intercept/pull/61 so many yaks, and some of them are camped out in other people's houses | 15:35 |
| tomwardill | I'd say 'networks, not even once', but it appears that my computer agrees | 15:59 |
| tomwardill | It's installing things into the build LXD, while the firewall is active! | 17:25 |
| tomwardill | shame that's as far as it gets, but still... it did a thing! | 17:26 |
| cjwatson | https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/378477 another py3 improvement, gradually disentangling stuff | 22:09 |
| pappacena | Done! | 22:19 |
| cjwatson | cheers | 22:45 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!