/srv/irclogs.ubuntu.com/2006/03/01/#launchpad.txt

=== mpt_ [n=mpt@219-89-153-81.jetstart.xtra.co.nz] has joined #launchpad
kikomatsubara, vai ou no vai? eu tenho que ir embora!12:12
matsubaravou onde?12:13
=== AlinuxOS [n=Ubuntu@d83-176-25-250.cust.tele2.it] has joined #launchpad
=== X1n3 [i=X1n3@cm-83-97-180-148.telecable.es] has left #launchpad []
=== X1n3 [i=X1n3@cm-83-97-180-148.telecable.es] has joined #launchpad
X1n3hi12:53
X1n3what is?12:56
X1n3that the free cds need to be aproved?12:56
X1n3i have 45 of the 100 that i say12:57
kikoX1n3, write to the email address I provided earlier.12:58
=== kiko [n=kiko@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== X1n3 [i=X1n3@cm-83-97-180-148.telecable.es] has left #launchpad []
=== ryanakca [n=ryan@d226-20-171.home.cgocable.net] has joined #launchpad
ryanakcawhat is "ubuntero"?01:08
kikoanother FAQ01:10
kikoa person who has signed the Code of Conduct01:10
ryanakcakk, ty01:13
kikonjy01:13
kiko-zzzdilys, give me the good news tomorrow01:18
=== kiko-zzz yawns
dilysMerge to devel/launchpad/: [trivial]  Fix for bug 31380: source package sort by version doesn't cope with invalid version numbers. Use apt_pkg.VersionCompare instead of Sourcerer's unmaintained Version method which blows up occasionally. Fixes an OOPS. Even removes an XXX (r3192: kiko)01:25
=== jsgotangco [n=jsg@210.4.38.43] has joined #launchpad
=== ohoel [n=eruin@unaffiliated/eruin] has joined #launchpad
=== stub [n=stub@ppp-58.10.74.179.revip2.asianet.co.th] has joined #launchpad
dilysMerge to devel/launchpad/: rs=BjornT though rather trivial; Fix for bug 31367: Specifying a non-published binary package when filing a bug causes an oops. Change the add bug process to ignore the package if it was never published in the distribution, and test it using Gentoo (which now uses Malone officially, yay. Or at least in sampledata). This fixes a topcrasher, so hooray for us. (r3193: kiko)02:04
Kinnisonjamesh: ping?02:13
jameshKinnison: pong02:23
Kinnisonjamesh: I'm desperate for a bzr conversion of the gnome-power-manager portion of gnome-cvs02:23
Kinnisonjamesh: but tailor can't do its job from gnome anoncvs02:23
Kinnisonjamesh: any chance you could grab me a tarball of that bit of cvs?02:23
jameshyou want a tarball of the repo?02:23
Kinnisonplease02:23
=== Kinnison hopes he has time to translate it before he falls asleep :-)
Kinnisonit's 01:2402:24
jameshhttp://www.gnome.org/~jamesh/gnome-power-manager-repo.tar.gz <- 3.8MB02:25
Kinnisonfetching now02:25
jameshKinnison: looks like time doesn't go backwards for that module, so Tailor shouldn't get confused02:26
KinnisonUnicodeDecodeError: 'utf8' codec can't decode bytes in position 0-3: invalid data02:27
Kinnisongo tailor02:27
Kinnisonany suggestions?02:31
Kinnisonif I do "LANG=C ....." then it whinges the equivalent with an ascii codec02:32
stubMight try LANG=en.iso-8859-1  -- text files without an explicit encoding have a tendancy to be iso-8859-1 encoded02:33
Kinnisonthat's an "unsupported locale" apparently02:34
Kinnisoneven after I've allegedly created the en_GB.ISO8859 locale python rants02:35
stubKinnison: You could try changing the default encoding in /usr/lib/python2.4/site.py (the setencoding() method), but make sure you change it back!02:36
=== stub cluches at straws
jameshKinnison: did you try putting "encoding = iso-8859-1" in the [cvs:source]  section of your .tailor file?02:37
Kinnisonnope 'cos I didn't know it existed02:38
Kinnisontrying02:38
Kinnisonoooh, jamesh wins02:38
Kinnisontailor running02:38
jameshiirc, tailor told me to do that when I tried02:38
Kinnisonjamesh: aah didn't tell me02:38
jameshKinnison: which version of tailor are you using?02:38
Kinnison0.9.20 apparently02:39
jameshTailor could certainly do with some robustness improvements02:39
stubAnyone have access to merge to the tailor trunk? pull out canonical.encoding and use encoding.guess as a fallback. Otherwise you won't be able to migrate trees where commit messages used mixed encodings.02:40
Kinnison<voice style="gump">Run tailor! Run!</voice>02:52
KinnisonHmm, up to the 19th January03:29
Kinnisoncor 10th Feb03:35
Kinnisontailor in getting faster shocker03:35
Kinnisonjamesh: you're a star03:42
=== Kinnison goes to bed
Kinnisonciau03:42
=== stu1 [n=stub@ppp-58.10.73.80.revip2.asianet.co.th] has joined #launchpad
=== merriam [n=merriam@84.12.156.26] has joined #launchpad
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
jameshstu1: do you have any ideas about this bug? https://launchpad.net/products/launchpad/+bug/3165105:16
Ubugtumalone bug 31651 in launchpad "IPlacelessLoginSource.getPrincipalByLogin(email) returns None with an email address that was just created" [Normal,Confirmed]  05:16
jameshstub: the second OOPS report seems very weird, given the statement log05:17
stubIf the insert was failing, the transaction would be screwed. So we can assume the inserts are working correctly05:22
stubI could see that behavior happening if we had multiple connection objects.05:22
stubI could see that happening if some of the code was creating a connection using sqlbase.connect(), as it opens a fresh database connection.05:23
stub(by default - I think I landed an option recently to make that call reuse the existing connection if a parameter is passed? Can't remember if it landed or if I backed it out)05:24
stubBut that wouldn't be happening because it is all SQLObject code05:24
jameshcould it be index corruption?05:25
stubUnlikely - postgresql is good on that front. Bugs like that seem to only occur in the pathalogical edge cases some people create.05:26
stubBut possible05:26
stubDo we log calls to the connections commit and rollback methods?05:27
stubEasiest way to explain that behavior would be some doofus sticking a rollback in the code, or rollback being called by Z3 for some pathalogical reason05:28
stubIs it happening often? I  can reindex the person table for a laugh but it will render launchpad unusable for maybe 10 mins05:30
jameshI don't know how often it happens: salgado only mentioned two oops reports05:31
stubI'll make a note to reindex the person and emailaddress tables next rollout (and hopefully remember to check my notes...)05:31
jameshI wonder how much more reliable things would be if SQLObject did "self._connection = self._connection" in its constructor?05:41
jameshit'd definitely help with problems related to garbage collection on the wrong thread05:41
stubSQLOS replaces self._connection with a descriptor, so in our case I don't think it would help much (?).05:53
jameshthe descriptor is the issue: it means that if an object gets garbage collected on a different thread, the destructor will use a different _connection than the one used to create the object06:03
jameshso the object doesn't get removed from the cache06:04
spivjamesh: Ouch, that's a nasty one.06:06
jameshspiv: SteveA pointed it out to me originally06:06
spivjamesh: Yeah, instances really should know what connection they're tied too.06:06
jameshspiv: with plain sqlobject they do: self._connection :)06:07
spivRight :)06:07
spivzopeless also has its own descriptor for _connection :/06:07
jameshyeah06:08
jameshI suppose one way to do it would be to make the descriptor's __get__ method do the assignment06:08
jameshso an sqlobject gets bound to a connection the first time it tries to use it06:08
=== mpt__ [n=mpt@219-89-133-196.jetstart.xtra.co.nz] has joined #launchpad
=== stub [n=stub@gb.ja.101.40.revip.asianet.co.th] has joined #launchpad
=== pkunk_ [n=pkunk@a84-231-5-36.elisa-laajakaista.fi] has joined #launchpad
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
=== carlos [n=carlos@84.76.255.40] has joined #launchpad
=== koke [n=koke@ubuntu/member/koke] has joined #launchpad
=== istbot [n=gokaye@88.224.183.203] has joined #launchpad
istbothi 09:15
istbotfuck you ChanServ09:16
=== fish [n=kvirc@h081217119213.dyn.cm.kabsi.at] has joined #launchpad
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has joined #launchpad
=== stub [n=stub@gb.ja.101.40.revip.asianet.co.th] has joined #launchpad
=== mpt [n=mpt@219-89-132-228.jetstart.xtra.co.nz] has joined #launchpad
=== dsaa [n=dsaa@210.1.85.218] has joined #launchpad
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== carlos_ [n=carlos@62.87.127.194] has joined #launchpad
=== mpt [n=mpt@219-89-132-228.jetstart.xtra.co.nz] has joined #launchpad
=== WaterSevenUb [n=WaterSev@195-23-238-209.nr.ip.pt] has joined #launchpad
carlosspiv: hi, around?11:21
ddaalifeless: can you make it more apparent, in emails sent to arch-commits, what is the branch being merged into?11:24
ddaaFor example, by telling the branch name and nick in the message body and in a header11:25
ddaaI'm quite confused with bzr commits being mixed with launchpad commits11:25
ddaaI have to actually think to tell what the commit is relevant to.11:26
ddaaIt hurts.11:26
stub+111:41
mptAnd also list the commits in chronological instead of reverse chronological order :-)11:43
mptAnd make the list of modified files as compact as it was in the baz days11:43
mptAnd, and, and ...11:43
=== mpt gets a headache from Launchpad permissions
mptcarlos, do you happen to know what the template equivalent of 'launchpad.Edit' is?11:44
mpttal:condition="..."11:44
mptI thought it would be tal:condition="view/edit", but no11:45
carlosI don't remember it, but I know where I can see it11:45
carlosjust a second...11:45
carlosmpt: well, you first need to associate the launchpad.Edit permission with the interface11:46
mptI think it already is, because it's used for this page's menu11:46
mpts11:46
mpt@enabled_with_permission('launchpad.Edit') works for one of the menu items11:47
mptbut now I want to do the same thing in TAL11:47
mpthmmm, other templates are using "view/edit"11:48
carlostal:condition="entry/required:launchpad.Edit"11:48
carlosentry is a SQLObject11:49
mptaha11:50
mptso in this case, tal:condition="context/required:launchpad.Edit"11:50
mptGreat, carlos, thanks11:50
carlosyeah11:50
carlosmpt: you are welcome11:50
mptanother bug fixed11:52
mptand another12:00
=== hunger [n=tobias@p54A62775.dip0.t-ipconnect.de] has joined #launchpad
hungerIs there any documentation on what I should do to report bugs in launchpad?12:02
hungerLike should I assign bugs? To whom? May I close bugs? Which ones? Which priorities/severities may I assign? That kind of stuff?12:03
hungerWhat does assigning a bug signify?12:04
hungerAll this workflow kind of stuff.12:04
BjornThunger: that kind of workflow is specific to each project using launchpad. for example for ubuntu bugs, http://www.ubuntu.com/community/participate#head-5f51744fe2ab198ea488499b475eedefb45860c2 is proably a good start.12:11
hungerBjornT: Thanks!12:11
hungerEven though the documentation is still about bugzilla there:-)12:13
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== jinty [n=jinty@196-28-45-170.jhb.netdial.co.za] has joined #launchpad
cprovgood morning 12:14
=== mpt [n=mpt@219-89-132-228.jetstart.xtra.co.nz] has joined #launchpad
BjornThunger: really? :) i looked only briefly, and saw that it mentioned launchpad and malone, so i assumed they were updated12:16
=== AlinuxOS [n=Ubuntu@d83-176-25-250.cust.tele2.it] has joined #launchpad
=== lbm [n=lbm@x1-6-00-13-10-7a-d1-e4.k233.webspeed.dk] has joined #launchpad
kiko-zzzhello hackers of the world12:53
kiko-zzzUNITE12:53
=== matsubara [n=matsubar@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
kikostub, ping?01:04
stubkiko: pong01:17
kikohello stub 01:17
kikohow's thailand today?01:17
kikoit is hot and humid here01:17
kikoand the air is sizzling with pqm commits01:17
stubkiko: same same01:18
=== beyond [n=beyond@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
kikostub, would you consider my commits from yesterday for the rollout?01:18
stubyes, I was thinking of rolling out...01:18
=== stub checks his email
stubr319301:18
kikostub, matsubara has one OOPS fix to land this morning01:18
stubok01:19
kikoit hasn't landed because he apparently hasn't had a haircut yet or something like that01:19
kikomatsubara, can you mail stub cc: launchpad with the revision you land your majestic fix for bug 5757 the bug from hell?01:19
Ubugtumalone bug 5757 in malone "OOPS: requesting an upstream fix doesn't check input for duplicates" [Normal,In progress]  http://launchpad.net/bugs/575701:19
stubCan I use that excuse?01:19
kikostub, it's harder in your case because it is painfully obvious you have not had a haircut in the last 19 years01:20
matsubarakiko: yes, I can. but let's see if pqm will accept it01:20
kikothanks01:20
matsubarakiko: 4 conflicts01:21
kikofix them and resubmit01:21
kikothey should be trivial01:21
ddaadoes the zopeless test harness actually commit to the database?01:22
kikodaf, ping?01:22
=== fabbione [i=fabbione@gordian.fabbione.net] has joined #launchpad
kikoddaa, I believe so01:22
kikootherwise, how would selects work?01:22
ddaai.e. if I use LaunchpadZopelessTestSetup, can I put some some test data in the database, spawn a subprocess, and have the subprocess retrieve the data?01:22
lifelessyes01:23
ddaa(after commit, of course)01:23
kikoyes01:23
kikoddaa, look at gina.txt01:23
kikoI do something like that in it01:23
ddaaI trust you.01:23
kikothere may be hints in that file01:23
ddaajust starting to hit the first "duh, didn't think of that" in implementing bzr imports...01:24
kikostub, ping 2.0, a bit more urgent01:24
kikohttps://launchpad.net/distros/ubuntu/+ticket/256/+index\01:24
ddaaokay, looking01:24
kikostub, this suggests to me that the migration for statuses we did wasn't complete01:24
kikostub, can you handle the database hack?01:24
stubok01:25
ddaakiko: it's the first time I see a use of "import transaction". Is that a relic, or is that a new preferred way of doing commits and stuff?01:26
kikoI think it's the way it's done in tests BIMBW01:26
ddaahow does that compare to canonical.database.sqlbase's begin, commit and rollback?01:28
stublaunchpad_prod=# select status,count(*) from ticket group by status;01:28
stub status | count01:28
stub--------+-------01:28
stub     30 |    1201:28
stub     20 |   16901:28
stub     10 |   22201:28
stub(3 rows)01:28
kikostub, wth is wrong with that page? is it a keyerror on something else? it still oopses01:29
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
stubThe ticket update has been done - not sure where that 40 is comming from?01:29
kikocould it be /gasp/ hardcoded?!01:29
stubWould the page template be broken?01:29
kikoit must be the case01:30
kikoprobably something wrong in the menu links01:30
kikobut where?01:30
kikostub, can you make sense of that traceback?01:31
BjornTkiko: ah, i think there is some history that needs to be migrated :(, i'll take a look.01:32
kikothanks BjornT 01:32
kikothat's okay -- it appears to be affecting few users01:33
BjornTstub: please run: select status,count(*) from ticketreopening group by status;01:34
stubJust found that table :)01:35
kikolaunchpad_staging=> select priorstate,count(*) from ticketreopening group by priorstate;01:35
kiko priorstate | count 01:35
kiko------------+-------01:35
kiko         30 |    2301:35
kiko         50 |     201:35
kiko         40 |     301:35
kiko(3 rows)01:35
kikoenjoy the pain01:35
kikoman this my bloody valentine album is excellent01:36
stubBjornT: Do you think just priorstate needs to be updated, or is it more complex like the ticket update patch?01:38
stubOtherwise I've got:01:39
stubUPDATE ticketreopening SET priorstatus=10 WHERE status IN (20, 30);01:39
stubUPDATE ticketreopening SET priorstatus=20 WHERE status=40;01:39
stubUPDATE ticketreopening SET priorstatus=30 WHERE status=50;01:39
BjornTstub: it's just the priorstate column that needs to be updated. yes, i was going to give you those queries :) although, the last 'status's should be priorstatus as well.01:41
kikoBjornT?01:41
BjornTkiko: yes?01:41
kikoI didn't understand your last phrase.01:41
stubbah - they are all priorstate01:41
kikoah, yes.01:42
stubtickets fixed01:42
kikogood work stub and BjornT 01:42
kikothe page works!01:42
=== dsa [n=dsaa@210.5.87.33] has joined #launchpad
kikostub, ping?01:50
stub?01:50
kikostub, could you find out how many bugwatches we have for each bug tracker type?01:51
kikoI'm in the middle of a hack or else I would do it myself01:51
kikoif you are uberbusy I can do it later01:51
stublaunchpad_prod=# select count(*),bugtrackertype from bugtracker,bugwatch where bugwatch.bugtracker = bugtracker.id group by bugtrackertype;01:54
stub count | bugtrackertype01:54
stub-------+----------------01:54
stub  3730 |              201:54
stub 22965 |              101:54
stub     9 |              401:54
stub(3 rows)01:54
kikothanks dude01:54
kikoappreciated01:54
=== carlos -> lunch
carlosIf you need me for anything, I will be available by phone. I need to recharge it01:56
carlossee you later01:56
kikocarlos, did you get cprov a bugfix01:56
kikoshit01:56
=== pkunk__ [n=pkunk@a84-231-5-36.elisa-laajakaista.fi] has joined #launchpad
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
salgadostub, can you do a select datecreated from person where name = 'daniangella'; on production?02:08
=== Keybuk [n=scott@213-78-32-60.ppp.onetel.net.uk] has joined #launchpad
stub 2006-02-23 10:20:46.87801902:08
salgadothat's UTC, right?02:09
stubyup02:11
=== raptoid [n=raptoid@85.96.210.207] has joined #launchpad
kikoBjornT, ping?02:33
kikobradb, ping?02:33
=== jsgotangco [n=jsg@ubuntu/member/jsgotangco] has joined #launchpad
bradbkiko: pong02:34
kikobradb, have we had the existing Ubuntu Dapper tasks closed or updated?02:34
kikobradb, if not, could you please take care of that today.02:34
bradbsure02:35
kiko(mark mailed me) thanks!02:35
bradbHm, this could be painful02:36
bradbThere are 49 + assignee and various status information to keep on them.02:36
kikoI wonder if we can offload that to someone else.02:37
=== kiko thinks
kikojblack?02:37
bradbMaybe an Ubuntu triager?02:39
kikocan you try finding one?02:39
bradbsure02:39
=== mgalvin [n=mgalvin@ubuntu/member/mgalvin] has joined #launchpad
jblackkiko:?02:43
kikojblack, was wondering if you could help us with something, but..02:43
jblackbut... 02:43
jblackwhat's the something?02:44
kikomoving some ubuntu bugs around 02:44
kikolet's see if dholbach is game first, otherwise you can help02:44
jblackheh. I don't know if I can help until I understand the scope of the help.02:44
kiko:)02:45
jblackIf its a company priority, I can drop work responsibilities to do it though.02:45
bradbjblack: looks like dholbach is on top of it, thanks02:47
BjornTkiko: pong?02:52
kikoBjornT, never mind, was the topic above02:52
BjornTok02:52
kikothanks though02:52
=== doko [n=doko@dslb-084-059-085-222.pools.arcor-ip.net] has joined #launchpad
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
mdke_hi all03:01
kikohello mdke_ 03:01
mdke_an upload to breezy-updates was published yesterday03:01
mdke_and it has appeared in the archive but is not being included in the index03:01
mdke_https://launchpad.net/distros/ubuntu/breezy/+source/ubuntu-docs/5.10-6.303:01
mdke_http://archive.ubuntu.com/ubuntu/pool/main/u/ubuntu-docs/03:01
kikowhen did it appear in the archive, mdke_?03:01
mdke_kiko, i don't know, it was built about 24 hours ago, so maybe soon after that?03:02
kikodoes the file time not suggest that03:02
=== mdke_ downloads it
mdke_about 11 hours ago? maybe03:03
mdke_i don't know03:03
mdke_it finished building about 26 hours ago according to https://launchpad.net/+builds/+build/17086603:04
kikomdke_, okay, we'll need cprov to look into it and figure out why apt-ftparchive isn't doing it's thing03:05
mdke_shall I file a bug, or will you guys look into it without?03:05
kikomdke_, file a bug and ping cprov03:05
mdke_thanks kiko03:06
kikoenjoy03:06
mdke_cprov, bug #32721 for you. Thanks!03:09
Ubugtumalone bug 32721 in soyuz "Package built for breezy-updates is not appearing in the package index" [Major,Unconfirmed]  http://launchpad.net/bugs/3272103:09
=== jsgotangco [n=jsg@125.212.123.59] has joined #launchpad
=== mdke_ is now known as mdke
=== tseng [n=tseng@unaffiliated/tseng] has joined #launchpad
=== hunger [n=tobias@p54A61C35.dip0.t-ipconnect.de] has joined #launchpad
tsengis it a known issue that closing any bug makes launchpad  oops?03:19
kikotseng, no, it is not. OOPS code please.03:20
tsengkiko: OOPS-55C32103:20
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/55C32103:20
tsengkiko: this is the second bug i've tried to close with the same result03:20
kikohold tight london03:21
kikoBjornT, ping03:21
BjornTpong03:21
kikoI need a band-aid for tseng 03:21
kikoBjornT,     from_addr = format_address(person.displayname, person.preferredemail.email)03:21
kikoAttributeError: 'NoneType' object has no attribute 'email'03:21
kikoBjornT, we need a band-aid and a needle to test it03:22
BjornThmm, i can't say that i understand why this is happening. person should be the currently logged in user, and the error indicates that it doesn't have a preferred email address. i'll take a closer look...03:27
tsengBjornT: there is a back story03:27
tsengwhich may or may not be relevant03:27
tsengi tried to merge two accounts, one with a non-working email address03:27
tsengso one account got stuck in limbo03:27
tsengstevea merged it for me by hand after gpg verification03:28
tsengthe account might be a bit non-standard due to this03:28
kikotseng, what's the account name?03:29
tseng'brandon'03:29
kikoyou have a preferred email address03:29
kikoit is listed on your page03:30
kikowhat was the old address?03:30
tsengbrandon@brandonhale.us was the broken one03:30
kikodo you remember the launchpad name for it?03:30
tsengbrandon-ubuntu was the other account03:30
ddaaokay, the commit-pass-to-subprocess thing does not work here :(03:30
kikotseng, are you logged in now?03:31
tsengkiko: as brandon03:31
kikoare you sure it's as brandon? can you click on the link next to Log Out to confirm?03:31
tsenghttps://launchpad.net/people/brandon03:32
bradbkiko: I noticed that the Dapper bugs are all cleaned up now. Yay to dholbach.03:32
kikotseng, okay, then I'll leave you in BjornT's hands because it is not an obvious case of bustage03:32
tsengkiko: ok.03:32
Kamionbradb: was there a reason why they *all* had to be closed? I was using the dapper tasks to remember certain things I wanted to fix for dapper, and I've just lost that information.03:32
Kamionbradb: because "target fix to release" creates a dapper task03:32
BjornTkiko: have you executed the last sql query in the oops report, to see what it returns?03:32
kikoKamion, it won't create them any longer after tuesday03:33
kikoKamion, you should use milestones instead03:33
kikoreleases are for backports03:33
bradbKamion: yeah, that's a UI bug that I've landed a fix for. things you want to fix for dapper should use milestones (again, our fault on the UI)03:33
kikomilestones are for the future03:33
Kamionwhere is the UI for that?03:33
bradbKamion: +editstatus03:33
BjornTkiko: also, please check 'select name from person where id=30159903:33
Kamionah03:33
dilysMerge to devel/launchpad/: [trivial]  Add an assert in PersonSet.getByEmail() to help debugging https://launchpad.net/products/launchpad/+bug/31755 (r3194: Guilherme Salgado)03:33
Kamionthat has both "dapper" and "ubuntu-6.04"03:34
kikorock on dilys 03:34
Kamionmake your minds up :P03:34
bradbKamion: Those are your guys' milestones, not ours03:34
kikoKamion, blame mdz! :)03:34
KamionI frequently do03:34
Kamionmdz is Canada03:34
kikoBjornT, brandon-ubuntu-merged03:34
bradbThe "dapper" one was there before the important, the "ubuntu-*" ones came from Bugzilla, IIRC03:34
bradbs/important/import/03:34
Kamionbradb: please add a way to search by milestone03:35
BjornTtseng: can you try to logout, and then login again, to see if it helps?03:35
tsengBjornT: mm, ok03:36
tsengBjornT: whatdyaknow03:37
tsengBjornT: works.03:37
bradbKamion: er, yeah, that's a bug. I've got a big landing in code review right now which changes all the bug listings and advanced search screen, so I can easily sneak in the milestone search.03:37
tsengBjornT: thanks a lot03:37
Kamionbradb: yes please, I need to not lose track of bugs where my manager has said "please don't forget about this for dapper"03:37
bradbjamesh: Any review news, btw? :)03:37
BjornTtseng: ok, that's good to hear. it was probably due to the manual merge03:37
=== BjornT heads out for a while
=== carlos [n=carlos@84.76.255.40] has joined #launchpad
carloscprov: hi03:42
cprovcarlos: hi03:42
carloscprov: I'm back with DSL, just in time ;-)03:42
cprovcarlos: good, have talked with pitti about our situation03:42
carloscprov: Thinking it twice, you don't need any patch from me. The changes I did are not executed by soyuz but from manually uploaded files03:42
cprovcarlos: uhm ...03:43
carloscprov: rollout your changes and I will try to get mine reviewed, landed and cherrypicked as soon as possible (that's today)03:43
carloskiko: hi, around?03:43
Kamionbradb: should I file a bug as a memory aid?03:44
cprovcarlos: I've already rolled out my relevant parts,i.e., your code is there ;)03:44
carloshmm, in fact, my changes are not really needed for soyuz, but for manual uploads so it's not even so urgent03:44
carloscprov: well, I mean, any other change that was not already on your branch03:44
cprovcarlos: AFAICS, you won't be able to cherrypick anything til monday, no stub03:45
carloscprov: when will martin upload the new pkgstriptranslations?03:45
cprovcarlos: the other changes I have are planned for tuesday03:45
bradbKamion: sure, that would help. can you please assign it to me, and mark it High priority and In Progress?03:45
cprovcarlos: so, I've fixed a deadline to tuesday03:45
carloscprov: that's ok, as I said, it's not affected directly by your uploads from soyuz, that should work anyway03:45
cprovcarlos: or as soon as you can release your fix03:45
carloscprov: so no translations for dapper until tuesday??03:45
carloscprov: ok, let me tell it again. You don't need any patch from me or on production for dapper imports. At least not that I'm aware of03:46
kikocarlos, always around.03:46
carloscprov: if your part is already done, let's do it now03:46
cprovcarlos: pitti doesn't like the idea of extra clicks exposed by the lack of your fix03:46
carloscprov: I did a mistake when I talked with you03:47
cprovcarlos: what was it ?03:47
carloscprov: those clicks should not affect soyuz imports03:47
carlosbecause the translations are not uploaded against a potemplate but a source package03:47
carlosso we need the initial review anyway03:47
cprovcarlos: so why those tests in mawson was wrong ? 03:48
cprovwere03:48
carloscprov: so the changes I'm doing atm are useful only for manual imports not for the automatic path03:48
ddaamaybe it's just connecting to the wrong db...03:48
cprovcarlos: I lost the track then, why did you said the mawson's results were wrong ?03:48
carloscprov: Because the IPOFile.path was wrong on the previous import03:49
carloscprov: it was for another problem that should not appear now03:49
=== tseng [n=tseng@unaffiliated/tseng] has left #launchpad ["thanks"]
carloscprov: I tested it manually here and it worked03:49
cprovcarlos: then it won't work now too, we have no fix 03:49
carlosbut you just remind me to check it with raw DB access03:49
cprovcarlos: why did it failed in mawson ?03:49
carloscprov: that's a problem only with old imports into dapper03:50
carloscprov: we imported them on december03:50
carloswith wrong settings03:50
Kamionbradb: done, bug 32728; thanks03:50
Ubugtumalone bug 32728 in malone "need search by milestone facility" [Normal,In progress]  http://launchpad.net/bugs/3272803:50
cprovcarlos: the same thing will happen again in production.03:50
bradbKamion: cheers03:50
kikospiv, ping?03:50
carloswe will need to fix them by hand and next time it will work03:50
kikoKamion, bradb: uhm, you can search by milestone today.03:51
carloscprov: but the amount of broken packages is low as we stopped the old import procedure long time ago already03:51
kikobradb, make sure your branch does not regress that, btw03:51
carloscprov: dude, but only for the 10-15 packages that were imported by mistake on dapper03:51
Kamionkiko: how?03:51
cprovcarlos: so, do it first in mawson, I really want to see them fixed03:51
bradbkiko: only on upstreams, as best i can tell03:51
kikoKamion, well, +bugs-advanced03:51
carloscprov: ok, then I will need to run the poimport script on mawson03:51
kikoKamion, however, it is currently broken in production -- fix is in PQM and it will work on tuesday03:52
Kamionkiko: which does not exist on my person page ...03:52
carloscprov: do I have permissions to sudo to the launchpad user as I used to have?03:52
Kamionand the advanced search option from there doesn't have a milestone option03:52
kikodamn03:52
kikobradb is right03:52
kikothat is CRACK03:52
cprovcarlos: nothing changed there, if you had, you still having, try ASAP03:52
=== kiko can't understand why this isn't done in the same way
carloskiko: sorry for ignore your pong... O:-)03:52
kikoblah03:52
bradbkiko: If I land the fix in the next hour or so, do you think it can make it into the next prod rollout?03:53
kikobradb, yes.03:53
carloskiko: I thought I would need an urgent review from you today, but it will not needed as urgent so just ignore the ping ;-)03:53
kikothank god03:53
carloscprov: ok03:53
bradbkiko: ok, working on it now03:53
cprovcarlos: I'm leaving for lunch now, you can do whatever you  want (can) in mawson's DB, when I come back we can redo the uploads and see if it works03:53
carloscprov: ok03:54
carloscprov: just a question... will emails be sent out from mawson?03:54
kikobradb, notify stub of the patch via email03:54
bradbkiko: will do03:54
cprovcarlos: never03:54
carlosok03:55
cprovcarlos: from initZopeless03:55
carloscprov: yes, initZopeless03:55
cprovcarlos: find pitti and tell him we may be able to switch pkgstriptranslation soon (today or tomorrow)03:55
cprovcarlos: will be back in 1 hours, play fair in mawson';)03:56
carloscprov-lunch: ok, thanks!03:56
=== seb128 [n=seb128@ubuntu/member/seb128] has joined #launchpad
seb128hi04:03
kikohello seb128 04:03
kikowhat's up?04:03
seb128how often is update the "assignee" to the bug list?04:03
kikosorry?04:03
seb128I've the feeling that I lot of bug I triage stay unassigned on https://launchpad.net/distros/ubuntu/+bugs?field.searchtext=&orderby=-datecreated&search=Search04:04
seb128open that page04:04
seb128look for #3271804:04
seb128it's "need info (unassigned)"04:04
seb128but the task is assigned to desktop-bugs in fact04:04
kikoseb128, you are in the wrong context, but.. weird04:05
seb128"wrong context"?04:05
kikoseb128, I think it's a bug04:05
seb128it happens a lot, I thought it was some of "bug list is update hourly or so", ie: lag04:05
seb128should I open a bug so?04:05
kikocould it be?04:05
kikobradb, can you check out seb128's question?04:06
seb128I don't know, I'm not an lp dude04:06
kikoseb128, I'd mail the mailing list instead04:06
seb128which one? :)04:06
seb128launchpad-users list ?04:06
bradbkiko, seb128: yeah, it's a bug04:06
kikoor launchpad, it's the same04:06
kikoaha04:06
bradbfor certain statuses, the code is erroneously appending a hardcoded " (unassigned)" to the display value04:07
=== __keybuk [n=scott@82.108.80.242] has joined #launchpad
kikobradb, argh. is that reported?04:09
kikogross04:09
seb128it makes bug triage quite annoying04:09
kikoBjornT, are you saying the merge isn't clearing out session information?04:09
seb128(ie: if it's very low on your priority list you may want to bump it)04:09
bradbkiko, seb128: bug 2967104:09
Ubugtumalone bug 29671 in malone "Listing shows bug "unassigned" even when it's assigned" [Normal,Confirmed]  http://launchpad.net/bugs/2967104:10
bradbI can fix that one today too, easily enough.04:10
kikocool04:10
kikothanks04:10
kikobradb, https://launchpad.net/distros/ubuntu/dapper/+source/nut/+bug/6585/+viewstatus04:10
seb128bradb: thanks dude :)04:10
Ubugtumalone bug 6585 in nut "Hotplug dependency in nut-usb package" [Major,Confirmed]  04:10
bradbno prob04:10
kikobradb, can you confirm that crashes, and see what you can do about it?04:10
carloscprov-lunch: 100% confirmed that the problem is what I told you.04:11
carloscprov-lunch: to do the test you want to do, I need that a package is uploaded twice (that's with different versions)04:11
bradbkiko: That's an easy enough one to fix today too, if you want.04:11
carloscprov-lunch: when you are back we will talk about it04:12
kikobradb, I do -- see if it's reported, btw04:12
kikobradb, BjornT: I'd like us to move forward on making the view/editstatus split go away, and have just +status04:12
kikowhat do you guys think of that?04:12
bradbkiko: Presumably it would be read-only if you can't edit it, right?04:13
kikocorrect.04:13
bradbI think that would be good.04:13
BjornTkiko: yeah, that seems to be the problem, that the session wasn't cleared out. could you file a bug about it? i don't know the procedure of how admins can merge accounts.04:14
bradbkiko: I can't find a bug on the +viewstatus problem. I'll report and take it.04:14
BjornTkiko: yeah, merging the two pages seems like a sane thing to do.04:15
kikosalgado, can you coordinate a bug report with BjornT on this matter? I need to skip out04:15
kikocool.04:15
jblackkiko: ping04:15
salgadoeh?04:15
BjornTbradb: maybe bug 32709?04:15
kikosalgado, merging apparently isn't clearing out session data for authed users.04:15
Ubugtumalone bug 32709 in launchpad "Cannot view status of bug in some cases" [Normal,Unconfirmed]  http://launchpad.net/bugs/3270904:15
bradbBjornT: oh, good call04:16
bradbthanks04:16
salgadoI'll have a look in one second.04:16
kikothanks salgado 04:16
salgadokiko, "Care must be taken to ensure that a branch actively being updated via rsync is not accessed by anything."04:17
salgadofrom https://wiki.launchpad.canonical.com/RocketFuelSetup04:17
kiko:)04:17
jblackkiko: a branch being written via rsync (which lp devs use) is inconsistant during write. Its on the RocketFuelSetup, but probably hasn't been publicised well enough04:17
kikoyeah, I know now :)04:17
jblackI was going to be cute with a sentence with about 20 knows in it. 04:19
jblackKnow that I now know that you didn't know what you now know... 04:19
jblackIs the lack of knowlege about this still a general issue?04:20
jblacksalgado: nope. its not your fault. That document postdates your move to bazaar-ng04:21
salgadowell, at least know we know it and matsubara won't have problems anymore04:25
salgadoand I don't think this is a general issue. it applies to us because we have a prebuilt branch that we sync from rocketfuel and that's shared between me, kiko and matsubara 04:26
kikoyeah.04:26
kikowe suck.04:26
jblackOk. Fine. To the back of the station wagon with you.04:27
=== bradb_ [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
jblackI think its my fault04:27
kikoI ride mexican-style with pride04:29
=== sladen [n=paul@193.28.45.41] has joined #launchpad
sladenis there a feature to automatically refile a malone-originated bug in a remote bugtracker;  is in the support only passive, rather than active04:45
sladenor a generated link to  bugzilla.upstream.com/...filebug?package=$this_one04:45
sladenat the moment I'm doing that by hand and then adding the links.  Having Malone do it would mean that it was linked automatically04:46
bradb_sladen: Malone doesn't support filing the bug upstream, ATM.04:47
bradbIt's not impossible that it might in the future.04:47
sladenbradb: nod. okie, thanks04:47
seb128bradb: it should not do a raw forward imho04:48
seb128bradb: but a pointer to "newbug" page from upstream could be nice :)04:48
bradbI can see issues with reporting dups upstream.04:48
bradbOr, at least, that being more likely to happen when done blindly through Malone.04:48
seb128and that's also quality of what we forward04:48
seb128often we get informations and cleanup the bug before forwarding04:48
bradbseb128: Yeah, the link to the upstream filing page would seem a good idea, at the least.04:49
seb128bradb: BTW how is placed the "fix formating of comments" on your list?04:49
seb128that makes backtraces $)$:! to read04:50
bradbseb128: Not on the list. That seems like an mpt thing, but kiko would know where that stands. I feel your pain.04:50
sladen. o O {if a little more of the source was opened, more people would be willing to fix the small irriating bugs like line-wrapping...}04:50
bradb(It's particularly brutal in a three-column layout.)04:50
seb128like have #7 .... #804:50
seb128function #904:50
seb128kiko: do you know about that? :)04:51
kikolet me see04:51
kikoseb128, indeed, mpt has not given us a good answer on that. is it frustrating you?04:52
seb128yep04:52
kikoI can escalate this and ensure we find a solution if you yell loud enough on the list04:52
kikotip: yelling VERY loud works04:52
bradbsladen: It'd be hard to usefully open source small pieces of LP for this kind of thing. e.g. To usefully Open Source Malone, you'd almost have to open up all of LP for anyone to have a hope of launching the app locally.04:52
seb128kiko: have a look on current comment from https://launchpad.net/malone/bugs/3203404:52
Ubugtumalone bug 32034 in gdm "GDMsetup immediately crashes in Dapper." [Major,Unconfirmed]  04:52
seb128kiko: the valgrind log04:52
seb128"==21124== by 0x8053DA0: setup_user_combobox_list (gdmsetup.c:1619) ==21124== by 0x8053F3D: setup_user_combobox (gdmsetup.c:1649) ==21124== by 0x805C9F8: setup_security_tab (gdmsetup.c:5002) ==21124== by 0x8060380: setup_gui (gdmsetup.c:6270)"04:53
seb128stuff like that04:53
kikoseb128, yeah. horrible. please cry out loud -- people don't believe me when I say our comment formatting is a disaster04:53
seb128are ...04:53
seb128grrrrrr04:53
kikoseb128, there would be a trivial workaround, not perfect, but perhaps:04:53
kikooffering a raw text view of a comment04:53
kikowhat do you think?04:53
seb128why  prevent you to fixing the comment rendering?04:53
seb128bugzilla does it right, it never screw backtraces .. should be possible :)04:53
kikoseb128, there's a couple of issues, IIRC04:53
kikofixed-width fonts04:54
seb128a new line is a '\n' no?04:54
seb128I mean current layout eats '\n' chars or something like that04:54
kikothe current layout is html04:54
kikowith some fancy formatting applied04:54
kikoit works well except when it doesn't04:54
sladenkiko: the HUGE FONT, narrow overfilled screen, urls that get wrapped and broken at '-' dashes and tables that don't fit make MALONE LOOK TEH CRAP.04:54
seb128kiko: anyway a "raw comment" would be better yep04:55
seb128it would give an easy way to get a decent formatting if you didn't get the mail04:55
kikosladen, the narrow screen is going to be fixed soon. the RHS portlet is dead, we just need to bury it.04:55
bradbThe huge font has me using <small> in any page that I want to look non-crap (e.g., the new bug listings, when jamesh gets around to reviewing them.)04:55
kikoand mpt needs to wake up and do it04:55
kiko<small> sucks.04:55
bradbYeah, I should probably use CSS04:55
kikounless it's used to mark up small portions of text04:56
kikosladen, the url wrapping is only in email, IIRC04:56
sladenbradb: how come the font is being touched in the first place, it should be left at the browser default04:56
kikothe tables that don't fit are just fuckage caused by the RHS portlets04:56
bradbsladen: Dunno.04:56
kikosladen, an mpt question04:57
bradbkiko: In this case, it's not really about content not fitting, so much as that our default font isn't very aesthetically pleasing, IMHO, (and apparently, IsladenHO)04:57
kikothere's a way to fix that04:58
kikotrivial it! :)04:58
bradbCan we do that? LP-wide font-size desuckage?04:58
kikowe could, yeah04:59
kikolet's talk about it with mpt04:59
bradbsounds like a plan04:59
sladenkiko: yes, wrapping only in email (the rest of the time it just goes off the RHS of my screen :)04:59
kikoI'm outta here today04:59
sladenbradb: I remember from trying to use 'smaller' and friends ages ago that they /don't work as expected/ and it's better to use 1.2^n scaling:  map(lambda x: 100*(1.2**x), map(float, range(-3,3)))  eg.  58%, 69%, 83%, 100%, 121%, 144% ... which match the "natural" scaling of fonts05:05
sladenwhich I found to work across all browsers/combos05:06
bradbright. i imagine mpt is aware of this too, so he should be able to help us sort it out05:08
sladenbradb: bug fileage time?05:09
sladen(what I don't understand is that all the scaling are < 100% and yet the font is clearly bigger...)05:09
bradbsladen: Sure, a bug should probably be filed. Do you want to file it, or should I?05:10
=== ian_brasil [n=vern@200.242.46.130] has joined #launchpad
sladenbradb: I'm playing with the CSS at the moment, I'll post a suggested patch for people when I'm happy05:15
sladenin the meantime, turning off the CSS altogether is quite refreshing :)05:15
bradbheh05:16
salgadoBjornT, can you give me more context on the "merging apparently isn't clearing out session data for authed users." problem? I can't seem to find anything in my backlog05:18
BjornTsalgado: sure. the oops in question was OOPS-55C32105:20
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/55C32105:20
BjornTwhat seems to have happen is that steve merged tseng's two accounts brandon and brandon-ubuntu, leaving only brandon05:21
BjornTbut it seems at the time that he was logged in as brandon-ubuntu, and since the session information didn't get cleared, he stayed logged in, and did stuff without having a preferred email address05:21
BjornTat least that's the theory. is it clear?05:22
salgadoas clear as it gets. :)05:23
salgadothanks BjornT, I'll check some things and will file a bug on that05:23
BjornTcool, thanks05:23
BjornTsalgado: how confident are you of the test coverage for person vocabularies? :) if i change __contains__ not to issue any db queries, should i add some more tests?05:28
cprovcarlos: duderino, what's up ?05:30
carloscprov: hi05:30
carloscprov: I found an easier way to show you that it's working05:31
cprovcarlos: show me ;)05:31
carloscprov: https://dogfood.ubuntu.com/rosetta/imports05:31
=== carlos is loading the page too... but seems to be a bit slow...
cprovcarlos: ehe, kinda slow at mawson (huge)05:32
cprovcarlos: yup05:32
carloscprov: is it a problem with the page? or the server?05:32
cprovcarlos: I've loaded it normally05:32
carloscprov: The Finnish entry was moved automatically to the first list of resources ready to be imported when I fixed manually the path05:33
cprovcarlos: mawson isn't fast, page isn't small05:33
carlosso the system was able to decide where should it go05:33
cprovcarlos: didn't get it very well05:33
carloscprov: so the problem was, as I supposed, the POFile.path field05:33
cprovcarlos: what did you fixed 05:34
carlosthat is empty always05:34
carloscprov: I did an update of that row05:34
carlosfixing the path05:35
cprovcarlos: what should it be ? rather than empty ?05:35
carlosthe same as the file being imported05:35
cprovcarlos: so, do we need to do those fixes in production in order to get it working properly ?05:36
carloswhen we fix it "manually" using the web interface, next time it's already correct so the import will be done automatically05:36
carloscprov: no, the website allows us to do that05:37
cprovcarlos: uhm .. that's good05:37
carloscprov: so I would say, go, go, go!05:37
carlosbut pitti is not online05:37
cprovcarlos: eheh, call him05:37
carloscprov: I was waiting for you05:37
carlosto know if you know why is not online ;-)05:38
carlosbefore calling05:38
cprovcarlos: he just left before me, don't know why 05:38
carloscprov: anyway, is all ready on your side to do the pkgstriptranslations update?05:38
carlosok05:38
cprovcarlos: what time is there, in Germany ? +1 by you ?05:39
carloscprov: I want to be sure so I call Martin to request him the upload ASAP05:39
carlossame as here05:39
carlos17:3905:39
sladenbradb: btw, do file it and post me the #, that way it'll get your thoughts too05:39
cprovcarlos: yes, call him and say the words: DO IT ;)05:39
carlos:-D05:39
carloscool, thanks05:39
cprovcarlos: I hope i can assist the first run of it today yet05:40
carloscalling....05:40
cprovcarlos: thank you, got the job organized and done just in time .05:40
carlosno answer05:40
=== carlos checks the phone number
carloscprov: seems like he left already for the weekend05:45
=== carlos mails pitti as seb128 suggests
cprovcarlos: ohh this is bad, can we upload his last pkg from p.u.c ?05:46
mdkecprov, did you see my breezy-updates bug? is there any more info I can give you on it?05:47
carloscprov: asking atm05:47
salgadoBjornT, I've refactored these vocabs a few times already (and the last time it was almost a complete rewrite), and I haven't seen bugs reported on them, so I guess the test coverage is good05:48
cprovmdke: sorry, not yet, one sec and I can handle it for you05:48
BjornTsalgado: ok, good. i'll let you review the changes later.05:50
mdzbradb,Kamion: I didn't create any milestones05:50
bradbsladen: bug 3274905:51
Ubugtumalone bug 32749 in launchpad "Launchpad fonts are too big" [Normal,Confirmed]  http://launchpad.net/bugs/3274905:51
salgadoBjornT, sure05:51
bradbmdz: right, I don't know who added the "dapper" milestone, but oh well05:52
sladenbradb: groovy05:53
=== bradb goes to get food, bbl
=== bradb &
mdzbradb: the sab?05:55
bradbcould be05:55
carloscprov: it's done05:56
carloscprov: Martin will do the upload as soon as he arrives home05:57
carloscprov: in 3 hours or so05:57
carloscprov: will you be online at that time?05:57
cprovcarlos: very good carlos 05:57
cprovcarlos: yes, but it will take another 3 hours to be compiled and published 05:57
carlosoh05:58
cprovcarlos: I will take care of it properly for us 05:58
cprovcarlos: don't worry05:58
seb128bradb: "dapper" was created before the bugzilla import05:58
carloscprov: well, I should be online at that time (unless my DSL line fails again)05:58
seb128and bugzilla import created the "ubuntu-..."05:58
cprovcarlos: let's see05:58
carloscprov: thanks for all05:58
carloskiko_smtb: did you see it?05:59
carlossmtb.... what's that?05:59
carloscprov: I'm going to update the status answering your email05:59
=== rraphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
cprovcarlos: please, CC LP don't forget06:00
carloscprov: sure06:00
cprovcarlos: something related to "montain bike", ohhh, crap !06:01
carlos:-P06:01
=== rraphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== WaterSevenUb [n=WaterSev@195-23-238-209.nr.ip.pt] has left #launchpad ["Fui]
=== dsa [n=dsaa@210.1.82.61] has joined #launchpad
carloscprov: done06:13
=== hannosch [i=hannosch@e176111247.adsl.alicedsl.de] has joined #launchpad
=== thierry [n=thierry@modemcable023.222-70-69.mc.videotron.ca] has joined #launchpad
cprovcarlos: good06:28
=== Keybuk [n=scott@82.108.80.245] has joined #launchpad
seb128bradb: around?06:35
seb128bradb, jamesh: could we get https://bugzilla.ubuntu.com/show_bug.cgi?id=13169 imported?06:37
Ubugtuubuntu bug 13169 in nautilus "evil breezy unmounting action due to async filesystem" [Normal,Upstream]  06:37
seb128bradb: 06:40
seb128  32620. poor string in the window switcher button06:40
seb128in update-manager (Ubuntu), unconfirmed 06:40
seb128why doesn't it say it's assigned to mvo?06:41
seb128same issue as for the unconfirmed pointed before?06:41
=== bradb returns
bradbseb128: It seems as though there were already two bugs opened that were watching that bug: https://launchpad.net/malone/bugtrackers/ubuntu-bugzilla/1316906:53
bradbWhich, I guess, is why it wasn't imported.06:53
bradbseb128: So, you want it imported anyway?06:54
seb128yep06:54
seb128those 2 bugs are useless, the bugzilla one has a lot of comments compared to them06:54
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
seb128one of the 2 is an another issue06:55
bradbok, i'll mail james06:55
seb128it says to a comment:06:55
seb128"06:55
seb128Bug #5049 (the eject failure) was fixed recently in Dapper.06:55
seb128As for the progress bar, this is https://bugzilla.ubuntu.com/show_bug.cgi?id=13169."06:55
Ubugtumalone bug 5049 in kaffe "kaffe: merge new debian version" [Normal,Fix released]  http://launchpad.net/bugs/504906:55
Ubugtuubuntu bug 13169 in nautilus "evil breezy unmounting action due to async filesystem" [Normal,Upstream]  06:55
seb128I don't really why you did import them in that case, that's the first time06:55
seb128since we worked with bugzilla before the import, the bugzilla is almost always the useful one06:56
bradbre: the bug not showing mvo being assigned, it's caused by the same bug you mentioned earlier. that code is just b0rked.06:56
seb128and the launchpad one is a one comment "known as bugzilla nnn"06:56
seb128bradb: ok, thank you :)06:56
BjornTsalgado: ping07:01
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
bradbseb128: Just to confirm, you'd ideally like us to have imported ALL Bugzilla bugs, right? Including the ones that already had manually added watches in Malone at the time the import was run?07:01
salgadoBjornT, pong07:02
seb128bradb: yep07:02
BjornTsalgado: could you please review https://chinstrap.ubuntu.com/~dsilvers/paste/filezhRe0X.html, it's quite small. it fixes bug 2045 and makes sure that 'obj in person_vocabulary' doesn't issue a db search.07:03
salgadocome on Ubugtu, what's bug 2045?07:04
salgadoah, right. it's private07:04
salgadoBjornT, Hmmm, I guess this is a good oportunity for me to read all 7 bugmail from that bug I have in my inbox. :)07:06
bradbseb128: You've got a copy of the email I sent. Thanks for letting me know of the problem.07:06
BjornTsalgado: could be :)07:06
seb128bradb: did get the mail yet, but thank you :)07:06
salgadoBjornT, ValidPersonOrTeamVocabulary.__contains__() uses the "foo_object in Foo.select(query)" syntax, which will issue a single "select * from foo where <query> and id = foo_object.id". I guess you knew that already, but you think it's worth adding around 30 lines of code to avoid a query that should run almost instantaneously?07:26
salgado(or maybe you have another reason that I can't see for the changes in the __contains__ method)07:28
=== seb128_ [n=seb128@ANancy-151-1-77-107.w81-50.abo.wanadoo.fr] has joined #launchpad
BjornTsalgado: well, my main motivation was to minimize the number sql queries issued. but as you say, the avoided queries are probably quite quick. if you want, i can revert that part.07:37
salgadoBjornT, I'd like to check with stub first if avoiding these queries are really worth something, because with the changes you've done, we now have two different places in each vocab where we need to code the extra logic for that vocab, and it used to be only one07:39
salgadowhat do you think of holding the __contains__() changes until we talk with stub?07:40
BjornTsalgado: sure, sounds sane.07:41
salgadoBjornT, so, you just removed the _getFormInput(), which was the one calling vocabulary.search()?07:48
BjornTsalgado: yes. then i modified popup.pt, so that the select box with search results get shown only if there's not an exact match.07:49
salgadoright, and why do we need to make IHugeVocabulary extends IVocabularyTokenized?07:50
BjornTsalgado: it's because SinglePopupWidget expects the vocabulary to provide IVocabularyTokenized. that dependency wasn't added with this patch, just added it since it should be like that.07:55
salgadoah, right07:55
salgadowell, I don't have anything to complain about that patch. you can merge wirh r=salgado07:56
BjornTthanks07:57
dilysMerge to devel/launchpad/: [r=salgado]  fix bug 2045, prevent IHugeVocabulary widget from performing searches if it gets an exact match. only display search results if there's no exact match. should prevent timouts from happening when assigning someone to a bug, spec, etc. (r3195: Bjorn Tillenius)08:35
bradbHm, that merge seems like a bug, to me.08:37
bradbAn exact match != the desired match necessarily.08:38
bradbI vaguely remember there being a bug open on that behaviour, but maybe I'm remembering incorrectly.08:40
salgadobradb, https://launchpad.net/products/launchpad/+bug/204508:42
=== bradb was just looking at that, yeah
salgadoI pointed out what you said too, but this is a temporary solution08:42
salgadoand I don't think it'll cause us trouble right now08:42
bradbUsers will decide, I guess. :)08:45
bradban example of how one could go easily wrong would be to search for, literally, "foo"08:47
bradbthere's an actual "foo", but there are also dozens of variants08:47
salgadothe same with 'carlos'08:48
bradbindeed08:48
carlosno dudes, I'm unique!08:48
carlos:-P08:48
bradbheck, even bradb08:49
BjornTis it just me, or has the test suite become quite a bit slower?09:17
BjornTit takes twice as long for me to run all the functional tests, compared with a few weeks ago09:18
BjornTbradb: re bug 2045, if the users want to search for a value, why not use the search form? ;)09:19
bradbBjornT: That makes logical sense, but the conversation between user and system is more a mutual co-operation trying to achieve some goal, rather than a set of discrete correct or incorrect actions. Users actions are often approximations of what they really mean.09:21
bradbBut, maybe I misunderstand the fix then: if it only specifically chooses "foo" on exact match when that was typed directly into, say, +editstatus, then it's probably ok.09:22
bradbFrom "only display search results if there's no exact match", I thought that was referring to the search form.09:24
BjornTbradb: exactly, the merge message is a bit ambiguous. and by looking at the oops related to that widget, that is what most users expect09:24
CarlFKwhat are the chances of getting the "add attachment" option to take a URL so that I can post files that arn't on my file system, like http://dev.personnelware.com/carl/temp/Feb24/ubuntu-installs/a/fdisk.txt09:24
BjornTbradb: yeah, that sentence referred to the search results that are displayed if you don't use the "Choose" popup search09:25
CarlFKand also, allow posting multiple files as a group so that I can stop using the file name as both the title and comment 09:25
bradbBjornT: ok, n/m then :P09:26
CarlFKis there a proper place to post sugestions like these ?09:27
BjornTCarlFK: the easiest way is probably to file bugs at https://launchpad.net/products/malone/+filebug09:30
CarlFKk - thanks09:31
CarlFKI like this - don't have to pick a package name ;)09:32
CarlFKworst part about reporting ubuntu bugs 09:32
mptYou don't have to pick a package name when reporting Ubuntu bugs either09:33
CarlFKhey look at that... 09:33
mptbug 3163809:33
Ubugtumalone bug 31638 in malone ""Package Name" field when reporting bug should be more obviously optional" [Normal,Confirmed]  http://launchpad.net/bugs/3163809:34
CarlFKgood deal09:34
=== lbm [n=lbm@x1-6-00-13-10-7a-d1-e4.k233.webspeed.dk] has joined #launchpad
Keybukbradb: does Malone *have* to have an "Are you sure?" for Unsubscribe09:51
Keybukit's damned annoying09:51
Keybukcouldn't it just do it, on the basis that subscribing again is easy09:51
bradbKeybuk: from the man who brought you three column layouts...09:53
bradbno, we needn't ask that annoying question09:53
seb128do we need it for subscribing too? :)09:54
seb128if I click to subscribe on a bug, that's because I want too09:55
bradbI wasn't even aware of that, because I never have to sub/unsub, but those annoyances should be removed, yeah.09:56
bradbBjornT: I wonder if you might have time to review a patch for an urgent bugfix? (5 files changed, 45 insertions(+), 36 deletions(-))10:00
bradbIt's just adding the milestone search widget to the person and distro search forms.10:00
BjornTbradb: i might have time, put it up somewhere and i'll have a look at it10:02
bradbBjornT: ok, thanks, just quickly cleaning up the diff10:02
BjornThmm, i wonder if someone would object if i suggested reverting the patch for testing individual pagetest stories :)10:04
bradbBjornT: https://chinstrap.ubuntu.com/~dsilvers/paste/fileQXuk4G.html10:04
seb128bradb: https://launchpad.net/malone/bugs/3150610:05
Ubugtumalone bug 31506 in malone "Remove "are you sure" page from subscriptions" [Wishlist,Unconfirmed]  10:05
bradbBjornT: it's pretty annoying, yeah10:05
seb128bradb: you should read the bugs so :p10:05
=== bradb is over a month behind on bugmail :/
bradbBut the triagers are meant to ensure timely response, and alert me of any critical issues while I ignore bugmail.10:07
BjornTi'm mostly concerned with that the time it takes to run 'python test.py -f canonical.launchpad' almost doubled for me :(10:07
bradbBjornT: Wow. I usually run the tests with pqm. (So yes, I usually wait until it's reviewer stamped, to save time.)10:08
seb128bradb: do you get that many bugs?10:08
seb128bradb: I got 847 mails for launchpad since monday and I think I read almost all of them ...10:09
bradbseb128: Since Jan 20th, my bugmail folder has 2500 mails. Not all are Malone-specific, of course.10:09
seb128replying or fixing is another topic :)10:09
seb128bradb: 4295 since jan 20th :p10:09
bradbseb128: Try confining your bugmail reading to within Normal Working Hours :P10:10
seb128anyway that's not the topic, I do think you should read them even if you don't reply, that's the best way to know what issues people have around10:10
bradbyeah10:10
seb128bradb: no way to do that, I usually use saturday,sunday afternoon without IRC to catchup10:11
bradbexactly10:11
seb128which I agree should not be required10:11
seb128and I would not ask somebody else to do the same :)10:11
lifelessBjornT: due to the db isolation ?10:11
lifelessBjornT: yes, that patch should not be reverted. There was a bug open on the fact that they were /not/ isolated.10:11
seb128bradb: anyway, I've noted, better to let you know on IRC what bothers me so :)10:12
seb128(I already sort of do that ...)10:12
BjornTlifeless: i would guess so. i compared running tests on a fresh rocketfuel, and on a rocketfuel with r3146 and r3131 reverted.10:13
bradbthanks. i'll aim to clear out my bugmail folder next week though too. it's getting pretty crazy.10:13
seb128bradb: dholbach speaks about "mail terror" from launchpad now :)10:13
seb128he had like 1800 bug mails not read this morning he was trying to catchup a bit today10:14
BjornTlifeless: sure, it shouldn't be reverted. but something should be done to make it faster to run the tests.10:14
bradbseb128: whoa10:14
seb128bradb: how is the "stop sending 5 mails when somebody edits a bug" placed on the "to fix" list?10:14
seb128(which is a side effect of the: "you have to use 5 different pages to edit a bug")10:15
bradbseb128: Still a while to go on that one, I think. Mainly to convince the appropriate people (i.e. person) that five different pages to edit a bug sucks.10:15
seb128not easy person to convince ... :)10:16
dilysMerge to devel/launchpad/: Re-applying patch 4845 as I inadvertently reversed it while merging patch 31872 r=bjornt (r3196: Diogo Matsubara)10:16
bradbMind you, kiko sent something to Mark a couple days ago talking about prioritizing changing status on the bug page, so maybe it's not /that/ far off. (Like, say, within a month.)10:16
bradbThat kind of change would kill a few birds with one stone: reduced bugspam, faster triage, less annoyance, etc.10:18
mptKeybuk, add your name to the BugWorkflow spec if you like it10:20
mptit makes subscribing/unsubscribing a checkbox, with no extra page10:20
seb128bradb: right, would be nice10:20
lifelessBjornT: well during the dev cycle you almost certainly want to be running only malone tests for instance10:20
lifelessBjornT: so performance is important, but we're so far out of the 'instant gratification' zone at the moment, I think the time here isn't all tht important compared to isolation.10:21
BjornTlifeless: how do i run only malone test? and i do think time is somewhat important. previously it took less than 15 minutes to run, it was ok, i could run them during a workrave, and while reading some email or looking and looking at the diff.10:23
BjornTlifeless: now it takes 25 minutes, and that makes it a lot harder to find time to run them.10:24
lifelessBjornT: give me an example malone story or standalone name10:25
lifelessspiv: I think we need to do that zope 'when I say search in the test name I mean the whole path fix'10:25
lifelessBjornT: I understand. 15 minutes is 14 minutes too long IMO ;010:26
lifelessBjornT: bzr runs 1500 tests in 3ish minutes. which is getting a bit slow.10:27
BjornTlifeless: oh, you mean only one pagetest story. i actually seldom have the need to run a single story. when i develop i mostly some doc/foo.txt. when i'm done i run all functional tests to see whether anything broke due to my changes.10:30
lifelessBjornT: I meant all tests with malone in the name actually, but the zope runner is rather borked.10:31
lifelessBjornT: it decides it knows more about the meaning of test python paths than we do.10:32
BjornTlifeless: well, quite often i do changes that can affect more than malone, so i need to run all tests. but sure quite often it's enough to run only part of the test, but with the current structure it's quite impossible.10:33
lifelessBjornT: yeah. best you can do is a path prefix like test.py -f lib.canonical.launchpad.doc10:34
BjornTwe should probably work on grouping the standalone tests, so that tests that are related to each other are in the same story10:34
lifelessthat would reduce the spurious setup time10:34
lifelessmyself, I'd like us to stop testing postgresql, we know it works ;)10:34
BjornTyeah, that's true :) although iirc, not using a proper db for testing SQLObject is quite hard, it requires a lot of work to get it to work using stub classes10:36
BjornTit would be great if you got it to work though :)10:36
BjornTbradb: tell me about the changes to MilestoneVocabulary, why is it necessary?10:37
bradbBjornT: because of showing the milestone widget on the people pages. we have to show all the milestones on those pages.10:38
bradbit's a crap UI, tbh, but it's the best thing I can think of for an urgent fix, without getting overly complicated and clever trying funky heuristics on the Bug and BugTask tables to "guess" what milestones the user wants to choose from10:39
=== mpt_ [n=mpt@222-154-152-43.jetstream.xtra.co.nz] has joined #launchpad
mpt_bradb, the batxhed notification (bug 1350) could be implemented independently of any merging of bug pages.10:45
Ubugtumalone bug 1350 in malone "Change notifications should be batched" [Wishlist,Unconfirmed]  http://launchpad.net/bugs/135010:45
cprovguys, see you tomorrow10:45
bradbmpt_: It's a lot more work though too.10:45
mpt_true10:46
BjornTbradb: hmm, is there really a need to search by milestone on the person pages?10:48
bradbBjornT: An urgent need, yeah.10:48
BjornTbradb: really, what is the use case? why can't they search in a context?10:49
mpt_BjornT, I thought that was what kiko suggested Kamion do earlier10:49
BjornTwasn't that searching for milestones for a distribution?10:50
bradbBjornT: That would require them knowing that they're supposed to switch to the context +bugs, for one. Ubuntu devs tend to drive Malone from their FOAF pages.10:50
bradbThere's no logical reason for them /not/ to be allowed to search by milestone on their personal reports, other than that "it's harder to implement", which is usually an excuse reserved for only the most painful UIs.10:52
BjornTbradb: ok, i guess it's ok for now. i do think that the comment you added deserves to be an XXX. and don't you think it would be wise to use shortlist, so that we get a warning when the number of milestones get too high, so that we are forced to do something about it?10:53
bradbBjornT: Oh, right, for some reason I thought shortlist raised an exception.10:54
bradbA warning would be useful.10:54
BjornTok, fix those two things and you can merge10:55
bradbgreat, thanks for taking some time out of your friday evening to look at it10:56
mpt_oh10:58
mpt_Different things have different sets of milestones10:58
mpt_and people can be assigned to bugs in different things10:58
mpt_So what milestones do you offer on a person's bugs page...10:58
bradbyeah, like i say, clever heuristics == pain10:59
bradbright now, i just put them all in there, so they can at least do their job10:59
mpt_All milestones for all distributions and products in Launchpad?10:59
bradbthis problem is much deeper than just milestones. e.g. a bug never disappears from a user's subscribed bugs report until it's fixed in all contexts.11:00
bradbmpt_: yeah11:00
bradbso, it'll say "Ubuntu: dapper", or "bzr: 1.0", etc, in the list11:00
bradblike i say, crap UI, but I can't think of anything better that can land within the next 15-30 mins11:00
matsubaraSee you guys on wednesday. Here I go to Carnival! Take care.11:03
jblackI don't see a link on https://lists.ubuntu.com/mailman/listinfo/ for lp-devel. Do we have an archive?11:14
elmolists.canonical.com11:15
elmoor just append 'launchpad/'11:16
jblackthanks\11:17
=== SteveA [n=steve@72-254-148-105.client.stsn.net] has joined #launchpad
dilysMerge to devel/launchpad/: Fix https://launchpad.net/products/malone/+bug/5757 (OOPS: requesting an upstream fix doesn't check input for duplicates) r=kiko (r3197: Diogo Matsubara)11:20
=== mpt [n=mpt@219-89-153-97.jetstart.xtra.co.nz] has joined #launchpad
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has left #launchpad []

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