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