lifelessdo we still use Pygpgme ?00:08
lifelesswgrant: https://bugs.launchpad.net/launchpad/+bug/18483500:09
_mup_Bug #184835: Add status '9' (which equates to WONTFIX) to the Roundup external bug tracker <bugwatch> <lp-bugs> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/184835 >00:09
lifelesswgrant: is that fixed incidentally?00:09
wgrantlifeless: No, but it's no longer an OOPS, so it's Low.00:09
lifelesswgrant: what about https://bugs.launchpad.net/launchpad/+bug/125068 ?00:13
_mup_Bug #125068: Bugzilla bug watch updater crashes on POSTs that return a HTTP error <bugwatch> <lp-bugs> <oops> <story-reliable-bug-syncing> <Launchpad itself:Triaged> < https://launchpad.net/bugs/125068 >00:13
wgrantlifeless: Some of those are fixed, others are not.00:13
lifelessI wonder if leonardr's work has fixed https://bugs.launchpad.net/launchpad/+bug/106338 ?00:13
_mup_Bug #106338: Editing a bug targeted to a release crashes if you directly edit the untargeted task  <lp-bugs> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/106338 >00:13
wgrantI need to check waht still shows up.00:13
lifelesshe says it hasn't fixed it00:14
lifelessbut I wonder if we're lucky00:14
wgrantHe tested it.00:14
wgrantAnd said it wasn't fixed.00:14
lifelesswgrant: hey - https://bugs.launchpad.net/launchpad-buildd/+bug/497772 - would love to get bm crashes oopsing00:25
_mup_Bug #497772: exceptions.AttributeError: 'DebianBuildManager' object has no attribute '_subprocess' <oops> <Launchpad Auto Build System:Triaged> < https://launchpad.net/bugs/497772 >00:25
wgrantlifeless: So, roughly, "fuck no"00:35
wgrantGenerating OOPSes from user-controlled code == bad00:36
wgrantIt's also really hard, since there is no access to the slaves except through lp-buildd, and that's what's broken here.00:37
lifelesswgrant: uhm, thats not what I was saying00:39
lifelesswgrant: if the build manager blows up, it should write an oops.00:39
lifelessditto lp-buildd00:39
wgrantWhere should lp-buildd write an OOPS to?00:39
lifelessif the *build* fails, thats not lp-buildd or the build manager blowing up00:39
lifelesswgrant: hand it off to the bm00:40
wgrantAnd how do we know a user isn't going to run their own lp-buildd and generate millions of OOPSes?00:40
lifelesswgrant: can they talk to the build master?00:40
wgrantlifeless: I can easily hijack a buildd and get my own lp-buildd to run there.00:40
lifelesswgrant: on the host os?00:40
wgrantlifeless: Outside the chroot, but inside the VM.00:41
lifelesswgrant: isn't that on a different IP ?00:41
wgrantlifeless: lp-buildd runs inside the VM.00:41
wgrantWe don't talk to the host at all.00:41
wgrantWe don't know that the host exists.00:41
wgrantWe just know that there is a command that we can run to reset the VM.00:41
wgrant(because we will need to use alternative virt solutions in the future *cough* ARM *cough*, where there is no Linux on the host)00:41
lifelessyou have a good scary story there00:42
lifelessif we get an oops from lp-buildd, its time to kill the build00:43
lifelessthat prevents millions of oopses from one instance00:43
wgrantWhat does killing the build mean?00:43
wgrantKilling the build does not kill lp-buildd.00:43
lifelesswgrant: AIUI we restart the buildd per build because of precisely the trust issues you allude to00:44
lifelesswgrant: so I mean that00:44
wgrantlifeless: At the moment we only restart it when the next build starts. But that is itself a security issue, so we should probably fix that.00:44
wgrantThere seem to be three different ways an externalbugtracker can indicate that the importance is unable to be converted :(01:02
wgrantlifeless: Bug #422960 is cheating.01:04
_mup_Bug #422960: appear to be failing to record oops for all +translate HTTP 503 errors <canonical-losa-lp> <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/422960 >01:04
lifelesswgrant: I disagree :)01:04
lifelesswgrant: it falls into the bucket of a bunch of operational improvements that we should do, IMO01:06
wgrantAlmost OOPS report time.01:08
lifelessalmost 11 second timeout time01:08
wgrantWhat did we start at?01:09
lifeless17 on edge01:09
lifelessstep 1 was making both 1701:09
wgrantI was going to say 17, but I thought that seemed a bit odd.01:10
lifelesssince then we've ratched as we fixed things01:10
lifelessedge was lower 'to catch slow pages' - but has a different user population, so not very effective at that01:10
StevenKAnd edge still isn't dead. :-(01:11
lifelessits in the queue01:11
wgrantBut at least it no longer has a shorter queue.01:12
wgrantlifeless: Your gardening has nearly been successful.01:15
wgrantBut not quite :(01:15
StevenKI'd like to rewrite my Firefox URL suggestions, since most of those tend to be edge URLs01:15
lifelesswgrant: thats just the first 7501:17
lifelessStevenK: sqlite301:17
lifelessStevenK: update ...01:17
lifelesswhere ...01:17
StevenKlifeless: Er, which of the 4,000,000 sqlite databases? :-)01:20
wallyworldthumper: got a sec on mumble?01:23
StevenKsqlite> select count(*) from moz_places where url like '%edge%';01:27
wgrantIf I "accidentally" break bug searching and manage to globally corrupt the history so the old code is irretrievable, does writing an implementation does not UTTERLY SUCK become critical?01:35
wgrants/implementation/implementation that/01:35
StevenKHow do you plan to globally corrupt the history? :-)01:37
wgrantThat's one part of the plan that is not yet entirely defined.01:37
StevenKwgrant: To quote Penny Arcade: Stay Good!01:38
wgrantlifeless: Is bug #558585 valid?01:40
_mup_Bug #558585: librarian outages cause OOPSes on pages showing BranchMergeProposal diffs <code-review> <lp-code> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/558585 >01:40
wgrantThat's like complaining that Launchpad doesn't work when the DB is down.01:40
wgrantExcept that mizuho is shit and a SPOF.01:40
StevenKNext p-s-c run will hit SPR 9800001:42
lifelesswgrant: its arguable01:43
lifelessI'm not sure if we should say 'yes, yes it does'01:43
lifelessor 'we should degrade gracefully'01:44
wgrantStevenK: Isn't the much faster than we thought?01:48
wgrantOr are lots of early SPRs missing?01:48
StevenKLots of early SPRs are missing changelogs01:49
wgrantI mean missing in general.01:49
wgrantI don't know what evil was done before the first import.01:49
StevenKAh. The first SPR id run across was 5951001:49
lifelesswgrant: are you still hacking on https://bugs.launchpad.net/launchpad/+bug/575450?01:49
_mup_Bug #575450: Archive:+copy-packages nearly unusable due to timeouts <lp-soyuz> <ppa> <timeout> <Launchpad itself:In Progress by wgrant> < https://launchpad.net/bugs/575450 >01:49
wgrantlifeless: The next piece of work in that is bug #733071.01:50
_mup_Bug #733071: Archive:+copy-packages slow when potential conflicts detected <timeout> <Launchpad itself:Triaged by wgrant> < https://launchpad.net/bugs/733071 >01:50
wgrantBut there is also work needed to speed up the UI, but I haven't analysed that yet.01:50
lifelesswgrant: there is a stall condition here01:50
lifelesswgrant: if you split out a bug like this, please assign yourself to all components, or only leaf components01:51
lifelesswgrant: you see why ?01:51
wgrantYeah, sorry.01:51
lifelessno worries01:51
lifelessbigjools pinged me about this morning when discussing the drop to 11 seconds01:51
lifelesshes concerned about usability during release01:53
wgrantThat's +queue.01:53
wgrantNot +copy-packages01:53
StevenKwgrant: If my one-liner of evilness is right, we've processed 38k SPRs since the 15th.01:54
wgrantStevenK: Sounds reasonably plausible.01:54
StevenKPity there is another 500+ k01:54
lifelesswgrant: bah, emybrain01:55
wgrantSpeaking of 11 seconds.01:55
wgrant424 Time Outs01:56
StevenKYou didn't want 424?01:57
wgrantWe had 252 yesterday.01:57
wgrantSo, no.01:57
StevenKwgrant: The BUILDMASTER oops concerns me01:57
StevenKSince the same thing happened yesterday too01:57
wgrantStevenK: That normally just means the source has been superseded.01:57
wgrantIt often happens.01:57
wgrantI thought jelmer had fixed that, but maybe not complete.01:58
StevenKI guess EB and ED are also appservers?02:02
wgrantEB and ED are edge appservers.02:02
StevenKThey need more fire.02:02
wgrantWe need to get rid of potassium and gandwana.02:03
wgrantThe +copy-packages GET timeouts are all high non-SQL on gandwana and potassium.02:03
wgrantlifeless: When are the new appservers happening?02:04
lifelesswgrant: probably not till mthaddon is back02:05
lifelesselmo was checking with charlies/nafallo about the memory02:05
wgrantIs the session queue normally sitting at 0?02:06
wgrantie. would it help to deprioritise gandwana/potassium instances?02:06
lifelesscve:+index spiked02:06
lifelessjcsackett: has a fix for that02:06
wgrantHe does.02:06
lifelessis it landed?02:06
wgrantKanban says it's in review.02:06
wgrantChecking it.02:06
lifelessdropping the timeout02:06
StevenKWow, apparently windmill is fixed02:33
lifelesshave we done anyting to improve https://launchpad.net/ubuntu/+source/kde4libs/+publishinghistory ?02:39
jtvStevenK: did they use the great "rm -rf" debugger?02:47
wgrantStevenK: Windmill is not fixed, but wallyworld fixed the broken test.03:06
wgrantlifeless: My SPR.copyright fix will have improved that.03:07
wgrantIt still times out occasionally, though.03:07
thumperhttps://code.launchpad.net/~thumper/launchpad/blueprint-linked-bug-tasks/+merge/53734 anyone?03:13
thumperI have three other reviews which are the start of the pipeline03:14
thumperbut they are trivial03:14
thumperand I'm tempted to self review03:14
thumperbut if someone wants to look at them03:14
thumperhere they are:03:14
wgrantIn bugtask-tales-addition you seem to be trying to reimplement structuredstring.03:15
thumperwgrant: what do you mean?03:16
wgrant_make_title escapes stuff.03:16
thumperwgrant: I implemented it in the same way that the main link is generated03:17
thumperwgrant: I didn't think too hard about the implementation03:17
wgrantYes, and most of our code is buggy :(03:17
thumperis that bit buggy?03:17
wgrantIt's not buggy, but it's not a pattern that we want to perpetuate.03:18
wgrantIf you are calling cgi.escape, you are probably doing it wrong.03:18
thumperwhere is structuredstring?03:18
wgrantcanonical.launchpad.webapp.menu, or somewhere stupid like that.03:19
wgrantYeah, canonical.launchpad.webapp.menu.structured is the constructor.03:19
StevenKwgrant: Hm, do you know the diff?03:22
wgrantStevenK: Which diff?03:22
StevenKwgrant: wallyworld's that fixed the failing windmill test03:23
wgrantStevenK: r1259303:24
thumperwgrant: can you think of a better place than lp.services.utils for structured?03:25
thumperStevenK: it isn't really LP specific03:27
wgrantI think lp.app is probably more appropriate.03:27
wgrantBut I'm not really sure.03:27
lifelessSpamapS: ping03:27
lifelesshttps://bugs.launchpad.net/launchpad/+bug/627974 - the url I got from you, did you get that from launchpadlib, or were you url hacking ?03:27
_mup_Bug #627974: MaloneApplication:CollectionResource is slow/times out <lp-bugs> <pg83> <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/627974 >03:27
thumperwgrant: lp.app.browser.string ?03:29
thumperor lp.app.browser.structured03:30
wgrantthumper: string, I think.03:30
wgrantI don't know.03:30
thumperit needs an interface module too03:31
thumperI'd not be happy putting it in lp.app.interfaces.launchapd03:31
thumperor launchpad even03:31
thumperI'd be happy to have the interface and implementation in a single file in lp.services.string03:31
wgrantIndeed not.03:31
thumperand include escape and structured03:32
wgrantBut lp.services is not webapp-specific, so lp.services.string is wrong.03:32
wgrantlp.services.escaping, maybe.03:32
lifelessl.c.l.w.string then03:32
wgrantBut I'd prefer lp.app somewhere.03:32
thumperlifeless: we are trying to remove things from c.l.w03:33
thumperwgrant: lp.app with two files?03:33
lifelessthumper: I know, but that seems pointless to me, the angst about it is dwarfed by the number production and deployment issues spent fixing moves03:34
lifelessI can't bring myself to care about labels03:34
lifelessthe arguments for moving are that the old structure impeded splitting the tree, but the analysis for splitting the tree was wrong03:35
* lifeless shrugs03:35
thumperperhaps we should just move canonical.launchpad.webapp to lp.app.webapp03:36
wgrantthumper: I am not sure if bugtask-repr is safe.03:36
thumperwgrant: why?03:36
wgrantthumper: It potentially exposes the bug ID and target, if you manage to get hold of a BugTask through the webservice.03:37
thumperwgrant: the webservice handles launchpad.View03:38
thumperwgrant: and a bugtask view permissions filter to the bugs03:38
thumperwgrant: so not a valid concern I believe03:38
wgrantthumper: The webservice handles launchpad.View for collections, sometimes.03:38
thumperhaving that repr certainly helped my debugging03:38
wgrant8 of todays top 10 OOPSes are fixed. Yay.03:43
wgrantThe next few are codehosting.03:43
wgrantBlue squad sounds like a good one to take those :P03:43
StevenKwgrant: I'm having a such a brain fade -- can you remind how to get the latest version of an SPR in a distroseries.03:47
wgrantStevenK: That doesn't really make sense. But getCurrentSourceReleases might do what you want.03:48
wgrantDepending on what you actually mean.03:48
wgrantIt is ambiguous in a couple of ways.03:48
StevenKwgrant: So, Julian would like to delete the DSD is the derived series and parent series have the same version published. This makes sense to me, but I'm having a brain fade looking for the method to help me.03:49
wgrantThat's deleting the comment history.03:50
wgrantBad idea to do that so early.03:50
wgrantYou can possibly justify doing that once a series is done.03:50
wgrantBut not before.03:50
StevenKWhy? If something has been synced, the comments are now irrelevant/03:51
wgrantStevenK: I want to know why it was synced.03:52
StevenKThen you can look up the bug.03:52
wgrantThere wasn't a bug, because we are no longer in the dark ages.03:52
wgrantThe sync button has been implemented.03:52
StevenKGood point.03:53
thumperhow do we avoid the ValidPersonCache queries?03:58
thumpercan we precache it?03:58
wgrantthumper: list(getUtility(IPersonSet).getPrecachedPersonsFromIds(ids, need_validity=True))04:01
thumperwgrant: I have the people already from a different join04:02
thumperalthough looking at this method, I may change it04:04
lifelessthumper: are you filtering on person ?04:06
thumperlifeless: I'm looking at two timeouts04:07
thumperfor spritns04:07
thumperone on +index seems to be getting valid persons04:07
thumperand +attendees-csv is just shite04:07
lifelessbug #'s ?04:07
thumperthe attendees-csv could be trivially fixed04:07
thumperexcept for the ircnicknames04:07
thumperbug 73599604:08
_mup_Bug #735996: Sprint:+index timeouts <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/735996 >04:08
thumperthere isn't a bug for +attendees-cvs db counts04:08
thumperircnicknames is a SQLMultipleJoin04:09
thumperso not sure how to precache the names for all the people :-(04:09
wgrantYou'll have to make it a cachedproperty returning a list.04:09
thumperwgrant: but that doesn't mean we can get them all at once04:10
wgrantthumper: It does...04:10
wgrantstore.find(IRCNick, IRCNick.personID.is_in(personIDs)04:10
wgrantgrep around for pre_iter_hook04:10
thumperI'll look around04:11
lifelessjtv: perhaps you could comment on https://bugs.launchpad.net/launchpad/+bug/70887504:28
_mup_Bug #708875: AssertionError: More than 6 plural forms were submitted <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/708875 >04:28
jtvlifeless: should be an unexpectedformdata if it happens from the web UI.04:28
jtvSeems a far-out guess that this is coming from the web UI, but I'm pretty sure I made the importer check for this condition way back when.04:29
wgrantI think the check is wrong.04:30
wgrantI looked at it yesterday and ran away.04:30
wgrantThere are only forms 0-504:30
wgrantWhich is 6.04:30
wgrantNot more than 6.04:30
jtvThe check doesn't even make sense to me.  It's an "else" attached to a "for" loop AFAICS04:36
wgrantThat does make sense.04:36
wgrantBut it's not right here.04:36
wgrant(the else block is executed if the loop runs to completion, without a break)04:37
wgrantPretty much.04:37
jtvSo yes, that explains the off-by-one04:37
lifelesswhich is the absolute opposite of what *everyone* reads it as04:37
jtvThe loop breaks when it hits the first plural form that is supported but is not in the form.04:37
jtvAnd when you submit exactly the maximum number of forms, it never does.04:37
wgrantBut I thought this must have come up often.04:38
wgrantDoes it not?04:38
lifelesswgrant: still working on bug 714414 ?04:39
_mup_Bug #714414: unstack debian sid branches <code-hosting> <stacking> <Launchpad itself:Triaged by wgrant> <Ubuntu Distributed Development:Fix Released by mbp> < https://launchpad.net/bugs/714414 >04:39
jtvProbably, yes, but plural forms are relatively rare, and only Arabic uses 6.04:39
wgrantjtv: Right, but wouldn't it occur on every plural arabic translation, then?04:39
wgrantOr is the last one uncommon?04:40
jtvEvery complete plural arabic translation made in the UI, yes.04:40
wgrantGiven the debate around ubuntu-l10n-ar, I guess we don't have many arabic translators.04:40
wgrantlifeless: Not really.04:40
jtvAn upstream translation. or one uploaded by a translation team member, wouldn't have this problem.04:41
jtvI'm expounding on the bug.04:41
lifelesswgrant: unassign?04:44
lifelessis https://bugs.launchpad.net/launchpad-buildd/+bug/719162 deployed?04:46
_mup_Bug #719162: [Natty] check-implicit-pointer-functions fails on natty, resulting possibly broken packages <qa-ok> <Launchpad Auto Build System:Fix Committed by julian-edwards> < https://launchpad.net/bugs/719162 >04:46
wgrantlamont was looking at that earlier this week. We need an amd64 builder on staging before we can QA it.04:48
wgrantSo no, it's not yet deployed.04:48
lifelessstub: hey04:52
lifelesscan you see how long the very big query in bug 722787 takes now on prod?04:53
_mup_Bug #722787: Product:+filebug timeouts <timeout> <Launchpad itself:Triaged> < https://launchpad.net/bugs/722787 >04:53
lifelessI suspect the index repack fixed it04:53
* stub tries to find a real query04:59
lifelesswgrant: and are you doing 729064 ?05:01
wgrantlifeless: Yes.05:08
stubI can't even cut and paste the query without making terminal explode :-P05:08
stubhttps://lp-oops.canonical.com/oops.py/?oopsid=1875O1575 has bigger problems than just timing out05:08
lifelessstub: :P05:08
lifelesslove our ui05:09
lifelessThere are currently no open bugs.05:09
stublifeless: So the insane query in that oops runs in 3 seconds atm. on one of the slaves.05:11
lifelessstub: great,thanks05:12
* lifeless closes05:13
lifeless224 critical bugs05:13
lifelessincluding 6 5-digit ones05:13
lifelessstub: going to cook dinner; catchup after that?05:14
pooliere bug 719271, are there multiple threads per scanner process?05:24
_mup_Bug #719271: Branch scanner jobs failing in bzrlib <branch-scanner> <bzr> <oops> <Launchpad itself:Triaged> < https://launchpad.net/bugs/719271 >05:24
lifelessstub: ping06:20
stublifeless: pong06:24
wgranthttps://staging.launchpad.net/successful-updates.txt is odd.06:28
lifelessstub: https://dev.launchpad.net/LEP/ReliableDBDeploys06:30
wgrantCan we move to Python 3 please?06:37
StevenKHow do you suggest we move Zope and Storm to Python 3?06:37
StevenKOh, you don't want problems, you want solutions?06:38
wgrantI want non-awful Unicode.06:39
wgrantI also want lucid_db_lp to fail sensibly.06:39
wgrantBecause its current failure is not sensible.06:39
stublifeless: tasque07:16
lifelesswgrant: happier with the bug count now ?07:30
wgrantlifeless: Mostly.07:35
wallyworldwgrant: there's a question about getting increased space for a ppa. who do i assign that to?07:47
wgrantwallyworld: A LOSA, or poke a commercial admin07:51
spmthat doesn't sound right. we haven't been doing space increases for ages. can you guys already do that?07:52
spmyourselves as in. ?07:52
wallyworldspm: if we can do it, i don't know how :-)07:53
spmit's an admin option off the ppa itself. or was.07:53
wgrantIt needs a commercial admin or a LOSA, and there is approximately one active commercial admin.07:53
wallyworldi also have a question about a confirmation code being rejected when someone tried to create an account. assignee?07:54
wgrantwallyworld: Ask them to file a ticket at the SSO support site.07:54
spmthat needs a level of sanity filtering first07:54
wgrantOr reassign the question to canonical-identity-provider07:54
wallyworldwgrant: thanks. will reassign to cip. what's the sso support site?07:55
wallyworldspm: if it is an admin option on the ppa i can't see it. can i ask you to increase the space for the user?08:01
stubwgrant: You were correct about Bug #73661308:01
stub(The ℃ bug since mup seems to have run out of blow)08:02
* wallyworld has soccer training. back later08:03
bigjoolsmorning hackers08:48
adeuringgood morning08:56
jtv1hi adeuring09:02
adeuringhi jtv1!09:03
jtv1StevenK: I realize it's late for you, so you may want to ignore this.  I have minimal tests running for cron.publish-ftpmaster, but basically without any packages.  Next on my wishlist is to make it produce meaningful data so I can test that I don't break it.09:04
wgrantYou managed to get it to run locally?09:05
=== jtv1 is now known as jtv
jtvIn /tmp, not /srv/launchpad.net09:05
jtvThe test should be able to reproduce it without any system changes.09:06
jtvAlso, I changed it so it'll run against ubuntutest.  Not sure yet if that's particularly useful.09:07
wgrantKnowing the state of the sampledata, it's probably best to run against a completely new distro.09:07
wgrantOr you are going to have fun when you start trying to publish stuff.09:07
jtvYes, that's what I was hoping to do.09:07
jtvProblem is, I'm lost in the dark.09:08
lifelessin a completely temp dir09:08
jtvall alone09:09
jtvand it's freezing09:09
wgrantDon't pollute my terminal with your wide characters again.09:09
jtvmy… what?09:10
wgrantOh wait, that was stub.09:10
jtvah—nasty stub09:10
jtvIs ญ a wide character?09:10
jtvAnyway, wgrant, what would I have to set up?  I'm guessing distro, distroseries, sourcepackagename, sourcepackagerelease, sourcepackagepublishinghistory… anything else?09:11
wgrantYes, but not as wide as the ℃  I was terrorised with earlier.09:11
jtvWait, there's a single character for °C?09:12
stubI can whine in ℉ if you would rather09:12
jtvHow do I type those?09:12
stubAnd if you think I'm bitching about the cold, you should hear my maid.09:13
wgrantjtv: Distribution, DistroSeries, DistroArchSeries to start. Then you can create some stuff to publish... SPPHs (you may have to add files manually) to test publish-distro, and PackageUploads with PackageUploadCustoms to run through process-accepted to test dist-upgrader signing.09:13
jtvHoly cow, it's 18°!   Insane!09:13
stubYou mean 18℃09:13
jtvwgrant: quite a shopping list already… thanks09:13
wgrantstub: At least there was no exclamation mark that time.09:14
jtvstub: yes, but I only know how to type that as °C09:14
jtvIs there a problem with an exclamation mark after ℃?09:14
jtvOr as the Danes would say, Åååå.09:15
* jtv is beginning to wonder whether wgrant's terminal is perhaps not entirely prepared for some non-ASCII characters somehow.09:16
stubBug #736613 is the "℃" problem09:16
jtvwgrant: thanks, that gives me something to aim for09:17
bigjoolsif you use an irc client developed this century, it would probably work09:17
lifelessit works in irssi09:17
wgrantThis irssi.09:17
wgrantIt depends on your terminal emulator and font.09:18
lifelessok, irssi with real terminal and font :P09:18
jtvwgrant: the files I may have to add manually are changelogs for the SPRs?  Or something else?09:20
wgrantjtv: SPRFs09:20
jtvIsn't that a view?09:20
wgrantThe view is SPFP09:20
jtvHow could I have been so stupid?09:20
wgrantI know, right.09:20
bigjoolsI knew that would provoke a reaction :)09:22
jtvhi bigjools :)09:23
bigjoolshelleau jtv09:25
sidneianyone around for a review?10:28
poolie(just waving, not volunteering :)10:31
LPCIBotProject devel build #547: FAILURE in 4 hr 38 min: https://hudson.wedontsleep.org/job/devel/547/10:34
wgrantlp.services.job.tests.test_runner.TestTwistedJobRunner.test_timeout as usual.10:37
sidneioh, hi poolie :)10:37
gmbThe Kanban board should pop a message when I move a card into the review lane that says "Are you sure, little man?"10:38
sidneigmb, with a picture of clippy?10:41
gmbYeah :)10:41
lifelessjml: ping11:25
jmllifeless: pong11:26
lifelessjml: https://dev.launchpad.net/LEP/ReliableDBDeploys11:26
jmllifeless: yeah, I saw your earlier IRC comment about it. It's on the list.11:26
lifelessjml: thanks11:26
lifelessjml: also, I just replied to bug 655385 - with some ideas about solving a few of our bug challenges all at once11:27
_mup_Bug #655385: Allow bug status change from Triaged only for bug supervisor <accesscontrol> <acl> <bugs> <easy> <lp-bugs> <Launchpad itself:Opinion> < https://launchpad.net/bugs/655385 >11:27
lifelessjml: I'd be interested in your thoughts on my comment (which was mailed in so possibly not visible yet11:27
jmllifeless: ok11:28
lifelessits a little adlib, but I felt the pieces going 'click' as I wrote11:28
=== henninge is now known as henninge-lunch
lifelessman, its *really* hard to enter a tag that is a prefix of an official bug tag11:32
lifelessjml: its there - https://bugs.launchpad.net/launchpad/+bug/655385/comments/1411:40
_mup_Bug #655385: Allow bug status change from Triaged only for bug supervisor <accesscontrol> <acl> <bugs> <easy> <lp-bugs> <Launchpad itself:Opinion> < https://launchpad.net/bugs/655385 >11:40
lifelesshmm, night all11:51
lifelessbigjools: if you think some of the things I changed should still be critical, please just change them back: I was doing a big sweep as I said, and 100% accuracy is hard to achieve11:51
bigjoolslifeless: ok, I didn't want to be too presumptuous :)11:52
lifelessbigjools: long as you include a reason (so that in 3 months I don't toggle again, blindly :)) it will be fine11:52
wallyworldjtv: there's a translation question i have no clue about. can i assign it to you?11:53
=== matsubara-afk is now known as matsubara
jmllifeless: !!!11:54
lifelessjml: ?11:55
lifelessjml: you caught me just as I was about to walk away from the keyboard... whats up?11:56
jmllifeless: oh, just that at least one of the bugs that were marked down had been previously marked Critical without comment.11:56
jmllifeless: it's no big deal11:56
lifelessjml: presumably by me? I do try to comment consistently, though I will admit I rarely bother when the bug is tagged oops or timeout from the start11:57
jmllifeless: yeah. it was one or two out of about thirty, and not bugs I care about personally.11:59
lifelessjml: if it was the getBranches on sourcepackage one, there was a comment already on the bug saying it no longer oopsed; I was lazy there ;)12:00
=== jtv1 is now known as jtv
jmlI've got some PPA stats12:58
jml- how many PPAs are there?12:59
jml- how many packages are being uploaded to PPAs?12:59
jmlbigjools: awesome :)13:00
wgrantThere are also a few graphs on lpstats.13:00
bigjoolsyeah w3as just searching for some good ones for jml13:00
bigjoolslike https://lpstats.canonical.com/graphs/NewPPAsWithUploads/13:00
wgrantThere aren't really any good ones.13:00
wgrantThat's the only interesting one.13:01
jmldo those "published sources" and "published binaries" numbers collapse versions?13:01
bigjoolspublished implies collapsed13:01
bigjoolsPPAs don't let you publish more than one version13:02
bigjoolsof the same package, I mean13:02
wgrantIf you mean distinct source versions? No.13:02
wgrantIf I have the same package in multiple series or PPAs, it will show up a few times in that number.13:02
jmlI guess what I mean is COUNT(DISTINCT (archive, sourcepackagename))13:02
bigjoolsah, no idea13:02
bigjoolsanyway I am desperately hungry13:02
jmlok, thanks.13:02
jmlI'm getting a failure in stable running ./bin/test -cvv lp.services.job.tests.test_runner test_timeout13:26
wgrantjml: Yes, that test fails on its own.13:26
wgrantAnd sometimes in the full test suite.13:26
jmlit's consistent, and it's not like either of the failures mentioned on bug #50591313:26
_mup_Bug #505913: TestTwistedJobRunner.test_timeout fails intermittently <lp-foundations> <spurious-test-failure> <Launchpad itself:Triaged> < https://launchpad.net/bugs/505913 >13:26
wgrantjml: Is it a failure to import _pythonpath?13:26
jmlwgrant: yeah.13:26
jmlwgrant: you also get this failure when running the test standalone?13:27
wgrantIt needs someone who knows that code to look at it.13:27
wgrantI spent a couple of hours on it and made very little progress.13:27
jmlwgrant: I'm keen to have a try today, if I can deal with prior obligations.13:28
jmlwgrant: just to be crystal clear, you get the _pythonpath import failure when you run the test by itself locally?13:29
wgrantjml: Yes.13:29
wgrantjml: Working around that in ways that I no longer recall (possibly hacking site.py in the custom PYTHONPATH it uses) gets the same message as the spurious failure, but I was unable to work out how to get the output of the subprocess.13:30
wgrantSo I made the failure spurious again and ran away.13:30
=== jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: jcsackett | https://code.launchpad.net/launchpad-project/+activereviews/
wgrantjml: I initially suggested doing what you considered, but could not think of a reasonable name for the method.13:57
wgrantSwap suggested and considered.13:57
jmlwgrant: ahh, yeah. I can see how that would be a problem.13:58
* bigjools waves at jcsackett14:04
jcsackettbigjools: publisher config schema review? :-)14:05
bigjoolsjcsackett: yarp :)14:05
jcsackettlooking now.14:05
=== henninge-lunch is now known as henninge
sinzuijcsackett: will you have time to review https://code.launchpad.net/~sinzui/launchpad/person-merge-job-0/+merge/5370614:17
wgrantderyck: I've tried running it 30 times. It doesn't hang :/14:17
deryckwgrant: *sigh*14:17
wgrantderyck: Yes.14:18
jcsackettsinzui: i'll do it next. already had it marked for looking at today.14:18
deryckwgrant: I was afraid of that.14:18
wgrantderyck: It's also only died a couple of times on Jenkins.14:18
deryckwgrant: can you point me at those builds?14:18
wgrantAnd it doesn't die often on ec2 when only running WindmillLayer.14:18
deryckwgrant: also, I'm going to try it in a vm with lower resources and see if that helps cause it.14:19
* deryck is grasping at straws14:19
wgrantThat's a good plan.14:19
wgrantI ran out of straws a week or two ago.14:19
deryckI'm spending today on it and if I get no hangs, then will look at other options -- decoupling from Zope test runner or moving to Selenium.14:21
deryckabentley: I'm good for a js chat now.14:22
abentleyderyck: cool.14:22
wgranthttps://hudson.wedontsleep.org/job/windmill/2/ https://hudson.wedontsleep.org/job/windmill/5/ https://hudson.wedontsleep.org/job/windmill/35/ https://hudson.wedontsleep.org/job/windmill/41/ https://hudson.wedontsleep.org/job/windmill/43/ are isolated spurious test failures. The build I thought was Windmill breakage was in fact the UEC slave going away.14:22
gmbleonardr: Is it possible to pass versioned annotations to export_write_operation (for example, if I want to export a write operation in devel but nothing else)?14:23
wgrantderyck: So we have had 62 builds without a failure.14:23
wgrantIs it something about being in a full test run? Low resources? A race? Who knows...14:23
leonardrgmb: yes, but you can't do it in export_write_operation, because the definition of a named operation takes place over multiple annotations14:24
leonardryou need to use @operation_for_version()14:24
gmbleonardr: So, do I do something like:14:25
leonardrto publish an operation in devel and nothing else, at the bottom of the annotation stack you would have @operation_for_version('devel')14:25
gmb... right.14:25
jcsackettbigjools: so, this branch is just adding these fields, not refactoring anything to make use of them yet?14:29
jcsacketti only see addition, and setting/changing of the fields, not anything using them.14:29
bigjoolsjcsackett: yep, just doing a small focused branch14:29
jcsackettbigjools: cool.14:29
bigjoolsbecause I know that the code change to use this stuff will be pain ridden :)14:30
jcsackettbigjools: in the factory, since those are optional fields, is it entirely appropriate that they're always set?14:30
bigjoolsjcsackett: interesting point14:31
jcsacketti don't know that it's actually a problem...just raising a possible different take on how it should work.14:31
bigjoolsit would be impossible to set them to None14:31
bigjoolswith that code14:31
bigjoolsI'll change that, thanks for spotting14:32
jcsackettbigjools: r=me, with that. :-)14:32
jcsackettsinzui: looking at your MP now.14:34
bigjoolsOur reviews rock lately14:34
jcsacketti think moving to the "just use the review page as the queue" is a big win.14:37
bigjoolsyou weren't around in the old days when we used a wiki page ...14:39
* bigjools shudders14:39
jcsackettthat sounds like it would be problematic, yes. :-P14:40
sinzuiproblematic was an understatement14:42
bigjoolsand a review team of about 5 people14:43
sinzuiWe had to locate a reviewer and prod each over several days to get approval14:43
bigjoolsr=trivial was great for that :)14:43
sinzuiexcept when something broken and kiko looked at the branch and ask how anyone could claim the changes were trivial14:44
LPCIBotYippie, build fixed!14:50
LPCIBotProject devel build #548: FIXED in 4 hr 15 min: https://hudson.wedontsleep.org/job/devel/548/14:50
deryckwgrant: sorry, was on call.  thanks for the build links.  And I'm not sure honestly what it is about the entire run that causes it.14:52
deryckif the Jenkins build is just WindmillLayer, then perhaps it could be changed to run the entire suite + windmill.14:52
jcsackettsinzui: is there a need for a test on mergeAsync in the failure case?14:55
jcsacketti.e. when it returns None instead of a job?14:55
jcsackettah, nevermind. that test exists, just in a different test file.14:55
sinzuijcsackett: That is handled by the base runner and tests. the base defines an empty set of addresses.14:55
jcsackettsinzui: yes; i saw test_mergeAsync_success on test_person and so nothing corresponding, but it's handled elsewhere. :-)14:56
bachi sinzui -- can you look at this paste as a mid-imp sanity check for the menu problem we discussed yesterday?  https://dev.launchpad.net/JavascriptUnitTesting14:58
jcsackettsinzui: r=me.15:01
sinzuibac: thank you for updated the docs15:02
gmbjcsackett: I have an MP for you to review, if you'd be so kind: https://code.launchpad.net/~gmb/launchpad/make-+subscribe-play-nice-bug-735397/+merge/5383915:03
bacsinzui: oops, that was the wrong paste.  :)15:03
bacsinzui: i meant for you to look at http://pastebin.ubuntu.com/581613/15:04
bacbut, yes, i was happy to update those testing docs15:04
jcsackettgmb: looking now.15:06
gmbThankee kindly15:07
sinzuibac: ugh. I do not think we should be injecting line-height into styles. The anchor is making a bad guess. Only <h3>s have a line-height of 20px. Maybe this is why firefox still shows the icons misaligned in pages15:09
bacsinzui: perhaps, but that isn't part of my change.15:09
sinzuiyou cargo-culted it15:09
bacsinzui: i'm happy to fix it at the same time, though15:09
bacsinzui: i replicated the existing code, assuming it was correct, for the case i was adding15:10
bacthanks TAL15:10
=== matsubara is now known as matsubara-lunch
bacsinzui: the gist of my change to the page template is the conditional addition of "display:none"15:10
sinzuibac: why do you use style instead of the invisible-link class. I think I would toggle between two classes <invisible-link|visible-linl>15:11
bacsinzui: good idea15:11
sinzuiwe have supported invisible link since 2006 to make link testing easy. I think we can really use it for something else now15:11
bacsinzui: otherwise a reasonable way to go?15:12
sinzuiyes. your implementation is what I would have done15:12
sinzuiI am worried about spurious line-heights now15:12
sinzuisweet. bac. there is only one occurrence. bac. try removing it and look at the page in firefox and safari. I think they will look the same now15:14
SpamapSlifeless: pong (no packet loss, but latency is close to interplanetary ;)15:14
bacsinzui: thanks.  i'll switch to use invisble-link.  and i'll make the line-height fix15:14
sinzuibac: firefox is still showing the sprites offset from where we intended them. I did not check that there was something overriding the line-height in the markup :(15:15
bacsinzui: these would be for links such as in the global-actions portlet?15:16
deryckhenninge: we're all sorted out for feature flag now?15:17
henningederyck: the request is on LPS15:18
deryckhenninge: ok, thanks.15:18
sinzuibac: this template is used by "fmt:link", which is most links15:19
deryckhenninge: the number in the feature flag has to be incremented above whatever the current highest number is.15:21
gmbjcsackett: I need to go and run some errands; I'll respond to your review comments and questions when I get back.15:21
deryckhenninge: so the 1 is probably something like 20 now?15:21
jcsackettgmb: righto.15:21
deryckjml: ping15:25
jmlderyck: hi15:25
henningederyck: hm?15:25
deryckhenninge: I forgot what that attribute is called, but it has to be current value + 1.15:25
henningederyck: oh, I did not know that. I think it's the priority.15:26
deryckyeah, that's it.  priority.15:26
deryckhenninge: rather, it has to be some value that isn't taken yet. ;)  So everyone does current + 1.15:27
henningederyck: I thought that was only relevant when you have multiple rules for the same flag.15:27
henningeit's to solve those ambiguities, I thought15:28
deryckhenninge: yes, but there's a db constraint it has to be unique across all flags.  the update will fail if not.  it did in the past anyway.15:29
henningederyck: hm, there are multiple rules with "1" and also with "0" in the current rule set.15:29
deryckhenninge: ah, ok.  So guess I'm wrong then. sorry for the noise.15:30
bacsinzui: that line-height styling doesn't seem to have any effect.  i've removed it and bumped it up to very high and see no difference in rendering15:31
sinzuibac: in firefox?15:31
sinzuiWell regardless, we need to remove it. I was hoping that firefox would be fixed15:32
bacsinzui: actually, i was only looking at the global-actions portlet, which uses the same styling15:33
bacsinzui: it *does* have an effect on those in the involvement portlet15:33
bacsinzui: on firefox, in the involvment portlet removing the 'line-height' styling does change the spacing between the lines (yay) but it does not appear to affect the vertical spacing of the sprite relative to the text.15:36
sinzuibac: I do not see a style attribute in the involvement portlet links on https://launchpad.net/gdp15:38
sinzuioh, no icon15:40
sinzuibac: set this aside. I will look into it later15:41
sinzuisorry for the diversion15:41
bacsinzui: np.  i'm easily diverted15:42
=== salgado is now known as salgado-lunch
sinzuijcsackett: I really like your TestTeamParticipationMesh test. I was able to add two new tests, one using setMembershipData on each member, and the other using deactivateAllMembers. The second test fails as I expect. I will know my refactoring is done when both agree15:59
jcsackettsinzui: fantastic.16:00
=== deryck is now known as deryck[lunch]
jcsackettgmb: r=me. left a few questions in a comment, but those are strictly for my education, not issues with the branch.16:12
gmbjcsackett: Thanks. I'll respond to your questions in the MP now.16:13
=== matsubara-lunch is now known as matsubara
=== salgado-lunch is now known as salgado
=== Ursinha is now known as Ursinha-lunch
=== deryck[lunch] is now known as deryck
=== beuno is now known as beuno-lunch
deryckI may have a Windmill hang now!17:19
jelmerderyck: Is that really something to get excited over ? ;-)17:32
deryckjelmer: it's the little things that make me happy :-)17:32
sinzuijcsackett: ping17:37
jcsackettsinzui: pong.17:37
sinzuijcsackett: I have a fix for team participation. Do you have a few minutes to mumble about it?17:37
jcsackettgive me just a few moments to grab a drink, and i will be on mumble.17:38
deryckah.  stuck in a sleep.17:42
deryckI told you all sleep was evil.17:43
bigjoolsjust ask lifeless17:47
deryckoh wow17:48
deryckwindmill imports time.sleep and uses it inside a while statement17:48
=== fjlacoste is now known as flacoste
jmlderyck: that sucks17:59
deryckjml: yes, that's a terrible, scary thing to do, IMHO. But turns out it's unrelated.  Just frightening, but not causing the hang.18:00
jmlderyck: huh18:00
jmlderyck: very glad you're working on this, btw.18:00
deryckjml: looks like were stuck in client.open.  still digging to know for sure. lot of backtrace to get through18:01
deryckand thanks!  glad to be working on it myself18:01
bigjoolsnight folks18:02
deryckcrap.  lost my hung process.18:03
derycknight bigjools18:03
=== beuno-lunch is now known as beuno
deryckMorning, lifeless18:39
lifelesshi deryck18:44
lifelesswgrant: jelmer: can either of you loook at  https://rt.admin.canonical.com/Ticket/Display.html?id=42954 and answer toms debugging question?19:00
lifelessabentley: hey, do you know - do the LP code import slaves need access to the librarian ?19:02
abentleylifeless: I don't know.19:02
jelmerlifeless: looking...19:04
lifelessjelmer: thanks19:04
lifelessjml: if you haven't actually gone - did I make any sense in that bug post I pointed you at?19:05
sinzuijcsackett: Can you review https://code.launchpad.net/~sinzui/launchpad/deactivate-all-members-fix-0/+merge/53885 today?19:29
jcsackettsinzui: sure. i'll take a look in just a bit.19:29
jcsackettsinzui: 458 lines? that's much smaller than i was expecting from your earlier concerns. :-)19:30
sinzuijcsackett: Deleting lots of code is often small then editing it19:34
lifelessjcsackett: hey, how is your cve:+index fix going19:34
jcsackettlifeless: made the changes you pointed out and it's out to land.19:35
jcsackettec2 instance died without output on first try, second try seems to be going alright.19:35
lifelessjcsackett: cool - third highest oops yesterday19:35
jcsackettlifeless: well, let's hope i got everything then. :-)19:36
lifelessjcsackett: if you didn't, we can iterate19:37
=== Ursinha-lunch is now known as Ursinha
jcsackettsinzui: r=me, and thanks for that branch. i feel that it's a pretty big win.20:07
sinzuijcsackett: thank you.20:07
jcsackettthumper: you around yet?20:10
thumperjcsackett: am now, with coffee even20:26
jcsackettthumper: cool. i am looking at https://code.launchpad.net/~thumper/launchpad/bugtask-tales-addition/+merge/5373220:26
jcsacketti think it looks pretty good, but i wonder if now all Formatters will need to define _title_values? it looks like calling it is baked into the base class, but the base class has it as NotImplemented.20:27
jcsackettthumper: it might be better for the base form to just return None on _title_values instead, since that's handled gracefully.20:29
thumperjcsackett: that seems reasonable20:29
thumpercan easily fix20:29
jcsackettthumper: with that change, r=me.20:30
thumperwgrant yesterday also suggested replacing the cgi.escape rubbish with "structured"20:30
jcsackettthumper: yeah, that might be better too.20:30
thumperjcsackett: that branch is one of a pipeline :)20:30
jcsackettthumper: dig. :-)20:30
thumperjcsackett: I only want to land the top20:30
thumperI just broke it up for review20:30
thumperit was getting kinda big20:30
jcsackettthumper: dig.20:31
jcsackettso, r=me on that little bit. :-)20:31
lifelessugh, checkwatches passwords are in lp-prod-configs :(20:43
lifelessabentley: are translation sharing jobs actually running yet ?20:55
abentleylifeless: no.20:55
lifelessabentley: I'm thinking of marking https://bugs.launchpad.net/launchpad/+bug/735954 qa-untestable then20:56
_mup_Bug #735954: Translation merge job display <qa-needstesting> <Launchpad itself:Fix Committed by henninge> < https://launchpad.net/bugs/735954 >20:56
=== jcsackett changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | firefighting: - | On call reviewer: - | https://code.launchpad.net/launchpad-project/+activereviews/
abentleylifeless: I don't think "untestable" is strictly accurate.20:57
abentleylifeless: too-painful-to-test, perhaps :-)20:58
lifelessabentley: indeed, and too little risk; I think we should check the page renders on qastaging20:58
lifelessabentley: what url should I look at to see the sharing stuff ?21:03
abentleylifeless: translations.*/$SOURCEPACKAGE/+sharing-details.21:03
lifelessabentley: I get a 404 on https://translations.qastaging.launchpad.net/ubuntu/+source/bzr/+sharing-details21:05
lifelessah, distro series source package21:06
abentleylifeless: yes, sorry.21:06
lifelessseems to render ok21:07
lifelessand i've clicked around and turned everything I could find on21:07
lifelessabentley: that seems sufficient to suggest production won't be broken by the change21:08
abentleylifeless: does it show that a job's pending?21:10
lifelessabentley: it says No upstream templates have been found yet21:11
lifelessI guess that that is impeding it21:11
abentleylifeless: doesn't sound right.21:12
lifelessabentley: darn, I'll undo my tag change then21:12
lifelessabentley: what should we do to be confident this is deployable21:13
lifeless[that is, that it won't regress or break anything]21:13
bacsinzui: would you have the time and interest to review the branch for the menu links that we discussed this morning?21:15
abentleylifeless: it should say something like this: http://pastebin.ubuntu.com/581807/21:15
abentleylifeless: the fact that it doesn't suggests that we may not have the correct revno on qastaging.21:15
lifelessabentley: 1261721:16
lifelessI'll have a poke at unity instead21:17
lifelessit should have strings21:17
abentleylifeless: I don't know what's going on.  Perhaps the job has not been created.  Perhaps the browser message is being suppressed somewhere.21:19
lifelessabentley: I think its because there are no templates for upstream21:19
abentleylifeless: Oh, did you actually create a packaging link?21:19
lifelessyes, but the upstream hasn't been imported21:20
abentleyI don't recall making job creation conditional on that.21:20
abentleyIt requires the package to be an Ubuntu package, but I think that's all.21:20
wallyworldsinzui: you still there?21:21
lifelessabentley: have a look here - https://translations.qastaging.launchpad.net/ubuntu/natty/+source/gtk+2.0/+sharing-details21:21
lifelessabentley: tell me what you think21:22
sinzuiI am21:22
abentleylifeless: it looks as though this was an already-existing packaging link.  If you just linked it now, then it should have the message.21:23
wallyworldsinzui: i have lost an email you may have sent about dealing with spam. there's a few open questions about removing spam links or accounts sending spam etc. if there a wiki page or email you can flick me which describes sop for that?21:23
wallyworlds/an email/any email21:23
lifelessabentley: oh, ok.21:23
sinzuiNot yet. I may write it tomorrow21:23
lifelessuhm, will look on needs-packaging21:23
abentleylifeless: look now.21:24
lifelessabentley: what did you do?21:24
wallyworldsinzui: ok. so with accounts sending spam, i assume i should try and see if it's a one off (in case they have been hacked) or if it's an ongoing issue and hence disable that account?21:25
abentleylifeless: I deleted the packaging link and then re-created it.21:25
sinzuiwallyworld: I can find the email I sent. I believe we can see suspended users now so the process is simpler. suspend the user, then send an email to the user's preferred email address asking the him to confirm he has control of his computer, browser, and isp21:25
lifelessabentley: ok, cool21:25
lifelessso it looks ok to you?21:25
abentleylifeless: yes.21:25
wallyworldsinzui: will do. thanks.21:25
lifelessabentley: thanks for the help21:26
abentleylifeless: you're welcome.21:26
sinzuiwallyworld: I just sent the email I think you remember. It has the text of the messages I send21:27
wallyworldsinzui: thanks. much appreciated. i need a better email filing system. there's sooooo much of it :-)21:28
sinzuiwallyworld: no you do not. I need to document what I have been doing for the last 15 months21:28
wallyworldsinzui: wow 15 months of dealing with spam! you poor soul. get it document so we all can help out better :-)21:29
bacsinzui: did you see my request ^^?21:31
sinzuibac: sorry I did not. I can review it now21:31
bacthanks, sinzui!  https://code.launchpad.net/~bac/launchpad/hidden_links/+merge/5391121:32
lifelessrockstar: whats jsoops?21:32
abentleylifeless: it's what Brendan Eich says when he reviews his life's accomplishments.21:35
rockstarlifeless, it's our way of logging js errors.21:44
lifelessrockstar: they get passed back to the server?21:45
rockstarIt's not really defined right now, but we're in the process of doing that.21:45
rockstarlifeless, *kinda*  It's still being defined.21:45
lifelessrockstar: you might like to add anything constraints wise you come up with to dev.launchpad.net/LEP/OopsDisplay21:45
rockstarThe very basics of it is "have javascript write a img with the src="/jsoops?<oops-contents>"21:45
lifelessrockstar: I'm putting together a redo of the entire oops stack to be more agile and reusable across teams21:46
=== matsubara is now known as matsubara-afk
mwhudsonrockstar: does this work of Y.on('error') or something?21:50
rockstarmwhudson, well, we had a global event called one:error that has some extra handling (like providing feedback to the user).21:51
mwhudsonah ok21:52
wgrantlifeless: Did you get the code import worker sorted out?21:52
rockstarWe don't yet have error handling inside YUI just yet, simply because it steps on our current jsoops infrastructure.21:52
lifelesswgrant: the librarian usage question? its for the librarian uploader ha rt ticket21:52
beunomwhudson, the idea is that if *anything* fails, even YUI, it's silently reported to our servers21:54
beunoI picked that up after talking to one of the gmail devs21:54
rockstarbeuno, unfortunately, YUI kills our existing jsoops stuff.21:54
beunosaid that's how they managed to roll out so much crack to so many browsers21:54
beunorockstar, yeah, need to give it some love again21:54
rockstarbeuno, already on it.21:54
rockstar(well, kinda)21:54
* beuno pretends to not have read that21:55
lifelessbeuno: lp will want that real badly21:55
thumperlifeless: can you mentor wgrant's review https://code.launchpad.net/~thumper/launchpad/add-publishing-for-factory-distro-sourcepackage-bug-tasks/+merge/53733 ?21:55
lifelessbeuno: I suspect my oops stuff will be highly relevant21:55
* thumper waves at beuno and rockstar21:55
* rockstar waves at thumper21:55
beunohi thumper!21:56
beunolifeless, yeah, I only spent a few days on it like 5 months ago, it needs some love and documentation to be able to be used company-wide21:56
lifelessbeuno: json format ?21:57
beunolifeless, well, it sends it back as a url, so we don't need js to do any mangling (it did fail, so it can't be depended on)21:57
beunoonce we finish making it play nice with yui, we need something in the backend to parse it21:58
lifelessbeuno: ok21:58
leonardrthumper: https://code.launchpad.net/~leonardr/lazr.restful/forbid-reference-to-entry-not-published-in-this-version/+merge/5391822:01
leonardrnot 100% happy with the implementation, but see what you think22:01
thumperleonardr: ok...22:02
thumperleonardr: with the VersionedObject addition, will this mean any locations in LP will need to be fixed?22:03
thumperleonardr: or is that wrapped in the entry code?22:03
sinzuihuwshimi: http://people.canonical.com/~curtis/out-4.ogv demonstrate call-to-action links create by an :after pseudo class22:15
huwshimisinzui: Very nice!22:16
huwshimisinzui: How do you make the arrows?22:16
huwshimisinzui: Are they an ascii character?22:16
sinzuihuwshimi: http://pastebin.ubuntu.com/581829/22:19
huwshimisinzui: Nice work22:20
sinzuihuwshimi: I wanted to use something more semantic and implicit <em><a>test..., but the pseudo class has to be on the parent element to hover correctly. So I used an explicit class22:22
thumperis LP on python 2.6 or python 2.7 ?22:28
wgrantLucid doesn't have 2.7.22:29
huwshimisinzui: I can hear you22:29
thumperhttps://code.launchpad.net/~thumper/launchpad/blueprint-linked-bug-tasks/+merge/53734 needs a mentor review :)22:36
LPCIBotProject db-devel build #463: FAILURE in 4 hr 56 min: https://hudson.wedontsleep.org/job/db-devel/463/22:41
lifelesswgrant: did you look at the rt I mentioned? - the buildmaster xmlrpc thing22:46
wgrantlifeless: The code import worker thing?22:47
wgrantlifeless: That I asked you about an hour ago?22:47
lifelesswgrant: I referenced two things22:47
lifeless08:00 < lifeless> wgrant: jelmer: can either of you loook at  https://rt.admin.canonical.com/Ticket/Display.html?id=42954 and answer toms debugging question?22:47
lifelessdo the LP code import slaves need access to the librarian ?22:48
thumperyes, I think sop22:48
wgrantlifeless: 42954 is code imports, not buildmaster.22:48
thumperwe store the logs there22:48
wgrantBut yes, I think the import slaves need librarian access.22:48
lifelessoh bah, it is22:48
lifelessand no, not sorted23:01
* thumper has to get out of the house23:13
wgrantlifeless: What is blocking the HA librarian? Complete identification of all the required firewall holes?23:36
lifelessthere is an rt for it, don't have the number offhand but you should be able to see it23:43

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