/srv/irclogs.ubuntu.com/2005/07/27/#launchpad.txt

bradb-afki'm still here (though i'm not supposed to be)12:06
dilysMerge to rocketfuel@canonical.com/sqlobject--test--0.6: [trivial]  Remove SteveA's turd (patch-32: stuart.bishop@canonical.com)12:06
bradb-afkSteveA: i'd be surprised if it caused that warning, because it's not a slice.12:07
SteveAit still counts12:07
SteveAplease check it out12:07
bradb-afkand, futhermore, because i think it's ordered12:07
bradb-afk(there should be a default ordering imposed, IIRC) checking now12:08
bradb-afkSteveA: i visited all the +bugs URLs and didn't see the warning. does that mean it's good to go?12:09
bradb-afker, one sec12:09
=== bradb-afk checks one other thing
bradb-afkchecked all the task pages too, no warnings.12:10
bradb-afkSteveA: good to go then?12:10
=== Keybuk [~scott@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
bradb-afkSteveA: merge request sent12:14
bradb-afkwhat's the magic incarnation to see pqm's queue again? it's gone from my shell history.12:16
elmohttp://pqm.ubuntu.com/12:16
Keybukelusive by its obviousness12:16
bradb-afkah, didn't know it got moved to a public url12:17
=== __keybuk [~scott@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
bradb-afkright, really afk now12:19
Keybukbradb-afk: before you go ...12:21
bradb-afk<out_of_office_reply>i've left!</out_of_office_reply>12:22
Keybukuh-huh12:22
Keybukconvincing12:22
SteveAhttp://sourceforge.net/tracker/index.php?func=detail&aid=1242657&group_id=5470&atid=10547012:24
SteveAhttp://python.org/sf/124265712:25
SteveA(shortcut to the same thing)12:25
bradb-afkbrutal12:26
bradb-afkok, off to mcgill now, mega seriously, mega late12:26
Keybukbradb-afk: just one thing ...12:31
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=SteveA]  remove IBugTaskSubset (patch-2149: brad.bollenbach@canonical.com)12:42
=== sabdfl [~mark@sabdfl.silver.supporter.pdpc] has joined #launchpad
sabdflhey bradb, *great* to see those IBugTaskSubsets disappearing!12:46
=== sabdfl [~mark@sabdfl.silver.supporter.pdpc] has left #launchpad []
=== asgeirf [~asgeirf@nat-pool-brisbane.redhat.com] has joined #launchpad
=== otep [~otep@AP-203.167.31.177.sysads.com] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  portlet and page fixes for distroarchrelease (patch-2150: mark.shuttleworth@canonical.com)02:37
=== XoloX [~Peter@cc220079-a.assen1.dr.home.nl] has joined #launchpad
XoloXHi all. This may be the wrong place to ask, but I'm trying anyway :P. Why is Ubuntu using Bugzilla aswell as Launchpad?05:47
BurgundaviaXoloX, ubuntu is switching over as launchpad is a new product05:47
XoloXAha05:47
XoloXOk05:47
XoloXThanks05:47
Burgundaviabugzilla was setup for a bug solution now05:47
Burgundavialaunchpad is for a next generation solution, that includes bug handling and other stuff05:48
XoloXI wanted to file a bug against Galeon from Bugzilla, chose the category Universe, and was redirected to Launchpad where I had to sign up again. I can understand that it's to much trouble to merge the accounts or something like that, but might it be a good idea to provide a small paragraph with an explanation of the situation (Launchpad replacing Bugzilla)?05:49
XoloXOr maybe I overlooked some FAQ :). Still, it was confusing to me.05:50
Burgundaviayour launchpad login already works on the wiki05:50
Burgundaviaonce bugzilla dies, there will only be one ubuntu identify system05:50
XoloXOK. Thanks for the info!05:51
XoloXEhm, one more thing, probably out of context here, but I can atleast try: If I have a problem with Yelp depending on mozilla-firefox, I should file it under Yelp right?05:52
Burgundaviayes05:52
Burgundaviaand yelp is supported, so that goes in bugzilla05:52
XoloXHehe, see what I meant with confusion :P05:53
Burgundaviabugzilla is for anything is main05:53
XoloXOK05:53
XoloXWell, thanks again, cheers05:53
Burgundavianp05:53
=== XoloX [~Peter@cc220079-a.assen1.dr.home.nl] has left #launchpad ["Leaving"]
=== _otep [~otep@AP-203.167.31.177.sysads.com] has joined #launchpad
dafspiv: around?11:47
spivdaf: yeah11:47
dafYou should not import connect from canonical.database.sqlbase:11:48
daf    canonical.lp.sql11:48
dafis this an incomplete __all__ or a bad import?11:49
spivIncomplete __all__ I think.11:51
dafthanks11:53
Kinnison/srv/launchpad.ubuntu.com/dogfood/launchpad/lib/canonical/archivepublisher/domination.py:135: UserWarning: Getting a slice of an unordered set is unpredictable.12:30
Kinnisonis there an "anyof" operator for the selectresults class?12:31
KinnisonI just want one of the results, any will do12:31
spivKinnison: Then order by id?12:32
KinnisonI don't really want to impose any ordering on the select because that'll increase db overhead12:34
Kinnisonbut I guess that's the least scary option12:35
=== Kinnison watches the process slowly chew 1.6G of ram
=== terrex [~terrex@84-122-73-155.onocable.ono.com] has joined #launchpad
spivKinnison: FWIW, iter(results).next() will workaround the warning.  But don't expect reviewers to treat that trick kindly :)01:02
Kinnisonspiv: *sigh*01:15
Kinnisonspiv: having a selectresults.anyof() which effectively did [0]  supressing the warning would be nice01:16
=== kiko [~kiko@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
spivKinnison: What's the use case?  So far no-one has had a good reason to want a random row.01:20
=== mpt [~mpt@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== Kinnison has a selectresults which contains a bunch of packagepublishing records
=== Kinnison wants a random row so he can cross through to the architecture of the package
=== carlos [~carlos@243.Red-83-47-24.pooles.rima-tde.net] has joined #launchpad
spivHow did you get the packagepublishing records?01:32
carloshi01:32
spivI'm wondering why you can't directly select an architecture.01:33
Kinnisonspiv: they're passed in as a list of records to be sorted ready for domination01:36
=== Kinnison shall alter the api to pass the arch tag down with it
Kinnisonit'll be easier01:36
=== jamesh [~james@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
dilysNew Malone bug 1549 filed on The Launchpad by Matthias Urlichs: System error clicking on "View Changelog"01:44
dilyshttps://launchpad.ubuntu.com/malone/bugs/154901:44
=== cprov [~cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== stub [~stub@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
jameshinteresting post from one of the Gnome developers working for Novell: http://primates.ximian.com/~federico/news-2005-07.html#2102:03
jameshsounds like he could really use some of the version control stuff we're working towards02:03
=== carlos [~carlos@243.Red-83-47-24.pooles.rima-tde.net] has joined #launchpad
dafjamesh: he's spot-on about undocumented interfaces, in my opinion02:06
=== debonzi [~debonzi@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
mptNot to mention Malone02:12
=== salgado [~salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/sqlobject--test--0.6: Force (by using parenthesis) set operations to be applied in the whole SelectResults the method was called. Add tests for this and some other bits of set operations in a separate file. r=spiv (patch-33: guilherme.salgado@canonical.com)02:38
=== BjornT_ [10183@82-135-221-189.ip.takas.lt] has joined #launchpad
=== Keybuk [~scott@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
bradbmorning03:35
bradbNo SteveA?03:36
kikobusy03:47
=== SteveA [~steve@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
bradbSteveA: Hi. Two things: 1. what's new in the world of page titles? 2. you've got mail.03:53
SteveAhi bradb 03:53
SteveAcool03:53
SteveA1. you can kick my ass when you get to brazil03:53
SteveA2. thanks, i'll read the mail03:53
bradbSteveA: 1. ! 2. ok, cool03:54
salgadoSteveA, I want to run sqlobject's tests on chinstrap, but we need py.test (which is not packaged) for that. should I import it into rocketfuel or package it myself?03:59
SteveAis that holger krekel's thing?03:59
salgadoyes, it's04:00
mptkiko: Do you have time to look at mpt@canonical.com/launchpad--footer--0? There's something missing in either launchpad.zcml or configure.zcml, but I don't know which or what04:01
dafmpt: what's the error message?04:01
mptKeyError: 'legal_should_link'          04:02
dafhmm, perhaps you can paste something more complete?04:02
mptI've specified a new feedback <browser:page> in configure.zcml, like the existing "legal" page04:03
dafwhere does the legal_should_link come into it?04:03
mpt<tal:feedback condition="not: view/feedback_should_link"><a href="/feedback">Give&nbsp;feedback</a></tal:feedback>04:03
mpterg, that "not: " shouldn't be there, but anyway04:04
dafI think you can just do:04:04
daf<a tal:condition="view/feedback_should_link">...04:04
mptoh, true enough04:05
mptI don't think that'll fix the underlying problem, though04:05
=== mpt tries
dafthat doesn't explain why the "legal_should_link" message, though?04:05
dafs/?//04:05
mptI've defined legal_should_link in a "GlobalLinks" class in launchpad.py04:05
SteveAsalgado: let's look at this together in person04:05
mptSo what I think is happening is that there's somewhere I should be saying "use the GlobalLinks class" and I haven't04:06
dafmpt: I think you can probably come up with a better name than that :)04:06
salgadoSteveA, sure04:06
mptok, "GlobalNavigationLinks"04:06
dafno, I mean the _should_link stuff04:06
SteveAwhat's this global links thing?04:07
dafyou mean "there should be a link to the feedback page", not "the feedback should link to something"04:07
mptthe links to the "Legalese" and to the (new) feedback page04:07
mptdaf: No, because if you're on that page, it'll be <strong>Give feedback</strong>04:07
mptIt's whether the text should link or not, not whether it should exist at all04:08
dafurg04:08
dafthis seems like a duplication of the menus logic04:08
mpttrue04:08
mptbut a menu just for two links in the footer seems like overkill04:08
dafmmm04:08
dafwhat's the view class in this case?04:09
mptI don't know. :-)04:09
dafaha04:09
SteveAdaf: mpt and i will talk about this in person now04:09
dafsure04:10
mptthanks for your time daf04:10
dafno worries04:10
dafmorgs: yo!04:12
bradbkiko: so, the best thing i can think of right now to replace <ul tal:condition="python: not request.getURL().endswith('+edit')">, is to move that logic into a method on the ViewWithBugTaskContext method. Still less than ideal, but an improvement on what's currently there, IMHO. Maybe SteveA will have a cleaner solution when he's around.04:14
bradbs/ViewWithBugTaskContext method/ViewWithBugTaskContext class/04:15
morgsdaf: hi04:20
dafmorgs: can I pick your brain about milestones?04:25
dafmorgs: I'm working on the traversal code04:25
morgsok04:25
dafso, milestones are accessed by /products/foo/+milestones, yes?04:26
morgsYes04:27
dafhmm, as far as I can tell, the only thing you do with milestones is edit them04:27
dafis that right?04:27
daf(i.e. /products/firefox/+milestones/1.0/+edit works, but /products/firefox/1.0/ doesn't do anything)04:28
dilysNew Malone bug 1550 filed on The Launchpad by Morgan Collett: Login link to localhost on production on system error page04:28
dilyshttps://launchpad.ubuntu.com/malone/bugs/155004:28
morgsproducts/firefix/1.0 would be a product release version 1.004:28
morgss/firefix/firefox/04:28
dafsorry, I meant /products/firefox/+milestones/1.0/04:29
morgsYes, you just edit them, and then they appear in malone04:29
dafgroovy04:29
daflooks like I'm done, then04:29
morgsAs far as I am aware, there is nothing else you would want to do with them...04:29
morgsOk04:29
dafthanks04:30
dafbradb: now then, what sort of a beast is a BugTasksReport?04:30
bradbIt's vile. Too bad people keep hacking it, really. It needs a spec.04:31
dafwell, my objective is to work out how not to import it into traversers.py, since it is in the DB code04:32
=== bradb looks
dafit's not a DB object04:32
dafcould it perhaps be an adaptor?04:33
dafon BugTaskSet, maybe?04:33
dafI'm not clear on what it does04:33
bradbok, here's how, i think04:33
bradbi /think/. you'll have to double check the URL once this is done though04:33
bradbin traverse_bugs, remove:04:34
bradb    if name == 'assigned':04:34
bradb        return BugTasksReport()04:34
bradb    else:04:34
bradband then remove the import04:34
dafthat sounds too simple to be true :)04:35
SteveAdaf: "adapter" please04:35
dafsorry04:35
bradbdaf: it's probably true04:35
bradb(i.e. it'll probably work)04:35
dafwell, if the tests pass...04:35
bradbit's possible that tests will fail04:35
bradbbut that's only because that report exists at two URLs currently, and not one04:36
bradbthe true path is /malone/assigned04:36
dafnoted04:36
bradbany tests that test /malone/bugs/assigned should instead test /malone/assigned04:36
elmouh, someone killed the librarian?04:36
elmoCRITICAL 22-07-2005 15:31:28 0d 0h 10m 4s 3/3 HTTP CRITICAL: HTTP/1.1 500 Internal Server Error 04:37
dafProfessor Troup, in the Library, with the candlestick04:37
elmocan it, SAS man.04:37
Keybukelmo: the librarian has been killing itself for a while now04:45
Keybukspiv did rude things to it04:45
Keybuk(at least on staging)04:45
elmoyeah, this is production tho04:45
=== cprov [~cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
stubOops... production librarian is my faut05:12
stubI bounced the database because I locked up something I shouldn't have05:12
bradbSteveA: Just curious, what's the reasoning behind documenting function keyword args like ":foo: bar" instead of, say, "foo -- bar" (the latter being a style borrowed from the example in pep 257)05:17
SteveAsome doc system uses it, some pep uses it somewhere05:17
jameshis it reStructuredText format or something?05:19
=== Virtuall [~virtuall@who.is.virtuall.info] has joined #launchpad
bradbDo you guys find this easy to read?05:26
bradb    Keyword arguments:05:26
bradb    :from_addr: a string05:26
bradb    :bugdelta: an IBugDelta05:26
bradb    :to_addrs: a string, list, or tuple. If a list or tuple, an email05:26
bradb               is delivered to each recipient individually.05:26
daflooks reasonable to me05:27
bradbok05:28
bradbstub: Where do I read about the proper way to add a new value to the LP config?05:36
elmostub/spiv/stevea: the librarian is up to 100G space usage btw - i can't remember what the cut-off figure you asked for warning was at tho05:44
SteveAit is in the spec05:45
SteveAi think .2 TB05:45
SteveAthanks for the note05:45
=== SteveA --> lunch
=== SnakeBite [~SnakeBite@84.242.143.64] has joined #launchpad
carlosstub, any chance to get the poimport script added to cronscript today?06:00
=== morgs -> supper
=== lamont [~lamont@15.238.5.154] has joined #launchpad
mptbradb: If you're fixing bug 1341, why haven't you accepted it?06:32
mptIs it a duplicate?06:32
mpt(if so, I can't find the original)06:33
carlosmpt, did you see my question (yesterday) about the missing css code for the po download pages in Rosetta? the language selector looks really ugly06:35
bradbmpt: because i didn't search for a bug report when i started working on it06:35
carlosmpt, https://launchpad.ubuntu.com/distros/ubuntu/hoary/+sources/gnome-panel/+pots/gnome-panel-2.0/+export06:36
mptcarlos: No I didn't06:36
carlosmpt, would you fix it, please?06:37
mptcarlos: sure06:37
mptthat's a one-line fix06:37
carlosmpt, cool, thanks06:37
mptbradb: Searching Malone needs to be made more attractive, then :-)06:40
bradbIt already is purtier in my menus branch, but that work is being delayed by other refactorings, as requested by SteveA 06:41
bradbmpt: btw, remind me, did you have an ETA on when you might have a little google-like HTML/CSS snippet that I can plug into the bugtask listing?06:42
mptbradb: no, sorry06:42
mptpester me when you get here, and I'll do it under your watchful eye06:43
mpts06:43
=== bradb adds that to BrazilTopics
=== mpt finishes wading through the complete list of Malone bugs
=== bradb eats lunch while make check runs
mpthttp://www.google.com/search?q=site%3Awiki.launchpad.canonical.com06:51
elmooh, right, I may as well de-SSLize that06:53
elmosay...07:00
elmowhy did I get:07:00
elmoR. [  21: The Launchpad Team  ]  Launchpad Account Creation Instructions07:00
elmothat today?  and how many other people did? :p07:00
=== Burgundavia [~corey@S0106000000cc07fc.gv.shawcable.net] has joined #launchpad
carlosstub, is librarian still down?07:07
carlosstub, I'm getting emails with errors '500' from librarian07:08
stubcarlos: Oh - I thought elmo had bounced it. I'll bounce it now.07:08
carlosstub, ok, thanks 07:09
stubcarlos: Should be fine now07:09
carlosok07:09
carloscould you take a look at the poimport script please? :-)07:09
carlosnow that you are connected...07:10
KinnisonSee you all on Sunday/Monday07:21
Kinnisonkiko: Will anyone meet me at the bus station on Sunday, or should I wander up to the hotel on my own?07:22
Kinnisonkiko: Or indeed should I come and bang on your front door?07:22
kikoKinnison, what time do you arrive?07:23
Kinnisonkiko: plane lands at 0510 as per the wiki07:26
KinnisonOther than that, I dunno, I imagine out of the airport by 0600, so hitting SC around 1000 ?07:26
=== Keybuk [~scott@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
salgadoSteveA, might you have some time to look at my changes in basicvoting (https://chinstrap.ubuntu.com/~jamesh/pending-reviews/guilherme.salgado@canonical.com/launchpad--basic-voting--0/filtered-diff) today?07:31
kikoKinnison, prolly07:35
stubcarlos: poimport reenabled as per email07:36
carlosstub, thank you07:37
=== guillem [~guillem@62-43-130-250.user.ono.com] has joined #launchpad
guillemhola com va tot?07:37
kikonot bad07:38
carlosguillem, hi07:38
guillemhi carlos07:44
guillemde donde eres?07:45
kikom/f?07:47
=== debonzi [~debonzi@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
carlosguillem, este canal es de habla inglesa, la mayora no entienden espaol.07:50
mptelmo: Oh, that was my fault, sorry07:53
mptelmo: Someone was asking me about the "merge accounts" page, and trying to find it, and we guessed that you wouldn't have a Launchpad account, but you did, or vice versa07:53
elmook, that's cool as long as it wasn't to _everyone_ ;-)07:53
bradbsalgado: did you see my reply to your code review of one-bugmail-per-recip?07:59
salgadobradb, yep08:05
Keybukit was a "what happens if we do X?  Does anyone know someone who won't have added himself to launchpad yet?" kind of thing08:06
salgadobradb, you use Mail-followup-to: header because you want the replies going to the list but not you?08:09
kikoI HATE THAT HEADER08:10
bradbsalgado: yes; getting two emails just annoys the heck out of me08:10
Keybukwhy does Malone send e-mails as "Blah but really Malone <xxx@malone>" ?08:11
Keybukcan't it use headers properly, and do08:11
KeybukFrom: Blah <blah@blah.com>08:11
KeybukSender: Malone <admin@malone.com>08:11
KeybukReply-To: Malong bug X <xxx@malone>08:11
=== salgado thinks about making mutt ignore that header just to annoy bradb. :P
bradbif I were lazier, I'd setup my filtering to autokill the second message just in case08:12
stubBjornT: Is there a reason you decided to use the From: address instead of the Reply-To: to get replys to Launchpad? I know Roundup does both for some reason, but I think Keybuk's suggestion would be fine.08:16
dilysMerge to thelove@canonical.com/dists--bazaar--1.5: new build (patch-51)08:17
mptarg08:17
dilysMerge to thelove@canonical.com/bazaar--devo--1.5: safe_flush at the end of arch_print_headers_summary, display revision info as soon as possible in archive-mirror (patch-40: Matthieu.Moy@imag.fr)08:17
mptAfter I've done a "make clean" on Launchpad, how do I get it running again?08:18
kikomake build iirc08:18
kikoand then make run08:18
kikomake schema and make run08:18
mptthanks08:18
kikoso carlos, daf?08:18
kikoProgrammingError: ERROR:  permission denied for relation translationgroup08:19
kikocarlos, daf: I've asked stub to fix up permissions for that table for the poimport user -- unless you scream08:19
bradbstub: I designed/implemented the From: header. Relied on the From: header because Reply-To seemed unnecessary.08:19
stubI'll update it on production but would like someone to update security.cfg and merge it as [trivial] 08:19
carlosstub, I will do it08:20
kikosalgado, how do you make mutt ignore that header?08:20
kikohmmm08:20
carlosstub, the tests didn't detect it because the transaction problems spiv is working on08:20
kikobradb, stub, what do you think of Keybuk's suggestion?08:20
mpt    ImportError: No module named gettextpo08:20
mpt*** ERROR: Trebuchet died prematurely!08:20
mptkiko, that didn't work08:20
bradbkiko: it's described why we didn't do that in the spec08:20
bradbkiko: it seems a bit confusing, IMHO08:20
kikobradb, point dear Keybuk to the spec then08:20
salgadokiko, I don't do that. neither I know if it's easy08:20
bradbKeybuk: from the spec:08:21
kikobtw08:21
bradbThe From: header will contain the name of the person that made the change. It is intended to help someone scanning their Inbox to "weight" each mail. We intend for users to take advantage of the X-Malone-BugXXX headers to filter and sort their mail if they want to, but being able to quickly scan your "critical" folder for reports from your manager can be quite useful. :)08:21
bradbIt's also intended to be clear that this email came through Malone ("...via Malone") and show the email address that will be replied to, rather than confusing the user with Reply-To trickery. 08:21
Keybukthat doesn't say why you're not doing that08:26
Keybukit just says that you're not08:26
carlosstub, the poimport security entry is completely empty??08:26
bradbKeybuk: "rather than confusing the user with Reply-To trickery." is a hint :)08:27
KeybukI've never seen a user confused by Reply-To08:27
bradb"show the email address that will be replied to", etc.08:27
Keybukshow the e-mail address is a misnomer, most modern mail clients *HIDE* the e-mail address08:27
kikostub, carlos, daf: translator also needs permissions.08:27
stub And a test that runs the script, connecting as the correct user.08:28
Keybukso you're going to confuse the user by making them think they're replying to the submitter, not Malone08:28
carloskiko, any table we use will need permissions, as I said, it's empty :-?08:28
kikocarlos, could you list the tables to stub, then, please?08:28
carlosstub, I told you already that I had to disable that test because transaction problems08:28
mptCan anyone tell me how to unbreak my Launchpad?08:28
mptSorry to be such a millstone08:28
carlosstub, spiv is working on it08:28
kikoah08:28
Keybuk(you just confused me, you see, and I don't consider myself an idiot user :p)08:29
bradbKeybuk: isn't that was "via Malone" is for?08:29
jameshmpt: is the GlobalLinks class in your branch meant to be used?08:29
KeybukI assumed that meant it came from Malone, not that my reply would go to Malone08:29
carlosstub, is there an easy way to get the list of tables the script needs?08:29
mptjamesh: no, I should have removed that08:29
carlosstub, is it possible that you were executing that script as the poexport user?08:30
bradbKeybuk: where did you think your reply would go to?08:30
Keybukthe original submitter only08:30
KeybukI actually hunted for a few seconds in the e-mail to find out how to copy that into malone08:31
Keybukand then noticed the From: address was mangled08:31
carlosit makes no sense that it does not have any right, we executed it before...08:31
mptKeybuk: How much do you use Bugzilla?08:32
Keybukoccasionally08:32
bradbKeybuk: hm, interesting. you're the first person to have mentioned being confused by this. which is not meant to suggest that obviously it's not a problem, but rather that i'd be curious to see what some proper user testing of the email notifications would discover, before we change anything just yet.08:33
mptAll Bugzilla mail comes from bugzilla-daemon@whatever08:33
Keybukbugzilla doesn't have a mail-reply-to-comment thing though, does it?08:33
mptno matter who caused the change, and no matter what bug08:33
KeybukI've always gone to the web page to reply08:33
jameshmpt: unleess the admin has changed the from: address08:33
stubcarlos: env LPCONFIG=production PYTHONPATH=$HOME/dists/launchpad/lib LP_DBUSER=poimport LP_DBNAME=launchpad_prod LP_DBHOST=emperor python $HOME/dists/launchpad/cronscripts/rosetta-poimport.py -q08:33
mptKeybuk: sure, but that's not your use case here, your use case is sending mail to the person who made the change08:33
jameshmpt: I've seen some with bugzilla@...08:33
mptNeither Bugzilla nor Malone let you do that08:33
Keybukwhen I get an e-mail from a BTS with some text in it, I want to be able to reply to it08:34
Keybukto reply to the comments that person makes08:34
Keybukthe headers of the incoming e-mail should be set up to make it obvious that the reply is going to be copied into the bug tracking system08:34
mptfair enough, I occasionally do that too (usually to say "that comment probably wasn't helpful)08:34
mptbut the most common case is adding a comment to the bug itself08:34
Keybukusing the standard From (person who wrote the comment), Sender (automatic system that generated the e-mail) and Reply-To (where your reply will go) headers would accomplish that08:34
Keybukright08:35
Keybukso I expect that replying to a mail from $BTS should add a comment to the bug itself08:35
mptIs this the Reply-To-Munging-Considered-Harmful flamewar in disguise?08:35
mptI think it is08:35
jameshmpt: we aren't munging emails -- we are generating them08:35
KeybukReply-To is harmful for mailing lists, not ordinary use08:35
carlosgrrr08:35
kikoI was about to say that.08:35
kikocarlos?08:35
carlosI forgot to remove the automatic imports from the notifictions08:36
carlosdaf, be prepared, we are being spammed...08:36
KeybukFrom: Matthew Paul Thomas <mpt@canonical.com>08:36
KeybukSender: Malone <malone@launchpad.ubuntu.com>08:36
KeybukReply-To: Malone Bug 1488 and Matthew Paul Thomas <1488@bugs.launchpad.ubuntu.com>08:36
Keybuk-- 08:36
Keybukwould be how I'd do the headers08:36
KeybukFrom mpt, really sent by Malone and replies go to the magic address that sends to both08:37
mptok, so what would a "Reply" button do?08:37
Keybukreply would go to the Reply-To address08:37
KeybukReply-To overrides From, From overrides Sender08:37
mptso some mailers have a special "reply to author" function?08:37
Keybukthis is spelled out very clearly in the RFC, and I've never seen a non-compliant mail client08:37
kikocarlos, automatic imports?08:38
mptThunderbird is sorta non-compliant because it refers to From as "Sender" in its UI08:38
Keybuksome mailers do provide a "reply to the From, ignoring Reply-To or Mail-Followup-To" function08:38
mptok08:38
carloskiko, hoary imports08:38
Keybukand, in my example, that button becomes useful08:38
Keybukbecause I can deliberately reply to you, in private, to avoid mail going to the list08:38
mptthat seems reasonable, bradb?08:38
carloskiko, the po imports are being executed so we are getting confirmation emails08:38
Keybukuh, s/list/bug/08:38
bradbsalgado: replied to your reply. do i get my r=salgado now? :P08:38
Keybukwhere in the current system, I can't do that ;)08:38
=== bradb catches up on scrollback
carloskiko, s/hoary/breezy/08:38
kikoI think Keybuk is correct.08:39
bradbhm08:41
bradbOf the headers mentioned, I only show From: in my mutt configuration08:41
bradbYou might be right, but I'll note that: 1. you'd be less confused but I'd be more confused and 2. would it be a good idea to user test this on a couple more people before we change functionality that has only recently been released?08:43
bradbOne other thing I was curious about: when's the last time you guys replied to an email where the address that was replied to was a different address than the From: address of the email? What about Normal People? Is it something they'd be used to?08:45
kikoI think so08:46
Keybukthe mutt default config includes reply-to08:49
Keybuk(me just checks)08:49
Keybukchecked08:50
kikobradb, I think Keybuk is right, and moreover, stub does too.08:50
salgadobradb, it should be good to go now.08:51
mptbradb: (1) The current behavior hasn't been user-tested either, and (2) I doubt this is the sort of thing you could user test at all08:51
bradbmpt: 1. exactly my point ;) as to whether it's testable or not, i'm not really sure08:53
bradbdo you guys have an example of a threaded email discussion where the From: address and the Reply-To: are different?08:54
=== Keybuk points at debbugs
KeybukFrom: Vagrant Cascadian <vagrant+bugs@freegeek.org>08:56
KeybukReply-To: Vagrant Cascadian <vagrant+bugs@freegeek.org>, 319421@bugs.debian.org08:56
KeybukSender: Debian BTS <debbugs@bugs.debian.org>08:56
=== Keybuk picks the bottom example in his inbox
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Minor cleanups to database/potemplate.py, and a lint.sh fix (patch-2151: christian.reis@canonical.com)08:56
bradblooking at my RSS feeds in tbird, i only see the From: address08:58
bradbso, there's this notion with transactional email being a customer service representative08:59
bradbthe From: field of a transactional email should show two things:09:00
bradb1. a recognizable brand name09:00
bradb2. something to distinguish this email from spam09:00
bradbif we make this change and i file a bug09:00
bradbi'm going to get a bugmail09:00
bradbFrom: Brad Bollenbach <brad.bollenbach@ubuntu.com>09:00
mptfrom "Brad Bollenbach"09:01
mpta highly suspicious name09:01
bradbif i ever get an email from "Brad Bollenbach", yes, i'm going to be highly suspicious :)09:01
mptbradb, if you're receiving bugmail when a bug is *reported*, it's because you're involved in software development, which means you're almost certainly on one or more development mailing lists, which means you're getting e-mail from random strangers all the time09:02
bradbso, does a From: Brad Bollenbach <brad.bollenbach@ubuntu.com> achieve both the stated objects? (as per Jakob Nielsen's words, not mine, but that doc was a foundation of the design)09:02
mptAnd if you're getting bugmail at any other time, it's either because the bug has been reassigned to you, which means you're involved in software development, etc; or it's because you've subscribed to the bug, in which case you recognize the subject line.09:03
Keybukpersonally, spam-wise, I'd fine09:04
Keybukuh, find...09:04
KeybukFrom: Brad Bollenbach <brad.bollenbach@ubuntu.com>09:04
jameshmpt: or if someone else asks a question about a bug you openned?09:04
Keybuknon-suspicious (ish)09:05
Keybukbut09:05
KeybukFrom: Brad Bollenbach <an-email-address-that-isn't-brad's>09:05
bradbKeybuk: sure, but if it were an email from yourself?09:05
Keybuksuspicious09:05
mptjamesh: That == you're subscribed (in the Malone model)09:05
Keybukbradb: I'd expect an e-mail to be Scott James Remnant <scott@netsplit.com>09:05
bradbKeybuk: agreed, hence the "via Malone" bit, which you've already said tells you it came from malone09:05
mpt(so I should have said "you're", rather than "you've"09:05
Keybuknot an e-mail address that isn't mine09:05
jameshmpt: ah.  That's "you're subscribed" rather than "you've subscribed"09:05
mptjamesh: snap.09:05
Keybukbut that's abusing things for which there are perfectly good additional headers09:05
KeybukI guess you're worry is that you might be exposing your e-mail address to spam harvesters?09:06
mptbradb: "... via Malone" is often hidden beyond the ellipsized part of the From address in the list pane of my mail clients09:06
jameshwe already expose email addresses, don't we?09:06
bradbmpt: yup09:06
kikojamesh, to logged in users09:07
bradbso, the other part is: does: Brad Bollenbach <brad.bollenbach@ubuntu.com> contain a recognizable brand name?09:07
Keybukif someone's getting an e-mail FROM malone, aren't they by definition someone with a Malone account? :p09:07
jameshkiko: okay.  and anyone receiving a bug mail would have been an authenticated user in order to subscribe to the bug ...09:08
bradbKeybuk: And after the recognizable brand name question, there was another thing I was curious about: looking at a bugmail in this way in tbird, how do i actually know that the bugmail even came from Malone at all until i hit reply (and even then, do i know?)09:09
=== SnakeBite [~SnakeBite@84.242.143.64] has joined #launchpad
Keybukwhy do you care?09:09
Keybuktbird should at least show the Reply-To ?09:09
Keybukwhich will have MALONE in it09:09
bradbKeybuk: because I want to know that it came from Malone :)09:09
mptbradb: This is a bug tracker, not buylotsofstuffwithoneclickshopping.com09:09
Keybukyou do ?!09:10
Keybukput it in the e-mail body :P09:10
bradbKeybuk: the more we shove in the body, the more litter in replies09:10
jameshbradb: isn't the Launchpad branding in the URL enough?09:10
KeybukI don't think that belongs in From:09:10
bradbjamesh: yup, but you have to be able to see it, right? :)09:10
bradbjamesh: or, otherwise, be given some hint in some way (e.g. "via Malone", whatever)09:11
mptbradb: Ok, if you want branding in the From, make it 12345@malone.launchpad.net.09:11
jameshbradb: it's in the second line of the email ...09:11
bradbmpt: yep09:11
mptoh, wait, that's not the author's address09:11
mptha09:11
dilysMerge to thelove@canonical.com/dists--bazaar--1.5: new build (patch-52)09:11
bradbjamesh: users won't even get that far if the email looks spammish09:12
dilysMerge to thelove@canonical.com/bazaar--devo--1.5: fixed a segfault in baz archive-version (patch-41: Matthieu.Moy@imag.fr)09:12
bradbdo you base your email filters on Reply-To or From, for example?09:12
Keybukpersonally?  neither09:13
Keybukbut in general, From:09:13
Keybukand if Malone was circumventing my Brad Bollenbach killfile, I'd be very upset with it <g>09:13
mpthaha09:13
bradbhm, this is all sounding a bit far-fetched to me so far09:15
jameshbradb: if users start assuming that emails with senders of the form "foo bar via Malone" and we get popular, I can guarantee that we won't be the only people sending emails of that form :)09:15
Keybukfor reference, procmail's filters check sender, from and to09:15
Keybukuh, reply-to09:15
mpt"via" doesn't seem a very robust way of inventing new headers09:16
Keybukplus the rfc already gives you perfectly servicable headers for this :p09:17
bradbI'm trying desparately to convince myself that it's a good thing that a From: address in no way indicates this email is connected to Malone.09:18
kikobradb, turn it around. try to convince /us/ that there is a need for the from address to indicate the mail comes from malone.09:19
kikobradb, remember that there is a [bug XXX]  bit in the subject09:19
Keybuk(and read  3.6.2 of RFC 2822 :p)09:19
mptbradb: And don't refer to people who are talking about e-commerce :-)09:19
bradbkiko: http://www.useit.com/alertbox/20031208.html, in particular, this passage:09:19
bradbIn most cases, the from field should clearly show two things: a recognizable brand name (if available), and a function that clearly distinguishes the message as a transaction rather than an advertisement. In our study, effective senders included reservations@hilton, tickets@amtrack.com, and ship-confirm@amazon. Note that most of these from lines were truncated by the in-box view: you typically have no more than 20 characters to conv09:20
mptbradb, what did I just say? :-P09:20
bradbkiko: [Bug 42]  foo, doesn't give any hint that it's from Malone09:20
bradbcould just as well be from bugzilla or somewhere else.09:20
mptbradb: How about "[Malone bug 42] ?09:20
Keybukbradb: those are for totally automatic messages though09:21
Keybukyour messages are retransmissions of an e-mail or comment supplied by an author09:21
bradbmpt: it's only Malone bug 42 if it's bug #42 on Malone, right?09:21
Keybukbtw09:21
Keybukthose three things happen to be in my killfile ;P09:21
mptbradb: Only if the same applies to "via Malone"09:22
bradbmpt: also, microcontent. do we have room for "[Malone..." at the beginning of every subject line?09:22
mptbradb: No we don't, kiko's just jumped on me for that09:22
bradbKeybuk: not necessarily.09:22
bradbKeybuk: could have been a status change, adding an external link, setting a bug private, etc.09:23
Keybukright, so _those_ e-mails should be From: Malone ...09:23
Keybukseriously, look at debbugs, it does this exactly right09:23
Keybukbecause it's had ten years of people who know how e-mail systems should and must work making absolutely sure it does09:24
Keybukyour argument seems to be that some random e-commerce spam site says you should put Malone branding in the From address09:24
bradbKeybuk: dude, you also think that three bug statuses exactly right. :) there are non-power-users involved here too, right? :)09:25
Keybukmine is that you're giving the recipient more power by using the headers properly09:25
Keybukuh, I think there are _two_ bug statuses; "open" and "closed" :p09:25
bradboh yeah, heh :)09:25
Keybukseparating the original author and autonomous system allows me to make a choice09:26
KeybukI can hit "reply" and send it to the autonomous system (in Reply-To)09:26
Keybukor I can "reply to author" and send a private mail to the person who made the comment09:26
Keybukthe Sender header clearly identifies the mail as been generated by malone09:26
bradbmpt: so, from a usability perspective, you think it's a good thing that that the From: address *not* in any way indicate that this email came from Malone?09:27
mptbradb: No, I don't think it's a good thing, but I think the current approach causes other problems that are worse overall.09:27
bradbas if life weren't already difficult enough for MLM's with the bugmail from address :)09:27
bradbmpt: such as?09:28
mptbradb: Such as not being able to reply to the commenter without a fiddly copy-and-paste work.09:28
Keybukor allowing people to use Malone to e-mail people who have killfiled them09:29
bradbKeybuk: this change won't really change that09:29
bradbKeybuk: remember, people can add comments when they make other changes09:29
Keybukthose should be From: person-who-made-the-change09:30
Keybukso I can reply to it to ask them why09:30
mptbradb: If a confirmation from a Nielsen-hiring e-commerce site to you gets lost, that's their loss. If a Malone message to you gets lost, that's your loss and yours alone. You have an interest in ensuring you get Malone mail, and we're not making it particularly hard to get such mail.09:31
bradbi concede that the not-being-able-to-reply directly easily is an annoying problem, but i'm not sure that i agree that that's even worse than not knowing that the email came from Malone09:32
Keybukyou know the e-mail came from Malone09:32
bradbKeybuk: how?09:32
Keybukassuming you're not using a broken mail client (<g>) you can see the Sender: header09:32
Keybukand the SECOND LINE in the e-mail says malone09:32
Keybukand the Reply-To (which your e-mail client is _very_ broken if it's hiding) says Malone09:33
bradbKeybuk: dude, inside the body is too late :) the point is to make it clear to someone swimming through their inbox.09:33
Keybukin the summary view, the fact it begins [Bug XXXX]  is a bit of a hint that it comes from Malone, or some other BTS09:33
=== debonzi [~debonzi@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
kikoI agree with Keybuk 09:33
bradblooking at tbird, it appears to me to be just showing From:, but maybe i'm doing something wrong09:33
Keybukpersonally, I tend to search on the XXXX as the duplication between different systems that e-mail me is low09:33
Keybukthe chances are the "via Malone" bit is going to get truncated off ANYWAY in the summary view09:34
Keybuk[Bug XXX]  is the summary hint09:35
bradbin my default tbird view i don't think it would09:35
bradbKeybuk: not if you're already using 2 other bugzilla instances, right?09:35
Keybukbradb: don't you have one of those mad widescreen ultra-hi-res laptops though? :p09:35
Keybukbradb: I use debbugs and several bugzillas every day, and have still never got the same XXX bit09:35
KeybukI just filter my mailbox on the Bug#09:35
KeybukI don't think about which BTS/Bugzilla/etc. it came from09:35
=== Mez [~Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=salgado]  change bugmail notification to one bugmail per recipient (patch-2152: brad.bollenbach@canonical.com)09:42
kikocarlos?10:04
kikoor daf?10:04
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=stub]  fix up security permissions for my stuff (patch-2153: scott@canonical.com)10:16
SteveAbradb: ping10:19
jordihey10:32
jordicarlos10:32
jordihmm10:32
mptjamesh: Did you want me to remove that stray class before you re-review the footer branch?10:37
SteveAspiv: ping10:38
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=stub]  fix up security permissions for my stuff (patch-2154)10:42
jameshmpt: I hadn't actually started reviewing it -- just took a quick glance10:46
=== terrex [~terrex@84-122-73-155.onocable.ono.com] has joined #launchpad
KeybukSteveA: is Launchpad "Plone" or "Zope 3" ?10:59
SteveAwhat does that mean?11:03
Keybuk<-- he asked11:03
KeybukI thought it was a Plone app written with Zope 311:03
Keybukor something11:03
Keybukbut then I confuse Plone, Zope and Twisted all the time11:04
SteveAwe are borrowing the plone CSS stylesheet11:04
SteveAthat is all of plone there is in there11:04
Keybukright11:04
Keybukso what's Plone?11:04
SteveAplone.org11:04
KeybukI thought Plone was a webapp framework on Zope?11:04
SteveAit is a content management system built on zope211:04
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  More lintage fixes, and FINALLY make lint.sh work with python files (patch-2155: christian.reis@canonical.com)11:14
=== terrex se va a cenar // is going to dinner
=== SnakeBite [~SnakeBite@84.242.143.64] has joined #launchpad
bradbgotta step out for a bit, back later11:22
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=kiko, rs=sabdfl]  add dummy 'Code branches' page for each person (patch-2156: mpt@canonical.com)11:43
=== Mez [~Mez@cpc2-lich4-3-0-cust115.brhm.cable.ntl.com] has joined #launchpad
=== SnakeBite [~SnakeBite@84.242.143.64] has left #launchpad [""]
=== ddaa [~ddaa@200-171-140-32.dsl.telesp.net.br] has joined #launchpad

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