wallyworld_wgrant: why did you remove the check for the bug created response at line 120?00:37
wgrantwallyworld_: That was previously the only success verification00:38
wgrantwallyworld_: Since it used /bugs to obtain the ID.00:39
wgrantBut now I check the body of the response, so the status code check isn't too important.00:39
wallyworld_ok, thanks00:39
wgrantwallyworld_: Thanks00:44
=== jtv1 is now known as jtv
wallyworld_lifeless: the new celery jobs infrastructure - if i were to write a new celery job, would there be any need to provide a db table? i would think not, since the celery/rabbit stuff would provide the necessary guaranteed delivery/execution?03:05
lifelesswallyworld_: that aspect is unchanged to date.03:16
lifelesswallyworld_: we don't run rabbit in HA mode, its only good for things we're willing to drop on the floor in times of failure.03:17
wallyworld_lifeless: ok, so i need to create the db table etc03:17
wallyworld_just wanted to check, thanks. i figure new stuff should be done using celery03:18
lifelessit should03:18
lifelesswhats the bug # for 'apport bugs generate multiple notifications'03:30
nigelbbah. that one's fixed.03:32
lifelesspossible 66760403:47
lifelesshah 274537 should be dead easy03:51
lifelesswgrant: I have a suspicion you fixed 626542 ?03:52
* wallyworld_ off to see doctor. bbiab04:03
wgrantlifeless: Not sure. All of that stuff sort of blurs together.04:10
lifelesswgrant: any idea on the bug # I'm looking for ?04:15
lifelessbefore I wont-fix all of LP ?04:15
StevenKlifeless: That sounds a little drastic?04:19
StevenK"BAH, I can't find the dinner plate I want, so I'm going to burn the house down."04:20
wgrantlifeless: 667604 is the only one I know about04:20
wgrantBut that's about new bugs.04:21
wgrantBug 42484904:22
_mup_Bug #424849: Launchpad should batch attachment notification emails <lp-bugs> <story-better-bug-notification> <story-better-notification-sending> <Launchpad itself:Fix Released by gmb> <apport (Ubuntu):Invalid> < https://launchpad.net/bugs/424849 >04:22
wgrantThe end of that bug reveals all04:24
wgrantBug #76899704:24
_mup_Bug #768997: Batch multiple comments from the same person/same bug <feature> <Launchpad itself:Triaged> < https://launchpad.net/bugs/768997 >04:24
wgrantlifeless: It's a dupe of that04:24
lifelesswgrant: good call, I went past that and ignored it.04:27
lifelessnow, where is the damn one about mailing list subscriptions -> user terrible confusion04:28
lifeless(that https://bugs.launchpad.net/launchpad/+bug/816373 is a dupe of)04:28
_mup_Bug #816373: Launchpad sends duplicate bug change notifications <Launchpad itself:Triaged> < https://launchpad.net/bugs/816373 >04:28
StevenKBah, bug change04:28
lifelesswgrant: fwiw I'm seeing nearly 3 second bug searches on LP04:31
lifeless423ms 04:32
lifelessSQL-main-slave: SELECT COUNT(*)04:32
lifeless681ms 04:32
lifelessSQL-main-slave: SELECT BugTaskFlat.bugtask04:32
lifelessso, most of it elsewhere04:32
wgrantYeah, they're rarely below 2s now.04:35
wgrantPartly because of slow COUNT(*)s, but it's mostly out of the DB Now04:35
wgrantYay, only major bugtaskflat timeouts were already a problem. Tag union searches, +subscribedbugs, +commentedbugs. But some obscure sort orders have regressed... I think most of them would be OK if we didn't have stupid COUNT(*).04:39
wgrantBut it seems I can sensibly denorm a few more cols onto BugTaskFlat to sort out sorts.04:39
StevenKAssertionError: Name "+secrecy" is not registered as a view or navigation step for "Bug" on "bugs".04:52
StevenKOh, it's on the bugtask04:54
wgrantI hate partner05:16
StevenKEveryone does.05:18
wgrantNot as much as me.05:18
nigelbYou hate having a partner?05:19
nigelb(j/k I know what he means)05:19
StevenKnigelb: I love having a partner. I can't speak for wgrant. :-P05:20
nigelbStevenK: Haha :)05:20
wgrantMaybe nobody cares about searching for partner bugs :)05:20
nigelbI'm very tempted to start launchpadmemes.tumblr.com in the footsteps of mozillamemes and webkitmemes05:22
nigelbThough, I suspect all it will have are rage faces ;-)05:22
wgrantI think it could be pretty well summed up by the original rageface :)05:23
nigelbFor April fools, lp should at some point show rage face icon for ~launchpad-dev05:24
spmyou could have a yellow rubber duck, and a cowboy hat05:24
wgrantwebops have the monopoly on amusing icons05:24
hloeungcanonical-sysadmin has a pigeon icon05:24
nigelbso we bribe spm? Or is he LOSA and not webops05:24
spmnigelb: s/losa/webops/ basically05:25
spmjust a rename a while ago05:25
wgrantBack in my day, canonical-sysadmins had the default icon!05:25
spmlosa was getting a trifle overloaded.05:25
wgrantBut indeed, it is a pigeon now. Odd05:25
wgrantLaunchpad/Landscape/ISDU1/CA/EA OSA? :)05:25
spmlaunchpad, landscape, ubuntu one, consumer apps, isd etc operational systems admin05:25
nigelbadmin ALL THE THINGS05:25
spmwgrant: right05:26
hloeungpes, engineering05:26
hloeungpretty much A-Z OSAs05:26
spmoh god. I forgot about them05:26
wgrantOh right, PES now too05:26
wgrantMust need to take on some more of CS soon05:26
nigelbSee, there's enough material in here to make a meme site.05:26
nigelbHah, the pigeon reminds of @feral_pigeon05:27
spmwebops is better than losa too. WeBops and use a Cyndi Lauper song as our theme; or Wh00psies! etc05:27
nigelb"We didn't start the fire" is a better theme song for ops teams ;-)05:28
spmnigelb: expect when we did05:28
wgrantA theme song doesn't have to be truthful :)05:28
spma good mate has a long running joke: he fixes everything with fire. as one of the UQ Datacentres had a fire in it - on his watch (he was networks dude at the time).05:29
spmI'd prefer "Burning Down The House" myself, anyway.05:29
bigjoolsspm: We Are The Cham^WWebops?05:47
spmbigjools: more like We're Too Sexy for LP Developers, too sexy by faaaarrr05:48
bigjoolsspm: now you're doing Mission Impossible05:49
nigelbhaha, well played bigjools.05:53
* nigelb slow claps05:53
StevenKwallyworld_: Are you back?06:01
lifelessspm: we're too sexy for bops ?06:12
spmI think my brain just abort retry ignored?06:14
wallyworld_StevenK: yes06:20
wallyworld_ah a review06:21
wallyworld_StevenK: TestBugTaskInterestingActivity - that test looks a little out of place. where are the other similar tests for the bug activity stuff on _index? a doc test?06:29
StevenKwallyworld_: Yes, a doctest. And I didn't really want to add to it.06:35
wallyworld_StevenK: is it worth porting the doc test to a unit test? as it stands now, there's a bit here and a bit there if you know what i mean06:36
wallyworld_the BugVisibilityChange class - that is only relevant if the show info type in ui feature flag is off right, and when turn on the ff for all, we delete that class?06:37
=== jtv1 is now known as jtv
StevenKwallyworld_: BugVisibilityChange and BugSecurityChange and related gubbins will all die when we drop the UI feature flag and switch to information_type fully.06:41
wallyworld_cool, just wanted to check - should we put in a XXX so we can keep track?06:41
StevenKI can mark the two classes as such, if you wish.06:41
wallyworld_with bug-change.txt - i'm all for ripping it out so long as there's coverage elsewhere06:41
StevenKwallyworld_: I'd love your opinion on that -- compare bug-change.txt to test_bugchange.py06:42
wallyworld_ok, let me look06:42
wallyworld_StevenK: perhpas if we raise a bug for the work to rip out the legacy stuff when the ff is on that would be good and we could put the relavant XXXs in as we find them06:42
wallyworld_StevenK: ffs, there's also test_bugchanges - note the extra 's'06:43
StevenKwallyworld_: To be fair then, I'd be putting in XXXs all over the place.06:44
wallyworld_ok, lets' leave it then. so long as we make sure we eventually deleting everything required06:45
StevenKwallyworld_: I like the idea of XXXs for BugVisibilityChange and BugSecurityChange -- pretty much everything else will fall out due to tests failing.06:45
wallyworld_right. my fear was that those classes would hang around, not be used, and we'd forget them06:45
StevenKwallyworld_: test_buchange.py doesn't exist, it's only test_bugchanges, sorry06:46
wallyworld_StevenK: lib/lp/bugs/adaptors/test/test_bugchange06:47
wallyworld_StevenK: so, about the fact that this branch introduces a new unit test but everything else is in a doc test (bug activity stuff). my personal preference is to be consistent otherwise it's too hard to keep track of what's where. so add the new test as a doc test or port the doc test to unit tests. do you share that view?06:50
wallyworld_StevenK: with bug-change.txt vs test_bugchanges.py: they are similar for sure but bug-change.txt mainly exercises the addChange() API which is a lower level one and is invoked indirectly in test_bugchanges when those tests are run. so if addChange() is broken, so too will be test_bugchanges bug there is an argument that all the current tests are required since strictly speaking they do test sifferent aspects07:03
StevenKwallyworld_: Right.07:04
wgrantwallyworld_: If you have time once you're done with StevenK, could you please look over https://code.launchpad.net/~wgrant/launchpad/bug-899776/+merge/103806?07:04
StevenKwallyworld_: I can move that one unit test into xx-bug-activity.txt if you wish07:04
wallyworld_StevenK: it would be great if you could because then everything is together07:04
wallyworld_StevenK: and the test_bugchange, test_bug_changes, bug-change.txt can stay as is for nowe07:05
StevenKwallyworld_: Aye07:07
wallyworld_StevenK: r=me but with an additional comment about the bug-change.txt deletions07:13
StevenKwallyworld_: I just remember the other reason I didn't want to use xx-bug-activity.txt07:13
StevenKRan 1 tests with 1 failures and 0 errors in 24.564 seconds.07:13
wallyworld_it failed as is ot with your additions?07:13
StevenKwallyworld_: No, look at the time!07:14
wallyworld_yeah, slow07:14
wallyworld_so are manu unit tests due to setup of db07:14
StevenKwallyworld_: I'll revert the drops in bug-change.txt with a little sadness.07:14
wallyworld_StevenK: i share the sadness but we would be removing test coverage07:14
wallyworld_you will need to wrap with a ff check as well07:15
StevenKI will? It won't be set, they'll be fine unchanged.07:15
wallyworld_StevenK: ah yes sorry. the classes are constructed directly07:17
wgrantwallyworld_: Thanks. There are some other existing tests, but they're mostly at the browser layer because apparently people hate doing things properly.07:27
wallyworld_wgrant: np. my main worry was that we have the test coverage to ensure any regression is caught07:27
wallyworld_seems like we do hopefully07:27
wgrantAnd my new test covers most interesting caes.07:27
adeuringgood morning07:33
=== adeuring changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: adeuring | Firefighting: - | Critical bugs: 3.47*10^2
stubOne more benefit of moving to PG 9.1 is that the enum types no longer suck, and we can use meaningful terms instead of cryptic integers at the DB level with things like our partial indexes.08:04
wgrantstub: Ah yeah, sorry, I just assume everyone knows those two08:10
stubOh, I'm used to that. Just seeing the index go past prompted me to confirm being able to add values to the enumerated type got into 9.1.08:11
stubwgrant: I assume this new index needs to go live?08:36
angelocHi guys, i'm writing some scripts for ubuntu accomplishments, until now i solved all my problems reading online api documentation, but I'm now in stuck08:39
angeloci have to find all packages uploaded by a person, can you suggest me the best way?08:40
=== almaisan-away is now known as al-maisan
lifelessstub: yes it does :)09:00
stublifeless: the index?09:01
=== al-maisan is now known as almaisan-away
* StevenK stabs gpg for being obtuse and terrible.09:04
wgrantstub: Yes please. Don't think backups are done yet, though.09:14
stublifeless, wgrant: My network is just far to bad. webops will need to do it if you don't want to wait for my SEA -> Los Angeles link to stop sucking.09:15
stubGot as far as typing 'scree' and waiting for the 'n' to come though :-/09:16
wgrantstub: It's not urgent; the rev won't be deployed until Monday or so.09:17
stubk. I'll stick it on my list and try later.09:17
wgrantThanks :)09:17
stubExcept my reminder list is 'in the cloud' and accessed via LA too...09:17
* stub grabs a pen09:18
stubLooks like all of level 3 from my POV... did we break level 3?09:20
StevenKstub: Haha, it's possible.09:22
stubIs it sucking from AU too?09:22
StevenKI've not seen any problems, what are you having trouble reaching?09:23
stubStevenK: Starts happening Level 3 in Los Angeles. Probably the Singapore -> LA link or Level 3 in LA.09:44
rick_h_bwuhahaha, https://launchpad.net/+combo/rev15149/?yui/yui/yui-min.js10:26
StevenKrick_h_: Indeed!10:26
StevenKrick_h_: I suggest you and deryck find some time to get the flag enabled on qas and then hit up every bit JS to see what breaks.10:27
StevenKActually, while I think of it.10:27
wgrantWe also probably need to get a reasonable subset loaded early.10:28
wgrantstub: Thanks.10:35
* wgrant lands.10:35
* StevenK declares victory over gpg.10:49
czajkowskiStevenK: every small victory counts!10:53
StevenKczajkowski: I've been fighting with it for a few hours spread over the last few days.11:02
StevenKYou'd think moving a private key to a different keyring is hard or something.11:03
jmlhey guys11:17
jmlso, uhh, I left a couple of Launchpad ec2 instances running somehow11:17
=== bac changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: adeuring, bac | Firefighting: - | Critical bugs: 3.47*10^2
bacmorning adeuring11:25
adeuringmorning bac11:25
bachi beuno11:26
beunohiya bac!11:26
bacadeuring: you're not reviewing ian's MP are you?  if not, i'll grab it now.11:38
bacadeuring: nm, done.11:58
adeuringbac: , right, I did had a look at the mp. (sorry for the late answer: had lunch...)12:04
adeuringerm i did _not_ have a look at the mp12:05
bacadeuring: perfect12:06
=== salgado_ is now known as salgado
=== wgrant_ is now known as wgrant
deryckMorning, folks.13:03
adeuringmorning deryck13:03
angelocStevenK: here I am!13:42
StevenKangeloc: Right, so as I said in #launchpad, the first step is to gather your requirements. What exactly do you want to search by, and what results do you want to get back. Once you have that, we can see.13:44
=== fjlacoste is now known as flacoste
angeloci have to verify that a person has uploaded at least one package to assign him a trophy14:07
angelocStevenK: sorry for delay14:07
StevenKangeloc: To a PPA? To the main Ubuntu archive? As the uploader itself, or is it okay if his upload was sponsored?14:12
angelocStevenK: sorry for not being explicit... as a first package uploaded, I can think of a sponsored one, I think I can create another tropy for people that has packages on their ppas14:14
StevenKangeloc: Right, so I can't recall any method exported over the API that will do that. If you're not afraid of diving in and writing some Python, some people here will probably answer any questions you have. As for me, it's time for bed.14:17
angelocStevenK: thank you so much! Wich package I have to download to dig inside launchpad api implementation?14:18
=== bac changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: adeuring | Firefighting: - | Critical bugs: 3.47*10^2
=== adeuring changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
czajkowskigmb: photo walk plan ?16:35
gmbczajkowski, Will email to you by Monday - bug me if I don't! Trying to find out if some of the suggested routes that I've found are actually a) sane b) interesting c) not liable to get us mugged.16:38
czajkowskigmb: cool ask pleia2  if you like16:42
czajkowskiahh you sent the mail to uds cool16:42
czajkowskiwas working on that saves em time :)16:42
gmbczajkowski, Ah, sorry, didn't realise I was duplicating effort. Oh well, you can relax now. Have a beer, knock off early... :P16:43
=== salgado is now known as salgado-lunch
=== salgado-lunch is now known as salgado
* deryck lunches afk for a bit, to enroll the youngest in kindergarten.17:04
rick_h_ooh, send the kids away!17:04
=== matsubara is now known as matsubara-lunch
=== salgado is now known as salgado-brb
=== salgado-brb is now known as salgado
=== matsubara-lunch is now known as matsubara
abentleybac: Are you OCR?19:11
abentleyrick_h_: There's no OCR.  Up for a review?19:17
rick_h_abentley: aure thing19:19
abentleyrick_h_: Thanks.  https://code.launchpad.net/~abentley/launchpad/remove-create-merge-proposal-job/+merge/10392919:19
abentleyrick_h_: It is long, but 90% deletion.19:19
rick_h_abentley: what's this --\x20 in line 619 of the diff?19:32
rick_h_some sort of unicode foobar there with the mp maybe?19:32
abentleyrick_h_: that's a lint fix.  It's the encoding for a space.  Having an actual space caused a lint complaint.19:32
abentleyrick_h_: Because we don't like having trailing whitespace.19:32
rick_h_abentley: ah, ok that's inside of some message body19:33
rick_h_abentley: thanks19:33
abentleyrick_h_: np.  Should have mentioned that in the description.19:34
rick_h_woot! no more code* my poor chrome auto complete is so confused19:42
=== benji___ is now known as benji
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
SpamapSHrm, I need the opposite of source_package.setBranch .. I need to delete the official branch pointer22:45
cjwatsonSpamapS: .setBranch(None)22:50
SpamapSThat easy?22:50
SpamapSWhy didn't I think of that. :)22:50
cjwatsonwell, branch=None obv.22:50
cjwatsonaccording to the code, yes22:50
cjwatson        else:22:50
cjwatson            # Delete the official DSP if there is no publishing history.22:50
cjwatson            self.distribution_sourcepackage.delete()22:50
cjwatsonmaybe try it on staging first22:51
SpamapSthis is for the 'charms' distribution :)22:55
SpamapSERROR:can't determine Launchpad branch from bzr branch22:58
SpamapSoh wait22:58
SpamapSthats my problem..22:58
SpamapScjwatson: first test did not result in its removal... Hrm23:03
SpamapSis there a way to get launchpadlib to spew out what it is doing? Some kind of logging decorator?23:04
cjwatsonimport httplib2; httplib2.debuglevel = 123:07
cjwatsonbut that may not help if I made a mistake in interpreting what's happening on the server side23:08
SpamapSI *just* fell far enough down the well to find that in pydoc :)23:08
SpamapSyeah I just want to see that it sends None :)23:08
SpamapSYeah, so I get a 200 OK with that, but it doesn't seem to actually be deleting the source package record23:15
cjwatsonOK, maybe I'm wrong then23:19
SpamapScjwatson: I'm looking at the same code.. but I haven't gone deeper to see what .delete() does23:21

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