/srv/irclogs.ubuntu.com/2005/09/21/#launchpad.txt

sabdflmpt: pls would you help ogra design a less bogn screensaver unlock dialog?12:11
sabdflbong, even12:11
mptsabdfl: It'd be a pleasure12:12
sabdflthanks muchly12:13
mptThe design isn't the problem, ogra and I both know what's wrong, but apparently even moving a button is a herculean code task12:13
sabdflerrr... why?12:13
mptsomething to do with the way the xscreensaver code is written, combined with it not using a toolkit, I guess12:14
mptask ogra12:14
Burgundaviasabdfl, is it worth pouring a lot of money into the xscreensaver dialog when we are likely to drop it for dapper?12:16
Nafallognome-screensaver wfm ;-)12:19
ddaaomg... svn is so BoOOoong...12:19
ddaasay you have a branch A with 3 commits, latest revision is 4 (1 is import). Then you create a branch B with one import and one commit (latest revision of B is 6).12:20
ddaaIt _looks like_ when you update A after creating B, then you update it to... revision 6!12:21
ddaaeven though revisions 5 and 6 have nothing to do with A!12:21
lifelesshere12:27
lifelesslooking aete it now12:27
sabdfllifeless: be great if i can land stuff tonight01:02
=== sabdfl [n=mark@pdpc/supporter/silver/sabdfl] has left #launchpad []
=== WaterSevenUb [n=WaterSev@bl5-43-178.dsl.telepac.pt] has joined #launchpad
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: rollout failed patch (patch-2418)01:16
dilysMerge to rocketfuel@canonical.com/hct--devel--1: revert failed patch (patch-19)01:18
dilysMerge to rocketfuel@canonical.com/sourcerer--devel--0: revert failed patch (patch-29)01:18
Nafallohmm01:19
Nafallolooks fun :-P01:19
=== sabdfl [n=mark@host217-37-231-22.in-addr.btopenworld.com] has joined #launchpad
lifelesssabdfl: scotts patch is backed out, running check again to be sure01:22
sabdflthanks rob01:26
lifelessno probs01:26
lifelessno idea how I let it slip through01:27
=== lifeless is sorry about that
sabdflnp01:27
sabdflwe'll sort it out in the morning01:27
sabdflsweet dreams01:27
lifelessyup01:27
sabdfldid you have fun this evening?01:27
lifelessthanks01:27
lifelessI'll be another 20+ while the final test runs01:27
lifelessyup, was goo d to catch up. we were as a 'mysociety.org' evening01:27
lifelesswhich is some sort of apolitical political hackers group01:28
lifelessand was interesting in its own right01:28
cprovlifeless: hi rob, is it solved already ?01:32
lifelesscprov: I've rolled the patch out01:32
lifelessbut pqm is still disabled while I check tests really do pass again01:33
cprovlifeless: ok, I can do it later, at home, thank you for come and fix it ;)01:34
lifelessnp01:35
lifelessok, pqm is back01:40
lifelessnight all01:40
=== cprov sent a PQM "bomb" request and leave ;)
cprovnight guys01:50
=== stub [n=stub@203-214-4-72.dyn.iinet.net.au] has joined #launchpad
=== mpt wonders why he's getting "Failed to verify signature" error messages from PQM without having sent anything to PQM in the first place
dilysMerge to rocketfuel@canonical.com/launchpad--production--1.32: Cherry pick patch-2416 into production (patch-5: stuart.bishop@canonical.com, rocketfuel@canonical.com)02:48
dilysMerge to rocketfuel@canonical.com/launchpad--production--1.32: Executive override on production link style (patch-6: steve.alexander@canonical.com)03:15
=== kiko [n=kiko@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== Burgundavia [n=corey@S0106000000cc07fc.gv.shawcable.net] has joined #launchpad
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
=== mdz [n=mdz@ca-studio-bsr1o-251.vnnyca.adelphia.net] has joined #launchpad
=== robitaille [n=robitail@d154-5-117-228.bchsia.telus.net] has joined #launchpad
sabdflmorning all08:07
sabdflstub: morning - did that landing go ok on production-branch?08:08
lifeless10:48 < dilys> Merge to rocketfuel@canonical.com/launchpad--production--1.32: Cherry pick patch-2416 into production (patch-5: stuart.bishop@canonical.com, 08:22
lifeless               rocketfuel@canonical.com)08:22
sabdfli don't think that's it08:22
sabdflmy code never landed in production08:22
sabdfli asked stub to merge directly from my branch08:23
sabdflit didn't land because of PQM fuckage last night08:23
sabdflstub: please ack?08:23
stubmark.shuttleworth@canonical.com/launchpad--cverework--0--patch-26 ?08:24
sabdflstub: yes08:24
=== sivang [n=sivang@box79162.elkhouse.de] has joined #launchpad
stubI thought you wanted that in the next production rollout, not cherry picked into current.08:24
sabdflstub: i want that in the rollout for tuesday08:24
stubYes - it will be.08:25
sabdflok, did it land smoothly?08:25
sabdfllifeless: is PQM landing things again now?08:25
stubProduction branch hasn't been created yet - I generally do it on Monday in case plans change re: what needs to be rolled out.08:25
stubI've got it here in my notes though so it won't be missed08:26
stubI'll do it now so you won't fret on your break ;)08:26
sabdfli'm away from this evening, so perhaps its worth confirming it merges cleanly now?08:26
sabdflthanks dude08:27
sivangmorning all08:28
sabdflhey sivang08:28
lifelesssabdfl: yes08:34
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=spiv]  script librarian logging (patch-2419: stuart.bishop@canonical.com)08:49
dilysMerge to rocketfuel@canonical.com/dists--devel--0: [trivial]  Prepare production 1.33 for Tuesday rollout (patch-114: stuart.bishop@canonical.com)08:53
stublifeless: Can you please mirror that production branch08:59
SteveAmorning09:02
stubmorning09:05
SteveAhm... so i need to tell pqm to merge that UI underlines thing...09:10
sabdflSteveA: i fear your patch will collide with mine09:18
sabdflplease drop mpt's bugtask changes09:18
sabdfli've made substantial cleanups to bug-headline-tasks and don't want them overridden09:19
=== BjornT [n=bjorn@office.pov.lt] has joined #launchpad
stubSo I need to cherry pick the underlines off patch into Tuesdays production release too?09:19
sabdfli already did the no-underline thing in the bugtask-headlines work that i did09:19
SteveAsabdfl: the patch is in pqm's queue.  http://pqm.ubuntu.com/09:22
SteveAhttps://chinstrap.ubuntu.com/~dsilvers/paste/filefkv7L3.html09:22
SteveAthat is the diff it is trying to apply09:23
SteveAi don't think pqm has a "cancel please" command09:23
SteveAunless you're rob.09:23
spivSteveA: It does if it's not at the head of the queue yet... temporarily break your mirror by e.g. moving it.09:24
SteveAspiv: is that going to break anything else?09:24
spivNot exactly elegant, but effective...09:24
spivWell, just anyone else trying to use the mirror of your archive.09:24
SteveAsabdfl: look at http://pqm.ubuntu.com/  is the third thing in the queue from you (launchpad--cverework--0) going to be broken by changes to launchpad.css and bug-headline-tasks.pt ?09:25
spivYou could narrow the scope by just breaking that bit of the mirror, I guess, but that's more slightly fiddly and thus slightly more likely that user error will cause real problems :)09:25
sabdflSteveA: yes09:26
sabdflbut it will be broken anyway09:26
sabdfli sent a prior merge in and it just came back failure09:26
sabdflthough all tests pass here09:26
spivSteveA: Hmm, too late anyway, it's now the head of the queue...09:26
sabdflah fahhhrk09:27
sabdflwe need a new dependency09:27
sabdflhow do i do that?09:27
SteveAas in, from hoary?09:27
sabdfl    import PIL.Image  ImportError: No module named PIL.Image09:27
spivpython-imaging?09:28
SteveAyou need to ask elmo to install python-imaging09:28
SteveAfor python 2.409:28
sabdflon which machines?09:28
SteveAon all the machines we use09:28
sabdflelmo: ping09:28
sabdflZnarl: ping09:28
SteveAso, chinstrap, macquarie (perhaps), gangotri, mawson09:28
SteveAdunno about any others09:28
sabdflSteveA: could you kill your merge please?09:28
sabdfli want to land this damn branch of mine09:28
sabdfli've spent an hour on it already this morning, and don't have much time today before leaving09:29
SteveAalso, it needs to be announced on the list, and docs updating09:29
stubchinstrap, gangotri, asuka, macquarie, mawson, possibly macaroni and emperor and the importd boxes depending on if they need to import modules that depend on it. 09:29
SteveAsabdfl: i moved my archive to a different name09:29
sabdflSteveA: you're a star. thanks very much.09:29
SteveAsabdfl: dunno if it screwed pqm's merge in time though09:29
spivsabdfl: No-one without pqm privs would be able to stop the current merge, unless Steve got lucky and moved it in time.09:29
sabdfllifeless: ping09:29
sabdfl^^09:29
stubI've killed it (by killing PostgreSQL, all the tests will fail)09:30
sabdflmy kingdome for a revision control system that does a status in less than 8 minutes09:30
sabdflthanks stub09:31
spivHmm, it was still doing build-config, so SteveA was probably fast enough.09:31
spivs/was/is/09:31
SteveAbe nice to have some pqm admin controls09:32
SteveAlike, changing the queue order09:32
SteveAcancelling jobs09:32
sabdflor stopping a merge09:32
spivSteveA: I'm sure lifeless would say patches accepted ;)09:32
SteveAonce bzr gets more "out there" i'm sure there will be improvements forthcoming09:32
SteveAschooltool want this stuff09:32
SteveAthey're frustrated with svn and its conception of branches, which isn't so useful in practice09:33
SteveAthe whole distributed concept is good when you have teams across continents09:33
spivstub: you can restart postgres, steve's job is done.09:33
spiv(presumably failed)09:33
SteveAand, they'd like to have tests run pre-commit09:33
SteveAyet not hold up development09:33
SteveAokay09:34
spivThe asynchrony of pqm is interesting... it's both good and bad.09:34
SteveAmy pqm job has finished09:34
SteveAand failed09:34
SteveAi'm moving my archive back09:34
SteveApostgres should be restarted09:34
stubI was going to make sure my merge failed too since I think sabdfls superseeds it09:35
spivGood thing baz gives you a good 5 or so minute window to break things after it starts a job, because of how long the build-config takes ;)09:35
SteveAthe failure message said that it couldn't merge from my mirror, so i was in time09:35
sabdflaiui we're basically on pause till elmo or Znarl can install python-imaging09:46
sabdfli still have a key to the DC..09:47
SteveAsabdfl: you could make it a soft dependency on PIL09:48
SteveAwhat are you using PIL for anyway?  hackergotchi?09:49
SteveAgraphs?09:50
=== dand [n=dand@gw.datagroup.ro] has joined #launchpad
sabdflSteveA: making sure images meet the size requirements09:53
sabdflZnarl is handling the sysadmin stuff, should be done in a sec09:53
SteveAokay.  PIL isn't actually necesarry for that iirc09:54
SteveAzope3 has some image sizing code09:54
SteveAthat just looks at the minimum necessary for the main image types09:54
=== Keybuk [n=scott@217.205.109.249] has joined #launchpad
sabdflSteveA: i'm happy for you to clean that out then09:54
sabdflbut right now, i just want it landed so i can start work on other bits09:55
SteveAokay, sure.09:55
sabdflwell. i got to fix one bug while we wait09:59
sabdflcheck that it's actually an IMAGE before trying to find its size09:59
SteveAtested with a .py file? ;-)10:00
sabdflSteveA: yes, mailnotification.py as it happens ;-)10:02
sabdflfuck. so predictable it must be time for a new girlfriend10:02
SteveAdude, breezy is blode and brunette *at the same time*10:03
sabdflshe's a brunette with a cold?10:05
SteveAwow... just got an email that i'll get my new laptop in the middle of next week10:06
SteveAi've been missing a laptop.  although, working only in the office has a certain discipline10:07
Znarlsabdfl : Added python-imaging.10:07
SteveAZnarl: to which machines?10:07
sabdflZnarl: much obliged, thanks10:07
Znarlmacquarie, asuka, gangotri, chinstrap and10:07
sabdflconfirm chinstrap?10:07
Znarl... and mawson.10:07
SteveAstevea@chinstrap:~$ python10:08
SteveAPython 2.4.1 (#2, Mar 30 2005, 21:51:10)10:08
SteveA[GCC 3.3.5 (Debian 1:3.3.5-8ubuntu2)]  on linux210:08
SteveAType "help", "copyright", "credits" or "license" for more information.10:08
SteveA>>> import PIL10:08
SteveA>>>10:08
SteveAcool10:08
SteveAi'll mail the launchpad list about the new dependency and update the docs.10:10
sabdflthanks stevea10:10
=== niemeyer [n=niemeyer@217.205.109.249] has joined #launchpad
sabdflstub: there's a new cronscript, can be daily10:14
sabdflcomes with my landing10:14
sabdflcronscripts/update-cve.py10:14
stubI'll turn it on staging and see what happens10:14
sabdfli don't think there's any point in running it more often than daily, i think the db it fetches is a daily update10:14
lifelessPIL ?10:33
SteveApithon imaging library10:35
SteveAbut speled rite10:35
SteveArather than john lydon's band10:36
lifelessheh10:42
dilysMerge to rocketfuel@canonical.com/launchpad--production--1.33: Cherry pick patch-2416 into production (patch-1: stuart.bishop@canonical.com, rocketfuel@canonical.com)10:50
SteveAniemeyer: hi.  had another case, in schooltool this time, of why that list() calls __len__ and swallows all exceptions thing was bad.  This time, it was a __len__ implementation that used list(self) and so got a max recursion error.10:58
SteveAso, major slowdown, and really hard to track down.10:58
bob2hah11:01
niemeyerSteveA: Hey11:02
niemeyerSteveA: Yes, that's really bad indeed :/11:02
SteveAat least it's been fixed now11:03
SteveAi haven't looked at the C code that fixes it11:03
SteveAbut from the schooltool report, i'm pretty sure rhettinger made it catch only specific errors like AttributeError and TypeError11:04
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== ddaa [n=ddaa@marvin.xlii.org] has joined #launchpad
niemeyerSteveA: 11:06
niemeyer                if (!PyErr_ExceptionMatches(PyExc_TypeError)  &&11:06
niemeyer                    !PyErr_ExceptionMatches(PyExc_AttributeError)) {11:06
niemeyer                        Py_DECREF(it);11:06
niemeyer                        return NULL;11:06
niemeyer                }11:06
SteveAgreat11:06
SteveAthanks gustavo11:06
sabdfllifeless: could you get bzr using LP for bugs, specs, support tickets please?11:07
=== ddaa [n=ddaa@marvin.xlii.org] has left #launchpad []
lifelesssabdfl: sure thing. I'll ring martin today and ensure one of us will set it up on monday11:08
sabdfllifeless: what's PQM doing baz merge --two-way scott@canonical.com--2005/launchpad--manifest-hints-and-ancestry--0 for?11:11
lifelesssabdfl: because I'm doing last nights merge again ?11:12
sabdfldo you know that i'm trying to land mark.shuttleworth@canonical.com/launchpad--cverework--1?11:12
sabdflbefore i can get cracking on the work i owe daniel?11:13
lifelesssabdfl: I didn't, sorry.11:13
sabdflok. could you let me merge drop in please?11:13
lifelesssabdfl: I'll reenable pqm for a bit. can you tell me when I'm clear to kill it again11:13
sabdflsure11:13
lifelessok, pqm cron is up again11:13
=== WaterSevenUb [n=WaterSev@bl5-48-106.dsl.telepac.pt] has joined #launchpad
sabdflpqm web site still giving bad gateway, lifeless11:15
lifeless1 sec11:15
sabdflfixed, thanks11:16
lifelessnp11:17
lifelessping me when I'm clear, or if you have trouble11:17
SteveAsabdfl: just seen the trickery in the TicketTargetView class.  eeewwwww :-)11:18
=== ddaa [n=ddaa@marvin.xlii.org] has joined #launchpad
lifelessI need a favico.ico for pqm11:28
Keybuksw11:28
Keybuklifeless: 11:30
lifelessgot that as an ico ?11:30
Keybukor is it  ?  can never remember which way it goes11:30
sabdflSteveA: i have a note to discuss it in UBZ11:33
SteveAyeah.  i added some notes to the note on my branch.11:33
sabdflwe need a general approach to "I want a page with a specific set of criteria against a general list"11:33
sabdflat least all the hackery is in one place11:33
SteveAthe standard way to do this in zope3 today is to use a base class TicketTargetView, and some tiny subclasses, each implementing self.getTickets() separately.11:34
SteveAthen register in zcml the appropriate subclass.11:34
sabdflthat would work too11:35
SteveAanother approach would be to have a way of giving "advice" of some sort from zcml11:35
sabdfltoo limiting11:35
stubI've got a ruber ducky going cheap11:35
stubrubber even11:35
SteveAso you'd say <page .... subsetoftickets="created_tickets" />11:35
sabdflyou've no sooner implemented all the "advice" zcml, when you need to do something totally different11:35
SteveAand the view class can read that11:35
sabdfli prefer the subclasses11:36
SteveAit is more pythonic, to be sure11:36
SteveAto use subclasses11:36
sabdflactually, i prefer to have it all in one view class, but we can discuss it at UBZ11:36
SteveAsure11:36
sabdfli don't think looking at the URL is evil11:36
sabdflit's just looking at the request, right?11:36
SteveAwell... i've added some notes about looking at the request in a more robust way11:37
sabdflthe current implementation is simplistic, but i think we can improve that11:37
sabdflright11:37
sabdflhow... long... must... i... wait... for.... the.... failure.... message....11:37
SteveAbut the issue is, if i want to rename a page from say +createdtickets to +ticketscreated11:37
SteveAi need to do it in two places -- the zcml and the view class11:37
SteveAwhereas, i should be able to do it just in zcml11:37
sabdfldisagree11:37
SteveAyou disagree that i need to do it in two places?11:38
SteveAor that i should be able to do it in just zcml?11:38
sabdflif you want to rename a field (statusexplanation -> whiteboard) you have to do it in 4 places11:38
SteveAi don't see that fields are the same as pages.  in general, needing to rename things in N-1 places is better than needing to rename it in N places11:39
SteveAwith one being the absolutely best case11:39
SteveAalso, all other pages in the system can be renamed just in zcml.11:39
SteveAin zope 3 in the future, you won't list templates in zcml.  they will be included in the view classes anyway.11:40
SteveAso at that point, you can have one view class11:41
bob2formlib!11:41
SteveAwith 4 (or however many) methods11:41
SteveAone for each of the variants of the pages -- as the entry point11:41
SteveAactually, that would work now also, although be a bit less elegant11:41
SteveAbut it would avoid the "programming complexity" and spready-out-ness of subclasses11:41
SteveAand achieve the goals of having one view class, an obvious flow of control, and needing to change the page name only in zcml.11:42
sabdfli think you could fake this now11:43
sabdflif you put a hook in the template11:43
sabdflthat runs a view method at the beginning of the page11:43
sabdfland sets up the iterators appropriately11:43
sabdflwe need this stuff for:11:44
sabdfl - lists of bugs (open, closed, search criteria, show rejected, other options)11:44
sabdfl - lists of specs and tickets and branches11:44
sabdfllots of places11:44
=== Kinnison returns having fl-cow'd his desktop
sabdflso we need to develop a pattern, and re-use it11:44
SteveAcan you point me at another view class that needs this stuff?  it will help me see the pattern11:45
SteveAthis hook you speak of is something i'm on the hook to do for upstream zope3 sometime.  haven't got around to it yet.  it also fixes the "don't do stuff in __init__" issues.11:46
SteveAin fact, i wrote in the comment:11:46
SteveA            #   Either wait for the Zope 3 improvement I'm on the hook to11:46
SteveA            #   land that makes templates called "template" in view classes,11:46
SteveA            #   or include it manually like Zope 3 will do in the future.11:46
SteveA            #   Then, have different methods as entry-points for the different11:46
SteveA            #   pages.11:46
SteveA            #     self.createdtickets()11:46
SteveA            #     self.assignedtickets()11:46
SteveA            #     self.answeredtickets()11:47
SteveA            #     self.subscribedtickets()11:47
SteveA            #     self.tickets()  # everything else.11:47
SteveA            #   Hook these up in zcml.11:47
SteveA            #   using the class and attribute style of registing pages.11:47
sabdflSteveA: the bug search stuff is a bit of a mess, so don't look there for inspiration11:48
SteveAokay.11:48
sabdflbut imagine all the places where you want to slice a set of data up 20 different ways11:48
SteveAyeah11:49
SteveAokay.  i'll leave the comment there for now, and mull it over the weekend11:49
sabdflSteveA: i have often used 1 view class with different templates, so don't make the binding at a view class level please11:49
SteveAthere's nothing in this that would stop that from working11:50
SteveAthe upstream change is that when you have 'template="..."' in zcml, the template ends up in a standard attribute name 'template'11:50
SteveAyou can use one view class with many template="..." directives, because a new class is generated for each use.11:51
SteveAit has to be that way to work currently.11:51
SteveAone of the reasons to deprecate template="..." in zcml, and make the standard way to do things to explicitly name templates in the view classes11:52
SteveAis to get rid of the "let's generate classes" stuff11:52
SteveAbeacuse it makes the system more obscure if zcml is going around generating classes for you11:52
sabdflah. +1 on that front, then11:52
SteveAi mean, it's cute... but in a "i want a rifle for bambi" way11:53
sabdflsrichter?11:54
SteveAnah, this was in zope3 from the very earliest time11:54
SteveAthe motivation at the very start was to make it the minimum lines of code to write a page template, and some in-python-code logic to help it do its job 11:55
SteveAbecause many people wanted the same power as having islands of python code in page templates11:55
SteveAbut others wanted the python code to be testable, and to look like regular python code11:55
SteveAout of that came the view class + template idiom11:55
SteveAand the desire to avoid the boilerplate that is so common in zope 211:56
SteveAso, generating classes was one way to avoid the boilerplate of using python code to explicitly include a page template, and to avoid the need to say def __init__(self, context, request): self.context = context; self.request = request11:57
SteveAit achieved the goal of avoiding boilerplate and making it easy to add bits of python code to templates, but at the cost of making the whole "views" system obscure when it didn't need to be so.11:57
SteveAnow it is time to undo that, and make things less magical.11:58
=== SteveA concludes today's zope3 history lesson
sabdflso, will the boilerplate become required?12:01
SteveAyou'll need to provide an __init__ for your view classes, or subclass something that provides a suitable __init__12:01
SteveAso, in launchpad, we'd end up subclassing LaunchpadView all the time, i expect12:02
SteveAwhich is nice and self-documenting anyway12:02
SteveAclass TicketTrackerView(LaunchpadView):12:02
SteveA ...12:02
SteveAso, no particular boilerplate12:03
SteveAbut still understandable -- you look up what LaunchpadView is if you want to see its __init__12:03
SteveArather than have to look in the dark corners of zcml implementation12:03
SteveAstub: did you ever track down the issues with sending mail async in production?12:05
stubI have not looked - it may well work if I turn it back on.12:06
SteveAdid it fail on staging also?12:06
SteveAit would be nice to be able to test it on staging first.12:06
stubStaging doesn't send email yet, so that needs to be sorted first12:07
SteveAi think sending it all to some list would be nice12:07
SteveAthen people can look at the list to see what staging is sending out12:07
KinnisonIf anyone here wants me to retain the contents of the dogfood database (soyuz tables in particular) speak now, or forever hold your peace12:14
Keybukone hopes people will be holding their piece in private12:18
Kinnisonwude.12:18
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=jamesh]  rework cve structure, and general polish (patch-2420: mark.shuttleworth@canonical.com, bjorn.tillenius@canonical.com)12:21
bob2filthmongers!12:22
Kinnisongah, talk about confusion12:23
KinnisonI got an arch-pqm success message from *MY* pqm, sat next to a change-to-rocketfuel tla mail-revisions mail12:23
Kinnisonmy brain froze for about 10 seconds due to the cognitive dissonance12:23
ddaaSteveA: okay, I'm looking at the pybaz problem right now.12:24
SteveAthanks ddaa 12:24
=== JanC [n=janc@lugwv/member/JanC] has joined #launchpad
SteveAstub: when you update the request type in servers.py, you also need to update it in canonical/functional.py12:27
sabdfllifeless: k, i'm done. phew. thanks12:28
stubThere a mock request in there or something? ok.12:28
sabdfljamesh: please grill me about those extra review points (schemas vs interfaces) in UBZ, ok?12:28
sabdflstub: did that branch land ok on your production branch, too?12:29
lifelesssabdfl: thanks12:29
lifelesspqm -> down12:29
stubits in pqm. If it doesn't land, is it ok to cherry pick that one instead?12:29
lifelessstub: your merge will complete12:30
lifelessthen I'm locking the queue to fight this hct patch12:30
sabdflstub: 2420? yes. though it might include a little more rocketfuel than the previous one. there's one little extra fix too, but not critical12:30
stubok.12:31
SteveAsabdfl: when do you leave today?  bjorn's working on a ticket tracker email interface braindump for you to look at? 12:33
SteveAstub: actually, i see you changed servers.py already.12:34
SteveAstub: i'll update functional.py because i'm making an improvement there anyway.12:35
sabdflSteveA: 6pm uk time12:35
SteveAstub: ... or did you.  i'm getting confused between Publication and Requests :-/12:36
SteveAthanks sabdfl 12:36
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== SteveA improves pagetest error output to get rid of HTML guff and present a nice traceback.
lifelessyay01:00
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== WaterSevenUb [n=WaterSev@bl5-41-98.dsl.telepac.pt] has joined #launchpad
SteveAlifeless: if i have some changes i've made but not committed on the wrong branch, can i baz switch, baz branch, baz commit?01:03
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
lifelessyes01:05
SteveAgreat01:05
lifelessstuping01:08
lifelessdoh01:08
lifelessI wanted stub01:08
=== Kinnison lunches
lifelessspiv: meet niemeyer 01:18
lifelessniemeyer: meet spiv01:18
niemeyer:)01:18
niemeyerspiv: Hello :)01:18
lifelessspiv: niemeyer might like to learn some more about twisted, to get a feel for what writing good twisted code is like01:18
niemeyer"good twisted code" 8)01:19
spivHow to freeze Python in a futex beyond the reach of Ctrl-C in one easy step: python -c "from threading import Event; Event().wait()"01:20
bob2I guess I have only myself to blame for that terminal being fucked now01:21
spivniemeyer: G'day01:21
SteveAspiv: normal kill killed it01:21
Keybukbob2: How to freeze your system by making it do lots of disk I/O: sudo rm -rf /01:21
lifeless...01:21
bob2Keybuk: monkey boy.01:22
Keybukbob2: get a haircut.01:22
spivSteveA: Yeah, so does Ctrl-\  (SIGQUIT).01:22
spivStill surprised me.01:22
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== stub [n=stub@203-214-4-72.dyn.iinet.net.au] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
KeybukKinnison: found another amusing point where my Cymraeg slips out for you ... I pronounce the C type "char" with a hard ch, and not a soft one01:36
=== Seveaz [n=seveas@seveas.demon.nl] has joined #launchpad
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
=== RapSinna [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
BjornTsabdfl: could you have a quick look at https://wiki.launchpad.canonical.com/TicketTrackerEmailInterface01:56
BjornTsabdfl: i will improve it later after lunch01:56
=== BjornT -> lunch
sabdflBjornT: i'll make some tweaks while you are out01:58
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
=== sabdfl [n=mark@pdpc/supporter/silver/sabdfl] has left #launchpad []
=== camilotelles [n=Camilo@20132139198.user.veloxzone.com.br] has joined #launchpad
=== ddaa [n=ddaa@ordo.xlii.org] has joined #launchpad
mptagh, I thought I'd be smart and leave baz merge running overnight02:40
=== asmodai [n=asmodai@sakko.demon.nl] has joined #Launchpad
mptbut it ran out of memory02:40
Nafallohehe02:41
Nafallo:-)02:41
mptServes me right for having only 1 GB02:41
=== mpt [n=mpt@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
ddaampt: I have only 1GB too03:14
ddaaand baz never runs out of mem here...03:14
ddaabut tend to use --star-merge...03:15
kikoddaa, but you have swap03:17
ddaaOh yes03:17
kikoand mpt in particular uses gnome with two browsers, gaim, etc03:17
kikoso there's only about 600-700mb for baz03:17
ddaa953MB of swap03:17
=== sabdfl [n=mark@host217-37-231-22.in-addr.btopenworld.com] has joined #launchpad
ddaaand I'm using gnome, with firefox, evolution, gaim, emacs, etc.03:18
ddaaI'm not sure exactly what is the root cause of the problem, but I found that when resident usage gets too high, I can free much ram by switching with R&R to 800x600 then back to 1600x120003:19
ddaaquitting evolution and firefox from time to time (about once a week) also helps.03:20
sabdflBjornT: comments submitted03:32
mptddaa: Does /products/foo/+branches show branches for any products yet? (it seems to be empty on the ones I try, though I don't know if that's just because the products aren't using it)03:41
mptoh, nm, the file does nothing03:41
ddaanot yet, the work we did on that during the sprint is blocked by a couple of big-urgent-really-I-mean-urgent tasks.03:42
mptok03:42
ddaathat is, samba import and fixing importd so the BranchDataStorage db schema changes won't break RCS imports.03:42
=== Lathiat [i=lathiat@gasp.bur.st] has joined #launchpad
LathiatJust wanted to stop by and say that malone is looking pretty rocking :)03:43
KinnisonKeybuk: aye, I know you do. We argued back and forth about the pronounciation of 'char' a few weeks ago IIRC at the BBQ03:45
bradbLathiat: glad to hear. feedback is always welcome.03:46
Lathiatits quite nice having things like cve references, separate tracking for hoary+breezy in the same bug etc03:46
mptKinnison: Depends whether you're talking about characters or about the BBQ :-)03:47
Kinnisonmpt: I always pronounce it with a soft ch03:47
kikoLathiat, thanks, we appreciate the feedback -- hard work it is03:47
Lathiatim sure :) 03:48
Lathiat1 question, can i bring up a more advanced bug search page?03:49
Lathiate.g. new and assigned to MOTU03:49
mptLathiat: No, but if you go to https://launchpad.net/people/motu/+assignedbugs and click the "Status" column header twice, all the New bugs will be listed first03:50
Lathiatmpt: ok, will that be added in future?03:51
bradbLathiat: fwiw, novemberish is when we'll be diving into making searching in Malone rock.03:51
Lathiatbradb: ok cool03:51
kikoLathiat, you can click on the (Advanced) button, does that help?03:51
KeybukKinnison: ah, it was you I was arguing with then :p03:51
Lathiathmm, when sorting by "severity" it should probably sort by severity type rather than alphabetically :)03:52
bradbLathiat: i'll file a bug on that, thanks03:52
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
Lathiatbradb: cool03:54
bradbLathiat: filed #2347 and subscribed you as well03:58
Lathiatbrcheers03:58
Lathiatbradb: 03:58
mptkiko, what's the shipit admin page url?03:59
kikompt, it's only available to shipit admins03:59
kikowhat should I do?03:59
mptI mean, on localhost :-)03:59
mptoh, I don't know who's the admin in the sampledata anyway04:00
=== ctrlsoft [n=jelmer@dsl16-123.fastxdsl.nl] has joined #launchpad
mptoh yes I do04:01
mptOk, I'll shut up now04:01
ctrlsofthi04:01
ctrlsoftAnyway to merge two user accounts?04:01
Kinnisonlifeless: any word on the HCT branches? Have you locked PQM or is it all still running?04:02
lifelessKinnison: pqm is still getting its knickers cleaned04:02
Kinnisonmmm laundry day04:03
kikompt :)04:07
SteveAhi mpt 04:09
bradbno pqm still eh? hrmph.04:09
Lathiatpqm?04:11
SteveALathiat: just a sec, i'll get you a description04:12
SteveAhttp://mail.python.org/pipermail/python-dev/2005-August/055376.html04:12
SteveAit's a bit long, but explains about pqm04:12
Lathiatthanks04:13
bradbLathiat: you don't wanna know :)04:14
kikogood man bradb 04:15
=== zyga [n=zyga@2-mi2-1.acn.waw.pl] has joined #launchpad
zygahello04:17
LathiatSteveA: good read, thanks04:18
ddaalifeless: [samba]  just put up something that I think might actually be correct. Would you have the time to review it today?04:22
ddaa(not tested yet, though)04:23
lifelessddaa: sure. is there a diff yet ?04:23
ddaaI'll send you the branch name as soon as it's gone through the test suite and committed.04:24
mpthi SteveA04:26
lifelessddaa: would like a diff please04:26
ddaaok04:26
mdkejordi, around?04:29
ddaalifeless: diff in the pipe04:33
bradbBjornT: hi. any news on the URL changes review?04:34
KinnisonOkay, since noone has complained, I'm going to blat at the dogfood DB for a while. dogfood will be offline for the duration.04:35
elmostuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuuub04:36
=== Kinnison tickles elmo
=== Keybuk wonders whether it's time for an IRC client plugin that recognises your nickname with repeated letters
BjornTbradb: well, i've come a long way,  but i postponed it temporarily, since i wanted to create a braindump of the ticket tracker email system for mark to look at04:37
BjornTbradb: it's not impossible that i still finish the review today, though04:38
bradbBjornT: ok, I understand that it's a big review, and not the easiest one to do at that. good to hear that it's coming along. thanks.04:38
kikoKeybuk, it wouldn't help in this case, though.04:38
Lathiatif i want to mark something as rejected in warty,hoary and leave it as new in breezy, would i first "request a fix in a distro" so that item is created and then change its status?04:41
Kinnisonelmo: When does mawson's copy of the archive get updated?04:41
elmo4am04:42
kikoLathiat, right.04:42
Kinnisonelmo: is that when it starts, or when it finishes?04:42
elmostarts04:42
bradbLathiat: are there already warty and hoary tasks?04:42
Kinnisonelmo: cool, so 6am will do for starting the import from it04:43
Kinnisonthanks04:43
elmoKinnison: easily eyah04:43
Kinnisoncoolio.04:43
=== Kinnison hammers mawson. (s'not hard)
bradbLathiat: or, in human terms, i should have ask that as "is the bug already reported in warty and hoary?"04:43
lifelessplease merge, kthnxbye04:44
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: r=bjornt merge support for hints and ancestry of manifests. (patch-2421: scott@canonical.com)04:44
Kinnisonlifeless: I.E. you're ready for PQM to be used again?04:44
dilysMerge to rocketfuel@canonical.com/hct--devel--1: magic marker upgrade from scotts hct branch (patch-20: scott@canonical.com)04:44
dilysMerge to rocketfuel@canonical.com/sourcerer--devel--0: magic marker upgrade from scotts sourcerer branch (patch-30: scott@canonical.com)04:44
Lathiathrm i cant see a disctinction between distro versions 04:44
Lathiatyet i see it on lots on bugs04:44
bradbLathiat: URL?04:45
Lathiathttps://launchpad.net/malone/bugs/205704:45
bradbLathiat: so, does bug #2057 appear to you to be filed in warty and hoary?04:45
Lathiatbradb: just "Ubuntu", i want to split out the status to say rejected for warty and hoary, but leave breezy as new04:46
Lathiathow would i do that?04:46
Lathiat(is that a sane thing to do?)04:46
bradbLathiat: ok, so, you want to target the fix to those releases and then reject the tasks04:46
Lathiatohh i see 04:46
=== mpt doesn't see how to ask for it to be fixed in a distro release in the first place
kikohttps://launchpad.net/distros/ubuntu/+bugs/2057/+target04:47
bradbmpt: If i were allowed to make the actions menu useful, you sure would.04:47
kikompt, you can't (at least currently)04:47
Lathiatmpt: "Request fix: ... in distribution"04:47
mptLathiat: That's what I thought too04:47
Lathiatand then theres "target release"04:48
mptah, "Target Fix to Releases"04:48
bradbmpt: I'd put that on the bug page, so you'd have noticed it, but I'm not allowed to do that.04:48
bradbSame with Edit Assignee/Status Details04:48
bradbAt the moment, assigning a bug is not only hard to discover, it's practically an easter egg.04:50
BjornTlifeless: can you please add a malone section to the production config? (just copy the section from the default config)04:51
lifelessBjornT: ?04:52
ddaarubber duck egg04:52
mptbradb: bah, who needs to assign bugs anyway :-)04:53
bradbBjornT: There already is a malone config section, no?04:53
BjornTlifeless: sorry, by config i mean launchpad.conf file04:53
bradbBjornT: At least, it's being used by the bugmail error sending code04:53
BjornTbradb: doesn't seem like it04:53
BjornT    return config.malone.bugmail_error_from_address04:53
BjornTAttributeError: 'NoneType' object has no attribute 'bugmail_error_from_address'04:53
bradbBjornT: ah, ok, i thought that's what might have tipped you off to asking for it to be added to the config file :)04:54
bradbLathiat: btw, you might not actually need to target-then-reject04:56
Lathiatbradb: oh?04:58
bradbLathiat: by default, an "Ubuntu" bug means that it should be "fixed in the current (development) release". to signal a backport fix request, you use the "Target Fix to Releases" page04:58
Lathiatbradb: oh ok04:58
bradbwe'll have to think a bit about how to make clearer what a bug filed on "Ubuntu acroread" actually means, perhaps04:59
Lathiatbradb: altho in this case, the reporter mentioned it was vulnerable in all three05:00
Lathiatbradb: (but didnt manually click the other two05:00
Lathiatbradb: also having breezy + "ubuntu" is confusing05:00
Lathiatand also annoying05:00
Lathiathowever thats not the easiest thing in the world to "transition" when we release, yo udont really want to just mark all bugs as "breezy" at release 05:01
bradbLathiat: do you mean that it's confusing to have the bug reported on both "Ubuntu" and on "Ubuntu Breezy"?05:02
Lathiatbradb: well that too05:02
=== kiko gets spammed
lifelessBjornT: done05:02
bradbLathiat: i agree, it's confusing05:02
lifelessI think there is a general issue.05:02
bradbLathiat: what did you mean by 'also having breezy + "ubuntu" is confusing', exactly?05:03
BjornTlifeless: thanks05:03
lifelesswhen we have sane defaults, why do we not have a default value if they are not present ?05:03
lifelesswe've had this sort of fuckage before05:03
Keybuk. o O { why are Bugs on Product and not ProductSeries }05:03
Lathiatbradb: i meant by that exactly what you just said05:04
kikoKeybuk, they will be. mpt, why are you waiting to land deactionizing?05:04
kikoKeybuk, they will be targettable to series, actually.05:04
kikolifeless?05:04
kikoah, right05:04
bradbKeybuk: from the backend implementation details, that's an easy one to explain. from a UI perspective, the difference between product vs. product series reports is communicated poorly, it would seem05:04
kikothanks BjornT, I was getting spammed like crazy.05:04
mptkiko: I've been trying to land it for the past half hour, I'm not waiting :-)05:05
kikowow05:05
bradbLathiat: ok05:05
Keybukbradb: yes, it would be a first for Malone to be really confusing ui wouldn't it O:-)05:05
=== Lathiat grins at Keybuk
Lathiatim impressed how well its doing now, some of the stuff is rather difficult to expose05:05
BjornTlifeless: would that have helped, though? the value was required, but since the section didn't exist, there was no error. no section, no default value. the malone section could get merged into the launchpad one, though.05:06
KeybukWHY CAN'T I FIX THE BUG FROM THE SCREEN WITH ALL THE COMMENTS ON IT, AND WHY CAN'T I VIEW OTHER BUGS IN THE SAME THING FROM THE SAME SCREEN. GNARGH!05:06
bradbKeybuk: Malone's UI is confusing, but improving.05:06
mptKeybuk: +105:06
lifelessBjornT: having a default and making the rest optional would definately fix it05:07
bradbKeybuk: I added a link to allow you do the latter in my URL changes branch (currently being reviewed by bjornt), and haven't yet removed it, but I think sabdfl may want me to, when he sees it.05:07
Keybukright now the work flow looks like:05:07
=== test [n=test@203.131.147.242] has joined #launchpad
=== asmodai [n=asmodai@sakko.demon.nl] has joined #Launchpad
bradbKeybuk: I also added a link to the bug page to at least make it easy to get to the *screen* where you can "Fix" the bug, but sabdfl told me to get rid of that too, so I did.05:08
test@find pdf05:08
Keybuk1) find bug, 2) read bug, 3) hunt randomly and scream until some helpful person points at the "upstream <product>" link which oh-so-unobviously takes you to the right screen, 4) fix the bug, 5) go back to the task page, 6) add a comment, 7) go back to the bug page, 8) click "other bugs in <product>"05:09
BjornTlifeless: but in this case a default value wouldn't have helped, since the section didn't exist. i wonder if it's possible to make a section required...05:09
bradbKeybuk: we want what you want, dude. "upstream <product>" is, practically speaking, *impossible* to figure out to be the place where one goes to fix the bug.05:10
lifelessBjornT: garh. MAKE THE SECTION OPTIONAL.05:10
bradbKeybuk: people hate this. people repeatedly complain about it. people should complain about it. i fixed the problem. i had to unfix it yesterday. :/05:11
kikoKeybuk, like a broken arm, you get used to it after six months05:13
mptKeybuk: step (3) didn't used to be necessary, but that was reversed a few months ago too05:13
BjornTlifeless: if you tell me how to do it in the config system, sure.05:13
lifelessBjornT: I wasn't saying *you* should do it, rather that *we* have a general issue05:14
BjornTlifeless: ah, that is true :)05:14
kikolifeless, BjornT: the other option is making a mandatory test that validates the config file...05:15
lifelesskiko: wouldn't help05:15
bradbfwiw, i added that config option to malone05:15
kikolifeless? how now?05:15
bradbi expected that adding a new config option would Just Work05:15
lifelesskiko: unless you meanaas a startup-of-launchpad thing05:15
Keybukkiko: no, you don't05:15
lifelesskiko: tests are not run on the production *servers*05:15
Keybukkiko: people don't hang around 6 months to get used to a bug tracking system05:16
Keybukthey take 15 seconds to install bugzilla, which they know05:16
lifelessbradb: you *HAVE* to tell stub and I about new config entries.05:16
Kinnison19697 postgres  25   0  110m  97m  86m R 98.5  2.7  11:40.43 postmaster05:16
Kinnisonyay for postgres05:16
dilysMerge to rocketfuel@canonical.com/pybaz--devel--0: [trivial]  disable OrderedTestLoader (patch-40: ddaa@ddaa.net)05:16
kikolifeless, perhaps an email to launchpad and a poke in the eye to reviewers?05:16
kikosladen, https://launchpad.net/malone/bugs/224905:16
BjornTkiko: there is some validation done when the config is parsed, but it seems that no sections are required, so it doesn't complain if one section is missing05:16
kikoright05:16
kikothat's what I meant05:17
bradblifeless: right, sorry, i will next time i make a change05:17
BjornTthere should be some way of making it required05:17
kikoyeah, agreed -- the instance should bomb out if missing05:17
lifelesskiko: either vaidation during startup or an email to lp + a poke to stub and I05:18
kikolifeless, I meant poking launchpad to say "If you add config sections, tell me or kiko will kill you" and then poking reviewers to watch out for it05:18
lifelesskiko: heh05:19
mptKeybuk: BoF it for UBZ05:26
Keybukmpt: no, I want to live05:27
=== rbelem is now known as rbelem-afk
=== cprov is looking for the gun, suicide makes lot of sense
cprov********************************************************05:30
cprov*  51 conflicted items in this tree. Please            *05:30
cprov* resolve each conflict with "baz resolved 'filename'" *05:30
cprov********************************************************05:30
bradbcprov: those can't be all real (...can they?)05:30
=== ddaa think people should do something about their workflow
cprovbradb: even if they are duplicated code, no way to recover the tree, will try with --star-merge 05:31
dilysMerge to rocketfuel@canonical.com/cscvs--devel--1.0: [trivial]  remove uses of pybaz OrderedTestLoader (patch-110: david.allouche@canonical.com)05:32
ddaaI think experience shows that mesh-merge does not work satisfactorily, it's really worth the trouble to try avoiding situation that cannot be handled with star-merge and diff3...05:32
bradbI always use --star-merge these days.05:36
ddaaI mean something a bit different: avoiding workflow and merge topologies that cannot be handled with star-merge.05:37
cprovbradb: yeah, i suspect this is the problem, once you rely on --star-merge you can't go back to simple merge 05:39
ddaacprov?05:39
cprovddaa: sorry ?05:41
Kinnisoncprov: If you're having problems even with star-merge, let me know and I'll have a look05:41
ddaaYou seem to imply that star-merge supercedes mesh-merge, but it's (supposedly) the other way around.05:41
ddaastar-merge is more limited (and more predictible) and forces you to be more disciplined, which does not prevent using mesh-merge.05:42
cprovKinnison: not really star-merge works, but still resulting in wierd failures on merge05:42
KinnisonNever use mesh merge with launchpad05:43
Kinnisonit's too risky05:43
ddaacprov: yup, but when you know the limitation it's often possible to avoid them.05:43
BjornTlifeless: btw. you don't happen to know where i can find a version of libgetopt++, that can be used to compile config-manager with?05:43
Kinnison(IME)05:43
=== cprov needs to go, the tree is recovered, soon I'll try PQM again
lifelessBjornT: apt-get source config-manager05:49
lifelessBjornT: or from my barch archive05:49
SteveAmpt: ping05:50
mptSteveA: pong05:52
sabdflKinnison: ping05:52
SteveAmpt: i want a  div traceback  style to use that puts the text in a smallish monospaced font, and makes the block of traceback distinct from the rest of a standard launchpad page05:52
SteveAthis is to improve the debug views on errors05:52
sabdflSteveA: class="highlight" or class="highlighted" will give you boxed, slightly coloured background05:54
sabdflyou could add the font style, and size, directly05:54
mptSteveA: Does it have its own <div>s?05:54
SteveAyes, it is in a <div class="traceback">05:54
mptSteveA: I mean for individual lines05:54
SteveAi can alter the class etc.05:54
mptor is it expecting a <pre>05:54
SteveAoh, it's got a <ul> with a <li> for each line05:55
BjornTlifeless: thanks, compiled fine now05:55
SteveAall inside a <p>05:55
sabdflBjornT: see those comments?05:55
SteveAso, <div class="traceback"><p>some text here <ul><li>line1</li><li>line2</li></ul>more text</p></div>05:56
mptSteveA: You're allowed <p> inside <li>, but you're not allowed <ul> inside <p> (yet)05:56
sabdflmain thing is the closing procedure (not just a simple set-to-closed), and then that i don't think we should allow opening tickets by mail yet, or that we should have a better system than "affects /foo/bar" if we do05:56
SteveAmpt: i don't have control over that part of the rendering05:56
mptSteveA: ok, so if this isn't going to be used anywhere else, <div class="highlight" style="font-family: monospace; font-size: smaller;">05:56
SteveAend users will never see this.  this is for developers05:57
SteveAok05:57
BjornTsabdfl: yes. i'll adjust how the status command should work, and will remove creation of tickets. i agree that it's probably not worth creating new tickets via email.05:57
Lathiatanother thing thats a little confusing is that the tabs in the top right stay the same but they change meaning through different contexts, a good idea in theory but it can be a little confusing and not easy way to get back to the main "bugs" page or whatever05:58
SteveAmpt: ah -- class="highlighted"05:59
BjornTsabdfl: i played around using only the person who created the ticket, so i didn't notice the answered state. i'll make sure that the email system will have the same semantics as the web ui05:59
bradbLathiat: Can you give an example of what the unexpected behaviour was that you experienced, WRT the tabs?05:59
bradbI think I know what you're referring to, but I want to be sure.05:59
mptSteveA: right, sorry (I haven't used that yet myself)05:59
Lathiatbradb: so i goto launchpad.net06:00
Lathiatbradb: and i click bugs, and see things about bugs06:00
Lathiatbradb: i then login, and i click on my name and look at my profile06:00
Lathiatbradb: i think hit bugs, which looks exactly the same06:00
Lathiatbradb: and it takes me to my personal bugs page instead06:00
Lathiatand the only way to get back is the breadcrumb in the top left06:01
Lathiatnifty, but a little confusing06:01
Lathiatthey should at least change colour or something to indicate they are in a different context, and perhaps a button next to the tabs which takes you back up to the top level rather than just the link up in the top left06:01
mptLathiat: yes, we don't do a good job of showing that "the tabs belong to this thing" at the moment06:01
ddaalifeless: can you review the patch?06:01
=== Lathiat nods at mpt
mptLathiat: It used to be better, but not *much* better06:02
ddaait seems to be working well with ubuntu-doc, but it's going so slowly that I do not expect that test to complete before I have to leave.06:02
bradbmpt: Is there currently a plan for what to do about that? (i.e. how to show to what thing the tabs belong?)06:03
mptLathiat: The hierarchy should be better explained when https://wiki.launchpad.canonical.com/LaunchpadHierarchyNavigation, and changing color is part of https://wiki.launchpad.canonical.com/LaunchpadBranding06:03
bradbheh06:03
mptwhen https://wiki.launchpad.canonical.com/LaunchpadHierarchyNavigation is implemented, I mean06:03
Lathiatalso 06:03
Lathiatfor new packages going into breezy06:03
Lathiathow do i get them into launchpad06:03
Lathiat(in my case im upstream, so im interested in a component, and having them assigned to me)06:04
mptLathiat: That should happen automatically, but currently isn't afaik06:04
mptanyway, time for me to go to class06:04
Lathiat"There are currently 1024 products registered in launchpad"06:05
=== Lathiat wonders if thats a real number
mptyes, it's counted from the database06:05
mptI implemented that :-)06:06
Kinnisonsabdfl: pong06:09
Kinnisonsabdfl: sorry, was helping cut laminate flooring06:09
bradbsabdfl: https://launchpad.net/malone/bugs/2353 -- the Specifications tab raises a 404 off the main page.06:10
sabdflKinnison: i'm afraid i'm going to fail to deliver this soyuz rework this evening06:10
sabdflit will be done by the time i get back from SA, but that's 2+ weeks from now06:10
sabdfli think it only really affects the web UI at this point06:11
Kinnisonsabdfl: right06:11
Kinnisonsabdfl: Don't sweat it too much06:11
sabdflbradb: please fix it, i'm packing06:11
bradbsabdfl: ok06:11
SteveAbradb: that's supposed to be fixed in production, and in RF06:11
bradbIt's not fixed in production, at least06:12
SteveAbradb: i'll sort out the spec tab problem -- it's a zcml registration / menus system problem06:12
bradbok06:12
SteveAthen the fix was lost06:12
SteveAit was fixed there before the last update06:12
sabdflbradb: it's fixed in staging06:12
=== bradb reassigns the bug to SteveA
SteveAokay, i'll sort it with stu06:12
sabdflKinnison: anyhow, i;m sorry06:12
Kinnisonsabdfl: Just make sure you don't guilt yourself into doing it instead of relaxing on your holiday06:13
SteveAbradb: what is /malone/bugs/ supposed to do?06:14
sabdflKinnison: ask no questions, i'll tell ya no lies...06:14
SteveAbradb: is it supposed to redirect to /malone ?06:14
SteveAwhat's another user that works in the sample data other than foo.bar@canonical.com ?06:17
=== SteveA sees in sampledata
=== Lathiat -> sleep
bradbSteveA: redirect, yeah06:29
SteveAbradb: thanks.  i was just debugging a problem with infinite redirect recursion on logging out, and was confused by the additional redirect06:29
sabdflSteveA: see pagetests/README.txt near the bttom06:31
bradbSteveA: I'm going to make IBug.id, IBug.private and IBugTask.id always publicly accessible today, btw.06:32
bradbSteveA: oh, and IBugTask.bug, perhaps?06:32
SteveAsabdfl: thanks.  i actually wanted to log in interactively, but found a bunch of users clearly marked in the sampledata06:33
SteveAbradb: the 'id' attributes, sure.  'private', sure, because it makes sense to be able to check if it is private without raising an error06:33
SteveAIBugTask.bug, sure, because you should always be able to go to a less-specific thing, and get that thing's id06:34
bradband be able to do IBugTask.bug.private, etc.06:34
SteveAnow, this is very odd06:35
SteveAthe code for production on gangotri actually has the correct link on the front page for the specs facet06:35
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  hide dups from the assigned bugs report; improve the wording and pagetest (patch-2422: brad.bollenbach@canonical.com)06:37
SteveAlifeless: got a sec?06:37
=== gneuman [n=guest@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
lifelessshoot06:40
SteveAlifeless: so, there's a problem in production.  that is, on the front page, the link on the specs facet goes to '+specs' not 'specs'06:41
SteveAi have looked on gangotri at the code that ought to be responsible for this.  it is correct, and the link is to 'specs'.  i asked stu to change this code a few days ago, and he did.06:41
SteveAso, either, i'm totally wrong about what code to change, or launchpad on production has not been restarted since stu made the change.06:42
lifelessits been restarted06:42
lifelesstwice today06:42
SteveAokay06:43
SteveAit is under revision control.  i'll get the tree, and check it out locally06:43
SteveAta06:43
lifelessddlooks like it might work06:44
lifelessbah, hes gone06:44
=== niemeyer_ [n=niemeyer@217.205.109.249] has joined #launchpad
=== lamont [n=lamont@15.238.5.82] has joined #launchpad
jordimdke: here06:46
SteveAhaha, you can tell what pagetests stu wrote06:47
SteveA  ... Accept-Language: en-au,en-gb;q=0.7,en;q=0.306:47
sabdflcheers guys06:49
sabdfllifeless: great work this sprint06:49
sabdflthanks06:49
sabdflniemeyer: you're off to an excellent start06:50
=== sabdfl [n=mark@pdpc/supporter/silver/sabdfl] has left #launchpad []
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
Kinnison19697 postgres  25   0  110m  97m  86m R 99.9  2.7 121:18.99 postmaster07:07
KinnisonFFS07:07
Kinnisontwo hours it has been deleting rows07:07
Kinnisontwo sodding hours07:07
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=kiko for most, trivial for the rest]  Major cleanup of portlets, minor cleanup of Codes of Conduct and ShipIt; fixes bug 1968 and bug 2103 (patch-2423: mpt@canonical.com)07:14
kikofinally07:15
=== cprov cries, his patch still missing
mdkejordi, we have a string called "translator-credits" that should be completed automatically, rosetta doesn't do it (known bug), do you have any advice to work around it?07:32
jordimdke: completed automatically how?07:33
mdkejordi, by inserting the names and emails of the translators who worked on the po07:34
mdkejordi, bug 11607:34
jordioh, right07:34
jordino, other than filling it by hand, no other ideas.07:34
mdkek07:37
=== rbelem-afk is now known as rbelem
mdkejordi, the other was, some translators have been complaining about having to translate the licences which are in our template because of the use of -e. I mailed danilo asking if there is any way he knows of to exclude such entities but had no response yet, you know anything?07:41
jordinope. You could leave the translation out.07:43
jordixml2po could easily grow an --exclude option.07:43
mdkeyeah that would be cool07:44
bradbSteveA: Given an interface IFoo, with a large number of attributes, is there a simple way to say "protect IFoo with permission Bar, but allow attributes x and y to be publically accessible"? Or do I have to <allow> x and y, and then name all the remaining attributes i want to protect in the <require> directive?07:49
SteveAbradb: no.  it is something i discussed with tres at EP, and have a plan to fix.  but that requires quite a bit of work on the zope component architecture.07:51
bradbok07:51
SteveAonce the rest of launchpad is well organized, and menus / ui issues are under control07:51
SteveAi'll spend some time making the security really rock rather than just get by07:52
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=stevea] . make bug notifictions concerning the same bug be part of the same email thread. (patch-2424: bjorn.tillenius@canonical.com)07:56
=== SnakeBite [n=SnakeBit@84.242.143.64] has joined #launchpad
kikoBjornT, many congratulations08:01
kikorock on08:01
SteveAkiko: cc-ed you on mail to stu and rob about a one line change needed in production.08:05
kikodoh!08:11
SteveAto get the root 'specifications' tab going to the right place08:12
SteveAi could do it myself, with ssh, vim and the restart script08:12
SteveAbut it's kinda rude08:12
kikono tests, tsk tsk.08:13
SteveAwe need link checking tests for that one08:14
SteveAbasically, for every page08:14
SteveA - parse out the facet and menu links08:14
SteveA - check they all work08:14
SteveA(or just do so for all links)08:14
kikowell08:15
kikoI did a scrubbing of pages for links08:15
kikoand then added them to xx-notfound-traversals08:15
kikomanual08:15
kikobut effective08:15
kikothere are many 404s in production, though08:16
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
=== cprov -> lunch, more 2 conflicts for dessert, back in 30 min
=== jbailey_ [n=jbailey@testhaus.cns.utoronto.ca] has joined #launchpad
=== cprov back
=== cprov -> wow, at least my devel branch pass in test, now it goes ;) (after solving the conflicts :()
Kinnison220 minutes of CPU time :-(08:49
=== Kinnison hates databases sometimes
=== rbelem [n=rodrigo@200.246.97.164] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: r=bjornt greatly improved error pages in pagetests and debug port.  introduced new url for a  debug root page.  fixed bug when redirecting for logging out. (patch-2425: steve.alexander@canonical.com)08:49
cprovKinnison: god forbid !08:51
cprovKinnison: it requires, indeed, request PQM twice for that ;)08:52
=== Kinnison grins
Kinnison19697 postgres  25   0  110m  97m  86m R 99.9  2.7 226:56.42 postmaster08:53
Kinnisonsee08:53
Kinnisonyeesh08:53
Kinnisonciao all08:53
cprovKinnison: what is that ? the publisher in action ?08:53
=== Kinnison heads off
Kinnisoncprov: No, that's "DELETE FROM Build;"08:53
=== Kinnison waves
cprovKinnison: shhhhh, good night08:54
=== depoll is now known as dePOLL
spivKinnison: file:///usr/share/doc/postgresql-doc-7.4/html/sql-truncate.html08:57
sivanghmmm, so I see now that using launchapd integration, if someone use "Get help..." he is acutally offered to opne a support ticket, is this what we really want to have there?09:12
sivangI mean, it might be mistaken that way that this is where you report bugs, so I'm thinking we need to also have the report a bug item for lpi in order to distinct the two09:14
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: Implement Distribution.getDistroReleaseAndPocket() for Keybuk. r=sabdfl (patch-2426: daniel.silverstone@canonical.com)09:25
bradbSteveA: Hm, it looks like I might have to wait until after the URL changes land to fix the ID/privacy thing. It's now causing 404's to be returned on private bugs for unauthorized users because of trying to access the relevant IBugTask through privacy-aware search APIs ("privacy-aware", as in, if you don't have access to view that task, it isn't returned by the search). I can think of ways to fix this, but I haven't thought of anyt09:33
kikosivang, well, we need to consider this rather carefully -- most of the end-users we expect won't know how to use a bugtracker!09:44
kikosupport tickets can  be linked to bugs09:45
sivangkiko: so the general idea is the users use the issue tracker, and devels us the bugtracker?09:45
sivangkiko: also, we need to make it easy for a group of users who are subscribed to a specific issues be notified when their underlying bug is fixed, so that they can be protected of the technical deails of the bug, but know when the issue is fixed09:49
bradbsivang: I'd be surprised if that wasn't the default behaviour (i.e. subscribe to issue != subscribe to bug.)09:52
sivangbradb: ah ok =) Sorry, I may be out of sync with the developments of the issue tracker 09:52
bradbsivang: I'm not saying that it actually is/isn't; I haven't looked at it at all really. :) /me takes a peek09:53
sivangbradb: I'm really interested in working/helping with the support/issue tracking stuff, I take it that the only missing link currently is the buffer application? (I wonder if you recall I mentioned it to you long before)09:58
bradbsivang: I don't know if there's plans for another application in between those two, but I sure hope not. :P09:59
bradbsivang: It seems to me that Launchpad itself could glue those two things together sufficiently (without, say, adding YAMT [Yet Another Menu Tab] )10:00
bradbDisclaimer: IMHO10:00
sivangheh10:03
sivangbradb: however, if we don't apply some kind of filtering we will end up with numerous support request that will require much work to sort out, attach to bugs etc10:03
sivangI forsee so many "my mouse don't work" etc10:04
bradbsivang: What "filtering" are you referring to exactly?10:05
bradbe.g. you can relate tickets to various contexts, e.g. distributions or upstreams, which seems like a significant step in filtering to me10:06
sivangbradb: this is filtering according to responsibility sources, what about filtering according to different parts of the OS, differnt classes of problems etc10:10
kikojordi?10:10
bradbsivang: it appears that you can attach it to a specific sourcepackage as well. i imagine there would be a lot of overlap with the same problems Malone has with that for bugs.10:11
bradbwhich, in both cases, keywords may be able to address (i.e. for arbitrary grouping of tickets or bugs)10:12
kikoJORDI!10:13
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [r=lifeless]  Launchpad Auto Build System User Interface Prototype (buildfarm UI) also minor fixes for buildd infrastructure, still needing mpt love. (patch-2427: celso.providelo@canonical.com, mpt@canonical.com)10:34
cprovWOW !!! 10:34
kikoYES!10:37
bradbkiko: Would you be interested in doing a drive-by review of a fix for #1378? (Ripping out database imports in another module where they shouldn't be.)10:45
kikohow big?10:45
bradbkiko: diffing now...probably 50-100 lines max10:46
kikosure10:46
bradbcool, thanks, just bazzing it now...10:47
=== camilotelles [n=Camilo@20132139198.user.veloxzone.com.br] has joined #launchpad
bradbhm, it appears that limit doesn't work with selectBy. @#*!!%11:08
kikowow, shipit test failures. wonder why.11:09
bradbkiko: you've got patchmail11:26
=== kiko waits for baz
mptarg11:27
=== mpt just typed "launchpad.net/people/mpt/+bugs" and got a 404
kikobradb, fix bug 2361 today11:27
kikompt, where was the error?11:27
mptkiko: Unimplemented feature11:27
kikoblame the bradb11:28
bradbmpt: You can find dozens of those types of URLs in LP.11:28
bradbs/URLs/404s/11:28
mptbradb: this isn't from a bad link, it's from an overly-consistent (for now) mental model11:28
mptkiko: https://wiki.launchpad.canonical.com/MaloneFrontPages11:28
bradbmpt: Yes, I know :)11:28
bradbanother example would be, say, https://launchpad.net/distros/ubuntu/+sources/mozilla-firefox11:29
mpthey, but I can see https://launchpad.net/people/mpt/+assignedbugs now11:29
bradbkiko: Dude, 2361 has been that way for months. Why is it suddenly urgent at 17:30 Friday night? :)11:31
kikobecause I SAID SO!11:31
kikobradb, it's not so urgent, and it's not night, but if you fix it I will love you11:32
kikobradb, will you get us mountain bikes to ride while I'm in montreal?11:32
bradbkiko: sure. how many?11:32
kikoone for me?11:33
kikoI'll take my pedals shoes and helmet11:33
kikowe can go for 6am rides11:33
bradbyeah, that could be fun11:33
kikoor freezing11:33
bradbthat too11:34
kikowhat pedals do you have?11:34
bradbJust the ones that came with the bike.11:34
bradbi.e. not clipless11:35
bradbI've been thinking of getting some though11:35
kikooh11:35
bradbyeah, i'm pretty tame atm11:36
bradbno pedals == sissy riding11:36
kikoyeah11:37
kikojohan's been using a spare pair of time atacs I have11:37
bradbkiko: so, From: <preferredemail>, Reply-To: <bug address>?11:38
kikocorrect11:38
kikoand the bug address should be11:38
kikobug666@bugs.launchpad.net or bug555@launchpad.net11:38
kikosomething like that11:38
bradbhm11:40
kikoI mean, others may correct me, but a lot of spamfilters block all-numbers email11:40
bradbkiko: can i merge the patch i just sent you?11:45
kikowait for baz!11:47
bradbwhat's the wait for? i waited for baz so that you don't have to!11:48
bradband, using mutt, you should have enough RAM left to read that email too11:48
kikoobviously you've never used baz11:49
bradbhave so! i'm waiting on it right now.11:49
bradbstatus before switching11:50
kikoit's comparing from and to!11:50
bradbFROM and TO, you mean?11:50
kikothat too11:51
kiko--  utilities/lint.sh11:51
bradbahhhehhe11:52
kikobradb, I like the patch. I'm a bit concerned with orderBy, though12:01
kikowhy am I concerned?12:02

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