/srv/irclogs.ubuntu.com/2009/08/13/#launchpad-dev.txt

* rockstar goes to search for dinner00:09
wgrantThe project owner has an implicit subscription to all of the project's bugs unless there is a bug supervisor set. The bug supervisor can unsubscribe from its subscription, but the owner cannot. Is this intentional?00:12
wgrantSo, basically, you must have a bug supervisor set or the owning team gets spammed.00:12
wgrantI wonder if it's a holdover from when bug supervisors couldn't unsubscribe either.00:13
maxbTotal: 23204 tests, 8 failures, 7 errors in 236 minutes 9.812 seconds.00:28
maxbkarmic, python2.400:28
maxbdevel r909800:28
wgrantmaxb: pastebin beckons.00:28
* kfogel is away: Smithwicks w/ Omar00:28
wgrantI must set LP up in a karmic chroot...00:28
* thumper wanders up the road for a coffee00:43
maxbHmm, so several appear to be because I was holding out making sqlite available to python 2.4 until I verified what wanted it00:46
maxbcanonical/lazr/doc/timeout.txt failure... http://paste.ubuntu.com/252217/ - mean anything to anyone?00:48
maxbtest_realiseUpload_for_delayed_copies - http://paste.ubuntu.com/252218/ - DONE != ACCEPTED assertion failure00:49
wgrantmaxb: There were no errors before that last one?00:50
maxbno00:50
maxband then lastly soyuz/doc/gina.txt and soyuz/doc/package-diff.txt are very unhappy00:51
wgrantgina.txt gets unhappy sometimes.00:51
wgrantubuntu-meta sometimes fails to unpack on Karmic.00:52
wgrantEven though it always works when I do it manually.00:52
wgrantI might look further into that today, if it's that close to working on 2.4 Karmic.00:52
maxbyup, confirmed, with the appropriate additional sqlite packages, down to just the 4 mentioned failures01:03
maxbdocumented on LaunchpadOnKarmic01:27
wgrantmaxb: I can only reproduce the package-diff.txt and gina.txt failures.02:36
wgrantThe package-diff.txt one was an easy fix (debdiff in < Karmic is buggy, and doesn't obey the manpage wrt. exit codes; Karmic's does)02:37
wgrantgina.txt I'm looking at now...02:37
wgrantBut it's non-deterministic :(02:37
* wgrant kicks gina a few times for being non-Zopey, non-Stormy, and non-deterministic.02:53
cprovwgrant: on the bright side it is still importing debian 2x day with no complaints, has its merits.03:12
* cprov has to setup an karmic chroot, it's about time.03:13
wgrantcprov: Indeed. It works mostly.03:14
wgrantBut has strange strange strange issues in karmic.03:14
cprovmaxb: on the test_realiseUpload_for_delayed_copies failure, print the 'logger.buffer' it might contain a hint about what's going worng.03:15
cprovwgrant: paste me the complete failure if you can.03:16
wgrantcprov: It's an error 29 (ESPIPE!?) running dpkg-source on a variable number of packages.03:16
cprovwgrant: did anything sensible changed in karmic dpkg ?03:17
wgrantcprov: I'm trying to work out exactly what it does so I can try to reproduce.03:17
wgrantSince the command it says errored works fine.03:17
wgrantSo it might be something in Python.03:17
cprovwgrant: `dpkg -sn` ?? there is no -n switch03:21
wgrantcprov: Maybe not, but it works.03:22
wgrantJust sometimes not when gina does it.03:22
wgrantThe number of failures differs each time.03:23
wgrantSometimes it succeeds.03:23
wgrantcprov: Ah, -sn is one option. Of course.03:23
* wgrant waits for slow slow uni proxies.03:24
cprovwgrant: yes, -sn exists, my bad03:25
* cprov goes to bed.03:27
wgrantNight cprov.03:28
kfogelbarry: are you actually on right now?03:33
wgrant(I noticed that it actually means the exit code was 29, not that an error with errno 29 occurred)03:37
kfogelthumper: thanks for the review.  I'm getting 'bzr: ERROR: unknown command "pqm-submit" '... isn't pqm-submit part of the launchpad plugin?03:38
thumperkfogel: no, it is bzr-pqm03:50
kfogelthumper: ah03:50
kfogelthumper: so I'm confused because it's *always* Just Worked from this box, and now suddenly it doesn't.  AFAIK I didn't do anything to change things... well, whatever.  I'll go look for the plugin.03:51
thumperkfogel: which bzr?03:52
kfogelthumper: trunk, recent03:52
kfogel/usr/local/bin/bzr03:52
kfogelsymlink to my src03:52
kfogelthumper: submitted03:53
thumperspm: ping04:04
spmthumper: heyo04:05
thumperspm: can I get logs for the scripts bmp creation jobs04:06
thumperspm: (both of them)04:06
thumperspm: logs synced that is04:06
thumperspm: I should see why I didn't get an email from karl04:06
spmkk04:07
spmthumper: bzrsyncd is done; doing codehost atm (can't recall offhand which one that stuff runs on)04:09
thumperspm: me neither04:10
thumperI'd love to rename the jobs so I can get it right04:10
spmignorance shared is.... hmmm. ignorance squared?04:10
spmthumper: mpcreationjobs is bzrsyncd, fwiw04:12
wgrantcprov, maxb: The gina.txt failure is a Karmic dpkg bug, which is probably maybe sort of tracked in bug #399938.04:15
mupBug #399938: unpacking the upstream tarball not working <apport-bug> <i386> <bzr-builddeb (Ubuntu):Fix Released> <https://launchpad.net/bugs/399938>04:15
wgrantHrm, the explanation in the bug was not clear.04:25
wgrantIt looks like it might actually be that the fixed dpkg bug has the same symptoms that are caused by Python ignoring SIGPIPE.04:25
* wgrant pokes harder.04:25
wgrantThere we go.04:32
wgrantFixed.04:32
thumperspm: still cherrypicking?05:09
spmthumper: ha. 4 hour test run? yes. :-)05:14
thumper:(05:14
thumperspm: plz get faster 'puter05:14
spmthumper: pls write faster tests05:14
spmlast word. I win. :-P05:15
wgrantI can haz ec2test and review?05:15
wgrantlp:~wgrant/launchpad/karmic-2.4-fixes05:15
wgrantI've only tested on karmic and jaunty, not hardy.05:15
thumperspm: arg05:22
spmthumper: hopefully should be finishing sometime in the next hour, hour & ½.05:27
thumperok05:27
jtvthumper: while you wait, can you help me figure out a problem I'm looking at?05:57
thumperjtv: me waiting?  hah, but sure - hit me05:58
jtvthat thumper... last time he was in a slapping mood, now he's in a be-slapped mood05:58
* jtv hits thumper05:58
jtv*any-way*...05:58
jtvMy script commits translations snapshots to a bzr branch.  Works fine generally.05:59
thumperI feel a but coming along05:59
jtvBut there's a project that exports its translations back to the same branch it also imports translations from.05:59
jtvThat _should_ work.  The imports are based on branch jobs when the branch changes, the exports are done from a cron job.06:00
jtvIf the exporter sees that there's an unfinished translations import job on the branch, it backs off from committing to it.06:00
jtvBut I notice that for several days in a row now, every daily run sees this situation and backs off.06:01
thumperok...06:01
jtvOn staging there are such jobs, except they're all completed.06:02
thumperhow do you determine an unfinished translation import job06:02
jtvExactly.06:02
thumperah06:02
thumperdon't we have a start time?06:02
jtvWell I look at type and status.06:02
thumperOK, the way I see it you need two things:06:02
thumperfirstly06:02
lifelessthing06:03
lifeless...06:03
lifelessprofit!06:03
jtvlifeless: thanks, that's very helpful06:03
jtvcan we skip straight to the profit?06:03
thumperunfinished should mean pending, that has started but not yet finished :)06:03
lifelessjtv: I aim to please.06:03
thumperrather than just pending in the future some time06:03
lifelessbetter than aiming at peas.06:03
jtvcringe06:03
thumperand the import job would need to know not to run while an export is running on the same branch06:03
thumperlifeless: better than aiming your pee06:04
jtvno, actually aiming your pee is quite useful.06:04
lifelessthumper: if you don't aim you will dis please06:04
thumperjtv: as long as it isn't at you or me06:04
jtvthumper: actually it's the other way around—I want the import job to complete peacefully and the export job to back off.06:04
thumperit all depends on the target06:04
jtvI'm just trying to figure out why the exporter hits that situation quite so often.06:05
thumperjtv: yes, but if there is an export job running, you don't want the import job to start right?06:05
thumperjtv: my guess is that when an import job has finished, it creates another at some future time06:05
jtvthumper: the exporter has the branch locked anyway.06:05
jtvI don't suppose the stuff I'm doing with the branch causes RosettaUploadJobs to be created even before I commit?06:06
jtvAFAICT the import job does not create a new instance of itself: it's created by the branch scanner when a change is detected.  Besides, on staging I only see completed ones.06:07
thumperhmm..06:07
thumperI'd say get a losa to run queries for you06:08
jtvit _might_ be an unlucky interference between cron jobs I guess06:08
thumperto see if production is like you think it should be06:08
thumperjtv: possibly06:08
thumperbut I don't think so06:08
jtvowww, do I _have_ to talk to spm again?  :-)06:08
thumperit seems wrong06:08
thumperjtv: no, you could wait for herb :)06:08
jtvspm it is then :)06:09
jtvFor the "I'm unwittingly creating the job myself" hypothesis I could ask a losa to give the export job an extra run...06:10
jtvthumper: ahhhhh I think I may have it06:13
jtvJob.id == BranchJob.id06:13
jtvBad join condition.06:13
thumperhaha06:14
thumperyes06:14
jtvShould be Job.id == BranchJob.job06:14
thumperright06:14
thumperjtv: I CC'ed you on my email about query performance06:19
thumperjtv: I thought you might have some ideas06:19
thumperjtv: it went to the launchpad-dev list anyway06:19
jtvthumper: I see it06:19
jtvand I owe you a look.  :)06:20
thumperjtv: I _think_ we may be able to speed up the queries with an index06:20
thumperjtv: I'm hoping06:20
thumperor we may need some other magic06:20
jtvthumper: I'll have a look.  Most well-documented "query performance post" I've seen so far.06:21
thumperjtv: thanks06:22
jtvthumper: this is a common pattern... IIRC BjornT had a similar optimization problem dealing with "either non-private or visible to this user."06:22
jtvMaybe we should just have a team for Everyone.  :)06:22
jtvthumper: you're always checking for ownership, even though you've got a done deal in the common case where the branch is public.06:23
jtvthumper: (assuming the "private = %s" means "private is false")06:24
thumperjtv: it is06:24
thumperin the subscription one we only look for private06:24
jtvyou'll want to limit the ownership check to private branches06:24
thumperdo you think that it makes that much of a difference?06:25
thumperperhaps it does06:25
jtvthumper: it's hard to say; it's right at the other end of the join so maybe not.  But even then we'd hate ourselves for not checking.06:26
thumperjtv: it could be huge06:26
thumperjtv: given that we have 150k branchesnow06:26
jtvchild's play06:26
thumperit is a one line fix06:26
jtvwanna play with the TranslationMessage table?06:26
thumperthat should break no tests06:26
thumperjtv: looked at the branch revision table?06:27
thumperI think we win06:27
jtvah now, branch _revision_ table is big I know06:27
jtvhow big is it?06:27
thumperbiggish06:27
jtvhow biggish?06:27
spmjtv: heyo06:27
thumperspm: want to compare some timings for me?06:28
jtvhi spm!  nm, thumper sat by the couch as I wrestled through my emotional problems and resolved the issue06:28
spmjtv: errr. oki. I'll try not to feel left out, unwanted etc06:28
jtv(It was something to do with my mother not loving me, or a bad join condition)06:29
spmthumper: is that a trick question?06:29
thumperspm: what is my person id?06:29
thumperspm: I'm assembling the query now06:29
jtvthumper: you can also move the selection of all public branches out of the union: "Branch.private = FALSE OR Branch.id IN (... UNION ...)"06:30
spmthumper: 43385206:30
thumperspm: http://pastebin.ubuntu.com/252311/06:34
thumperspm: there are three queries there06:34
spmthumper: oki06:35
thumperspm: much appreciated06:35
spmthumper: in return. does this look spurious or serious to you? https://pastebin.canonical.com/21094/06:36
jtvspm, thumper: also, try this with a partial index on private branches only.  Indexing just the "private" attribute may not be quite as effective.06:36
spmcan't find any mention of that failed test beyond that.06:37
thumperspm: spurious I think06:37
thumperjtv, spm: the partial index thing is probably best done on staging :)06:37
thumperjtv: can you give spm an example of what you mean?06:37
spmI wasn't planning on doing it on prod! :-)06:37
thumperspm: I wouldn't think you would06:37
thumperspm: are you running my three on staging?06:38
thumperspm: staging is fine I think06:38
spmthumper: prod, but can shift?06:38
thumperspm: I just need comparitive results06:38
thumperspm: prod is fine too06:38
thumperI think the first one takes around 1200ms06:38
spmthumper: coolio. do you need the output or, sure you've got it right?06:38
thumperI don't need the output, just the times06:38
* spm ndos06:39
thumperthe output should be the same for all three06:39
thumperif it isn't06:39
thumpersomething is wrong06:39
thumperspm: it is just a count of active reviews I'm doing06:39
thumperspm: it won't be very big06:39
jtvI'd try "CREATE UNIQUE INDEX foo ON Branch(id) WHERE private IS TRUE" and "CREATE INDEX bar ON Branch(owner) WHERE private IS TRUE" and then run the query with the "private IS TRUE" condition on the ownership check.06:40
jtvOne or the other might catch something.06:40
* thumper waits 06:40
spmthumper: 1st was 2.2secs, then 1.7 to 1.9secs on later runs. result is count = 1.06:40
thumperI'm EODing after this06:40
thumperspm: ok06:41
thumperspm: that matches what I'm seeing in the oopses06:41
spmthumper: 2nd was 2.9 for first; then consistant 1.8's; again count == 106:42
thumperok, not much in that one then06:42
* thumper taps his foot06:42
spmthumper: wow. I think you want to use #3. 0.127secs from the get go06:43
thumperw00t06:43
thumperjtv: you tha man06:43
spm10x improvement is impressive06:43
thumperI think I can submit that fix easily06:43
spm+1 from me for it at any rate :-)06:43
jtvI haven't exactly followed the channel...  what was it that did it?06:43
* thumper branches now06:44
thumperjtv: the third query in the pastebin06:44
thumperjtv: moving Branch.private = False out of the union06:44
jtvthumper: cool, I think I've repaid you for the day then.  :)06:45
thumperjtv: that you have, thanks06:45
jtvmy pleasure, sir06:45
thumperok...06:46
thumpernot as trivial as I hoped06:47
jtvuh-oh06:47
thumperbut I think I can get it done tomorrow06:47
thumperit is more how we handle visibility checks in the branch collection code06:47
thumperI was hoping for a two line fix06:47
thumperand it will be longer06:47
thumperbut still relatively simple I think06:47
thumperbut not for today06:47
thumperI'll look at it tomorrow06:47
* thumper acts like a tree06:48
jtvOh, I think I've looked at that code once and the function calls to build up the query go pretty deep.06:48
jtvWhat do trees do?  Grow?  Metabolize carbon dioxide?  Get hugged by hippies?06:48
spmtrees? cut down and turned into beautiful furniture? a la my lovely jarrah desk I built that I'm sitting at atm06:49
jtvspm: we don't want thumper to be get cut down and turned into furniture, now do we?06:50
jtv(*Beautiful* furniture could be hard, but let's generalize the principle)06:51
spmah. that'll teach me for reading approximately ONE line of scrollback and responding. Doh.06:51
jtvNo worries.  All makes for good entertainment.06:51
spmLOSA Duties: operational sysadmin; general entertainer; etc06:52
jtvspm: dance for us!06:54
jtv(At this point I bet ara is wondering...)06:54
spmjtv: I dance... on  the inside06:55
jtvooh, nice comeback06:56
jtvso nice in fact that I'll refrain from making jokes like "inside of what?"06:56
spmheh06:57
maxb+  * lib/lp/soyuz/tests/../doc/gina.txt - tar broke dpkg. dpkg was fixed. python breaks the fix. wgrant has a fix.07:27
maxbwhoa, sounds hellish :-)07:27
wgrantmaxb: Nasty signal stuff.07:28
wgrantBut see lp:~wgrant/launchpad/karmic-2.4-fixes07:28
wgrantAll fixed.07:28
wgrantSoyuz tests pass now.07:28
wgrantThat one took a little while to track down.07:29
* maxb wonders how on earth we get valid archives wherein tar doesn't consume all the compressed data07:40
wgrantWho knows.07:40
wgrantBut it happens...07:41
wgrantOften.07:41
wgrantmaxb: What arch did you run the tests on?08:03
wgrantBecause I have no idea where those other failures came from...08:04
maxbamd6408:07
wgrantSeems unlikely, but that's one difference.08:08
maxbI'd try again, but I'm in the middle of a py2.5 testrun08:11
maxbI need to make myself a separate lp-sourcedeps for 2.4 vs 2.508:11
wgrantI need a separate sourcedeps for amd64 vs. i386 :(08:11
wgrantThat's why all my instances are i386; my first one was.08:12
maxbI'm thinking symlink download-cache, bzr checkout --lightweight sourcecode/*08:12
wgrantPossibly.08:13
* wgrant heads home.08:15
adeuringgood morning08:28
mrevellMorning09:18
wgrantNow that Bugs people are around:10:21
wgrantThe project owner has an implicit subscription to all of the project's bugs unless there is a bug supervisor set. The bug supervisor can unsubscribe from its subscription, but the owner cannot. Is this intentional? I wonder if it's a holdover from when bug supervisors couldn't unsubscribe either.10:22
wgrantThis confused when I moved a project to Launchpad last week, and couldn't unsubscribe the team.10:22
intellectronicawgrant: i'm not sure i understand. what were you expecting to be able to do?10:33
wgrantintellectronica: The project owner always has an implicit subscription to a project's bugs if the project has no bug supervisor.10:35
wgrantintellectronica: This subscription is impossible to remove.10:35
wgrantIt is the only one which is irremovable.10:36
wgrantThis seems strange.10:36
intellectronicawgrant: well, it's not really a subscription. it's an "also notified"10:37
intellectronicathe point is that _someone_ needs to be notified, and project owner or bug supervisor make sense10:37
wgrantintellectronica: It's a forced structural subscription.10:37
wgrantintellectronica: The bug supervisor can easily unsubscribe.10:37
wgrantAs the bug supervisor is no longer implicitly notified; it just normally has a structural subscription.10:37
wgrantThe project owner *is* implicitly notified, without a subscription.10:38
wgrantThis is entirely unobvious.10:38
wgrant'Subscribe to bugmail' is the obvious place to look, but doesn't help.10:38
wgrantSo I checked the code, and indeed it is hardcoded.10:38
wgrant(lp.bugs.model.bug:620)10:38
wgrantintellectronica: So, it is quite easy to get into a condition where nobody gets email, but also quite hard for users to work out how to stop email.10:44
wgrantAnd other bugtrackers don't by default email anybody about all new bugs.10:44
wgrantSo it wouldn't be entirely unthinkable to not have the owner notified by default.10:44
intellectronicadon't know, maybe you're right and we don't have to have any subscribers by default. haven't though about this enough. i still think that it would be bad if a project had noone notified10:46
wgrantintellectronica: It sounds like a question that should be asked during the guided project creation.10:49
wgrantThat can then create a structural subscription if the creator desires.10:50
intellectronicawgrant: i'm not sure it's worth asking, but it might make sense to simply create a structural subscription (which can then be removed)10:51
intellectronicathere's one serious implication of having no subscriptions, though, and that's private bugs10:51
wgrantintellectronica: That does get slightly confusing when the ownership changes.10:51
wgrantintellectronica: Private bugs do not care about indirect subscriptions, do they?10:51
wgrantDon't they just subscribe either the security contact or the bug supervisor?10:51
wgrantOr owner if either doesn't exist?10:51
intellectronicathey do, in the sense that indirect subscriptions are converted to direct ones when a bug becomes private10:52
wgrantRight.10:52
wgrantBut I'm not sure why that's of concern.10:52
wgrantOr do you mean that you'd expect the project team to be able to see them always?10:52
wgrantI think bug privacy implications need to be much more obvious, which would alleviate this.10:53
intellectronicabecause you can end up with orphan bugs. bugs which nobody will be able to see. this is risky in particular with security-related bugs10:53
intellectronicai agree10:53
wgrantBugs that are filed as security bugs will always have somebody subscribed, though.10:53
wgrantAnd maybe marking a bug as private should warn if only you'll be able to see it.10:54
wgrantBut if a structural subscription is created initially, the projects with nobody subscribed should be minimal. In my case the interested developers subscribed separately.10:54
wgrantBut there are managerish people in the team, and they probably didn't want notifications.10:54
mrevelljtv: What do you reckon we should call these "policies"? Review policies?13:25
jtvmrevell: access or permissions policies13:26
mrevelljtv: Okay, let's stick with permissions policies as that's what we've had pretty much all along.13:26
jtvmrevell: cool13:27
beunog'mornin13:27
jtvbeuno: buenos13:28
beunojtv, que tal?13:30
jtvbeuno: bien, bien... ¿tu?13:30
jtvoh, I can do better: tú13:31
=== mrevell is now known as mrevell-lunch
jtvdanilos: cp requests are both there now... what else have I forgotten?  :)13:32
beunojtv, "vos"13:32
jtvbeuno: ahhh, you funny folks  :)13:32
beuno:)13:32
jtvThe two countries whose Spanish is _really_ weird and different: Argentina and Spain13:33
beunowe like being unique13:34
=== bigjools is now known as bigjools-lunch
salgadohey beuno!  any news about those breadcrumbs?  I'd like to see some more examples to make sure the new infrastructure can cope with them13:36
beunosalgado, I will write them *now*13:37
salgadobeuno, great, thanks!13:40
=== bigjools-lunch is now known as bigjools
beunosalgado, Im putting examples for:  a bug, a branch, a merge proposal, a package, a bug in a package and a blueprint. Need anymore?14:08
=== dpm_ is now known as dpm
salgadobeuno, how about https://translations.edge.launchpad.net/ubuntu/jaunty/+source/gfxboot-theme-ubuntu/+pots/bootloader/af/+translate ?14:12
beunosalgado, will add that as well14:13
beunoworking on the wiki page14:13
beunogimmi a few more minutes14:13
salgadosure thing!14:13
=== mrevell-lunch is now known as mrevell
BjornTsinzui: ping?14:19
sinzuiHi BjornT14:19
* gmb => caffeine. BBIAB.14:19
BjornTsinzui: hi. i'm looking at c.l.javascript/registry/milestoneoverlay.js14:19
* sinzui saw the bug14:19
BjornTsinzui: when you show an error message in the overlay, why don't you use overlay.showError()?14:20
sinzuiBjornT: It did not exist when the widget was created14:20
BjornTsinzui: ok. so it's ok if i change it to use showError()?14:21
sinzuiBjornT: I think rockstar added the feature after looking at competing implementations. I talked to him about it14:21
sinzuiBjornT: +1 to fix the widget. If you do not have time, I think we will get to it before 3.014:21
BjornTsinzui: cool. i'll do it, since i want to update it to use a new method i wrote, and the custom error handling got in the way.14:22
beunosalgado, that was a tricky one14:31
beunosalgado, danilos, https://wiki.canonical.com/Launchpad/UI/Navigation14:31
beunoI did something weird there14:32
beunoUbuntu > Jaunty 9.04 > Afrikaans translations > gfxboot-theme-ubuntu > Translating bootloader14:32
beunoinstead of:14:32
beunoUbuntu > Jaunty 9.04 > gfxboot-theme-ubuntu > Afrikaans translations > Translating bootloader14:32
beunoit was an arbitrary decision, to fix a long-standing bug for translators who can't get back to their language to translate more templates14:33
beunoI will continue adding the remaning examples, but let me know what you think14:34
danilosbeuno: right, it's something we discussed before, and what we want to fix anyway14:34
danilosbeuno: we need that on product translations as well14:35
beunodanilos, yeah, and the links don't make sense in the second option, even though the hierarchy may14:35
beunoI'm not 100% sure this isn't too crazy, but I'm 65% sure14:35
danilosbeuno: yeah, we can easily move the links around as well14:35
beunodanilos, will do one for product translations. Do you have a link handy?14:36
danilosbeuno: https://translations.launchpad.dev/evolution/trunk/+pots/evolution-2.2/es/+translate (I think, I made half of it up :))14:36
danilosbeuno: or, if you want something on edge...14:36
beunodanilos, yes  :)14:37
danilosbeuno: https://translations.edge.launchpad.net/tangocms/trunk/+lang/es and any link from there14:37
beunodanilos, thanks14:38
daniloskfogel: flacoste is in a crapper :)15:10
matsubarastub, herb, flacoste, rockstar, bigjools, henninge, sinzui, intellectronica: LP production meeting in 47 min in #launchpad-meeting15:13
intellectronicathanks for the reminder, matsubara15:13
herbmatsubara: thanks15:13
salgadosinzui, one of the tests failed because I forgot to re-add the link to '+branding' that was removed from the actions menu. should I leave it in the nav menu for now?15:19
sinzuisalgado: Can you add the link to details page.15:21
sinzuisalgado: change details page.15:22
sinzuisalgado: EdwinGrubbs: Beuno provided a good plan to allow us to move branding links into the page, so we only need to provide the link from the change details page for now.15:22
salgadosinzui, at the bottom of the +edit page, using the related pages thing? or something else?15:23
sinzuisalgado: That is the immediate kind of fix (and we can hack the h2 and ul into template, no menu) , the other route is add a sentence that includes the link in extra-form slot15:25
salgadosinzui, +edit has been converted to 3.0 in this branch of mine, and it already has a <tal:menu replace="structure view/@@+related-pages" />15:28
salgadoso, if there isn't anything preventing me from defining a menu to be used there, I think I'll just do it15:28
sinzui+115:29
beunokfogel, http://googletesting.blogspot.com/2009/07/call-for-attendance-google-test.html15:49
salgadosinzui, https://pastebin.canonical.com/21116/ fixes most of the tests that failed.  only 2 more to go15:54
sinzuisalgado: +115:56
sinzuisalgado: if the product-menu test failed, make sure you have the latest version...the lastest version is not bittle15:57
salgadosinzui, I've removed portlet-milestones from project-index.pt, so now there's a test failing because there's no 'See all milestones' link in that page15:57
sinzuiWe should add the link I think. I do not think there is any other way to get to that page.15:58
salgadosinzui, any suggestions for where to put the link?  I don't see any obvious place for it in the current layout15:59
sinzuiAt the bottom of projects portlet, where we list milestones16:01
beunosinzui, could you comment on bug 412971?16:02
mupBug #412971: Convert bug-edit.pt to 3.0 template <Launchpad Bugs:In Progress by deryck> <https://launchpad.net/bugs/412971>16:02
deryckbeuno, sinzui -- I'm adding notes/questions/etc. to that bug right now with questions I have so far.16:03
beunoderyck, I commented16:04
beunothought I did16:04
beunobut something... odd happened16:04
beunoor even better, "can we make it an open team"16:05
deryckbeuno, ah the "no portlets" rule will solve half of my questions then :)16:07
deryckbeuno, I should have refreshed first :)16:07
beunoderyck, perfect  :)16:07
deryckbeuno, some questions are still relevant though16:07
beunoderyck, will look and answer16:07
=== cprov is now known as cprov-lunch
beunosalgado, finished with the breadcrumb detailing: https://wiki.canonical.com/Launchpad/UI/Navigation16:15
beuno(I will move that to the public wiki soon)16:15
EdwinGrubbssinzui: the base-layout.pt moved the informational messages just above the maincontent div. Is there any reason it can't be moved back in? I have one test that fails, and I assume there will be others that we'll have to fix, because we usually use find_main_content() instead of importing BeautifulSoup directly.16:23
sinzuiEdwinGrubbs: I think I can move back. I did not intend to move it16:24
EdwinGrubbsok16:24
=== matsubara is now known as matsubara-afk
gary_postermatsubara-afk: lemme know when you can give me some background for the librarian-gc failure.  no rush, just trying not to forget my part ;-)16:48
sinzuisalgado: I would like your thoughts about how to proceed from this: https://pastebin.canonical.com/21124/16:50
bigjoolssinzui: woohoo that did the trick thanks16:58
bigjoolsI don't get rounded corners on the boxes, is that intentional now or am I missing some css?16:58
salgadosinzui, sorry, was on the phone; looking now16:59
sinzuibigjools: is your css uptodate? There was a typo in the css where the *border-radius properties uses % instead of px.17:00
bigjoolssinzui: I think so, the project page has rounded corners in .dev for me17:01
sinzuibigjools: they should appear in the side for all .side .portlet classes17:01
sinzuibigjools: then I wonder if some divs are missing class="portlet"17:02
bigjoolsnot that I can see17:02
bigjoolsthe yui-u is the outer div and the portlet is the inner div right?17:02
barrysinzui: so ~person index should be a main_only, right?17:05
sinzuibarry: no way an objects index will be main_side in most cases. see the design that was sent out months ago17:06
=== beuno is now known as beuno-afk
barryright, so main_only is the right thing to use17:07
sinzuibarry: no, main_side because it is a primary object's index page https://devpad.canonical.com/~curtis/LP_userdetail.png is not ready. I want it17:08
sinzuibarry: do not work on this. I want the team page to land first. It has layout changes that the user page should use.17:09
barryyep, i know17:09
barryi'm trying to get some basic grounding here.  okay, so ~person does have side actions17:10
bigjoolssinzui: I see the problem.  I was using Konqueror, it doesn't round the edges.17:14
sinzuibigjools: It should...there is something missing from the style. Which version are you using...the latest uses webkit and we cover that case17:15
bigjoolssinzui: I am using the very latest in kde4.317:15
sinzui    -webkit-border-radius: 5px;17:17
bigjoolsI see it17:18
salgadosinzui, have you considered a new interface for objects that provide a registrant/registration-date, plus changing existing classes to provide that interface? I think that'd be the best way forward as it'd also get us on the right path for standardizing the names we use for this17:18
sinzuisalgado: So add properties or fix models to support this?17:19
salgadosinzui, I'd add aliases to the existing properties and mark the existing ones as deprecated17:20
intellectronicaEdwinGrubbs: around?17:21
sinzuisalgado: okay. I will take that approach17:21
EdwinGrubbsintellectronica: hi17:21
=== cprov-lunch is now known as cprov
=== salgado is now known as salgado-lunch
mrevellcatch you tomorrow guys17:56
=== deryck is now known as deryck[lunch]
sinzuideryck[lunch]: I replied to the bug, sorry for the delay18:19
sinzuideryck[lunch]: I can get you a diff of another edit page that was modified to show all the changes you might make to convert a modification page to ui 3.018:20
=== matsubara-afk is now known as matsubara
sinzuideryck[lunch]: I added an example diff of a full conversion of a modification page: https://dev.launchpad.net/VersionThreeDotO/UI/Conversion18:31
=== salgado-lunch is now known as salgado
=== deryck[lunch] is now known as deryck
derycksinzui, thanks for all that.  *very* helpful work.19:05
gary_posterabentley, rockstar, on #launchpad someone is having problems pulling.  They say it is neverending and then they get19:56
gary_posterbzr: ERROR: http://bazaar.launchpad.net/%7Eturl/%2Bjunk/ircbot/.bzr/repository/packs/dd8ac238ab24bfcbbb8620946cf7d032.pack is redirected to https://launchpad.net19:56
gary_posterdo one of you have time to check that out?19:56
rockstargary_poster, abentley is on holiday this week. I'll figure chat with them.19:57
gary_posterrockstar: ok thanks much19:57
derycksinzui, ping.20:25
sinzuihi deryck20:25
derycksinzui, hi! With generic-edit.pt, there's no way to get messages/notifications back to the template, right?20:26
sinzuideryck: those are supplied by base-layout.20:26
sinzuideryck: EdwinGrubbs discovers a bug this morning. the notifications are in the wrong place20:27
sinzuideryck: EdwinGrubbs may have a fix ready20:27
EdwinGrubbsderyck: yes, I changed it in my branch, which I will put into review today.20:27
derycksinzui, ah, ok.  So it should work soon if I continue to use generic-edit.pt?  I have broken tests that test for confirmation messages is why I ask.20:28
deryckEdwinGrubbs, awesome!  Good news then :)20:28
intellectronicabigjools: bear in mind my branch is going to spend quite some time in pqm, it's lazr so it gets a full run. if you rather have it killed so that your testfix gets in quickly it's fine by me20:43
intellectronicalet me know if you do so that i can resubmit20:43
bigjoolsintellectronica: ah bugger, might be a good idea to do that20:44
bigjoolsor trunk will sit broken20:44
intellectronicayeah20:44
bigjoolsmthaddon: would you mind doing the honours please? ^20:44
mthaddonbigjools: stop pqm, remove intellectronica's branch?20:46
bigjoolsmthaddon: yeah please, so mine gets in quicker20:46
bigjoolsand then tom can resubmit20:46
mthaddonintellectronica: ok, feel free to resubmit20:48
intellectronicamthaddon: cheers20:48
mthaddonyah20:48
rockstarHow do I enable the storm tracer for make run?20:51
rockstarEdwinGrubbs, ^^ Maybe you know?20:53
thumperrockstar: skype20:57
rockstarthumper, hi.20:57
bigjoolsthanks for pointing that one out intellectronica21:01
intellectronicanp21:02
=== beuno-afk is now known as beuno
sinzuibeuno: is there a reason we do not want to show distribution custom icons in links?21:35
beunosinzui, none at all21:36
beunoit's a bug21:36
sinzuibeuno: I seem to have fixed it. It was an accident.21:36
=== cprov is now known as cprov-afk
beunosinzui, thank you21:38
maxbajax bug commenting is entirely shiny :-)21:53
EdwinGrubbsrockstar: do you still need info on the storm tracer?22:00
EdwinGrubbssinzui: is there a bug open already for the team edit pages?22:00
sinzuiNo22:00
=== salgado is now known as salgado-afk
sinzuiEdwinGrubbs: maybe22:01
* sinzui looks at EdwinGrubbs assigned bugs22:01
EdwinGrubbshuh, how did I miss that?22:02
sinzuiEdwinGrubbs: You have two bugs. One for the index and one for the edit pages22:02
rockstarEdwinGrubbs, I think we're okay now, thanks.22:02
=== matsubara is now known as matsubara-afk
maxbuhoh. ajax bug commenting is shiny, but not when it fails to actually save your comment despite looking as if it did!22:04
maxboh22:05
maxbno, it did save, but there was a quirky browser caching issue22:05
beunosinzui, https://staging.launchpad.net/bzr22:07
=== _thumper_ is now known as thumper
sinzuiThose file names are going to be difficult. What part can we hide? Maybe we need a smaller font22:10
sinzuibeuno: I think we need a santise the packages portlet. Show the the latest pacckage, or limit it to 322:12
beunosinzui, exactly22:12
beunoI think the length is not a problem, but the large amounts of them is22:12
EdwinGrubbssinzui: mp sent22:13
sinzuibeuno: the file names are a font-size larger than the body text22:13
sinzuiEdwinGrubbs: thanks!22:13
beunosinzui, ah, indeed they are22:13
Fly-Man-Evening all22:16
Fly-Man-I have succesfully installed the LaunchPad on a local server22:16
Fly-Man-and made some Wiki changes on how to get it running so you can work it on a internal network22:17
Fly-Man-but I have a Q about the source importer22:17
Fly-Man-does it work in the development branch /22:17
maxbI'm fairly sure that's illegal unless you rebrand it22:17
maxbI wish it wasn't though, I'd quite like to use Malone myself22:18
Fly-Man-okay, so the code importer doesn't work in the development branch ?22:18
maxbI have no idea22:18
Fly-Man-because waiting for 1 hour now and I see no change on the pages22:18
Fly-Man-maxb: https://dev.launchpad.net/Running/LocalNetwork22:19
Fly-Man-For getting it to run on a local network22:19
maxbI'd imagine it probably needs to be kicked off from a cronjob22:19
maxbWhilst it's nice that Canonical have open-sourced LP at all, it seems that they've diligently avoided distributing a configuration capable of running a local instance for production purposes.22:21
Fly-Man-Hmm, but so far it's working like a charm22:21
Fly-Man-yeah, and still need to find out how i can delete the "test" stuff and projects22:21
Fly-Man-so I can have a clean tree22:22
maxbFly-Man-: What do you mean by code importer, ooi?22:38
Fly-Man-maxb: the part where I put in a SVN url22:38
Fly-Man-and it starts to grab that SVN22:38
Fly-Man-and downloads the SVN history into the trunk on the Launchpad22:38
Fly-Man-so I can fiddle with it22:38
Fly-Man-and checkout the source with bzr22:39
Fly-Man-Like it tells on this page:22:39
Fly-Man-https://help.launchpad.net/VcsImports22:39
maxbIt probably has something to do with cronscripts/code-import-dispatcher.py22:40
Fly-Man-And also what I notices22:41
Fly-Man-the version of the trunk build is still 2.2.622:41
maxbhuh, still? I though wgrant landed a branch to fix that22:42
Fly-Man-I see this:22:42
Fly-Man- Launchpad 2.2.6 (r9108)  devmode22:42
Fly-Man-I grabbed the source 4 hours ago22:42
maxbhmm, wonder what happened to that branch22:43
Fly-Man-no clue ...22:43
thumperFly-Man-: yes the importer stuff works22:44
Fly-Man-thumper: is there any page on the Wiki about it ?22:44
Fly-Man-how to setup ?22:44
thumperno22:44
thumperand right now I'm firefighting22:45
* Fly-Man- throws a fire blanket to thumper 22:45
=== matsubara-afk is now known as matsubara
Fly-Man-thumper: hope it's not your pc that's on fire ....22:53
thumperflacoste: no, production issues22:53
Fly-Man-okay ...22:54
thumperflacoste: sorry, tab completion fail22:54
Fly-Man-well, i'll have some sleep22:55
Fly-Man-and maybe tomorrow when it's not fire time we can have a talk ?22:55
thumperit is friday for me today :)22:56
Fly-Man-thumper: it's almost friday for me here22:56
* Fly-Man- yawns23:03
Fly-Man-K, time for me to get some sleep23:03
Fly-Man-Nite all :)23:03
Fly-Man-and thanks for the help so far23:04
wgrantmaxb: Mine was r910923:13
wgrantLanded 12:41 UTC23:13
maxbnot that branch, different branch - what happened to your "fix the version numbers to say 2.2.7" branch?23:13
wgrantOh.23:14
wgrantI didn't have a branch to fix that.23:14
maxbah. someone did. I must be misremembering23:15
wgrantI tried to run the test suite in Karmic 2.4 last night, but my LVM snapshot filled up :(23:16
intellectronicawow, something very strange is happening. i've submitted a lazr.restful branch to pqm and pqm spat it back claiming there are merge conflicts. but there are no revisions in trunk that are not present in my branch!23:26
thumperintellectronica: crisscross merges?23:27
intellectronicathumper: no no, my branch is exactly one revision on top of what's already in trunk. should be a very straightforward merge23:28
thumperintellectronica: are you sure you are submitting to the correct branch?23:28
intellectronicayeah, i'm wondering that too. i'm submitting to ~launchpad-pqm/lazr.restful/trunk23:29
intellectronicawhich is the branch i branched from, so it is, actually, the correct branch23:30
wgrantintellectronica: That's not really trunk.23:30
intellectronicawgrant: oh?23:30
wgrantRecent commits have been to ~lazr-developers/lazr.restful/trunk23:30
wgrantNot via PQM either, AFAICT.23:31
intellectronicawhich doesn't conflict with my branch either23:31
intellectronicai find it very hard to believe that whatever we're using in lp accept merges not through PQM23:32
intellectronicamaybe it's a parallel branch or something23:32
intellectronicayou're right, though, that this is the branch currently registered as trunk23:33
intellectronicaleonardr: any idea what's going on? ^^^^^23:36
intellectronicatrying to submit to that branch fails too, which sort of makes sense, because it isn't owned by pqm23:38
intellectronicaha ha, ok, i get it23:39
intellectronicawgrant: thanks, your clue helped23:40
intellectronicaso my branch doesn't conflict with the branch i thought is the one i'm submitting too, which is the lazr-developers branch, but it conflicts all over the place with the branch managed by pqm23:40
wgrantintellectronica: Excelent.23:40
wgrantAh.23:41
wgrantMakes sense.23:41
wgrantApart from why the branch isn't PQMed.23:41
intellectronicayeah, that's very strange, i don't remember any mention of that change23:41
wgrantI've seen a couple of mentions of moving to a non-PQMed trunk in lazr.restful merge proposals as they fly past.23:42
maxbWhat's the bin/test incantation to run a single doctest?23:43
wgrantbin/test -vvt testname.txt23:44
wgrantThat's what I use.23:44
wgrantYou can also give the full path.23:44
=== EdwinGrubbs is now known as Edwin-afk
maxbwgrant: many thanks for sorting gina.txt. Unfortunately package-diff.txt is still broken.23:51
maxbhttp://paste.ubuntu.com/252805/23:52
wgrantmaxb: Huh.23:53
wgrantMy Karmic LP chroot is currently rebuilding, so I'll look in a moment.23:53
wgrantBut that's nothing like the failures I had yesterday.23:53
* maxb rants furiously at shhh.py23:58

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!