StevenK | I've spent a few minutes trying to work out where file uploads are handled in lazr.restful | 00:00 |
---|---|---|
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:03 |
StevenK | wgrant: ... which I can't find in lazr.restful | 00:10 |
wgrant | class BytesFieldMarshaller(SimpleFieldMarshaller): | 00:11 |
wgrant | (is it just me, or does lazr.restful have marshall/unmarshall around the wrong way?) | 00:13 |
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:17 |
StevenK | wgrant: Hmmm, pdb.set_trace() inside IProductRelease.addReleaseFile() is probably far too late | 00:24 |
wgrant | StevenK: Probably, but it might vaguely hint where it could get the data | 00:25 |
wgrant | I'd rather break in the Bytes (un)marshaller | 00:26 |
StevenK | wallyworld___: https://lh6.googleusercontent.com/-awustg4A1HQ/UQ1pZ49aYjI/AAAAAAAB0_k/ZXhib0JPDXQ/s728/YoomaShreddedBedText.jpg | 00:29 |
wallyworld___ | what you saying 'bout me? | 00:30 |
* StevenK stabs this script | 00:52 | |
StevenK | Unknown consumer (System-wide: Ubuntu (undermined)). | 00:54 |
mwhudson | undermined? :) | 00:55 |
StevenK | mwhudson: Matches my current naming scheme | 00:56 |
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:16 |
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:17 |
StevenK | Right, deleted two of them | 01:18 |
StevenK | The authorization page: | 01:18 |
StevenK | ... | 01:18 |
StevenK | Success! | 01:18 |
wgrant | Great | 01:18 |
StevenK | Hmm, but my pdb did not fire | 01:19 |
StevenK | Oh | 01:21 |
StevenK | I edited the wrong version | 01:21 |
wgrant | Hah | 01:23 |
StevenK | wgrant: http://pastebin.ubuntu.com/1606599/ | 01:33 |
StevenK | I can't see the tarball content being called in _decode, but the GPG sig is | 01:34 |
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:35 |
StevenK | Hmmm | 01:39 |
* StevenK stabs lazr.restful for being obstreperous. | 03:20 | |
StevenK | wgrant: QA could be fun | 04:30 |
wgrant | Quite | 04:34 |
wgrant | StevenK: SPRB and TTB are populated on DF, BPB is about 1/3 done... | 04:38 |
StevenK | Wow | 04:38 |
StevenK | Nice | 04:38 |
wgrant | It's been going for about 3 hoursish, though | 04:39 |
wgrant | Or 2 | 04:39 |
wgrant | Something like that | 04:39 |
StevenK | Bleh | 04:58 |
StevenK | launchpadlib, forget your auth token! | 04:58 |
StevenK | wgrant: Is DF done? | 05:42 |
wgrant | No | 05:43 |
wgrant | f | 2112345 | 05:43 |
wgrant | t | 1721777 | 05:43 |
wgrant | Almost half way | 05:44 |
StevenK | wgrant: Put me out of my misery trying to work where lazr.restful gets it values from? | 06:12 |
wgrant | StevenK: ResourceOperation.validate | 06:26 |
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:27 |
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:29 |
StevenK | Unknown consumer again! And Seahorse is unhelpful | 06:32 |
wgrant | You probably nuked your DB | 06:33 |
wgrant | Or it expired | 06:33 |
StevenK | I did nuke my DB | 06:36 |
StevenK | Your denorm columns not existing required a make schema | 06:36 |
wgrant | Or a database/schema/upgrade.py :) | 06:38 |
StevenK | I will usually just run make schema | 06:39 |
=== Nigel_ is now known as G | ||
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:30 |
wgrant | Pretend they're not there, and I'll fix them as I merge the pipe :) | 07:31 |
StevenK | 183>>>>>>> MERGE-SOURCE | 07:32 |
StevenK | Haha, looks like they're earlier up the chain | 07:32 |
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:33 |
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:34 |
StevenK | No, was just trying get the name 'PackageBuild' killed everywhere | 07:37 |
StevenK | wgrant: Oh, when does IPackageBuild die? | 07:39 |
wgrant | StevenK: It doesn't | 07:41 |
wgrant | It's an interface for package builds | 07:41 |
StevenK | wgrant: So IBinaryPackageBuild will also implement IPackageBuild ? | 07:44 |
wgrant | StevenK: It in fact inherits from it | 07:44 |
wgrant | BinaryPackageBuild implements it, with the help of PackageBuildMixin | 07:45 |
StevenK | Right | 07:45 |
StevenK | wgrant: r=me | 07:45 |
wgrant | Thanks | 07:45 |
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:22 |
wgrant | cjwatson: Oops, sorry | 08:24 |
wgrant | Should have time to look at that tomorrow | 08:24 |
cjwatson | Great, thanks | 08:24 |
wgrant | StevenK: Can you QA 16468 tonight? | 08:29 |
wgrant | Should be on qas shortly | 08:29 |
StevenK | Is now, apparently | 08:31 |
StevenK | wgrant: Done | 08:33 |
wgrant | StevenK: Thanks | 08:34 |
wgrant | We shall garbo it up on prod tonight | 08:34 |
StevenK | wgrant: All gree | 08:41 |
StevenK | *green | 08:41 |
adeuring | good morning | 08:46 |
=== _mup__ is now known as _mup_ | ||
=== gary_poster|away is now known as gary_poster | ||
=== Ursinha-afk is now known as Ursinha | ||
=== teknico_away is now known as teknico | ||
=== salgado is now known as salgado-lunch | ||
=== salgado-lunch is now known as salgado | ||
=== slank_away is now known as slank | ||
* 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 > | 15:05 |
=== teknico_away_ is now known as teknico | ||
=== teknico_ is now known as teknico | ||
=== deryck is now known as deryck[lunch] | ||
=== deryck[lunch] is now known as deryck | ||
=== gary_poster is now known as gary_poster|away | ||
=== gary_poster|away is now known as gary_poster |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!