/srv/irclogs.ubuntu.com/2006/04/30/#launchpad.txt

mdkes/3 or 4/10 or 1112:08
=== mdke wonders if the majority of open ubuntu bugs are duplicates which have been fixed
sfllawWe shall try to tell you soon.12:12
mdkesfllaw, mm?12:12
sfllawWell, for some definition of soon.12:12
mdkeand what definition of we?12:12
sfllawmdke: Hi.  I'm the new guy.12:13
mdkehello, are you on malone?12:13
sfllawhttps://launchpad.net/people/sfllaw12:14
mdkeah, I thought you meant you were working on launchpad12:14
=== baconbacon [n=bacon@modemcable020.207-130-66.mc.videotron.ca] has joined #launchpad
salgadolifeless, noted that12:15
sfllawOh no.  We're going to be doing this the old fashioned way.12:15
=== sfllaw sighs.
mdkekiko's last comment on bug #3796 tends to suggest that duplicates aren't included in most bug lists, not sure though12:15
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/379612:15
kikoeu?12:15
salgadolifeless, would you like to review that branch or shall I fix this and resubmit it for review?12:15
kikomdke, they aren't -- in any [by default]  that I know of12:15
mdkekiko, oh, I just don't understand geek-speak then12:16
mdkeso all those 10k ubuntu bugs are probably lots of dups?12:16
kikoI was agreeing with you!12:16
mdkeoh damn12:16
lifelesssalgado: I've emailed you :)12:16
kikoheh12:16
mdkekiko, I got excited there for a moment12:16
kikoas I said -- they aren't [displayed by default] 12:17
lifelesssalgado: I'd say, fix it, and the duplicate code, then resubmit it12:17
kikoI fixed the last bug I knew there, which was in the recent bugs portlet12:17
mdkek12:17
mdkeso there are 10k real bugs, that is crazy12:17
salgadolifeless, great. thank you12:19
=== salgado => home
=== Lorenzod [n=lorenzod@63.218.103.162] has joined #launchpad
=== Lorenzod [n=lorenzod@63.218.103.162] has joined #launchpad
=== sabdfl [n=mark@ubuntu/member/pdpc.silver.sabdfl] has joined #launchpad
baconbacon10k but for many apps01:00
baconbacon...about 16k apps iirc01:00
lifelessits < 1 per app01:00
=== WaterSevenUb [n=WaterSev@195-23-238-238.nr.ip.pt] has joined #launchpad
=== mpt__ [n=mpt@222-154-112-147.jetstream.xtra.co.nz] has joined #launchpad
=== tambaqui [n=tambaqui@200-183-98-222-mns.cpe.vivax.com.br] has joined #launchpad
=== mpt__ [n=mpt@222-154-112-147.jetstream.xtra.co.nz] has joined #launchpad
=== cprov [n=cprov@201-43-145-181.dsl.telesp.net.br] has joined #launchpad
=== mpt [n=mpt@219-89-132-63.jetstart.xtra.co.nz] has joined #launchpad
=== stub [n=stub@ppp-58.8.1.90.revip2.asianet.co.th] has joined #launchpad
=== stub [n=stub@ppp-58.8.1.90.revip2.asianet.co.th] has joined #launchpad
=== PenguinOfDoom [n=PenguinO@spleen.xzrq.net] has joined #launchpad
PenguinOfDoomHow do I increase the severity of a bug?04:15
PenguinOfDoomoh, it's per-distribution04:15
PenguinOfDoomgrumble04:15
=== mpt_ [n=mpt@222-154-181-148.jetstream.xtra.co.nz] has joined #launchpad
=== PenguinOfDoom [n=PenguinO@spleen.xzrq.net] has left #launchpad []
=== LeeJunFan [n=junfan@adsl-69-210-207-5.dsl.klmzmi.ameritech.net] has joined #launchpad
=== mpt__ [n=mpt@219-89-151-229.jetstart.xtra.co.nz] has joined #launchpad
=== auth00_ [n=auth@fiji.grd.sgsnet.se] has joined #launchpad
=== mpt_ [n=mpt@219-89-138-28.jetstart.xtra.co.nz] has joined #launchpad
=== poningru [n=poningru@ip68-226-0-76.ga.at.cox.net] has joined #launchpad
=== nictuku [n=yves@ubuntu/member/nictuku] has joined #launchpad
nictuku"Your subscription to this team has been deactivated. You can't join this team. " what does this mean? I was banned from the group?05:05
=== stub [n=stub@ppp-58.8.1.90.revip2.asianet.co.th] has joined #launchpad
=== mpt__ [n=mpt@219-89-156-29.jetstart.xtra.co.nz] has joined #launchpad
=== mpt_ [n=mpt@219-89-156-29.jetstart.xtra.co.nz] has joined #launchpad
=== _drak0__ [n=rob@user-10cmeb8.cable.mindspring.com] has joined #launchpad
=== _drak0__ [n=rob@user-10cmeb8.cable.mindspring.com] has left #launchpad []
=== carlos [n=carlos@105.Red-81-34-77.dynamicIP.rima-tde.net] has joined #launchpad
carlosmorning08:39
=== tambaqui [n=tambaqui@200-183-98-222-mns.cpe.vivax.com.br] has left #launchpad []
sivangmorning all08:47
carlosstub: hi, around?08:51
stubcarlos: yes08:51
carlosI'm having problems with the code updates you do on mawson in the afternoon, or at least I think that's the problem...08:51
carlosI breaks the language pack exports08:52
carloss/I/It/08:52
stubI don't do code updates on mawson. Do you mean asuka?08:52
carloshttp://mawson.ubuntu.com/~carlos/rosetta-dapper-2006-04-24.log08:52
carlossorry, asuka, yes08:52
carlosI have lock problems08:52
stubI can't stop that without disabling the afternoon code updates. They were really only needed for the sprint, so I don't have a problem with that.08:53
carlosstub: would the problem be: 08:53
carlos        for (id,) in cur.fetchall():08:53
carlos            yield POTemplate.get(id)08:53
carloswould it be fixed if I create a list first? and stop using the cursor?08:54
stubNot really - locks you open remain that way until you commit or rollback.08:54
carloshmmm, no, the break is when doing the query....08:54
carlosyeah, and another process is doing the lock08:55
carlosstub: well, disabling the code update would fix it now, but when we move this script into production...08:55
stubYou are competing with a script that removes and rebuilds the permissions on all the tables - no way you can avoid locking. This is why we shutdown the production launchpad for database updates.08:55
carlosit would be worse08:55
carlosok08:55
carlosso you think it would not be an issue on production?08:56
lifelesscarlos: can you reschedule teh exports to not conflict ?08:56
carlosI did it already08:57
carlosLet me see how long it takes now and try to find a better time to execute it....08:57
carlosstub: when does the morning update finish?08:57
carlosthe one with the database mirror included08:58
stubcarlos: If it triggered by the database updates, then this won't be an issue on production08:58
carlosok08:58
carlosmy export script takes around 3 hours 08:59
carlosand I would need as close as possible to the db mirror to be able to test it earlier instead of one day after doing changes on production (like new .pot/.po imports)09:01
carlosstub: I'm executing it at 12:30 (mawson time)09:01
stubYou shouldn't get too close, as the finish time isn't that fixed09:01
=== stub waits for his trash mailbox to load
carlosyeah, that's why I moved it later, but seems like I moved too late and the afternoon update broke it again09:02
stubStart at approx 8:30 and finish approx 12:00 server time09:02
stubI'll disable the midday update09:03
carlosthat means I should delay it a bit more or the afternoon update should be done later (3 hours after the morning update is a bit early, isn't it?)09:03
stubIt was requested at that time for the sprint.09:04
stubLike I said - it is pretty pointless now that the sprint is over09:04
carlosstub: I think lifeless prefer that we leave it there instead of removing it09:04
carloslifeless: ?09:04
lifelessI don't care09:05
lifelessI think steve might09:05
lifelessand its night to have staging really close to prod09:06
lifelessplus the po exports really should not be broken every time we sprint09:06
lifeless-> should solve it permanently09:06
stubit is just a code update - doesn't make any difference to data freshness09:06
lifelessI'm talking code!09:06
stubpo exports shouldn't be running on staging.09:07
lifelesssorry, I meant 'close to HEAD'09:07
lifelesslanguage packs are .. mo exports  not po, sorry.09:07
lifelessI meant language packs though09:08
carloslifeless: well, the po export should move to production servers before next sprint09:09
carloswe are runing them on staging because that give me much more flexibility, but I hadn't to change any code recently related to them, or at least I hadn't to change any code that would not wait for the usual review and rocketfuel merge procedure, atm we are running there rocketfuel code directly09:11
lifelessso why not run on production now ?09:17
stubhe is chicken ;)09:18
lifelesssquawk squawk squawk09:19
=== stub goes for a swim
carlos:-P09:23
carloslifeless: if I get a full run today, and works as expected, I will start planning its movement to production09:23
=== doko_ [n=doko@dslb-088-073-100-191.pools.arcor-ip.net] has joined #launchpad
=== BjornT [n=bjorn@213.226.190.253] has joined #launchpad
mpt_stub, are you able to cherrypick r1815?10:04
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has joined #launchpad
carloslifeless: hi, we had a weird behaviour with bzr, I'm not sure if it's related but it happened with the revision number change, the r3488 that is know now as r1814 (I think that was the last commit with the old revision numbers)10:18
carloslifeless: I added a new file on that revision and merged it into rocketfuel10:18
carlosbut rocketfuel doesn't have that file10:18
mpt_carlos, lifeless said that the revision numbers will all fix themselves once he finds that bug10:19
carlosand my mirror has it so I'm sure I didn't forget its addition10:19
carlosmpt_: do you mean that the missing files is a known bug?10:20
carlosmpt_: or just that the revision number will be fixed?10:20
carlosin that case, is not just the revision number, but data lose...10:20
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
ddaacarlos: the revno thing should not affect the merge logic, I think.10:25
mpt_carlos, I don't know anything about any missing files10:26
ddaawhether a branch has new stuff compared to another one is based on the revid of the tip10:26
ddaacarlos: what is your branch with the problem?10:27
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== SteveA [n=steve@195.182.78.95] has joined #launchpad
SteveAhi10:35
carlosddaa: chinstrap:/home/warthogs/archives/carlos/launchpad/bug-252910:37
carlosSteveA: hi10:37
carloshmmm10:40
carlosddaa: seems like it's there now...10:40
stubmpt_: Probably. I was going to do a production update today and bring it in then.10:40
mpt_great10:41
SteveAmpt_: aren't you on holiday? :-)10:41
mpt_SteveA, yes, but I said I'd be on IRC some of the day10:42
=== jinty [n=jinty@84.Red-83-55-199.dynamicIP.rima-tde.net] has joined #launchpad
ddaacarlos: well, if you say the file is in rocketfuel now, no need for me to investigate.10:43
carlosddaa: right, thank you10:45
carlosstub: the migration script for the cherry pick you did for me is now on rocketfuel, could you execute it now?10:46
stubcarlos: Do you know what revision it landed in?10:47
carlosstub: well, I know the revision where it should be, and is the same you cherry picked...10:48
carlosI didn't ask for any other merge since then10:48
carlosr3488 or r1811 with the new numeration10:49
stubDo you have a rough idea on how long it will take to run?10:49
carlosstub: no, but I don't think it would take more than one hour, it depends on the number of POFiles we have in our database10:50
carlosI think it would do the job in 15-20 minutes10:51
carlosbut that's only a guess10:51
carlosif you want10:51
carlosI could time it on staging10:51
stubno - rough is fine.10:51
carlosok10:55
stubLooks like rolling out head is the thing to do - almost everything landed since friday are desirable cherrypicks11:00
stubLaunchpad will be going down in just over 45 minutes at 10:00 UTC for its regular code update. Estimated downtime is 15 minutes.11:13
doko<doko> carlos, pitti: I don't see any translations on rookery from the last OOo build?11:44
doko<pitti> doko: hm, maybe you still had NO_PKG_MANGLE defined?11:44
doko<doko> pitti: no, and that wouldn't explain that carlos had something imported11:44
stubLaunchpad will be going down in 15 minutes at 10:00 UTC for its regular code update. Estimated downtime is 15 minutes.11:45
carlosdoko: I don't control rookery tarballs, I cannot do anything there...11:45
=== tortho [n=tortho@6.84-49-112.nextgentel.com] has joined #launchpad
=== WaterSevenUb [n=WaterSev@195-23-238-173.nr.ip.pt] has joined #launchpad
BjornTmpt_: ping12:05
mpt_BjornT, pong12:05
BjornTmpt_: do you have any suggestions how to make it possible to unsubscribe a team, without adding another menu item?12:06
mpt_BjornT, the current UI structure kind of requires another menu item12:07
=== koke [n=koke@ubuntu/member/koke] has joined #launchpad
mpt_hmmm, unless ...12:08
mpt_... unless "Subscribe" becomes "Subscribe/Unsubscribe" if a team you are a member of is subscribed to the bug12:08
mpt_and then the resulting "Subscribe/Unsubscribe" page has radiobuttons for the things you want to do12:09
mpt_(*) Subscribe me12:09
mpt_( ) Unsubscribe Team I'm a Member Of A12:09
mpt_( ) Unsubscribe Team I'm a Member Of B12:09
mpt_That would be a gross hack, but it would avoid adding a menu item12:09
mpt_Ideally, this would be done by the people/teams that you had permission to modify in the list of subscribers having checkboxes12:10
mpt_To unsubscribe yourself or a team you're a member of, uncheck the relevant box12:11
SteveAwell12:12
SteveAthere are a number of states12:12
BjornTyeah, i'll go for the "Subscribe/Unsubscribe" solution for now. it's quite easy to do, and the menu won't get any longer than it already is.12:12
SteveA - not subscribed12:12
SteveA - subscribed as me12:12
SteveA - subscribed as team / teams12:12
SteveA - subscribed as me and team / teams12:12
SteveAthe menu item can say12:12
ddaahey SteveA12:12
SteveA - unsubscribe12:13
SteveA - unsubscribe my team12:13
SteveA - unsubscribe me and my team12:13
SteveAin the (hopefully) rare case you want to unsubscribe a team, but not yourself, we can either offer a UI, or you unsubscribe it all, and resubscribe yourself 12:13
mpt_Three menu items?12:13
SteveAno12:13
SteveAwell, yes12:14
SteveAbut only one ever visible at one12:14
SteveAonce12:14
BjornTSteveA: so, if the menu item says 'unsubscribe my team', how do i subscribe myself?12:14
ddaais Launchpad eligible for SOC?12:14
mpt_SteveA, that would be two extra page loads for the rare case, and the same number of page loads for every other case12:15
SteveABjornT: it would say "unsubscribe me and my team"12:16
mpt_so, what is the advantage of doing it that way?12:17
SteveAah...12:17
SteveA<mpt_> (*) Subscribe me12:17
SteveAso, that wasn't a type12:17
SteveAtypo12:17
SteveA(how ironic i typo the word typo)12:17
BjornTSteveA: i don't quite understand. if one of my team is subscribed, but i'm not, what will it say?12:17
SteveAi was confused by one radio button being "subscribe me" and the rest being "unsubscribe team"12:17
SteveAmixing "subscribe" and "unsubscribe" in the same control is confusing12:18
mpt_Yes, it is12:18
mpt_We already have such evilness for unmarking duplicates12:18
mpt_but the alternative is to add another menu item12:18
mpt_or to add more page loads12:18
SteveAmy overall point, confusion aside, is that we can make the menu item give more of a clue as to what it is offering12:19
SteveAby making it more context-specific12:19
BjornTwell, it's hard to get much information into the item and still have it short enough to fit into the menu box.12:24
=== SteveA thinks of icons
SteveAa person going into a waste-disposal device, vs a group of people going into the same waste disposal device ;-)12:25
=== SteveA got a degree from the school of inappropriate icons
jordicarlos?12:25
SteveAmpt_: magic dhtml extra menu items...12:25
carlosjordi: hi12:25
mpt_Soylent Green Subscriptions12:26
=== mpt__ [n=mpt@219-89-140-108.jetstart.xtra.co.nz] has joined #launchpad
=== BjornT -> lunch
=== mpt [n=mpt@219-89-150-174.jetstart.xtra.co.nz] has joined #launchpad
=== SteveA --> lunch
=== Keybuk [n=scott@quest.netsplit.com] has joined #launchpad
Keybukuh, help01:35
Keybuksync-source is bailing out most of the time saying things like:01:35
KeybukE: bsdgames_2.17.orig.tar.gz (from bsdgames) returns multiple IDs ([(1332643,), (1494584,)] ) for orig.tar.gz.  Help?01:35
=== SnkBite [n=SnkBite@212.25.63.82] has joined #launchpad
elmoKeybuk: that shouldn't happen, that means there are multiple different orig.tar.gz's with different content, that's stunningly bad01:46
elmoKeybuk: and it's not a sync-source problem01:46
Keybukthe content is the same in both Ubuntu and Debian, if that's what you mean ?01:46
elmono, I mean in soyuz01:46
KeybukKinnison says that the uploader may be just making duplicate aliases for the same actual .orig.tar.gzs01:47
Keybukit's doing this for pretty much every package I'm trying01:47
elmosync-source already deals with that01:47
elmoit has an explicit "is the sha1 and size the same?  if so, ignore it" check01:47
Keybukso who has to be beaten up to get this fixed?01:47
Kinnisonelmo: yargh, okay I need to looksee01:48
elmokeybuk: well, probably not me, but I need to run to a meeting anyway01:48
=== Kinnison needs to grab some lunch, then I'll take a good look at the db
KeybukKinnison: thanks *hug*01:49
=== bradb [n=bradb@modemcable092.66-130-66.mc.videotron.ca] has joined #launchpad
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
cprovmorning guys02:02
mptbradb, pong02:02
=== matsubara [n=matsubar@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
matsubaragood morning!02:06
bradbmpt: Was going to ask you to have a look at the UI for the fix for bug 977. (It's in rf.)02:09
UbugtuMalone bug 977 in malone "Commenting on bug should optionally subscribe you" [Major,Fix committed]  http://launchpad.net/bugs/97702:09
mptThat would require me to go off IRC to get rocketfuel02:10
mptHave a demo/screenshot?02:10
mptor shall I just look at it tomorrow?02:10
bradbmpt: Tomorrow'd be fine.02:14
bradbIt's nothing life changing.02:14
mptok02:15
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
ddaamatsubara: good catch02:26
ddaaI mean bug 4116402:26
matsubaraddaa: ?02:26
UbugtuMalone bug 41164 in launchpad "IPerson.getBranch() should return only one result" [Critical,Confirmed]  http://launchpad.net/bugs/4116402:26
ddaacritical db constraint missing02:26
salgadolifeless, around?02:26
matsubaraddaa: thanks to the oops report :)02:26
ddaano idea how that ended up this way, I cannot believe I just _forgot_ to add that constraint02:27
matsubaraddaa: can you fix it?02:28
ddaaShould be trivial to fix.02:29
ddaawell, that also means that some form do not do proper input validation02:29
ddaabut that's MUCH less of a problem02:29
=== mpt_ [n=mpt@219-89-142-164.jetstart.xtra.co.nz] has joined #launchpad
=== mpt_ [n=mpt@219-89-142-164.jetstart.xtra.co.nz] has left #launchpad ["http://mpt.net.nz/"]
Keybukok, that's kinda interesting02:42
Keybukmy sync queue just got moved to failed, not accepted02:42
Keybukno errors, just lots of02:43
Keybuk12:41:44 WARNING Exception during processing made it out of the main loop.02:43
Keybuk -> http://librarian.launchpad.net/2339224/lEYjJjYCKy0XmqVqfzZaazPPLYK.txt (No To: header)02:43
=== mpt_ [n=mpt@219-89-142-164.jetstart.xtra.co.nz] has joined #launchpad
=== mpt [n=mpt@219-89-142-164.jetstart.xtra.co.nz] has joined #launchpad
salgadoKeybuk, I think there's a bug reported for that02:47
salgadoyep, bug 4095802:47
=== carsten [n=carsten@wikipedia/carsten] has joined #launchpad
salgadoUbugtu, you slacker!02:48
UbugtuMalone bug 40958 in qprocd "all sync attempts fail" [Normal,Confirmed]  http://launchpad.net/bugs/4095802:48
Keybuk*sigh*02:48
=== BjornT [n=bjorn@clt-84-32-240-183.dtiltas.lt] has joined #launchpad
kiko-zzzcprov, is that codepath not tested?03:11
kikohey BjornT 03:11
=== carlos -> lunch
carlossee you later03:11
kikohey carlos 03:12
cprovkiko: barely only, I think we never reached this subtle situation03:12
carloskiko: hey dude!03:12
kikodid you give matsubara a hand trying to figure out that crasher in rosetta?03:12
BjornThi kiko 03:12
kikocprov, really? isn't it that all syncs are busted?03:12
carloskiko: which crash?03:12
kikocarlos, the TypeError?03:12
matsubarakiko: yes he did.03:12
kikothe only rosetta crasher open that I know of03:12
kikoah cool03:12
carloskiko: I talked with matsubara last week03:12
cprovkiko: yes, I think so03:12
carlosmatsubara: but did you manage to reproduce it?03:13
kikocprov, so what I'm saying is that we are not functional-testing the sync process03:13
matsubaracarlos: unfortunately, no03:13
cprovkiko: just investigate that zakame issue 03:13
kikocarlos, well, notice that there are /two/ string 13s in the form03:13
carlosmatsubara: let's talk when I'm back from lunch about it, ok?03:13
kikocarlos, how could that be possible?03:13
kikoanyway k03:13
matsubaracarlos: ok.03:13
carloskiko: no idea ;-)03:13
kikocarlos, seems to be user-dependent too03:13
carloskiko: I don't even know how is possible that we get a python list dump there....03:14
kikoi.e. BjornT sees two string 13s but matsubara and I only see one.03:14
kikocarlos, it happens when there are two fields with the same form name.03:14
kikothat's how you get the python list03:14
cprovkiko: I'd say so, it's not properly tested, as you can see 03:14
carloskiko: oh, that's good03:14
kikoyeah03:14
carlosI mean if you found a way to reproduce it03:15
kikocarlos, we'll need to find out what codepath causes two string 13s to be displayed03:15
carloskiko: I need to leave now, I will take a deeper look when I'm back03:15
carlosthanks for the extra info03:15
carloskiko: right03:15
matsubarakiko: it's string 14. The garbage I told you in string 13 is the html that should be rendered as the second 14 textarea03:15
kikosorry03:15
kikostring 1303:15
kikostring 1403:15
carlosI guess it depends more on your locale preferences or IP address03:16
kikomatsubara, that "garbage" is actually entered by the translator, I suspect03:16
kikocarlos, exactly what I think03:16
carlosok03:16
carlossee you later!03:16
matsubarakiko: do you think so? I run a diff between the html generated by opera and by firefox and there's no difference03:16
kikomatsubara, I am /sure/ there is a difference03:17
kikomatsubara, and, for the 10th time, this is /not/ browser-dependent03:17
kikoit most likely depends on the user's calculates languages03:18
matsubaraI understood that and I'm not saying it's browser dependent.03:18
kikobut you said "opera" versus "firefox" above!03:18
=== niemeyer [n=niemeyer@200.103.151.176] has joined #launchpad
ddaamh... apparently, the db constraint just got forgotten...03:23
ddaahow embarassing03:23
=== stub [n=stub@ppp-58.8.1.90.revip2.asianet.co.th] has joined #launchpad
ddaastub!03:25
stubddaa!03:25
kikoheya guys03:25
ddaacan you make a itsy bitsy littly db patch like, right now, with a cherrypick?03:25
=== kiko looks at ddaa
ddaaI'll try phrasing what I want in sql if you want.03:26
stubsql or english03:26
ddaain english03:26
ddaadrop all Branch with duplicate (owner, product, name), keep the one with smallest id03:26
ddaaadd unique branch constraint on (owner, product, name)03:27
stubok03:28
ddaanote that product may be NULL, dunno how that can affect unique matching03:28
ddaathe delete should delete just 2 rows03:29
kikoddaa, that constraint needs a code check, I assume?03:30
ddaakiko: code check?03:30
kikowell, something to avoid OOPSing when it happens03:30
ddaawell, the form that allowed violating the unspoken constraint definitely needs fixing03:30
ddaaand other forms too03:30
ddaabut that's a low priority03:31
kikoif it generates an OOPS it is high priority03:31
kikoyou don't need to do it yourself though03:31
ddaaI mean lower than fixing the db constraint.03:31
kikojust make sure it gets done03:31
ddaasince the constraint violation cause _everything else_ to blow up03:31
kikoyou mean not having the constraint I guess03:32
kikoyeah03:32
ddaait's a "violation" in that a lot of things rely on that constraint, in the code (selectOneBy) and in the design.03:33
kikoindeed03:33
ddaamh...03:33
ddaastub: please run the following query and mail me the output before running the delete in production: select branch.id, branch.owner, person.name as owner_name, branch.product, product.name as product_name, branch.name from (select owner, product, name from branch group by owner, product, name having count(*) > 1) as dup_branch join branch using (owner, product, name) join person on branch.owner = person.id join product on branch.produ03:35
ddaait may require some cleanup on the supermirror filesystems03:35
stubddaa: pastebin please - that got cuttoff.03:35
ddaahttps://chinstrap.ubuntu.com/~dsilvers/paste/filejTBhBh.html03:36
ddaabasically, I just want to know which branches got deleted, because the branch mirroring stuff does not care about that constraint03:37
=== radix [n=radix@dsl092-068-248.bos1.dsl.speakeasy.net] has joined #launchpad
=== lamont [n=lamont@mib.fc.hp.com] has joined #launchpad
=== fabbione [i=fabbione@gordian.fabbione.net] has joined #launchpad
=== lifeless [n=robertc@dsl-43.1.240.220.rns01-kent-syd.dsl.comindico.com.au] has joined #launchpad
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== zorglub [n=zorglub@194.146.226.84] has joined #launchpad
stubddaa: https://chinstrap.ubuntu.com/~dsilvers/paste/fileay8ntH.html03:38
ddaastub: you have already applied the constraint?03:38
stubddaa: Not yet03:39
stub('ddaa', NULL, 'fred') and ('ddaa', NULL, 'fred') should conflict I assume?03:39
ddaayes03:40
ddaaNULL product == pseudo product with name '+junk'03:40
stubddaa: Done03:46
ddaathank you, I'm looking forward to see it landing in RF03:47
=== ddaa is sick, going for a nap
=== sabdfl [n=mark@ubuntu/member/pdpc.silver.sabdfl] has left #launchpad []
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
kikostub, we're only rolling out head-ish next week, right?03:52
=== stub [n=stub@ppp-58.8.1.90.revip2.asianet.co.th] has joined #launchpad
stubkiko: Almost everything that had landed since last week was either requested for cherry pick, or was a good candidate for a cherry pick. So yes - one off.03:54
kikostub, wonderful. next tuesday, then?03:54
=== stub [n=stub@ppp-58.8.1.90.revip2.asianet.co.th] has joined #launchpad
=== beyond [n=beyond@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
bradbSteveA: Any chance of some code review love today?04:03
=== bradb is in desparate need
kikobradb, what's up?04:04
bradbCode review blockage04:04
kikothat was obvious; more specifically?04:04
bradbalso, a decision on binary package name, as per my email04:04
kikoI thought we had agreed to just put it in the description. I don't care about data migration, it's your call.04:05
bradbkiko: Yeah, it's already in the description, just the data migration issue. But if you don't care, then I say we just blow it away. We'll offend two or three people, and life will go on.04:05
kiko"Have you ever killed anybody? Me? No. But I hurt someone's feelings once."04:07
bradbheh04:07
=== bradb lands it then
=== jkakar [n=jkakar@204.174.36.228] has joined #launchpad
sivangkiko: hehe04:21
kikowhat movie is that from sivang?04:21
SteveAbradb: you know i'd love to give your code some lovin'04:25
Kinnisonstub: is it possible to create temporary tables inside a transaction when connected as user 'ro' ?04:27
Kinnisonactually, that question goes to anyone who might know04:31
bradbSteveA: !04:33
SteveAtough love04:34
bradbbooty call04:34
SteveAthe bradb code review booty call04:35
SteveAi like it04:35
bradbheh04:35
SteveAddaa: why is the docs stuff in my review queue?04:36
stubKinnison: Dunno ;-)04:36
Kinnisonstub: hhe04:37
Kinnisonstub: it seems it is permissible04:37
=== Kinnison continues doing scary sql research as ro
Kinnisonjust with much shorter queries now04:37
SteveAjamesh: nice improvements to the pending reviews page!04:38
Kinnisonif I have a table with three columns in it, namely aliasid, filename and sha1, how can I select all the rows which have a unique (filename,sha1) tuple?04:40
=== sits [n=sits@cpc3-cwma2-0-0-cust739.swan.cable.ntl.com] has joined #launchpad
SteveAbradb: i'm looking at bug 36866 now, in preparation for the code review04:43
UbugtuMalone bug 36866 in malone "Searching for bugs after selecting a certain status from the Right-Hand-Menu resets the search" [Major,In progress]  http://launchpad.net/bugs/3686604:43
SteveAbradb: one thing about bug reports like this: always edit the description to include an actual URL to start the process from04:43
SteveAthat way, looking at the problematic workflow is much easier for someone looking at the bug04:43
SteveAmatsubara: please note the above too04:43
bradbSteveA: updated :)04:45
matsubaraSteveA: ok04:48
=== lbm [n=lbm@82.192.169.174] has joined #launchpad
kikoKinnison, select distinct on (filename, sha1)05:00
kikoKinnison, you can use that in a subselect if necessary05:00
kikoand order by to choose which row you want05:01
Kinnisonkiko: yah, someone gave me that in another channel just now, thanks dude05:01
=== Kinnison never knew of the ON(blah) bit of select distinct
kikoif you need some help using it let me know as I rewrote some code to use it recently05:01
Kinnisonthanks, I think I've got the answer I needed, but I may be back for more hints05:01
kikocarlos, ping?05:02
=== lakin [n=lakin@S01060013101832ce.cg.shawcable.net] has joined #launchpad
=== ddaa crawls out of his nap
ddaaSteveA: not sure why doc-bazaar is in your queue, after you replied to to lifeless' question during your vacation, I made some changes to the README, quicked it back to needs-review, then it went dormant. Only this week was I asked to get it moving again, so I said I already did...05:10
ddaaat some point in the meantime, I guess it was moved out of spiv's queue (who is in vacation if I am correct)05:11
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
=== radix [n=radix@dsl092-068-248.bos1.dsl.speakeasy.net] has joined #launchpad
=== lamont [n=lamont@mib.fc.hp.com] has joined #launchpad
=== fabbione [i=fabbione@gordian.fabbione.net] has joined #launchpad
=== lifeless [n=robertc@dsl-43.1.240.220.rns01-kent-syd.dsl.comindico.com.au] has joined #launchpad
=== zorglub [n=zorglub@194.146.226.84] has joined #launchpad
ddaaSteveA: I guess your review is requested to confirm that the level of build documentation (good) and cross-referencing (none so far) is satisfying.05:12
stubddaa: That constraint is causing branch-pages.txt to fail (possibly others too)05:13
ddaaduh?05:13
ddaa???05:13
ddaano way _adding_ this constrain would cause anything to fail05:13
ddaaexcept some forms with insufficient validation05:14
stubhttps://chinstrap.ubuntu.com/~dsilvers/paste/fileYAYBeQ.html for the constraint that needs to land in rocketfuel....05:14
ddaastub: yes05:15
stubhttps://chinstrap.ubuntu.com/~dsilvers/paste/fileJRit8h.html for the failure05:15
ddaaholy fucking christ :(05:16
ddaacascading breakage05:16
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
ddaaI'll be working on fixing the tests and stuff, right now05:16
stubddaa: ok. It would be great if you can land that db patch as patch-40-49-1.sql05:17
=== Martolod [n=jeremy@ARennes-257-1-104-50.w81-48.abo.wanadoo.fr] has joined #launchpad
stubddaa: The person merge tests are also failing - we need to decide how to handle duplicates caused by a merge.05:21
ddaa*argl*05:21
stubI don't think we can delete them, so we might just have to leave them in the db as noise05:22
ddaastub: we should append "-N" to the name, where N is an integer.05:22
ddaai.e. name-1, name-2, etc.05:22
stubAhh... yes. that is a good idea.05:22
stubI'll sort the people merge code if you like.05:23
ddaathere might be better ways to do it, but I people merge is rare corner case, so that will be good enough05:23
ddaastub: I'd like if you could05:23
ddaasend me a paste, when you have05:24
mdkeis spiv on vacation, does anyone know?05:24
ddaamdke: he is05:24
Kinnisonstub: Would there be anything wrong with coalescing libraryfilealias records?05:24
mdkeddaa, thanks. When is he back?05:24
ddaamdke: back at the end of the week05:24
mdkegreat05:24
mdkegood telepathy too05:25
Kinnisonstub: I.E. if two lfa records have the same name, content, mimetype and expiry, reduce it to one lfa by updating FKs everywhere else?05:25
carloskiko: pong05:27
kikocarlos, read my mind, I was about to reping05:27
kikoso 05:27
kikoyou have more database corruption. :)05:27
carlosnot really read your mind, I just came back from lunch :-D05:27
carlosI have more database corruption?05:27
kikocarlos, yes. when and why did you choose to tal:replace="structure  inside the pomsgset textarea?05:28
carlosI don't remember05:28
stubKinnison: If you do that, we lose the ability to determine the last access time of the individual aliases (as there is now only one)05:28
stubKinnison: Apart from that, no problem at all05:28
carloscan you point me to the code so I can try to get the context?05:28
=== Kinnison tries to think of a case where the last access time on the individual aliases would be important
kikocarlos, look at the pomsgset translation template.05:29
carlosI'm there05:31
carlos<tal:content replace="structure python:pomsgset_view.getTranslation(index)" />05:31
cprovKinnison: last_access is only important for gc, if we set it for now it will just prevent them to be removed for a while. Does it sound right ?05:31
carlosare we talking about that?05:31
Kinnisoncprov: I was just imagining chosing the most recent last_access05:32
kikocarlos, yes.05:32
stubKinnison: Removing LibraryFileAliases will cause URLs that used to work to stop working, so we don't want to add this to the Library garbage collector. Just special cased in areas we feel are worth the extra complexity.05:32
kikocarlos, when and why did that structure get added there.05:32
Kinnisonstub: right05:32
Kinnisonstub: that's a good point, removing an LFA renders a URL nonfunctional05:32
KinnisonThis might be a one-off for sourcepackages so I'll look into doing a one-off script instead05:33
stubAre you creating a large number of dupes or something?05:33
stubThat will never be garbage collected?05:33
carloskiko: last time i touched that code was with my branch to split out the pomsgset specific template from the pofile-translate template05:34
carloskiko: I don't know if I added it then05:34
carloskiko: about why, I don't know05:35
carloslet me check if it's on that branch...05:35
kikocarlos, okay. it is wrong.05:35
carloskiko: don't we have a bzr blame yet?05:35
carlosit's really useful to know the revision when a code line was touched....05:36
kikocarlos, we have bzr annotate.05:36
carloslifeless, mpool:  ^^^05:36
kikocarlos, but I have little time, so let's go over why it is wrong.05:36
Kinnisonstub: at some point, a large number of dupes was created05:36
Kinnisonstub: I'm trying to work out if it's still going on05:36
kikocarlos, what happens when an end-user types into a translation textarea the following literal text: "Foo <em>bar</em> baz"05:36
Kinnisonstub: so I need to clear the dupes05:36
carloskiko: let me read some extra code first, please...05:36
kikocarlos, what do we save to the database?05:36
kikocarlos, there is no extra code to read -- the error is there. :)05:37
stubHow large though? Is it significant in a table designed to hold tens of millions of records?05:37
kikocarlos, I have 5m05:37
Kinnisonstub: It's significant in a tool sense05:37
carlosok05:37
Kinnisonstub: tools are crashing because of finding more than one lfa for a given filename in a given distro pool05:37
carloskiko: I guess the form submission should escape the '<' and '>' chars05:37
carlosso we get &lt; and &gt;05:38
kikocarlos, the browser does that for you. we correctly store in the database "<" and ">".05:38
kikocarlos, the problem is when we /render/ the translation in the page.05:38
stubKinnison: Ok. So we can either merge the dupes, or fix the tools to choose an arbitrary file in that case.05:38
kikowhat happens is you end up with a <textarea> that can contain HTML elements.05:38
kikocarlos, have you ever tried, for instance, to type in </textarea> into a translation?05:38
carlosright05:39
carloskiko: no, I havent05:39
kikocarlos, test it in a local tree to see what happens. 05:39
kikocarlos, now, this is very unfortunate05:39
carloskiko: I know that the 'structure' thing should not be there05:39
kikoright05:39
Kinnisonstub: I want to merge the dupes because it's good for the tools to rant in case they're not dupes but a bug which has allowed different contents into the pool05:39
kikobecause it has caused some weird side-effects that need to be dealt with05:39
carlosbecause the html tags are preserved that is what you are pointing at05:39
kikoright05:39
kikocarlos, we have translations now that include portions of the translation template05:39
Kinnisonstub: we used to have a bug which has allowed three broken orig.tar.gzs to happen05:39
Kinnisonstub: Celso and I corrected that bug, but now we need to cleanup so we have an easier time of it debugging any future issue05:40
kikocarlos, we will need to look for translations that contain the text </textarea> and fix them.05:40
stubKinnison: ok. If you don't remove the existing LFAs, but just change the stuff that points to them to point to the canonical entry, then existing URLs won't break.05:40
kikocarlos, hopefully there are not too many05:40
kikocarlos, but it will require a script to fix them I suspect05:40
carloslet me check the number...05:40
kikocarlos, can you a) do an analysis on staging and tell me how many are broken b) add a test c) fix the bug and d) email me cc:launchpad with the details? :-)05:41
Kinnisonstub: aye, but won't librariangc then remove the unreferenced LFAs ?05:41
carloskiko: we have 5 translations with '</textarea>'05:41
Kinnisonstub: given the only things using the LFA URLs are the internal tools I'm fairly happy if they disappear anyway05:41
kikocarlos, wonderful. 05:41
stubKinnison: And if the number of irrelevant LFA entries becomes a size problem, we can update the librariangc to remove unreferenced library file aliases that have not been accessed in the last 6 months or something.05:41
kikocarlos, if you implement "translation search" we can fix them using rosetta :-)05:42
carlosa) yes, b) yes, c) yes d) yes05:42
kikothanks!05:42
kikoI love jackpots!05:42
carlos:-P05:42
Kinnisonstub: aah cool05:42
carloskiko: so, is this related to the problem matsubara is fixing?05:42
kikocarlos, this is the exact same problem, but I have assigned matsubara to something else -- I think this is something you're better off testing and fixing.05:43
carloskiko: I'm not asking for matsubara to fix it, I'm just asking to know if I need to deal also with the other bug ;-)05:44
ddaastub: okay for me to send https://chinstrap.ubuntu.com/~dsilvers/paste/filezQs3xR.html with r=stub?05:44
kikocarlos, yeah, and no, it's just one bug. 05:44
carloskiko: ok, seems like this came from my branch, or at least it's there too05:44
kikocarlos, yeah, tough luck, your bug, you fix it :)05:45
carlos:-P05:45
ddaa(might need extra adjustment, full test suite running)05:45
stubddaa: Sure, but it won't land without the people merge code being updated too05:45
ddaaha, I did not realise that this was actually breaking on people merge...05:46
carloskiko: so do you want a migration script or just leave them and fix with the search feature when it's implemented?05:46
stubddaa: Do you need it landed now, or do you want to pass the branch over to me to land?05:46
kikocarlos, a) how long is the search feature going to take to get done b) can you at least contact the owners of the pofiles and tell them what is wrong so they know and can fix it?05:46
stubddaa: The people merge code is paranoid - it analyzes the database schema and fails when it detects a structure we are not explicitly handling that it can't deal with itself.05:47
ddaathe sooner the better, I do not know what people merging look like, but I'm willing to give it a shot. Not sure I will have the time tonight though, as I'm on a collision course with my SO.05:47
stubddaa: ok. I'll apply your patch and land a fix tomorrow.05:48
carloskiko: a) I don't have any idea as the spec is not even approved b) Will do as soon as the 'structure' tag is removed and they can actually fix it05:48
=== ddaa hugs stub
kikocarlos, agreed. 05:48
kikocarlos, I'd love the spec, at least a small part of it, to be implemented soonish.. but get your dapper shoes polished first. 05:48
carlosdapper thing is almost done05:49
kikowonderful05:49
carlosat least the critical part, I need to ask some extra removals and I think we can deal with whatever is missing from time to time05:49
carloskiko: there are 'only' 2000 entries on the queue from the 29000 we had last week05:50
kikocarlos, wonderful -- does this mean that the translation statistics for today are going to look great?05:51
carloskiko: they should, but they are not looking great....05:51
carloskiko: the export log says that we have twice the number of pofiles we had last week, but pitti's email says we have no changes...05:52
carlossomething went wrong here...05:52
=== bradb & # dner
SteveAbradb: !05:53
carloshmm, and pitti is not around so I cannot check it with him now...05:53
carloskiko: if you look at http://mawson.ubuntu.com/~carlos/rosetta-dapper-2006-04-25.log vs http://mawson.ubuntu.com/~carlos/rosetta-dapper-2006-04-21.log05:53
carloskiko: you can see the difference05:53
carloswe had a problem with the exports from 22 until today, a lock was breaking the exports, but we fixed it today05:54
kikocarlos, perhaps that's the problem -- the script ran too early?05:54
kikocarlos, that looks awesome!05:55
carloskiko: not sure, I need to check with martin 05:55
kikook05:55
carloskiko: yeah, and most of those changes are just KDE being imported... I think KDE is 2/3 of dapper's .po files05:56
kikowow05:56
carlosyeah they split them a lot05:57
carlosok, let's fix some bugs ;-)05:57
carloskiko, matsubara: thanks for debugging the problem05:58
kikocarlos, no worries, enjoy 05:58
kikoand matsubara was right after all05:58
kikoit /was/ browser dependent05:58
=== kiko dons paper bag
kiko-fudcarlos, make sure your test catches the fact that "<" and ">" are /not/ present inside the textarea content05:59
=== beyond is now known as beyond-rango
kiko-fudcarlos, it's tricky making pagetests that ensure that something is not in the page06:00
carloskiko-fud: well, I don't think is so tricky, just test for &lt; and &gt; if they are not there, '<' and '>' would be there, right?06:01
kiko-fudcarlos, well, sorta -- what if the sampledata has a literal &lt; in it?06:01
stubThere are funky new tools in Z3.2 for writing page tests and page test like things easier - just needs someone to go through and work out what is useful to us and throw together some examples.06:02
kiko-fudmaybe you should just test that the view's getTranslation method returns a string with no "<" or ">" in it, and fiddle with sampledata adding a string that contains it.06:02
carlosit will appear as &lt; but never as '<', right?06:02
SteveAstub: bjorn and i were talking about that over lunch06:02
SteveAstub: i asked bjorn to try using the new stuff for his current work06:02
SteveAsee how it goes06:02
kiko-fudtime for fud06:02
SteveAand then we can write new docs depending on how that goes06:02
carloskiko-fud: yeah, the idea is to submit and fetch texts that has a '<' char06:03
carlosthe form should not have it ever06:03
Kinnisonstub: when did librariangc last run?06:04
carlosSteveA, stub: I guess it would mean that we will need to rewrite our pagetests, right?06:04
SteveAyes, over time06:04
carloskiko-fud: enjoy it06:04
SteveAwe'll just rewrite the ones we touch, to start with06:04
carlosok06:04
=== ddaa starts hacking a doctest for people merging
stubIf I'm right, it will be much easier rewriting a failing pagetest than fixing a broken one - I find fixing them is really fiddly and time consuming06:05
SteveAyeah06:05
mdkeI think " status: fix released" should work when in a bug mail, what do you guys think?06:08
Kinnisonstub: I'm concerned that I'm finding duplicated LFC records06:11
stubKinnison: Hmm... looks like the cron job has gone missing06:12
kiko-fudmdke, sounds reasonable. doesn't it?06:12
Kinnisonstub: it's been missing a long time then06:12
Kinnisonstub: any chance of it being started pretty soon?06:13
Kinnisonstub: then I'll hold off on my stuff until the LFCs are coalesced06:13
stubSure.06:13
Kinnisonthanks06:13
Kinnisonit'll be easier to just use lfa.content rather than having to drill into lfa.content.sha1 and lfa.content.filesize06:13
Kinnisonprobably quicker too :-)06:14
mdkekiko-fud, well, I tried closing two bugs that way. It would be useful for those people who made a vague attempt at reading the documentation, but didn't really do so thoroughly06:15
BjornTmdke: i agree, it should work. could you please file a bug about it?06:20
mdkeBjornT, you bet06:20
mdkeBjornT, bug 4132906:23
UbugtuMalone bug 41329 in malone "allow spaces in status name when changing status by email" [Minor,Unconfirmed]  http://launchpad.net/bugs/4132906:23
BjornTthanks06:24
ddaastub: here is a doctest for branch handling when merging people06:26
ddaahttps://chinstrap.ubuntu.com/~dsilvers/paste/fileraBhNY.html06:26
ddaaI have to run now, I might not have the time to do any more today. Anyway, you can do the sql stuff better and faster than me even in your sleep.06:27
ddaastub: and one w/o broken line breaking: https://chinstrap.ubuntu.com/~dsilvers/paste/fileJ6Ax6x.html06:29
=== ddaa runs
=== sivang_ [i=sivan@muse.19inch.net] has joined #launchpad
=== sivang_ is now known as sivang
=== saik0 [n=saik0@adsl-065-007-195-142.sip.mia.bellsouth.net] has joined #launchpad
saik0Launchpad Ubuntu CoC signing problem https://launchpad.net/distros/ubuntu/+bug/4133406:42
UbugtuMalone bug 41334 in Ubuntu "Cannot sign Ubuntu Code of Conduct 1.0.1" [Normal,Unconfirmed]  06:42
=== saik0 pets Ubugtu
sfllawsaik0: I can confirm it.06:45
sfllawI tried to sign it yesterday.06:45
sfllawNo dice.06:45
sfllaw1.0 works, though.06:46
carloskiko-fud: pitti's script was run 7 minutes before the language pack generation finished06:46
carloskiko-fud: he's running it again now. and we moved it 30 minutes later06:46
carlosto prevent this to happen again06:46
saik0sfllaw, same. I can sign 1.0 fine06:46
salgadosaik0, sfllaw, that's a bug in launchpad actually06:48
salgadostub, are we running head on production?06:50
saik0salgado, can you file launchpad bugs in launchpad?06:50
salgadoyeah, sure. https://launchpad.net/products/launchpad06:51
=== beyond-rango is now known as beyond
saik0salgado, thanks. wow, why could'nt I find that06:53
salgadosaik0, probably because it's (unintentionally) well hidden. :/06:56
=== carlos -> out
carlossee you later tonight07:06
stubsalgado: yes07:08
salgadocprov, are you using mawson's db?07:18
salgadostub, are you going to be around during the next 15min or so?07:18
cprovsalgado: yes, it has edgy oppened07:18
salgadocprov, and you're running a launchpad instance there, I guess?07:18
cprovsalgado: yep07:19
salgadocprov, which codeline?07:19
cprovsalgado: it's not mandatory, though07:19
cprovsalgado: current07:19
cprovsalgado: it has lp-upstream (lastest or so)07:20
cprovsalgado: if you need update, I don't mind07:20
salgadoyeah, I saw that... I'm going to run the mirror-prober script from there. don't think it can cause you any harm07:21
cprovsalgado: it must not ;), no problem 07:23
mdzbradb: is it possible to have the "subscribe me" checkbox closer to the "add comment to this bug" rather than having to mouse down below the comment box to uncheck it?07:24
Seveasmdz, heh, I was about to complain about that one too 07:24
mdzbradb: those of us who trawl through a lot of bugs without wanting to be subscribed would appreciate it07:25
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
bradbmdz: I'm confused. You have to scroll down to hit the button to submit the comment, so how is it more effort to uncheck that box when it's near the submit button? (not that I'm saying the UI's great, just curious.)07:34
mdzbradb: because I click it before I type in the comment, so that I don't forget07:35
mdzso my clicks go: add a comment, uncheck subscribe me, focus the comment box, then submit07:36
bradbah, so you're saying that where it currently is, it's easy to forget to uncheck it before submitting07:36
mdzmaybe; personally I see it immediately, notice that I want to change it, and do it right away07:37
bradbah, ok, i know the feeling07:39
bradbmpt was going to look at that UI tomorrow to smooth it out, so I'll send him an email noting what you guys said07:39
mdzthanks07:39
mdzI like the checkbox, by the way; too often I've had to subscribe someone who commented because i need to ask them a question07:40
bradbyeah, should make life a little easier, i hope07:41
SeveasI even didn't see the checkbox, I use <enter> to send the comment 07:45
Seveasand the checkbox wasn't visible 07:45
Seveaslp broken? I get timeouts...07:48
Seveasstarted ~1 minute ago07:49
Seveasah, works again07:51
jordicarlos: ping08:06
jordihttps://launchpad.net/people/uzbek/+review was never added to ubuntu translators08:07
jordierr08:07
jordihttps://launchpad.net/people/ubuntu-l10n-uz08:07
sitsah fudge08:31
sitsI didn't mean to start an argument on the devel mailing list with my bug voting comment08:31
mdkesits, it's not an argument, don't worry.08:32
sitsI dunno. One of the replies was kind of long08:33
mdkethat's not a bad thing. We should move channel if you want to discuss more though08:33
sitsyou know you've failed to express yourself properly when someone starts talking about elitist developers08:34
sitsmdke: oh right08:34
sitswhich channel?08:34
mdkesits, #ubuntu-devel?08:34
sitsok08:35
SteveAkiko: ping08:42
kikoSteveA, sorry, see privmsg need 5 more minutes :-/08:46
bradbsalgado: Any chance of a followup to the bug dates review today?08:51
cprovbradb: how do you build an rfc2047 compilant recipient for sending email from malone ? is there any specific code ?09:02
kikoor BjornT 09:03
kikocarlos, I saw we generated another report.. but it still says we miss some 300 templates09:03
bradbcprov: No idea. Maybe BjornT would know.09:04
BjornTcprov: do you mean generating: 'Name <email>', where Name can contain non-ASCII characters?09:07
kikoyes09:07
BjornTok, then use canonical.launchpad.mail.format_address09:07
kikowhy don't you use it for the recipient name in Malone mail?09:08
kikoBjornT, also, I wonder whether that code is actually in accord to rfc-2047.09:08
BjornTdon't know, should we? at the moment we use only the email address, i think that's enough09:09
BjornTkiko: what makes you think it doesn't work as specified in rfc 2407?09:09
=== bradb reads up on zope.testbrowser.browser
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
kikoBjornT, I think we should, but I'm not picky.09:12
SteveArfc 2047 ?09:13
SteveAoh, 240709:13
kikoBjornT, because I think rfc2047 is a bit more complicated than that -- at least the code we have from katie does it differently.09:13
SteveArcf 2407 ?09:13
kikoare you guys crazy09:13
kiko2 0 4 709:13
kiko2407 is something else09:13
BjornTyes, 204709:13
SteveAhurra09:14
SteveAnothing to do with triple DES encoding of IP domain thinggie09:14
BjornTkiko: where can i see the katie code? i'd like to see what it does.09:14
kikocprov, can you email the code to BjornT?09:14
cprovBjornT: apparently, there are fancy things related with names containing '.' and ',' in the rfc. Nonetheless, I; m going to use your format_address09:14
cprovBjornT: look at lib/canonical/archivepublisher/utils.py on fix_maintainer function09:16
kikoanyway09:17
BjornTbradb: if you're going to use testbrowser you can get r3499 from bjorn/launchpad/bugwatches-rework, it modifies test.py so that you can use testbrowser. you also need to install python-mechanize.09:18
SteveABjornT: we should make that a formal dependency09:18
kiko-afkbradb, what BjornT is saying is "wait a week"09:18
bradbhmph09:18
BjornTSteveA: yes, i was going to do that tomorrow. what's the procedure for adding a dependency to launchpad?09:20
SteveAi don't know anymore09:20
SteveAideally, get the dependency added to the launchpad-dependency packages09:20
SteveAjeff b. would do that09:20
SteveAstu would know better09:20
kiko-afkpretty much09:20
BjornTyeah, i'll ask stub tomorrow.09:22
SteveAok09:22
stubhttp://stuartbishop.net/Software/EmailAddress/index.html handles Unicode email addresses, including Unicode domain names.09:23
stubI have no idea about launchpad-dependancies. I submit them into Launchpad and waited until Lifeless updated things last time.09:23
BjornTkiko-afk, cprov: format_address produces an rfc 2047 compliant string.09:23
cprovBjornT: okay, will use it 09:24
SteveABjornT: please email launchpad dev list, and cc jeff b. about adding that dependency.09:24
bradbSteveA: So, re: the patch for bug 36866, it's not really feasible to write a test that /really/ tests what the bug report is about until we can use zope.testbrowser.Browser.09:25
UbugtuMalone bug 36866 in malone "Searching for bugs after selecting a certain status from the Right-Hand-Menu resets the search" [Major,In progress]  http://launchpad.net/bugs/3686609:25
SteveAbradb: that's not going to be long.  i think you can wait, and meanwhile, you can get it working on your own branch, ready to merge09:25
bradbok09:26
bradbI was planning to convert all of bugtask-search-pages.txt to use z.tb.B anyway. Should make it much more readable.09:26
bradbBjornT: So, even still, I'll need to merge from your branch first?09:27
cprovBjornT: are the doupble quotes expected in __"Zak B. Elep" <zakame@ubuntu.com>__ ?09:27
BjornTcprov: yes, since it contains '.'09:27
kiko-afkaha09:27
cprovBjornT: when sending emails, the To: headers remains only the email 09:28
kiko-afkBjornT, did you look at the code in fix_maintainer?09:28
cprovBjornT: uhmmm09:28
BjornTkiko-afk: i looked at it briefly. 09:31
kiko-afkit does weird things09:32
bradbBjornT: Can you give me just the diff for test.py so I don't have to merge the whole branch?09:33
=== welshbyte [n=welshbyt@cpc3-cwma2-0-0-cust276.swan.cable.ntl.com] has joined #launchpad
BjornTkiko-afk: yeah, it looks kind of broken, but i'm not sure what format the input is in.09:37
BjornTbradb: well, you don't have to merge the whole branch, just the revision i gave you. but hold on, i'll give you the diff.09:37
cprovBjornT: DISPLAYNAME <EMAILADDRESS>, as docummented in tests/test_utils.py09:38
cprovBjornT: to have that much obsolete code sounds really depressing, it might have something special from debian09:39
ddaagrah... I'm so bad at SQL :(09:40
BjornTcprov: what about if DISPLAYNAME contains characters that need to be quoted?09:40
ddaaanybody feels like teaching me, or should I just wait for stub to do the hard work tomorrow?09:41
cprovBjornT: it does weird things like returning '<EMAIL> (NAME)'09:41
=== lucasvo [n=lucasvo@wservices.ch] has joined #launchpad
lucasvohow can I register a project in launchpad? does it cost anything?09:42
lucasvois it possible to have commercial projects in launchpad?09:42
ddaaprofit!09:42
ddaalucasvo: you just need a launchpad account09:42
BjornTcprov: only if DISPLAYNAME contains '.' or ','. there are a lot of other characters than need quoting.09:42
ddaayou can register anything, but the system is mostly designed to be useful to free software projects09:43
ddaabut any openly developped project can benefit09:43
lucasvook09:43
ddaathe sabdfl has some plan for "private" projects and distros, mostly for things that are planned to be released eventually, but it's not there yet (and it's a difficult problem)09:44
cprovBjornT: but this format is compilant with rfc2047 ? I mean the '<email> (DISPLAYNAME)'09:44
ddaathough I'm sure you will have his ear if you have some money to put in it :)09:44
lucasvoddaa: ok, thanks09:46
BjornTcprov: AFAIK, yes. but what would be interesting to know is the restrictions of the Maintainer and Changed-By fields are. what characters can DISPLAYNAME contain, and are they properly quoted.09:51
BjornTbradb: https://chinstrap.ubuntu.com/~dsilvers/paste/file41BJqa.html09:52
bradbBjornT: thanks09:53
bradbBjornT: Hm, it seems that our version of python2.4-clientform doesn't work with z.tb.B10:04
bradbAssertionError: ClientForm >= 0.2.1a is required10:05
kiko-afkbradb, is that dapper?10:05
bradbvery dapper10:06
BjornTbradb: really? what did you do to get that error?10:06
bradbBjornT: ran a simple test that uses z.tb.B10:07
BjornTbradb: can you paste the test somewhere? i did some simple tests and it worked.10:07
cprovBjornT: so, using the default format_address seems to be a fair solution, I miss an API based EmailAddress content class10:08
bradbBjornT: Just:10:10
bradb    >>> from zope.testbrowser import Browser10:10
bradb    >>> browser = Browser()10:10
bradbis enough10:10
bradbAre your packages right up to date?10:11
BjornTbradb: hmm, that works for me. what's the traceback?10:11
bradbI just updated as of 10-15 mins ago10:11
BjornTbradb: yes, i updated today, not long ago. i'll update again now.10:11
bradbBjornT: https://chinstrap.ubuntu.com/~dsilvers/paste/file7gbgJ3.html10:13
=== JanC [n=janc@lugwv/member/JanC] has joined #launchpad
kikosalgado, ping?10:17
salgadokiko, pong10:19
BjornThmm, i have the 0.2.0.99-1 package installed, but ClientForm.VERSION returns '0.2.1b'10:19
kikosalgado, privmsg10:19
kikoBjornT, no /usr/local versions installed?10:19
bradbBjornT: 10:19
bradb>>> ClientForm.VERSION10:19
bradb'0.1.17'10:19
BjornTkiko, bradb: i just uninstalled the package, resulting in that i couldn't import ClientForm. i installed it again, and it still shows 0.2.110:21
BjornTbradb: 0.1.7 seems wrong, though. do you have any old versions installed?10:21
bradber, yeah, looking at the pkg version, i must have an old one floating around10:22
bradban old CF, that is10:22
bradb__file__ tells all10:24
kikostop waving red herrings around10:24
kikoBjornT, remind me -- do we now notify by email when the remote bugwatch's status changes?10:40
BjornTkiko: no, not yet. it's next on my list after the current bug watches work.10:44
kikoBjornT, thanks. and I had another consideration.10:46
kikoBjornT, you know the plan to do away with priority?10:46
BjornTyeah10:46
kikowell10:46
kikoI was thinking. what will we do about other bug trackers that implement it?10:46
kikoI mean, that information doesn't really map well 1-to-1 with our Importance.10:47
=== lucasvo [n=lucasvo@wservices.ch] has left #launchpad []
kikoand any conversion we do will just end up being confusing, won't it?10:47
BjornTi'd imaging we map it into importance. other bug tracker's statuses don't map 1-to-1 to our statuses, we still map them.10:47
=== tortho [n=tortho@6.84-49-112.nextgentel.com] has left #launchpad []
=== LeeJunFan [n=junfan@s64-186-37-120.skycon.net] has joined #launchpad
salgadokiko, https://dogfood.ubuntu.com/distros/ubuntu/+mirror/brazilllll11:01
salgadono time outs during the last probe11:01
lifelessmoin moin11:05
kikosalgado, you are DE MAN11:06
kikosalgado, however, I should tell you one thing11:06
kikothat list is VERY long and not really sorted or sortable11:07
kikoso I'm asking myself if the tabular format is actually appropriate11:07
kikoor if we should, instead, have multiple tables11:08
salgadoor if we should do that only for the in-developement distrorelease and the latest stable one?11:09
salgadoand have some more concise for the other ones?11:09
kikoright11:09
kikolet's ask the people who matter!11:09
salgadoanyway, I've already asked these questions myself11:09
kikooh!11:10
salgadoI first wanted to check with mpt if he has any suggestions. before start stripping things off 11:10
kikookay, cool11:10
kikosalgado, how long timeout are you using there?11:11
salgadoyou don't wanna know!11:11
salgadowith 40 seconds I got lots of timeouts. then I changed it to 90 seconds and didn't get any11:11
=== bradb loves this "ItemNotFoundError: insufficient items with name 'summary'" error. I AM TEH ROBOTO.
kikosalgado, I guess that's.. okay11:15
kikosalgado, the breadcrumbs on that page.. seem wrong.11:19
salgadowhy? It's the same as in other pages under /distros/ubuntu/, isn't it?11:20
kikosalgado, colour-coding the status would help as well, perhaps11:20
=== lucasvo [n=lucasvo@wservices.ch] has joined #launchpad
kikosalgado, what happened to the content on that page?!11:24
kikoI was showing it off to mdz!11:24
salgadoI did another probe11:25
mdzhehe11:25
mdzdemo effect11:25
salgadoand this one failed11:25
kikowith 90 seconds, no less?11:25
kikoyou sandbagged my demo. snif11:25
lucasvois it possible to change the email address in launchpad?11:25
salgadoit takes less than one minute to probe that mirror11:25
salgadokiko, the content should be back quickly11:26
kikookay11:26
salgadoyes, it's back now11:26
=== sits [n=sits@cpc3-cwma2-0-0-cust739.swan.cable.ntl.com] has left #launchpad []
salgadoyou need to tell me when you're demoing something that I'm testing.11:27
kikoyeah yeah11:27
salgadolucasvo, yes, it is. on your personal page you should see an 'Email Addresses' link11:28
salgadolucasvo, on the top left corner11:28
lucasvoYour preferred contact address for all Launchpad e-mail is:11:28
lucasvoI want to change this11:28
salgadolucasvo, in this case you need to add a new one and confirm it11:29
lucasvoic, ok11:30
lucasvosalgado: thanks11:30
welshbytelucasvo: 1) add new email address and confirm 2) set new email address as contact address 3) remove original email address11:30
=== bradb heads off, later all
=== bradb [n=bradb@modemcable092.66-130-66.mc.videotron.ca] has left #launchpad []
lucasvohm, strange, I can't sign the COC11:38
lucasvoit reports a str error11:38
lucasvohttp://pastebin.com/68185311:38
kikohttps://launchpad.net/products/launchpad/+bug/3954711:39
UbugtuMalone bug 39547 in launchpad "Code of Conduct 1.0.1 signatures not accepted" [Normal,Confirmed]  11:39
kikolucasvo?11:39
lucasvoyeah11:40
lucasvokiko: thank11:40
jordigood night lp!11:41
=== viajero [n=viajero@wikipedia/Viajero] has joined #launchpad
viajerohi, anyone around?11:51
=== ruffneck_ is now known as ruffneck
welshbytehm i've just realised that i have a duplicate launchpad account that i registered a while before the one i now use and must have forgotten about11:55
lucasvowelshbyte: just delete it :)12:00
welshbytelucasvo: i might do, if i could remember the email and password i used to log into it :)12:01
lucasvowelshbyte: you can try to search there: https://launchpad.net/people12:02

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