| thumper | mwhudson: ping | 01:11 |
|---|---|---|
| thumper | bug 298284 | 01:11 |
| thumper | comments? | 01:11 |
| mup | Bug #298284: No headers suitable for filtering branch subscription emails for Gmail users <email> <Launchpad Bazaar Integration:Won't Fix> <https://launchpad.net/bugs/298284> | 01:11 |
| thumper | wgrant: since you are around, any comments on that bug? | 01:13 |
| wgrant | "fuck stupid mail clients", basically. | 01:13 |
| thumper | wgrant: well... around 50% of the launchpad users use gmail | 01:14 |
| wgrant | Silly people. | 01:16 |
| mwhudson | silly gmail too | 01:18 |
| mwhudson | thumper: i can't see how setting list-id as stub suggests would hurt | 01:18 |
| lifeless | perhaps you submit bugfixes.... | 01:18 |
| lifeless | oh right, its proprietary | 01:18 |
| lifeless | what you can do is write a imap client to do filtering for you on the gmail boxen | 01:19 |
| wgrant | Is there a good reason that revision emails come from noreply@ rather than the committer? | 01:26 |
| thumper | wgrant: no | 01:32 |
| thumper | wgrant: at least I don't think so | 01:32 |
| thumper | wgrant: what if the person doesn't want their email address shown? | 01:32 |
| thumper | wgrant: or we don't know who they are... | 01:32 |
| thumper | mwhudson: I'm beginning to think we should just do it (add a list id that is) | 01:33 |
| wgrant | thumper: If they're committing to a public branch with an email address but don't want it disclosed, they are probably insane. | 01:33 |
| thumper | mwhudson: conduct an experiment for code email, and if it ends up working, get other LP email sources to add it too | 01:33 |
| thumper | wgrant: heh | 01:33 |
| mwhudson | wgrant: some of our users are insane | 01:33 |
| mwhudson | thumper: +1 | 01:33 |
| * thumper is finally through the inbox | 01:34 | |
| * mwhudson has a desk to put together, will be in and out irc-wise | 01:34 | |
| thumper | mwhudson: my first balsamiq mockup sent to the list (and you) | 03:24 |
| thumper | mwhudson: do you think this is the right approach to starting? | 03:24 |
| thumper | (I do) | 03:24 |
| * mwhudson looks | 03:26 | |
| * thumper afk for a bit to get dinner on | 03:47 | |
| thumper | Ursinha: are you really here | 04:12 |
| thumper | ? | 04:12 |
| * thumper EODs | 04:58 | |
| lifeless | ciao | 05:00 |
| noodles775 | G'day all. | 07:38 |
| mrevell | Morning | 08:46 |
| wgrant | bigjools: I don't think anybody cares in the slightest about the "maintainer defaults". We have been saying this for years! | 10:25 |
| lifeless | wgrant: ? | 10:26 |
| wgrant | Bug 521722 | 10:26 |
| mup | Bug #521722: Display of Component on distro source summary page may be wrong (but we know the right value) <Soyuz:Incomplete> <https://launchpad.net/bugs/521722> | 10:26 |
| bigjools | wgrant: fine, we should just remove that column then | 10:28 |
| wgrant | bigjools: Why not replace it with the real value? | 10:35 |
| stub | bigjools: re: https://code.edge.launchpad.net/~wgrant/launchpad/sprbu-columns-to-sprb/+merge/18995 , do we have a record on who did the upload elsewhere? (SourcePackageRecipeBuildUpload.registrant at the moment) | 10:35 |
| bigjools | what real value? | 10:35 |
| bigjools | it's got many | 10:35 |
| bigjools | which are listed below | 10:35 |
| stub | (or wgrant) | 10:35 |
| wgrant | stub: SourcePackageRecipeBuild.requester. | 10:35 |
| bigjools | what he said | 10:35 |
| wgrant | bigjools: The current component in the development series is the current component. | 10:36 |
| stub | k | 10:36 |
| bigjools | wgrant: I don't see the benefit of repeating it | 10:36 |
| wgrant | bigjools: Hm, true, it is just there. | 10:36 |
| wgrant | Also, I wonder if we can make https://launchpad.net/debian/+source/dpkg more useful. The table doesn't take Pending SPPHs into account, so it's empty for gina'd archives. | 10:37 |
| bunjee | I'm currently downloading the sources of Launchpad through the installer, and this seems to be rather slow, I'm stuck at 33760 or something and it freezes from time to time | 10:39 |
| bunjee | Can we download them from another place? | 10:39 |
| wgrant | stub: Thanks. | 11:06 |
| adiroiban | Ursinha: hi, any idea why for bug 509252 the status was not updated? | 11:22 |
| mup | Bug #509252: Remove AdminPoTemplateSubset from security.py <cleanup> <Launchpad Translations:In Progress by adiroiban> <https://launchpad.net/bugs/509252> | 11:22 |
| leonardr | allenap, let me know when you have some time to help me with the bug problem i emailed you about | 11:26 |
| allenap | leonardr: Very soon, I just saw your message. | 11:26 |
| leonardr | i've made a little progress in diagnosis | 11:28 |
| allenap | leonardr: Okay, there are some weird things to do with bug comments/messages. I'll get your branches and try to replicate here. | 11:29 |
| leonardr | allenap, i can tell you it's not a navigation problem. a stock launchpad branch also turned up with a BugComment object there | 11:29 |
| allenap | leonardr: So I should be able to replicate with only your lazr.restful branch? | 11:33 |
| leonardr | allenap: my launchpad branch includes changes necessary to integrate the new lazr.restful branch | 11:34 |
| leonardr | so you'll need both | 11:34 |
| leonardr | allenap: i've confirmed that in a stock launchpad, the same code runs and self.entry.parent returns None | 11:44 |
| leonardr | so i think it's something to do with the way the entry adapter was generated | 11:44 |
| allenap | leonardr: Okay. | 11:46 |
| allenap | leonardr: I think I'm being stupid. How do you normally set up your lazr.restful branch in a Launchpad branch? | 11:49 |
| leonardr | allenap, good question | 11:50 |
| leonardr | symlink it to a directory in your launchpad root directory | 11:50 |
| leonardr | and change buildout.cfg to mention it | 11:51 |
| leonardr | develop = | 11:51 |
| leonardr | . | 11:51 |
| leonardr | lazr.restful.dev | 11:51 |
| leonardr | then run bin/buildout | 11:51 |
| allenap | leonardr: Ah ha, that sounds good. Thanks | 11:51 |
| leonardr | allenap: does this make any sense to you? i've got both versions of launchpad running side-by-side | 11:55 |
| leonardr | and it looks like the object returned by traversal is a BugComment in both cases | 11:55 |
| leonardr | but in stock Launchpad, the object that gets made into an EntryResource is a Message, and in my changed Launchpad, the object that gets put into the EntryResource is the same BugComment that was retrieved from traversal | 11:55 |
| leonardr | obviously a Message has a .parent and a BugComment does not | 11:56 |
| allenap | leonardr: Cool, that's useful info. I'm still trying to get the branch to run the test :-/ But I know that there's some smoke and mirrors in the bug message/comment area, and I helped to create some of it, so I'm hopeful I'll find the culprit. | 11:56 |
| leonardr | cool | 11:57 |
| leonardr | let me know if i can help you get it running | 11:57 |
| allenap | leonardr: It keeps failing in create-lp-wadl.py with: AttributeError: 'WebServiceTestRequest' object has no attribute 'version' | 11:57 |
| leonardr | allenap: that makes me think you're not using the new lazr.restful | 11:58 |
| leonardr | allenap: i did make clean and am now getting the same error | 12:09 |
| leonardr | this should be easy to fix | 12:10 |
| allenap | leonardr: I have to go and feed children, so I'll be back in ~45 minutes, sorry about that. | 12:13 |
| leonardr | ok | 12:13 |
| === henninge is now known as henninge-lunc | ||
| leonardr | flacoste, i need some help from you today. i'm trying to integrate the new lazr.restful into launchpad so that we can make launchpadlib use '1.0' before the feature freeze | 12:49 |
| leonardr | my gut feeling is that i won't make it, but if i'm going to make it i'll need advice from you since gary is not in today | 12:50 |
| james_w | leonardr: is /beta/ going to start changing at that point? | 13:04 |
| leonardr | james_w: no, beta will _stop_ changing at that point | 13:04 |
| james_w | nice | 13:04 |
| james_w | thanks :-) | 13:05 |
| leonardr | james_w: the goal is to retire 'beta' at the same time that karmic is retired | 13:05 |
| leonardr | if i don't make this goal then we will be retiring 'beta' when lucid is retired | 13:05 |
| leonardr | allenap: i've pushed an update that will make the wadl generation work (in doing so i discovered a much bigger problem with the wadl generation, but you don't need to worry about it) | 13:22 |
| === henninge-lunc is now known as henninge | ||
| leonardr | allenap: found it! | 13:40 |
| allenap | leonardr: Awesome :) | 13:40 |
| leonardr | i need your help figuring out what to do | 13:40 |
| leonardr | the problem is bugcomment_to_entry | 13:41 |
| allenap | leonardr: Phew, at least I might be able to help eventually. | 13:41 |
| leonardr | allenap: basically, in the new system, you can no longer adapt just an object to an IEntry | 13:42 |
| leonardr | you need to multi-adapt a 2-tuple (object, version_marker) | 13:42 |
| allenap | leonardr: Ah. What does the version_marker signify? | 13:43 |
| leonardr | allenap: which version of the web service the user is trying to access | 13:43 |
| leonardr | ie. beta, 1.0, etc | 13:43 |
| leonardr | i'm going to try just adding the marker interface to the adapter function and ignoring it. since this mechanism doesn't change between versions that _should_ work | 13:44 |
| leonardr | allenap: i think it's working | 13:55 |
| allenap | leonardr: Cool. I still don't quite get it all, but I'll pull your branch and figure it out. I've not been very useful to you, except perhaps as an over-the-shoulder viewer. | 13:56 |
| leonardr | allenap: i've pushed my revision. looking at webservice.zcml that seems to be the only special adapter launchpad has | 14:00 |
| leonardr | which would explain why everything else is working | 14:00 |
| allenap | leonardr: Ah, I get it now. Awesome. | 14:03 |
| flacoste | hi leonardr | 14:07 |
| leonardr | hi | 14:08 |
| leonardr | flacoste: right now i know of two unresolved issues w/r/t making launchpad multi-version | 14:09 |
| leonardr | the cached WADL file and the apidoc derived from that file | 14:09 |
| leonardr | right now my code is getting the 'devel' version of the WADL | 14:10 |
| leonardr | we need to get separate WADL and generate separate apidoc for each version | 14:10 |
| leonardr | but, maybe we don't need to do that right now | 14:11 |
| leonardr | flacoste: i'm going to add a launchpad test to make sure that the 1.0 web service is _accessible_. then i can get launchpad/lazr.restful reviewed and landed, and then i can do a new launchpadlib | 14:14 |
| leonardr | good plan? | 14:15 |
| flacoste | leonardr: i agree, you don't need the separate apidoc right now, nor the separate WADL file | 14:16 |
| flacoste | one thing we don't want though | 14:17 |
| flacoste | is that people start developping 1.0 launchpadlib application | 14:17 |
| flacoste | while we are still changing it | 14:17 |
| flacoste | might be better to freeze beta now | 14:17 |
| flacoste | cut devel | 14:17 |
| flacoste | and release a launchpadlib that talks to devel by default | 14:17 |
| flacoste | and discuss with the Ubuntu side that they would be ok with us uploading a version that switch devel to 1.0 in a couple of weeks | 14:18 |
| flacoste | before beta | 14:18 |
| flacoste | once we have frozen 1.0 | 14:18 |
| leonardr | flacoste: what changes were you hoping to make in 1.0? | 14:18 |
| flacoste | leonardr: remove obsolete mutators | 14:19 |
| flacoste | as named operations | 14:19 |
| leonardr | ok, that's the only one i was planning to do in the next couple weeks | 14:19 |
| flacoste | and maybe a general call to the teams to think about any API they wanted to change | 14:19 |
| flacoste | break backward compatibility | 14:19 |
| leonardr | flacoste: i don't think i agree about releasing a launchpadlib that talks to devel by default | 14:21 |
| leonardr | what is the worst that happens if we release a 1.0 launchpadlib and people start using it? | 14:21 |
| leonardr | 1.0 will start out one way and then change | 14:21 |
| leonardr | that's the same as if they start using devel | 14:21 |
| leonardr | and if something goes wrong and we end up releasing a devel launchpadlib in lucid, we're in big trouble | 14:22 |
| flacoste | leonardr: expectations management | 14:22 |
| leonardr | right now people don't even notice the version | 14:22 |
| flacoste | we'll have to announce that the version change is happening and what it means | 14:22 |
| flacoste | people will have to know that they can revert to beta if they want | 14:22 |
| flacoste | i think doing devel -> 1.0 is a less confusing story | 14:23 |
| leonardr | flacoste: do you know who i should talk to on the ubuntu side? | 14:24 |
| flacoste | james_w: do you have a suggestion about the above suggestion? | 14:24 |
| leonardr | james_w, maybe? | 14:24 |
| flacoste | james_w: s/suggestion/conversation/ | 14:24 |
| leonardr | flacoste, james_w: sorry, my laptop battery died. what have you been talking about? | 14:31 |
| flacoste | leonardr: no news from james_w yes | 14:31 |
| flacoste | yet | 14:31 |
| leonardr | ok | 14:31 |
| james_w | hi | 14:32 |
| james_w | sorry, was at lunch | 14:32 |
| james_w | you want to know if we would be ok with a devel -> 1.0 default change in a couple of weeks | 14:33 |
| james_w | ? | 14:33 |
| flacoste | james_w: basically, yes | 14:36 |
| james_w | I don't have a problem with it | 14:36 |
| james_w | I'm not on the release team though | 14:36 |
| flacoste | james_w: has launchpadlib been moved to main? apt still shows it as Ubuntu MOTUY Developers | 14:37 |
| james_w | it has | 14:37 |
| flacoste | james_w: are you the maintainer? | 14:37 |
| james_w | it's in Debian now, so the Debian python team take care of it | 14:37 |
| flacoste | james_w: and who is that? | 14:38 |
| james_w | there's a whole bunch of them | 14:38 |
| james_w | Luca is the one that has done most of the work so far | 14:38 |
| flacoste | so I guess we should talk to him | 14:39 |
| flacoste | and someone on the release team | 14:39 |
| flacoste | pitti or slangasek i assume | 14:40 |
| james_w | I can talk to Luca if you get the go ahead | 14:42 |
| leonardr | flacoste: can you sort things out with the distro developers while i complete the integration? | 14:42 |
| james_w | yeah, #ubuntu-release works | 14:43 |
| flacoste | leonardr: i think it's best if you or gary handles it directly, let me know if there is problem | 14:43 |
| leonardr | ok | 14:43 |
| kfogel | I never modified sendmail.py in my branch. But when I merged from today's devel, I got a conflict there. I resolved it in the obvious way (half of it was empty text, and it looked like a bzr artifact), I still have this diff: | 15:10 |
| kfogel | http://paste.ubuntu.com/376899/ | 15:10 |
| kfogel | (I'm sure this is related to r10306 in devel) | 15:11 |
| kfogel | the sha part I can explain from my resolution, and can fix. the set thing mystifies me | 15:13 |
| kfogel | oh, duh, nm. devel vs db-devel issue sigh | 15:15 |
| === jamlata-afk is now known as jamalta | ||
| === beuno is now known as beuno-lunch | ||
| === henninge is now known as henninge-afk | ||
| kfogel | adeuring: got a sec? I'm beginning to wonder if I'm on the right track here. http://paste.ubuntu.com/376948/ | 16:30 |
| adeuring | kfogel: LET ME LOOK... | 16:30 |
| kfogel | adeuring: caps lock? :-) | 16:31 |
| adeuring | kfogel: yeah... This key should be banned... | 16:31 |
| kfogel | :-) | 16:31 |
| adeuring | or I should remove it from my keyboard | 16:31 |
| thekorn | hi thumper, when you are around, and have a minute, I've a question about bug 520412 | 16:32 |
| mup | Bug #520412: fix review_types argument of the API's Branch.createMergeProposal method <code-review> <oops> <Launchpad Bazaar Integration:Triaged> <https://launchpad.net/bugs/520412> | 16:32 |
| kfogel | adeuring: what I'm trying to do is make a portlet template that can be reused on other pages as well, not just the product-index page | 16:40 |
| adeuring | kfogel: is patches-view.test an IProduct ? | 16:41 |
| adeuring | erm, patches-view-test | 16:41 |
| kfogel | adeuring: yes | 16:42 |
| adeuring | odd... I that case, I'm lost. In theory, things look correct, I'd say... But: Did you consider to define the protlet for IHasBugs instead of IProduct? | 16:42 |
| adeuring | kfogel: I know, that question is not related to your problem, but anyway... | 16:43 |
| kfogel | adeuring: phone, one sec | 16:48 |
| adeuring | kfogel: no problem... I'm afraid you need to as somebody else anyway :( | 16:48 |
| === beuno-lunch is now known as beuno | ||
| adeuring | kfogel: Just tired the diff you posted. Looks good; no problems to access http://launchpad.dev/firefox ; the new portlet appears. But there is a little bug in your ZCML data: a duplicate tag </browser:pages> | 17:14 |
| adeuring | s(tired/tried/ | 17:14 |
| kfogel | adeuring: d'oh | 17:14 |
| kfogel | adeuring: thank you (off phone now) | 17:14 |
| kfogel | adeuring: I wonder why I'm getting that OOPS. | 17:14 |
| kfogel | adeuring: you're right that the portlet should be for IHasBugs, I think | 17:15 |
| adeuring | kfogel: A guess: Your test object is something else than an IProduct, or | 17:15 |
| adeuring | you did not test the diff that you posted. | 17:15 |
| adeuring | more likely the latter -- the web server should simply not start wirth that ZCML data ;) | 17:16 |
| kfogel | adeuring: aaaaaah, that may be it | 17:18 |
| kfogel | adeuring: thx | 17:18 |
| kfogel | adeuring: working for me now | 17:26 |
| adeuring | kfogel: great! | 17:26 |
| jtv | Ursinha: poimport error reporting seems to be broken on staging... do you know anything about it? | 17:29 |
| kfogel | adeuring: when you asked "Did you consider to define the portlet for IHasBugs instead of IProduct?", did you mean something like http://paste.ubuntu.com/376995/ ? | 17:41 |
| adeuring | kfogel: yes | 17:42 |
| kfogel | adeuring: cool, thanks | 17:42 |
| kfogel | adeuring: man, I wish we were sitting in the same room :-) | 17:42 |
| adeuring | kfogel: shall we organize a mini sprint ;)? | 17:42 |
| kfogel | adeuring: I'm all for it sometime soon, actually. I have some travel plans solidifying right now, but once I know those, maybe I can come out to Berlin or you to NY in the next month or two... | 17:43 |
| adeuring | kfogel: sounds like an interesting idea! But as a nitpick, I don't live in berlin, but in Bielefeld (ca 3 hours by train from berin) | 17:45 |
| adeuring | s/berin/Berlin/ | 17:45 |
| kfogel | adeuring: oh, sorry -- dunno why I thought you lived in Berlin. | 17:45 |
| adeuring | kfogel: because both names start with 'B' and sound somewhat foreign ;)? | 17:49 |
| kfogel | adeuring: I'm not *that* American, man :-) | 17:49 |
| * kfogel skirts around self-examination | 17:49 | |
| adeuring | kfogel: perhaps is because Bielefeld does not exist: http://www.dw-world.de/dw/article/0,,1400913,00.html | 17:51 |
| kfogel | adeuring: I always suspected you might live in a place that does not exist. Don't think we didn't all notice that you showed up in Alabama with an empty suitcase, or that your ticket back listed a non-existent terminal at Atlanta airport. | 17:53 |
| adeuring | kfogel: you didn't noticed my other suitcase! | 17:54 |
| kfogel | adeuring: that's because that one didn't exist at all! | 17:55 |
| kfogel | .oO (sheesh) | 17:55 |
| adeuring | kfogel: bullshit. I travel _always_ with two suitcases! | 17:55 |
| adeuring | though one most times invislibke | 17:55 |
| adeuring | ...invisible | 17:55 |
| kfogel | Now you're talking crazy, clearly. | 17:56 |
| adeuring | well, I'm from a town that doesn't exist -- what do you expect ;)? | 17:57 |
| jtv | mthaddon, if you're still here: could you (a) stop for the day or (b) give me another import run on staging? :-) | 17:57 |
| jml | g'night folks. | 18:07 |
| * jml will be back later for kiwi chat! | 18:07 | |
| === flacoste is now known as flacoste_lunch | ||
| mwhudson | morning jml | 18:24 |
| jelmer | hello mwhudson | 18:34 |
| mwhudson | jelmer: hi | 18:34 |
| kfogel | adeuring: http://paste.ubuntu.com/377042/ | 18:49 |
| kfogel | adeuring: background: I realize that using the same patch batcher as we use in the main +patches view is not a good long-term solution; I was just trying to get something working, and then to refine to somehow show the "top 5" youngest patches or something in the portlet. | 18:49 |
| kfogel | I failed at the "get something working" stage, as you can see :-). | 18:50 |
| === danilos is now known as daniloff | ||
| lifeless | morning morning people | 19:33 |
| === flacoste_lunch is now known as flacoste | ||
| adeuring | kfogel: that's because lp.registry.browser.product.ProductView does not define batchedPatchTasks . You should use the view class that has this method in the ZCML attribute 'class="..."' | 19:48 |
| adeuring | kfogel: and sorry for the late answer. was out to buy some food | 19:49 |
| kfogel | adeuring: no problem -- it's already late for you! | 19:49 |
| kfogel | adeuring: edits to .zcml files require restart or no? | 19:49 |
| kfogel | adeuring: it seems to have been working without restart... | 19:50 |
| kfogel | adeuring: ...but I wonder if I'm just getting lucky or something. | 19:50 |
| adeuring | kfogel: I think youmust restart the server | 19:50 |
| * kfogel winds up the mouse | 19:50 | |
| beuno | rockstar, thumper, did you guys know that branch icons seem to be missing on edge? | 19:52 |
| thumper | beuno: yes, bug has been filed | 20:01 |
| thumper | rockstar: ping? | 20:01 |
| kfogel | adeuring: it must be late there -- curious, how much longer you plan to be on? | 20:20 |
| adeuring | kfogel: perhaps an hour or so | 20:20 |
| kfogel | adeuring: *nod* thx | 20:20 |
| rockstar | thumper, hi. | 20:31 |
| leonardr | mars, can you help me with a windmill test failure? | 20:37 |
| kfogel | adeuring: oh wow, while adding some more test data, I just got this great confirmation box: | 20:38 |
| kfogel | This file does not look like a patch. What is a patch? | 20:38 |
| kfogel | Is this file a patch: | 20:38 |
| kfogel | yes | 20:38 |
| kfogel | no | 20:38 |
| kfogel | adeuring: that is so awesome | 20:38 |
| mars | leonardr, sure, I can have a look. what is the problem? | 20:38 |
| leonardr | mars, http://paste.ubuntu.com/377137/ | 20:38 |
| adeuring | kfogel: thanks :) | 20:40 |
| james_w | I just hit the bug that caused me to propose https://code.edge.launchpad.net/~james-w/launchpad/sync-source-negative-versions/+merge/16861 again, and I was a little surprised to realise that it's been approved for 6 weeks and isn't yet merged, let alone in production, could someone move it along please? | 20:42 |
| mars | leonardr, did you try running those tests locally? | 20:43 |
| mars | leonardr, bin/test --layer=BugsWindmillLayer -t test_security_settings_form_overlay | 20:43 |
| leonardr | mars, trying now | 20:44 |
| kfogel | james_w: can you not land via PQM? | 20:45 |
| james_w | kfogel: I cannot | 20:46 |
| poolie | hello james_w, kfogel | 20:46 |
| kfogel | james_w: two questions: one, why not? (we should fix that?) | 20:46 |
| james_w | hi poolie | 20:46 |
| james_w | kfogel: I'm not on the LP team | 20:46 |
| kfogel | two: I notice the repeat of "if dest_version is None" in the diff... looking closely at the redundancy now. | 20:47 |
| kfogel | poolie: hey there | 20:47 |
| kfogel | james_w: oy vey | 20:47 |
| kfogel | james_w: ok | 20:47 |
| poolie | hi kfogel | 20:47 |
| james_w | I don't know if that's a rule, but it's why I don't have it currently, and I'm not even sure that I would want the ability :-) | 20:49 |
| poolie | james_w: i may see about batch-upgrading those branches soon | 20:49 |
| leonardr | mars: i get a different error, but it's one that looks like my fault... | 20:50 |
| james_w | thanks | 20:50 |
| kfogel | james_w: yeah, looking at the diff and at the full file's source... I can't take a break from what I'm working on right now long enough to puzzle out whether that apparently-repeated conditional makes sense, but it might be worth a second look from you? | 20:51 |
| leonardr | mars: here's the local errors: | 20:51 |
| leonardr | http://paste.ubuntu.com/377148/ | 20:51 |
| leonardr | The "could not fulfil proxy request" looks like it could have started failing due to multiversion. could that have caused the other errors? | 20:52 |
| james_w | kfogel: it does make sense (at least to me) the first case is for the actual catching of the case where the package isn't in Ubuntu, the second is to not crash trying to do "ubuntu" in None. | 20:52 |
| james_w | kfogel: it may be possible to restructure everything to avoid it, but there may well be other duplication if that is done | 20:53 |
| kfogel | james_w: if (dest_version is None | 20:59 |
| kfogel | or apt_pkg.VersionCompare(dest_version, source_version) < 0): | 20:59 |
| kfogel | if (not Options.force | 20:59 |
| kfogel | and dest_version is not None and dest_version.find("ubuntu") != -1)): | 20:59 |
| kfogel | james_w: (off the top of my head, just for clarity) | 20:59 |
| kfogel | james_w: but I see what you mean; it's tricky | 20:59 |
| kfogel | adeuring: got a sec for quick discussion? (might be useful to check out lp:~kfogel/launchpad/255868-link-to-patches-view for this) | 21:01 |
| adeuring | kfogel: yes | 21:02 |
| kfogel | adeuring: basically, I'm trying to figure out a) the best way, visually, to present this "latest patches" list, and b) how to limit it to the five (or whatever) youngest patches. | 21:02 |
| adeuring | kfogel: I am really bad for aestheical things ;) | 21:04 |
| kfogel | adeuring: no problem, let's limit it to question (b) :-) | 21:04 |
| adeuring | ok ;) | 21:04 |
| adeuring | kfogel: To limit the result set, just use something like storm_result_set[:5] | 21:05 |
| kfogel | adeuring: right now it will just show batch-size number of bugtasks with patches -- could be 50. | 21:05 |
| kfogel | adeuring: aaaaaaaah | 21:05 |
| kfogel | adeuring: in the .pt file?? | 21:06 |
| adeuring | kfogel: it is probably a bt nicer to do that in the view class. But it should also be possible in the template | 21:06 |
| kfogel | adeuring: well, the view class is BugsPatchesView, the same view we're using for the full +patches view. Ah, but maybe I should make a new method in the view, instead of using batchedPatchTasks. Something like latestPatchTasks or something. Is that the usual route? | 21:08 |
| mars | leonardr, got lost in a context switch, looking | 21:08 |
| adeuring | kfogel: yes, that's what I meant | 21:08 |
| kfogel | adeuring: thanks, all clear now | 21:08 |
| adeuring | great | 21:08 |
| leonardr | mars: when i run it locally and watch the browser window i see 2 failures | 21:08 |
| leonardr | the first one is that we're expected to be at the login page when we're actually at the bug detail page | 21:09 |
| leonardr | and the other is an exception generated by the web service, which i believe is caused by the attempted request to /api/beta/api/devel/ | 21:09 |
| mars | leonardr, have you tried running it stand-alone, and changing the privacy? | 21:09 |
| leonardr | mars, trying that now | 21:09 |
| mars | leonardr, don't worry about the first error, that is something in windmill we need to silence | 21:10 |
| mars | it does not cause a failure, just a lot of noise | 21:10 |
| leonardr | so the ERROR is not the same as a bug failure? | 21:10 |
| mars | line 9 is just a loud BANG, line 11 is your branch dying | 21:11 |
| leonardr | mars: yes, changing the bug privacy settings POSTs to /api/beta/api/devel/bugs/15 | 21:13 |
| mup | Bug #15: PO file import errors should be more verbose <feature> <Launchpad Translations:Fix Released by carlos> <https://launchpad.net/bugs/15> | 21:13 |
| leonardr | that's definitely my fault but i don't know how it happens... | 21:13 |
| mars | how what happens? | 21:13 |
| leonardr | how the /api/devel/ gets in there | 21:13 |
| mars | client.js | 21:14 |
| mars | or base-template-macros.pt | 21:14 |
| mars | grep for the string '/api' in *.js and/or *.pt | 21:14 |
| leonardr | mars: well, i know that if someone was in fact sending a request to /api/devel it would get munged by LP.client.normalize_uri into /api/beta/api/devel | 21:15 |
| leonardr | but no one should be doing that yet | 21:16 |
| mars | leonardr, are there any in-page <script> nodes that have JS with the old path in there | 21:18 |
| mars | leonardr, looking at base-layout-macros.pt, there is a fmt:api_url call | 21:18 |
| leonardr | mars: aha! | 21:18 |
| leonardr | that will give the url of the latest version | 21:19 |
| leonardr | let me try just changing everything to use /api/devel and see how much damage it does | 21:19 |
| mars | most of the JS is driven by variables pulled from on-page script nodes. So a bad webservice URL probably comes onto the page via template somehow. | 21:20 |
| leonardr | mars: attempting to use the devel web service gives incredibly weird results, so i'm going to hack fmt:api_url | 21:32 |
| leonardr | the weird results: | 21:32 |
| mars | leonardr, heh, ok | 21:32 |
| leonardr | (Pdb) self.entry.private = True | 21:32 |
| leonardr | *** AssertionError: Expected int for Person foreign key reference, got <type 'object'> | 21:32 |
| mars | you should be able to capture that with pdb | 21:33 |
| mars | leonardr, I need to sign off for tonight. Please ping me tomorrow if you need more help debugging the templates | 21:33 |
| leonardr | mars, sure | 21:33 |
| leonardr | i think i will just come back to this tomorrow as well, i've been working all day | 21:34 |
| === jamalta is now known as jamalta-afk | ||
| noodles775 | Hey thumper, mwhudson, jml: is 22UTC still ok? (ie. in 5mins) | 21:56 |
| thumper | noodles775: good for me | 21:56 |
| mwhudson | noodles775: sure | 21:57 |
| thumper | noodles775: skype? | 22:01 |
| noodles775 | thumper: yeah, absoludity is my id. | 22:03 |
| === jamalta-afk is now known as jamalta | ||
| thumper | noodles775: lets try a conf call | 22:04 |
| thumper | noodles775: skype still failing for me on a conf call | 22:05 |
| mwhudson | that all sounded fine to me | 22:05 |
| thumper | noodles775: can you try hosting? | 22:05 |
| mwhudson | i can host | 22:05 |
| thumper | either or, I don't care | 22:05 |
| noodles775 | OK, otherwise voip is fine too. | 22:05 |
| mwhudson | or noodles775 | 22:05 |
| * noodles775 hosts | 22:05 | |
| mwhudson | ah | 22:05 |
| wgrant | win 23 | 22:06 |
| wgrant | Argh. | 22:06 |
| lifeless | lose 45 | 22:06 |
| mwhudson | noodles775: ... | 22:06 |
| wgrant | lifeless: Indeed. | 22:06 |
| mwhudson | noodles775: are you trying to host too | 22:06 |
| mwhudson | ? | 22:06 |
| mwhudson | i've just started a call... | 22:06 |
| noodles775 | yes, I did. | 22:06 |
| noodles775 | OK, add me in :) | 22:06 |
| mwhudson | jml: poke | 22:07 |
| jml | mwhudson, back online | 22:09 |
| jml | please invite | 22:09 |
| mwhudson | jml: am doing | 22:09 |
| noodles775 | thumper: https://dev.launchpad.net/BuildBranchToArchiveUI/UseCaseManualBuild | 22:11 |
| noodles775 | thumper: https://dev.launchpad.net/BuildBranchToArchiveUI/UseCaseComplexRecipe | 22:13 |
| mwhudson | skype fail | 22:18 |
| mwhudson | or possibly new zealand internet fail | 22:19 |
| noodles775 | https://edge.launchpad.net/+apidoc/#distribution_source_package | 22:26 |
| thumper | :( | 22:32 |
| mwhudson | thumper: yay | 22:32 |
| mwhudson | skype | 22:32 |
| thumper | mwhudson: you dropped too? | 22:32 |
| mwhudson | thumper: no, just you | 22:32 |
| poolie | mwhudson: you legend (re upgrades) | 22:49 |
| jml | noodles775, stupid question, but which mockup are you looking at right now? | 22:57 |
| noodles775 | https://dev.launchpad.net/BuildBranchToArchiveUI/UseCaseManualBuild | 22:58 |
| === jamalta is now known as jamalta-afk | ||
| noodles775 | https://dev.launchpad.net/BuildBranchToArchiveUI | 23:08 |
| * wgrant loves LP_PERSISTENT_TEST_SERVICES. Sub-20s test runs are actually getting back into the realms of practicality. | 23:25 | |
| thumper | wgrant: what is that? | 23:30 |
| wgrant | thumper: As of a few days ago, setting that will keep librarian and memcached running between test runs. | 23:31 |
| thumper | hmm... | 23:31 |
| wgrant | thumper: So you save the entire startup time. | 23:31 |
| thumper | wgrant: and how do you set it? | 23:31 |
| wgrant | export LP_PERSISTENT_TEST_SERVICES=1 | 23:31 |
| thumper | wgrant: and how do you tell it when you're done? | 23:31 |
| wgrant | No idea. | 23:31 |
| thumper | wgrant: where is it documented? | 23:31 |
| wgrant | thumper: I only know of it from seeing devel r10273 fly past. | 23:32 |
| wgrant | Ah, bin/kill-test-services | 23:34 |
| poolie | sheesh, you guys need to talk more | 23:34 |
| wgrant | Sssh. | 23:35 |
| mwhudson | poolie: i can supply stupid shell scripts at little notice! | 23:35 |
| thumper | mwhudson: do you have any feel for how long it is taking? | 23:36 |
| mwhudson | thumper: i think it's probably about 5% done? | 23:36 |
| thumper | mwhudson: through 3k? | 23:37 |
| mwhudson | ish | 23:37 |
| mwhudson | maybe less | 23:37 |
| mwhudson | "a couple of days" | 23:37 |
| mwhudson | particular as i'm going to have to kill it and go into town in a minute | 23:37 |
| mwhudson | (i don't have an ssh key on devpad) | 23:37 |
| thumper | screen FTW | 23:37 |
| thumper | I created an SSH key for devpad :) | 23:38 |
| mwhudson | doesn't help with agent forwarding | 23:38 |
| mwhudson | thumper: well, in that case... | 23:38 |
| thumper | mwhudson: how soon are you going? | 23:39 |
| thumper | mwhudson: I have a trivial bug fix :) | 23:39 |
| mwhudson | thumper: kinda now ish | 23:39 |
| thumper | mwhudson: I'll submit it normally and you can look later :) | 23:39 |
| mwhudson | thumper: want to run "cp ~mwh/.bazaar/plugins/l.py ~/.bazaar/plugins/" | 23:40 |
| wgrant | That's a descriptive name. | 23:40 |
| mwhudson | and then "for i in `cat ~mwh/non-2a-branches`; do echo $i; date; bzr upgrade lp:$i ; bzr flibble lp:$i ; done" | 23:40 |
| mwhudson | thumper: on devpad? | 23:40 |
| thumper | flibble? | 23:40 |
| mwhudson | wgrant: the command name is even better! | 23:40 |
| mwhudson | yes, flibble | 23:40 |
| wgrant | mwhudson: Indeed. | 23:40 |
| thumper | heh | 23:40 |
| mwhudson | it write locks and then unlocks the branch | 23:40 |
| mwhudson | thus triggering the puller to run | 23:41 |
| wgrant | Isn't there a convenient in-LP method to do the upgrade now? | 23:41 |
| thumper | wgrant: yes (ish) | 23:41 |
| mwhudson | not as convenient as a stupid shell script | 23:41 |
| thumper | mwhudson: is non-2a-branches ordered corrently for upgrades? | 23:42 |
| thumper | mwhudson: also what about the ones that are done already? | 23:42 |
| mwhudson | thumper: doesn't matter | 23:42 |
| thumper | ok | 23:43 |
| mwhudson | thumper: upgrade will just upgrade the repo if it can't open the branch | 23:43 |
| mwhudson | thumper: ones that are done already will cause "already at most recent format" type errors | 23:43 |
| mwhudson | but well, that's not really a problem | 23:43 |
| thumper | mwhudson: running now | 23:44 |
| thumper | mwhudson: bzr: ERROR: RemoteRepository(bzr+ssh://bazaar.launchpad.net/~ubuntu-branches/debian/squeeze/autoconf-nonfree/squeeze/.bzr/) | 23:45 |
| thumper | is not compatible with | 23:45 |
| thumper | CHKInventoryRepository('bzr+ssh://bazaar.launchpad.net/~ubuntu-branches/debian/sid/autoconf-nonfree/sid/.bzr/repository/') | 23:45 |
| thumper | different serializers | 23:45 |
| mwhudson | yeah | 23:45 |
| thumper | why? | 23:45 |
| mwhudson | doesn't actually seem to matter in the end though :-) | 23:45 |
| poolie | thumper: known bug about cross-format stacking | 23:45 |
| mwhudson | dunno | 23:45 |
| poolie | sucks | 23:45 |
| poolie | feel free to bump up the bug | 23:45 |
| thumper | what is the issue around it? | 23:45 |
| poolie | bzr doesn't support stacking across branches of varying formats | 23:58 |
| poolie | and lp configures package branches to be stacked on each other | 23:58 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!