/srv/irclogs.ubuntu.com/2005/10/12/#launchpad.txt

=== dand [n=dand@83.103.205.67] has joined #launchpad
=== stub [n=stub@203-214-4-72.dyn.iinet.net.au] has joined #launchpad
kikoheya stub 12:25
stubyo12:26
lifelessyoiifcation12:29
kikohow goes it12:31
kikohey sabdfl 12:31
sabdflkiko: excellent thanks. you?12:32
kikopretty good. glad to see you back in the london seat! had a good rest?12:32
ajmitchhi12:33
stubIs Steve's navigation work ready for production rollout Tuesday?12:34
kikostub, I need to merge them in to test, first12:34
kikoit's a pretty big change12:34
stubkiko: There was a patch from salgado improving the shipit export file name. Do you happen to know if it unguessable (since it contains very private data?)12:35
=== niemeyer_ [n=niemeyer@200.138.52.87] has joined #launchpad
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
stub(or is he emailing the results instead of sticking them in the Librarian?)12:46
=== stub waits for his revlib cleanup to finish
stubCurrent one in my branch is guessable and needs to be fixed. Don't know about the updated version yet12:48
=== RWG [n=RWGraphi@69.177.65.86] has joined #launchpad
=== RWG [n=RWGraphi@69.177.65.86] has left #launchpad ["Connection]
=== spiv [n=andrew@adsl-66-203.swiftdsl.com.au] has joined #launchpad
kikostub, both are guessable. jane wants them guessable. what should we do?12:53
stubkiko: They can't be guessable until we work out how to put security into the Librarian and implement it. The only place we can put a secret at the moment is the filename, so any private data going into the Librarian *must* have a strongly unguessable filename.12:55
stubJane or marileze or whoever sends the files to the producer will need to manually rename it before resending it if necessary.12:56
kikostub, can you email jane directly and raise this concern? at any rate, I think this will have to be dealt with next week.. jane really wants them to run this week12:57
=== lamont__ [n=lamont@129.19.2.8] has joined #launchpad
stubIt will be fixed today and merged in. I can't run the export until it s done. There is no way I'm going to create a permanent URL containing the shipping addresses of 20,000 geeks incudeing a fair number of privacy nuts and make it publicly available on the web. There is currently no alternative.12:59
=== Virtuall[BZA] [n=virtuall@who.is.virtuall.info] has joined #launchpad
kikostub, whatever -- negotiate with the client, is all I ask.01:01
kikoshe's not a lax customer either01:01
stubkiko: There is no negotiation. We cannot do it. It would be illegal in most countries.01:01
kikoyou're not listening.. :)01:02
stubI am. If I wait until Jane is online and tell her the problem *before* fixing it, there will be no shipit run this week. If I fix it now, there will be.01:02
Nafallodude... we are releasing _next_ week :-)01:03
kikoI agree you don't need to wait for her; I am pointing out however that this will need to be discussed with her, because she already asked salgado to change the filename to something less obscure.01:03
stubkiko: So there will be no test shippit run this week, and I still will not do the export until it is fixed.01:04
stub*My* address is in that list.01:04
Nafalloah. test. :-P01:04
mptnight all01:06
stubnight01:07
stubkiko: So shall I email Jane saying 'This is the situation, there is no alternative, and it would be fixed already except Kiko wants Jane to be aware of the problem before we fix it for no particular reason since there is no choice or decision to be made', or 'This is the issue, which I have fixed' 01:16
kikostub, I already conceded your point above -- do the fix, and notify her of the issue.01:18
stubkiko: ok ;)01:19
stubbah - I read that sentance three times and missed 'don't need to wait for her' each time ;-P01:19
stubWeee... utilities/library-relink.py supposedly just saved me 2GB of a 2.7GB revlib01:20
kikoI feel like crying when baz gives me conflicts in files I didn't tough01:23
kikotouch either01:23
stubYer - I got that on a feature branch last night. I typed 'baz undo' and went to bed to will be having fun later today ;-/01:24
lifelesskiko: during merg e?>01:25
=== eruin slaps kiko discreetly with a search tool while making duck-like sounds
kikolifeless, yeah, during merge01:27
kikoI'm also getting the weirdest thing01:27
kiko--  {arch}/launchpad/launchpad--devel/launchpad--devel--0/rocketfuel@canonical.com/patch-log/patch-217001:28
kikolifeless, any clue what that is?01:28
stubBah - I think salgado broke the sampledata on that branch so some build tests are failing during the cherry pick ;-/01:29
kikooh, gross01:30
stublifeless: I might have to roll out a branch where some of the tests are failing due to sampledata issues01:30
kikoman, I'm really baz-fucked today01:30
kikolifeless?01:35
sabdflkiko: me too01:41
sabdfl128 conflicts01:41
sabdflnaaaice01:41
sabdflnight all01:41
=== sabdfl [n=mark@pdpc/supporter/silver/sabdfl] has left #launchpad []
kikoI got only 301:42
kikomaybe I should just retire this branch01:42
kiko00:09:02 INFO    lock /var/lock/launchpad-poimport.lock already exists, exiting02:11
kikostub, that message is happening quite often, perhaps we should increase the interval with which it runs?02:11
stubkiko: That would make a longer possible delay between someone uploading a file and it being imported. It isn't an issue except we are running it without '-q' at the moment, on Carlos' request02:12
stubMight be better to reduce the verbosity of that message to 'DEBUG' ?02:13
stubBah - crontab doesn't know the output it is emailing is utf-8 encoded ;-/02:14
kikoyeah, hmmm02:14
matsubaralifeless, ping02:31
lifelessmatsubara: ?02:31
lifelesskiko: pong02:31
matsubaralifeless, could you help me solve this: https://launchpad.net/products/launchpad/+bug/750 ?02:31
matsubaralifeless, do you know what happened to needsSyncReview?02:32
lifeless?02:32
lifelessneedsSyncReview is/was a flag used in the product series workflow by the importd team, which is currently ddaa02:33
stubkiko: Do you happen to know if salgado is retricting shipit order information to ASCII at input time?02:36
matsubarastub, yes, for everything (but the name originally -- it's now for everything, period)02:41
stubkiko: And where are the shipit tests living?02:41
matsubarathere are both functional doc and pagetests02:41
stubmatsubara: ok. There is still code in the export that is stripping accented characters, which is dangerous because it is lossy. I'll remove it.02:41
stubmatsubara: But where. I can't find them ;)02:41
matsubaradoc/shipit.txt?02:42
stubmatsubara: Ta. I need glasses.02:42
matsubaraalso standalone/xx-shipit-*02:42
matsubaralifeless, do you know why the flag is missing?02:43
matsubaralifeless, or if it was database-backed?02:43
lifelessmatsubara: I do not recall, sorry02:43
=== rbelem [n=rodrigo@200.246.97.164] has joined #launchpad
matsubarakiko, remember to close 53803:01
=== tritium [n=michael@pcp0011975002pcs.sandia01.nm.comcast.net] has joined #launchpad
=== interalia [n=interali@adsl-60-232.swiftdsl.com.au] has joined #launchpad
=== rbelem_ [n=rodrigo@200.246.97.164] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: r=kiko Fix for bugs 2468 and 2467: System error when targeting a fix for a release, when no source package is specified, and System error when requesting fix in a distribution without specifying a source package. Unifies the way BugTask target names are rendered throughout Launchpad, making it consistent and ensuring we use a single codepath. Adds a test for bug 2467; pending a test for bug 2468. Patch 05:19
=== philiKON [n=philipp@222.70.121.120] has joined #launchpad
philiKONis there someone here who can give me access to https://launchpad.net/products/zope/+series/zope3.1/+pots/zope/+upload ?05:35
philiKONi'm the maintainer of the zope translations05:35
philiKONuploading a POT used to work, no idea why it doesn't anymore05:35
philiKONget a "Forbidden" now05:35
lifelessspiv: ping05:38
lifelessphiliKON: I'm not sure.05:41
lifelesscarlos does not seem to be around right now05:42
philiKONthat's ok, i'll wait05:42
lifelessand hes Da Man for this05:42
philiKONok :)05:42
jameshphiliKON: maybe because you aren't listed as registrant of zope?05:43
jamesh(or belong to a team listed as the registrant05:44
philiKONjamesh, i'm the team administrator05:44
philiKONhttps://launchpad.net/people/zope3-dev/+members05:44
philiKONfor some reason, "mohsen" is listed as the owner of https://launchpad.net/products/zope/+series/zope3.1/+pots/zope though05:45
jameshphiliKON: sure, but https://launchpad.net/products/zope shows the registrant as stub05:45
jameshmaybe stub knows :)05:45
jameshstub: is there any reason you are listed as owner of https://launchpad.net/products/zope rather than the zope3-dev team?05:55
stubBecause I registered it? And the zope3-dev team doesn't necessarily have anything to do with Zope2, which will be a separate product series?05:56
stubI have no ideas what rights it conveys besides setting metadata on the product and series05:57
=== robitaille [n=robitail@d154-5-117-228.bchsia.telus.net] has joined #launchpad
jameshI wonder why philiKON used to be able to upload a template then05:58
philiKONi think i created the zope3.1 series05:59
philiKONhttps://launchpad.net/products/zope/+series/zope3.1/+pots/zope is owned by "mohsen" though. no idea why.06:00
tritiumlifeless, I was approved as a member many months ago, before we started using launchpad.  Who approves the proposed members of "Ubuntu Members"?06:00
philiKONi definitely remember uploading the first POT06:00
lifelesstritium: mdz I would presume06:01
ajmitchor any of the CC06:01
tritiumlifeless, ok, thanks06:01
tritiumyou too, ajmitch 06:01
ajmitchspeaking of zope, how would I get a team renamed?06:02
ajmitchsince I in my ubuntu-centric ways registered a zope team on launchpad, intended for ubuntu & debian packaging06:03
=== bradb_ [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
=== bradb_ [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has left #launchpad []
philiKONajmitch, we might want to merge that with the zope3-dev team...06:05
philiKONi created the zope3-dev team out of my zope3 developer-centric view :)06:05
ajmitchright :)06:05
ajmitchthe debian/ubuntu team handles zope 2.x & 3.x packaging06:06
bob2hah06:06
bob2is Five in ubuntu/debian yet?06:06
philiKONbob2, well, it is through zope2.806:06
philiKONfive 1.0 is part of zope 2.806:06
bob2oh, great06:06
stubI'm happy to change ownership of the Zope product and projects to a suitable team06:07
lifeless'five' ?06:08
philiKONheh06:08
stubZope 2 + 306:08
lifelesserk06:08
bob2lifeless: brings some zope3 things to zope206:08
philiKONwhy not just 'zope'06:08
ajmitchfun stuff :)06:08
philiKONif i can be an admin of 'zope', i would vote for 'zope'06:08
lifelessman, thats gonna be fun when z4 comes out, 'nine'06:08
philiKONthen we can move people over from 'zope3-dev' to 'zope' slowly...06:08
philiKONlifeless, there won't be a z4... hopefully06:09
ajmitchmaybe rename the existing zope team to pkg-zope, as it is in debian :)06:09
stubHow about you create a zope-admins team philiKON and I change ownership to that?06:09
philiKONstub, phew, that would be a lot of teams then.06:09
philiKONstub, but ok06:09
stubTeams can be members of teams. Lots of rope for everyone!06:10
philiKONah, yeah :)06:10
=== ajmitch should really tweak .procmailrc to sort ozzope mail into a separate folder
stubYou probably just want zope-admins and zope-developers (or zope) for the product side.06:11
mdzlifeless: the CC deals with approving Members06:11
philiKONright now we have 'zope' and 'zope3-dev'06:11
stubAnd the distro stuff should be seperate teams probably once members can automatically upload new packages into Ubuntu Grumpy Groundhog.06:11
mdzlifeless: I deal with developers06:11
lifelessmdz: ah righ06:12
philiKONi guess we want to end up with 'zope-dev', 'zope-admin' and 'pkg-zope'...?06:12
lifeless't'06:12
ajmitchstub: the distro stuff is useful being separate now for bug assignment06:12
lifelessajmitch: mmm, you have 'upstream' and 'distrorelease' for that06:12
ajmitchtrue06:12
stubphiliKON: You could collapse 'zope-dev' and 'zope-admin' into one team if you don't mind giving full control over everything to eveyone ;)06:12
philiKONnah, having them separate is better06:13
philiKONstub, can teams be renamed?06:13
tritiummdz, should I just wait for the CC to review the proposed members, or ping somebody?06:13
stubAccording to the spec, yes.06:13
=== stub checks reality
mdztritium: if they were approved at a CC meeting but not processed in launchpad yet, then I'd say poke someone06:13
philiKONstub, i propose to rename "zope3-dev" to just "zope-dev" and "zope" to "pkg-zope", like ajmitch suggested06:14
tritiummdz, thanks.  I will, then.06:14
stubBah - there is no team rename in there. Thankfully the DBA is online...06:14
philiKONstub, i created the 'zope-admin' team. would be great if you could give ownership of the zope package to that team06:15
stubRenames are done06:16
philiKONajmitch, does anybody from the package guys need to be in this zope-admin team?06:16
philiKONah, cool06:16
philiKONthanks06:16
stubExtra points for someone creating a 16x16 icon for the emblem...06:16
ajmitchphiliKON: probably not, afaik06:17
philiKONajmitch, ok... lemme know if that becomes the case06:17
ajmitchwhat does the zope-admin team do? :)06:17
stubzope-admins now owns the zope product06:18
philiKONbe the owner of the 'zope' project06:18
ajmitchright06:18
spivlifeless: pong06:18
ajmitchwe'll apply if needed then06:18
stubChange the product description, create new product series, that sort of stuff06:18
ajmitchstub: thanks for the rename06:18
lifelessspiv: whats a good exception for 'cannot add FOO to container BAR, FOO is already present'06:18
spivHmm.  Probably make an AlreadyExistsError?06:21
lifelessyah, so there is not top level one06:21
lifelessthanks06:21
stubrorrEyeK ?06:21
stubProbably a subclass of ValueError06:21
spivI'm not sure about that.06:22
lifelessworks for me, TODOing it - thanks06:22
spivDo you really want random "except ValueError:" clauses catching this?06:22
lifelessnoodle time06:23
stubMmm.... noodles....06:23
philiKONajmitch, stub, you guys know what's up with this team: https://launchpad.net/people/zope3-dev-zope ?06:26
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
stubphiliKON: That is actually a person. The zope3 developers mailing list email address must have gotten sucked in an an account created.06:28
philiKONah06:28
=== stub nukes it
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  ShipIt paranoia updates (patch-2594: stuart.bishop@canonical.com)06:35
lifelesswhilst you are nuking stuff07:05
lifelesshow octopus-like are the sprint ?07:05
lifeless(s)07:05
stubdunno ;-)07:10
stubDoesn't look bad. Just a cluster of three tables.07:11
lifelesscare to nuke the 'test print one'07:11
dilysMerge to rocketfuel@canonical.com/launchpad--production--1.35: Merge ShipIt export into production (patch-8: guilherme.salgado@canonical.com, stuart.bishop@canonical.com, rocketfuel@canonical.com)07:12
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Add shipit indexes (patch-2595: stuart.bishop@canonical.com)08:22
=== carlos [n=carlos@243.Red-83-47-24.staticIP.rima-tde.net] has joined #launchpad
carlosmorning09:04
carlosstub, hi09:04
carlosstub, the language pack script is using the db as read only09:04
carlosam I able to use commit in that situation?09:04
stubcarlos: yes.09:05
carlosoh!09:05
carlosok, didn't know that, then  I will commit after every pofile export09:06
stubcarlos: You can start abusing it again if you want - the upgrade I was doing has finished.09:06
carlosok09:06
carlosthanks09:06
stubcarlos: I had to kill the process because I didn't notice what was happening until half way through09:06
carlosstub, added a commit and running the script now. Could you confirm if it's enough?09:12
carlosstub, btw, I think there is something wrong with the change you did to prevent that poimport fail more than once per day with a file09:13
carlosstub, we have more than 500 pofiles that are not being imported and I don't see anything in our logs that indicates that those pofiles are giving us problems09:14
stubcarlos: Shall I nuke the history and see what happens?09:14
carlosstub, yes, please09:14
stubcarlos: https://chinstrap.ubuntu.com/~dsilvers/paste/filenRPozP.html09:17
stubcache nuked09:18
carlosstub, thank you09:19
carloslet's se what happens...09:19
philiKONman, launchpad is sure taking long to upload my zope3.1 POT...09:24
=== GoRoDeK [n=gorodek@p5083E981.dip.t-dialin.net] has joined #launchpad
=== Virtuall[BZA] [n=virtuall@who.is.virtuall.info] has joined #launchpad
carlosphiliKON, when did you upload it?09:29
philiKON3 hours ago maybe09:29
carlosphiliKON, I cannot check it now, but I detected some problems with the import queue, debugging it atm09:31
stubI might have interrupted that import - I had the system down for about 30 minutes.09:34
philiKONstub, should i just resubmit?09:37
stubphiliKON: I don't know ;) Carlos? 09:37
carloshmm, I'm not sure09:38
=== sabdfl [n=mark@host217-37-231-22.in-addr.btopenworld.com] has joined #launchpad
carlosstub, please, could you execute SELECT ptn.name, pt.rawimportstatus from potemplatename ptn, potemplate pt where pt.potemplatename=ptn.id and ptn.name like 'zope'; ?09:38
stub3 entries, all with rawimportstatus==309:39
carlosthat means that the import is done09:39
carlosphiliKON, could you confirm, please?09:39
philiKONcarlos, ok. how do i do that? i have no idea what changed in the POT after the last upload (i  just know *that* it changed)09:40
sivangGood morning everybody09:41
carlosphiliKON, Project-Id-Version: Development/Revision: 38234/Branch: Zope-3.109:41
carlosphiliKON, is that one?09:41
carlosor that's the old one?09:41
carlossivang, morning09:41
philiKONcarlos, lemme check09:42
philiKONcarlos, looks like the new one09:42
philiKONcarlos, thanks09:43
carlosphiliKON, cool09:43
carlosI did nothing ;-) but you are welcome09:43
=== dand [n=dand@gw.datagroup.ro] has joined #launchpad
KinnisonMarning09:58
carlosstub, ok, seems like the changes you did are the problem here10:04
carlosstub, the script is importing files from Wednesday now10:05
stubok. probably just a logic fault in the 'recently seen' code.10:05
SteveAhi10:05
stubcarlos: I've scheduled a daily deletion of the cache, which should last until we can have a closer look at it.10:08
SteveAyo stub 10:08
stubReview my branch bitch it is growing hair!10:08
SteveAdid you get the shipit export crack sorted? 10:08
carlosstub, ok, thank you10:08
stubStill working on shipit10:08
SteveAstub: that's kinda fitting, coming from you10:08
SteveAstub: i had a thought... export to a place on the filesystem10:08
SteveAa directory given in the config10:09
SteveAand point apache ,with auth at it10:09
stubOh... exports are sorted for now. Although we should refactor to not use the Librarian in this case I think.10:09
stubI'm trying to rollout the updated shipit stuff to production10:10
stubOr implement authorization into the Librarian somehow.10:11
=== thebigearl [n=thebigea@dialin-80-228-157-042.ewe-ip-backbone.de] has joined #launchpad
thebigearl#ubuntuusers10:13
thebigearloops... sry10:13
=== Jonex [n=Jonex@80.65.200.134] has joined #launchpad
carlosSteveA, spiv hi, around?10:31
SteveAhello carlos10:32
carlosSteveA, spiv http://librarian.launchpad.net/560358/560391/lSkFwO4AG0ocgPLZEaZ8JaScmVg.txt10:32
carlosWe still have problems with librarian...10:32
carlosI can implement a workaround for that with a try/except and doing a full export instead of using the cache from librarian10:33
carlosbut that will hide that problem10:33
SteveAcarlos: that's an isolated traceback10:33
SteveAit doesn't tell me what was running10:33
SteveAspiv: ping10:34
carlosSteveA, It came from a .po export request10:34
carlosthe failure is when we fetch a file from librarian10:34
SteveAand, this is a file that was recently added?10:34
carlosnot sure10:35
carlosbut spiv already checked the code and we have the needed commits10:35
carlosso in theory is not a transaction problem if that's what you are thinking on10:36
SteveAstub: is commit() synchronous?10:36
carlosspiv is aware already of that bug and we don't see where is the problem exactly...10:36
SteveAwell10:36
carlosmy question is if I can "hide" that problem10:36
stubeh?10:36
SteveAwhen spiv and i talked a while ago, we have a few ideas of what might be the root of the problem10:37
carlosor what should I do so users don't get this problem10:37
SteveAand spiv was going to work on finding out which of the ideas is correct10:37
carlosSteveA, oh! didn't know it10:37
SteveAstub: when in python i say commit(), does that call block until the data is in the "current committed data for all threads" on the postgres server?10:37
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
stubSteveA: So I'm told.10:38
SteveAcarlos: can you try putting a try/except around the call to the librarian in pofile.py, and if there's a LookupError, then log a warning, sleep for 1s, try again10:39
stubI don't think spiv has tried lowering the transaction isolation a notch which would be safe and probably solve this10:39
carlosSteveA, Is that better than just request a full export?10:40
SteveAyes10:40
SteveAit also tells us when such an error has occured10:40
SteveAso we can keep investigating it10:40
SteveAstub: my worry about doing that is we may never find the cause of the problem10:41
carlosSteveA, I mean, log the warning and do a full export so if the error persists the user will get the file...10:41
carlosSteveA, a full export does not depend on librarian10:41
stubIf it solves it it tells us a lot about the cause of the problem.10:41
carloswell, it does but to feed it not to retrieve anything10:42
SteveAstub: would you lower the isolation level for the librarian, for the exporter, or both?10:43
stubSteveA: The Librarian10:43
SteveAcarlos: okay.10:43
SteveAstub: it's tough though, beacuse all we will see is the absense of problems, not problems being solved.10:43
SteveAstill, provided carlos is logging this, we can see the frequency of errors, and try this out10:44
SteveAwithout shitting on things10:44
stubSteveA: It will mean that *yes*, the commit is happening in the export and *yes* the commit is visible to the Librarian. Which means either the Librarian has a transaction issue (not resetting the transaction at the start of the request). 10:44
SteveAstub: sure, so i'm saying, let's get carlos to make this change, watch the logs for a while, then lower the isolation level, and watch the logs for a while.10:45
SteveAand note the differences.10:45
stubc/carlos/spiv10:45
stuboh10:46
carlosstub, no, it's my work :-P10:46
carlosstub, is the commit that I added enough to fix the lock problem you told me this morning?11:04
stubcarlos: Dunno ;)11:05
stubcarlos: Should be11:05
carlosok11:05
stubThe easiest way for me to tell could break your export run again, so I won't do that just yet ;)11:06
KinnisonWhat is the correct way to invoke your superclass' __init__ ?11:06
Kinnisonit it simply MySuperClass.__init__(self)11:06
Kinnisonor is there some more 'correct' solution?11:06
stubIf you are a classic class, yes.11:06
KinnisonI'm a __metaclass__ = type11:07
SteveAKinnison: are you intending your class to be used in a crazy heap of mixins?11:07
SteveAdo you have a single base class?11:07
Kinnisonno11:07
Kinnisonyes11:07
SteveAthen that is the correct way11:07
KinnisonIt's a very simple abstractparent ==> (child1, child2) heirarchy11:07
SteveAgood11:07
SteveAthen use what you wrote11:08
Kinnisonokay11:08
Kinnisonthanks11:08
jameshauthserver down?11:08
SteveAstub: i'm seeing a result ordering issue in shipit code11:09
KinnisonSteveA: Is there a method run on class construction?11:09
SteveAhttps://chinstrap.ubuntu.com/~dsilvers/paste/file2tLS5P.html11:09
lifelessKinnison: __init__11:10
KinnisonSteveA: I want to register each of the child classes in an array11:10
SteveAKinnison: what do you mean "class construction"11:10
Kinnisonlifeless: that's instance construction11:10
SteveAKinnison: there are such things11:10
SteveAKinnison: don't mess with them11:10
jameshSteveA: I did some fixes to fmt:text-to-html that should fix https://launchpad.net/products/launchpad/+bug/174911:10
SteveAdo it by hand11:10
Kinnisonokay, so I should just register them by hand in the module level code?11:10
SteveAyes11:10
Kinnisonsure11:10
Kinnisonthanks11:10
jameshit uses 77% less regular expressions11:10
SteveAanything more contrived gets scary11:10
lifelessKinnison: then not that I know of, you'd have to hook into the module loader.. but __new__ may be close to what you want, which is run to create instances, or just put the stuff you want in the class decl..11:10
SteveAexcept for gurus11:10
lifelessclass Foo:11:10
SteveAwho love that shit11:11
lifeless    something_here_is_run_at_'class construction'11:11
stubSteveA: Looks like the usual11:11
SteveAlifeless: i'd be inclined to use a class advisor11:11
SteveAlifeless: but, really... it is too complex for what Kinnison  needs11:11
lifelessSteveA: I don't know that term11:11
SteveAlifeless: like implements(IFoo)11:11
lifelessSteveA: nor what Kinnison needs, was just answering the question ;)11:11
lifelessSteveA: oh right, just code executed during class parsing11:12
SteveAi came up with the idea when jim said something was a nice idea but couldn't be done ;-)  i showed it to PJE who coined the term 'class advisor' and improved the implementation and added to his PEAK library.  jamesh wrote a weblog entry on them11:12
lifelesssweet11:12
SteveAlifeless: code executed during class parsing which changes the metaclass11:12
SteveAand remembers the original metaclass11:12
lifelessSteveA: decorates the metaclass ?11:13
SteveAthen uses the metaclass to hook into offering some advice11:13
SteveAkind of11:13
lifelessgone again..11:13
lifelessbye-for-now11:13
SteveAit isn't really a decorator in the sense we usually use the term11:13
SteveAi think11:13
SteveAstub: usual?11:14
jameshlifeless: http://blogs.gnome.org/view/jamesh/2005/09/08/0 <- that's my understanding of the code11:14
SteveAjamesh: the rules you give for determining the true metaclass are i think slightly simplified11:16
SteveAthen again, they aren't documented properly anywhere except in the C source code that does this stuff11:16
SteveAeven GvR generally documents them wrongly11:16
SteveAfortunately PJE is kinda pedantic on these things ;-)11:16
SteveAi think the simplified version is better for a blog article anyway11:17
=== Seveas [n=seveas@seveas.demon.nl] has joined #launchpad
stubSteveA: Someone forgot to put in an orderBy argument so we get indeterminate SQLObject result ordering11:19
SteveAand it isn't warned11:20
SteveAbecause the test uses a list comprehension11:20
SteveAi'll let salgado fix it in a few hours11:21
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Split a shipit db patch in two so it can be run seperatly (patch-2596: stuart.bishop@canonical.com)11:21
SteveAunless it's going to crap all over pqm merges11:21
KinnisonI am creating some "context" control classes for the uploader. They allow the uploader to decide what tests are appropriate for an upload in a given context. E.g. for uploads from the buildds, we don't check for signatures.11:41
=== zyga [n=zyga@2-mi2-1.acn.waw.pl] has joined #launchpad
KinnisonNow, I could just set up things like in __init__ do self.checksigs = True/False11:42
Kinnisonor I could have effectively static properties11:42
Kinnisonso that I can docstring them11:42
Kinnisonwhat is the best style?11:42
=== _Rappy_ [n=hunt-pre@dsl-253-122.monet.no] has joined #launchpad
KinnisonSteveA: ^^ ?11:59
SteveAKinnison: hello12:00
SteveAKinnison: can you possibly find a different term than "context" ?12:01
SteveAor a more specific term?12:01
KinnisonIt's an UploadContext12:01
KinnisonOr rather, one of InsecureUploadContext and BuildDaemonUploadContext12:02
SteveAokay.  just, don't use the variable name 'context' anywhere12:02
KinnisonNot even in process-upload.py ?12:02
SteveAyou have an OO design choice here12:02
SteveAno not even there12:02
SteveAalways uploadcontext or ucontext or something12:02
Kinnisonokay12:03
SteveAit was very confusing reading 'request' in salgado's original shipit database classes12:03
KinnisonFair enough12:03
SteveAso, the OO design choice is12:03
SteveAdo you make your UploadContext classes express information about policy, or do you get them do implement policy12:03
SteveAthat is, you can say12:04
=== ddaa [n=ddaa@ordo.xlii.org] has joined #launchpad
SteveA  class UploadContext:12:04
KinnisonThey are to express policy12:04
SteveA      checksigs = False  # the we trust all uploads cos we are on crack12:04
SteveAor you can say12:04
SteveA  class UploadContext:12:04
KinnisonThe NascentUpload class will apply policy provided by a context object12:04
SteveA      def verify_upload(self, upload):12:04
SteveA          ... this implementation doesn't check the sig for the above reasons12:05
=== Kinnison was pondering
SteveA12:05
Kinnisonclass InsecureUploadContext:12:05
Kinnison    @property12:05
SteveAyou could also have a single ContextPolicy class12:05
SteveAand make singleton instances of it12:05
SteveAto express the different variations in policy12:05
Kinnison    def check_sigs(self):12:05
SteveAif there are only a few of these12:05
Kinnison        """We check signatures on insecure uploads."""12:05
Kinnison        return True12:05
SteveAi suggest you have some classes, and use class attributes12:06
SteveAand define an interface IContextPolicy (or whatever)12:06
SteveAwhere you document the expected attributes12:06
Kinnisoncontexts need to verify their arguments though12:06
KinnisonE.g. The BuildDaemonUploadContext will verify that the process-upload instance was given a build-id to attach to12:06
SteveAthis is a different thing than what you were asking originally12:07
SteveAi would need to understand how the parts fit together in order to make a recommendation about this12:07
KinnisonOkay12:07
SteveAbut12:08
=== Kinnison ponders how best to describe this.
SteveAmaybe start by writing the interfaces for the various moving parts12:08
SteveAand write a decent interface docstring for each12:08
SteveAso, you have your upload context part, and your process upload part 12:08
=== Kinnison nods
SteveAthat should help to communicate the larger picture12:08
KinnisonMy NascentUpload stuff is what I was going to start writing next12:08
Kinnisonbecause I imagined that as I try and apply a context to a NascentUpload it'll give me a chance to work out the best interface12:09
SteveAyou can write in the docstring what you imaging the scope and responsibilities of an UploadContext are12:10
SteveAthese things are not clear to me12:10
Kinnisonyep, I'll do that12:10
KinnisonThe doctests have some of this information in them already12:10
KinnisonI guess I need to migrate that to the docstrings12:10
SteveAokay12:11
SteveAperhaps i've been some help clarifying things12:11
KinnisonYep12:11
Kinnisonthanks12:11
=== Kinnison gets on with things
=== Kinnison imagines that once it comes down to code review more will come out and perhaps more directed suggestions will be possible
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
sivangSteveA: so , I don't recall (after checking the logs) what book did you recommend for Zope, besides the "Web component development with Zope 3" or wasn't any other? :)12:18
SteveAi haven't read any of the books for learning zope312:19
SteveAbob2 has, i think12:20
SteveAi know the authors of both books12:20
SteveAstephan's book is available online12:20
SteveAso, that's a good start12:20
SteveAit is a wiki site12:20
SteveAbut you can buy the paper book too12:20
sivangk thanks12:25
=== sabdfl [n=mark@pdpc/supporter/silver/sabdfl] has left #launchpad []
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  rename uploader.py to poppyinterface.py in canonical.archivepublisher (patch-2597: daniel.silverstone@canonical.com)12:30
KinnisonWhen a doctest is run, what is its CWD?12:41
lifelessKinnison: reading between the lines, look at the gpg tests, they figure out where to find files12:45
KinnisonRight12:46
lifeless(the answer is, any, you don't (cant) know)12:47
SteveAKinnison: i suggest to do the following: import a module.  inspect the module's __file__ attribute.  get files relative to that.12:48
=== Kinnison fortunately has exactly what he wants :-)
SteveAwe could have a open_file_from(module) or get_filename_from(module)12:48
SteveAif these are needed12:48
Kinnisonfrom canonical.archivepublisher.tests import datadir12:48
Kinnison:-)12:49
SteveAyay12:49
=== zyga [n=zyga@2-mi2-1.acn.waw.pl] has joined #launchpad
KinnisonHmm, interesting test system bug12:53
Kinnisonif you run: python text.py -f --test="foobar"12:53
Kinnisonwhere foobar matches some pagetest names too12:53
Kinnisonthen the pagetests are also run12:54
Kinnisonbut the story isn't run in its entirety12:54
Kinnisonwhich causes failures12:54
jordicarlos: was Tagalog added to the database in the end?12:55
jordi(tl code)12:55
carlosjordi, no, I don't think so12:56
carlosI'm going to request a change to the DB so I will add that request too12:56
SteveAKinnison: yes, known limitation in the pagetests12:56
KinnisonSteveA: this is very annoying12:56
jordiPlural-Forms is nplurals=2; plural=n>1;12:56
=== Kinnison has two doctests he wants to run, namely nascentupload.txt and uploadpolicy.txt
Kinnisoncan I simply give multiple --test= args?12:56
SteveAprobably not12:57
jordicarlos: great. We have a few others in the queue, but we don't have reliable info yet.12:57
SteveAjust run one, then run the other12:57
Kinnisonbut test startup is hideously expensive12:57
SteveAor start a wiki page / bug on what you want the test runner to do12:57
carlosjordi, send me it by email, please12:57
jordisure12:57
jordicarlos: it's in the launchpad mailing list though12:58
carlosok12:58
jordiDate: Fri, 23 Sep 2005 18:34:41 +020012:58
=== Kinnison reports https://launchpad.net/products/launchpad/+bug/2933
lifelesstheres a bug on ita laready01:00
KinnisonI couldn't spot it01:01
Kinnison:-(01:01
=== Kinnison is sure kiko will dedupe it later
stubjamesh: So now that the GPG web-of-trust code is available on staging, what should I run to set it off?01:07
carlosdo we have any testing account on production?01:08
SteveA    +   Module canonical.launchpad.database.bugtracker, line 68, in __getitem__01:14
SteveA    +     raise KeyError, id01:14
SteveA    + KeyError: <built-in function id>01:14
SteveA01:15
SteveAhahahaha01:15
lifelessdid that do what I think it did ?!01:18
SteveAi imagine that someone changed the name 'id' to the name 'name' in that function01:20
SteveAbut left it raising KeyError, id01:20
SteveAso id became the global builtin01:20
lifelessyah01:21
stubIt could also mean someone passed __builtin__.id into the __getitem__ method01:26
stubid might be a valid local that happens to be bound to the __builtin__ ;)01:27
SteveAand furthermore01:28
SteveAa rogue C extention could be corrupting the heap and causing a legitimate string id to be replaced with that function01:29
stubNah - it is a buggy __repr__ method on the id object01:31
dilysMerge to rocketfuel@canonical.com/launchpad--production--1.35: [trivial]  sync (patch-9: stuart.bishop@canonical.com)01:31
stubdef __repr__(self): return repr(id)01:31
carlosjordi, request sent01:38
stubcarlos: any reason to keep the -v option on poimport?01:42
carlosstub, no, you can remove it now that we detected why it was not working01:42
jordicarlos: thansk mate01:52
carlosnp01:53
=== mpt [n=mpt@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
mptGooooooooooooood morning01:58
mpt"Unable to fork for patch", eh01:58
stubThe same file has been added on two branches, and I get two conflicts (the file, and the .arch-ids/thefile.id). How to I resolve it?01:58
stublifeless, ddaa: ^^^02:00
ddaabaz resolved --all?02:01
ddaaI mean02:01
ddaais that two files with different contents or same name02:01
ddaaor is that just the merge bug that cannot deal with redundant adds?02:01
stubbaz resolved .arch-ids/uuid.py.id02:02
stubresolved: One or more of the paths supplied doesn't exist.02:02
ddaabaz resolved --all02:02
ddaaI know baz resolved is buggy with conflicting adds.02:02
stubok. So leave it until I resolve everything else, and do the global resolve.02:02
lifelessstub: no02:03
lifelessstub: choose one of the fiells02:03
lifelessstub: and one of the ids02:03
ddaawell, you also need to remove one of the ids02:03
lifelessstub: move them into the the right place02:03
lifelessstub: delete the other two02:03
ddaabut that will appear in status anyway02:03
ddaabetter again, try doing the merge without spurious conflicts02:04
stublifeless: Did that. The .id still stays on the conflict list and I can't resolve it individually (I avoid --all incase I miss one)02:04
lifelessstub: ignore the conflict list02:04
lifelessstub: ;002:04
lifelessddaa: if its added twice it will always conflict, its not spurious02:05
ddaaI think you are wrong, for subtle reasons02:05
lifelessddaa: you have the same file, with two different ids, but sharing the same path02:05
ddaaBut I do not wish to have an argument about it at this current time and place02:05
lifelessddaa: that is a conflict in any VCS I know02:05
ddaaoh sure, except there's a bug in merge where that happens for files with the same id02:06
lifelessddaa: there is a separate bug, where if the file path is different, it doesn't know how to choose the right move target02:06
lifelessddaa: but thats not what I understood to be the case here02:06
ddaafrankly I've given up trying to interpret most baz bug reports02:07
ddaathere are few stock solutions to ease the immediate pain02:07
ddaabut most of the time what the users say is so out of line with what the problem actually is that's it's not worth it02:08
stubIt isn't a spurious conflict. I committed the same file on another branch because I needed the module and someone hadn't reviewed it yet. And now I'm cleaning up.02:08
=== matsubara [n=matsubar@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
matsubaragood morning02:08
=== bob2 forwarded all the debian baz bugs to lp and left them
lifelessddaa: well, fwiw, when I've looked at the 'why am I getting random conflicts' stuff, its always been history shortcuts02:09
bob2etoohardtofix02:09
lifelesswhich is a known issue in baz, and not relevant to bzr02:09
lifelessso I don't think we're getting stuff thats out of line, its quite predictable within the bounds of c-baz02:09
=== Jonex [n=Jonex@80.65.200.134] has left #launchpad []
SteveAjamesh: great news aobut bug 174902:11
SteveA(launchpad was offline when i wanted to look earlier)02:12
ddaahistory shortcuts?02:14
ddaalifeless: most of the time here, it turns out to be a criss-cross situation.02:19
ddaaApparently pqm encourages a pattern that causes criss-cross:02:19
ddaa1. send merge request02:19
ddaa2. merge rocketfuel before starting working on something else02:20
ddaa3. merge performed by pqm02:20
=== terrex [n=terrex@84-122-90-162.onocable.ono.com] has joined #launchpad
lifelessddaa: criss-cross is ok with baz merge, as long as you are not cherry picking02:20
lifelessddaa: not ideal either, but not bad, it will just take the common base before.02:21
ddaaexcept when baz merge goes out of memory trying to cache the whole history since the big bang02:21
lifelessddaa: usually you get one too far back, and thats due to history shortcuts02:21
ddaalifeless: let's abandon this line of discussion. I have no desire to rant about baz.02:22
lifelessddaa: ok02:22
lifelessddaa: note I'm not defending accuracy or reliability, just cause02:22
=== ddaa ignores lifeless' last message
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== lifeless goes back to counterstrike
=== niemeyer [n=niemeyer@200-193-158-172.ctame7006.dsl.brasiltelecom.net.br] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  converted a bunch of browser:traverse into navigation (patch-2598: steve.alexander@canonical.com)02:29
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
lamontlifeless: bazaar_1.4.2-1ubuntu1 has unaligned loads and/or stores in it's sha1 code.  fix that. kthxbye.02:38
lamontI suppose I should track down _where_, huh?02:39
=== lifeless channels anthony baxter
lifelessbug reports - you know things with patches - are welcomeed02:40
bob2I think he stole the sha1 code from elsewhere02:40
lifelessblame colin walters02:41
=== camilotelles [n=Camilo@firefumsoft.fumsoft.softex.br] has joined #launchpad
carlosSteveA, I have a request for the sysadmin02:50
carlosshould I add it to https://wiki.canonical.com/LaunchpadSysadminRequests ?02:50
SteveAcarlos: you can mail it to RT02:51
SteveAi'll msg you the email address02:51
carlosok02:51
carlosthanks02:51
SteveAit's a good idea to cc me or kiko02:51
SteveAor both02:51
carlosok02:51
ddaalifeless: in your "ignore annoying mac filenames in python" cscvs patch02:53
ddaawhat does "note that hiding all should be ok - validation checks the final result." mean?02:53
ddaalifeless: that's a clarification requested to me by spiv, since your patch ended up in one of my reviewed branches02:54
lifelessddaa: it means that hiding all invalid file names is ok, as live ones will trigger a end-result validation failure02:58
ddaaokay02:58
ddaathanks, it would be nice if you were a bit more proactive at getting your branch merged, though02:58
ddaaI'll handle that one.02:59
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
lifelessddaa: that one I deliberately left for you03:00
lifelessddaa: as you had other stuff you said to me you didn't want interrupted03:01
lifelesssome time back03:01
ddaaha... I guess so. It's hardly an extensive patch, I do not think it would have blocked me.03:01
ddaais that the same issue with your pybaz patch? :)03:02
ddaa(I know it's not, I'm just teasing)03:02
lifelessno03:03
lifelessthat one is me having to do a fix to hct03:04
lifelessand I keep getting halfway to doing it03:04
lifeless:/03:04
ddaaBTW, keep in mind that my next big chunk next week is working on the BranchDataStorage db patch.03:05
ddaaSo large amounts of HCT borkage are to be expected.03:05
bradbkiko-zzz: Hey. Any news on the sortorder widget patch I sent you yesterday?03:05
ddaalifeless: also, I'd like to know who is likely to work on cscvs/bzr and improtd/bzr?03:05
stubSteveA: BrowserNotificationMessages is now conflict free03:09
SteveAthanks03:09
SteveAi'm about to change that ;-)03:09
SteveAprobably not actually03:09
SteveAbut i'll review it after i send this next merge off, and get some lunch03:09
SteveAstub: i've converted all NotFoundErrors from zope to launchpad03:10
=== carlos -> lunch
kikohey ho03:19
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Added plural forms for Tagalog and associated Philippines's languages to that country (patch-2599: carlos.perello@canonical.com)03:28
bradbhey kiko 03:29
bradbkiko: might you have a chance to review the sortorder widget patch this morning?03:29
=== lamont [n=lamont@mix.mmjgroup.com] has joined #launchpad
=== gneuman [n=gneuman@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
kikobradb, yeah, but I need to spend some time with matsubara and gneuman first -- is it blocking you?03:35
bradbnot really blocking me, no03:35
kikocarlos the bugspammer :)03:37
SteveAsalgado: hi03:39
salgadohi SteveA 03:39
SteveAso, in my merge that is with pqm, i commented out two shipit doctest lines03:39
SteveAbecause there's a query with ambiguous ordering03:39
SteveAi mailed you a link to a traceback03:40
salgadoSteveA, I've been looking into that for the past 15 minutes03:40
SteveAokay03:40
SteveAi commented them out so that pqm wouldn't maybe fail03:40
salgadocan't find what could be wrong, as the requests are always ordered by the date requested03:41
SteveAmaybe there are two things with the same date03:41
SteveAare they ordered by id also?03:41
salgadono, they're not03:41
salgadoI guess that would solve it03:42
salgadobut first I want to make sure what's going on03:42
SteveAso, make the test return the name and the date03:43
SteveAthen you'll be able to see03:43
salgadoI just did that. and yes, there's three things with the same date03:44
SteveAgreat03:45
SteveAtvarka03:45
SteveAyou might want to either wait for PQM to process my latest stuff03:45
SteveAor merge from steve.alexander@canonical.com/launchpad--Navigation--0--patch-3003:45
SteveAbefore you send that to pqm03:45
salgadoI'll wait for pqm. how long since you sent your branch there?03:46
SteveAit will arrive 55 minutes in the future03:47
SteveAat Fri Oct 7 14:39:15 2005 UTC03:47
salgado7 minutes. that probably means at least 30 more minutes to run all the tests03:47
SteveAlifeless: the times on http://pqm.ubuntu.com/ are 1 hour in the future03:47
salgadothat should be enough time for me to poke kiko and get my shipit-searching branch reviewed03:47
lifelessSteveA: oh, are they GMT not UTC ?03:47
salgadokiko, that's an easy one. ;)03:48
lifelessSteveA: erm, BST I guess I mean03:48
kikosalgado, do you really want to land that today? I was thinking we should focus on getting any trivial fixes landed first03:48
kikoalso03:48
SteveAlifeless: the page says UTC03:48
kikoI need your help on a frigging merge crapola that baz inflicted on me03:48
lifelessSteveA: yes, but what is it really ?03:48
kikoit nuked cronscripts/shipit-exports.py, salgado 03:48
kikocan you believe it?03:48
SteveAbut if you remove UTC and say BST then they will read right, at least today03:48
SteveAno idea about after DST changes03:48
lifelessSteveA: ok, so its getting localtime not UTC.03:48
lifelessSteveA: can you file a bug on pqm in launchpad for me ?03:49
salgadokiko, of course I can!03:49
kiko#@!#!$!@!!@*&03:49
SteveAlifeless: done03:50
lifelessSteveA: thanks03:50
lifelessSteveA: now I will ignore that for a year or so03:51
SteveAyeah, and one hour03:51
lifelessSteveA: and then when you complain next time dst comes around03:51
lifelessI'll be enthused and work on it03:51
SteveAi'll ask elmo to set the local time on that box to NZ03:51
lifelessROTFL03:51
=== SteveA --> lunch
jameshNZ time is a fair bit different to Sydney time03:52
lifelessjamesh: but so much more accurate03:52
lifeless:)03:52
jameshlifeless: UTC+14:00?03:52
lifelessjamesh: +1203:55
jameshlifeless: I met some kiwis pushing for 2 hours daylight saving when I went over there :)03:55
lifelessjamesh: city folk :P03:56
jameshthey wanted 1 hour DST in the winter and 2 hours in the summer03:56
lifelessnight all04:00
=== sabdfl [n=mark@217.205.109.249] has joined #launchpad
mptcarlos: ping04:07
carloskiko, ;-)04:23
carlosmpt, pong04:23
mptcarlos, let's talk about https://launchpad.net/products/rosetta/+bug/203604:24
=== mpt thinks we should have Ubugtu in here
kikoit's the unbelievable sabdfl!04:24
kikoskin and bones!04:24
carlosmpt, ok04:24
mptcarlos: What templates are we currently showing, and why?04:24
kikoUbugtu?04:25
carlosmpt, we are showing all templates that distribution has that have at least one translation 04:25
carlosmpt, the idea is to show all of them04:25
carlosthe fact that some are missing is a bug04:25
mptHow can they ever get from 0 translations to 1 translation?04:25
carlosmpt, because you reach them using another paths or a new .po file is imported04:26
mptcarlos: other paths such as what?04:26
carlosmpt, for instance /rosetta points to many of those translations04:27
mptok04:27
mptcarlos, why are you hiding the ones with 0 translations in the first place?04:28
carlosmpt, and as soon as gina is executed on production distros/ubuntu/breezy/+sources04:28
carlosmpt, as I said, it's a bug 04:28
carloswell, I guess it's a bug04:28
carlosI doubt mark had any interest on hidding them04:28
mptok04:29
mptso there's nothing really that I need to design here04:29
mptcarlos, one more question: I'm looking at https://launchpad.net/distros/ubuntu/hoary/+lang/hi04:30
carlosmpt, a good way to prevent that that page sucks so much04:30
mptcarlos: I can't work out what order the templates are listed in04:30
carlosmpt, no order at all04:31
carloswe need to add an orderBy to that query04:31
mptok, I'll report that bug04:31
carlosI doubt it has it04:31
carlosI think we have that one already04:31
mptI don't see it04:34
mptwhen searching for "order" or "sort"04:34
mptthere's bug 20, but that's not quite the same04:35
carlosok, then file it, please04:38
=== grayman [n=grayman@85-64-195-124.barak-online.net] has joined #launchpad
mptdone, bug 293804:42
=== mpt blows a ladybug off his keyboard
carlosmpt, thanks04:47
=== camilotelles [n=Camilo@fireincub.fumsoft.softex.br] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  lots of tidying up.  converting all database classes to use NotFoundError consistently, and to import it from launchpad.interfaces in preparation for the move to a new zope3.  Also, introduced a NameNotAvailable error.  removed browser:traverse rdirective.  commented out shipit test that fails sometimes. (patch-2600: steve.alexander@canonical.com)04:57
mptbradb: will your package nameage get into the next production update?05:04
=== zorglub_ [n=53915de2@wahe.diwi.org] has joined #launchpad
bradbmpt: I think so, since I merged it before what I understand to be the cutoff date.05:05
zorglub_hello, I simply haven't found how to edit a bug's details (severity, ...)05:08
bradbI have to add back that link, this is just too silly. ;)05:08
bradbzorglub_: you have to click on the package name05:08
zorglub_ahah ok05:09
mptbradb: or restore the underlining and the bug icons, like I wanted to do05:09
=== mpt grates his teeth
bradbmpt: that doesn't help much, IMHO05:10
mptYou haven't seen it05:10
bradbi can imagine it though05:10
bradbmpt: even with a neon sign saying "THIS IS CLICKABLE", it still gives basically no hint whatsoever that that's where you're supposed to go to edit the assignee/status details05:11
bradbparticularly when seen through the eyes of someone who isn't that familiar with Malone05:11
mptSure, the controls should already be on the page you're looking at05:12
=== salgado preferred the [Edit] link
salgadothan having to click on the package/upstream name05:12
bradbmpt: yeah, that would be even better. ;)05:12
mptbut any design that's repeating the same word over and over again is wrong05:12
bradbsalgado: [Edit]  was shown to be invisible, in user testing05:12
bradbbecause [Edit]  still doesn't say "this is what you click to edit the assignee/status details"05:13
salgadoindeed, but the package name doesn't do that either05:14
bradbsalgado: correct :)05:14
sabdflwe should have a mini-icon that means "edit", and use it liberally.05:15
mptsabdfl, we do have an edit mini-icon (/@@/edit.gif)05:20
dilysMerge to rocketfuel@canonical.com/cscvs--devel--1.0: [r=spiv]  python high bit filename filtering (patch-113: david.allouche@canonical.com, robert.collins@canonical.com)05:20
mptI think the problem is that having to go to a separate page at all is so unnecessarily different from (and less efficient than) other bug trackers, that even that might not make it obvious05:20
mptif "[Edit] " didn't work, I doubt an icon for the same would work.05:21
sabdflmpt: until the edit form controls can be made tight enough to fit in the current task headline bar, we will retain a separate page, 'k?05:21
mptsabdfl: sure, I'm not allowed to change the page at all, so need for more specific restrictions :-)05:22
mpthooray, mirror finished05:23
=== Kinnison [n=dsilvers@haddenham.pepperfish.net] has joined #launchpad
Kinnisonre05:27
sivanghey Kinnison 05:39
Kinnisonhi sivan05:40
bradb"The world will change this week" -- said by a passerby who saw my new bugtarget search portlet05:42
bradbsalgado: Do you have time for a code review in the next hour or so?05:43
=== Virtuall[BZA] [n=virtuall@who.is.virtuall.info] has joined #launchpad
salgadobradb, how big is it?05:48
bradbsalgado: I don't have the diff ready yet, but it's pretty small (it *looks* a lot bigger than it is, because of adding the new portlet in a bunch of different places, but the core code is probably 100 lines)05:49
salgadobradb, I can do it after lunch05:50
salgadoin 1h, aprox.05:50
bradbsalgado: awesome, thanks. i'll send it off in a bit.05:50
mptheh05:56
mptI thought I cleared the status notes, but left behind carlos's word "Cheers"05:56
kiko:)05:58
sabdflbradb: ping06:04
bradbsabdfl: pong06:04
sabdfljust discovered your DistroSourcePackage work06:04
sabdfli have a mostly-fledged DistributionSourcePackage06:05
sabdflhow much work did you do on DistroSourcePackage? which pages are there? url's?06:05
sabdfli'll need to integrate our work06:05
bradbhm06:06
bradbthere's the bug listing06:06
bradb /distros/ubuntu/+sources/mozilla-firefox/+bugs06:06
sabdflddaa: niemeyer joins you on LP next week06:06
SteveAbradb: pqm will shortly be nuking browser/traversers.py06:06
sabdflhas all of that branch work landed?06:07
sabdflSteveA: urgh06:07
SteveAwhy urgh?06:07
sabdfldoes it all migrate to class files?06:07
SteveAyes.  all migrated already06:07
SteveAthis is just rearranging things06:07
SteveAinto the proper modules06:07
SteveAnow, i'm in a position to solve the breadcrumbs problem06:08
sabdflnice06:08
sabdflwith that, and a pillars-of-launchpad tweak or two, our page layout will start to become very usable06:08
SteveAtraversers.py was the last hold-out of "everything from everywhere in one module" in the browser code06:09
sabdflit's already a zillion times better than it was six months ago06:09
bradbsabdfl: basically other than the bug page, there's just little bits of infrastructure (like the *Set, the interfaces) and tests. nothing too serious.06:09
SteveAyeah.  i'm finding launchpad not too bad to use for bugtrackingn etc. now06:09
bradbi did the minimum required to make it work for me06:09
sabdflbradb: is it done, i.e. you don't have more branches that are landing with changes to it?06:10
bradbsabdfl: I'm changing the dsp bug page a little bit now, to make it work with the bugtarget search portlet.06:10
bradbthat should be landed within a few hours06:12
bradbsabdfl: I also added back the menu on another branch (seemed to have gotten blown away during the menus integration.) that patch is awaiting kiko's review.06:13
mdkejordi, ping06:14
=== klaatujr [i=klaatujr@12-217-115-46.client.mchsi.com] has joined #launchpad
sabdflwhat's a dsp url with sampledata?06:20
mdkejordi, unping, i'll email06:21
bradbsabdfl: /distros/debian/+sources/mozilla-firefox/+bugs, IIRC06:22
sabdflbradb: i do like the move to non-table listings...06:23
bradbIt's nice when they aren't that many bugs being displayed, IMHO.06:23
=== klaatujr [i=klaatujr@12-217-115-46.client.mchsi.com] has left #launchpad []
sabdflbradb: i'll tidy up the layout a little06:27
sabdfldrive-by polishing06:27
bradbsabdfl: Can you wait on that for just a bit? You might conflict with me and mpt06:27
=== Kinnison waves sabdfl's drive-by polishing at soyuz
sabdflok06:27
bradbthanks06:28
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  Make sure reloads on the shipit-myrequest page won't blow up in user's face. (patch-2601: guilherme.salgado@canonical.com)06:32
kiko-fudheh06:32
kiko-fudalways nice to see somebody that cares for the end-user's good looks06:32
kiko-fudMIRROR BAZ MIRROR06:33
sabdflbradb: conflicts ahoy on DistroSourcePackage...06:33
=== zyga [n=zyga@2-mi2-1.acn.waw.pl] has joined #launchpad
bradbsabdfl: Maybe it's due to the nav stuff.06:34
sabdflbradb: no, i just have to merge to entirely different "distribution source package" implementations :-)06:34
SteveAthat's touched mainly browser code06:34
SteveAalthough i have made the not found exception stuff better in database code06:35
sabdflbradb: yours should not have been inside files called "packages.py" and "sourcepackage.py"06:35
SteveAcarlos: ping06:35
sabdflplease observe the pattern06:35
SteveAaha!06:35
sabdflit should have gone into browser/distrosourcepackage.py and database/distrosourcepackage.py06:36
sabdflok?06:36
bradbsabdfl: right06:36
SteveAi thought that was odd when i was adding browser navigation code for that 06:36
sabdflwhy do you need the __eq__ and __ne__ methods?06:37
mdkei have two wikinames06:38
mdkeis it because I am special?06:38
bradbsabdfl: so that .target comparisons Just Work, regardless of what kind of object .target is.06:38
sabdflok06:38
mdkedamn, everyone has two06:38
sabdflbradb: remember we talked about the quality of the current release and the whole distro w.r.t. bugtasks?06:39
sabdfli.e. closing a bug in breezy now would also close it in ubuntu?06:39
sabdflis that implemented?06:39
sabdfland in email interfaces?06:39
bradbsabdfl: not implemented yet06:39
sabdflis there a spec?06:39
bradbno that i know of offhand06:39
sabdflcould you create a one-paragraph braindump, register in LP, and put on the UBZ agenda please?06:40
bradbsure, i'll do that now06:40
sabdflthanks06:40
SteveAsabdfl: is that because breezy is the "current dev release" of ubuntu?06:41
SteveAor is that a special case?06:42
sabdflSteveA: yes06:45
sabdflsorry06:45
sabdflcurrent dev release fixing should also fix distro bugtask06:45
SteveAright, figures06:46
zygais there any # for translators06:50
sabdflbradb: ok, looks like we need to do a bugzilla import script...06:51
bradbindeed :)06:51
sabdflwe have about 2,900 open/new/pendingupload bugs on ubuntu06:51
sabdflmalone will easily handle that06:51
bradbyup06:51
=== zyga invites interested people to #ubuntu-translators
bradbsabdfl: the spec i added for the task res. issue is here: https://wiki.launchpad.canonical.com/ResolvingTargetedBugTasks06:53
carlosSteveA, pong06:55
SteveAcarlos: pitti sent an update about language packs06:55
carlosok06:55
carlosbad news... need to investigate a bit, I think that perhaps a reimport should fix that...06:58
SteveAcarlos: let me know what happens with it please07:00
=== SteveA --> home
carlosSteveA, sure07:00
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  nuke browser/traversers.py (patch-2602: steve.alexander@canonical.com)07:07
jordicarlos: fantastic!07:10
jordicarlos: thanks for fixing; ie, loading me with a ton of work :P07:10
carlos;-)07:11
jordimdke: next time, for Rosetta issues, mail rosetta@ so Carlos will read too. This export thing seems something he'll be able to analyse correctly07:15
bradbsalgado: patch sent!07:16
jordicarlos: the ubuntu-doc people are getting a pair of files that don't validate, with duplicate msgids07:17
bradbKinnison: ping07:17
bradbKinnison: From what header is the sourcepackage maintainer set?07:18
Kinnisonpardon?07:19
bradbmaybe i'm asking the wrong person07:19
Kinnisonperhaps you're asking the wrong question07:19
bradbKinnison: do you know how our maintainership data is populated?07:19
Kinnisonno07:20
bradbhm07:20
bradbkiko-fud: do you know how our maintainership data is populated?07:20
kiko-fudvia gina07:22
Kinnisonbradb: What table are you on about?07:22
bradbKinnison: maintainership07:22
Kinnisonbradb: I thought that was a fairly manual process07:23
bradbkiko-fud: Mm, last I asked (though I don't recall who I asked), I was of the understanding that maintainership data comes from the uploads.07:23
kiko-fudbradb, we don't deal in uploads yet -- all our data comes from gina07:23
bradbI guess gina polls that data every so often?07:23
kiko-fudright07:24
kiko-fudit looks at the archive07:24
bradbkiko-fud: what piece of data does gina look at to get the maintainer?07:24
kiko-fudthe releases file IIRC07:24
bradbBecause I'm thinking: is this going to work for inherited packages?07:24
KinnisonI hate to say this, but gina doesn't touch the Maintainership table at all07:25
kiko-fudit should though :)07:25
KinnisonThe archive can't tell us who maintains what07:25
Kinnisonit doesn't always know07:25
KinnisonPlus in Ubuntu we don't always have a specific maintainer07:25
Kinnison(per-se)07:25
bradbso, this field will be manually overrideable, presumably?07:26
bradbi.e. in the web UI?07:26
carlosjordi, I'm aware of that07:27
bradbI have to give a clear answer to Kamion and mdz about how they can set the maintainer in cases where the data we've loaded is incorrect.07:27
kiko-fudKinnison, at least in the old sourcpackage model we set the maintainer correctly07:27
carlosI think they filed a bug and I already answered them07:27
carlosjordi, I think it's related with obsolete messages07:28
Kinnisonkiko-fud: setting the maintainer in the sourcepackagerelease is *NOT* the same as setting a maintainership record07:28
Kinnisonsabdfl: can you chime in here please?07:28
kiko-fudKinnison, it used to be, in the old sourcepackage model.07:28
Kinnisonkiko-fud: it hasn't been for about 9 months07:29
kiko-fudit's not my fault you were born into the 90s07:29
sabdflwe don't have a "make me the default assignee for bugs on this" table07:30
sabdflwe have package subscriptions07:30
sabdflwhich are "cc me on bugs on this package"07:30
sabdflbut it makes sense to add this07:30
=== sabdfl [n=mark@pdpc/supporter/silver/sabdfl] has left #launchpad []
bradbheh07:32
kiko-fudhe hath spoken07:32
bradbThat still leaves the original problem unresolved, it seems.07:33
bradbISTM that the maintainer of the package must be set correctly, however that is done.07:33
kikowell07:33
kikoyou first need to ask yourself07:33
bradbInherited package or not, otherwise the user will be mislead and confused.07:33
kikowho is the maintainer of the package?07:33
bradbkiko: the person responsible for making sure the fix ends up in $distro07:34
kikocarlos, what do you think pitti's mail looks like07:34
jordicarlos: yes, it's not the first time obsolete msgs give us trouble07:34
kikobradb, I'm not sure we know who that person is for all packages.07:34
bradbkiko: We probably don't. But shouldn't there be a way of letting the user manually enter that data?07:34
kikofor packages?07:35
bradbyeah07:35
kikoI guess -- it's not a very difficult change07:35
kikobut I'm not sure that's what maintainership is07:35
carlosjordi, I know, it's a know bug07:37
jordicarlos: got a bug number?07:37
bradbto me "default assignee" is a way of saying "i maintain this thing (even though it may be a non-forked package from an upstream distro)"07:37
kikobradb, is that the same as maintainership? Kinnison?07:38
bradbit's a bit impure perhaps, but in the context of launchpad, i think it is.07:38
bradbi'm just guessing though. i don't have enough user input to do any more than that.07:40
kikome neither. tell you what: is there anywhere in our code that currently uses that table?07:41
bradbyup, bugmail07:41
kikoanywhere else?07:41
kikoif not -- dude, go for it!07:41
kikoI'd rename it perhaps07:41
bradbgo for it, i.e. make it editable?07:42
kikouse it to your advantage!07:43
carloskiko, I think the problem is that we rejected many pofiles when we started with Rosetta and Hoary so it's a matter to review them by hand07:43
carlosjordi, https://launchpad.net/products/rosetta/+bug/262107:50
kikocarlos, but dropped individual translations -- is that because the template is out of sync?07:50
carloskiko, either the potemplate is out of sync or we missed some imports for Hoary07:51
carloskiko, if breezy is not losing translations, hoary should not do that either07:51
carloswith breezy, every day we get new uploads07:51
carloshoary is frozen already07:51
carlosso we need to force a new import to fix any issue we had07:52
=== Kinnison preps to run away
kikoSteveA, will you STOP CONFLICTING WITH ME?!07:58
kikoit's the 4th time I request a merge!07:58
kikocarlos, I don't know if I understand exactly what you mean08:00
kikooh08:00
carloskiko, we did many fixes since the Hoary release08:01
kikocarlos, so we only include in the pofile translated strings -- no blanks?08:01
carlosso many .pot and .po files that were failing to import at that time will work now08:01
carloskiko, ? I don't get your answer, sorry08:02
kikoit was a question :)08:02
kikothe pofile, it doesn't include blank translations, right? it only includes msgid/translation pairs for translation != ""?08:02
kikothe dropped plural form is weird08:02
Kinnisonciao08:02
carloskiko, right, was a question, sorry08:05
carloskiko, no, we have them too, but to compare the rosetta output martin removes them08:06
carloskiko, about the dropped plural form, it's normal as long as we don't have any msgid_Plural08:06
carlosI need to debug it to be 100% sure that it's the case, but I have a test for that and I doubt it's incorrect (but all things are possible with real data...)08:07
kikoah, I see.08:08
kikoyeah, you learned the language.08:08
bradbsalgado: did you get my patch?08:10
salgadobradb, yep. reviewing it now08:11
bradbawesome08:11
jordicarlos: if mdke does an upload of a single po file and there are some strings that differ between the rosetta data and the po file he's uploading, what takes precedence, and what gets relegated to the translation memory?08:12
mdkei was thinking more of strings that are translated in rosetta, and not at all in the po file08:12
jordioh, then rosetta should be smart.08:13
carlosjordi, if it's a published upload, Rosetta has preference if it's not a published upload, the .po has preference08:14
jordicarlos: good to know08:14
jordicarlos: and if the translations are missing in the upload, but are in rosetta, they get filled up, right?08:14
carlosjordi, they will be there, yes 08:16
carlosor at least they should O:-)08:16
jordihehe08:18
kikosalgado, ping?08:24
salgadokiko, pong08:25
bradbmpt: I'm attempting to look at your Malone front page changes now08:25
kikosalgado, you know person.activememberships and person.myactivememberships?08:25
=== jinty [n=jinty@205.134.224.215] has joined #launchpad
salgadokiko, yes08:26
kikowell08:26
kikothe names kinda suck08:26
kikoI wanted to know08:26
kikois it possible to add some asserts to them08:26
kikoto ensure you ran them on the right thing08:26
kikofor a person it doesn't even make sense to run activememberships, does it?08:26
salgadono, it doesnt. I even thought about merging them into a single method and act according to what self is08:27
kikohmmm08:27
kikotricky08:28
kikobecause a team can have activememberships and myactivememberships08:28
kikookay08:28
kikoI'll just add a note08:28
kikoddaa, ping?08:37
kikocarlos, ping08:44
bradbmpt: hm08:45
bradbmpt: you completely blew away the portlets from the front page? :)08:45
carloskiko, pong08:45
bradbmpt: This also assumes that global searching will work08:46
kikocarlos, https://launchpad.net/products/rosetta/+bug/109808:46
kikois it still relevant?08:46
carloskiko, could be I didn't fix it, perhaps it was fixed as a side effect of another fix...08:49
carloswell, I wrote code before that bug that should prevent that error to happen, but seems like I did a mistake08:49
kikoit's an old bug08:50
carlosnow that the language packs are more or less on their track I'm caring much more about bug reports so next week I will try to handle all big bugs and the permissions ones from Jordi08:50
carloskiko, I know 08:50
kikookay08:50
jordiOn the Translations page of a new project:08:50
jordi"If you would like to help translate Silva then you should ask the08:50
jordiRosetta team to set it up by sending an email to the Rosetta mailing08:50
jordilist. You should give them the URL to the Silva.pot and .po files. You08:50
jordiwill be able to start translating once they are imported."08:50
=== RWG [i=RWGraphi@69.177.14.20] has joined #launchpad
jordicarlos: can we change this to suggest that the URLs are a tar.gz with all the files?08:51
bradbKinnison, kiko: do you know if our data model supports finding out if a package is 1. inherited (i'm guessing almost certainly yes) and 2. no different to the original package? can gina be made smart enough to make that distinction?08:52
kikoin theory yes08:52
carlosjordi, sure08:52
carlosjordi, file a bug and set me as the owner and set the urgency to high08:53
jordicarlos: thanks08:53
jordik08:53
bradbkiko: hm. i wonder if there should be a distro default maintainer for inherited packages that aren't forked?08:53
RWGCarlos Marin08:53
RWGCrad Bollenback in Canada08:54
RWG*Bollenbach08:54
RWG*Brad08:54
carlostime to leave...08:56
carlossee you!08:56
carlosjordi, do you need anything from me08:56
RWGMy fridge is rampaging in the next door neighbor's garden; I need to go cheer it on.08:56
=== RWG [i=RWGraphi@69.177.14.20] has left #launchpad ["Connection]
jordicarlos: don't think so08:57
jordicarlos: have a nice weekend dude08:57
jordicarlos: only that I duoubt I can modify severity and owner08:57
carlosjordi, ok, don't worry08:59
carlossee you08:59
jordihttps://launchpad.net/products/rosetta/+bug/294709:00
carlosthanks09:00
jordiheh09:03
jordiI *rule* the karma top 5.09:03
kikoddaaaaaaa09:06
ddaathat won't highlight my nick09:06
kikotwo things09:06
ddaaI will have to leave in a few minutes09:06
kikoa) I did a baz replay --reverse and committed. is that okay? I ended up with an UNKNOWN patch in my history. Will PQM hate me?09:07
kikob) Are you going to look into bug 2915?09:07
ddaaIIRC that tends to make log-for-merge unhappy, wich causes noise in rockefuel merge log09:08
kikoforever?09:08
ddaayou should do sync-tree after replay --reverse to restore the patchlog09:08
kikodarn09:08
ddaauntil somebody fixes rocketfuel09:08
kikois it too late for that now?09:08
ddaaI do not remember the specifics, it's not a critical problem anyway09:08
kikocan I go on working on the branch?09:09
=== kiko prays for pqm
ddaakiko: 2915 is FIXED!09:09
kikooh09:09
ddaaepiphany is removed from the GNOME project, epiphany-browser is now in the GNOME project09:10
kikothanks09:10
kikoyou're so sweet09:10
ddaakiko: I think you should restore the patchlog for the reversed patch before sending a merge to rocketfuel09:10
kikoI will try09:11
jordikiko: got a min?09:11
kikojordi, sure09:11
ddaabut I might be confused, I remember there used to be a problem, but I'm not sure exactly what it was09:11
bradbkiko: how's it looking for that patch to get reviewed? it'll make your triaging life a lot easier. ;)09:11
jordikiko: I've got a request to add a Shona team.09:11
jordihttps://launchpad.net/people/shona09:12
bradbkiko: and it conflicts with the patch i sent to salgado, so i hoping to get them both sorted out and landed today09:12
jordibut this dude has messed up things it seems.09:12
kikoyeah09:12
kikojordi, what is the issue?09:12
=== ddaa leaves to watch Voyager
jordishould we rename this "shona" thing to ubuntu-l10n-shona, and when someone else joins, switch from shepard to this team for ubuntu translators?09:13
jordikiko: we need to add shepard to ubuntu translators for the Shona language.09:13
jordishona = sn09:13
kikowe can rename it now and update the ubuntu translators. are you ok with that?09:13
kikojordi?09:14
jordiwrong, it's sna it seems09:14
jordiyes09:14
kikoso ubuntu-l10n-shona or -sna?09:15
jordibut, per sab policy, sheaprd should be enabled in ubuntu translators, not the team09:15
jordinot until there are more people in the team at least09:15
jordior we can make an exception, now that the team is already created anyway09:15
jordiwhatever you think is better09:15
jordi-sna09:15
kikoright09:16
kikoexceptions-r-us09:16
kikooh-oh09:16
jordino no no09:16
kikoit appears I can't rename it09:16
kikosalgado, how do I change a team's name?09:17
jordithey do have a two letter code "sn"09:17
jordikiko: make it "sn"09:17
kikook09:17
kikobut I can't rename the team09:17
kikoso no luck yet09:17
salgadokiko, you can't. someone droped that09:17
kikoah09:17
kikothat's what neuman's patch did09:17
kikojordi, can you remind me to rename it next week?09:17
jordikiko: yes. Should I mail you?09:18
salgadokiko, yes. I asked him to do that09:18
kikonext week, yes09:18
jordimail you now, so you have it in your queue?09:18
kikoif you email me now, jordi, it will get lost09:18
jordiok. let's see how I do to remember. :)09:18
kikoput an email in YOUR inbox :)09:18
kikoshephard appointed for ubuntu-sn09:19
kikohttps://launchpad.net/rosetta/groups/ubuntu-translators/09:19
kikoProgrammingError: ERROR: canceling query due to user request09:20
jordishouldn't that be "cancelling"?09:24
dilysMerge to rocketfuel@canonical.com/cscvs--devel--1.0: [r=spiv]  use svn_oo.util.pysvnClient instead of pysvn.Cleant, tweak inventory for bzr (patch-114: david.allouche@canonical.com)09:24
jordikiko: many thanks09:24
jordikiko: I have the same mail problem as you do I'm afraid09:25
jordiI'm slowly learning to flag CRITICAL stuff09:25
jordicritical = gets you fired if ignored :)09:25
bradbor "severity is meaningless", depending on who you talk to09:26
bradbin fact, we have a spec proposing to get rid of it09:27
jbaileybradb: Err. I trust that the spec talks about current use cases and what replaces them? =)09:36
kikoare there valid use cases for severity?09:36
bradbkiko: dude, my patch!09:37
=== bradb nags salgado too, while he's at it
bradbjbailey: not in detail, but in summary09:37
jbaileykiko: I use severity to know which bugs I need to fix before I go home, (critical), and which bugs I need to fix before release (major).  Trivial is cosmetic bugs that I can do when I don't feel like working and enhancements are things that probably need specs to go along with them.09:38
jbaileykiko: Everything else is normal.09:38
bradbjbailey: What do you use priority for?09:38
kikojbailey, that sounds like priority tome.09:38
jbaileybradb: I don't. =)09:38
kikojbailey, then you're misusing priority to represent severity09:38
kikoseverity is something else.09:38
kikoerrr09:39
kikothe opposite of what I said three sentences above.09:39
bradbjbailey: It sounds like you're using the thing that describes what effect the bug has on the user to prioritize fixing it09:39
jbaileyPriority tells me which ones I would need to do first.09:39
jbaileyI should probably prioritize the normal bugs.09:39
kikono09:39
kikowell09:39
kikonot exactly :)09:39
kikoseverity is how severe the bug is09:39
jbaileyI'd rather give priority to my boss to tell me what order he'd like things done in.09:39
kikoif it's a showstopper, critical, world-crashing issue 09:40
kikoseverity doesn't really make sense for features and enhancements09:40
kikopriority indicates what order you should fix them09:40
kikoand you're right, priority is something that's more of a management thing09:41
jbaileyI'd love mdz or people dealing with community etc to let me know which bugs people care about more.  In bugzilla land, it could even be through the vote system.09:42
=== bradb is warming up to the voting system
jbaileyUltimately I need to know what *must* be done today.  Then I need to know what I ought to do tomorrow when this is out of the way.09:43
jbaileyBasically, there will always be more bugs than time, so I think that's really what I'm looking for.09:44
bradbjbailey: we'll give you that09:44
kikojbailey, what must be done today is P109:45
kikothen you can order the rest using P2-P509:45
kikoor alternatively09:45
kikoyour manager can order it09:45
jbaileyRight.09:46
jbaileyThe complication is...09:46
jbaileySome tasks are trivial and easy to get out of the way.09:46
jbaileySo should be a higher priority so that we still take care of the little details.09:46
kikoright09:47
kikoso smaller, cheaper tasks should get higher priority09:47
kikoit's a traditional scheduling problem :)09:47
jbaileyYup =)09:48
jbaileyAnd I hate to say it, but it would be nice if bugs from people with higher karma got a boost. =)09:48
bradbThat would make Rosetta users happy.09:50
bradbKarma: 293828209:51
kikowho's that?09:53
bradbnoone, but rosetta's karma system is wacked ;)09:54
jordibradb: could be me ;)09:58
jordicurrent karma is nearly 20k09:59
bradbheh09:59
jordiheheheheheh bradb has 813. :)09:59
bradbthat's pound sterling09:59
mdzkiko,jbailey: things come in as P2; I bump them up to P1 if they're particularly important10:04
mdzjbailey: currently, what's important is the 5.10 milestone report I posted to -devel10:04
=== Ubugtu [n=bugbot@www.binaries4all.com] has joined #launchpad
Seveasbug 100010:11
UbugtuMalone bug #1000: There are too many bug reports in Malone Fix req. for: upstream launchpad, Severity: Normal, Assigned to: Nobody, Status: Rejected http://launchpad.net/malone/bugs/100010:11
Seveasmpt, see it works :)10:11
mptgreat, thanks Seveas10:12
Seveasyw10:12
mdkenice one10:13
bradbcool10:13
kikoSeveas!10:14
Seveaskiko?10:15
kikothat rocks10:15
Seveasit knows the ubuntu and gnome bugzillae too if neccessery, justreplace 'bug' with gnome or ubuntu10:16
jordikiko: where does rosetta get the name for a language?10:16
Seveasbugzilla/malone urls are triggers too10:16
jordiAsturian is incorrectly called "Asturian; Bable", when it should be "Asturian".10:16
kikohmmm10:16
kikojordi, I think that requires a database patch, file a bug10:16
jordikiko: done.10:18
jordispoke too fast10:18
jordiPRogrammingError10:18
jordihttps://launchpad.net/errors/showEntry.html?id=1128716317.920.88850478234710:19
jordiI don't know if its related10:20
jordibut I changed the title from10:21
jordiIncorrect language name for "ast": "Asturian; Bable"10:21
kikojordi, request expired right?10:21
kikowhat's the bug #?10:21
jordiI don't know10:21
jordinow it got filed as 295110:21
jordiafter changing the title from "Incorrect language name for "ast": "Asturian; Bable"10:21
jordifuck10:21
jordidamn10:21
jordiIncorrect language name for "ast": Asturian; Bable10:21
jordito10:22
jordi"Incorrect language name for "ast": "Asturian; Bable"10:22
jordiie, added quotes.10:22
kikomatsubara, I'm still waiting for your test -- am I going to get it today?10:22
jordikiko: it didn't take long to show that error10:22
jordia few seconds10:22
kikoyeah, it's weird10:22
jordiare you ok with what you have, or should I file a bug?10:23
salgadobradb, review sent. sorry for the delay10:34
kikoit's okay10:36
kikoa timeout, I think, jordi 10:36
bradbsalgado: cool, thanks10:36
jordikiko: good10:40
jordidinner time. Have a nice weekend folks.10:40
jbaileymdz: Ah cool, I didn't know that you used the Priority flags.  I hadn't been using them, so I uncluttered my display.10:44
=== cprov -> whyyyy baz status get so long in a just switched tree ? 5 min !!
=== Ubugtu [n=bugbot@81.171.100.21] has joined #launchpad
bradbsalgado: I've got a reply coming up in about 5 mins...11:16
bradb(with view tests, etc.)11:16
kikobradb, sent too11:17
bradbkiko: thanks11:18
bradbi won't get to that one until monday...conflicts, etc.11:18
=== palmTree [n=chatzill@213.210.238.185] has joined #launchpad
bradbsalgado: reply sent11:23
=== zygis [n=zygis@84.46.151.241] has joined #launchpad
dilysMerge to rocketfuel@canonical.com/launchpad--devel--0: [trivial]  mention gettext in About Rosetta page (bug 712) (patch-2603: mpt@canonical.com)11:26
UbugtuMalone bug #712: Rosetta doesn't mention gettext Fix req. for: upstream rosetta, Severity: Normal, Assigned to: Matthew Paul Thomas, Status: New http://launchpad.net/malone/bugs/71211:27
=== asmodai [n=asmodai@sakko.demon.nl] has joined #Launchpad
mptdilys, meet Ubugtu. Ubugtu, dilys. I'm sure you'll be very happy together.11:30
salgadobradb, back to Mac OSX?11:34
bradbrunning tiger on this machine, but ssh'ing into my other powerbook to do lp work11:34
salgadoI noticed when got this html email11:36
bradbwha?11:36
bradbshit11:36
bradbchanged11:37
bradbthat's an embarassing default. i'm suitably humiliated.11:37
salgadobradb, I had the feeling the only way to test that would be with something like ClientForm, but asked just to make sure11:39
bradbok11:39
salgadoanyway, the tests looks fine11:39
bradbshould i doit?11:40
salgadoyou mean, the merge? sure!11:40
bradbthanks11:40
ddaacprov: you want the technical answer, or the real answer?11:47
=== mgalvin [n=mgalvin@cpe-69-205-47-165.nycap.res.rr.com] has joined #launchpad

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