/srv/irclogs.ubuntu.com/2011/06/10/#launchpad-yellow.txt

=== Ursinha is now known as Ursula
=== Ursula is now known as Ursinha
=== Ursinha is now known as Ursula
=== Ursula is now known as Ursinha
=== Ursinha is now known as Ursula
gary_posterbac, benji, objectives today if possible (especially since I'm going on vacation)13:15
gary_posteralso, is it safe to run lp2kanban?  Should I just run it with13:16
gary_posterbin/py src/lp2kanban/bugs2cards.py -e gary.poster@gmail.com -p <yourpassword>13:16
gary_posteror do I need to specify the board?13:16
gary_posterIt looks like the ones I've done as test cases have not yet been synced13:17
benjigary_poster: I did mine yesterday... hopefully correctly.13:17
gary_postercool benji I'll look.  thanks13:17
gary_posterbenji, bac, oddly, I get13:18
gary_posterTraceback (most recent call last):13:18
gary_poster  File "src/lp2kanban/bugs2cards.py", line 5, in <module>13:18
gary_poster    from argparse import ArgumentParser13:18
gary_posterImportError: No module named argparse13:18
benjigary_poster: did you run the buildout? that's a new dependency13:19
gary_posterI did, but maybe before I updated.  lemme retry13:19
benjiif that doesn't work, perhaps it wasn't added to the install requires entry13:20
gary_posterbenji, bin/buildout did nothing, and retrying failed again.  We can look into it later13:20
gary_posterbenji, updating setup and buildout made it work.  I'l commit in a sec.  Looks like it is updating ythe test board?13:24
benjigary_poster: no idea, I haven't touched it lately13:24
gary_posterlooks like -b yellow should work...13:24
bachi gary_poster13:26
gary_posterhey bac13:26
gary_postergot it to work13:26
bacgary_poster: yes, yellowtest is the default.  safer that way13:26
gary_posteragreed13:26
bacgary_poster: and cards must be marked to get synced13:26
gary_posterbut no cards are reported as updated (I mean, that's what the script reports).  I'm seeing what the cnstraint is...13:26
gary_posterah ok13:27
bactitle.startswith("sync:") or a tag in the description13:27
gary_postercoo;13:27
gary_postercool, I see that13:27
gary_postertrying13:28
gary_posterbac benji danilos, call in 213:28
bacok13:28
danilosack13:28
benjigary_poster: for a second there I thought Julia had taken over your IRC client13:29
gary_posterbac, benji, ta-dar! :-)  I synced my two bugs13:30
benjiyay!13:30
gary_posterga ga goo goo13:30
danilosfwiw, does anyone find that leankitkanban is much less responsive lately13:30
bacgary_poster: ooo, try again13:30
danilosmaybe that was while the script was running :))13:30
gary_posterheh13:30
* benji wonders if it's onomatopoeia like a pirate day.13:30
gary_posterbac, running13:30
gary_posterI'll start the call when it is finished13:31
gary_posternote you have to reload the board13:31
bacnice13:31
gary_posterit's done13:31
bacno13:31
bacit reload automatically13:31
gary_posteroh I see13:31
gary_posterI just wasn't patient enough the first time13:31
gary_postersweet!13:31
danilosyeah, I got a notice how the board is updated13:31
benjiyeah, in my experience it behaves the same as when people update the board13:32
bacgary_poster: how did you resolve the argparser issue?13:32
gary_posterbac, just updated setup.py and buildout.cfg.  will commit13:32
bacwhy did it work for me?  is that required for things that are in the standard distribution?13:33
=== Ursula is now known as Ursinha
benjibac: my psychic powers tell me that you're running Python 2.7 which has argparse and Gary is running 2.6 which doesn't13:39
bacbenji: hmm, i thought i'd used 2.613:40
benjibac: that or you're using a python with argparse already installed13:41
bacbenji: i guess.  i don't recall installing it.  /me looks13:41
danilosgary_poster, lp:~danilo/launchpad/bug-772754-other-subscribers-remove-cruft13:42
bacbenji: i guess i'm wrong!  it is in /usr/lib/pymodules so i guess that is something i installed?13:42
benjiI can't tell from here. ;)13:43
* benji supresses a system-python-is-evil rant.13:43
bacyep, i'd installed python-argparse at some point13:44
bachey benji i was thinking it might be a good idea to save off the initial state of the kanban board in case we mess it up in some way.  i assume there is nothing in their API that would allow recovery.13:46
benjiyeah, I don't know of a way to set the entire board state (although, as you know, they let you retrieve it)13:47
bacbenji: i forgot to tell you that you and i swapped #launchpad help contact days this week!  i covered for you on monday.  you mind picking it up today?13:48
benjibac: sure13:49
benjiI swear!  Now the Gnome shell is crashing on me.  Maybe it wan't Unity's fault.13:50
benjigary_poster: after looking at bug 697735 for a little while, I've realized that either I don't think it should be considered a bug or I don't understand what it should do instead -- if you hand craft odd URLs you'll get wierd errors, what's wrong with that?13:56
_mup_Bug #697735: ValueError raised submitting bogus query string <lp-foundations> <oops> <Launchpad itself:Triaged by benji> < https://launchpad.net/bugs/697735 >13:56
gary_postersorry, just saw that; looking14:01
gary_posterbenji, from a high-level perspective, the problem is that OOPS reports are supposed to be actionable14:02
gary_posterbenji, it's fine for the user to get an error14:02
gary_posterbenji, but as you say, there's nothing for us to fix--so we should not get an OOPS14:03
gary_posterThat's my understanding14:03
benjimakes sense, I was missing that angle14:03
gary_posterOf course, fixing it may be a little exciting :-/14:03
benjiyeah, I'm starting to wonder if I made a bad pick.  I'll timebox at least the first stab at fixing it to see where it goes.14:04
gary_posteranyone, what date did you choose for your allhands objectives' target dates?  I have responded to all of them so I can't see them anymore  :-P15:02
gary_posterOh Apr 1215:03
gary_posternm15:03
danilosgary_poster, something random for me, mostly in March/April next year15:05
gary_poster:-)15:05
gary_postercool15:05
gary_posterdragnob said in her email today april 1215:05
gary_posterI dunno if that is real or not15:05
gary_posterbut whatever, it's in the ballpark :-)15:06
gary_posterthanks15:06
danilosgary_poster, fwiw, all the MPs are up and linked to the bug, in order of how they are in my pipeline (and in order of their dependencies; -lp-names is the only one which is relatively independent)15:07
danilosgary_poster, -sections, -subscribers, -activity, and -loading should be most interesting15:08
gary_posterdanilos, awesome.  I got your last branch and I think it is built now; I'll look at the UX, and the diffs, soon.15:08
danilosgary_poster, fwiw, anything after -loading (inclusive) but before the last one (exclusive) will have two subscribers lists :)15:09
gary_posterdanilos, heh, ok, good to know :-)15:09
danilosgary_poster, I wouldn't mind you reviewing a branch or two while I add tests for -actions :))15:09
gary_posterdanilos, ok cool.  I should be able to start in about 10 min.15:10
danilosgary_poster, yeah, no worries, you don't have to review them (especially not all of them, that's why I split them up :)15:12
danilosgary_poster, I am also asking on #launchpad-dev, so if are starting a review, please claim it15:12
gary_posterack15:12
benjigary_poster: what is an "informational" oops?15:16
Ursinhaan oops that's not an oops? :P15:16
gary_posterbenji, yeah15:17
gary_posterwe track them separately15:17
gary_posterwe don't need to drive them to zero15:18
benjigary_poster: so would inappropriate use of :int be an informational oops?15:18
gary_posterbut it is convenient to use the OOPS machinery to collect the data15:18
gary_postereh15:18
gary_posterno15:18
gary_posternot as far as I know15:18
benjibut we don't really want to collect anything15:18
gary_posterbecause we don't want to track itright15:19
gary_posterbenji, do you know how to generate api docs from a lazr.restful app, like api.launchpad.net/+apidoc ?  This is a question from ISD15:22
benjigary_poster: not off the top of my head, but I'm sure I could figure it out quickly enough15:23
gary_posterbenji, if that's ok, please do; my stack is getting too deep for last day before vacation :-)15:24
benjiheh, ok15:24
benjigary_poster: there's no super-straightforward way, but it's not too bad.  The best thing would be for them to look at utilities/create-lp-wadl-and-apidoc.py in LP.15:37
gary_posterbenji, ok thanks15:38
gary_posterbenji, I told Ricardo Kirkner, "pindonga," that he could ping you if he had questions.  I suggest that the first time he asks you for help, you give it to him as soon as convenient; subsequent times give him focus no more than once or twice a day if possible, unless, in your judgement, you think you ought to give him more.  Good relations are good, and they are doing important things.15:42
benjik15:43
danilosgary_poster, fwiw, I found one of the problems "generic" webkit JS engine (eg. epiphany) borks on: "class" as a variable name, or more frequently, a function parameter :/16:02
gary_posterdanilos, interesting!16:04
gary_postergood to file away16:05
gary_posterand unfortunate :-/16:05
danilosgary_poster, do you happen to know if that's a problem for Safari as well? I find it interesting because webkit JS should be identical in both, minus the version differences16:05
gary_posterI don't know off-hand danilos.  I mosty use Chrom these days, which has a different JS engine AFAIK16:06
danilosgary_poster, yeah, it does16:06
danilosgary_poster, looking at the code, it seems your unlanded branch is the only one which tries to use it, though :) http://paste.ubuntu.com/623491/ if you want to apply it to your branch directly16:12
gary_posterdanilos, cool, thanks, will do16:13
danilosgary_poster, ftr, it seems to be a problem with safari as well: http://blog.solution10.com/2010/02/use-of-class-in-javascript16:14
gary_postercool16:16
gary_posterthanks16:16
* danilos -> off, pushing branches up took longer than I expected, so still tests missing for the final branch16:30
* danilos needs to add task cards to the kanban board as well... :/16:31
* gary_poster lunches16:59
gary_posterbac, I want to reorder the sections in danilos' final branch.  I don't know if it affects yours.  I think that "Maybe notified" should be last, as it is in the mockup (and I wonder if we should have a help icon with the explanation for what "Maybe notified" means).17:01
gary_posterI suspect that is in a later branch17:01
gary_posterbut wanted to share now, in case it is in that one17:02
gary_posterI'll dig in more after lunch17:02
bacok18:24
gary_posterI'll just put the comment in the last MP and not worry about it, I think.18:31
benjigary_poster: I have a pretty good solution to the bug we discussed earlier (bug 697735) but I wanted to get a consult to be sure it doesn't go too high on the hack-o-meter: https://pastebin.canonical.com/48394/18:41
_mup_Bug #697735: ValueError raised submitting bogus query string <lp-foundations> <oops> <Launchpad itself:Triaged by benji> < https://launchpad.net/bugs/697735 >18:41
gary_posterk, looking18:41
benjiit scores a 3/10 on my unit here, but mine is known to be unreliable18:41
gary_poster:-)18:41
gary_posterbenji, mm, BrowserRequest.__processItem is so tasty.18:46
benjiheh, how so?18:47
gary_postersarcasm, just that it is so big and dense that copying and modifying is really unappealing, and then combine that with the __* munging and you have a really great reason to do the monkeypatch, which is a shame18:48
gary_posterbenji, did you consider having a marker interface for the request instead?18:48
gary_posterwe do that kind of thing a lot18:49
benjioh, indeed18:49
benjire. marker; well, it'd be a marker on the exception, right?  That'd be fine with me.18:49
gary_posterbenji, wouldn't it have to be on the request?18:50
benjithe reason I did it this way is because it mirrors the way we did something similar for the web service18:50
gary_postersince _isIgnoredException takes strtype18:50
gary_posternot the actual instance18:50
benjiit also (after I changed it) takes the exception instance18:50
gary_posteroh, duh, yeah18:51
gary_posterso yeah, on the exception sounds good18:51
gary_posterthat way you could do it on the fly or as a general statement in code18:51
gary_poster("this kind of exception is not oops worthy")18:52
benjisounds good18:52
gary_postercool18:52
gary_posterbenji, the only thing that really irks me is the monkeypatch, and I see why you did it.  Overriding __processItem does not seem like a nice option18:52
gary_posterwell...18:53
benjiright, it seemed like the lesser of the two evils; the other option would be to modify zope.publisher to make this easier18:53
gary_posterI looked for the word "monkeypatch" in the code, and I only saw mailman do it, which is not regarded as one of our high marks :-)18:57
gary_posterbenji, I'd get bac's opinion, which will be easy since he'll probably be your reviewer.  I think I'd trust his opinion more than mine.  The only other thought I had was that you could consider moving the monkeypatch to lib/lp_sitecustomize.py.  It is where we do other similar sorts of things, and is where people will expect to find stuff like this AFAIK18:59
gary_posterbenji, other than that, I think the interface-on-the-exception method of squelching OOPSes will be quite welcome, and should be announced.19:00
benjik; I'll look in lib/lp_sitecustomize.py, and add a comment there that hopedully would have helped me find it when I was looking for a home for this thing19:00
gary_postercool19:00
benjik19:00
* benji orders bright white, embossed announcement cards.19:01
gary_poster:-)19:01
* benji is reminded by his phone to do CHR tasks.20:05
benjihmm, we seem to be behind in project/license review (or I'm doing it wrong)20:12
benjigary_poster: your lost soul is still lost: https://support.one.ubuntu.com/Ticket/Display.html?id=2433  What should I do for him?  Ask a LOSA to change his account's email address?20:28
gary_posterbenji, I was bad (and/or swamped in other things) today and didn't do CHR, and we are missing gmb, so you are getting a bunch.  Sorry. :-(  bac, is there any chance you could give some thoughts on help we could give to that guy--the oops that he mentions?20:31
gary_posterI *think* we just need to set a preferred email, but I'm not really sure, and going int the raw SQL feels a bit like diving into the deep end which might have some hungry sharks hanging around.20:31
benjiheh20:31
gary_posterSo I'm hoping you have some insight on "tried and true" things to do to help that guy, bac20:32
gary_poster(from your registry days)20:32
benjik, I'll set his email address; how do we know that we're not being soccially engineered into giving control of someone's account to someone else?20:32
=== Ursula is now known as Ursinha-lunch
bacgary_poster: i'll look20:36
gary_posterbenji, we contacted him at this email address20:37
gary_posterthanks bac20:37
gary_postergetting kids from school, biab20:37
benjiHmm, I don't understand that.  He said he doesn't have access to the account's current email address any more.20:38
bacbenji: i'm confused too20:41
gary_posterbenji, bac, it may have been because it was compromised?  The situation was that his account was sending spam20:57
bacbenji: so here is my take.  the dude was suspended for spamming.  it is unclear whether he confirmed that he resolved his spam problem, i.e. changed password, got rid of browser issue20:57
baci think when we reactivate his account he has to go to the 'forgot my password' dance to get reinstated20:58
bacis that right, gary_poster?20:58
gary_poster(he said there was no browser issue)20:58
bacif so, he's currently stuck as he won't be able to retrieve the link20:58
gary_posterforgot my password: well...we suspended his LP account20:58
gary_posternot his SSO acount AFAIK20:59
bacgary_poster: then why can't he log in now?  why is his account still showing as suspended (grey avatar)?20:59
gary_posterI don't know.  I think it is because of an LP bug20:59
gary_posterif you look at the state in the administartion page20:59
gary_posterthen I think you'll see that he is actuve21:00
gary_posteractve21:00
gary_posteractive21:00
gary_poster(he said, typing slowly)21:00
gary_posterwhat I thought we needed21:00
gary_posterwas to set a preferred email21:00
gary_posterbecause that's what the OOPS points to21:01
gary_posterbut maybe logging in via SSO would work?21:01
gary_posterShould we tell him to make a new account, and then merge the new into the old?21:01
gary_posterif there's no accessible email associated with the old account, that won't work either21:02
bacoh, i missed that there was an OOPS21:02
bacno, a merge will not work21:02
gary_posterok21:03
bacgary_poster: what is the OOPS?21:03
baci don't see it21:03
baci don't think lifeless understands my silly attempts at humor.21:03
gary_posterbac, argh, sorry.  This is not the most pitiful of souls, nor the one I was thinking about or talking about.  I meant https://support.one.ubuntu.com/Ticket/Display.html?id=212021:04
gary_posterwho has been unable to log in for two weeks now21:04
bacaha!21:04
gary_posterbac, OOPS-1973H9021:04
gary_posterbug 79367021:05
_mup_Bug #793670: User account missing preferred email after suspension/reactivation <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/793670 >21:05
benjigary_poster: what's the bug number for the redoing of the bug subscription overlay you were working on?21:05
benji(I can't find it on the board.)21:05
gary_posterbenji bug 772754 (Feature Work: (BIG BUGS))21:06
_mup_Bug #772754: After better-bug-notification changes, list of bug subscribers is confusing <qa-ok> <story-better-bug-notification> <Launchpad itself:In Progress by gary> < https://launchpad.net/bugs/772754 >21:06
benjidoh!  I've gotten so used to not looking in there that I totally missed it.21:06
gary_poster:-)21:07
benjiI'm tempted to be "conversational" when chaning bug statuses or marking them duplicates, i.e., adding a comment about what I'm doing and why, but that doesn't seem to be the culture so I'm abstaining21:08
gary_posterbenji, dunno.  I do it when I think it adds value21:09
benjithe only "value" I was feeling was along the lines of softening the "they just marked my bug as Low, they must hate me" reaction21:10
gary_posterI have about one hour before vacation...I could help with CHR or read about CoffeeScript.  Which one sounds like more fun?  Which one should I actually do?21:10
gary_posterYeah21:10
gary_posterI've written those21:10
gary_posterI've become more...blunt lately with the new standards21:11
gary_posterSo I have not21:11
gary_posterbut do what feels right21:11
bacgary_poster, benji: i have not idea what to do for amichai221:13
bacgary_poster, benji: short of asking a LOSA to do a SQL mod to give him a preferred email address21:13
benjigary_poster: I just finished CHR (whew, it took more than an hour), so I doubt there's anything to do21:13
gary_posterbenji, I was pondering https://support.one.ubuntu.com/Ticket/Display.html?id=251421:14
benjioh, feel free handle that one ;P21:14
gary_posterbac, that's what I was thinking.  Would you feel comfortable assembling such SQL?21:14
bacgary_poster: will it fulfill my SQL goal?  :)21:15
gary_posterbac, lol21:15
gary_posterbac, it would help21:15
bacsure, i'll see what i can do21:16
gary_postercool21:16
bacam i going to need a TL or higher to sign off on my losa request?21:16
gary_posterbac, yeah, I think TL can do it21:17
gary_posterbenji, FWIW, I assigned it to Matthew Revell and sent a message to the requester and Matthew to that effect21:19
benjicool21:20
gary_posterbenji, fun: today Danilo alerted me that webkit falls over if you use "class" as an argument because it is a non-functioning reserved keyword in JS.  All other browsers are fine with it.  coffeescript compilation (and repl) clearly disallows.  another win.21:25
benjiheh, cool21:25
benjigary_poster: I assume that goes for all the other reserved, but unused words.  There are lots of them.21:27
gary_posterbenji, I'd hope so. dunno21:27
gary_posterI'd expect so tbh.  this obviously has been around the block21:27
bacgary_poster: it looks like there is enough info at https://wiki.canonical.com/InformationInfrastructure/ISD/Docs/SSO/ErrorSettingPreferredEmail for a losa to work with21:32
gary_posterbac, oh interesting21:33
gary_posterbac, +1 I guess...assuming he has an email at all, which I hope he does21:33
gary_posterI mean, registered in LP21:33
bacbenji: did you see statik talking about playing with coffeescript?21:37
benjinope, I'll look in my logs21:37
bacbenji: it was only 15 minutes ago or so21:45
bacgary_poster: is there any chance we messed up with that user?  should he have gone from suspended to unactivated?21:45
bacwhich would require following an activation link?21:45
baci can find no docs21:45
gary_posterbac, I went from suspended to activated, and that failed21:46
gary_posterthen21:46
gary_postersinzui said we'd had truble with that in the past21:46
gary_posterand suggested I go from deactivated to activated21:46
gary_posterso I did that21:46
gary_posterand that failed too21:46
bacdid you try unactivated?21:47
bacgary_poster: the guy's email address was marked 2-validated.  we forced it to preferred22:03
bachttps://launchpad.net/~amichai222:03
baclooks much better22:03
bacGo Banana Slugs!22:03
gary_posteryay bac!22:12
gary_posterthanks22:12
bacgary_poster: the other gent was in the same boat.  i think we've got a suspend/reanimate issue here22:13
gary_posterbac, huh22:13
gary_postermaybe make a comment on the related bug?22:13
bacthis guy: https://support.one.ubuntu.com/Ticket/Display.html?id=243322:14
bacok22:14
gary_posteryeah, I figured that was who you meant22:14
bacif the bug is not critical it should be22:14
gary_posterI think I marked it critical22:14
gary_posterbug 79367022:14
_mup_Bug #793670: User account missing preferred email after suspension/reactivation <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/793670 >22:14
gary_posteryeah22:15
gary_posterbye22:22
gary_postertty;22:22
gary_posterl22:23
gary_posterneed to get better at this typing while walking thing :-)22:23
=== Ursinha-lunch is now known as Ursinha

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