[02:54] <StevenK> allow-picked-versions = false now complains about setuptools 0.7.4 no matter what versions.cfg states.
[02:54]  * StevenK stabs buildout for being obtuse and obstreperous.
[03:09] <StevenK> wgrant: I've been victorious over buildout
[03:09] <wgrant> Great
[04:58] <StevenK> wgrant: https://code.launchpad.net/~stevenk/launchpad/export-pu-auditor/+merge/171460
[05:20] <wgrant> StevenK: Do the bulk auditor load calls also bulk load the relevant people?
[05:21] <wgrant> Are the auditor calls logged in the statement log?
[05:21] <wgrant> What are the timeouts on the auditor calls?
[05:22] <StevenK> wgrant: I don't think the enterpriseid stuff is that smart
[05:25] <wgrant> It'll need to be
[05:29] <StevenK> wgrant: Have you read enterpriseid_to_object() ?
[05:32] <wgrant> StevenK: Not in maybe a year. Why?
[05:33] <StevenK> wgrant: Because I can't think of a way given a list of enterpriseids to objectify them in one or two queries
[05:34] <wgrant> StevenK: That doesn't obviate the need to do it in one or two queries.
[05:34] <StevenK> Certainly not
[05:35] <wgrant> You'd want to split them up by class, then bulk load
[05:35] <wgrant> You'd also ideally introspect the storm cache to grab those that exist.
[05:35] <wgrant> But that might be a bit difficult
[06:24] <wgrant> StevenK: https://code.launchpad.net/~wgrant/launchpad/sha256-archiveuploader-pre/+merge/171464 https://code.launchpad.net/~wgrant/launchpad/sha256-archiveuploader/+merge/171465
[06:31] <StevenK> wgrant: r=me for the pre branch
[06:39] <StevenK> wgrant: And slamming you with Needs Fixing on the second
[06:43] <wgrant> That was more approve with comments material, but OK :)
[07:04] <wgrant> StevenK: Updated
[07:09] <StevenK> wgrant: r=me
[07:09] <StevenK> wgrant: Are you going to bend gina to your will in terms of hashes, too?
[07:10] <wgrant> No comment
[07:11] <wgrant> But I might look at switching it to use some of the archiveuploader infrastructure
[07:11] <wgrant> Rather than reimplementing the world
[07:11] <StevenK> Sounds like a good idea
[08:23] <ttx> wgrant: ok, I'll try to drop the subscriber and see how it goes :)
[09:23] <ttx> wgrant: looks like it's passing tests now: https://code.launchpad.net/~ttx/launchpad/lp1193389/+merge/171491
[11:17] <wgrant> ttx: Looking good, thanks. I'll review it properly tomorrow.
[11:40] <ttx> wgrant: awesome, thx