/srv/irclogs.ubuntu.com/2012/04/25/#launchpad-dev.txt

wgrantStevenK: Erm00:06
wgrantStevenK: Why does https://code.launchpad.net/~launchpad/+recipe/launchpad-convoy use nest-part rather than merge?00:06
nigelbAh. It's a holiday today?03:43
wgrantnigelb: Yeah, ANZAC Day for AU/NZ.03:54
StevenKwgrant: :-(03:54
nigelbwgrant: Ah. I get worried when this channel has very little activity ;-)03:59
Gnigelb: haha, especially since it's the middle of the week :)04:43
nigelbG: Exactly04:44
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
adeuringgood morning07:32
czajkowskialoha08:02
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
gary_posterHey StevenK.  The convoy packaging problem is an issue for parallel testing because our automated system to initialize ec2 instances for tests breaks.  It looks like you worked on it but didn't get it working, seemingly because of a trivial version-already-exists-in-ppa issue.  Should I just try making a trivial change to convoy and firing off the recipe to see if that helps?11:55
wgrantgary_poster: Perhaps we should revive the old known-good version and try fixing it in a staging PPA or something?11:57
gary_posterwgrant, yeah, that sounds reasonable.11:58
wgrantgary_poster: Do you know how to do that?11:59
gary_posterwgrant, no, but it didn't sound too daunting & I was going to give it a try.  hints much appeciated.11:59
gary_posterr11:59
wgrantgary_poster: Delete the current version from the PPA, then copy the last known-good versions back from the grave (search for convoy with "Any status" on +copy-packages, select the last good version for each series, copy with binaries)12:01
gary_posterwgrant ack.  Then I have good versions in the PPA.  Then we can temporarily change the current recipe to build into a different PPA while we work out the kinks?12:02
wgrantLooks like 0.2.1-0~21+r21 is the version to go for, I think.12:02
wgrantgary_poster: Right, you can manually ask the recipe to build into a different PPA12:03
wgrantSo we don't break everyone.12:03
rick_his there any way to get the old recipe? I broke things by adding a Makefile to help run tests and upload to pypi, I can change that back if I knew what the old working recipe was12:04
rick_hthat's what I was trying to figure out yesterday12:04
wgrantThe recipe probably hasn't changed. You might have changed the packaging, which is fully versioned in the bzr branch.12:05
wgrantIt's likely that the makefile broke it12:05
wgrantSimply by existing12:05
rick_hyea, I know that broke it, but then he went in and 'fixed' it and I couldn't tell what his 'fix' was12:05
wgrantSince the debian/rules just uses dh $@, which tries to autodetect the build system.12:05
rick_hright12:05
=== mrevell_ is now known as mrevell
gary_posterwgrant, rick_h, I believe I have done the right temporary fix in the PPA.  Verification appreciated.12:07
wgrantgary_poster: Looks good12:07
wgrantHeh12:07
gary_postercool thank you12:07
wgrantYeah, seems OK12:07
gary_poster:-)12:07
rick_hwgrant: so where would this branch be Stevek had going with the debian dir? https://code.launchpad.net/~launchpad/+recipe/launchpad-convoy12:07
wgrantShould be published in a couple of minutes.12:07
gary_postercool12:07
rick_hok, cool12:08
rick_hsorry gary_poster, forgot the holiday12:08
wgrantrick_h: The second branch referenced in the recipe12:08
wgranthttps://code.launchpad.net/~launchpad/convoy/trunk12:08
rick_hah, gotcha. Sorry, read that as convoy/trunk12:08
wgrant(neither StevenK nor I are here today, but we can look tomorrow)12:08
* rick_h drinks more coffee12:08
wgrantYeah, that's a bit confusing.12:08
wgrantShould probably s/trunk/packaging/12:09
rick_hgary_poster: so if I can bug you a sec, where did you make a change that's now building?12:12
gary_posterrick_h, I went to https://launchpad.net/~launchpad/+archive/ppa12:13
gary_posterclicked on the delete link on top right12:13
gary_posterdeleted convoy12:13
gary_posterwent back to https://launchpad.net/~launchpad/+archive/ppa12:13
rick_hah ok, I thought you were making a 'trivial' change to it somewhere and didn't see an edit12:13
gary_posterclicked on copy link12:13
rick_hgotcha, thanks12:14
gary_postersearched for convoy with Any status and copied the one that seemed like it might not be brokacool np12:14
StevenKI did question rick_h adding a Makefile, I felt it was pointless, but I utterly missed that dh would try it first. :-/12:18
wgrantA Makefile is handy here12:22
rick_hI've gotten too used to having make test and make upload, make sdist for all my setup.py etc needs12:24
wgrantYup12:24
rick_hdidn't realize I was going to bork things, I need to find some time to get back into playing with packaging bits12:24
wgrantLots of our stuff has Makefiles for that purpose.12:24
rick_hyea, since LP I've gone through the Make oreilly book and gone nuts12:25
rick_hhttps://github.com/mitechie/Bookie/blob/develop/Makefile bwuhahahaha12:25
StevenKrick_h: I tend to put .PHONY at the end with everything12:26
StevenKSince make parses it in one-shot12:26
rick_hStevenK: yea, I started going that, but trying to remember/find them in teh list got to be a pain12:26
rick_hso I've gone the python "explicit" route12:26
=== matsubara-afk is now known as matsubara
deryckMorning, all.13:00
dobeymorning deryck13:14
dobeydid some part of the LP DBs get rolled back at all last night or earlier this morning?13:15
wgrantNo.13:15
wgrantWhat's the issue?13:16
dobeyhttps://code.launchpad.net/~ralsina/ubuntuone-control-panel/unique_in_ubuntu/+merge/10333713:16
dobeythis branch had 2 reviews last night, and even a couple of comments from tarmac failing to land it13:16
dobeybut now it has nothing13:16
dobeyoh weird13:17
dobeynevermind13:17
dobeyi see the problem13:17
wgrantIt was probably deleted and recreated or something.13:17
wgrantWe don't just randomly revert bits of the database :)13:18
wgrantHmm13:18
wgrantI indeed have emails from 11 hours ago about that MP13:18
dobeyno, it looks like he made 2 proposals at the same time13:18
wgrantBut it has a different ID13:19
wgrantSo yeah13:19
dobeysorry :)13:19
wgrant103336 is the one tarmac attacked13:19
cjwatsonSome people get very confused by the MP interface and think they need to re-propose any time they make a change13:19
wgrantIn this case it looks like a swift double-click of the Propose Merge button13:20
wgrantThe two were created seconds apart.13:20
dobeycjwatson: nah, this wasn't that case. more likely twitchy finger. i've made it very clear to our team to not do re-submit all the time as well :)13:21
=== abentley changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: abentley | Firefighting: - | Critical bugs: 3.47*10^2
abentleyadeuring: I'm the OCR.  Could you please review https://code.launchpad.net/~abentley/launchpad/celery-everywhere-7/+merge/103483 ?14:20
adeuringabentley: sure14:21
adeuringabentley: r=me14:39
abentleyadeuring: Thanks14:39
=== almaisan-away is now known as al-maisan
gmbderyck, I've bumped https://bugs.launchpad.net/launchpad/+bug/924378 up to Critical; it's now caused two build failures on db_lp in as many days (though not on consecutive runs, AFAICT).15:03
_mup_Bug #924378: buildbot spurious failure UncleanReactorError for TestPullerMasterIntegration <Launchpad itself:Triaged> < https://launchpad.net/bugs/924378 >15:04
gmbJust wanted to give you the heads up.15:04
gmbderyck, If you don't have resources enough to tackle it, gary_poster has suggested that ~yellow take it on.15:04
gary_posterBecause we only have 8 other bugs in the queue now :-P15:04
deryckgmb, thanks. We can add it to our board, but it will likely be a bit before we can get to it.15:04
gary_posterthanks deryck.  We'll coordinate via the bug if we tackle it15:05
deryckgmb, but we could disable the test and wait on one of our squads to get bandwidth to fix it properly.15:05
deryckgary_poster, ok, cool.15:05
gary_posterderyck, +1 on disabling fwiw :-) although the bug makes reference to some reason why that's a bad idea15:05
deryckgary_poster, I think that was speculation from flacoste back when I filed it.  if no one recalls, we could disable and see the fallout to be sure ourselves. :)15:07
gary_poster:-) fair enough15:07
mrevellApologies jcsackett, you weren't on the calendar entry. I've invited you now.15:19
mrevellsinzui, jcsackett, danhg, flacoste, matsubara, czajkowski: https://plus.google.com/hangouts/_/extras/talk.google.com/better-privacy15:30
jcsackettmrevell: having a bit of fun trying to get to the hangout. be there in a moment.15:33
mrevelljcsackett, no worries15:34
=== deryck is now known as deryck[lunch]
sinzuirick_h, do you know what I can do to convoy so that I can run Lp locally?16:52
rick_hsinzui: so gary_poster updated the ppa I thought with an older version that should work16:54
rick_hso update the packages?16:54
sinzuialready done16:54
sinzuiImportError: No module named convoy.meta16:55
gary_posterrick_h, sinzui that will only work if you have not updated, or on a fresh install, IIUC16:55
sinzui^ still present16:55
gary_posterBecause all I did was remove the newer package and reinstate the old16:55
rick_hcan you remove/reinstall?16:55
sinzuigary_poster, you mean visit lp and locate an older package, then lock the version?16:55
gary_posterAlternatively, sinzui, you could do (not exact spelling) sudo easy-install convoy and that would probably fix things up at the expense of a mess in your system python16:56
gary_postersinzui, no I mean in the PPA, I deleted the new package16:56
gary_posterthen copied the old one back into the PPA16:56
sinzuieasy install is crack in that case16:57
gary_posterIf I understand you, I agree that it would be nasty, but I think it would work16:57
sinzuiI see precise is 4 version behind16:57
gary_posteryes16:57
rick_hI think r20 will still work for you16:59
rick_hthe path stuff is only needed on production and I don't think we use it on dev, but not 100% sure16:59
=== al-maisan is now known as almaisan-away
=== matsubara is now known as matsubara-lunch
gary_posterjml, just a heads up in case you are around: the test failure I emailed you about is also affecting our production usage, and therefore blocking our progress.  We will be working on it.  Any insights from you will of course be welcome.  benji, could you file a bug for this so we have a coordination point?17:34
benjigary_poster: sure17:35
gary_posterthanks benji.  benji, I doubt it affects us, but there are two subunit patches in new, recent bugs for subunit that I noticed.17:35
gary_posterbug 98749017:35
_mup_Bug #987490: Test failures with Python 2.6 <subunit:New> < https://launchpad.net/bugs/987490 >17:35
gary_posterbug 98793817:35
_mup_Bug #987938: subunit trunk packaging breaks subunit-* commands <subunit:New> < https://launchpad.net/bugs/987938 >17:35
gary_posterno not the last one17:35
gary_posterbug 98751417:36
_mup_Bug #987514: Regressions in support for Python 3 <subunit:New> < https://launchpad.net/bugs/987514 >17:36
gary_posterthat one17:36
jcsackettis there a good/easy way to get +/- LoC for a branch out of bzr? like some magical incantation to pass to diff, maybe?17:38
jelmerjcsackett: you can feed the diff through diffstat17:39
jelmer"bzr send -o - | diffstat"17:39
jcsackettah, awesome.17:39
jcsackettthanks, jelmer.17:39
jelmerwe should add a lp-dev-tools script that does that :)17:42
=== deryck[lunch] is now known as deryck
=== matsubara-lunch is now known as matsubara
abentleyderyck: Could you please review https://code.launchpad.net/~abentley/launchpad/celery-everywhere-8/+merge/103545 ?19:09
deryckabentley, sure.19:09
abentleyderyck: thanks!19:09
deryckabentley, np!19:09
deryckabentley, looks good to me. r=me.19:13
abentleyderyck: Thanks.19:13
deryckabentley, np.  I've done enough of these now.  It's easy to spot the pattern. :)19:14
lifelessjcsackett: bug 988510 confuses me20:19
_mup_Bug #988510: Bug supervisor should not be subscribed on ubuntu bugs when transition from security to userdata <disclosure> <regression> <Launchpad itself:In Progress by jcsackett> < https://launchpad.net/bugs/988510 >20:19
lifelessjcsackett: I thought we'd checked that the vanilla policies around information types defined would work for ubuntu ?20:19
jcsackettthe policies work, it's a matter of handling auto subscriptions.20:20
lifelessauto or implicit?20:20
lifelessis this perhaps a mixed-mode issue until we stop doing auto subscriptions entirely?20:21
flacostesinzui: you did send a sketch on how to implement agreements in Launchpad on launchpad-dev a while back right?20:41
flacoste"Proposed team agreement" feature20:42
sinzuiflacoste, I did20:48
flacostegary_poster: who is chairing this week, you didn't update the wiki page after last week meeting20:51
gary_postersorry flacoste, lemme find the email20:51
gary_posterflacoste, sinzui20:51
sinzuiflacoste, That is the email thread I started. I cannot determine the real dates of when it happened because Shitehawk^hThunderbird does not honour my date formats. I cannot tell months and days apart20:52
flacostesinzui: i found it in the archive, thx, it was 2011-11-0820:52
sinzuiI see a sensible iso approved big-endian date just like I configured my desktop to use.20:53
gary_postersinzui, you are still ok with hosting team lead call?20:58
gary_posterlifeless, you are unavailable so we do not need to try and invite you right?20:58
bigjoolsmorning20:59
sinzuigary_poster, I am, I wish my wife would stop blocking me from doing it20:59
gary_posterok20:59
=== abentley changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
=== matsubara is now known as matsubara-afk
timrcIt feels like its taking excessively long to do ppa2ppa binary copies21:39
timrcstill pending, wtf :(21:49
=== salgado is now known as salgado-afk

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