[00:08] ugh, i'm sure apis are supposed to work in read-only mode... [00:09] mwhudson: I'm seeing distribution['ubuntu'] failing. [00:09] I never have before. [00:09] Are you seeing something similar? [00:09] 503s? [00:10] Yeah. [00:10] I've definitely seen them before [00:10] For some things, sure. [00:10] I've not tried to correlate it to anything [00:10] But never this. [00:13] Ugh. [00:14] wee! [00:14] c.l.d.oauth tries to use the master store. [00:14] i wonder what's happening [00:14] But that hasn't changed lately. [00:15] the response is being generated by launchpad, it seems? === mbarnett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: abentley | https://code.launchpad.net/launchpad-project/+activereviews [00:15] so there should be oopses [00:15] maybe? [00:16] mwhudson: Unlikely. [00:16] That probably won't generate an OOPS. [00:16] But I can see the issue locally. [00:16] ah ok [00:16] http://paste.ubuntu.com/565226/ [00:17] That's with login_anonymously [00:34] wgrant: looking [00:35] gary_poster: Thanks. Although it's less critical now, since the rollout failed so we'll probably have to cowboy. [00:35] wgrant, ah :-/ [00:37] wgrant, qa instructions already exist for the first: last para of description of https://code.launchpad.net/~gary/launchpad/bug548/+merge/48850 [00:37] looking at the other one... [00:38] I also did the same for the other one, wgrant (https://code.launchpad.net/~gary/launchpad/bug713392/+merge/48939) [00:38] gary_poster: Ah, sorry. Just about nobody does, so I stopped checking there a while ago. [00:39] wgrant, np, understood :-) . I'm not consistent either, but I try to do that === Ursinha is now known as Ursinha-afk [01:21] so... [01:21] what's the status with writing new windmill tests? [01:21] we are supposed to? [01:40] spm, i sent bug mail to lp during the rollout and it does not yet seem to have any effect.... [01:52] nm, they have come through [02:12] * thumper nipping out to what the girls' first touch game of the season [02:15] Project devel build (429): STILL FAILING in 6 hr 6 min: https://hudson.wedontsleep.org/job/devel/429/ [02:15] * Launchpad Patch Queue Manager: [r=jtv][bug=714820] Ignore malformed responses when probing remote [02:15] Bugzilla instances for XML-RPC capabilities. [02:15] * Launchpad Patch Queue Manager: [r=adeuring][bug=181368] Gut BinaryPackageFilePublishing view. [02:15] * Launchpad Patch Queue Manager: [r=jtv][bug=715659] Use zope.exceptions to format logged exceptions [02:15] so that __traceback_info__ is reported. [02:15] * Launchpad Patch Queue Manager: [r=leonardr][bug=715116] Add a big informational notice on the PPA [02:15] pages if publishing is disabled. [02:15] * Launchpad Patch Queue Manager: [r=lifeless][bug=713234] Remove duplicated queries issued when [02:15] calling ArchiveView.latest_updates [02:36] https://code.launchpad.net/~lifeless/launchpad/bug-704446/+merge/49167 [02:51] hi lifeless [02:52] hi poolie [02:54] do you have any idea whether https://bugs.launchpad.net/launchpad/+bug/716175 would be shallow? [02:54] <_mup_> Bug #716175: api calls fail while launchpad is readonly < https://launchpad.net/bugs/716175 > [03:03] poolie: I suspect its not, but without an oops can't really confirm (could simulate locally I guess) [03:04] poolie: but I'm not hugely interested in fixing readonly mode, I don't think its a particularly good use of engineering time [03:05] poolie: I'm going to reply to jam about loggerhead and try and clear up the confusion [03:05] poolie: however I have a few fires to deal with first today. [03:14] huwshimi: ping [03:14] lifeless: Hey [03:14] huwshimi: contrast this [03:14] http://bazaar.launchpad.net/~jameinel/loggerhead/merge_pqm_updates/files [03:14] and [03:14] http://bazaar.launchpad.net/~jameinel/loggerhead/merge_pqm_updates/changes [03:14] note the former has a 'help' tab [03:14] its in the lp loggerhead branch only [03:14] I think it was added during the drive to have 'help everywhere' in LP [03:14] but if you click on it [03:15] it takes you to a sad panda wiki page [03:15] I'm seeking a second opinion on just discarding that noddy tab [03:15] lifeless: It's easy to replicate locally. [03:15] I pastebinned the traceback earlier. [03:15] I don't see how it could have ever worked, but it apparently did.d [03:15] wgrant: I was not connected to irc :) [03:15] Right. [03:15] wgrant: feel free to annotate the bug ;) [03:18] lifeless, yeah, much better to just never go into readonly mode [03:18] hearty +1 from me to removing the help tab [03:18] lifeless: Help that is not helpful is useless. It appears to me that that help is not really helpful. You could either remove the tab or provide some useful help. [03:19] those little 'help' drawers were such an awful idea [03:19] lifeless, in that case i might bump 716175 down to low [03:19] poolie: I have done so [03:20] Useless help just results in what I like to call "help page rage". [03:23] I once worked on a massive application where all the help documentation for every form field was "This is the Foo field". The documentation guy didn't understand why this was a bad thing. [03:24] huwshimi, that's what we had [03:24] i cannot think of a better way to train people to never click on things labelled Help [03:24] poolie: Ouch. [03:24] if we'd kept the same mechanism but only used it when we had something important to say, it would have been half decent [03:25] but better to just put it inline if it's really important and you can't think of a way to make it obvious [03:27] If there is a bug that has been fixed by another bug do I change it to "Fix released"? [03:28] yes, or mark it as dupe, or as invalid [03:28] whatever you think best expresses the situation or is easiest [03:28] poolie: Cheers [03:35] abentley: are you still OCR? [03:37] lifeless: somewhat late for abentley don't you think? === thumper changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews [03:37] yes :) [03:38] * thumper still feels strangely protective [03:39] :) [03:42] Does anyone know what research might be alluded to in this comment? https://bugs.launchpad.net/launchpad/+bug/490058/comments/7 [03:42] <_mup_> Bug #490058: Launchpad shouldn't present itself as a database < https://launchpad.net/bugs/490058 > [03:42] she has done some user testing sessions [03:42] mm [03:43] i don't think the title of that bug is very accurate [03:43] it should be something more like "Launchpad home page should show cool stuff that's happening" [03:43] "not presented as a database" would be more like getting away from table-type views [03:46] poolie: Hmm.. I should try and get her findings [03:47] she'd be a good person for you to know [03:47] There were a couple of surveys last year too. [03:47] I never saw anything from them; were they even distributed internally? [03:47] mrevell posted something, but not very prominently [03:47] mm [03:48] fyi. am doing the stabby dance on the builtbot master to remove the living dead problem it's caused. [03:48] it's kind of connected to the timeline view concept [04:00] w00t: Committed revision 12345. [04:00] nice number [04:00] Project db-devel build (354): STILL FAILING in 5 hr 20 min: https://hudson.wedontsleep.org/job/db-devel/354/ [04:00] Launchpad Patch Queue Manager: [rs=buildbot-poller] automatic merge from stable. Revisions: 12348 [04:00] included. [04:54] Ugh, what am I doing wrong. I updated download-cache and did an update of devel and merged it into my branch and now getting this error: http://paste.ubuntu.com/565245/ . I know there was an update to YUI that removed a whole bunch of files. Is it possible there's something else I need to update? [04:58] huwshimi: try a "make clean build" [05:02] thumper: Trying but really weird things are going on [05:08] thumper: That's fixed it. Thanks mate [05:14] wgrant: Did you end up finding any of those facet images? [05:16] huwshimi: They should be in the history of lib/canonical/launchpad/icing. [05:16] Removed in the last 18 months. [05:17] wgrant: Cheers [05:35] wgrant: Would you use bzr explorer or something to find these files? [05:40] huwshimi: Try checking the directory around r10000 or so. Just create a new branch and revert to then. [05:40] wgrant: ah right ok [05:44] thumper: ping [05:45] thumper: I know its paste your EOD, but perhaps you could mentor Ian's review on https://code.launchpad.net/~lifeless/launchpad/bug-704446/+merge/49167 [05:54] lifeless: that sql bind params review - you approved it but you also need to claim it so it can be landed https://code.launchpad.net/~wallyworld/launchpad/profile-sql-bind-values/+merge/48872 [05:54] pretty please :-) [05:56] roger [06:32] huwshimi: Did you find the images? [06:32] wgrant: yeah got them. I remember then now [06:33] Someone went through and cleaned up c/l/icing a while back. === almaisan-away is now known as al-maisan === al-maisan is now known as almaisan-away === almaisan-away is now known as al-maisan [07:05] ok time to head off [07:19] stub: could I have a review on https://code.launchpad.net/~lifeless/launchpad/bug-704446/+merge/49167 [07:29] lifeless: What is the "+ AND BugMessage.index > 0" for? [07:29] end of the diff [07:33] tunes the query for 'commented on a bug' [07:34] oic. 0 is first comment == description [07:34] https://bugs.launchpad.net/launchpad/+bug/421901/comments/10 [07:34] <_mup_> Bug #421901: Person:+bugs timeouts < https://launchpad.net/bugs/421901 > [07:36] approved [07:37] thanks [07:43] hmm [07:43] bringing back fti columns seems entirely pointless [07:43] jkakar: is there some way to tell storm /not/ to load a column from the db ? [07:45] stub: can you compare the execution time for these on prod ? http://pastebin.com/raw.php?i=Lib9DXHz [07:54] lifeless: second 11 seconds, first 12.7 seconds [07:55] grr, really? [07:55] Time: 1355.438 ms on staging [07:55] [qa]staging [07:58] http://paste.ubuntu.com/565267/ [07:59] lifeless: qastaging is patched up remember [07:59] stub: this doesn't depend on any db patches [07:59] oh... [08:00] we haven't indexed BugMessage.index on qastaging yet. [08:00] stub: can I get a plan of that second query, once hot ? [08:02] stub: also please try http://pastebin.com/TeHbFEwD [08:02] That was the second query. This is the plan of the first one: http://paste.ubuntu.com/565268/ [08:03] seq scan on bugmessage [08:03] urg... that pastebin includes linenumbers in c&p [08:03] http://pastebin.com/raw.php?i=TeHbFEwD [08:04] stub: click on 'raw' [08:05] I suspect there are a lot of bugs with only a single comment [08:06] reverse that... [08:06] indeed [08:07] We have an index on (bug,index) [08:08] http://pastebin.com/HMY6QqJd btw [08:08] well, 8 seconds is better than 12 [08:09] I think I'll unlink the bug fro mthis branch, its going to need more work [08:13] stub: do you think a separate index on (index,) is needed, or perhaps two indices (bug,) and (index,) and drop the composite index? [08:13] I'll test a separate index in a tic. Looking at some alternative querys atm. [08:15] the one in https://bugs.launchpad.net/launchpad/+bug/421901/comments/5 I think you tried already for me, but was the best I found when experimenting last week [08:15] <_mup_> Bug #421901: Person:+bugs timeouts < https://launchpad.net/bugs/421901 > [08:16] different indexes don't seem to help on BugMessage.index (or message,index) [08:17] There are 177k messages from that user... [08:18] probably why it shows up in the timeouts [08:20] but I don't see a sane way to execute the query. Traversing from bug -> Message means materializing all the bugtasks with those statuses, traversing from message -> bug means materializing all 177k messages... [08:22] perhaps if owner was on BugMessage [08:22] So for that data, I'm not sure if we can beat explain analyze [08:22] SELECT DISTINCT ON (BugTask.id) BugTask.* [08:22] FROM BugTask, Bug, BugMessage, Message [08:22] WHERE [08:22] Bug.id = BugTask.bug [08:22] AND Bug.id = BugMessage.bug [08:22] AND BugMessage.message = Message.id [08:22] AND BugTask.status in (10, 15, 20, 21, 22, 25) [08:22] AND Bug.duplicateof IS NULL [08:22] AND Bug.private = FALSE [08:22] AND BugMessage.index > 0 [08:22] AND Message.owner = 931129 [08:22] ORDER BY BugTask.id; [08:23] Yer - we would have to denormalize [08:23] Or we just store a table of commenters on bugs [08:23] distinct on is just more efficient, right ? [08:24] it's a little more efficient than DISTINCT BugTask.* [08:24] yeah [08:24] But probably not noticebly faster [08:25] stub: how many seconds is that ^ [08:26] I'm not actually sure why BugMessage is separate to Message [08:26] 8.7 seconds [08:26] hmm, I had a 6 second one [08:26] Yippie, build fixed! [08:26] Project devel build (430): FIXED in 6 hr 11 min: https://hudson.wedontsleep.org/job/devel/430/ [08:26] * Launchpad Patch Queue Manager: [r=flacoste][bug=716109] Update Launchpad to r202 of lazr-js, [08:26] which includes an update to YUI 3.3. [08:26] * Launchpad Patch Queue Manager: [r=jtv] [ui=none][bug=316694][incr] Take advantage of web_link when [08:26] changing the URL in the browser bar, [08:26] rather than hacking self_link into web_link. [08:26] * Launchpad Patch Queue Manager: [r=jtv][bug=705657] Fix mirror-prober.sh wrapper script to export the [08:26] correct config. (Fix bug 705657) [08:26] * Launchpad Patch Queue Manager: [r=allenap][bug=710603] The new direct subscription overlay allows [08:26] <_mup_> Bug #705657: cronscripts using production/launchpad-lazr.conf config file with REPORTIFSEEN prefix < https://launchpad.net/bugs/705657 > [08:26] subscribing, [08:27] unsubscribing and editing subscriptions and is available to the [08:27] BugMessage contains bug message specific data. Message contains data relevant to all messages (answer comments etc.) [08:27] ~malone-alpha team. [08:27] Spambot [08:27] http://pastebin.com/raw.php?i=UdFwWx33 [08:28] 10s now :-) But the estimated cost is lower, and I think that is what we need to go by here. [08:31] stub: the distinct on is 10 seconds? [08:31] stub: whats the http://pastebin.com/raw.php?i=UdFwWx33 one ? [08:32] They are all around the same range in seconds. The distinct on has a cost of 391000, your variant 325000. [08:32] ok [08:32] so I'll code up my variant next week [08:32] how should we go about assessing the right denormalisation (e.g. owner in bugmessage; fold bugmessage and message togeter; have a table listing commentors) [08:33] That last one has pretty much the same cost [08:34] Even with denormalizing, we are going to have similar issues - materializing 170k rows. Lots of messages there. [08:36] Doing a count of 'messages owner by 931... and index > 0' has a cost of 280k [08:36] stub: if we had (bugtask.id, commentator), and updated it when adding a bugtask ? [08:36] Bug,commentator makes more sense [08:37] stub: we filter on bugtask status, and we search and report rows in bugtask; thats why i thought bugtask would make more sense [08:37] -> what do I have wrong [08:38] That user has commented on 43k distinct bugs [08:38] stub: is that user lifeless? [08:38] People comment on bugs, not on bugtasks. [08:39] apport [08:39] actually not apport [08:39] !janitor [08:39] janitor [08:39] who I never wanted as an actual user. [08:40] apport has made the most comment [08:40] and seb128 the next least [08:40] what is ~janitor for? [08:40] I think expiry and stuff [08:41] its the launchpad system robot anyway [08:41] so, this query makes no sense for their outliers anyway. Are we tackling the wrong problem? [08:41] c/their/these [08:42] seb128 is an outlier :P [08:42] [yes, he is, I'm trolling] [08:42] so there is a bug saying that the default Person:+bugs view is useless because it includes these bugs [08:43] heh... so we can't just turn this off for robots (unless we just make it official and designate seb a robot) - he has almost as many messages in the system as the janitor. [08:44] I know :) [08:45] good morning [09:18] Guten morgen === jtv-afk is now known as jtv [09:56] jml: oh hai [09:57] lifeless: hello. I got your message. [09:57] great [09:57] since i'm still conscious, and thinking work; do you want to chat now? [09:57] lifeless: definitely would like to talk, but not now. [09:57] ok [10:07] good morning [10:19] morning Ursinha, up early again [10:20] bigjools, trying to make it a habit [10:38] lifeless: totally loving the render time data at the top of the page [10:49] henninge: fancy a small review? https://code.launchpad.net/~adeuring/launchpad/bug-702468/+merge/49205 [10:50] ma kuckn === matsubara-afk is now known as matsubara [11:02] I am getting this error when doing "make schema" http://paste.ubuntu.com/565319/ [11:02] Anybody can tell me what is wrong? [11:02] henninge: Upgrade lp-dev-deps. [11:02] hm [11:02] You don't have postgresql-8.4-debversion installed. [11:04] wgrant: I see, thanks. [11:04] Please send more Internets. Thailand has run out. [11:05] I'm rather surprised the scripts get that far when the debversion package isn't installed. === al-maisan is now known as almaisan-away [11:07] Why does it keep these packages back? Anything I might have done? http://paste.ubuntu.com/565321/ [11:07] * henninge realizes this is probably turning into Ubuntu support. [11:08] dist-upgrade instead. [11:08] upgrade will try to avoid installing new packages. [11:08] I only get this on this machine, though. My laptop upgraded fine (using update-manager). [11:08] Right, update-manager does more than just apt-get upgrade. [11:10] currently update-manager gives me the "Not all updates can be installed. Run a partial update to install as many updates as possible." message and offers to "Partial upgrade". Is that a dist-upgrade? === Ursinha-afk is now known as Ursinha [11:12] henninge: Hmm, that's not good. [11:12] What does apt-get dist-upgrade say? It sounds like it may want to remove stuff, which would be bad. [11:13] Or there is some version skew. [11:13] http://paste.ubuntu.com/565323/ [11:13] This looks about right, though. [11:13] That does. [11:13] Do it. [11:13] Doing ... [11:16] ... done. [11:17] Now it wants to reboot, of course. (kernel update) [11:17] But update-manager is quiet about partial updates. [11:17] wgrant: tanks a lot! [11:17] Great. [11:17] Hopefully make will be happy now. [11:20] jtv: does these queries look like they'll do the same thing? http://pastebin.ubuntu.com/565325/ [11:20] s/does/do/ [11:20] bigjools: looking… [11:22] bigjools: what's the %s in the COALESCE stand for? [11:22] jtv: False [11:23] bigjools: NULL OR TRUE equals TRUE, so no need for that I think. [11:23] jtv: see lib/lp/buildmaster/model/buildfarmjob.py -> getBuildsForBuilder() [11:24] for the storm code that generated that [11:25] jtv: null equals true in PG? [11:25] No! [11:25] :) [11:25] But unusually for binary operators, OR has the property that (NULL OR TRUE) evaluates as TRUE. [11:26] the second query is missing the coalesce [11:26] Rather than to NULL, as one might otherwise expect. [11:26] the second is Rob's attempt to optimise the first [11:26] I'm not convinced it's right [11:27] bigjools: I don't see the "TeamParticipation.person = 2" reflected in the new query. [11:27] Oh wait, found it [11:28] It's another % in the old query. [11:28] yeah [11:30] bigjools: but all the nastiness, complexity-wise, is in the question of which foreign keys may be null. [11:31] the nastiness is in the fact that not all build farm jobs will have a related package build [11:31] ie TTBJs [11:31] I get an impression that this won't show jobs for public archives that the user is not an owner of. [11:33] No, that's not right. [11:33] But the other way around? Will this respect privacy? [11:34] Say that Archive.private = TRUE and there's no matching TeamParticipation. [11:36] looks wrong [11:36] There'll be no matching (PackageBuild, Archive, TeamParticipation), since those are inner-joined together on the conditions for visibility to the given user. [11:36] Which _sounds_ like what we want, [11:36] except then that tuple is being outer-joined to BuildFarmJob. [11:36] it's making my head explode [11:36] Which means that if there is no (PackageBuild, Archive, TeamParticipation) that proves visibility to the user, [11:37] the outer join will just make up a (PackageBuild, Archive, TeamParticipation) of all nulls. [11:38] And the WHERE clause doesn't check for that, so AFAICT it'll happily return a BFJ for an archive that the user isn't supposed to see. [11:40] The important thing there is that with an outer join, a join condition is crucially different from a "where" condition in that the latter filters out tuples from the entire join, whereas the former may just result in a tuple of nulls where nothing matches. [11:40] the problem with the original query is that it's doing a seq scan on archive [11:40] Use Union. [11:40] right [11:41] This is the wide-shallow/narrow-deep pattern that we keep seeing with privacy. [11:41] I'll ditch the left joins [11:41] Just write out separate cases for public and private archives, then factor for identical parts that you can share. [11:42] What should happen if there's no PackageBuild? Should be shown anyway? [11:42] yes [11:42] there's no privacy outside of anything that has a packagebuild [11:42] So for public archives you get something like… [11:42] SELECT DISTINCT BuildFarmJob.* [11:43] LEFT JOIN PackageBuild ON PackageBuild.build_farm_job = BuildFarmJob.id [11:43] LEFT JOIN Archive ON Archive.id = PackageBuild.archive [11:44] WHERE Archive.private IS FALSE [11:44] UNION [11:44] SELECT DISTINCT BuildFarmJob.* [11:44] LEFT JOIN PackageBuild ON PackageBuild.build_farm_job = BuildFarmJob.id [11:45] LEFT JOIN Archive ON Archive.id = PackageBuild.archive [11:45] LEFT JOIN TeamParticipation ON TeamParticipation.team = Archive.owner [11:46] WHERE Archive.private IS TRUE AND TeamParticipation.person = %s; [11:46] If there are a lot of BFJs without PBs, you could go even further: [11:47] SELECT DISTINCT BuildFarmJob.* [11:47] LEFT JOIN PackageBuild ON PackageBuild.build_farm_job = BuildFarmJob.id [11:47] WHERE PackageBuild.id IS NULL [11:47] UNION [11:47] SELECT DISTINCT BuildFarmJob.* [11:48] FROM BuildFarmJob -- I've been forgetting these lines, haven't I? [11:48] JOIN PackageBuild ON PackageBuild.build_farm_job = BuildFarmJob.id [11:48] I think you need to paste [11:48] JOIN Archive ON Archive.id = PackageBuild.archive [11:48] OK, OK, I'll paste. :) [11:49] I can't follow this here [11:54] bigjools: https://pastebin.canonical.com/43093/ [11:55] Ahhh, there's the LIMIT/OFFSET too isn't it? [11:55] Need to work that in as well. [11:55] We can use that to solve the remaining optimization problems. Hang on. [11:56] jtv: don't worry about that, it's the batchnav [11:56] Doesn't mean we shouldn't worry about it. It's actually most likely the single biggest factor in this problem. [11:56] the problem is that a slow query is issued twice, once to count, the next time with the limit/offset [11:57] But part of the problem is that slicing happens all the way at the end, and the query planner currently has no way to push that down into the nodes for preliminary optimization. [11:57] Hence the seq scan on Archive. [11:57] we need ordering as well [11:58] Yes. [11:58] and, is the distinct honoured across unions? [11:58] or did you guarantee they don't intersect? [11:58] Yes, I required that "Archive.private = TRUE" in the last query so that makes this a partitioning. [11:58] No overlap. [12:04] jtv: the queries are also missing a BuildFarmJob.builder = XXX [12:04] ah [12:04] but I shall experiment and see what else I can do! [12:04] bigjools: I'm not done either :) [12:05] Morning, folks. [12:05] hi deryck [12:05] howdy deryck [12:09] jtv: explain analyze on your new query shows a seq scan on archive still [12:09] bigjools: not entirely surprising. [12:09] because there's no index on private [12:10] Well that's why I said index it. :-) [12:10] :) [12:13] bigjools: somewhat elaborate attempt here… https://pastebin.canonical.com/43094/ [12:14] jtv: why order each inner query and then again in the outer? [12:14] bigjools: to shortcut their execution when the limit is reached. [12:14] order and limit in fact [12:14] ok [12:14] I don't know how we can control that in the batchnav [12:14] The backend may be smart enough to do that for itself, but not sure. [12:15] one can try [12:15] I've seen 8.4 backends push the ORDER BY down into sub-plans, so maybe we can leave that out. [12:15] well I can't do it in our current infrastructure anyway [12:16] OK, I'll drop that bit. [12:16] BTW I've been assuming that by far most archives are public… is that correct? [12:17] yep [12:18] 3 seq scans in that query [12:18] buildfarmjob, which is ok [12:18] packagebuild [12:18] 2 on Archive? [12:18] and archive [12:18] Oh [12:18] the good news is that it seems to return the right number of rows :) [12:19] We could try the TDD approach: write the absolute minimum that satisfies our tests... SELECT %(expected_number_of_rows)d [12:19] Is this less elaborate version any worse? https://pastebin.canonical.com/43095/ [12:19] assuming the tests are good, yes [12:21] jtv: same sort of performane [12:21] performance [12:21] same seq scans [12:22] we definitely need an index on archive.private [12:22] it would help elsewhere [12:23] Or in this last version of the query, one on (private, owner) [12:23] yeah [12:25] bigjools: can you see which of the two parts the packagebuild query is coming from? [12:25] the seq scan? [12:26] Sorry, yes. [12:26] the second I think [12:27] there's a hash cond for archive.id above it [12:27] Could you paste me the plan? [12:27] so prob need an index on packagebuild.archive [12:28] which query? :) [12:28] they're all similar but ... [12:29] https://pastebin.canonical.com/43095/ [12:30] Quite possible that we need that index, yes, or on (archive, buildfarmjob) or vice versa. [12:30] There's a patch now for hypothetical indexes in postgres. That'd be cool to have. [12:30] http://pastebin.ubuntu.com/565348/ [12:31] yes, you mentioned that, it would be very useful [12:31] Doesn't create real indexes, but lets you say "EXPLAIN HYPOTHETICAL" and it'll explain your query as it would do it if the hypothetical indexes were real. [12:31] so you do CREATE INDEX HYPOTHETICAL ... ? [12:32] hmmm so packagebuild already has indexes on archive and BFJ [12:35] Yes, like that [12:35] A unified index may help. [12:35] So one on both columns at the same time. [12:35] is that hypothetical index stuff packaged? [12:35] No [12:35] It's a patch. [12:35] urgh [12:36] But there's a PostgreSQL eXtension Network coming. [12:36] so, I am seeing the original query run quicker than your new ones right now [12:36] heh [12:36] I suspect it's cause it only has one seq scan [12:36] The seq scan on archive doesn't look at all costly. [12:37] rows=23596 [12:39] bigjools: oh! And have you tried indexing on the sort order? [12:39] yeah that adds 2 seconds alone, good idea [12:39] Indexing slows it down? [12:39] removing the ORDER BY speeds it up by 2 seconds [12:40] I'll add the index on dogfood and see what happens === almaisan-away is now known as al-maisan [12:43] Another big cost seems to be the PackageBuild→BFJ join. [12:44] * jtv slowly figures out that bigjools wasn't being sarcastic with the "that adds 2 seconds, good idea" :) [12:44] heh [12:47] jtv: actually date_finished is already indexed [12:47] bigjools: but (date_finished, id)? [12:48] no, but how often will date_finished need a tie breaker to ID? [12:48] If the sort order is completely covered, that may open up new plans. [12:48] index on private makes bugger all difference [12:48] :( [12:49] Well, scanning Archive wasn't that costly. [12:49] no, I mean there's still a seq scan [12:49] why is it scanning :/ [12:50] A seq scan is the most efficient thing to do unless it's clear that only a surprisingly small percentage of the table will actually be needed. [12:50] Think disk seeks: you can read quite a lot of data in a seq scan in the same time that it might take to seek to the next table record that an index identifies as matching. [12:51] indeed [12:51] (Bearing in mind that the index is in key order, not tuple storage order, so likely to be very random access patterns) [12:52] That's also why we have bitmap scans. [12:54] bigjools: hang on, I just stumbled onto a major simplification [12:54] adding an index on (date_finished, id) makes no difference :/ === matsubara is now known as matsubara-brb [13:01] bigjools: boo [13:01] But here's another version. I'd guess it was slower, but knowing trumps guessing: https://pastebin.canonical.com/43096/ [13:04] so far, a lot slower .... [13:08] bigjools: that's fine; it's just a stepping stone to https://pastebin.canonical.com/43097/ [13:08] Oh, that needs a DISTINCT obviously [13:09] Like so: https://pastebin.canonical.com/43098/ [13:09] oooooo quick [13:09] Then presumably it's incorrect in some way, but still nice to see a bit of speed. :-) [13:09] nope, looks fine! [13:10] I suspect the UNION is a very effective optimization barrier. [13:10] returns correct number of rows [13:10] down to 2 seconds from 9 [13:10] This makes it possible for the backend to materialize "archives this user is an owner of," which just has to be fast because so few records will be involved. [13:10] it's quicksorting in memory now [13:10] was using external merge on disk last time [13:11] The joins can't have helped. [13:11] "Disk: 142912kB" - yes :) [13:11] The key to this is that the Archive seq scan is actually relatively cheap. [13:11] it is [13:11] actual time=0.011..21.494 [13:11] So I'm materializing one, as the first part of the union! [13:12] That's even cheaper than previously, for some reason. [13:12] bizarro [13:12] There was just no way to get around a seq scan on archive, unless we managed to cram enormous amounts of selectivity into the PBs first—and even then. [13:13] I suspect that selectivity just isn't there. [13:13] Which can't have helped the size of the ultimate sort buffers either, for what it's worth. [13:14] Also, the PackageBuild join was very slow, and an outer join didn't look very different from an inner one, so I hoisted it out of the subquery. [13:14] That way it gets done only once. [13:14] thanks jtv, I'll Stormify this and run the tests against it [13:14] now, I need food [13:14] OK, glad I could help. [13:16] jtv: you certainly did [13:16] :) [13:22] bo hoo, no reviewer [13:22] boo, even === matsubara-brb is now known as matsubara === lionel__ is now known as lionel === jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: jcsackett | https://code.launchpad.net/launchpad-project/+activereviews [13:32] gary_poster: looking for a review for https://code.launchpad.net/~gary/launchpad/bug713382/+merge/49145 ? [13:32] yes, jcsackett ! :-) [13:33] * jcsackett is looking at it now. [13:35] thank you [13:43] gary_poster: in bug.py, what's the reason for working with "temp_recipients" instead of on "recipients"? [13:43] nm, reread proposal. [13:43] :-) l [13:43] k [13:43] * jcsackett gets more coffee while reviewing. [13:43] m [13:43] n [14:00] henninge, we see you connect, but never hear you [14:06] gary: r=me. i've requested follow up from sinzui. let me know if that becomes a blocker and i'll hunt him or someone else down. [14:06] gary_poster ^ [14:07] I can review it now [14:07] ah, i missed that you had logged on, sinzui. :-) [14:13] jcsackett: many thanks. will be back in afternoon === Ursinha is now known as Ursinha-afk === Ursinha-afk is now known as Ursinha [14:27] jcsackett, thanks for your review. You'll see that all the items I've put up for review build toward the goal of merging translations using a job. [14:27] abentley: i am indeed noticing that trend. [14:28] If you have any questions, I'm happy to answer 'em here. [14:34] adeuring: Two things about your branch. [14:34] henninge: yes? [14:34] 1. I am pretty sure this is only supposed to work one way. [14:35] i.e. first "upstream" translation overwrites "Ubuntu" translation but not the other way round. [14:35] henninge: right, but this is guarded by share_with_other_side [14:36] this flag is always true for product and projectgroups [14:36] and must be expicitly enabled for SPs [14:36] so I think we don't need extra guards [14:37] hm [14:37] Where is the flag "always true"? [14:37] In the policy? [14:38] henninge: right [14:44] ok, I see what you mean but AFAICT this is not the what you want. [14:44] s/the// [14:44] adeuring: ^ [14:45] henninge: well, I think we want upstream translations propagated to ubuntu, and that's done by share_with_other_side == True for upstreams [14:46] Or am I missing something? [14:48] Yippie, build fixed! [14:48] Project db-devel build (355): FIXED in 6 hr 1 min: https://hudson.wedontsleep.org/job/db-devel/355/ [14:50] adeuring: no, they are only to propagate if the Ubuntu translation is either empty or identical to the upstream one. [14:51] adeuring: if they are different, they stay different. [14:52] henninge: well, the bug says "In the old model, providing the first upstream translation would have overwritten the Ubuntu translation, too. " [14:52] adeuring: yes, that is the exception to the rule I just described and which is missing from the code. [14:52] and which you implemented correctly. [14:53] But it does not work the other way round. [14:53] The exception, I mean. [14:53] henninge: ah, so you think thatg we should restrict this feature explicitly for the flow upstream -> ubuntu? [14:53] exactly! ;-) [14:54] even if sharing the other way round is enabled? [14:54] yes [14:54] The reasoning for this exception goes like this: [14:55] We import templates and translations from upstream, but there are no (or little ) translations yet. [14:55] Then Ubuntu translators go to work and start translating. [14:55] The upstream translators get going, too, and after a while we get updated upstream translations. [14:56] Without this exception, these upstream translations would not show up in Ubuntu. [14:56] We have been getting bad press about how "Ubuntu ignores upstream translations2 [14:56] ". [14:57] But really, Ubuntu translations should be fed back to upstream anyway, but that is not something we can solve by code. [14:57] right. But how likely is the situation that the other "sharing variant" does something wrong? I'd prefer to not make the code even more complex ;) [14:58] I mean, somebody must explicitly enable this sort of "sharing flow" [14:58] adeuring: It needs to retain the old behavior, though. It's documented and the translators are used to it. [14:59] What do you mean by "enable"? [14:59] well, ok, I'll add a sort of diode then [14:59] jtv: so I converted your SQL into STORM syntax. I think I might be about to faint that I got it right first time and the tests all pass. [14:59] adeuring: please do. [14:59] bigjools: congratulations! [14:59] henninge: and what is the other issue? [14:59] jtv: http://pastebin.ubuntu.com/565404/ [14:59] adeuring: secondly, I would expect a test or two for this specific behavior. [15:00] adeuring: you adapted some tests but I guess they just failed because the incumbent message happened to be empty. [15:01] henninge: hmmm, I think the test method names indicate that the incumbent message is supposed to be empty, right? [15:01] but nevenr mind, i can add another test ;) [15:01] adeuring: hm, you may be right [15:02] but then perhaps a test is missing for when it is not? [15:02] ok, i'll check it [15:02] adeuring: thanks. [15:04] adeuring: I fear you will run into some work with the test when you add the "diode" because I think it is designed highly symmetrical atm. [15:04] but it would be good to show the assymmteric behavior in the test. [15:05] deryck: do you know of any docs about how to use LP.client.Launchpad() to interact with the web service from JavaScript? [15:05] benji, on call, will respond just a minute [15:05] thanks [15:16] Project devel build (431): FAILURE in 5 hr 33 min: https://hudson.wedontsleep.org/job/devel/431/ [15:16] abentley: i'm looking at your last branch now. i'm not familiar with lazr configs--the purpose of your additions is just proper error capturing when the job runs? [15:17] jcsackett, no, the configuration glues it all together. Nothing could be run without it. [15:17] jcsackett, It specifies the database user to use, the JobSource to use, the error handling, etc. [15:18] abentley: ah, i see. i had only looked at the first two config changes. [15:18] jcsackett, lazr-js is a layered configuration system, so those first two overlay the last one. [15:18] abentley: dig. thanks. [15:29] benji, hi, back now. [15:29] hmmm, so lp_client docs basically> [15:30] benji, I don't think we have any actually. I've always looked at the client code itself, or followed examples in other js code. [15:30] sorry === matsubara is now known as matsubara-lunch [15:32] deryck: that's what I suspected; I'm writing the start of some docs that will hopefully fill the hole [15:32] ah nice [15:35] abentley: all branches are r=me; sinzui has already followed up on several. [15:35] really nice work. [15:35] jcsackett, thanks. [15:38] abentley or rockstar -- has the minus icon on bug pages to unlink a branch been ajax before now? Or it always went to a confirmation page? [15:38] * deryck is QAing new lazr-js and not sure [15:38] deryck, on bug pages? No idea. gmb did that work. [15:38] deryck, unsure. [15:39] ah ok. gmb ^^ ? [15:39] abentley: I think everything is cleared to land now [15:39] * gmb reads scrollback [15:39] sinzui, great, thanks. [15:39] deryck: I *think* it's always gone to a confirmation page. [15:40] Yet another we-only-did-half-the-work-before-moving-on. [15:40] gmb, ok, cool. thanks. [15:44] deryck: assuming you're concerned about a branch that recently hit, i can confirm that as of last monday unlinking went to a page, not ajax. [15:44] jcsackett, indeed that is my concern. Thanks! === al-maisan is now known as almaisan-away [15:52] heh, oops. Somewhere a link from qastaging pointed to lpnet. And now there is a dumb project on launchpad. [15:53] or a new team rather. [15:53] heh, or a project that thinks it's a team. gah! [15:54] sinzui, can I or you delete https://launchpad.net/deryck-test-qa-team/, or do we need an admin for that? [15:54] deryck: you cannot see the delete link? [15:55] deryck: oh sorry, that is a project. the cannot be deleted. you can rename it and deactivate it [15:56] sinzui, ok, thanks. I'll just deactivate it. Sorry about that. [15:59] maybe I hand entered a URL and didn't realize I forgot the qastaging bit. [16:01] sinzui, I want to create a Job every time a Packaging is created, to merge the translations between the sourcepackage and the productseries. === allenap changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: jcsackett, allenap | https://code.launchpad.net/launchpad-project/+activereviews [16:02] sinzui, Packaging doesn't have a constructor, but I'm thinking of adding one, and having it call notify. Make sense? [16:02] yes it does [16:02] sinzui, cool, thanks. === salgado is now known as salgado-lunch [16:04] sinzui, are Packagings ever modified, or just created and deleted? [16:05] abentley never modified [16:06] sinzui, great. [16:06] * sinzui wonders though if changing a ds packaging workflow calls delete? [16:06] * sinzui lookse [16:10] abentley: definitely not modified. We create, and occasionally delete. Packaging acts as a historical record [16:10] Good to know. [16:14] jcsackett: got an MP for you! Are you free? https://code.launchpad.net/~jtv/launchpad/bug-711077/+merge/49242 [16:14] jtv: in the middle of investigating a test failure. can look in half an hour? [16:14] jcsackett: if you like, though I could ask allenap first [16:15] jtv: if he's free now, sure. if not, i'll be happy to take a look at it soon. :-) [16:15] jtv: I'm up for a review. [16:15] great! [16:15] see mp ^^^^ === deryck is now known as deryck[lunch] === beuno is now known as beuno-lunch [16:30] bigjools: it seems the parallel a-f is on qastaging… want to walk me through the Q/A process? [16:30] jtv: I'm in the middle of something, happy to do so later [16:30] OK [16:31] jtv: passing tests were some sort of weird miracle earlier, they don't actually pass and my storm expression won't compile [16:31] I am starting to really detest storm syntax [16:32] bigjools: it can be annoying, yes. Anything there I can help with? [16:32] jtv: maybe, one sec [16:32] Debugging the things isn't fun either. [16:32] tell me about it [16:36] jtv: http://pastebin.ubuntu.com/565450/ [16:36] won't compile, it doesn't like the PackageBuild.archiveID or PackageBuild.archive [16:36] bigjools: have you tried PackageBuild.id.is_in(Select(…))? [16:36] jeez [16:36] ok [16:37] it would be PackageBuild.archive.is_in of course [16:38] AttributeError: 'Reference' object has no attribute 'is_in' [16:38] Grrrr [16:38] Try PackageBuild.archive_id.is_in or PackageBuild.archiveID.is_in [16:38] this ID vs reference stuff is seriously irritating [16:39] This stuff is way too hard. [16:39] Yes [16:39] that too [16:39] I suspect the _id will do it, I was using ID before [16:39] grrrr [16:39] yay it worked [16:39] \o/ [16:40] thanks jtv [16:40] np [16:40] you were blocking my lane, had to help :) [16:40] I think I knew about this, it's clearly been a while since I wrote storm [16:40] But you keep hoping someone will fix this stuff while you're away, don't you? [16:43] now that part is fixed, I get the problem somewhere else === salgado-lunch is now known as salgado [16:43] That's progress. [16:43] What is it now? [16:43] same compile error about a Reference - but no indication of which one [16:43] Ah yes that one [16:43] it's always fun. [16:44] FSVO [16:44] I do TLAs, not ETLAs [16:44] ha [16:44] OMG NFW BMX [16:44] BMX? WTF!? [16:45] Anyway, with the Reference one at least you know you're missing a _id or ID suffix on a reference attribute. It's not that you might have mis-spelled one. [16:45] (That comes later.) [16:46] yes [16:49] shockingly, my first guess found it [16:49] the gods are toying with you [16:55] so is dogfood [16:56] "…they play with us for their sport" [16:58] dogfood is doofgod backwards. [16:59] mawson should be aergia it's so bloody slow [17:01] aegria? [17:02] aergia [17:02] Ah, not familiar with her [17:02] darn, this query is bust [17:02] What now? [17:03] it's returning the wrong stuff - yet the tests pass [17:09] Project devel build (432): STILL FAILING in 1 hr 53 min: https://hudson.wedontsleep.org/job/devel/432/ [17:09] * Launchpad Patch Queue Manager: [r=thumper][no-qa] Mechanical move of [17:09] canonical.launchpad.windmill.testing to lp.testing.windmill. [17:09] * Launchpad Patch Queue Manager: [r=allenap][bug=528367] Added alt tag to remove icons. === beuno-lunch is now known as beuno [17:15] Project db-devel build (356): FAILURE in 2 hr 27 min: https://hudson.wedontsleep.org/job/db-devel/356/ [17:15] Launchpad Patch Queue Manager: [rs=buildbot-poller] automatic merge from stable. Revisions: 12357 [17:15] included. [17:16] jtv: hurray at last. [17:17] What was the problem? [17:17] the history page on dogfood finally loads for adare [17:17] jtv: pebkac [17:29] jtv: where's your branch that you need testing? [17:31] bigjools: lp:~jtv/launchpad/bug-181368-view [17:31] That's against devel though. [17:31] 'sok [17:31] (In fact it's already _in_ devel) [17:32] I'm going to pocket copy a package to create publications in lots of places (it has 6 arch binaries) [17:34] moin [17:35] bigjools: glad you like it [17:35] lifeless: saves a bit of ctrl-u/page-down [17:35] I described it to elliot as 'tastefully IN YOUR FACE' [17:36] jtv: ok, I've copied acl in maverick to maverick-proposed, with binaries. See https://dogfood.launchpad.net/ubuntu/+source/acl/+publishinghistory [17:36] * bigjools runs df publisher [17:36] bigjools: I'm not sure what to look out for though [17:36] jtv: I'll send you a log file when it's done. You'll see that source go from pending to published [17:38] jtv: I'm not going to finish your review today. Do you mind if I complete it in the morning? [17:38] allenap: then maybe I should ask jcsackett to give it a go after all. [17:38] allenap: or were you already partway through it? [17:38] jtv: also this publisher may not finish within the 20 minutes I have left today [17:39] bigjools: ah… and this is df innit? [17:39] jtv: yeah I limited it to just the suite we want, but it will still take some time [17:39] a full run takes *hours* :/ [17:39] jcsackett: I am partway through it, but not that far, so I'm happy for him to take it if he is too. I have been pulled away by non-work things. [17:39] Oh well, have to check up tomorrow then. [17:40] jtv: ah it finished [17:40] bigjools: so it seems to have [17:40] So how do we know it did something sensible? [17:40] lifeless: good morning. [17:41] jml: hi hi [17:41] jtv: you want to wait on allenap, or shall i take a look at it now? [17:41] lifeless: now or in the next hour would be a good time for a call. [17:41] jtv: I am checking the Packages files [17:41] ok, I'll ping you in a few minutes [17:41] jtv: how many does it kick off in parallel BTW? [17:41] jcsackett: I'd appreciate a look—I'm pretty much EOD, but I'm slightly ahead of allenap timezone-wise. [17:42] bigjools: one for each architecture (including "source") [17:42] jtv: okay, i'll take a look and ask any questions in the comments. [17:42] jtv: so 6 [17:42] bigjools: but that's the parallel run. This branch is a different change. [17:42] lifeless: ok. [17:42] it is? [17:42] bigjools: yes, circumventing an awkward view. === deryck[lunch] is now known as deryck [17:42] seemed quicker :) [17:43] Good! [17:43] bigjools: the parallel run is still waiting for the RT [17:44] jtv: only 4 of the binaries got published, I don't know if that's right or not [17:44] but the ones that did look ok [17:44] bigjools: hmmm for proper Q/A we'd have to know whether that was right. [17:44] yes [17:45] could be PaS [17:45] PhotoAcoustic Spectroscopy? [17:45] ah I should try an arch indep, that'll not matter [17:46] Package Arch Specific [17:46] it's a last-ditch override file that the publisher uses [17:47] jtv: while you're still here, i note in the diff you're adding some columns, but this isn't targeted to db-devel; i assume it's not introducting change to the db, but don't really understand why not. [17:47] jcsackett: adding columns!? [17:48] jcsackett: sounds as if something went wrong with the MP… just a mo' [17:48] jtv: several adds of IntCol (branch_id), but not replacing similar statements. i may be misinterpreting. [17:49] jcsackett: oh, those aren't columns—they're attributes that allow me to read the FK attributes directly [17:49] Pretty standard stuff. [17:49] jtv: ah. i figured it was something like that, but wasn't sure. and the use of "Col" in the declaration raised alarms. :-) [17:50] Seems I have to go help with a little accident here. [17:50] jtv: so a tentative ok, but please ask wgrant to double check since this really needs more eyes to verify it before we unleash on the distro [17:51] bigjools: ack === matsubara-lunch is now known as matsubara === jtv is now known as jtv-afk [17:54] jtv-afk: so meh, I forgot to notice that copy-package only copied some of the binaries, so it seems perfectly ok. [18:00] jml: ping [18:02] jml: skype me [18:03] deryck: question for you, why are BugMessage and Message separate tables? [18:04] lifeless, I literally have no idea. :-) I asked once before and didn't understand the response. I think.... [18:04] lifeless, it was written in a time when we imagined different kinds of messages, of which bug message was one type. [18:04] there are 640 shared messages [18:04] lifeless: k [18:04] that is, one message on several bugs. [18:05] hmmm, I don't know why that would be. [18:05] deryck: I think we could make a bunch of queries faster if we deep sixed the sharing and just copied the message when that happens [18:05] deryck: email control interface controlling many bugs? [18:05] deryck: in case its not obvious I mean a db migration too :) [18:05] lifeless, perhaps. but I thought you could only affect multiple bugtasks, not bugs, from email [18:06] lifeless, yes, I think moving all that across to copies, not shared messages, is best [18:12] what's the difference between using IStore and Store.of? they seem to result in the same thing, but i'm not certain. [18:19] jcsackett, so IStore is what we provide to help manage connecting to the db, instead of just straight import Store from Storm. [18:20] jcsackett, see lib/canonical/launchpad/doc/storm.txt [18:20] deryck: thanks. [18:20] jcsackett, actually, I meant to point you at: lib/canonical/launchpad/doc/db-policy.txt [18:20] well, either is probably ok ;) [18:20] * jcsackett changes which file he is opening... [18:22] jml: http://gems.github.com/list.html [18:26] today has been a very busy review day... === jcsackett_ is now known as jcsackett [19:09] sinzui: can i get you to look at something? [19:09] yes [19:09] i'm having trouble understanding the "why" of a bit of publisher code: https://pastebin.canonical.com/43132/ [19:10] it looks like it's trying to rebuild web service based requests, but i don't understand why--commenting out that code doesn't seem to hurt webservice functions. [19:10] (this is the actual culprit for the api.launchpad.net pillar aliases redirecting outside of the api) [19:12] jcsackett: I confess I do not understand it [19:12] jcsackett: btw [19:12] jcsackett: see my last comment on https://code.launchpad.net/~jtv/launchpad/bug-711077/+merge/49242 - thats something to bear in mind [19:12] my head may be filled with straw today. I can struggling to refactor what I think is a simple test [19:13] lifeless: good thought. [19:13] sinzui: yeah, after a morning filled with the largest review queue i've seen, i'm feeling pretty muddled. [19:13] (and since you follow up on them all, i'm not surprised you feel the same). [19:14] lifeless, any chance you know what's up in this bit? https://pastebin.canonical.com/43132/ [19:14] other than the gratuitious pass? [19:15] hm, you can ignore that--left over from when i commented out a bunch of code to see what changed. [19:15] * jcsackett edits paste. [19:15] what file is that in ? [19:15] jcsackett: have you been introduced to bzr's annotate functionality ? [19:16] lifeless: i've seen annotate used once, and heard much of it. i do not, as yet, know how to do anything useful with it. :-P [19:16] file is canonical.launchpad.webapp.publisher [19:16] it seems to rebuilding webservice requests (and in the process losing the api layer) [19:17] so the comment says: [19:17] - no rootsite [19:17] - handling shipit [19:18] blame says the outer if was done in rev 3564.1.43 [19:18] and the inner one in 7675.289.2 / 3 / 4 [19:19] bzr log -r 7675.289.2..7675.289.4 tells me that bug-376990 was at issue [19:19] 'canonical_url() needs to return browser URLs when generating XHTML representations' [19:19] yeah. [19:20] that seems to be the inverse of the problem leonardr pointed out to me. [19:20] so, it wants to get canonical urls for browser links some of the time [19:20] so, this only happens when canonical_url is called for pillars that we've gotten by alias in the webservice. i think i can just change a call up the pipe to pass the request along. [19:20] this also tells us how to test it [19:20] inline bug commenting [19:22] but we have a stale comment or something, because the block *above* the comment was changed by tim in rev 3758.1.1 [19:22] hi sinzui [19:22] jcsackett: so, what are the symptoms? [19:23] hi bac [19:23] lifeless: bug 715992 [19:23] <_mup_> Bug #715992: Pillar aliases redirect outside of webservice on webservice calls < https://launchpad.net/bugs/715992 > [19:24] wow, we are digging up some chestnuts [19:24] this should be critical otherwise its a priority inversion [19:25] this came up because it needs to be whacked to deal with bug 623099 [19:25] <_mup_> Bug #623099: AttributeError filing a bug using the API < https://launchpad.net/bugs/623099 > [19:25] yes [19:25] agreed; I'm just saying bump it to critical [19:25] ah, dig. [19:25] done. [19:25] because its otherwise holding a critical hostage, which wouldn't make sense [19:26] so [19:26] looks to me like you need to pass a request in [19:26] ' If there is no request available, the protocol, host and port are taken [19:26] from the root_url given in launchpad.conf. [19:26] ' [19:27] as leonardr says [19:27] 'Possibly there's a method that needs to have the current request passed into it.' [19:27] jcsackett: whats the backtrace leading into this call of canonical_url ? [19:28] yeah, i think i've found it; i just hit this block of code and was wondering what on earth it was doing. [19:28] bit of a rabbit hole, really. [19:28] its guessing at the url the client used [19:28] and the default is to guess 'web browser request on main site' [19:28] worth enhancing the comment [19:29] and adding a :param request: up in the docstring (subsuming the 'if there is no request available' prose.) [19:29] ? [19:31] lifeless: yeah, i'm cleaning up the comments. [19:34] lifeless: thanks. [19:53] deryck: I'd love any feedback or additions you might have to my first draft of "how to access webservice APIs via JavaScript": https://dev.launchpad.net/yellow/JavaScriptAPIAccessDraft [19:54] benji, ok, taking a look now [19:57] any chance i can get a review on https://code.launchpad.net/~jcsackett/launchpad/api-pillar-redirects-715992/+merge/49279? [19:58] benji, yeah, it's good. Do you have plans to fill in more about PATCHPlugin? [19:58] deryck: I hope to. I have to figure out how to use it first. :) [19:59] benji, you can look at how the inline editor widgets and commenting use it. [19:59] My plan is to put in everything I know so people can get a better start. I spent way too much time figuring out what little I know thus far. [19:59] cool, thanks for the pointer [19:59] benji, I can add to this too. But I'll need pings to remind me about it. [20:00] you can count on me [20:00] benji, our use of the js client as evolved over time, and we're probably named_post and named_get heavy, when we should favor patch with patch_field set true. [20:01] interesting [20:01] and then setting accept to xhtml, we get back the html for the part we patched. instead of the full resource. [20:02] well, unless we need the entire thing. it depends on use, I guess. but it seems more common to do attribute patching. [20:02] can the client understand xhtml? why does the server not return partial JSON? [20:03] benji, well xhtml can be created into a node easily in YUI. it's just a string, I think, not try xhtml. [20:03] s/try/true/ [20:03] so... var stuff = result_of_my_patch(); var my_html = Y.Node.create(stuff); [20:04] hmm, I don't understand. Is that for when we only patch one attribute at a time? [20:05] benji, yes. which is actually common. And the attribute will be a FK to some other object on the backend. [20:05] ah, ok that makes sense [20:05] that pseudo code might not be exactly how we do it either. Just going off sketch from memory. [20:07] benji: js replies on the the DOM's engine to construct elements You can create xhtml or html using DOM nodes, but what is rendered could be the other :( I have also see cases where the code tries to place a list in a paragraph, but what is rendered is a paragraph, a list, and another paragraph [20:07] s/replies/relies/ [20:08] interesting [20:13] morning [20:18] jcsackett, I have a sequel for you: https://code.launchpad.net/~abentley/launchpad/job-on-new-packaging/+merge/49285 [20:27] * jcsackett looks [20:42] deryck: ping [20:42] deryck: do you know most about windmill tests? [20:42] thumper, I guess I'm your only help where windmill is concerned ;) [20:42] I have a question about lines like: start_url = (windmill.settings['TEST_URL'] + 'bugs/%d' % bug.id) [20:43] windmill.settings['TEST_URL'] is what? [20:43] and why? [20:43] and why does only the code windmill tests use it? [20:43] and lib/lp/testing/windmill/lpuser.py:35 === almaisan-away is now known as al-maisan [20:44] thumper, I actually don't know what that is. [20:44] deryck: I think it may have to do with that wallyworld did, to do with parallelizing the test suite [20:44] thumper, only code tests made use of it, and I'm not sure why. [20:44] and not using hard coded ports [20:45] yeah, that would be my guess. but that's not the way he originally mentioned about not hard coding ports. [20:45] deryck: I have a day of doing windmill today with three branches needing some form of windmill testing [20:45] deryck: I was thinking of a base class helper method [20:45] deryck: that uses canonical_url(obj, force_local=True) [20:45] deryck: and appending that to the base test url [20:46] deryck: there are too many uses of this already [20:46] thumper, why do you need windmill tests? What are you wanting to test? [20:46] deryck: mumble? [20:46] sure [20:46] is there a python-launchpadlib that can run without compability problems on 8.04? [20:46] SWAT ^ [20:50] abentley: r=me. [20:50] jcsackett, thanks. === al-maisan is now known as almaisan-away === matsubara is now known as matsubara-afk [21:03] wallyworld, windmill.settings['TEST_URL'] [21:05] wallyworld, thumper -- client.open(url='%s/bugs/1' % BugsWindmillLayer.base_url) [21:05] <_mup_> Bug #1: Microsoft has a majority market share later on, everyone [21:25] thumper: http://people.canonical.com/~ianb/request-build-error.png [21:26] lifeless: wgrant ^^ [21:26] Ronnie: how far up are you wanting me to look ? [21:27] a few lines: (21:46:52) Ronnie: is there a python-launchpadlib that can run without compability problems on 8.04? [21:27] the one we ship in 8.0.4 ? [21:27] blah, 8.04 [21:27] checking rmadison [21:28] no, nothing before karmic [21:28] dapper is EOL in a few months [21:28] so no plans to chane this [21:28] Ronnie: why? [21:29] the ubuntu-nl server still runs on 8.04. we want in one case to communicate with launchpad (creating bug reports) [21:29] if thats in the canonical datacentre, they are migrating them all to lucid [21:29] just tee that up, it should be queued already [21:30] lifeless: its an own managed server [21:30] ah [21:31] hoe flexible are loco's if we use canonical data centre> [21:31] can we deploy our apps ourself, are there limitations, whats the connection speed in europe etc [21:32] theres a range of configs, really up to IS what and where [21:32] in terms of capabilities and performance, pretty darn good; the dc is in London [21:32] that said [21:32] uhm, you should be able to get launchpadlib on 8.04, it will just require some work [21:34] lifeless: currently were thinking about staticcaly linking [21:35] Ronnie: well, its pytho n:) [21:36] you can probably backport the karmic set of packages fairly easily [21:36] so, is it known that the page to reqeust a merge proposal is slightly broken? [21:37] the "Other" option shows up as: "Other:'>" [21:37] which I assume is a broken template [21:37] beuno: yeah, I saw that today too... file a bug? [21:37] there is a bug open [21:37] its critical [21:37] lifeless: again about the hosting on canonical servers. how would such thing look like. if we have problems can we fix it ourself or do we have to ask canonical-isd, and what are there response times? [21:37] sinzui is looking at it [21:38] cool [21:38] Ronnie: I'm a poor source of info for this; #canonical-sysadmins is I think the channel for contact === salgado is now known as salgado-afk [21:39] lifeless: spel error on channel? [21:39] #canonical-sysadmin [21:39] No s. [21:40] thx wgrant [21:43] beuno: the fix is waiting for the rollout [21:47] abentley: I had a few remarks about your last branch. Maybe you have good reason to not do as I expected [21:47] sinzui, no, mostly copying bad ideas from existing code. [21:52] i'm seeing an odd typo on the merge proposal page. https://bugs.launchpad.net/launchpad/+bug/716713 [21:52] <_mup_> Bug #716713: typo in propose page < https://launchpad.net/bugs/716713 > [21:52] Specifically, the "Other: "line is rendered as "Other: '>" [21:52] Looking at the HTML it looks like invalid html [21:52] sinzui: whats the bug number, so we can dup jams bug :) [21:52] ah, beuno, you beat me to it [21:54] lifeless: beuno jam: bug 714527 address the issue on the recipe and Mp pages [21:54] <_mup_> Bug #714527: owned widget structured strings render as bits of quoted html < https://launchpad.net/bugs/714527 > [22:01] sinzui, thanks for your review. I've pushed changes. Diff still updating, though. [22:02] abentley: I approved it. [22:03] sinzui, I saw, I just wanted to close the loop. === jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews [22:33] wgrant: mumble? [22:52] sinzui: Can I get you to review the colour unification stuff I've done? Not now as it sounds like you've had a long day, but when you have a chance? [22:52] huwshimi: I sure can [22:52] sinzui: Thanks heaps. [22:52] I think I have done part of it already since I looked at it the other day [22:54] sinzui: OK great. Would be good if you could have a quick look at the css if you haven't already. [22:54] just putting together the mp [22:59] sinzui: For whenever you can get to it: https://code.launchpad.net/~huwshimi/launchpad/unified-colours-710446/+merge/49316 [22:59] thanks. I will wait for the diff to generate [23:00] * thumper is hacking the windmill base test case class [23:02] interesting.... windmill hanging on start at http://code.launchpad.dev:8085/windmill-serv/start.html [23:03] sometimes only... [23:06] * thumper wanders into the kitchen in search of food [23:06] sinzui: There's 60 or 70 lines removed from style.css which is nice [23:09] Yippie, build fixed! [23:09] Project db-devel build (357): FIXED in 5 hr 43 min: https://hudson.wedontsleep.org/job/db-devel/357/ [23:11] wgrant: the linkification of post-filing messages is nice [23:11] wgrant: I'm really glad you jumped on that [23:37] sinzui: I removed that line. All ok? [23:37] yes [23:38] sinzui: Thanks :)