[00:00] <StevenK> I've spent a few minutes trying to work out where file uploads are handled in lazr.restful
[00:03] <wgrant>         file_content=Bytes(constraint=productrelease_file_size_constraint),
[00:03] <wgrant> So it'll be in the Bytes handler
[00:03] <wgrant> Support was only added relatively late, AFAICR
[00:10] <StevenK> wgrant: ... which I can't find in lazr.restful
[00:11] <wgrant> class BytesFieldMarshaller(SimpleFieldMarshaller):
[00:13] <wgrant> (is it just me, or does lazr.restful have marshall/unmarshall around the wrong way?)
[00:17] <StevenK> wgrant: I can't see anything in either BytesFieldMarshaller or SimpleFieldMarshaller that would call _decode
[00:17] <wgrant> StevenK: No, so you'll have to trace through to see how it happens
[00:17] <wgrant> And why it doesn't happen for normal file uploads
[00:24] <StevenK> wgrant: Hmmm, pdb.set_trace() inside IProductRelease.addReleaseFile() is probably far too late
[00:25] <wgrant> StevenK: Probably, but it might vaguely hint where it could get the data
[00:26] <wgrant> I'd rather break in the Bytes (un)marshaller
[00:29] <StevenK> wallyworld___: https://lh6.googleusercontent.com/-awustg4A1HQ/UQ1pZ49aYjI/AAAAAAAB0_k/ZXhib0JPDXQ/s728/YoomaShreddedBedText.jpg
[00:30] <wallyworld___> what you saying 'bout me?
[00:52]  * StevenK stabs this script
[00:54] <StevenK> Unknown consumer (System-wide: Ubuntu (undermined)).
[00:55] <mwhudson> undermined? :)
[00:56] <StevenK> mwhudson: Matches my current naming scheme
[01:16] <StevenK> wgrant: How can I debug Unknown consumer?
[01:16] <wgrant> StevenK: Probably means your gnome-keyring has a token that your Local DB no longer has
[01:17] <wgrant> Assuming it's not production
[01:17] <StevenK> It's dev
[01:17] <wgrant> Right
[01:17] <wgrant> Poke around with seahorse
[01:17] <wgrant> And delete the key that is no longer valid
[01:18] <StevenK> Right, deleted two of them
[01:18] <StevenK> The authorization page:
[01:18] <StevenK> ...
[01:18] <StevenK> Success!
[01:18] <wgrant> Great
[01:19] <StevenK> Hmm, but my pdb did not fire
[01:21] <StevenK> Oh
[01:21] <StevenK> I edited the wrong version
[01:23] <wgrant> Hah
[01:33] <StevenK> wgrant: http://pastebin.ubuntu.com/1606599/
[01:34] <StevenK> I can't see the tarball content being called in _decode, but the GPG sig is
[01:35] <wgrant> StevenK: You might have to look through the code to see how lazr.restful gets its values, and how Zope forms do it differently.
[01:35] <wgrant> Because Zope forms clearly don't use the decoded version
[01:39] <StevenK> Hmmm
[03:20]  * StevenK stabs lazr.restful for being obstreperous.
[04:30] <StevenK> wgrant: QA could be fun
[04:34] <wgrant> Quite
[04:38] <wgrant> StevenK: SPRB and TTB are populated on DF, BPB is about 1/3 done...
[04:38] <StevenK> Wow
[04:38] <StevenK> Nice
[04:39] <wgrant> It's been going for about 3 hoursish, though
[04:39] <wgrant> Or 2
[04:39] <wgrant> Something like that
[04:58] <StevenK> Bleh
[04:58] <StevenK> launchpadlib, forget your auth token!
[05:42] <StevenK> wgrant: Is DF done?
[05:43] <wgrant> No
[05:43] <wgrant>  f        | 2112345
[05:43] <wgrant>  t        | 1721777
[05:44] <wgrant> Almost half way
[06:12] <StevenK> wgrant: Put me out of my misery trying to work where lazr.restful gets it values from?
[06:26] <wgrant> StevenK: ResourceOperation.validate
[06:27] <wgrant> (I just stuck a pdb.set_trace() in addReleaseFile, called it with lp-shell, and looked through the traceback to see what would be parsing args)
[06:29] <StevenK> Bleh, I did that too, but didn't see ResourceOperation there
[06:29] <wgrant> It wasn't
[06:29] <wgrant> Well, I guess it was
[06:29] <wgrant> But that method wasn't
[06:32] <StevenK> Unknown consumer again! And Seahorse is unhelpful
[06:33] <wgrant> You probably nuked your DB
[06:33] <wgrant> Or it expired
[06:36] <StevenK> I did nuke my DB
[06:36] <StevenK> Your denorm columns not existing required a make schema
[06:38] <wgrant> Or a database/schema/upgrade.py :)
[06:39] <StevenK> I will usually just run make schema
[07:30] <wgrant> StevenK: https://code.launchpad.net/~wgrant/launchpad/flatten-bfj-5-app-cleanup/+merge/146349 is a big bit boring and mostly red diff
[07:30] <StevenK> And contains two conflicts
[07:31] <wgrant> Pretend they're not there, and I'll fix them as I merge the pipe :)
[07:32] <StevenK> 183>>>>>>> MERGE-SOURCE
[07:32] <StevenK> Haha, looks like they're earlier up the chain
[07:33] <StevenK> wgrant: PackageBuildMixin does not die yet?
[07:33] <wgrant> StevenK: It never dies
[07:33] <wgrant> It's a mixin
[07:33] <wgrant> Used for functionality that's shared between BPB and SPRB
[07:33] <wgrant> Like uploads and such
[07:34] <StevenK> Possibly rename it? Or you don't care enough?
[07:34] <wgrant> Hmm?
[07:34] <wgrant> It's a mixin for package builds
[07:34] <wgrant> So PackageBuildMixin seems like an apt name.
[07:34] <StevenK> Fairy nuff
[07:34] <wgrant> Did you have an alternate suggestion?
[07:37] <StevenK> No, was just trying get the name 'PackageBuild' killed everywhere
[07:39] <StevenK> wgrant: Oh, when does IPackageBuild die?
[07:41] <wgrant> StevenK: It doesn't
[07:41] <wgrant> It's an interface for package builds
[07:44] <StevenK> wgrant: So IBinaryPackageBuild will also implement IPackageBuild ?
[07:44] <wgrant> StevenK: It in fact inherits from it
[07:45] <wgrant> BinaryPackageBuild implements it, with the help of PackageBuildMixin
[07:45] <StevenK> Right
[07:45] <StevenK> wgrant: r=me
[07:45] <wgrant> Thanks
[08:22] <cjwatson> Has anyone had a chance to look at https://code.launchpad.net/~cjwatson/launchpad/bpph-phase/+merge/144154 ?  I have another piece of work to do that touches some of the same code, so it would be nice to get this out of the way.
[08:24] <wgrant> cjwatson: Oops, sorry
[08:24] <wgrant> Should have time to look at that tomorrow
[08:24] <cjwatson> Great, thanks
[08:29] <wgrant> StevenK: Can you QA 16468 tonight?
[08:29] <wgrant> Should be on qas shortly
[08:31] <StevenK> Is now, apparently
[08:33] <StevenK> wgrant: Done
[08:34] <wgrant> StevenK: Thanks
[08:34] <wgrant> We shall garbo it up on prod tonight
[08:41] <StevenK> wgrant: All gree
[08:41] <StevenK> *green
[08:46] <adeuring> good morning
[15:05]  * mpt rediscovers bug 533044
[15:05] <_mup_> Bug #533044: Resummarizing bug report doesn't change page title <lp-bugs> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/533044 >