/srv/irclogs.ubuntu.com/2006/02/13/#launchpad.txt

=== andrewski_ [n=andrew@pool-141-158-41-67.phil.east.verizon.net] has joined #launchpad
andrewski_what does it mean to "sign" a code of conduct?  is that like saying that you've read it?  because i know very little about OpenPGP and such, but i'm not sure if signing it is a measure of good faith or somesuch.12:18
Kaiser_Sleepsandrewski_: it means you read it and agree to abide by it, then you can downlod it, clearsign it with gpg,  and paste the output into that window12:21
andrewski_Kamping_Kaiser: ah, ok.  so i need to download it first to sign it, eh?  could that be made a bit clearer on the site?12:21
Kamping_Kaiserandrewski_: http://ubuntu.pastebin.com/54150712:22
Kamping_Kaiseryeh. i commented yesterday12:22
andrewski_Kamping_Kaiser: ah, ok. :)  cheers.12:22
Kamping_Kaiser:)12:23
andrewski_should i set up gpg first?  do i need to "personalize" it?12:23
Kamping_Kaiseryou need a gpg key12:23
Kamping_Kaiserif you have that, you can sign it12:23
Kamping_Kaiseroh12:23
andrewski_ah, ok.  i think i have one on my desktop... lemme fire up ssh. :)12:23
Kamping_Kaiseryou need to have your key registed with launchpad IIRC12:23
andrewski_ok, i see that page too.12:24
andrewski_Kamping_Kaiser: any idea which key i'm supposed to send with 'gpg --send-key'?12:28
Kamping_Kaiserhm? wichever you want to use in future work with launchpad/other ubuntu stuff12:29
andrewski_ok, so pubring.gpg is fine?12:30
andrewski_(like i said, i know little about gpg. :)12:30
Kamping_Kaiserandrewski_: i know little as well :) i had to get help setting this up myself :$, and i cant remember12:30
andrewski_yeah, it doesn't like pubring.gpg.  i'll check the wiki for something.  how do i go about making a suggestion for launchpad stuff?12:31
Kamping_Kaiseri commented here. 12:32
Kamping_Kaiserjust a minute, I'll try find the gpg page12:33
andrewski_sounds good to me.12:33
Kamping_Kaiserhttps://wiki.ubuntu.com/GnuPrivacyGuardHowto - look at point 5.1 as well :)12:34
andrewski_Kamping_Kaiser: thank you much; you got to it faster than i. :)  i must be off, but i'll stick around to see if anyone backtracks to read our comments.  tsch!12:36
Kamping_Kaiserok mate :)12:38
=== lfittl [n=lfittl@83-65-244-232.dynamic.xdsl-line.inode.at] has joined #launchpad
=== mpt_ [n=mpt@219-89-149-187.jetstart.xtra.co.nz] has joined #launchpad
ddaalifeless: I saw you just sent a mail about branch status feedback01:15
ddaalifeless: here I just sent an update to the "runnig baz2bzr periodically" task, giving a lot of detail to implement the daily error reporting for buildbot.01:16
spivddaa: Did you see my mail about the buildbot tests?01:24
ddaaspiv: yes, and I wondered...01:24
ddaaspiv: did you make all the changed you mentioned?01:25
spivddaa: Yes, whY?01:25
ddaait does not read very obviously01:26
ddaa"the CVS version with the SignalMixin is probably straightforward to backport, if we want it (although I'm not convinced that that's the best way either...)."01:26
spivI made sure to run "make check" in sourcecode/buildbot before reporting my results :)01:26
spivOh, right.01:26
spivNo, I didn't try that.01:26
ddaa"Here's a diff, against rocketfuel"01:26
ddaa"Yeah, I think that test is totally irrelevant to us, so skip it."01:27
spivSorry, I sent a diff of what fixes it now (partly by skipping broken tests).01:27
ddaaSo I wondered what was stopping you  from sending the changes to rocketfuel.01:27
spivAnd also suggested ways to unbreak some tests.01:27
spivI wanted to run it by you, seeing as you've been working in this area.01:28
ddaaspiv: I am currently beyond suggestions about anything not involving drugs, sex, and the importd->bzr transition :)01:28
spivI'm happy to send the merge request right now if it all makes sense to you :)01:28
=== sabdfl [n=mark@ubuntu/member/pdpc.silver.sabdfl] has joined #launchpad
spivOk :)01:28
sabdflhey all01:28
sabdfljust got a timeout on /products/gnomebaker/ which was very surprising01:29
ddaaspiv: please do whatever you think is right to let me merge my pending cscvs changes01:29
sabdflis something going on with LP performance?01:29
spivNot that I know of, there was a rollout yesterday though.01:31
spivWhat's the OOPS id?01:31
ddaasabdfl: I received your mail about the pybaz packaging problem01:32
ddaasabdfl: you realize the problem is that pybaz and python-baz are just two packages for the same bit of software, don't you?01:32
sabdflspiv: lost the OOPS ID when I reloaded, but it just happened a minute ago01:32
sabdflddaa: in that case, should they not Provide: the same thing? Because installing config-manager pulls in one, and something else pulls in the other.01:33
spivsabdfl: Ok, I'll see if I can find it.01:33
ddaasabdfl: obviously, the fact they package the same thing shows that the two packagers did not communicate...01:34
ddaabesides, it looks like only pybaz is on dapper...01:36
sabdflddaa: i'm running dapper01:37
spivsabdfl: OOPS-39A1301:37
sabdflthanks spiv. anything seem out of the ordinary?01:37
spivAll fine until a SELECT COUNT(*) with a subselect.01:38
spiv(on BugTask, Bug, subselect on Bug, BugSubscription, TeamParticipation)01:38
spivI wonder if the offending SELECT COUNT(*) will go away next rollout with jamesh's __len__ fix.01:39
spivBut the page usually is very fast, so I guess there must have been some contention.  I'll get stub's opinion when he wakes up.01:40
sabdfljust got another timeout01:43
sabdflOOPS-39D2301:43
sabdflthis time on a rosetta page01:43
sabdflwe need to make eliminating these timeouts amajor goal of ours01:43
ddaaI'd bet it's browser/bugtask.py:102801:43
ddaa        tasklist = self.context.searchTasks(params)01:43
ddaa        return tasklist[:quantity] 01:43
=== Mez [n=Mez@ubuntu/member/mez] has joined #launchpad
ddaathat's run twice by the bugtarget-portlet-latestbugs.pt01:45
ddaabut spiv is right, this page is usually very fast, so it's probably a contention with something else that's really slow01:45
dilysMerge to devel/launchpad/: [r=kiko]  extend fmt:text-to-html to turn OOPS codes into links for developers (bug #30645) (r3097: Dafydd Harries)02:11
Kinnisonhey sabdfl 02:13
Kinnisonhow's the tour?02:14
sabdflhiya Kinnison, happy your baby is in production?02:14
Kinnisondude you have no idea how high I've been flying02:14
sabdfltours going well, not much sleep but otherwise wonderful reception and lots of work02:14
KinnisonI think I've given C1 a run for its money02:14
=== niemeyer [n=niemeyer@200-103-135-181.ctame7006.dsl.brasiltelecom.net.br] has joined #launchpad
sabdflwondered what the hell that was outside the window02:14
=== Kinnison grins
=== Kinnison is just finishing the "how to ensure drescher is happy during production rollouts" document for stuart
Kinnisonwe did a completely smooth rollout today together02:15
Kinnisonso I'm turning my notes from the process into a document for him02:15
sabdflv cool02:20
sabdflonce we get cron.daily's equivalent back down to once per 30 minutes we will have full equivalence to dak, AIUI02:20
KinnisonPretty much02:20
Kinnisonand we're down to an average 20 minute run02:20
KinnisonJames and I seriously improved the dominator's performance on Saturday02:21
=== jsgotangco [n=jsg@210.4.38.43] has joined #launchpad
sabdflyou beat the dominator into submission. i like it.02:34
=== sabdfl [n=mark@ubuntu/member/pdpc.silver.sabdfl] has left #launchpad []
=== stub [n=stub@gb.ja.97.164.revip.asianet.co.th] has joined #launchpad
=== Kinnison heads to bed
Kinnisonstub: I've sent you a doc02:57
KinnisonOr at least I thought I had02:58
Kinnisonit has bounced02:58
Kinnisongrah02:58
stubBut I'm fine!02:58
Kinnisonemail?02:58
Kinnisonwhat's your @canonical.com address?02:58
stubstuart.bishop@canonical.com02:59
Kinnisonsent03:00
Kinnisonand now I sleep03:00
Kinnisonciau03:00
stubNight03:03
=== makaveli [n=nnn@c-69-139-7-167.hsd1.pa.comcast.net] has joined #launchpad
=== makaveli [n=nnn@c-69-139-7-167.hsd1.pa.comcast.net] has left #launchpad []
=== stub [n=stub@gb.ja.97.164.revip.asianet.co.th] has joined #launchpad
spivstub: sabdfl got OOPS-39A13, but usually that page renders quite quickly.  I guess it was just contention on the tables in the query?03:29
stubI guess03:30
stubNot the prettiest of queries03:31
spivstub: Was the librarian upgraded in yesterday's rollout?03:33
stubspiv: Yes03:33
spivOk, so it now has the database name checking in it.03:34
spivSo it's a matter of upgrading all the clients :)03:34
stubHmm... it should enforce that.03:37
spivAt the moment, it's optional, depending on the client sending the relevant header.03:38
stubAll the time critical stuff is using the new code - anything else that blows up is a backend system that can afford some downtime03:38
spivI should make it log a warning everytime a client doesn't send that header.03:38
spivHeh.03:38
stubLogging warnings is only useful if someone is watching the logs :)03:39
spivWell, the theory would be that we could look back at the logs to figure out if there's any unupgraded clients left. 03:41
spivstub: But here's a 10-second, untested patch to enforce it: https://chinstrap.ubuntu.com/~dsilvers/paste/fileOMXlxG.html03:42
lifelessspiv: exponential degradataion on the warnings :)03:45
lifelessspiv: or for cuteness, have a web page on the librarian showing the old clients seen in the last day03:45
spivlifeless: ...or, someone could grep the log files that are rsynced to chinstrap already.03:46
lifelessspiv: I did mention cute didn't I ?03:46
spivEven if it is the boring way :)03:46
stubWe don't want old clients. Bad enough tracking down bugs in uptodate clients. Just refuse to talk to them.04:04
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
stublifeless: Can you install config manager into ~pqm/bin on balleny? (or tell me where to find it)04:58
=== makaveli [n=nnn@c-69-139-7-167.hsd1.pa.comcast.net] has joined #launchpad
jameshstub: "bzr branch http://www.robertcollins.net/config-manager/trunk/"05:49
jameshstub: would you be able to delete bugwatch id 353?  It should have remotebug='-42' and bug=572306:00
mpt_jamesh, how often are the Oops reports going to be sent?06:11
jameshmpt_: I had the script disabled while I was fixing it up to do a short summary to the list + long summary on the web06:12
jameshmpt_: I should have that going again today06:13
mpt_great06:13
stubjamesh: Deleted06:26
jameshstub: thanks06:26
mpt_What's up with staging?06:44
mpt_I get Bad Gateway errors for staging.ubuntu.com and staging.launchpad.net06:44
jameshthe staging config might need updating06:46
jameshboth bradb and daf added some required config keys that don't seem to be in configs/staging/launchpad.conf06:47
jamesh(or production[12] )06:47
dilysMerge to devel/launchpad/: [r=jamesh]  Export GPG fingerprints in person/$name/+rdf, to make it easier for people to e.g. create GPG keyrings for a team. (r3098: Andrew Bennetts)07:08
=== koke [n=koke@ubuntu/member/koke] has joined #launchpad
lifelessstub: ~/source/ has it all08:10
stublifeless: ta08:10
stub(I've just been using ~pqm/bin)08:10
SteveAhello08:15
stubyo08:18
SteveAstub: do you recall what the conflicts were in merging jamesh's __len__ stuff?08:19
stubSteveA: Nope. I was busy trying to rollout at the time :)08:20
stubSteveA: I'll be doing some cherry picks once a patch of mine lands and will check it out then08:22
=== tambaqui [n=patricia@200-208-49-85-mns.cpe.vivax.com.br] has joined #launchpad
=== carlos [n=carlos@84.76.255.40] has joined #launchpad
carlosmorning08:31
dilysMerge to devel/launchpad/: [trivial]  Make some config sections not required to ease maintenance (r3099: Stuart Bishop)08:41
lifelessstub: for pqm I use that too08:52
SteveAspiv: OOPS-39A13 is a __nonzero__, not a __len__, but jamesh's patch converts __nonzero__ to use a select-limit query rather than a count().08:58
sivanghowdy all09:02
SteveAhi sivan09:02
jameshSteveA: I don't think the fix to make __nonzero__ not do a count might have missed the rollout09:07
jameshSteveA: my original __nonzero__() implementation was erroneously doing a count still09:08
SteveAjamesh: yes, i think they both missed the rollout09:08
SteveAjamesh: stub said he will look at getting the rest cherrypicked later09:08
lifelessis __nonzero__ invoked on 'if foo:' ?09:09
jameshlifeless: yes09:09
jameshiirc, it will invoke __nonzero__ then __len__ then check if it is not None09:10
SteveASeveas: hello09:13
BjornTSteveA: hi. do you have a minute to discuss a fix to bug 3796?09:21
Ubugtumalone bug 3796 in malone "Duplicated bugs still show up as New in a list of bugs (also affects the Latest bugs portlet)" [Normal,Confirmed]  http://launchpad.net/bugs/379609:21
SteveABjornT: yes, after a workrave09:27
BjornTok09:27
jameshBjornT: I filed https://launchpad.net/products/malone/+bug/30307 a little while ago.  Does it sound like a good idea for searching?09:30
Ubugtumalone bug 30307 in malone "omit_dupes and searching" [Normal,Unconfirmed]  09:30
SteveABjornT: for bug 3796, i agree with you09:32
Ubugtumalone bug 3796 in malone "Duplicated bugs still show up as New in a list of bugs (also affects the Latest bugs portlet)" [Normal,Confirmed]  http://launchpad.net/bugs/379609:32
SteveABjornT: perhaps have self.status and self.display_status09:33
SteveAwhere the latter is a computed property09:33
BjornTSteveA: yes, that seems like a good idea. we were thinking of having self.status a property. that would avoid having to change all the call-sites, on the other hand it's more complicated.09:38
=== doko [n=doko@217.205.109.249] has joined #launchpad
SteveABjornT: we should keep the direct-to-database properties separately named from the computed ones09:42
SteveAyou can change all the callsites in an hour or two of work.  many people can lose many hours debugging problems from a more complex system over the course of months.09:43
BjornTjamesh: yeah, matching searches on dupes sounds like a good idea. 09:43
BjornTSteveA: true09:44
=== fabbione [i=fabbione@gordian.fabbione.net] has joined #launchpad
stublifeless: Where can I find the environment setup for using cm.py out of ~pqm/source ? Nothing seems setup in a sane path that I can find, and invoking cm.py directly from ~/source/config-manager can't find pybaz libraries for a start09:50
=== stub gives up and rolls out untested cherry picks again
KinnisonMorning10:16
=== dous [n=dous@efreet.edu.ms] has joined #launchpad
SeveasSteveA, bon giorno10:19
SeveasSteveA, I've gotta run, if it's important /msg or e-mail works too :)10:25
=== Lorenzod [n=lorenzod@63.218.103.162] has joined #launchpad
BjornTjamesh: does the "Launchpad Errors" mail get sent before the report has been generated? the link in the mail is a 40410:30
jameshBjornT: I had an error in the script.  See the second email10:31
BjornTjamesh: yeah, i saw that now10:32
jameshBjornT: should be reliable now, giving a short summary with the full HTMLised report linked10:32
BjornTyep10:32
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
=== merriam [n=merriam@84.12.81.236] has joined #launchpad
sivangmorning folkies10:41
SteveASeveas: okay10:56
=== doko [n=doko@217.205.109.249] has joined #launchpad
=== carlos [n=carlos@84-123-7-4.onocable.ono.com] has joined #launchpad
SteveAjamesh: it looks very good11:00
SteveAjamesh: i'd like you to make a couple of changes11:00
SteveAhard timeouts are far more important to fix than soft timeouts11:00
jameshyeah.11:01
jameshI'll split them out11:01
SteveAalso, i'd like to see soft timeouts arranged not in order of how many occurred11:01
SteveAbut instead in order of length of request11:01
SteveA(if that is possible)11:01
SteveAi don't know if that is how the soft timeout system works11:02
SteveAalso, at the top of the html, a table of contents, with #anchor links to the various sections11:03
SteveAand perhaps a summary would be nice, like 88 timeouts, 25 soft timeouts, 87 programming errors, 1 attribute error, 4 value errors11:04
jameshit'd also be goot to work out some regexps that could be used to help group the SQL errors11:04
SteveAwe should aim to have no non-404 non-timeout errors.  it is more useful to have non-404 and non-timeout errors ordered by the error type11:05
SteveAthan by frequency11:05
SteveA(or number occurred, rather)11:05
jameshcurrently there is a regexp to replace the memory location used in many object's repr() with INSTANCE-ID, which helps with grouping similar OOPS reports11:06
jameshthat doesn't really help for the ones where an SQL statement is included in the exception value11:07
SteveAmpt_: please take a look at jamesh's summary report sometime, and create a stylesheet for it11:07
SteveAstub: if you made a wee script that you can stick revision ids in, and get revision ids + logmessage out of, then your emails about what revisions got into production could be more informative without much extra effort.11:10
Seveasre11:12
SteveAhi Seveas 11:12
Seveashi11:12
SteveAi have a feature to request for Ubugtu.  it isn't very useful for anyone (including you), except launchpad developers11:13
SteveAthe feature is for Ubugtu to listen for OOPS-oopscode11:13
SteveAand show a URL to jamesh's oops-viewing cgi URL for that oops code11:13
Seveasthat11:13
Seveas's doable11:13
Seveaswhat's the CGI url?11:14
jameshSeveas: make OOPS-[A-Z\d] + => https://chinstrap.ubuntu.com/~jamesh/oops.cgi/$OOPSID11:14
mpt_SteveA, sure11:14
carloslifeless: hi, could you take a look to PQM? my request is taking too long to be processed  (more than 1 hour) and when I sent it, the queue was empty11:15
SteveAjamesh: as the oops report is rather large, what do you think about splitting it into 4 or so pages?11:16
=== stgraber [i=steph@xeon.stargate-server.com] has joined #launchpad
SteveA - hard timeouts11:16
SteveA - soft timeouts11:16
SteveA - 404s11:16
SteveA - errors11:16
SteveAwith an index page pointing to each with a summary11:16
jameshSteveA: I suppose so.  The alternative would be to remove some of the information11:17
Seveasjamesh, that url needs authentication11:17
SteveAi think the level of information is good11:17
jameshSeveas: yes.11:17
SteveASeveas: i was thinking just to display the URL, for ease of launchpad developers clicking on it from their irc client11:18
Seveasok11:18
=== jamesh just uses a keyword bookmark
SteveAyou're always ahead of the curve :-)11:18
Seveas@reload Bugtracker11:19
jameshSteveA: is it useful to have the OOPS numbers categorised by URL, or do you think it would be as useful to just list them all together for the particular error?11:19
UbugtuError: I tried to send you an empty message.11:19
Seveashmm11:19
SteveAjamesh: displaying the URLs helps me understand what the OOPSes are about.11:20
SteveAif the URLs are displayed, then the OOPSes may as well be categorized with them11:20
jameshSteveA: I wasn't suggesting removing the URLs, just changing it to display the URLs, then the OOPSs11:21
SeveasOOPS-21312AD11:21
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/21312AD11:21
SteveAalmost all have just one URL, so i don't think the change is important either way right now11:21
jameshokay11:21
SeveasSteveA, ok this way?11:22
SteveAOOPS-39A1311:22
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39A1311:22
SteveAperfect!11:22
SteveAthanks Seveas!11:22
Seveasnp11:22
SteveAjamesh: the text that says "0% from search bots, 100% referred from local sites" is needed only for 404s in my opinion11:23
=== WaterSevenUb [n=WaterSev@azevedo.astro.up.pt] has joined #launchpad
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
cprovmorning, hackers11:29
SteveAgood morning11:30
SteveAsalgado: there's a few shipit errors in jamesh's error report.  some interesting not found errors, and some other application errors11:30
SteveAhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/OOPS-38A65  <-- AttributeError11:31
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/38A6511:31
SteveAOOPS-38B59 <-- interesting NotFound error11:32
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/38B5911:32
SteveAjamesh: do you have shipit.ubuntu.com listed as a local site?11:33
mpt_cprov, hello, you wanted to talk about Soyuz?11:35
SteveAactually... mpt could put the css in his home on chinstrap, and jamesh's scripts could refer to it there in the html11:36
cprovmpt_: it was yesterday, wasn't it ?11:36
=== ..[topic/#launchpad:SteveA] : launchpad.net | developer meeting: Thur 9 Feb, 1200UTC (wiki:MeetingAgenda) | launchpad-users@lists.canonical.com (wiki:MailingLists) | Channel logs: http://tinyurl.com/72w39
cprovmpt_: I'm happy with you decision (wait +build clean up land then discuss the content issues)11:36
Kinnisoncprov: I think that getting the permission stuff sorted and the functionality sorted in the builds pages is more important than making them prettier11:37
Kinnisoncprov: don't you?11:37
mpt_cprov, my cleanup is done and waiting for (re-)review. That shouldn't be blocking us talking about anything else.11:38
mpt_You can try the branch if you want.11:39
cprovKinnison: uhm, personally I think the permission are ok, View is public, edit + cancel + mode  are Edit and admin is Admin. What am I missing ?11:40
Kinnisoncprov: erm, we need to get the launchpad-buildd-admins celebrity done and made able to do the admin work11:40
Kinnisoncprov: otherwise we can't hand over to adam and lamont11:40
=== Kinnison perhaps didn't communicate this well last week
cprovKinnison: right, good point. I almost forget this11:41
Kinnison:-)11:41
Kinnisonadam and lamont won't :-)11:41
cprovmpt_: It's hard ... we have a lot of bugs but I can define myself those who needs UI efforts, the simplest thing to do is some triage on soyuz bug recently added, would you do it for me ?11:44
SteveAcprov: we can ask daf  and matsubara to focus on soyuz bug triage for a while11:47
cprovSteveA: good idea, I'll ask them, thx11:47
=== SteveA wonders if daf is around yet
=== seb128 [n=seb128@ubuntu/member/seb128] has joined #launchpad
seb128launchpad keep Oopsing today, is that known?11:48
seb128ie: OOPS-39A24411:48
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39A24411:48
SteveAseb128: thanks.  probably a regression in the rollout.  i'll look into it11:49
SteveAseb128: did you *just now* get that oops?11:49
cprovKinnison: bug 3580, is the begin of buildd celebrity11:49
Ubugtumalone bug 3580 in launchpad-buildd "Missed Launchpad Celebrity for Auto Build System" [Major,Unconfirmed]  http://launchpad.net/bugs/358011:49
SteveAif so, i'll need to wait a short while until it is visible on chinstrap11:49
Kinnisoncprov: right11:50
seb128yep11:50
Kinnisoncprov: I've added a note to that about the team in production11:51
SteveAseb128: okay, i see it now.  this OOPS is to do with contention with people concurrently updating a bug.11:51
SteveAstub is working on a solution to that issue in general, which will be ready soon11:51
seb128I get timeout error too, like OOPS-39B20211:51
cprovKinnison: right11:51
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39B20211:51
stubeh?11:51
SteveAstub: are you not working on retries?11:52
stubOh - serialization exceptions. es11:52
stuby11:52
SteveAexactly11:52
SteveAi didn't know the right name for it :-)11:53
SteveAsalgado: the person vocabulary search stuff -- how is that going?  i think that's what OOPS-39B202 is about11:54
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39B20211:54
lifelessstub: carlos would like pqm live again11:56
lifelessstub: are you finished ?11:56
salgadoSteveA, yes, that's the vocab timing out. it was probably caused by contention.11:56
stubOops. Yup.11:56
salgadoSteveA, I tried it two times here. the first it timed out and the second it run pretty fast11:56
stublifeless: reenabled11:56
seb128timeout OOPS-39B214 ...11:56
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39B21411:56
SteveAsalgado: timeout problems with contention are an issue.  we can't automatically retry these, because it will take 1 minute to get a response11:57
SteveAsalgado: you have been working on a 2-stage query for this, right?11:58
stubserialization issues should not be confused with locks being held open by other processes. serialization issues are expected to happen and we need to cope. locks being help open by other processes are bugs that need to be dealt with individually11:58
=== matsubara [n=matsubar@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== SteveA waits until rsync processes seb128's most recent OOPS
matsubaragood morning!12:00
carloslifeless: oh, is it down? didn't know :-P12:00
SteveAstub: do you know if elmo changed the rsync of OOPS reports and other logs to be more selective about what it processes?12:00
salgadoSteveA, no, what I did was to make all vocabs not fetch all the results at once, but instead fetch only the results of the batch they're displaying12:00
stubSteveA: No idea.12:00
=== jinty [n=jinty@196-28-47-56.jhb.netdial.co.za] has joined #launchpad
lifelesscarlos: stub disabled it during prod rollout testing12:00
lifelesscarlos: and forgot to undisable 12:01
SteveAsalgado: okay.  did you discuss with people an approach of making two queries, one direct and one broader?12:01
SteveAsalgado: OOPS-39B214 would benefit from exactly this approach12:02
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39B21412:02
salgadoSteveA, something like bug 2045?12:02
salgadohttps://launchpad.net/products/launchpad/+bug/204512:02
SteveAbecause in this case, the long query was a search for a person matching 'iwj'12:03
SteveAthis could be matched instantly on person.name12:03
=== Mez [n=Mez@ubuntu/member/mez] has joined #launchpad
sivangmorning matsubara :)12:05
=== carlos_ [n=carlos@84.76.255.40] has joined #launchpad
SteveAsalgado: bug 2045 is part of it, yes12:05
=== thisfred [n=thisfred@a80-127-80-154.adsl.xs4all.nl] has joined #launchpad
SteveAsalgado: when we're entering names into a form, i think a launchpad-name or launchpad-email-address search would be sufficient.12:08
salgadoI definitely agree12:09
SteveAwhat do we need to do to make this change?12:09
SteveAthis would remove a whole bunch of timeouts for the distro team12:09
salgadowell, I told you I was waiting for some input on that bug12:10
salgadonobody ever commented on it12:10
SteveAif that happens again, add it to the weekly meeting agenda12:10
salgadofor that reason I haven't checked what's needed to make that change.12:10
stubI don't see OOPS-39B214 benefitting from doing more queries. The slow query was just a select on the person and validpersonorteamcache tables (with an unnecessary join with the emailaddress table)12:11
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/39B21412:11
salgadoif you have some time to discuss this now, it'd be great12:11
SteveAstub, maybe you can discuss it with salgado?12:11
SteveAseeing as you already commented on the bug before12:12
SteveAand you proposed exactly "Would it make sense for the search() method to only search on exact matches for email address?"12:12
dilysMerge to devel/launchpad/: [r=spiv]  Allow admins, rosetta experts and product owners to edit translations if using CLOSED/STRUCTURED mode. (r3100: Carlos Perell Marn)12:14
carlosdilys: thanks ;-)12:14
stubsalgado: For some reason, ILIKE is no longer hitting the index. Changing the query to do """ lower(email) LIKE 'iwj' || '%' """ again hits the indexes and runs fast (50ms)12:28
salgadoouch12:28
stubsalgado: Want me to shove that through with a [trivial]  ? I have a branch free12:29
salgadostub, that'd be great12:30
salgadostub, do you think it's worth leaving a comment there saying that we shouldn't use ILIKE because it might not hit the index?12:31
=== dous is now known as suodla
=== lbm [n=lbm@x1-6-00-13-10-7a-d1-e4.k233.webspeed.dk] has joined #launchpad
stubsalgado: Sure.12:31
salgadodaf, around?12:37
salgadothe link to the oops page from https://launchpad.net/products/launchpad/+bug/30807 is broken12:38
Ubugtumalone bug 30807 in launchpad "Links to a oops" [Normal,Unconfirmed]  12:38
SteveAmatsubara: hi12:38
SteveAmatsubara: i just had a phone call with daf about the process we will use to take OOPS reports from jamesh's report and users' reports, and process these into bugs with a particular milestone.12:38
SteveAdaf is going to write this up today, and talk with you about it, a bit later12:39
dafsalgado: yeah, I just noticed12:39
matsubaraSteveA: ok, we scheduled a meeting every day around 16:00 utc12:39
dafstub: my code assumes that the configuration value ends with a slash12:39
SteveAtwo of the most important QA tasks at the moment are to respond initial bug reports in some way, and to keep on top of oopses12:39
stubdaf: eh?12:40
matsubaraSteveA: also cprov asked me to take special attention to soyuz bugs12:40
dafstub: the configuration value oops_root_url12:40
stubOh - ok. I'll fix that now.12:41
dafit's a bug, but changing the config will work around it12:41
SteveAmatsubara: yes.  seeing as we've just rolled out soyuz, you and daf should pay particular attention to soyuz bugs, so that cprov and others have good information about what needs fixing.12:41
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
salgadocarlos, have you seen bug 3176? we just got two duplicates of it12:42
Ubugtumalone bug 3176 in launchpad "Error when trying to save AbiWord pt-BR translations" [Normal,Confirmed]  http://launchpad.net/bugs/317612:42
stubdaf: I generally use urllib.join12:42
dafSteveA: I'll put up a wiki page outlining what we discussed12:42
stuberm... urllib.urljoin12:42
SteveAdaf: thanks12:42
matsubaraSteveA: all right.12:42
dafstub: ok, I'll do a [trivial]  merge with that, assuming the tests still pass12:43
SteveAdaf: ideally, this needs an additional test12:43
carlossalgado: WTF...12:43
carlossalgado: thanks, I will take a look. btw, the OPPS link you gave is broken12:43
dafSteveA: oh, good point12:44
salgadocarlos, no, I didn't give any link12:44
carlosoh, is it a new malone feature?12:44
salgadocarlos, it's the code that turns the oops code into a link that is broken12:44
carlosthen I guess it's broken...12:44
carlosyeah12:44
SteveAcarlos: yes.  will be fixed in a few minutes12:44
salgadoyes, it was cherry picked today12:44
dafSteveA: did you see https://launchpad.net/products/launchpad/+bug/30746?12:45
Ubugtumalone bug 30746 in launchpad "LaunchBag.developer is not updated when ftest login() is called" [Normal,Unconfirmed]  12:45
SteveAcarlos: with a config change to work around the problem12:45
SteveAdaf: no, i didn't.12:45
SteveAi guess it should do12:45
carlosok12:45
SteveAwe should use events more for logging in etc.  but i'll look into that later.12:45
SteveAany reasonable fix will do for now12:45
dafwell, I worked around it, it's an infrastructure thing12:46
lifelessjamesh: did you get my mail this morning ?12:47
dilysMerge to devel/launchpad/: [trivial]  improve oops analysis script to generate html too (r3101: James Henstridge)12:47
carlossalgado: that problem is already fixed at rocketfuel12:47
salgadoahhh, cool12:48
carlossalgado: I rewrote that class and the new version seems like fixed it as a side effect12:48
carlossalgado: thanks anyway, I'm going to note that on that bug report12:48
=== SteveA --> lunch and exercise
carlossalgado: which bugs are the duplicated ones that you told me?12:49
salgadocarlos, don't remember, but I've marked them as dupes12:50
carloshmm I don't see that information on the bug report you gave me..12:51
carlosOh, sorry12:51
carlosI saw it12:51
jameshlifeless: yeah.  I had the report script disabled while I was cleaning it up to send a short report to the list + a longer web version.  It's running daily again now.12:51
lifelessjamesh: I mean the other email :)12:51
lifelessjamesh: the one about the branch status stuf12:51
lifelessf12:51
=== doko [n=doko@217.205.109.249] has joined #launchpad
jameshah.  found it12:52
=== jsgotangco [n=jsg@210.4.59.162] has joined #launchpad
dafjamesh: perhaps the OOPS web page could put hard and soft timeouts into separate categories12:54
jameshdaf: already suggested by Steve12:55
dafgreat12:56
dafanother suggestion12:56
dafit would be great if we could group by query12:56
dafe.g. ERROR: could not serialize access due to concurrent update UPDATE SessionData SET last_accessed = CURRENT_TIMESTAMP WHERE client_id =12:56
dafthat one turns up a lot in the latest report12:56
dafbut they're separate because the client_id is different each time12:56
dafI imagine we could strip out constants from the queries in order to compare them12:57
=== kjcole [n=kjcole@dsl092-145-217.wdc2.dsl.speakeasy.net] has joined #launchpad
dafwhat do you think?12:57
jameshdaf: yeah.  that'd be good.  lots of queries differ only by string or integer constants12:58
dafcool12:58
jameshI wonder how many falst matches you'd get by replacing all digit strings with "$int" and strings with "$string" (or similar)12:58
=== beyond [n=beyond@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
carloshmmm01:00
carlosI'm not able to get the Malone's 'Advanced' search form01:00
dafs/the//01:00
dafwhat happens?01:01
carlosdaf: it does nothing01:01
carlosWell, it changes the URL01:01
carlosfrom https://launchpad.net/products/launchpad/+bugs01:01
carlosto https://launchpad.net/products/launchpad/+bugs?field.searchtext=&orderby=-priority%2C-severity&advanced=Advanced...&field.milestone_assignment=&field.milestone_assignment-empty-marker=101:01
carlosbut the webpage remains the same01:01
dafyes, same here01:02
dafit's a bug01:02
dafhttps://launchpad.net/products/launchpad/+bugs-advanced seems to still work01:02
carlosIt works with a person as a context01:03
spivdaf: there's a bug about that.01:03
salgadocarlos, yes, that happens only if you're in the +bugs page. brad had a fix for that a few days ago01:03
salgadoI think he didn't manage to merge it. :-(01:03
carlosok, thanks01:03
spivdaf: bug 30690 I think01:04
Ubugtumalone bug 30690 in malone "'Advanced...' button on bugs listing doesn't do anything" [Normal,Confirmed]  http://launchpad.net/bugs/3069001:04
dafcool01:04
jameshlifeless: assuming that the two use cases are independent, then it would probably make sense to use xml-rpc for the branch-puller use case and go through the main web app for the diff URL case.01:07
jameshlifeless: given that authserver can only be reached on our network.01:08
carlosHmmm01:09
carlosI think Rosetta is completely broken atm..01:09
=== irvin_ [n=irvin@203.213.220.193] has joined #launchpad
carlosAnd I don't really understand it as the production update doesn't have any change from my side01:09
=== carlos mails launchpad mailing list
lifelessjamesh: works for me01:10
=== jblack [n=jblack@static-209-158-45-74.scr.east.verizon.net] has joined #launchpad
=== jblack [n=jblack@static-209-158-45-74.scr.east.verizon.net] has left #launchpad []
=== jblack [n=jblack@static-209-158-45-74.scr.east.verizon.net] has joined #launchpad
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
dilysMerge to devel/launchpad/: [trivial]  Use LIKE instead of ILIKE to use indexes and tweak schema.xml defaults (r3102)01:23
dafstub: empty merge?01:24
stubBah.01:27
dilysMerge to devel/launchpad/: [trivial]  Use LIKE instead of ILIKE to use indexes and tweak schema.xml defaults (r3103: Stuart Bishop)01:45
=== niemeyer [n=niemeyer@200-103-135-181.ctame7006.dsl.brasiltelecom.net.br] has joined #launchpad
=== carlos -> lunch
=== kiko [n=kiko@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
dafSteveA, matsubara: I've updated LaunchpadBugTriage and LaunchpadProjectMilestones to talk about oopses02:07
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
lifelessnight all02:09
dafnight Rob02:09
matsubaradaf: thanks daf, I'll take a look later.02:09
salgadostub, ping02:22
kikoKinnison, thanks for the document02:23
Kinnisonkiko: You're welcome. I figured it would be best to turn my notes into a real doc otherwise I'd forget things too02:25
kikoindeed. we should wikify it -- stub, any suggestion on where to put it? I'd like to see the other rollout procedures documented as well02:25
KinnisonFYI, I'm currently working on bugs 29635 and 29639 for cprov02:25
stubsalgado: pong02:25
stubI'd put it in the launchpad tree where it will be under RCS and more likely to be maintained. wikis are just where you put docs to forget about them.02:27
kikocool02:27
kikostub, where do such documents go?02:27
stubThere is a top level 'doc' directory in the launchpad tree. The database docs I'm keeping in database/schema where I notice them (although a symlink would do just as well)02:28
stubWow... some old docs in there :)02:29
salgadostub, is there anything wrong with https://chinstrap.ubuntu.com/~dsilvers/paste/fileQUMFzM.html ?02:29
dilysMerge to devel/launchpad/: [trivial]  make oops links work if oops_root_url config value doesn't end with slash (r3104: Dafydd Harries)02:29
kikocool.02:30
kikostub, so this doc/ directory is where we put docs to forget about them too?02:31
kikoperhaps the problem is more that nobody gardens any of our documentation.02:31
=== daf -> lunch
stubkiko: Thats why I keep my docs next to the code - I see them when changing the db scripts and can keep them up to date (or get reminded that I haven't)02:32
kikostub, that's fine, but where should rollout docs go, then?02:32
stubkiko: I'd stick 'em in the docs directory in the root of lp02:32
stub(but am not really fussed if you have a better idea)02:35
stubsalgado: that query should be fast, but isn't. I'm having a closer look.02:36
salgadostub, well, let's leave the speed aside for now02:37
salgadothere's something weird going on that query02:37
salgadostub, if you run it with the sampledata from chinstrap.warthogs.hbd.com/~salgado/current.sql02:38
salgadostub, it will give you only one result. which is correct02:38
salgadobut, if you join that query in a single line, it'll give you two results, which is wrong02:38
stuboh... your missing some brackets02:39
=== stub tries to remember if AND or OR has higher precidence
salgadoI know that if I add a parenthesis around the first condition in the where clause it'll work, but I wanna know why it's failing only when I join it in a single line02:39
stubwhat do you mean 'join it in a single line'?02:40
salgadostub, https://chinstrap.ubuntu.com/~dsilvers/paste/fileQqjjda.html02:40
salgadostub, if you paste it like it is in the first page I gave you, it'll get me only one result02:40
salgadoand if you paste it like it is in the second page, it'll give me two results02:41
salgados/me/you02:41
matsubaraSteveA, cprov: I've finished the triaging of all untriaged bugs on soyuz, would you take a look on the 4 left ones that are user requests?02:42
cprovmatsubara: yes02:42
salgadostub, IOW, if it's broken into multiple lines, the parenthesis aren't needed. but we need them if it's a single line query02:43
stubsalgado: There are some spurious spaces in there02:43
matsubaracprov: https://launchpad.net/products/soyuz/+bugs-untriaged02:43
salgadostub, in the single line version?02:43
stubyup02:43
salgadoI thought spaces where ignored02:43
stubEmailAddres. email, david. alloche02:44
matsubaracprov: I've changed them to wishlist, so you could easily find and take care of them.02:44
salgadoaha02:44
=== Kinnison heads out to lunch, back in a short while
salgadostub, even if you remove that space, it'll still give two results02:45
salgadomaybe not02:45
=== doko [n=doko@217.205.109.249] has joined #launchpad
salgadono, it won't02:45
salgadobah. what a shame02:46
cprovmatsubara: they are hard to solve, I'll triage them with Kinnison later today, thanks for your good job in the other bugs .02:47
lamont-awayKinnison: I don't _mind_ pestering you guys every time we need something done that requires lp-buildd-admin privs... :-)  But I think you guys might...02:48
matsubaracprov: no problem, shout if you need anything else. :)02:48
cprovmatsubara: ok, dude02:48
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has joined #launchpad
=== lamont-away -> work
stubsalgado: I don't think we can do that efficiently on production anyway... is this for a batch job, or to be used interactively?02:53
=== Keybuk [n=scott@descent.netsplit.com] has joined #launchpad
stubHmm... maybe not. I can get it down to 3.5 seconds once everything is in RAM02:55
salgadostub, it's for interactive use. but that's only for shipit admins02:55
=== mgalvin [n=mgalvin@ubuntu/member/mgalvin] has joined #launchpad
=== auth00 [i=auth@fiji.grd.sgsnet.se] has joined #launchpad
=== jbailey [n=jbailey@modemcable139.249-203-24.mc.videotron.ca] has joined #launchpad
jbaileyWhat should the bug be filed against for ubugtu?03:05
KamionUbugtu's just a bot Seveas runs, I don't think it has a bug tracking component03:05
jbaileyAh, hmm.03:06
=== jbailey bats his eyelashes at Seveas.
stubsalgado: This seems to be giving the best results so far: https://chinstrap.ubuntu.com/~dsilvers/paste/filevj65AW.html03:06
salgadostub, is the speed gain worth the fix?03:09
salgadoI think it should be trivial to fix. just want to make sure it's worth it03:09
stubbtw. I get one line on my sampledata using both single line and multiline queries you posted before03:10
salgadostub, yes, the problem was the spurious spaces. I should've read the queries carefully before complaining03:11
stubHmm... queries are performing equivalently now.03:13
Seveasjbailey, 'sup?03:18
jbaileySeveas: I'm just running out.  But the quick summary is that ubugtu seems to randomly pick which task to give status on, I'm wondering if there's a better way.  Bug #30621 is an example.03:20
jbaileyerr.03:21
jbaileybug 3062103:21
Ubugtumalone bug 30621 in malone "glibc changelog seems to list two versions and have eaten a line for the changelog" [Normal,Rejected]  http://launchpad.net/bugs/3062103:21
jbaileySeveas: Shows as rejected, even though the rejected task is simple a thing I clicked on by mistake.03:21
jbailey(I added a task that I hadn't intended to)03:21
Seveasit picks the first task listed in the /+text url03:21
jbaileySeveas: I'm not sure where to put wishlist bits for ubugtu.03:21
Seveasjbailey, in my mailbox or in here :)03:22
Seveasjbailey, what would you prefer? first non-fixed one?03:22
jbaileySeveas: I haven't got a comprehensive bit of logic worked out.03:22
Seveasor max(severity), pseudomax(status)03:22
jbaileyIn this case, I simply wrongly filed it against malone because I was fiddling with the URL and thought that the button was a text box.03:23
jbaileySo I rejected the extra task that I Added by accident.03:23
jbaileyAnyhow, appointment in 7 minutes, back in 45. =)03:23
Seveas:)03:23
salgadostub, that query I pasted to you return the two values on pqm's box03:23
=== ddaa received funny spam
ddaa"Finally a Patch that works!"03:24
Seveashahaha03:24
salgadostub, is that box running breezy's postgres?03:25
stubddaa: Bounce it to pqm - you might have better than your usual luck (and Launchpad could do with some performance enhancement!)03:25
stubsalgado: yes03:25
ddaastub doing03:25
ddaastub: you think I should forward the pqm reply to launchpad if it rejects it?03:26
salgadostub, any idea what could cause that query to fail on pqm's box but not locally?03:27
stubsalgado: Maybe your sample data is different to what is in the trunk?03:27
=== sNKe [i=ircap8@56.Red-83-43-233.dynamicIP.rima-tde.net] has joined #launchpad
=== sNKe [i=ircap8@56.Red-83-43-233.dynamicIP.rima-tde.net] has left #launchpad []
salgadostub, no. I committed and mirrored my sampledata changes03:32
salgadoand I just rebuilt my db here in case I had changes in it03:32
stubAnd pushed the sampledata changes to chinstrap?03:32
salgadoand the tests still pass here and fail on pqm03:32
salgadoyes, I meant that when I said mirrored03:33
stubHave you merged rocketfuel in recently? There was a change I made last week to the text search stuff that could cause what you describe (changed how '.' is handled in text searches)03:35
salgadoaha03:35
salgadoit must be it, then03:36
stubYou will need to rebuild the database after merging too, to ensure the updated stored procedures are in there.03:36
salgadoI don't remember when was my last merge on that branch, but it wasn't this week03:36
=== salgado merges
SteveAdaf, matsubara: how about we go through bug stuff in 50 mins time?03:38
=== daf returns from lunch
dafSteveA: fine by me03:44
SteveAthat's at 30 minutes past the next hour03:44
SteveAfor whatever timezone you're in03:44
Seveasjbailey, when you're back, the tasks are now first ordered by status and severity (status first, severity second). The priority from low to high for status is now: ['Rejected', 'Fix Committed', 'Fix Released', 'In Progress', 'Unconfirmed', 'Needs Info', 'Confirmed']  - any thoughts on that ordering?03:46
matsubaraSteveA: fine to me too.03:48
SteveAthanks matsubara, daf.03:48
matsubaraI should get some lunch now then.03:50
=== BjornT heads down to the post office for a while
ddaastub: re lock metrics03:51
ddaastub: what about some statistical profiling, taking periodic snapshots of taken locks and associated queries03:51
ddaalike, every minute03:51
ddaaanyway, the "oops" lock thing would be misleading at best, since it would actually do this sort of measurement, with no garantee that the lock that caused contention is still taken03:53
kikobradb, how's the feedback been on the package bug contact reports?03:53
bradbkiko: Most of it was what we got from yesterday during that discussion with Kamion et al.03:54
jbaileySeveas: I'm just back.  Trying to understand wht you mean.03:54
kikobradb, have people found it useful, annoying, excellent, hidden, a lifesaver, confusing, etc?03:54
jbaileySeveas: I don't understand how this affects the output.03:54
Seveasjbailey, what I mean is that if there is an unconfirmed low priority task and a rejected high priority one, Ubugtu will show the unconfirmed low priority one03:55
dafcarlos: yo03:55
carlosdaf: hi03:55
dafcarlos: I just met Tim Morley03:55
Seveasubugtu picks one task only, it's now trying to be intelligent about which task03:55
bradbkiko: Dunno, I haven't specifically asked yet. It hasn't been in production for that long.03:55
dafcarlos: he says the number 1 thing stopping him from using Rosetta to translate OpenOffice is the ability to add/edit comments03:56
jbaileySeveas: So rejected is least interesting, and confirmed is most?03:56
kikoit would be nice to collect that feedback, bradb, though not mandatory03:56
carlosdaf: we are aware of that03:56
bradbkiko: Definitely.03:56
Seveasyes, and I'd like to hear your comments about that ordering. For status it's less obvious than for severity03:56
dafcarlos: a) do we have a bug open?03:57
carlosdaf: I think so, yes03:57
carloslet me check03:57
dafcarlos: b) is it a high priority for us?03:57
carlosdaf: kiko asked me on December to look into it after PoMsgSetView implementation03:58
kikodaf, carlos: tell me about bug 1681, the bug that nobody wanted to kill.03:58
Ubugtumalone bug 1681 in rosetta "Viewing a translation page fails in unix2newlines" [Major,In progress]  http://launchpad.net/bugs/168103:58
kikocarlos, and pomsgsetview, hopefully, is on the way to pqm? :)03:58
carloskiko: I'm working on it atm now that the needed changes part of the  PoMsgSetPage spec is done03:59
carloskiko: most of the main changes is already merged since Friday03:59
kikoyeah I saw that03:59
kikogood work03:59
dafcarlos: "it"? 1681 or PoMsgSetView?04:00
carloskiko: I just found a problem with Rosetta and had to do some checking before requestin a fast review + cherrypick of that Friday merge 04:00
carlos168104:00
dafcool04:00
carlosbut also PoMsgSetPage, I detected a small problem04:00
dafwhat are your plans when #1681 is done?04:00
carlosdaf: Finish the suggestions fixes using AJAX04:01
carlosI have many things open atm and want to close all them04:02
carlosas soon as that is done, If kiko agrees, I think I could take a look to the comments feature04:02
dafkiko: can we make that a priority?04:03
dafkiko: I want to make Tim happy04:03
=== mgalvin [n=mgalvin@ubuntu/member/mgalvin] has joined #launchpad
carlosdaf: I have also pending dapper imports into Rosetta04:03
dafcarlos: perhaps you can make a list of all your pending tasks on the wiki04:04
carloskiko, Kinnison: Do we know when will be ready the new system to do the imports as planned?04:04
kikodaf, the dapper imports are a lot more importan, unfortunately.04:04
dafok04:04
kikocarlos, I have no idea on the status of that -- I thought everything just worked04:04
dafmaybe after those04:04
carloskiko: Kinnison and mdz told me that there is a missing part that would take more time than use soyuz to develop Ubuntu04:05
carlosbut I don't know the details04:05
kikofind them out04:05
kikoI think that is handwaving04:06
carlosKinnison, mdz?04:06
dafSteveA: before our meeting, could you take a look at what I wrote on the LaunchpadBugTriage page about oopses?04:07
jbaileySeveas: I'm wrestling with the placement of Confirmed and Unconfirmed.  Needs Info seems like the highest priority.04:09
Kamionjbailey: depends who's asking, really04:09
jbaileySeveas: I'm almost tempted to place Confirmed second from the left in that, aside from Rejected, its the state where something is waiting and may or may not have anything interesting happening with it.04:09
kikocarlos, make sure you find out and email launchpad with your analysis04:09
jbaileyKamion: Yeah.04:09
Kamionfor a maintainer, needs info is lower than confirmed, I'd say04:10
Kamionfor a submitter, it's higher04:10
carloskiko: ok. I thought you know about that issue already...04:10
kikocarlos, I don't know any of the specifics04:10
kikoddaa, daf: what of optional branch names, and the buildbot test suite fixes?04:12
SteveAdaf: i'll take a look now04:12
dafkiko: o-b-t is needs-reply04:12
jbaileyKamion: Who do you think is more important for ubugtu to serve?  I'd guess maintainer.04:12
dafkiko: (from me)04:12
kikodaf, get on it, don't miss this week's cutoff04:13
mdzcarlos: hmm?04:13
mdzcarlos: I don't know what you mean04:13
kikomdz, carlos would like to know the specifics of what needs to be done to get translation imports working again, if you know of such a thing.04:13
carlosmdz: pitti told me that he's not able to deploy the new script to extract translations04:14
Kamionjbailey: I am hopelessly biased04:14
carlosbecause we are missing something to include the tarball with translations as part of the build output04:14
=== beyond is now known as beyond-rango
carlosmdz: I talked with you about that and you confirmed it to me but I didn't ask for details04:15
mdzcarlos: what you told me was that imports were already broken before soyuz was rolled out04:15
carlosmdz: not broken but 'disabled'04:15
carlosmdz: waiting for the soyuz roll out04:15
mdz?04:16
kikobradb, can you detail to me what sort of work you're doing on searching this week?04:16
carlosmdz: ok, let's start again04:16
mdzJan 26 14:57:37 <mdz>   pitti: why aren't they being imported into rosetta anymore?04:16
mdzJan 26 14:58:12 <carlos>        mdz, because we changed a lot the way of doing imports and instead of porting the old script I decided to wait for the new system as it was supposed to be ready earlier04:16
jbaileybug 3062104:16
Ubugtumalone bug 30621 in soyuz "glibc changelog seems to list two versions and have eaten a line for the changelog" [Major,In progress]  http://launchpad.net/bugs/3062104:16
bradbkiko: Hang on, I'll show you a picture.04:17
=== thisfred_ [n=thisfred@a80-127-80-154.adsl.xs4all.nl] has joined #launchpad
carlosmdz: right,  I 'disabled' it04:17
carlosnot porting the old script to the new system04:17
carlosmdz: the new system relies on soyuz04:17
mdzcarlos: you disabled it because it wasn't working anymore04:17
Seveasjbailey, how about ['Rejected', 'Confirmed', 'Fix Committed', 'Fix Released', 'In Progress', 'Needs Info', 'Unconfirmed'] , that looks like an interesting-to-developer-ordering to me04:18
carlosmdz: it didn't broke, I just implemented the new way.04:18
KamionSeveas: to a developer, Confirmed is more interesting than either kind of fixed04:18
carlosmdz: Anyway, the question is... could pitti upload to the archive the new pkgstriptranslations version?04:18
mdzcarlos: I don't see any reason why he shouldn't04:19
carlosmdz: if that's done, dapper translations will appear on Rosetta04:19
mdzcarlos: oh really?04:19
=== Kinnison returns
Kinnisonsorry, mortgage conversation overran04:19
carlosmdz: yes, that's why I didn't ported the script04:19
mdzcarlos: how does the new pkgstriptranslations work?04:19
mdzmy understanding is that it was being changed to include the translations in the upload04:20
carlosmdz: generates a new tarball with the usual binaries, source and diffs and adds it to the changes file04:20
carlosmdz: right04:20
mdzbut that soyuz didn't know how to handle such uploads yet04:20
mdzKinnison: is that inaccurate?04:20
KinnisonIf carlos is happy with the code he added to distroreleasequeue then we can go for it providing carlos has ensured that the queued user has access to all the relevant tables04:20
carlosmdz: that was exactly what I'm asking for. Rosetta has the needed bits in place04:20
dilysMerge to devel/launchpad/: [trivial]  Fix a regression on shipit: untested code path was calling len() on a SelectResults object. Also changes sampledata in order to test that code path. (r3105: Guilherme Salgado)04:21
carlosKinnison: I asked sometime ago for a way to test this functionality04:21
carlosand I was told that until soyuz deploiment we cannot test it04:21
bradbkiko: Implementing this: http://flickr.com/photos/84096161@N00/97165744/ but still fleshing out the design, namely sectionalizing the page and possibly making each criteria a bit more distinct on the page.04:21
Kinnisoncarlos: right04:21
carlosKinnison: do we have a way to test it now?04:21
Kinnisoncarlos: so now that deployment is done we can look at testing it04:22
carlosok04:22
mdzcarlos: ok, I can't help you with rolling out the LP changes04:22
mdzupdating pkgstriptranslations is trivial04:22
bradbkiko: This also means updating the linkified search filter code to understand the extra params.04:22
carlosKinnison: could we prepare a test plan for it?04:22
Kinnisoncarlos: Right, so launchpad/doc/distroreleasequeue.txt is where we test the queue04:23
SteveASeveas: is ubugtu using the new text pages now?04:23
jbaileyKamion: I wonder if that's true.  If it's a bug I know that I'm interested in, isn't it better for me to know that somewhere in the launchpad universe, someone has fixed it?04:23
bradber, "Reporter" "Nobody", whoops04:23
=== bradb removes that option
carlosmdz: yeah, I didn't know the details of what's missing so I had to check, Thanks04:23
SeveasSteveA, yes04:23
kikobradb, safari displays <select multiple> as checkbuttons?04:23
Kinnisoncarlos: That area has changed in our deployment branch and cprov and I are working toward getting that merged in04:23
SteveASeveas: cool.  thanks for writing the initial spec for that!04:23
bradbkiko: No. Those <select multiple>'s are just a horrible UI. :)04:24
kikoReporter: nobody?04:24
kikoreally?04:24
bradbkiko: See previous comment. I removed that. :)04:24
carlosKinnison: I suppose I will need to add testing for my code there as soon as you get it merged, right?04:24
kikoyou will conflict with daf on Upstream Status04:24
bradbGot a little too copy and pastey there.04:24
bradbkiko: I won't. It's a totally different screen.04:25
bradb(For now.)04:25
kikoI so wish you guys actually communicated more04:25
Kamionjbailey: suppose it depends why I'm asking04:25
kikothat is crack04:25
kikoa different screen?04:25
bradbkiko: I reviewed his patch.04:25
kikothat does the same thing?04:25
Kinnisoncarlos: that's the ideal method, yes04:25
Kinnisoncarlos: It shouldn't be too much longer we hope04:25
Kinnisoncprov is intending to ask kiko to do a new review as soon as we've merged in these response fixes we're doing right now04:25
kikoI also am troubled by this new wave of search forms if we have no plan for updating the existing ones.04:26
carlosKinnison: ok, please, ping me when you are done04:26
kikoKinnison, make sure carlos and you guys are sorted before you leave today04:26
carlosKinnison: anyway, I will ping you again on Thursday just in case you forgot it, ok?04:26
bradbkiko: Yeah. The idea is to replace the existing advanced search screen which sucks because 1. it uses multi-selects, 2. it presents search results, even when you haven't searched for anything (which you almost surely don't intend to happen when you click "Advanced Search"), and 3. it's stuck in between portlets.04:26
=== thisfred_ [n=thisfred@a80-127-80-154.adsl.xs4all.nl] has joined #launchpad
Kinnisoncarlos: cool04:26
Kinnisonkiko: Yep, we're doing our best.04:26
Kinnisoncarlos: it looks like it wouldn't be too hard for you to do, once we get it merged04:27
kikoKinnison, carlos: today -- don't let this one slip, blocking Rosetta work is a big no-no04:27
Kinnisoncarlos: the distroreleasequeue test is pretty comprehensive thanks to cprov04:27
jbaileyKamion, Seveas: I'm not sure it's solvable without alot more complex logic.  I'm happy that Rejected isn't the highest now, given how easy it is to make a mistake where the only undo is "reject"04:27
carlosKinnison: ok, cool04:27
Kinnisonkiko: What do you suggest we do? Should carlos work on a branch of cprov's branch to get the tests written?04:27
carloskiko: I'm blocked on get that branch merged04:27
Kinnisoncarlos: could you branch off our work to get your tests done?04:28
carlosKinnison: are you changing that file?04:28
Kinnisoncarlos: Not currently, no04:28
kikoKinnison, just make sure carlos knows what he needs to do -- he can wait on that branch to merge, but I don't want him to be left in a vacuum and then discover later that he needs to change his code again.04:28
Kinnisonkiko: indeed04:28
carlosKinnison: ok, then it's not a problem04:28
Kinnisoncarlos: I believe that lot is pretty stable04:28
Kinnisoncarlos: if you can get your tests done (and any fixes you need if any) off our branch then we can merge your code in at the same time04:29
KinnisonThat would minimise the times I think04:29
carlosKinnison: path?04:29
kikobradb, I am confused, but I need to have lunch. talk to you in 30 minutes?04:29
Kinnisoncarlos: .../cprov/launchpad/uploader-tests04:29
bradbkiko: Sure.04:29
carlosKinnison: ok, thanks04:29
SteveAmatsubara, daf: skype?04:30
Kinnisoncarlos: if you have problems, bug me and/or cprov04:30
carlosok04:30
=== Kinnison is just finishing a refactor of the soyuz-upload test
=== doko [n=doko@217.205.109.249] has joined #launchpad
matsubaraSteveA: ok04:31
kiko-fudtime for uncertainty04:31
matsubaraSteveA: what's your skype username?04:32
SteveA 'implied'04:32
kiko-fudand don't ask04:32
matsubarajust added04:33
ddaayay! got my new toy04:33
ddaaI read that rockbox now works on the ipod nano, I thought I would have to check.04:33
LarstiQheh04:33
kiko-fudddaa, you were going to say something about pgsql locks?04:34
ddaakiko-fud: I asked stub if I thought that statistical profiling would be more practical04:34
dafmatsubara: what's your Skype username?04:35
ddaajust periodically polling the database for open locks and related transactions04:35
SteveAmatsubara: ?04:36
ddaakiko-fud: since anyway it appears that finding out locks at oops time is not reliable, I thought it would be better to avoid the temptation to misinterpret this data.04:36
matsubaraSteveA, daf diogo.matsubara04:36
kiko-fudhmmmm.04:36
matsubaraSteveA: added you already. it says: pending authorization04:36
kiko-fuddamn04:37
kiko-fudstub sent that email too late for me04:37
kiko-fudI missed 4 cherry-picks04:37
SteveAmatsubara: okay.  i need to restart skype.  then i'll make a conference call and call both you and daf04:37
kiko-fudand I wish he hadn't rolled out the __len__ changes04:38
SteveAkiko-fud: stub will be doing extras over the next day or so04:38
SteveAkiko-fud: we still have __len__ to roll out properly04:38
matsubaraall right04:38
kiko-fudSteveA, I think it was rolled out -- shipit broke.04:38
elmohow do I reply to a comment in malone?  and/or is the mail interface up yet?04:40
=== jsgotangco [n=jsg@210.4.59.162] has joined #launchpad
elmoas in reply to a comment, with the original quoted for me, a la bugzilla/rt etc.04:40
siretartelmo: just answer to the email. the mailinterface for bugs works (at least for me)04:41
LarstiQelmo: that works for me on bugs I get mailed to begin with, not sure how to do it with ones I'm not04:41
elmothanks04:42
elmo(kinda sucks that you can't do it through the web tho)04:42
=== beyond-rango is now known as beypnd
=== beypnd is now known as beyond
LarstiQelmo: web has always worked?04:44
siretartcprov: you wanted to talk to me because of special workflow needed for uploading to -updates?04:44
elmoLarstiQ: how?  'add a coment to this bug' just starts from empty?04:44
LarstiQelmo: ah hmm, I see what you mean04:45
=== LarstiQ confesses ignorance
cprovsiretart: not at this time, I'm sorry, we are not ready yet 04:45
siretartcprov: so uploading to breezy-updates is not possible at all at the moment?!04:45
SteveAmatsubara: https://chinstrap.ubuntu.com/~jamesh/oops-summaries/2006-02-07.html04:46
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/summaries04:46
SteveAthanks Ubugtu ;-)04:46
cprovelmo: ^^^, do you have any news about siretart comment ?04:46
Kinnisonpocketed uploads are not permitted yet04:46
KinnisonThis is mostly because I don't know what the deal is about who can upload to -updates and if there's an accepted/approved workflow on -updates04:47
mdkewhat seems to happen is that people upload, then Kamion/mdz have to approve it before it goes through04:48
mdkethey can tell you more, I'm sure04:48
KinnisonRight, so -updates needs the approval queue working04:48
mdzcorrect04:48
Kamionanyone who can upload to breezy can upload to breezy-updates, subject to approval as above04:49
mdzno one can upload to breezy ;-)04:49
mdkebut presumably someone can upload to breezy-updates at the moment, right?04:52
KinnisonOkay, so I need to ensure that uploads to pockets go to -unapproved basically, yes?04:52
KinnisonOnce I've finished this refactor for celso I'll take a look04:52
Kamions/can/could/ for the pedants :)04:54
KamionI suppose "anyone who can upload to dapper" would be more accurate04:54
SteveAmatsubara: https://chinstrap.ubuntu.com/~jamesh/oops.cgi/2006-02-07/C29004:54
=== raptoid [n=raptoid@unaffiliated/raptoid] has joined #launchpad
=== gneuman [n=gneuman@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== raptoid is now known as ceviz-asigi
kikodoes anyone know what was made of lifeless' story branch?05:48
KinnisonIs there a way in doctests to say "this snippet must produce these lines of output, but I don't care in what order they come" ?05:50
SteveAKinnison: either you sort them, or use a regex or use the 'in' operator05:50
KinnisonSteveA: https://chinstrap.ubuntu.com/~dsilvers/paste/filekKBenB.html05:51
SteveAyou can use '/n'.join(sorted(somestring.splitlines())) for example05:51
KinnisonSteveA: that's the bit of doctest05:51
KinnisonSteveA: but the order of those two lines is unreliable05:51
Kinnisonat the whim of the db05:51
SteveAi would...05:52
SteveAcreate a list called L05:52
SteveAand instead of printing, append to L05:52
SteveAand then sort L, and print it out05:52
SteveAor use the 'in' operator on L05:52
=== ..[topic/#launchpad:kiko] : zope_3_2
KinnisonI think I'll have to order queue_items somehow05:53
kikoerr05:53
kikohow did that happen05:53
Kinnisonbecause some of the messages are generated deep in the code below set_accepted()05:53
kiko(can someone restore the topic?)05:53
=== kiko shakes head
=== SteveA will
=== ..[topic/#launchpad:Kinnison] : launchpad.net | developer meeting: Thur 9 Feb, 1200UTC (wiki:MeetingAgenda) | launchpad-users@lists.canonical.com (wiki:MailingLists) | Channel logs: http://tinyurl.com/72w39
SteveAta05:54
SteveAbeat me to it05:54
Kinnisonno problem05:54
=== Kinnison glares at getFancyQueueItems
Kinnisonit's supposed to order them just fine05:55
Kinnisonunless sqlobject's .union() method is unreliable05:55
kikosalgado-afk knows all about union05:56
Kinnisonpoo :-(05:56
Kinnisoncprov: as soon as I can get a reliable ordering out of getFancyQueueItems, I'm done fixing up this test, it's neater and more reliable05:56
cprovKinnison: good05:56
Kinnisonbut currently the snippet I nopasted for stevea is unreliabl05:57
Kinnisone05:57
Kinnisonit fails about one time in four for me05:57
SteveAKinnison: i already said how to make the test reliable05:57
SteveAbetter to make that change, and then choose whether to sort out deeper changes later on05:57
KinnisonSteveA: okay, I figured it'd be nicer to make getFancyQueueItems ordering-stable since it clearly tries to be, but I guess for now I can short-circuit the test05:58
=== Kinnison reruns the tests
SteveAKinnison: thanks for keeping indeterminately-ordered tests out of RF :-)06:03
kikoyes as a public service06:03
Kinnisonhttps://chinstrap.ubuntu.com/~dsilvers/paste/fileyKmqjf.html06:03
KinnisonSteveA: is ^^ that okay?06:03
=== Kinnison waits for the joyful bzr commit to complete
Kinnison27 seconds :-(06:08
niemeyer"The following 674 words could not be found in the dictionary of 7 words and are highlighted below:"06:08
Kinnisoncprov: the fix for all that is pushing now06:08
niemeyerThat's *SO* useful.. :)06:08
Kinnisoncprov: after you've reviewed it, let me know if you're happy for me to close those bugs06:08
Kinnisonniemeyer: dictionary of seven words?06:08
cprovKinnison: ok06:08
Kinnisoncprov: pushed06:08
Kinnisoncprov: i have a bug report from colin about the queue tool. Shall I investigate that now?06:09
niemeyerKinnison: Yeah.. :)06:09
Kinnisonniemeyer: how... useful06:09
cprovKinnison: yes, in the meanwhile I can merge your changes and review06:09
Kinnisoncprov: cool06:09
=== Kinnison goes after this bug
SteveAKinnison: looks good to me06:11
SteveAi would add in the doc part of the doctest that the output can be in any order, hence the L and sort()06:11
KinnisonSteveA: is https://chinstrap.ubuntu.com/~dsilvers/paste/fileqCjeZE.html the result of a lack of execute_zcml_for_scripts?06:20
KinnisonSteveA: or should I be asking for IPersonSet instead of IPerson?06:21
SteveAKinnison: without seeing code, it is hard to say06:22
Kinnisonn/m I should be using the Set I'm pretty sure06:22
SteveAIPersonSet is a utility06:22
SteveAIPerson is a content object06:22
Kinnisonyep06:22
Kinnisonta06:22
=== Kinnison was just being braindead
KinnisonI blame the mortgage people06:22
Kinnisonthey confused me earlier06:22
Kinnisonwith talk of interest rates06:22
sivangrehi all06:24
=== sivang is finally, at home :)
Kinnisoncarlos: Did you manage to get that branch okay to work on your tests?06:27
=== doko [n=doko@217.205.109.249] has joined #launchpad
carlosKinnison: not yet, still fixing a more urgent issue. I need to request a fast review and a cherry pick06:28
Kinnisoncarlos: righty, no rush from me06:28
carlosKinnison: anyway I think I have enough information, thank you06:28
Kinnisoncprov: I've fixed a bug in queue and pushed a fix along with a security.cfg change06:29
cprovKinnison: ok06:29
Kinnisoncprov: I'll mail stuart about doing the grant until we can get it merged06:29
Kinnisondone, CCed to you06:30
cprovKinnison: good, thx06:30
dafcarlos: about bug #399106:38
Ubugtumalone bug 3991 in rosetta "Timeout error on translation page (+translate)" [Normal,Confirmed]  http://launchpad.net/bugs/399106:38
dafcarlos: er, I mean bug 575106:38
carlosdaf: yes?06:39
dafStuart asks: "It returns over 400 rows - are we actually using all of06:39
daf  these results? If not, why arn't we doing more filtering in the database?"06:39
dafcould you answer that in a comment?06:40
dafalso, have you implemented Steve's suggestions for code cleanup?06:40
dafif so, could you say so?06:40
carlosdaf: that's not 5751...06:41
carlosat least, I don't see any comment from Stuart...06:41
dafsee the description06:41
=== Kinnison goes to cook dinner
Kinnisonciau06:41
salgadoKinnison, the results of a union() are always orderred by the value of the orderBy argument of that union() call. the orderBy arguments of the SelectResults that make that union are ignored06:42
dafcarlos: also: does this bug need to be private?06:42
carlosoh, right....06:42
salgadoI mean, that's how it should work. if it doesn't work that way, then it's a bug and I'd be glad if you could find how to reproduce it06:42
carlosdaf: it has SQL code that shows part of our DB schema06:42
dafso does #3991, and that one isn't private06:43
dafwe should be consistent06:43
dafSteveA: what do you think?06:43
SteveAi think i'm on the phone06:44
carlosdaf: I didn't open 3991, I filed 5751... I didn't noticied that 3991 was not private06:44
dafmaybe we should just remove the traceback from the description06:45
dafthey look like differenct queries06:45
=== kjcole [n=kjcole@pchb1f.gallaudet.edu] has joined #launchpad
carlosdaf: also, malone ate my comment when I first closed that bug (5751)06:46
dafthe query that goes It returns over 400 rows - are we actually using all of06:46
carlosdaf: I implemented Steve's solution06:46
dafbah06:46
dafthe query that goes SELECT DISTINCT POSubmission.id FROM POSubmission JOIN POMsgSet ON POSubmission.pomsgset = POMsgSet.id JOIN POFile ON ...06:46
dafalso seems to time out quite a bit06:46
dafhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/2006-02-07/C29006:46
dafdo we have a bug open on that?06:46
carlosdaf: and added a new one saying that it failed and that we are going to use AJAX to solve it06:46
dafcarlos: that's good, you should be proud and say so!06:46
daf:)06:47
carlosdaf: it's noted on that bug already!06:47
carlosI added it this morning when I wake up06:47
dafsorry, what's noted on the bug?06:47
dafI suppose your comment could be read to imply "I implemented what Steve suggested and it's still too slow"06:48
dafbut it's better to say it explicitly06:48
=== daf -> yoga
carlosdaf: I did it but as I just told you.... malone ate that comment06:49
carlosseems like the comments when you change the status of a bug are now stored as normal comments06:49
carlosbut the old ones are removed when you add another comment06:50
carlosbradb: around?06:50
dafcarlos: weird06:50
dafcarlos: sorry, I understand now06:50
=== daf really gone
carlosdaf: don't worry06:50
carlosI will add the comment again06:51
BjornTcarlos: what exactly did you do when malone ate the comment? is it reproducable?06:58
carlosBjornT: I don't know if we can reproduce it06:58
carlosbut I suppose it's possible yes06:58
carloslet me look for a closed bug...06:58
carlosBjornT: ok, https://launchpad.net/products/rosetta/+bug/1604/+editstatus06:59
Ubugtumalone bug 1604 in rosetta ".tgz not recognized as tarball" [Normal,Fix committed]  06:59
carlosBjornT: that bug was set as fixed before the change that makes malone to store the status comments as a normal comment to the bug07:00
carlosBjornT: if you submit that form with a new comment, malone will store the new one and the old status message (that is not a bug comment) will disappear07:01
BjornTcarlos: ah, so it's that text that disappear. i thought that the comment you added got eaten up.07:03
carlosspiv: do you have time for a really fast review? I left out some changes you requested with PoMsgSetPage review and I need to request a cherrypick of that branch + this small fix 07:03
carlosBjornT: well, that's also a comment07:03
carlosand will only happen with old bugs07:04
BjornTcarlos: still kind of a bug, so you should report it.07:04
carlosok07:04
carlosBjornT: so it's not a feature, ok ;-)07:04
carloshmmm perhaps it's a bit early for spiv.... salgado, around?07:05
salgadocarlos, yes?07:06
carlossalgado: do you have time for a really fast review?07:06
=== jinty [n=jinty@196-28-44-84.jhb.netdial.co.za] has joined #launchpad
salgadocarlos, if it's realy fast, I think so07:09
salgadowhere's the patch?07:09
carlossalgado: https://chinstrap.ubuntu.com/~dsilvers/paste/filewmAh5G.html07:09
carlossalgado: if you think it's not easy to review, just tell me and I will request a normal review07:09
=== iwj [n=ian@xenophobe.extern.relativity.greenend.org.uk] has joined #launchpad
iwjAm I supposed to be able to see how my builds are coming along in soyuz ?  https://launchpad.net/distros/ubuntu/dapper/+source/firefox is a bit gnomic ?07:14
salgadocarlos, I don't feel confortable reviewing it because I'm not used to that part of the code and I won't have time to do it now. :-(07:16
carlosok07:16
carlosI will wait for spiv07:16
carlossalgado: thank you anyway07:17
salgadonp07:18
lamont-workbradb: so... I'm looking at the binary package page, and I want to file a bug...   how now, brown cow?07:18
lamont-workbradb: say from https://launchpad.net/distros/ubuntu/breezy/+package/libmp4v2-dev/07:19
SteveAdaf: did you make public those soyuz UI things we discussed?07:19
SteveAdaf: this is important for some of the questions like lamont's07:19
bradblamont-away: For now, best to specify the bin package name at: https://launchpad.net/distros/ubuntu/+filebug .07:21
bradbBinary packages seem sadly neglected atm in Launchpad.07:22
bradbWRT to the web UI.07:22
bradbcarlos: Malone eats comments?07:23
carlosbradb: I'm filing the bug atm07:23
carloslet me finish it and we comment it07:24
bradbok07:24
bradbI know that it eats status explanations, but that's usually a good thing.07:24
carlosbradb: https://launchpad.net/products/malone/+bug/3086107:26
Ubugtumalone bug 30861 in malone "Malone ate my editstatus comment" [Normal,Unconfirmed]  07:27
carlosbradb: then is the same problem07:27
carlosbradb: why is that a good thing??07:27
lamont-workbradb: that was what he noticed... :-)07:28
bradbcarlos: Malone doesn't have a status explanation field anymore.07:28
carlosbradb: the old status explanations are not migrated as comments so we are losing information07:28
carlosbradb: and that's fine, but if we migrate them as comments first....07:28
bradbcarlos: Yeah. I suggested migrating them as comments, but SteveA didn't want to do that. I think his alternative suggestion (the way we do it now) is also pretty reasonable though.07:29
carlosbradb: what's the alternative suggestion?07:29
carlosis that on production?07:29
bradbcarlos: The way we do it now.07:29
bradbYeah, with the most recent comment on change, etc.07:29
carlosbut we still lose data...07:30
=== Mez [n=Mez@ubuntu/member/mez] has joined #launchpad
bradbcarlos: yeah, no getting around that with the decision to not turn them into comments.07:30
carlosanyway, if the field is removed, how is that we still see old explanations?07:30
bradbcarlos: When a new comment on change is made, they're gone forever.07:30
SteveAcarlos: we looked through all the status whiteboards.  there were very few that were significant.07:31
BjornTbradb: it could be that malone does eat comments, though. look at bug 30832. i'm not sure what's wrong, haven't looked closely at it yet.07:32
carlosSteveA: well, it's an unexpected behaviour, perhaps we should warn the users... or at least document it or something... now that I know that I can "workaround" it.07:32
carlosanyway, I suppose it's not a big deal if you checked all those comments and nothing important was found.07:33
SteveAyeah.  the issue is, if we migrated them to comments, we'd be adding to confusion07:34
SteveAadding out-of-step comments where none was before07:34
bradbBjornT: What's the issue with bug 30832?07:34
Ubugtumalone bug 30832 in rosetta "+translation page gives me an AttributeError" [Normal,Confirmed]  http://launchpad.net/bugs/3083207:34
SteveAif you're changing the status whiteboard, then you're changing the status, so you need to decide whether the old status still applies07:34
carlosSteveA: yeah, I understand the problem07:34
BjornTbradb: he added a comment, a notification was sent, but the comment wasn't added to the bug. when he tried to add it again, it worked though.07:35
BjornTbradb: oh, wrong bug number...07:35
BjornTbradb: bug 3082307:36
Ubugtumalone bug 30823 in malone "Malone swallows comments" [Normal,Unconfirmed]  http://launchpad.net/bugs/3082307:36
bradbBjornT: Hm, this one might be hard to reproduce.07:38
=== zyga [n=zyga@ubuntu/member/zyga] has joined #launchpad
BjornTyeah.... does anyone have access to production db?07:39
kikoyes07:39
kikoread-only but yes07:39
zygahello hard working friends :-)07:39
BjornTkiko: could you check if the message table contains a message with rfc822msgid: <20060208124516.11152.78792.malone@gandwana.ubuntu.com>07:40
kiko-afkif you precook the query it will help07:40
bradbTV dinner DBA!07:41
kiko-afkbusted!07:41
kiko-afkL O L07:41
BjornTkiko-afk: select * from message where rfc822msgid = '<20060208124516.11152.78792.malone@gandwana.ubuntu.com>';07:41
kiko-afk0 rows07:42
BjornTjust be sure: select * from message where rfc822msgid = '<20060208124706.32277.45742.malone@gangotri.ubuntu.com>';07:42
kiko-afk1 row07:42
kiko-afkis there anything I can do for you? I will be back in 20 minutes07:43
BjornTkiko-afk: no, not atm, thanks07:43
kiko-afkgreat.07:43
BjornTso it seems like the transaction got aborted, but the bug notification email was sent despite that. he didn't recall any error messages either.07:44
bradbHard to say even if a transaction got aborted, I think. It could just be pebcak.07:45
SteveABjornT: can we see from the web request logs?07:45
bradbIf he can't reproduce it, it might not be worth spending too much time on. I followed up to the bug to get more information.07:45
=== poningru [n=poningru@n128-227-61-6.xlate.ufl.edu] has joined #launchpad
BjornTSteveA: we should be able to see something, yes. are they located anywhere accessible?07:45
SteveAyes07:45
SteveAchinstrap07:46
SteveA /srv/ and then you need to guess07:46
SteveAdepending which app server it was07:46
BjornTok07:46
BjornTisn't gandwana an app server?07:47
elmoyes, it is07:50
bradbsalgado: Hm, interesting, bug 30862. I guess our vocabs don't yet understand private email addresses?07:53
tsengis anyone able to merge my accounts given gpg signed confirmation?07:54
tsengone of the email addresses involved is acting up07:54
salgadoSteveA, I guess you're the only one who can help tseng here07:55
salgadobradb, yeah, that's a known problem. we (me, kiko an stub) decided that this change to the vocabs would be done in a later07:57
salgadowhat was urgent was to not show people's email addresses on their home page07:58
SteveAtseng: hello07:58
tsengSteveA: hi :)07:58
=== carlos -> out
carlossee you later07:59
bradbsalgado: ok07:59
SteveAtseng: so, you have two launchpad accounts, but cannot access the email address on one of them?07:59
tsengSteveA: yep07:59
tsengSteveA: unfortunately the one with the bad address left behind is joined to all the relevant groups07:59
SteveAi see07:59
tsengubuntu-dev core-dev etc07:59
SteveAlet's talk in private08:00
tsengnp.08:00
=== dholbach [n=daniel@ubuntu/member/dholbach] has joined #launchpad
dholbachHello.08:00
ajmitch_hi08:00
dholbachIs an issue about Malone not displaying links to attachments known already?08:01
bradbdholbach: I don't think so. Can you describe the problem, with URLs?08:01
dholbachhttp://launchpad.net/malone/bugs/29253 for example is a bug that has 2 attachments (I got that from mails that were sent to me because of desktop-bugs@) - it doesn't display the links to the attachments on the page.08:02
Ubugtumalone bug 29253 in gdm "Crash on second login" [Normal,Needs info]  08:02
bradbdholbach: How were the attachments fed into malone?08:04
dholbachNo idea, sorry - as you can see from the Activity Log, I was not involved.08:05
=== ogra [n=ogra@ubuntu/member/ogra] has joined #launchpad
ograhttps://launchpad.net/distros/ubuntu/dapper/+source/ltsp/+pots/pkgconf-ltsp/de/+translate08:06
ogragives me an opps 08:06
matsubaraogra: known problem08:07
ograoki08:07
matsubaraogra: bug 317608:07
Ubugtumalone bug 3176 in launchpad "Error when trying to save AbiWord pt-BR translations" [Normal,Fix committed]  http://launchpad.net/bugs/317608:07
bradbBjornT: The email interface doesn't know about attachments, right?08:07
ograoh, i was only accessing it ...08:07
ogra(not saving) 08:07
matsubaraogra: but it's the same bug. lots of reports about it lately08:08
BjornTbradb: that's right. there's a bug open on that08:08
ograoki, then i'm fine ...08:08
ograwill be patient ...08:08
bradbdholbach: So if these attachments were fed in via email, that would explain it.08:08
=== licio [n=licio@ubuntu/member/licio] has joined #launchpad
dholbachOh, if it's that known bug, then it's ok for me.08:09
bradbdholbach: One thing: were the emails you received Cc desktop-bugs, and addressed to malone?08:10
ogradholbach, seems working for the launchpad team is way easier ;) they know all their bugs in advance :)08:10
bradbWell, i.e., was the Malone submit bug address a recipient of the email08:10
dholbachbradb: I can forward them to you.08:11
bradbdholbach: sure, that'd be good. brad.bollenbach@gmail.com.08:11
dholbachok08:11
=== zyga [n=zyga@ubuntu/member/zyga] has joined #launchpad
=== doko [n=doko@217.205.109.249] has joined #launchpad
=== zyga [n=zyga@ubuntu/member/zyga] has joined #launchpad
bradbBjornT: Just so we can let dholbach know, when will that attachment junk stripper be rolled out?08:51
BjornTbradb: it has been rolled out, i think it was last week08:52
bradbah, ok, great.08:52
dholbachWouldn't it be more useful to just display the list of links? (as they seem to be in librarian already)08:54
kikothat is an idea08:54
kikocprov, could you answer to siretart (Reinhard)'s email to launchpad-users?08:58
cprovkiko: will see08:58
kikoDate: Sun, 5 Feb 2006 20:10:07 +010008:58
kikothank you very much08:58
bradbBjornT: One thing still confuses me: why was this stored as an attachment: http://librarian.launchpad.net/1538638/unnamed ?08:59
bradbThese particular attachments would be useless to show in the UI, because it's just the entire original message.09:00
zygahi09:00
zygaFAQ: is dapper open yet 09:00
kikomatsubara, I am looking at your fix for bug 93309:00
kikoI have a question09:00
zygaI can make a wiki I'd just need a reliable answer09:00
kikowhat happens if you post a second bugtracker with the same name?09:00
kikozyga, yes, it is open for development09:01
zygakiko: for translations09:01
kikoah. that is a carlos question09:01
zygacarlos: ^^^09:01
matsubarakiko: BugTaskNameField enters in action, why?09:01
kikomatsubara, not BugTrackerNameField?!09:02
BjornTbradb: every email, with all its attachments, is stored in the librarian. we'll probably move away from that, and extract the parts we're interested in, but it has low priority atm.09:02
matsubarakiko: ops, yep BugTrackerNameField09:02
kikookay.09:02
kikothanks.09:02
bradbBjornT: That's what I thought. So the emails that I forwarded to you indicate that HMRubuntu used the email interface in these interactions with Malone, right?09:03
BjornTbradb: yes09:03
bradbriiiight. It all fits together now.09:03
bradbSo, dholbach, normally we do show links for "real" bug attachments, but what happened here is that (1) HMRubuntu used the email UI to interact with Malone so (2) his emails were stored as attachments and then (3) there was a Malone bug that spewed attachment junk at the end of messages like that, which BjornT fixed. But it appears that HMRubuntu did add an attachment to this bug, per se. Does that sort of make sense?09:05
bradbs/did add an attachment/didn't add an attachment/09:06
dholbachOh yeah, might be. What is the plan? Strip attachments off?09:07
dholbachI'd like it better for them to remain and be indexed for a portlet on the right side (as normal attachments do as well).09:08
bradbdholbach: Normal attachments are. But there were no real attachments added to this bug. It was an implementation detail of Malone (that it stores all incoming mails as attachments) that leaked up to the interface, in this case.09:09
bradbBut if you were to add a patch to a bug through the web UI (because it's not possible through the email UI), it would be linked in a portlet from the bug page.09:09
dholbachHm, how about adding it by mail?09:09
bradbAssuming we still have that portlet. :) At worst, attachments are also linked with the comment that was made when they were submitted.09:10
bradbdholbach: bug 3022509:11
Ubugtumalone bug 30225 in malone "Attach files via email" [Normal,Confirmed]  http://launchpad.net/bugs/3022509:11
dholbachok09:11
dholbachgood night.09:17
bradbsee ya09:18
=== tseng [n=tseng@unaffiliated/tseng] has left #launchpad []
dilysMerge to devel/launchpad/: [trivial]  Removing obsolete changelog 'linkification' process and added tests for the remaining code. The missed author's changelog line is fixed in the Soyuz codeline. (r3106: Celso Providelo)09:45
=== ajmitch__ [i=ajmitch@port161-160.ubs.maxnet.co.nz] has joined #launchpad
=== ajmitch__ is now known as ajmitch
=== doko [n=doko@217.205.109.249] has joined #launchpad
kikoajmitch: your link to the Launchpad firewalls spec entry is broken at https://wiki.ubuntu.com/Firewalls10:09
kikojust fyi10:09
ajmitch_no surprise, I should fix that to point to the right wiki page10:11
kikoactually the reverse link is what's broken10:11
kikowiki to lp10:11
ajmitch_but the wiki page is wrong anyway :)10:12
kikoI love error reports10:17
cprovgood night guys10:17
ajmitch_kiko: looks like it's pointing to the right wiki page (https://wiki.ubuntu.com/SoC-Firewall)10:30
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has left #launchpad []
ajmitch_ https://wiki.ubuntu.com/Firewalls is old, I added the lp spec link by mistake10:30
kikoaha10:31
=== jinty [n=jinty@196-28-45-139.jhb.netdial.co.za] has joined #launchpad
kikolifeless, how's the story testrunning going?10:51
=== carlos [n=carlos@84.76.255.40] has joined #launchpad
carloszyga: no, dapper is not yet open to translate but I'm working on testing the new process and if all things go well we will start importing translations.11:15
lifelesskiko: it works, I just need to do spivs review requests11:15
zygacarlos: I'd like to make a FAQ on the wiki11:16
lifelessI might ask spiv to apply them actually, bzr stuff is pretty intense for the next 3 weeks - Major release and data format stability stuff11:16
zygacarlos: can you give me any estimate (even from the top of your head?)11:16
lifelessessentially sprinting every day11:16
lifelessspiv: what do you think ?11:16
zygait's mainly for translators that bump in and ask: 'ah, what about dapper'11:16
kikolifeless, please hand it off to spiv.11:17
lifelesskiko: you want it bad dontcha11:17
kikoyeah :)11:17
carloszyga: I really don't know. I will try to start doing imports next week. I suppose you could say at the end of the month and if it's earlier (I hope that) people will be more happy ;-)11:18
zygacarlos: k11:18
zygacarlos: any suggestions to translators that ask: "what do to till then?"11:19
carloszyga: translate Hoary and Breezy11:19
carlosthey are still supported11:19
carlosand the translations will be reused with dapper11:19
zygak, thanks11:20
kikoBjornT, thanks :)11:23
BjornTnp :)11:23
kikogo get some sleep!11:24
BjornTsounds like a good idea11:25
=== gneuman is now known as gneuman-zzz

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