/srv/irclogs.ubuntu.com/2006/02/18/#launchpad.txt

=== SteveMyers [n=SteveMye@255.26.118.70.cfl.res.rr.com] has joined #launchpad
=== ntsjk [i=ircap751@eu85-86-12-171.clientes.euskaltel.es] has joined #launchpad
ntsjkzein12:07
=== ntsjk [i=ircap751@eu85-86-12-171.clientes.euskaltel.es] has left #launchpad []
=== SteveMyers [n=SteveMye@255.26.118.70.cfl.res.rr.com] has left #launchpad ["Leaving"]
=== mpt_ [n=mpt@219-89-158-224.jetstart.xtra.co.nz] has joined #launchpad
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has joined #launchpad
=== spiv [n=andrew@adsl-66-203.swiftdsl.com.au] has joined #launchpad
CarlFK"Patch                (Required)"01:04
CarlFKrequired? ! ;)01:04
jameshCarlFK: it is required that you either leave the checkbox checked or unchecked :)01:19
mpt_Goooooooooooooooood afternoon Launchpadders!01:29
mpt_jamesh, oh boy, that's a bug01:29
CarlFKheh01:39
=== stub [n=stub@gb.ja.97.6.revip.asianet.co.th] has joined #launchpad
=== jsgotangco [n=jsg@ubuntu/member/jsgotangco] has joined #launchpad
=== mpt_ adds that to https://wiki.launchpad.canonical.com/FormLayout
=== mpt_ is now known as mpt
mptI can never change the people of a spec01:49
mptit always times out01:49
mptwhoa, I take that back01:50
mptit took about 25 seconds, but it worked01:50
=== poningru [n=poningru@n128-227-13-13.xlate.ufl.edu] has joined #launchpad
=== dous [n=dous@efreet.edu.ms] has joined #launchpad
=== xhaker [n=xhaker@213.201.220.218] has joined #launchpad
=== jsgotangco [n=jsg@210.4.38.43] has joined #launchpad
=== ulinskie [n=yolynne@202.57.88.34] has joined #launchpad
jameshso, MultipleJoin is really bad for pretty much any case where we'd get lots of results03:46
=== ulinskie is away: visit wahoy.com, zamboanga's free online classified ads
spivjamesh: Yep.04:00
spivjamesh: newer versions of SQLObject have SQLMultipleJoin or some-such that are better.04:01
jameshspiv: I noticed that.  I wonder why you'd ever want to use MultipleJoin instead of SQLMultipleJoin though?04:01
spivjamesh: The only reason I can think of is backwards compatibility.04:02
jameshif the aim is backward compatibility (i.e. don't make something start returning a SelectResults when it used to return a list), then it would probably be better to make MultipleJoin just convert the SelectResults to a list04:02
jameshso you still get one query04:03
spivjamesh: So for MulipleJoins that are hurting us, we can either manually replace them with properties that generate the appropirate SelectResults, or perhaps look at borrowing the newer joins from upstream.04:08
spivI think there's already one or two places where we've taken the ad hoc property road.04:08
jameshI converted a few MultipleJoins to SelectResults properties, but that wasn't performance related04:11
jameshthat was where we had two classes implementing the same interface but one returning a list and the other returning a SelectResults04:12
jameshmaybe we should look at upgrading our sqlobject04:19
spivDefinitely.04:19
spivThere's a bit of pain, because we've diverged a little bit.04:19
spivAnd upstream has rearranged a lot of internals for "class sqlmeta:", so most of our patches need work to keep them applying.04:20
stubAre our patches suitable for applying upstream? I can't remember much of what local changes we rely on.04:21
spivSome are, some aren't.04:21
jameshthe Table.selectFirst() and SelectResults.__nonzero__() stuff might be appropriate04:22
spivE.g. our postgres-specific unicode stuff isn't.04:22
spivselectOne got a mixed reception on the list, but I think if we synced it up to current upstream, they would probably take it.04:22
spivThey didn't like SelectResults.__len__ when I proposed that :)04:23
jameshdo you think they'd like selectFirst()?04:23
jameshI can understand them not liking __len__()04:23
spivI'm not sure, it's a relatively uncommon use-case, so it might be hard justify.04:23
spivI think there's a reasonable argument to be made that there's something inelegant about having lots and lots of select* variants, although I can't think of any obviously better alternatives.04:24
mptmeh04:29
mptAnyone here an expert on wpasupplicant? My Internet connection was working on Friday...04:30
ajmitch_mpt: and what broke since then? why is it wpasupplicant?04:31
mptok, maybe I'm misattributing blame04:31
mptWhat broke is that I have no Internet connection, no DNS, nothing04:32
ajmitch_brave/silly enough to be running dapper?04:32
mptno, 5.1004:32
ajmitch_hm04:33
mptand I know it's not the AP, because if it was, you wouldn't be able to see what I'm typing on my Mac04:33
jameshtry turning off WPA :)04:34
mptIf I did that, I'd break everyone else's Internet connection04:34
mptor setup, anyway04:34
ajmitch_ah yes, shared flat connection :)04:34
jameshspiv: looks like upstream sqlobject has SelectResults.getOne()04:39
jameshspiv: so you'd do Table.select(...).getOne()04:39
spivjamesh: Hmm, instead of selectOne, it seems.04:40
jameshyeah04:40
jameshactually, you'd do Table.select(...).getOne(None)04:41
jameshgetOne() will raise SQLObjectNotFound if you don't give a default04:41
spivIt would be good to get closer to upstream.04:42
jameshI suppose the other big change we've got is the set operations stuff for SelectResults04:43
spivYeah.04:43
spivThat's suitable for upstream, I think.04:43
spivAlthough not necessarily portable to all backends, which may be a problem.04:44
mptwell, that's progress04:44
mptdhclient works now, but still no Internet04:44
spivlifeless: what's up with pqm.ubuntu.com?05:09
stublifeless: star-merge sftp://chinstrap.ubuntu.com/home/warthogs/archives/stub/psycopgda/devel/ sftp://chinstrap.ubuntu.com/home/warthogs/archives/rocketfuel/psycopgda/3.0/test05:14
spivstub: pqm appears to be down... do you have access to it?05:21
stubyes. pqm isn't down, but the web interface maybe05:21
spivThe web interface certainly is :)05:22
spivAnd because it's not on chinstrap anymore, I can't check if it's alive with ps anymore :)05:22
stubwould have been the weekend sheduled outage05:22
spivYeah, I expect so.05:22
spivWell, I'll just be patient then.05:23
lifelessspiv: up05:29
spivlifeless: Thanks05:29
spivHmm, where did my branch merge request go...05:29
spivlifeless: I sent a request to pqm@, Message-ID: <20060213040821.GB10611@home.puzzling.org>, but have got no reply, and the web interface says the queue is empty.05:31
lifelessspiv: bzrlib.errors.UnrelatedBranches: Branches have no common ancestor, and no base revision specified.05:31
lifelessI got a traceback05:31
spivlifeless: Oh, heh.05:32
=== stub can't work out how to fire up the web ui
spivlifeless: Why didn't I get that traceback?05:32
lifelessstub: cd pqm; twistd -oy pqm/ui/twisted.py05:32
lifelessstub: I need to make an init script at some point05:32
lifelessspiv: unhandled exception05:33
lifelessspiv: to debug this, try doing the merge on chinstrap - make a branch from the rf psycopgda and merge into that05:33
spivlifeless: Oh, I know what my snafu was.05:33
spivI just want pqm fixed so I don't have to ask to find out what my embarrassing snafu was :)05:34
lifelessspiv: oh, it will be.05:34
lifelessI have a note now05:34
stublifeless: psycopgda was my request. I just wanted you to toss that at pqm since it tells me I don't have permissions :)05:38
stub(or I could just do it manually on balleny)05:38
spivlifeless: So, buildbot merge still doesn't work :)05:39
lifelessstub: if you could do it manually right now that would be great.05:41
lifelessspiv: details details details05:41
spivlifeless: are in the mail :P05:42
spivlifeless: (I'm keeping launchpad@ in the loop this time)05:43
spiv(or would, if I could spell "canonical"... ahem)05:43
jameshspiv: in the upstream sqlobject, SelectResults.__iter__() seems to evaluate every row in the result set before iterating :(05:49
spivjamesh: Yuck :(05:49
jameshreturn iter(list(self.lazyIter()))05:49
jameshhas a comment saying that it could be optimised05:50
spivAhem.05:50
jameshthe SelectResults.lazyIter() looks like the __iter__() in our version05:52
spivI wonder what tests will break (if any) if I make the obvious change...05:52
=== ajmitch__ [i=ajmitch@port169-55.ubs.maxnet.net.nz] has joined #launchpad
=== ajmitch__ is now known as ajmitch
=== catnip [i=meowmix@asnet01-1-165.aus.datafoundry.com] has joined #launchpad
=== mpt [n=mpt@219-89-158-224.jetstart.xtra.co.nz] has joined #launchpad
spivjamesh: So, replacing "return iter(list(self.lazyIter()))" with "return self.lazyIter()" doesn't break any SQLObject tests in SVN (although two tests are already broken...)06:15
spivBut then SQLObject only has 167 tests all up.06:15
jameshspiv: I wonder why they changed things?06:16
spivjamesh: r946, log message of "Made select results aggressively fetch objects, instead of lazily"06:19
jameshthat doesn't help much :)06:19
spivNo, not really :)06:20
spivI'm trying to see if any other files changed in that revision, but my svn-fu is too wek.06:20
spivweak, rather.06:20
jameshsvn diff -r 945:94606:20
spiv(and "svn log -r 946" doesn't work for some reason I don't understand, and the cryptic error message doesn't help)06:20
spivIt's things like this that make me feel a little less worried about the little warts that bzr still has :)06:21
jameshonly other change appears to be in docs/News.txt explaining that it now does something different :)06:21
spivI wonder why.06:22
jamesh"When iterating over select results, a list is now immediately created with the full list of instances being selected."06:22
spivI suppose it's meant as an optimisation for the expected "average" use of SQLobject.06:22
spivIf the result set is small enough, and is going to be entirely consumed, fetching it all at once probably is a small win.06:23
spivBut I don't think I like the tradeoff.06:23
spivAnd I definitely don't like the lack of tests :)06:23
jameshI wonder if it is a transaction isolation type issue06:23
spivThe history of that function has flip-flopped on this, I think.06:24
jameshor to cover some issue of iterating over a result set while modifying those rows06:24
spivI vaguely recall there used to be a comment that actually justified the call to list(...)06:25
spivOr rather, iter(list(...)))06:25
spivjamesh: See dbconnection.py:635 in our sqlobject snapshot06:27
spivWhich sort of does the same thing, and says:06:27
spiv        # We can't keep the cursor open with results in a transaction,06:27
spiv        # because we might want to use the connection while we're06:27
spiv        # still iterating through the results.06:27
spiv        # @@: But would it be okay for psycopg, with threadsafety06:27
spiv        # level 2?06:27
spivSo maybe it is to allow "for result in Foo.select(...): do_another_query(result)"?06:28
spiv(In which case there should be a test for it, dammit!)06:28
=== mpt_ [n=mpt@219-89-158-224.jetstart.xtra.co.nz] has joined #launchpad
=== jsgotangco [n=jsg@210.4.38.43] has joined #launchpad
=== mpt__ [n=mpt@219-89-158-224.jetstart.xtra.co.nz] has joined #launchpad
mpt__whoo, I now have DNS06:52
mpt__TCP here we come06:53
=== mpt__ is now known as mpt
stubAverage use? Toy use more like.07:00
stubAlthough for us, if we ever actually want to call __iter__ on a SelectResults containing more than 100 items or so then our code is broken.07:02
stubCause that would indicate  performance issues on the webapp. Batch jobs might have use cases, but I suspect that it would just be asking for memory bloat problems.07:04
jameshstub: we'd want to for /distros/ubuntu/+allpackages07:04
jameshor other pages primarily aimed at spiders07:05
stubjamesh: We can't list all packages on one page - it is broken07:05
stubjamesh: ZPT just isn't fast enough. spiders are happy following links.07:05
jameshI suppose so07:06
stubWe might have a large batch size, but no batch size is just a scalability problem waiting to be triggered.07:06
jameshand if we're just interested in spiders, something like google site maps might be more appropriate07:06
SteveAspiv, jamesh: the stuff you point out about sqlobject upstream worries me07:07
stubGoogle would be more likely to pay attention to properly batched pages anyway. A huge page full of 10,000 odd links looks like someone attempting to futz with their googlerankings.07:07
SteveAit looks like the project is actively moving in the direction of being just for noddy applications.07:07
spivstub: To be fair, that's pure speculation on my part07:08
=== stub tries to work out what spiv was speculating
spivstub: That it was an optimisation07:09
spivOr should I say, "optimisation" ;)07:09
spivstub: It's possible it's meant as a way to avoid holding a cursor open when something else may want to use it in the middle of an iteration, maybe.07:10
jameshSteveA: on the subject of join columns, they seem to be moving to more scalable versions (SQLRelatedJoin, OneToMany, etc)07:10
stubcursor sharing and iterating over results across multiple transactions seem more likely to me.07:10
dilysMerge to devel/launchpad/: [trivial]  Updates required for serialization and deadlock handling in upstream psycopgda, and tests (r3128: Stuart Bishop)07:11
stub(the answer to the first being don't be so stingy with cursors. The second is trickier if you really want to support it. We don't do it in Launchpad because it is scary)07:11
SteveAjamesh: that's good to hear.  things i'm concerned about are test coverage, leaving broken tests checked in, lack of rationale in commit logs for changes07:15
SteveAas well as not considering large datasets07:16
stubUpstream is really just the work of one developer at the moment (Oleg). Nothing to stop us contributing, although not all of our work has been wanted (we didn't convince upstream that selectOne was a good idea, them prefering to leave the burden of getting it right up to the developer)07:18
spivSteveA: I suspect the tests pass with their default backend, sqlite.07:18
spivstub: Oleg was ambivalent on selectOne after lots of people on this list piped up and said they wanted it07:19
jameshstub: there is something equivalent to selectOne() in there now07:19
spivstub: And since then SelectResults has sprouted a getOne with a similar meaning07:19
jameshstub: rather than Table.selectOne(...), you do Table.select(...).getOne(None)07:20
=== stub tries to decide what syntax he prefers
jameshslightly more verbose, but it covers both the select() and selectBy() variants07:20
jameshand anything else that returns a SelectResults07:20
stubSteveA: I'm being rained out. Ok if I take my public holiday next Monday?07:21
=== stub sods off to lunch
SteveAstub: oleg?  broyntmann?07:25
SteveAnot ian bicking anymore?07:25
spivOleg is much more active on the lists, at least.07:26
spivI get the feeling Ian is a bit more of a ceremonial leader, and Oleg does more of the dirty work.  Ian still does stuff, though.07:26
SteveAi wonder if oleg lives anywhere near me07:27
SteveAspiv, jamesh, mpt: i'd like to have a series of phone calls or skype calls with you (one at a time) to catch up with what's on your todo lists, what you're blocked on and that kind of thing07:28
=== mantiena-baltix [n=vytis@bonamens.lt] has joined #launchpad
SteveAoleg is in moscow, so not *so* far away07:34
SteveAkaip gaila, kad a nekalbu rusikai07:35
=== ulinskie is back (gone 03:45:43)
mptSteveA, currently I'm blocked hacking-wise on https://launchpad.net/distros/ubuntu/+ticket/36307:38
mptwell, push-/pull-wise, anyway07:39
=== lbm [n=lbm@cpe.atm4-0-1301006.0x50a0824e.vgnxx6.customer.tele.dk] has joined #launchpad
SteveAmpt: let's spend a couple of minutes trying to get your network back up07:53
SteveAbut on a different channel07:53
SteveA /join ##wpa-supplication07:53
=== mpt [n=mpt@219-89-155-139.jetstart.xtra.co.nz] has joined #launchpad
CarlFKdo launchpad submissions get announced in any #channels ?08:25
SteveAsubmissions?08:28
SteveAsome stuff gets announced here by dilys 08:28
CarlFKwhat's that?08:34
SteveAdilys is an irc bot that recieves notifications when certain things happen in launchpad08:36
CarlFKlike when I post a bug - is there a bot somewhre that blerts out "Bug #31285 in synaptic (upstream): "Add repository dialog doesn't close"08:36
=== mpt_ [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
SteveAi just checked the logs.  dilys is reporting here merges into the launchpad codebase.08:43
SteveAi'm sure i've seen some ircbot report new bugs filed here before...08:43
BjornTdilys used to report when new bugs were filed08:47
=== mpt_ wonders what https://launchpad.net/binarypackagenames is good for
mpt_anyone know?09:01
=== mpt [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
=== irvin [n=irvin@ubuntu/member/irvin] has joined #launchpad
=== carlos [n=carlos@212.166.228.170] has joined #launchpad
carlosmorning09:23
mpthi carlos09:23
mptHow can I reliably find an URL that uses a particular template?09:25
mptSome templates have lp:urls associated with them in the ZCML, but many don't09:26
mptin this case, binarypackagerelease-index.pt09:26
SteveAmpt: in general...09:28
SteveAmpt: find what the template is for09:28
SteveAto do this, you need to grep / look inside the zcml or the browser code for that template's name09:28
mptI got that far :-)09:28
SteveAlike: grep 'foo-bar.pt' zcml/*.zcml browser/*.py09:28
SteveAif it is in zcml, then look at what kind of object the template is used for09:29
SteveAlike09:29
mptit's for BinaryPackageRelease09:29
SteveA  <browser:pages for="....ISourcePackage" 09:29
SteveAright09:29
SteveAso next, you need to find what URL a binary package release has09:29
SteveAand then put the name of the page for that template on the end of that URL09:30
mptwell, /distros/ubuntu/hoary/i386/pmount/0.1-1 looks promising09:30
SteveAthe easiest way to do that is to look in09:30
SteveA doc/canonical-url-examples.txt09:30
mptbut it turns out that's handled by distroarchreleasebinarypackagerelease-index.pt09:30
SteveAand look for an example of an IBinaryPackageRelease09:30
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has joined #launchpad
mptThe text "BinaryPackageRelease" was not found.09:31
mptMaybe this template isn't used at all...09:32
SteveAthen such things don't have canonical urls09:32
SteveAanother thing to do09:32
spivmpt: you could always delete it, and see if any tests fail ;)09:33
SteveAunregister the page the template is used in in zcml09:33
SteveAand then see if tests fail09:33
SteveAspiv: it is registered in zcml, so deleting it will stop functional tests from starting09:33
SteveAspiv: voice call?09:33
mpt<!--browser:page ...09:33
spivSteveA: I'm just about to head out for dinner, unfortunately.09:33
SteveAmaybe when you get back?09:34
spivI think so; I need to be around in a few hours for the review team meeting anyway.09:34
lifelessjamesh: how are you going on the branch status page ?09:34
ddaahello guys09:35
mptddaa, what did https://launchpad.net/projects/+syncreview used to be for?09:35
mpthi, btw09:35
ddaagood question09:36
=== ddaa looks for the template
spivOuch, I can't assign a bug due to timeouts... https://chinstrap.ubuntu.com/~jamesh/oops.cgi/2006-02-13/C18709:36
mptddaa, bazaar-sync-review.pt09:37
ddaalooking at it already09:37
ddaabaffling09:38
ddaait looks like it would be an earlier version of https://launchpad.net/bazaar/series?text=&state=4&search=search09:39
=== ulinskie is away:
ddaabut seriously, I do not remember ever using something called +syncreview09:39
ddaathe whole rcs-imports stuff in launchpad starts to feel like 3 generations of programmers have gone over it...09:40
spivddaa: bzr annotate suggests it was added by sabdfl, with the log message of "work on revision control import workflow"09:40
mptyeah :-(09:41
ddaampt: date?09:41
spivWay back in Nov 2004.09:41
spiv(revno 891)09:41
ddaawait a min...09:41
mptddaa, perhaps you could offer your professional opinion in https://launchpad.net/products/launchpad/+bug/75009:41
Ubugtumalone bug 750 in launchpad "Oops at /projects/+syncreview" [Normal,Confirmed]  09:41
=== spiv -> dinner
ddaaI have no idea what we were up to in Nov. 2004...09:42
=== mpt was picking onions in Nov. 2004 :-)
mptSteveA, when I unregister the page, no tests fail09:42
ddaaShould be like the time of the Oxford conference...09:43
mptIn which situation would you have a binary package that doesn't belong to a distro arch release, anyway?09:43
ddaampt: do you need me to write a comment in the bug tracker saying "I have no freakin' memory what this page was meant to do, and from the look of it it appears superceded by /bazaar/series"?09:44
ddaaor do you feel comfortable enough about rm it without that?09:44
mptddaa, if that will be enough for matsubara to delete it confidently, sure09:44
SteveAddaa: do we have a meeting in 14 mins?09:45
ddaaSteveA: we usually do09:45
ddaaI think this one is going to be quick09:47
dilysMerge to devel/launchpad/: [r=salgado]  First part of SupportTrackerTweaks, reduce the number of status in the support tracker to three. (r3129: Bjorn Tillenius)09:50
ddaajamesh: you around?09:50
dafSteveA, BjornT: yes, dilys used to report new bugs09:53
dafI can work out why she stopped and fix her if there's demand09:53
SteveAdaf: CarlFK was asking about it.09:54
SteveAwe should have a dilys product in the launchpad project09:54
dafyes. we should09:54
dafperhaps we should keep dilys in RF09:55
jameshddaa: yeah09:55
ddaanice, I see you are already in #canonical-meetingh09:55
SteveAdaf: so, add a product, and file a bug09:55
=== mvo [n=egon@pD951DF50.dip0.t-ipconnect.de] has joined #launchpad
SteveAdaf: and then invite CarlFK to subscribe to the bug09:55
SeveasOOPS-44C196 - subscribing someone else (I tried to subscribe pitti) takes too long09:56
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/44C19609:56
SeveasShould I file it or is it known>09:56
dafI think this is known09:56
dafI'll check09:57
dafhmm, where's mpt today?09:57
sivangmorning all!09:57
SteveAmpt is having network problems09:57
dafhi Sivan09:57
dafSteveA: thanks09:57
sivangmorning daf :)09:57
Seveasanother thing: what is the usefulness of the "Link to CVE" feature? No link appears on the bug page...09:58
SteveASeveas: daf and matsubara are regularly filing bugs for OOPSes, particularly timeouts09:58
jameshSeveas: check the portlets09:58
SteveASeveas: see https://wiki.launchpad.canonical.com/LaunchpadBugTriage09:58
SteveASeveas: as a workaround, you can subscribe pitti by gpg mail09:58
Seveasah narf, didn't look good enough :)09:58
jameshSeveas: also, you can check the bugs associated with a CVE under /malone/cve09:58
jameshand there is a distribution cve report where you can see the open bugs with CVEs associated with them09:59
jamesh(although it is currently busted for ubuntu due to lack of handling of private bugs)10:00
dafCarlFK: bug #3129010:00
Ubugtumalone bug 31290 in dilys "Dilys doesn't announce new Malone bugs any more" [Normal,Unconfirmed]  http://launchpad.net/bugs/3129010:00
=== mpt [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
mvois there a reason that I can't "Edit Source" in launchpads display of a product series ? I try that for gnome-app-install (to point to the current bzr tree) and get a no permission error 10:02
mvo(https://launchpad.net/products/gnome-app-install/+series/main)10:02
ogramvo, are you the owner of the product ? 10:09
mvoogra: I'm registrant at least10:09
ograhmm, i see 10:09
=== WaterSevenUb [n=WaterSev@195-23-238-147.nr.ip.pt] has joined #launchpad
ograi had a similar prob with ltsp ... but it was sufficient for me just to add my branch as bazaar branch ...10:10
SteveAdaf: hello10:10
ogra(even if we are upstream for the new implementation)10:11
dafSteveA: hi10:13
dafmpt: I think the checkbox required thing is a Zope bug rather than a Launchpad one10:14
SteveAdaf: please would you talk with mvo and ogra about the permissions issues they are encountering10:15
SteveAdaf, mpt: in fact, it is both10:16
ograSteveA, mine seems valid, i didnt register the product 10:16
ograupstream changed to us (in fact we're our own upstream and there is still the old one) after the product was registered ...10:16
dafmvo: hmm10:18
mvodaf: it would be enough if I can remove one series and add a new one with the changed vcs information10:20
dafperhaps, as a temporary measure, one of the Launchpad admins can make the change for you10:23
dafwhat was the change you wanted to make?10:23
mvodaf: gnome-app-install no longer uses cvs but bzr at http://people.ubuntu.com/~mvo/bzr/gai--main/10:26
dilysMerge to devel/launchpad/: [trivial]  more suggested changes to analyse-error-reports.py script (r3130: James Henstridge)10:26
mvodaf: the same for update-manager. no cvs anymore but http://people.ubuntu.com/~mvo/bzr/gai--main/10:26
dafboth are from the same bzr branch?10:27
mvodaf: ups, no10:28
dafhttp://people.ubuntu.com/~mvo/bzr/update-manager--mvo/10:28
daf?10:28
mvodaf: yes, sorry. 10:28
=== mvo blushes
daf:)10:28
mvocopy'n'paste error10:28
dafSteveA: can you arrange that?10:29
=== carlos_ [n=carlos@gandalf.pemas.net] has joined #launchpad
dafSteveA: I can't work out how to work out who has launchpad.Edit on ProductSeries10:29
=== carlos_ [n=carlos@gandalf.pemas.net] has joined #launchpad
dafSeveas: I've looked at your OOPS10:32
SteveAdaf: basically, it is the buttsource group10:32
dafSeveas: I think we should have a bug on that, but I can't find it right now10:33
dafSeveas: how do I work that out?10:33
dafhola Carlos10:33
dafer10:33
dafSteveA: how do I work that out?10:33
carloshi ;-)10:33
SteveAdaf: it is tricky :-(10:33
dafSteveA: hmm, that's bad10:34
SteveAdaf: this will get easier when i've landed the CrowdControl optimisations10:34
SteveAwhich have some architectural improvements too10:34
dafso they will optimise for clarity as well as performance?10:34
SteveAyes10:35
dafI don't see why the owner of a product shouldn't be able to modify product series10:35
dafthere's also IProductSeriesSource10:36
dafit's not clear what it does10:36
dafclass IProductSeriesSource(Interface):10:36
daf    # revision control items10:36
SteveAit's various permissions hacks10:36
SteveAto work around a shortcoming in the authorization system10:37
dafthat's what I suspected10:37
SteveAthe work i'll be doing this week is the first step in fixing that10:37
dafI'm not sure that the UI makes it clear that certain bits of productseries have different permissions to the rest10:37
dafso, it is a UI problem as well as an infrastructure problem10:37
mptddaa, you wanted to talk with me about RCS imports?10:38
dafmpt: https://launchpad.net/products/+new has completely mad tab ordering in the form10:39
mptdaf, that's because it's a standard form10:40
mptsee the FormLayout spec10:40
dafmpt: right -- we have a bug on this, don't we?10:40
mptyes, bug is cited in that spec10:40
dafta10:40
mptalong with BjornT's description of how to fix it10:40
SteveAmpt: ddaa, me, lifeless and jamesh are in a meeting10:40
mptok10:40
dafthis spec seem to imply that it will be necessary to specify tab indexes in the code10:41
dafwhereas I'd like the form machinery to be able to generate tab inidces itself10:42
dafBjornT: am I correct?10:42
dafmpt: the only bug I can see cited in that spec is #3736, which appears to have nothing to do with tab indices10:43
dafmpt: ah, never mind, I see it10:43
BjornTdaf: hmm, i have to take a look at that again. but do we have to use tabindex at all?10:43
dafgood question10:44
dafit seems that some forms use it partially10:44
BjornTyeah, and that causes problems10:44
dafyes, I think you have to use it all or not at all10:45
BjornTexactly10:45
BjornTi think the current implementation plan is based on that we won't use tabindex at all, since it's hard to generate them automatically10:45
dafwell, FormLayout includes:10:46
daf    submit_button = SubmitButton('Add', 'UPDATE_SUBMIT', tabindex=1000)10:46
BjornTah yes, i think i added that since the current implementation adds tabindex=1000, and i wanted to show that it'd be possible to still specify a tabindex if you want that. i'll take a quick look at it again, to make sure it isn't needed, and update the spec.10:48
dafthank you10:48
dafis there any reason to add the view/focused_element_id stuff to *form.pt rather than just the main template?10:50
daf(i.e. do we need the duplication?)10:50
=== catnip [i=meowmix@asnet01-1-165.aus.datafoundry.com] has joined #launchpad
=== thisfred [n=thisfred@a80-127-80-154.adsl.xs4all.nl] has joined #launchpad
BjornTjust that the only use case for it so far is for forms. if we make it into a macro it should be ok to include it only in *form.pt i think.10:53
dafok10:53
dafhmm, that reminds me -- I think some pages with custom forms are using an existing JavaScript hack to focus the first form element10:54
dafsetFocus() in launchpad.js10:54
mvodaf: should I file a but about my g-a-i and update-manager request? or is that not needed (now that I have spoken to you)?10:56
dafit is needed, I think10:57
dafsomething like "product registrant should be able to modify product series details"10:57
dafplease include the URL of the +edit page10:57
=== carlos [n=carlos@62.87.126.29] has joined #launchpad
BjornTdaf: true, i forgot about that. they should be modified to use the same macro, and the macro should include the functionality of that function, that is focus on either the first error or the control with the explicit id.11:00
dafBjornT: agreed -- can you put that in the spec? :)11:00
BjornTdaf: yes, i will update the spec later today.11:00
dafgreat, thanks11:00
BjornTwell, thanks for commenting on the spec and giving good suggestions :)11:01
dafyou're welcome!11:01
dafstub: I wonder if the fix for #30979 is cherrypickable11:01
dafstub: and awful lot of people are running into it11:01
daf*an11:01
dafstub: also, does your mail to the list mean that you've been working on #2088?11:05
stubBug 208811:05
dafhttps://launchpad.net/products/launchpad/+bug/208811:05
Ubugtumalone bug 2088 in launchpad "psycopgda reconnection and conflict handling" [Normal,Confirmed]  11:05
dafbug 208811:05
Ubugtumalone bug 2088 in launchpad "psycopgda reconnection and conflict handling" [Normal,Confirmed]  http://launchpad.net/bugs/208811:05
dafSeveas: maybe Ubugtu should be case-insensitive11:06
mptwhy?11:06
stubdaf: Yes. Fix committed.11:06
mptoh, insensitive11:06
=== mpt goes back to sleep
dafstub: that's really excellent news11:07
Seveasdaf, it is11:07
Seveaswell, it should be11:07
dafI'm wondering why "<stub> Bug 2088" didn't trigger earlier11:07
dafBug 208811:07
stubdaf: What revision did the fix for Bug 30979 land in?11:08
Ubugtumalone bug 30979 in malone "Oops in popup to select a Ubuntu package" [Normal,Fix committed]  http://launchpad.net/bugs/3097911:08
mpthaha11:08
dafstub: not sure -- I'll look11:08
Seveasdaf, malone may just be too slow today - Ubugtu will silently fail on timeouts11:08
dafah, I see11:09
dafstub: looks like it was #311911:09
dafstub: which includes an unrelated change to shipit, seemingly11:10
dafstub: but it's [trivial]  so hopefully shouldn't be risky11:10
dafstub: I guess we can run the test suite on the current prod code + that cherrypick11:10
stubyup11:10
=== lifeless has not observed a correlation between risk and [trivial]
daftogether with the #2088 fix, that accounts for the biggest causes of not-from-search-bots crashes11:12
daflifeless: :)11:12
mvodaf: bug #31294, does that look ok? or do you need more information?11:12
Ubugtumalone bug 31294 in launchpad "product registrant should be able to modify product series details" [Normal,Unconfirmed]  http://launchpad.net/bugs/3129411:12
dafmvo: looks great to me11:13
dafmvo: I'm sure we can pester you later if we need more from you :)11:14
mvodaf: yep :)11:17
dafstub: any idea why http://localhost:9030 would be turning up as a URL in the OOPS reports?11:19
stubdaf: We havn't been able to repeat cases where that URL pops up. No idea why it happens.11:20
stubWhat is the oops?11:20
dafOOPS-43A111, OOPS-43A112, OOPS-43A114, OOPS-43A116, OOPS-43B140...11:21
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43A11111:21
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43A11211:21
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43A11411:21
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43A11611:21
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43B14011:21
stubAhh... oops happening before auth and traversal stuff has happened, confusing Zope as the virtual hosting gumph hasn't happened yet.11:22
dafurg11:22
dafah, presumably the SELECT on Person is during the auth process11:23
stubYup11:23
dafand it's timing out due to contention11:23
stubyup11:23
dafdo we know what's causing the contention?11:24
stubNup11:24
dafit must be something writing to Person, right?11:24
stubYup11:24
dafhmm11:24
dafit can't be the karma cron, because that now uses a separate table, IIRC11:25
SteveAstub: so, i guess we should change publication so that it does vhosting before the first person lookup11:26
dafcan we selectively log INSERTs and UPDATEs to Person, and would that help?11:26
SteveAstub: maybe i'll do this as part of some shipit work i want to do, so that we don't do URL mangling in apache any more11:26
dafSteveA: please file a bug11:27
SteveAstub: and instead use httpds on separate ports, with a conf file entry for where they're supposed to be11:27
SteveAdaf: why?11:27
dafSteveA: so that the problem is not forgotten, can be discussed, and can be tracked11:28
SteveAdaf: we're discussing it now.  i think it is premature to file a bug.11:28
dafwell, the solution may not be settled on11:28
stubSteveA: Sure. I'm not sure what sort of priority it would be though.11:28
SteveAstub: do you think the approach is sound?11:29
SteveAi probably need to do the shipit stuff fairly soon, for shipit business reasons11:29
dafI suppose the problem is very minor, the symptom being that OOPS reports are missing some information that isn't critical given that we know what the problem is (contention on Person)11:30
stubWe can't selectively log INSERTs and UPDATES. We can log everything and grep, but there would be too much info to wade through. Most likely the trigger is another launchpad request that also timedout, so the easiest source would be other  OOPS reports that occurred in the previous 30 seconds.11:30
dafI thought INSERTs and UPDATEs to Person were relatively rare11:30
SteveAstub: i talked a little on irc the other day about having a section in oops reports presented via cgi saying "other oopses that occurred concurrently with this one".11:31
SteveAi'm curious though, why the previous 30s ?11:31
dafSteveA: or perhaps a separate page which lists OOPSes chronologically11:32
SteveAdaf: now that the karma cache is in its own table, i expect they would be rare.  but maybe there is some other cache... ?11:32
SteveAdaf: that would be interesting to see.  it could use an html table11:32
SteveAto lay them out with concurrency11:32
dafyes11:32
SteveAor svg11:33
BjornTdaf: it's not missing the information entirely,  you do have the url in the PATH_INFO variable11:33
SteveAfor those using dapper, or possibly a backported firefox11:33
SteveAmy main concern about the vhosting being messed up is that the error page contains broken links11:33
dafI think it would be very informative to know the relative frequencies of the queries that modify Person11:33
stubThe oops you are looking at took 25 seconds to occur. If another request was interfering with it, that request may have started up to 50 seconds before the one you are looking at. The closer the time between them, the more the contention will have contributed to the timeout.11:33
ddaampt: yes11:34
ddaampt: I have issues with the RCS import details being recorded in productseries.11:34
SteveAdaf: from oops reports, we just get long queries, not modifying queries11:35
ddaampt: BTW, that's probably related to mvo's being confused a bit earlier today.11:35
SteveAdaf: we could extend the launchpad application logging to record timing information for POSTs11:35
stubThe karma cache should not be an issue now. It only gets updates to it for a few minutes each day by a single process. It might cause issues in very rare circumstances (a page needs to query the karma cache twice, but the updater happened to update the cached value between the selects).11:35
daf<scifi>make Postgres on emperor log to a socket which has a Python program listening on the other end which greps for INSERT/UPDATE to Person and logs them cumulatively with constants stripped out</scifi>11:35
stub(And if that happened, you would get a serialization exception anyway)11:35
SteveAdaf: we need to consider transactions, really11:36
stubActually, we could log inserts and updates using a trigger bug again it is too much information to wade through.11:36
SteveAi think we will want to log the time taken for POST requests at some point anyway11:36
SteveAas those are the ones that will be hardest to optimise in the long run11:36
dafthen let's do that11:36
ddaamvo, daf: the limits on editing rcs details for syncing imports is a half assed implementation or requirements which will soon be obsolete.11:36
dafddaa: what's being done about it?11:37
dafSteveA: is it much work to log POSTs in addition to GETs?11:37
ddaanot much, last time I checked there was clearly a bug that even I was not able to use the web UI to update such details...11:37
SteveAwe just discussed two different issues.  the first is that some error pages display incorrect URL links.11:37
ddaadaf: so I had to do raw SQL...11:37
dafddaa: do we have an updated set of requirements?11:37
SteveAthis is almost certainly caused by a timeout before zope does traversal, so that the virtual hosting stuff isn't processed11:38
dafyes11:38
SteveAthe mid-term solution to this is steve's refactoring of how we tell launchpad about virtual hosting11:38
SteveAso that we get a decent error page11:38
SteveAthe second issue is that it is hard to see when database updates occur concurrently with other problems11:39
dafI wouldn't prioritise that very highly -- it occurs relatively infrequently11:39
stubAnother approach would be to flag oops timeouts that contain UPDATE, INSERT or DELETEs as high priority, as these may be vicimizing other requests.11:39
ddaadaf: in short, not really. The rational for all that was preventing fucking up the Arch namespace. And the complete (old) requiremest are quite complex.11:39
SteveAsome database updates are caused by launchpad POSTs11:39
SteveAstub: jamesh will be adding the HTTP method to the urls in the oops summary11:39
SteveAstub: so, when we get fascist about readonly GETs, this will serve11:39
ddaadaf: since the priority for rcs imports is leaving arch stuff behind, the only reasonable thing to do would be leaving the stuff broken, and maybe just unbreak it just enough to allow buttsource to edit syncing productseries.11:40
SteveAand for other reasons to do with monitoring scalability, we should record the start and end times of POST requests11:40
=== ddaa wails
SteveAthis does not account for scripts and other daemons that use the database directly, though11:40
dafstub: OOPS-43C97 -- timeout on INSERT to Person11:40
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43C9711:40
SteveAi'll file a bug for the virtual hosting/timeout issue11:41
stubdaf: Daisy chain :-)11:41
SteveAas i'll probably fix that while i do some shipit work in a few weeks11:41
dafstub: :)11:41
ddaadaf: but maybe filing a but would help documenting the issue. People seem to have enough trouble with the current situation that we hear about that quite often.11:41
dafddaa: mvo just filed a bug11:41
SteveAhmm... daf: maybe in the text representation of bugs, grep out any OOPS code in it...11:41
SteveAthen, when you say a bug number, ubugtu could give the bug URL and OOPS URLs11:42
ddaadaf: in the meantime, I can use sql superpowers to fix his stuff.11:42
SteveAdaf: not a high priority, of course :-)11:42
dafSteveA: indeed11:42
SteveAon the recording of POST timestamps and durations, i don't think this will help so much now11:43
SteveAand stu's suggestion of seeing in the OOPS report when we have database writes is a good idea11:43
dafstub: hmm, INSERT into LoginToken that timed out: OOPS-43C11011:43
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/43C11011:43
SteveAdaf: is "bad urls in error pages" already filed as a bug?11:43
dafdon't think so11:44
stublogintoken has a foreign key relationship to the person table, so if the person table is locked the insert is blocked (I think this is improved in PG8.1)11:44
SteveAbug 3129911:49
Ubugtumalone bug 31299 in launchpad "bad URLs some error pages" [Normal,Confirmed]  http://launchpad.net/bugs/3129911:49
SteveAstub: can we turn down the serialization of login tokens or something like that?11:55
SteveAdoes the FK relationship help us in a tangible way here?11:56
stubFK relationship just gives us integrity guarantees. We can drop them. It might help - not 100% sure.11:57
lifelessreview meeting time11:59
lifelesswhos here?11:59
BjornTi'm here11:59
spivI am11:59
lifelessagenda:12:00
lifeless next meeting12:00
lifeless queue status12:00
lifeless20ths at the same time ok ?12:00
spivFine with me.12:00
SteveAyes12:00
BjornTsure12:00
lifelessok.12:00
lifelessqueue wise there are still a significant number of 2005 merge-conditional or other such pending merges12:01
lifelessi.,e.12:01
lifelesssftp://chinstrap/home/warthogs/archives/salgado/guilherme.salgado@canonical.com/sqlobject/2/smallfixes12:01
lifelessand12:01
lifeless sftp://chinstrap/home/warthogs/archives/jamesh/sqlos/select-results-len-fix12:02
lifeless    *12:02
lifelesswhat is the best way to handle this ?12:02
lifelessI'm concerned that noise will make it hard to tell how busy reviewers are, and also whether we are dropping things.12:02
jameshthe sqlos/select-results-len-fix one can be removed12:02
lifelessjamesh: it would be nice if already merged branches could be automatically trimmed or something12:02
lifelessby the merge-calculator12:03
SteveAi suggest an email to salgado about the other branch12:03
lifelesswell it was an example 12:03
lifelessthere are more12:03
SteveAwould be nice this meeting and salgado could meet12:03
stubpqm is down for 30 mins while I run the test suite on balleny12:04
lifelessanyhow, lets keep a low priority eye on ones own queue12:05
lifelessfeel free to drop people a mail - its not your problem as reviewers to drive branches through, but neither should we let the page rot.12:06
lifelessi.e. what is debonzi's branch still on there for ?12:06
lifeless(I've mailed salgado)12:07
SteveAlifeless: maybe have a "compost" section to the page12:07
SteveAfor branches that have been around for a while, and can probably be decomposed12:07
lifelessSteveA: then we need a gardener...12:07
lifelessI'd rather an email to lp cced the person direct12:08
lifelessif they don't reply, and noone takes it on, we can delete it after a couple of days12:08
=== cprov [n=cprov@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
cprovmorning guys12:09
lifelessspiv: ? jamesh? BjornT ? ok with you ?12:09
BjornTlifeless: sure, fine with me12:09
spivI'm ok with that.12:10
jameshlifeless: sounds okay12:10
lifelessok, lets do that12:10
lifelessany other business ?12:11
lifeless512:11
BjornTlifeless: i have a branch in your queue which is more than two weeks old, it'd be good if it was reviewed soon12:11
=== sivang LOLs at lifeless and SteveA 's gardnering analogies :-)
lifelessBjornT: I will either do it or ask another to do it tomorrow.12:12
BjornTthanks12:12
SteveAjamesh: i'll look at the reviews i have in my queue shortly12:12
lifeless412:12
lifeless312:12
lifeless212:12
lifeless112:12
lifeless012:12
lifelesstick tock tick tock tick12:12
SteveAddaa: i'm expecting an email from you about open issues12:13
lifelesss/issues/wounds/12:13
=== lifeless is tired
spivlifeless: Your story branch... I've made the trivial changes specified in my review.  I'm a bit reluctant to merge it myself r=spiv though :)12:13
SteveAddaa: i'd like to get that, and reply to it this morning, before i get lunch.  after lunch, i'll be away from irc.12:13
lifelessspiv: rs=lifeless12:13
lifelessspiv: I agreed with your review comments12:14
lifelessand it has been reviewed.12:14
spivlifeless: Good idea.12:14
spivAh, the the rsync just finished, excellent.12:14
spivlifeless: Also, when will bzr on remote branches stop taking an eternity? ;)12:14
lifelessspiv: 0.812:15
lifelesshttp://mail.opensolaris.org/pipermail/tools-discuss/2006-February/000165.html covers this a little12:15
stubdaf: Test failures cherry picking 3119. I'll try and roll in out tomorrow with the production update12:16
lifelessstub: reminder to reenablepqm12:17
=== stub actually remembered this time :)
lifelesssweet12:17
dafstub: ah, shame; thanks for trying12:18
BjornTcprov: hi. it seems that you forgot to push your soyuz-ui branch12:19
SteveAjblack: ping?12:20
cprovBjornT: really, let'me check 12:20
jblackstevea: pong12:21
SteveAhi jblack 12:21
jblackHiya12:21
SteveAcan you help me with some questions about RocketfuelSetup?12:22
jblackWhat can your local friendly neighborhood bzrguy do today?12:22
jblackSure thing. What's on your mind? 12:22
SteveAi have some code review to do12:22
jblackOk.12:22
SteveAif you take a look here https://chinstrap.warthogs.hbd.com/~jamesh/pending-reviews/12:22
SteveAthere are two branches from jamesh12:22
SteveAone of them, i can get a nice diff for on that page12:23
ddaaSteveA: okay, I'll do that right now.12:23
SteveAthe other is a new tree that isn't in RF yet12:23
SteveAso, i need to get this to review it12:23
SteveAit is quite small, so a "bzr get" worked12:23
jblackAll the way new.12:23
jblackOk.12:23
SteveAin only a few minutes12:23
jameshSteveA: there is no previous revision in rocketfuel for pygpgme12:23
SteveAis there a better way to get it, thoough?12:23
jameshit is all new code12:23
jblackThats the way.12:23
SteveAif it was bigger, should i use rsync somehow?12:24
jblackOh, I see what you're asking. You mean time.12:24
SteveAright12:24
jblackYeah, you can definitely use rsync if you have bzrtools, which you should12:24
SteveAwould you add this "get an entire branch from someone, from chinstrap" use-case to the docs?12:24
SteveAthen i can direct people to it when it comes up again12:25
jblack"bzr branch chinstrap:/full/working/dir newdir" I believe12:25
jblacksure.12:25
SteveAi see12:25
SteveAi didn't think about using "bzr branch"12:25
cprovBjornT: the chinstrap branch is up to date, let me check the permission issues 12:25
SteveAbecause i just wanted to see it, not edit it12:25
SteveAbut i see the point now12:25
lifelessSteveA: you can also use 'bzr get'12:25
jblackYeah. thats a misnomer. branch does the same thing as pull when you dont' have it.12:25
jblackbzr pull chinstrap:... may very well work as well.12:26
lifelessit wont12:26
lifelesspull only uses existing branches12:26
SteveAi tried "bzr pull" first, and it said I wasn't in a branch12:26
lifelessSteveA: there are three rules of thumb here:12:26
SteveAso i tried "bzr get" and it was got, in a few minutes12:26
jblackYeah, then you neeed branch.12:26
lifeless To make a new line of development use branch12:26
lifeless To retrieve an existing branch use 'get'12:26
SteveAjblack: on another issue, jbailey agreed to maintain the launchpad-deps packages.  would you talk with him about getting your package of scripts combined into that?12:27
lifeless To make a branch you have look like another branch use 'pull'12:27
BjornTcprov: ok, the pending-reviews page was updated a few hours ago, and it shows no diff.12:27
jblackstevea: Ok.12:27
SteveAlifeless: okay.  so, could i have used "bzr get" with the rsync syntax?12:27
jblackAny particular reason?12:27
SteveAany particular reason for what?12:27
lifelessSteveA: no, get, like branch, does not understand rsync.12:27
jblackfor jbailey taking it over?12:27
SteveAjbailey is taking over the launchpad-deps packages12:28
SteveAand it would make sense to combine the scripts into the package that developers should install12:28
jblackstevea: sure.12:28
SteveAi'd still like you to work on it, from the point of view of improving the scripts etc.12:28
SteveAbut for jbailey to do the dependency and packaging parts12:29
jblackwhat the? I could have sworn I've used rsync.12:30
lifelessjam has an rsync transport12:31
lifelessbut I think its bitrotten12:31
jblackstevea: no problem. we can rsync directly.12:31
SteveAwhat is 'jam' ?12:32
SteveAi had a databases lecturer once called 'jam'12:32
lifelessJohn A meinel12:32
jblackstevea: that should be something like "rsync chinstrap:/home/warthogs/archives/jamesh/branchname/ nameyouwant12:33
jblackThe slash after branchname/ is significant12:33
jblackAnd be in the directory that you want nameyouwant to show12:33
SteveAlifeless: aha, john.  i hadn't connected the initials with the name/person12:33
SteveAfor a second i thought 'jam' was an add-on for bzr ;-)12:34
jblackIt is, but its probably old12:34
=== niemeyer [n=niemeyer@200.103.244.5] has joined #launchpad
jblackSteveA: Ok. I added a rsync section. Read it over and tell me if it reads clearly to you?12:39
niemeyerjblack!12:41
niemeyerWhat's up doc? :)12:42
jblackniemeyer!!12:42
jblackThe sun is coming up. 12:42
dilysMerge to devel/launchpad/: [r=spiv,rs=lifeless]  Prevent test filtering breaking stories up - allows running of stories easily. (r3131: Robert Collins, Andrew Bennetts)12:42
ddaaSteveA: jam is an add-on, it's the code name for our experimental patch generator based on genetic algorithms ;)12:42
ddaait works pretty well overall, but we have stability issues with its IRC UI.12:42
jblackhow are you my man?12:42
niemeyerjblack: That's good news. ;)12:42
=== salgado [n=salgado@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
=== ddaa -> dinner
niemeyerddaa: Enjoy it12:43
=== ddaa -> lunch
=== ddaa needs coffee, or something
jblackanything bothering you today, buddy?12:43
SteveAjblack: it's good.  i would change the title to something other than "Adding a new branch for review".  Perhaps "Getting a whole branch for review" ?12:43
niemeyerddaa: I was wondering where you were12:43
ddaaniemeyer: so do I12:43
niemeyer:-)12:43
SteveAjblack: also, i'd add a note about the importance of the trailing slash12:43
SteveAother than those comments, it reads very clearly.12:44
ddaaSteveA: btw, email sent12:44
niemeyerjblack: Not at all.. lot's of flying birds around my head with different ideas. :)12:44
jblackSteveA: A whole branch?12:44
SteveAddaa: i will read now12:44
niemeyerSteveA: Morning12:44
jblackniemeyer: Oh, you need a bb gun.12:44
SteveAjblack: or maybe "getting a new branch for review"12:44
SteveAjblack: or maybe "getting a branch that is not in rocketfuel for review"12:45
niemeyerSteveA: Have you had the chance to look at the G arch spec?12:45
SteveAi guess the point is, you can use "bzr get" for it, but this will be slow right now.12:45
jblackGetting a new branch for review is good12:45
SteveAniemeyer: no, i haven't looked at any stuff from the G-Men for a while12:45
niemeyerHumm.. G-Men sounds.. humm G.. :)12:46
SteveAjblack: so, you might also point out that the reason to use rsync today is that we're waiting for the fast stuff to land in bzr12:46
niemeyerSteveA: Let me know if you do, please..12:46
SteveAniemeyer: men in black :-)12:46
jblackDone.12:46
jblackSure.12:46
jblackDone12:47
jblackOhh. the owner of spacepants.org is using bzr.12:48
=== _koke [n=koke@169.Red-217-127-113.staticIP.rima-tde.net] has joined #launchpad
lifelessjaq12:49
SteveAjamesh, stub, spiv: what's happened about the   sqlobject =='f'  to sqlobject?12:53
cprovBjornT: looks daf (or me, not sure) already merged his branch by accident ...zero diff locally too, I'll remove the branch from PendingReviews12:53
=== Progressive` [i=PRS@cl165-093.parabole.lt] has joined #launchpad
=== Progressive` [i=PRS@cl165-093.parabole.lt] has left #launchpad []
spivjblack: why are you looking at spacepants.org?12:54
spivjblack: I am curious :)12:55
spivSteveA: stub was going to do the necessary surgery to sqlobject, I thought12:55
jblackspiv: because I was searching for "RHEL" and "bzr"12:55
SteveAspiv: phonecall?12:56
stubSteveA: I'll be sorting that tomorrow12:58
SteveAok01:01
jblackspiv: He has something called bugtool01:10
salgadoSteveA, do you know in what port shipit is running on production?01:11
SteveAsalgado: same as launchpad is01:11
dafcprov: I don't think it was me01:16
=== niemeyer [n=niemeyer@200.103.244.5] has joined #launchpad
SteveAbug 3129901:17
Ubugtumalone bug 31299 in launchpad "bad URLs some error pages" [Normal,Confirmed]  http://launchpad.net/bugs/3129901:17
cprovdaf: so do I ;), that's way the zero diff is weird01:17
SteveAsalgado: ^^^^01:17
salgadoah, good. thanks SteveA01:18
cprovdaf: anyway, I can't investigate it right now, high priority things on soyuz01:18
=== carlos [n=carlos@62.87.118.235] has joined #launchpad
carloscprov: hi01:32
carlosdid you see my email?01:32
cprovcarlos: hi, yes, did't have time to sort it yet01:33
carloscprov: I have another problem with that branch. Do you have sometime to help me?01:33
cprovcarlos: yes 01:34
=== beyond [n=beyond@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
carloscprov: I'm adding a new test01:35
carlosto test the translations upload01:36
carlosand I get: pmount_0.9.7-2ubuntu2_amd64.deb: Unknown architecture: 'amd64'.01:36
carloscprov: but I already created the distrorelease 'dapper'01:36
carlosand the distroarch amd6401:36
cprovcarlos: did you add an correspondent distroarchrelease for amd6401:36
cprovohh01:36
cprovcarlos: send me the test,01:37
carlosjust a second...01:37
cprovcarlos: ok01:37
carloscprov: https://chinstrap.ubuntu.com/~dsilvers/paste/fileigZV4z.html01:39
salgadojamesh, still around?01:40
cprovcarlos: send me also the changesfile, please01:43
carloscprov: https://chinstrap.ubuntu.com/~dsilvers/paste/fileFpM2ZU.html01:46
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has joined #launchpad
cprovcarlos: here i can't find you key -> UploadError: Signing key not found within launchpad.01:52
carloscprov: that key is Kinnison's one01:53
carlosit should be there01:53
cprovcarlos: uhm ...01:53
cprovcarlos:keys imported, now got FileNotFound, which is good, send me the files 01:56
carlosok01:56
carloscprov: sent02:00
cprovcarlos: thx02:00
cprovcarlos: look on this https://chinstrap.ubuntu.com/~dsilvers/paste/fileXUSEDH.html, it passes here and points you are missing pmount source in dapper 02:12
=== jsgotangco [n=jsg@125.212.122.9] has joined #launchpad
carloscprov: yeah, I saw that other error, but the arch error is still there....02:12
cprovcarlos: you were probably appending data to nascentupload, don't do it anymore, work with your own test02:13
cprovcarlos: here I don't have the arch error, run this new one and see if it pass02:13
carlosI'm using distroreleasequeue.txt02:13
carlosanyway, I will add the new test as you suggest02:14
carloscprov: thank you!02:14
=== SteveA --> lunch
cprovcarlos: ok, try to work with this separated test (less moving parts) then I can help you on demmand, by copying it in my tree everytime you need me to run it for you02:15
carloscprov: your test works here02:23
carlosso It's a matter of fixing the missing pmount package and I suppose that's all02:23
carlosthank you02:23
cprovcarlos: fine, sort out the missing source 02:23
cprovcarlos: yes yes, be happy ;)02:24
=== doko_ [n=doko@dslb-084-059-106-239.pools.arcor-ip.net] has joined #launchpad
doko_could somebody enlighten me, why gcj-4.1_4.1ds8-0ubuntu8 was rejected with "Uploads to dapper are not accepted." ?02:36
salgadostub, around?02:38
Kinnisondo that's a good one02:41
Kinnisondoko_: ^^02:41
=== carlos_ [n=carlos@62.87.118.166] has joined #launchpad
carlos_cprov: hi, sorry, my line went down02:42
doko_Kinnison: ?02:42
carlos_cprov: did you see my question?02:42
Kinnisondoko_: when did that reject happen?02:43
cprovcarlos_: nop, type it again, please02:43
carloscprov: I checked it and I already have a pmount sourcepackage in our sample data02:43
doko_Kinnison: solved, that was still jackass ...02:43
carloscprov: so I suppose it's missing s source upload into the system, right?02:43
Kinnisondoko_: oops :-)02:43
cprovcarlos: not for dapper, I guess02:44
carloscprov: what's the table to do that?02:44
carloscprov: I thought the SourcePackage object was created automatically 02:44
carlosfrom the SourcePackageName and DistroRelease02:44
cprovcarlos: uhm, nop ... there is no SourcePackage in DB world, but a SourcePackageRelease published in a given DistroRelease02:45
=== matsubara [n=matsubar@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
carloscprov: ok02:46
cprovcarlos: query SourcePackagePublishing, for a that name and that version within dapper distrorelease. 02:46
carlossoyuz testing is really a pain in the ass....02:47
=== carlos -> lunch
carloswill do that after lunch02:47
carloscprov: thanks02:47
cprovcarlos: I'll be here to help you02:48
carloscprov: when are you going to merge your branch?02:48
carlossee you later02:49
=== dsaa [n=dsaa@210.213.84.183] has joined #launchpad
cprovcarlos: in RF ? no established ETA yet, depends on the reviewer's02:49
=== mgalvin [n=mgalvin@ubuntu/member/mgalvin] has joined #launchpad
carlosso it's ready to review, right?02:50
carlosok02:50
=== _mvo_ [n=egon@pD951DF50.dip0.t-ipconnect.de] has joined #launchpad
stubsalgado: pong02:57
salgadohi stub, I'm having some issues trying to write a sql query. can you give me some help?02:59
stubsalgado: sure03:01
salgadoso, we have the MirrorProbeRecord, where we store when a mirror was last probed (together with the log file). I need to do a query to get the mirrors that need probing. these should be the mirrors that were never probed or that were probed more than N hours ago03:03
=== BjornT heads out for a while
salgadostub, https://chinstrap.ubuntu.com/~dsilvers/paste/fileJX1jTn.html is what I have so far, but I think that's not the right way to do it. apart from the fact that it won't do what I want03:07
stubsalgado: Does https://chinstrap.ubuntu.com/~dsilvers/paste/file1ArzpA.html give you the results you want?03:11
stubuse 'is true' btw instead of = true03:13
=== carlos_ [n=carlos@84.76.255.40] has joined #launchpad
stubsalgado: https://chinstrap.ubuntu.com/~dsilvers/paste/file6uGRaw.html with the 'enabled' flag check 03:14
salgadostub, yeah, that seems to does the job. I'll write a test just to make sure. thank you!03:16
=== mpool [n=mbp@57.16.168.202.velocitynet.com.au] has joined #launchpad
mpoolthanks for making the login timeouts more reasonable!03:19
=== kiko [n=kiko@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
kikomorning03:20
mpoolhi kiko03:21
mpooli was just saying03:21
mpool"thanks for making the login timeouts more reasonable!"03:21
mpoolanyhow, goodnight03:22
kikoenjoy both03:24
kikostub, rollout planned for tomorrow? what revno?03:26
=== LarstiQ seconds mpool
bradbhey kiko. I think I've come up with a pretty simple macro API for keeping the search filter, than I can land (or get into code review at least) today including giving +assignedbugs the new look and feel, and possibly having time to migrate another of the reports today. should i do it, or should i still drop it?03:31
kikothat's interesting03:31
kikoif it's simple, give me an overview of how it works03:31
bradbthe macro would expect certain variables (default nothing): status_filter_links, searchtext, package, etc. it would use tal:conditions to know which of these need to be rendered. the views could have a base mixin class for common filter criteria extraction, and could specialize as needed.03:33
bradbactually, it would be more like searchtext_filter_link and package_filter_link03:35
kikohow would the view provide these variables?03:35
bradbkiko: the view for +assignedbugs wouldn't bother providing package_filter_link, where the view for +packagebugs would provide it with a method, like getPackageBugsFilterLink => tal:define="package_bugs_filter_link view/getPackageBugsFilterLink"03:37
stubkiko: I'm thinking of r3123 with r3128 cherry picked03:37
kikobradb, it might be easier for the macro to use the view directly03:39
stubkiko: But I'm open to opinions03:39
bradbkiko: It could do.03:39
stubkiko: r3126 should probably go out too03:41
kikostub, that sounds like a good plan, though I'd try to pick r3126 too -- basically let the support tracker stuff bake.03:41
kikoright03:41
kikomatsubara has a fix for bug 31158 that might be nice too, if we can land it now03:42
Ubugtumalone bug 31158 in soyuz "BinaryPackageRelease page is crashing because of portlet details" [Normal,In progress]  http://launchpad.net/bugs/3115803:42
ddaaholy shit...03:54
ddaathe sourcepackage field in productseries is just plain wrong...03:55
ddaaa series can be associated to multiple source packages...03:55
ddaasee database.ProductSeries.setPackaging03:57
kikowhat does the field link to?03:57
ddaait's not actually a field in the database03:57
ddaait's an input in the series/+source form03:57
ddaain the database is something that exists somewhere in the packaging tables03:57
kikopackaging sounds right, though03:58
ddaalook at database.ProductSeries.setPackaging03:59
ddaathat will update any number of existing associations, or create a new one...03:59
ddaalooks to me like a simple text input is not the right UI for that kind of functionality...04:00
ddaanevermind it has nothing to do with RCS imports...04:00
=== LarstiQ [n=larstiq@cust.7.157.adsl.cistron.nl] has joined #launchpad
=== kiko scratches head
ddaadon't scratch too hard, or you'll end up with no HEAD again ;)04:01
=== kiko [n=kiko@200-171-140-32.dsl.telesp.net.br] has left #launchpad ["Left]
=== kiko [n=kiko@200-171-140-32.dsl.telesp.net.br] has joined #launchpad
kikowhat happened?04:04
kikoanyway04:04
kikobradb, I think you should leave the filter stuff for later.04:04
bradbkiko: ok04:04
kikoit sounds like it will get in the way, and if we centralize the forms it is easy to add it in to the single central location, later.04:04
kikostub, are you on vacation today?04:05
bradbI'm removing it on a separate branch then, so that we can get the code back if needed, whilst not leaving it sit there unused in the codebase.04:05
kikosounds good04:05
kikobradb, what did bug 3123 actually fix?04:06
Ubugtumalone bug 3123 in python-numeric "python-numeric-tutorial package does not depend on python2.3 but requires it" [Normal,Confirmed]  http://launchpad.net/bugs/312304:06
kikooops sorry04:06
kikobradb, what bug did r3123 actually fix?04:06
bradbkiko: Just the bug in the errors report. I couldn't find a bug report open for it.04:07
stubkiko: sort of04:07
kikobradb, how did it happen?04:07
kikoa null bugtask delta?04:07
kikostub, cprov wanted to ask you to do some work on mawson if you could04:08
kikocprov, email stub of the details, is my suggestion, anyway04:08
cprovkiko: nevermind, I'm talking with him04:08
=== BjornT [n=bjorn@clt-84-32-240-183.dtiltas.lt] has joined #launchpad
bradbkiko: The BinaryPackageName vocabulary was returning an int instead of the BPN object, so in the specific case where a BPN is filled in for a task, it can look like a change was made when one wasn't, and thus it gets processed through the change machinery, causing an empty task delta.04:09
bradbBecause the code that checks if a change was really made would be comparing a BPN id to a BPN object04:09
kikoI see04:10
kikobradb, do you think it happened when changing a bugtask's binary package name?04:10
bradbno, that would accidetally work correctly04:11
bradbi /think/ it happens when you submit a "Comment on this change" without having changed anything.04:11
kikoso what would crash? not changing it? :)04:11
kikoI see04:11
bradbNormally you get an error that says that you submitted a change comment without changing anything.04:12
kikocprov, did Kinnison manage to finish off the work to support pockets?04:13
kikonon-release pockets, in particular04:13
kikostub, do you know if anyone is working on the fix for boolean columns?04:13
cprovKinnison: not yet, we are working on it 04:13
kikoI  see04:14
stubkiko: I'm looking at that tomorrow04:15
kikostub, thanks04:18
kikostub, in your reply to the Vocabulary... email I sent, I had two questions04:27
kikoa) did you do any code changes or would you like to delegate this to salgado?04:27
kikob) how do you "redo the ORDER BY" to use the index?04:28
=== Alinux [n=Ubuntu@d81-211-224-82.cust.tele2.it] has joined #launchpad
ddaastub: can you set the owner of https://launchpad.net/people/ddaa/+branch/gnome-app-install/main and https://launchpad.net/people/ddaa/+branch/update-manager/dev to mvo, please?04:32
ddaa(using "admins" priv)04:33
kikoddaa, is there no "reassign" yet?04:34
Alinuxhello, when Dapper's lauchpad import ?04:34
ddaakiko: you mean somethnig like bug #29863 ?04:35
Ubugtumalone bug 29863 in launchpad "Workflow to transfer ownership" [Wishlist,Unconfirmed]  http://launchpad.net/bugs/2986304:35
ddaakiko: as far as I know, there is nothing to implement any of the use cases.04:36
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
ddaakiko: there is Branch.author, but it's for a different use case.04:36
=== Alinux is now known as AlinuxOS
kikoddaa, that's what I was referring to. it would be nice to have..04:38
ddaakiko: well, it's not going to happen soon since it was marked as wishlist... feel free to expand on the strawman I proposed.04:38
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
ddaakiko: in the meantime, an admins intervention is required.04:39
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== raphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== ubuntulog [i=ubuntulo@trider-g7.fabbione.net] has joined #launchpad
=== Topic for #launchpad: Datacentre migration, services down | launchpad.net | developer meeting: Thur 16 Feb, 1200UTC (wiki:MeetingAgenda) | launchpad-users@lists.canonical.com (wiki:MailingLists) | Channel logs: http://tinyurl.com/72w39
=== Topic (#launchpad): set by lifeless at Sat Feb 11 23:58:55 2006
=== fabbione [i=fabbione@gordian.fabbione.net] has left #launchpad ["Ex-Chat"]
dilysMerge to devel/launchpad/: [trivial]  required MirrorProbeRecord indexes (r3132: Stuart Bishop)04:51
=== fabbione [i=fabbione@gordian.fabbione.net] has joined #launchpad
ddaamvo: your branches are set up and linked from the productseries, now you just need an admin to make you the owner of the branches.04:54
ddaahere is the list of people you can nag: https://launchpad.net/people/admins04:54
mvoddaa: thanks04:54
ddaaI had to file like 3 or 4 new bugs related to that user's email...04:55
ddaamvo: can you handle replying to launchpad-users?04:56
mvoddaa: hm, clicking on https://launchpad.net/products/update-manager/+series/main gives me a oops 04:56
ddaawhat... the...04:57
mvoddaa: I'm not on launchpad-users, sorry. my mail awaits moerations there too04:57
=== ddaa cries
mvoddaa: OOPS-44A491 if that helps04:57
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/44A49104:57
kikoI'll approve04:57
ddaaFUCKING BUGGY PRODUCTSERIES DIE DIE DIE04:58
ddaahere's what you get when you have functionality with no owner, no use case, and nobody who cares...04:58
ddaamvo: I'll clear the ProductSeries.branch, I bet that's the culprit...04:59
ddaaI sort of believe that field was actually never used04:59
mvoddaa: I answered the mail "gnome-app-install, update-manager sources?" on ubuntu-desktop/launchpad-users now04:59
mvoddaa: thanks for tracking that issue 04:59
ddaamvo: fixed05:00
ddaaholy fuck05:00
ddaamvo: thanks a lot, you helped me uncover a critical bug I needed to be aware of for the bzr transition.05:01
ddaathis transitions starts to really feel like bug slalom, or dodging bugllets, or whatever...05:02
ddaaI mean really, all the bugs in the system are the main design constrainsts...05:02
mvoddaa: thanks, I can see the bazaar branch now (but the old cvs branch is still there too)05:03
ddaamvo: sure, it's still being updated, I see no reason to remove it05:04
ddaaArguably, it should not be in the productseries, but I filed a bug about that.05:04
mvoddaa: it's not used for development anymore, maybe it can be moved to a less prominent place?05:05
mvoddaa: or renamed to "OLD" or something?05:05
ddaarename the productseries to OLD?05:05
ddaaman, it's your main productseries...05:05
mvoddaa: it isn't anymore, the main one is the bzr branch. and I seem to be unable to change that myself05:06
ddaait's your main productseries05:06
ddaaproductseries contains rcs import05:06
ddaathat's a bug in my opinion05:06
ddaayou cannot have rcs import w/o productseries05:06
ddaawe will be able to clear up the rcs details when the bzr branch for the import is registered and published, but right now it would cause the data to just never be converted.05:07
ddaawell, actually it's probably already converted, but won't be published05:07
ddaaWe can do that if you think it's right not to publish the bzr branch for that RCS import.05:07
dafddaa: re your bug:05:08
daf"not possible" -- the UI doesn't support it?05:08
dafor it's a permissions problem?05:08
ddaadaf: which bug, which "Not possible"?05:08
mvoall the fuss started because CVS is no longer my main branch for u-m and g-a-i. I would be happy to change that myself in launchpad but apparently I'm unable to. so I would like to have some way to indicate that it is no longer the mainline05:08
dafddaa: bug #3130805:08
Ubugtumalone bug 31308 in launchpad "Cannot set branch associated to a productseries" [Normal,Unconfirmed]  http://launchpad.net/bugs/3130805:08
daf"t's currently not possible to set the branch associated to a productseries."05:08
ddaadaf: there's no UI for that. And actually the productserie page OOPSes when you set the branch with sql.05:09
ddaaWill comment on the bug.05:09
dafthanks05:09
dafincluding an OOPS reference might be helpful05:09
ddaamvo: I understand that, but the "main branch" concept in launchpad is current braindead.05:09
ddaaadd in an importd bug05:10
mvoddaa: so there is no way to change a "main branch" once it was setup? (with the current lp)?05:10
ddaamvo: so the only options I can give you is either leave the productseries as it is, or clear the RCS details and never have the RCS import updated again and never have it published as baz.05:10
=== mvo weeps
ddaamvo: there's no main branch concept in launchpad. There is a main productseries.05:11
ddaamvo: I weep too05:11
ddaathis stuff is braindead05:11
mvook, please keep it as it is then people in gnome-cvs translate there it would be good to have that stuff importet05:11
ddaaI just did not realise how serious it was before we had the ability to register branch...05:11
mvoI will add some comment in the bzr branches stuff and hope that interessted people read it and get the bzr stuff instead of the cvs one05:12
ddaamvo: I'm really sorry for the trouble, but when I told you the rcs import stuff was in dire shape I was not weening. I just know how seriously broken it's broken, and you just happen to hit most of road bumps (and find some new ones too).05:13
ddaamvo: in that sense, you are an invaluable help to me :)05:13
=== Ubugtu [n=bugbot@ubuntu/member/seveas] has joined #launchpad
mvoddaa: ok, thanks. I'm happy that it will help fixing the issues. I will be able to work around the issues until then :)05:15
dilysMerge to devel/launchpad/: Fixes https://launchpad.net/products/soyuz/+bug/31158 (BinaryPackageRelease page is crashing because of portlet details) r=kiko (r3133: Diogo Matsubara)05:26
carloscprov: ok, so I have the sourcepackagerelease but it's still failing05:30
carloscprov: I think it's related to the manifest entry I choosed05:31
cprovcarlos: manifest isn't really important, IFAICS05:31
carloscprov: let me show you the new test05:31
matsubarakiko: https://launchpad.net/products/launchpad/+bug/31287 Could you try to access this bug?05:31
cprovcarlos: sure05:32
carloscprov: https://chinstrap.ubuntu.com/~dsilvers/paste/file7CyY91.html05:32
cprovcarlos: right, you have to create a Build for amd64 arch05:35
cprovcarlos: and pass it via to the policy05:35
carloshow?05:35
cprovcarlos: spr.createBuild()05:36
carlosok05:36
kikomatsubara, targetname is forbidden to me.05:37
cprovcarlos: also to publish the just-created sourcepackagerelease05:37
kikostub, r3133 would be nice as well, soyuz topcrasher05:37
matsubarakiko: same with me and salgado.05:37
ddaadaf: I commented on bug 3130805:39
Ubugtumalone bug 31308 in launchpad "Cannot set branch associated to a productseries" [Normal,Unconfirmed]  http://launchpad.net/bugs/3130805:39
kikobradb, can you check on permissions for bug 31287? seems busted.05:41
=== bradb looks
bradbhm, weird05:43
kikonot only weird, it's a bug :)05:43
kikoor perhaps not, but..05:43
bradbyes, almost surely a bug05:44
=== BjornT_ [n=bjorn@clt-84-32-240-183.dtiltas.lt] has joined #launchpad
=== bradb tries to reproduce locally
salgadoisn't that a bug that was filed as private?05:45
kikoit might have been, but are there no subscribers?05:46
bradbit's almost surely a privacy-related bug. i'm just trying to confirm that it is what i think it is.05:46
bradbreproduced locally05:50
bradbIt would have been easy to overlook before because most of us were admins.05:50
kikothank god somebody decided to revert that, but matsubara and salgado were never admins that I know of05:50
bradbthey probably weren't looking at a lot of private bugs05:51
bradbthere aren't many for LP05:51
kikotru05:51
kikoadd test and fix and we can cherry-picketh05:51
bradbyeah05:51
salgadowhat would be the fix for that? (just curious)05:52
=== hannosch [i=hannosch@e176104125.adsl.alicedsl.de] has joined #launchpad
bradbThis is actually already tested in the privacy stored, and perms denied is the right error here, but the perms denied appears to be happening when rendering the ZPT, instead of when traversing the URL, which causes it to spit out an exception, instead of a human-readable "not allowed here" type page.05:53
bradbs/privacy stored/privacy story/05:54
bradbsalgado: I /think/ the error may be in the page perms config, but I'm verifying that now.05:54
salgadoright, but I'd expect that the launchpad team would've been subscribed to that bug when it was filed05:55
bradbsalgado: Not if it were reported private.05:55
bradbThe SecurityTeams spec is set to address this hostility.05:55
salgadoah, right05:55
bradbright, here's the problem:05:57
bradb    <browser:pages05:57
bradb        for="canonical.launchpad.interfaces.IBugTask"05:57
bradb        class="canonical.launchpad.browser.BugTaskView"05:57
bradb        permission="zope.Public">05:57
bradb      <browser:page05:57
bradb        name="+index"05:57
bradb        template="../templates/bugtask-index.pt" />05:57
bradbzope.Public, when it should be launchpad.View.05:57
kikohey stub?05:58
dilysMerge to devel/launchpad/: [trivial]  remove the search filter UI. plan to resurrect it at the London sprint. (r3134: Brad Bollenbach)05:58
ddaakiko: replied to your mail with CC to the launchpad mailing list06:12
kikothanks 06:12
ddaakiko: your suggestion was wrong, and I ended up writing a long explanation of why... I thought it would be of interest to other peopel.06:12
kikosaying "your suggestion was wrong" isn't a good way to garner support...06:13
=== jbailey [n=jbailey@modemcable139.249-203-24.mc.videotron.ca] has joined #launchpad
jbaileyIn Malone, what's the difference between "milestone" in the status field, and targetting a fix at a release?06:15
kikojbailey, the latter seems to be a bad suggestion06:15
bradbjbailey: milestones are a forward-looking goal, targeting a fix to a release is for backport/security fixes. this ui sucks.06:16
ddaakiko: sorry I did not mean it to get out this way...06:17
bradbjbailey: Making this UI suck less is on my top 5 priorities list, among some other tasks involving making bug listings/batch/searching suck less.06:18
jbaileyHmm,okay.06:18
jbaileyIn this case I'm targetting a fix to a release because I'm uploading to -updates.06:19
jbaileySo I want to track the upload to Hoary and Breezy separately.06:19
jbaileySo it looks like I sould target a fix to the release.06:19
ddaakiko: put another way "registered branches cannot be a superset of authored branches because there are distinct overlapping sets".06:19
jbaileyThanks for the help. =)06:19
bradbjbailey: yep, that's right. no prob.06:19
kikoddaa, have you considered that it may be that the end-user does not benefit from this separation?06:20
ddaaI have, and they do.06:20
ddaaI can find the specific bug where that was discussed, if you want.06:20
kikoI've read it. I'm asking because I, as an end-user, see things in a similar manner as the user initially suggested in said bug report.06:20
=== ddaa goes back to read the bug report
kikothat going to registeredbranches and not seeing branches he registered /and/ authored is confusing.06:21
kikoso I'm suggesting that 3 + 2 + 4 might be a better solution06:21
kikohowever06:21
ddaakiko: that's an interesting solution06:21
kikoI don't want to discuss this any further than this, as I have a busy Monday :-)06:21
ddaaI'm not dismissing what you are saying now, but what you said initially.06:22
ddaakiko: so you suggest 1+3 for authoredbranches and 2+3+4 for registeredbranches?06:23
bradbIs it normal that our "Not allowed here" page shows an exception?06:23
kikoexactly06:23
kikobradb, only for launchpad devels, I think06:23
bradbkiko: I'm looking at this page with the no-privs user locally.06:23
ddaakiko: my concern is that I think that the places where the model does not match reality are important to see easily...06:24
=== bradb tests another page in production to see.
ddaaI'm not really happy with the current trade-off either.06:24
bradbHm, but I'm a dev in production, so screw that.06:24
kikoddaa, right, I think that /we/ as launchpad people find that distinction important, but I don't know about the end user06:24
bradbstub: Is it normal that our "Not allowed here" page shows an exception?06:24
carloscprov: ok, next step... How could I publish the package?06:25
kikoit is not normal, bradb 06:25
kikoit should only be so for launchpad devels06:25
kikoare you sure no-privs is not in launchpad devels?06:25
bradbYeah, I checked that: No Privileges Person is not an active member of any Launchpad teams.06:26
bradbI'm wondering if we also have a config that says always show tb's in dev.06:26
ddaakiko: you have a point... I guess we'd need to ask users. You know that old rule: when in doubt, do what's most useful to you, then you know it will match the expectations of at least one user.06:26
cprovcarlos: invoke its correspondent IDistroReleaseQueueSource.publish()06:27
kikothat's reasonable06:27
cprovcarlos:  then move the resulted SSPPH to status PUBLISHED06:28
carlosok06:28
bradbkiko: ah, check it out:06:29
bradb        # If the config says to show tracebacks, or we're on the debug port,06:29
bradb        # or the logged in user is in the launchpad team, show tracebacks.06:29
bradbit all makes sense now06:29
kikoyeah06:29
=== rraphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
=== rraphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
carloscprov: this is not working... the queue is empty06:45
carloscprov: https://chinstrap.ubuntu.com/~dsilvers/paste/fileCmeZoJ.html06:46
carloscprov: the for loop does not executes any iteration06:46
=== rraphink [n=raphink@ubuntu/member/raphink] has joined #launchpad
cprovcarlos: on sec06:51
carlosok06:52
cprovcarlos: you only have DRQ after perform the an upload 06:52
carlosso I need to upload the tar.gz?06:53
carlosok06:53
carlosdude, this is soooo complicate. The test Is getting big and big to just test a small part of the system06:53
cprovcarlos: in fact you need, since you have shortcut the source upload process, to create the SSPPH record by hand as we do in that method I've pointed you06:54
carlosI suppose that the fact that I don't know the details about soyuz it's the problem here...06:54
carloscprov: so what's the right way to do it?06:55
cprovcarlos: add the the SSPPH record by hand, I think, so at the end you'll have the source published 06:56
kikocarlos, cprov: would it be easier for carlos to put things in a directory like the sync process does?06:56
kikojust place them in a directory06:56
kikoand run process-upload for a specific policy06:56
kikowould that work?06:56
cprovkiko: he'd need real source 06:57
kikonot if you move the source check to the policy06:57
kikoinstead of hardcoding it06:57
carloscprov: I have the real source06:58
cprovkiko: this is not trivial 06:58
cprovcarlos: how big ?06:58
carlossmall06:58
carlos36104 bytes06:58
cprovcarlos: I'd not run the tools for it, it's expensive and as dificult as doing what you are doing. 07:01
carlosok07:02
cprovcarlos: concentrated in what you need, as soon as you have the source published you binary upload (already done) will pass07:02
cprovcarlos: will send the SSPPH insert for you, one sec07:02
carlosok, thank you07:03
kikoone sec, let me go up.07:03
bradbBjornT: For the "API" of a ZPT macro, is it more idiomatic for it to expect certain variables to exist, or for it to expect the view to have certain methods, or does it matter?07:05
BjornTbradb: good question, i'm not quite sure. i would say it depends on the situation, so it doesn't matter that much. choose what seems the easiest thing to do.07:10
bradbok07:10
dilysMerge to devel/launchpad/: [trivial]  fix bugtask view permissions so the 'Not allowed here' page shows the correct exception when user visits a private bug without launchpad.View permissions. (This is already tested to ensure that a 403 Forbidden is raised.) (r3135: Brad Bollenbach)07:16
kikocarlos, I now understand better what you are doing -- simulating a build that uploads translations to rosetta, right?07:18
carloskiko: right07:18
kikoI thought there were tests that did that already,but there are none07:18
carloskiko: if the test works, then we can start doing dapper imports07:19
kikoyeah, I see.07:19
carlosWe have tests for the imports07:19
kikobut not for builds :)07:19
carlosbut not for the glue between soyuz and rosetta07:19
carlosright07:20
cprovcarlos: wth -> (16:19:50) carlos <AUTO-REPLY> :  I'm not here ;)07:20
carlosoh, sorry07:20
cprovcarlos: don't you receive priv ?07:20
carlosyes, I do07:21
=== jinty [n=jinty@196-28-44-233.jhb.netdial.co.za] has joined #launchpad
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has left #launchpad []
=== ddaa [n=ddaa@nor75-18-82-241-238-155.fbx.proxad.net] has joined #launchpad
=== SteveA [n=steve@195.182.78.95] has joined #launchpad
kikoddaa, do you know if permissions are broken in https://launchpad.net/products/gnome-app-install/+series/main/+addpackage -- ?07:35
kikoddaa, I'd like product owners to be able to add those links..07:35
ddaadunno about this page...07:35
ddaabut I think there's alread a package for gnome-app-install...07:35
ddaahttps://launchpad.net/products/gnome-app-install/+packages07:36
ddaaThere's a bug about this being hidden and useless: bug 3131907:37
Ubugtumalone bug 31319 in launchpad "Association between source package and product is not discoverable" [Normal,Unconfirmed]  http://launchpad.net/bugs/3131907:37
kikois it useless?07:38
kikoor just too hidden to be useful?07:38
ddaauseless for navigation07:38
ddaaand, well, too hidden to be useful, too...07:39
ddaathe table is not hyperlinked07:39
=== lfittl [n=lfittl@85-125-145-79.dynamic.xdsl-line.inode.at] has joined #launchpad
carloscprov: ok, next step done07:50
carlosbut I keep getting "Policy permits only one build per upload."07:51
cprovcarlos: pass the build.id to the policy07:52
carlosright, I forgot that... sorry07:52
ddaacprov: you have admins superpowers07:53
cprovddaa: yes, I have07:53
ddaacan you give mvo ownership of https://launchpad.net/people/ddaa/+branch/update-manager/dev and https://launchpad.net/people/ddaa/+branch/gnome-app-install/main please?07:54
=== mvo hugs ddaa
ddaaand maybe sets the name of the latter to "dev" for consistency, too07:54
=== mpt [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
carloscprov: hmmm... sorry but I don't see a way to link the build with the policy....08:06
cprovddaa: mvo: check it, please08:06
cprovcarlos: one sec ... (set MockOptions().buildid = build.id08:08
ddaacprov: looks right, thank you08:08
cprovddaa: you're welcome08:08
cprovcarlos: you need to use buildd policy too08:09
mvocprov: thanks, looks good08:09
=== merriam [n=merriam@84.12.81.236] has joined #launchpad
carloscprov: buildd policy?08:11
cprovcarlos: instead of anything08:12
carlosok08:13
carlosI need to go out for 15 minutes when I'm back I will try that08:13
carloscprov: thank you very much!08:13
ddaakiko: re product preload, I think it's just because this guy is bored. http://jarufe.monkey.cl/08:20
kikothat may be08:21
ddaahttp://foros.tux.cl/viewtopic.php?p=11114&sid=e6af236771b13b61fb98ec83fef51e2e08:22
ddaaI think this guy needs a dating site more than launchpad.08:23
cprovcarlos: when you get back, see this test prototype -> https://chinstrap.ubuntu.com/~dsilvers/paste/filefNbWHF.html, hope it does what you want08:23
carlosI'm back08:24
carloscprov: that's more or less what I did. Thanks!08:26
cprovcarlos: good, did you see that test .. have isolated you tarball.08:26
cprovcarlos: how are you going to process it now ?08:26
cprovcarlos: don't forget to abort the transaction when you finish you test (I forgot)08:27
carloscprov: we have code in place for that08:27
carlosso it's a matter of checking the translation import queue to know if the import was done08:28
cprovcarlos: I'm pushing my branch, you'll need it for a small fix in DRQ interface08:28
carlosok08:28
cprovcarlos: tricky, I'm curious to see how your things will work, maybe in the next conference ;)08:29
carloscprov: it's easy, Kinnison did a hook on soyuz for translations, let me check where is it...08:29
kikomatsubara, how's it going?08:29
cprovDRQC.publish()08:30
cprovcarlos: ^^08:30
carloscprov: yes08:31
=== merriam [n=merriam@84.12.81.236] has joined #launchpad
carloscprov: publish_ROSETTA_TRANSLATIONS08:31
cprovcarlos: the magic is in spr.attachTranslationFiles()08:32
carlosoh, you want those details?08:32
carlosthen best for the next conference, or at least not today... I wan to finish this today and stop.08:33
matsubarakiko: wanna take a look of what I have so far?08:33
cprovcarlos: sure, as I said, next conf ;)08:33
kikomatsubara, what are you working on?08:33
cprovcarlos: let me know if you need any other help with upload-and-queue system08:34
carloscprov: I don't think so, If this triggers Kinnison's hook, that's all I need. Next step is Rosetta specific08:35
matsubarakiko: that broken traversal on build problem, and trying to fix a portlet on binary package release page08:35
carloscprov: thank you very much for your help08:35
carlosThe test is passing now08:35
carlosIt's time to test my Rosetta stuff ;-)08:35
cprovcarlos: anytime08:35
cprovcarlos: good luck !08:38
ddaakiko: I think I construed a convincing scenario of what happened with "prelink", and came to an interesting conclusion.08:38
carlosthanks!08:38
kikomatsubara, the broken traversal thing should be trivial08:38
matsubarakiko: it's already fixed.08:38
kikowhere's the patch08:39
=== Seveas [n=seveas@ubuntu/member/seveas] has joined #launchpad
carloshmm08:41
matsubarakiko: https://chinstrap.ubuntu.com/~dsilvers/paste/fileADvMEg.html08:41
carloscprov: I need some extra help. I cannot create the entry published as the hook is executed when we publish it08:41
carloshmm08:42
cprovcarlos: ?? again, I lost the track 08:42
carlossorry08:42
carlosthe source code is published08:42
carlosthen, I upload a binary08:42
carlosand execute .process()08:42
cprovright08:43
carlosbut the publish hook are not executed08:43
=== carlos talks about queue.py code
cprovcarlos: wait you need to execute DRQ.publish()08:44
cprovcarlos: in fact DRQC.publish()08:44
SteveAdaf, matsubara: hi guys.08:44
carlosinstead of NascentUpload.process() ?08:45
matsubarahi SteveA 08:45
carlosor before it?08:45
SteveAi have received a user-support request by email.  someone needs to sign the CoC in launchpad, but is having trouble doing so.08:45
SteveAi'll reply to the email, and cc matsubara and daf on it.  okay?08:45
SteveAyou can sort out between you who will reply08:45
cprovcarlos: no, that is right, leave it there, since you have upload an NEW bin to dapper you need to accept then the 'process-accepted' can publish it or by hand you can directly publish it 08:47
cprovcarlos: I'm extending the test for you08:48
carlosok08:48
matsubaraSteveA: ok08:50
cprovcarlos: add this before kill librarian 'queue_item.customfiles[0] .publish(MockLogger())'08:54
cprovcarlos: spr.attachTranslationFiles() is broken08:54
SteveAthanks08:54
carloscprov: that's the point for this ...08:55
cprovcarlos: I see :)08:55
carloscprov: from where could I get queue_item?08:55
carlosOh, I have it already08:55
carlossorry08:55
cprovcarlos: yup08:56
carloscprov:  ForbiddenAttribute: ('customfiles', <DistroReleaseQueue at 0x35989f0c>)08:57
cprovcarlos: (17:28:11) cprov: carlos: I'm pushing my branch, you'll need it for a small fix in DRQ interface08:58
cprovcarlos: you need my branch or fix it by hand in yours08:58
carlosoh08:58
carlosok08:58
carlosthat's the fix08:58
=== carlos merges
AlinuxOScarlos, buenas dias :)09:00
AlinuxOSwhen there will be Dapper in Rosetta ? :)09:01
carlosAlinuxOS: hola :-)09:01
AlinuxOS;)09:01
=== _mvo_ [n=egon@pD951DF50.dip0.t-ipconnect.de] has joined #launchpad
=== dsas [n=dean@host81-129-228-69.range81-129.btcentralplus.com] has joined #launchpad
=== mpt [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
dsashi, there are two teams in launchpad that are seemingly the same team. One has the various members of the doc team as members, but no packages, the other has the tech board as a member but no-one else and owns the doc packages09:25
dsasThe teams being ubuntu-doc and ubuntu-doc-lists, should the two teams be merged together?09:26
=== ajmitch_ [i=ajmitch@port163-136.ubs.maxnet.co.nz] has joined #launchpad
Seveasdsas, you'd better discuss that with the members of those teams09:33
=== bradb heads off, later all
=== bradb [n=bradb@modemcable033.209-70-69.mc.videotron.ca] has left #launchpad []
dsaswell Corey Burger of the docteam said it seems that it's likely, I'll ask someone from the tech board too.09:35
=== Alinux [n=Ubuntu@d81-211-214-77.cust.tele2.it] has joined #launchpad
=== Alinux [n=Ubuntu@d81-211-214-77.cust.tele2.it] has left #launchpad ["Ex-Chat"]
=== cprov points the joke of the day: ... drescher.ubuntu.com (82.211.81.167): icmp_seq=13 ttl=48 time=3802 ms
dsasSeveas: there's no need for a merge afterall, sorry.10:01
=== mpt [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
=== lfittl [n=lfittl@85-125-145-79.dynamic.xdsl-line.inode.at] has joined #launchpad
=== _dsas [n=dean@host81-129-228-69.range81-129.btcentralplus.com] has joined #launchpad
=== _dsas is now known as dsas
=== luka74 [n=luka74@clj46-234.dial-up.arnes.si] has joined #launchpad
luka74Is Malone down? I get error: OOPS-44C718 when trying to submit bug 10:39
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/44C71810:39
luka74(or just clicking on "Choose..." for package)10:39
matsubaraluka74: known problem, that is bug 30979. 10:40
Ubugtumalone bug 30979 in malone "Oops in popup to select a Ubuntu package" [Normal,Fix committed]  http://launchpad.net/bugs/3097910:40
kikolet me see10:41
kikoluka74, this should be fixed in production by tomorrow noon UTC10:41
luka74OK, actually on "Add" bug I get different code: OOPS-44D752.10:42
Ubugtuhttps://chinstrap.ubuntu.com/~jamesh/oops.cgi/44D75210:42
luka74(I try to add bug to "linux-image" package)10:42
sivangSeveas: how does Ubugtu know where the traceback of an OOPS ?10:50
Mezgodamn this annoying package thing10:50
kikosivang, it just uses a hardcoded URL 10:51
kikoMez, what's p?10:51
MezIs there a simple way in malone to import a bug from debian's PTS10:51
Mezkiko: pacakge was the wrong word10:51
MezI meant GPG sig thing10:51
kikooh. 10:51
Seveassivang, OOPS-(?P<oops>\d+[a-z] \d+) -> https:://chinstrap.ubuntu.com/~jamesh/oops.cgi? + match.group('oops')10:51
kikoI don't think debzilla has been ported yet.10:52
Mezkiko: damn - It's annoying cause am using malone as central bug tracking - want to bring in bug from debia n -have to make it myself then link10:52
kikothat's annoying, yeah. I need to check with mdz what the status is on debzilla, if he needs us to port it.10:53
MezSeveas, I'm assuming the OOPS lookup is meant to be protected?10:53
luka74BTW: submit to "linux-686" package works - strange!10:54
kikoprotected?10:54
Mezkiko: looking at https://chinstrap.ubuntu.com/~jamesh/oops.cgi asks me for username and password10:54
kikochinstrap is private, yeah10:55
kikowe need to move this stuff elsewhere10:55
kikoreason it's private is just that it shares responsibility with other apps10:55
=== ogra [n=ogra@ubuntu/member/ogra] has joined #launchpad
=== dsas [n=dean@host81-129-228-69.range81-129.btcentralplus.com] has joined #launchpad
=== mpt [n=mpt@219-89-153-216.jetstart.xtra.co.nz] has joined #launchpad
kikoCan somebody check out https://chinstrap.ubuntu.com/~jamesh/oops.cgi/2006-02-13/D752 and file a bug as relevant?11:26
kikoBjornT or SteveA or mpt or someone?11:26
kikogone!11:27
=== lbm [n=lbm@x1-6-00-13-10-7a-d1-e4.k233.webspeed.dk] has joined #launchpad

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