| lifeless | wgrant: don't you love release bugs ;0 | 07:58 |
|---|---|---|
| lifeless | wgrant: I've asked ttx to grab some timing data, it looks like milestone assignment on bugs may be rather slow - we'll try to create a decent bug report | 07:59 |
| lifeless | wgrant: he's saying ~2s to dereference the bug, assign a milestone, and call lp_save() | 08:00 |
| wgrant | lifeless: On OpenStack projects? Yes. | 08:00 |
| wgrant | You have thousands of structsubs. | 08:00 |
| lifeless | wgrant: still, should be deferrable work | 08:01 |
| wgrant | I am aware. | 08:01 |
| lifeless | wgrant: heh | 08:01 |
| lifeless | wgrant: ok, well we'll gather the data | 08:01 |
| wgrant | It's on the list for this year, as it's becoming quite unpleasant. | 08:02 |
| lifeless | since assigning milestones is relatively rare I figure it doesn't show in the % failure rates much | 08:02 |
| wgrant | Most other things are performing OK nowadays. | 08:02 |
| cjwatson | Re discussion from yesterday, Diff.generateDiffstat works just fine for git diffs; it just needs a small tweak to run in -p1 mode. | 13:26 |
| cjwatson | (We could change the diff format from turnip, but I'd rather leave that as close as possible to the default git style.) | 13:27 |
| wgrant | Nice. | 13:29 |
| wgrant | Yeah, let's notdiverge pointlessly. | 13:30 |
| cjwatson | wgrant: And it's just the format_diff stuff in lp.app.browser.stringformatter that should need to be tweaked otherwise, right? | 13:35 |
| cjwatson | In which case I'm nearly done with preview diffs, just need to have the alternate diffs stuff in place in turnip and to finish the job changes. | 13:38 |
| wgrant | cjwatson: I think so. The rest is just bsed on the classes. | 13:38 |
| wgrant | I'll get to your two remaining not-quite-mega-branches soonish. | 13:41 |
| cjwatson | https://code.qastaging.launchpad.net/~cjwatson/germinate/+git/germinate/+ref/exp/+merge/254302 renders now. | 13:41 |
| cjwatson | Thanks. | 13:42 |
| wgrant | Ah excellent. | 13:58 |
| wgrant | cjwatson: https://code.launchpad.net/~wgrant/launchpad/do-not-reject-ddebs/+merge/257758 | 14:06 |
| cjwatson | wgrant: r=me, thanks | 14:10 |
| cjwatson | wgrant: That's hopefully all the MP stuff proposed now. I'm moving on to see if I can sort out at least a little bit of repo listing display. | 14:42 |
| cjwatson | wgrant: Should I shove contextlib2 into turnip-dependencies and land Kit's branch? (Though, that's only worth it if you're going to get all the way up my review stack today, which may be a bit to ask ...) | 14:57 |
| wgrant | cjwatson: Sounds reasonable to me. | 15:01 |
| === Spads_ is now known as Spads | ||
| cjwatson | wgrant: This is basically awful: http://people.canonical.com/~cjwatson/tmp/git-target-listing.png | 16:48 |
| cjwatson | I think the best way to improve that would be to provide a list of branches in all associated repositories sorted by last-modified and privileging the default repository, or something | 16:49 |
| cjwatson | OTOH this was quick. Any suggestions that might be doable short-term? | 16:49 |
| cjwatson | Deploying Kit's turnip branch for cross-repo diffs now. | 17:32 |
| cjwatson | Had to "rm -r /srv/mojo/mojo-stg-ue-launchpad-git/trusty/qastaging/build/storage-ceph/" so that the new auto-repo stuff stopped being confused | 17:38 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!
