/srv/irclogs.ubuntu.com/2010/01/18/#launchpad-dev.txt

wgrantal-maisan: Why do you need SPRBJ.gettitle now?00:42
wgrantTher's an implementation in the integration branch, IIRC.00:43
al-maisanbecause the db-devel branch as it stands fails a test00:43
wgrantHuh.00:43
al-maisanJust doing a testfix00:43
wgrant(wireless laaaaaaaaaag)00:44
al-maisanthe failing test is test_makeJob00:44
al-maisanSourcePackageRecipeBuildJob does not implement getTitle()00:44
wgrantAhh.00:44
wgrantWe need to get the other couple of unreviewed integration pieces landed.00:45
wgrantBut Internet here sort of sucks.00:46
* al-maisan could not agree more00:47
jmljelmer, al-maisan: my conference bag is in the hotel lobby01:28
jmlal-maisan, jelmer: are either of you there?01:28
thumperoops01:29
thumperI'm not there01:29
jml...01:29
jmlthumper, hmm.01:29
jmlmaybe I can go there & return before mwh gets to the talk.01:29
thumperjml: do you remember monty's nick?01:30
jmlproblem solved01:46
jtval-maisan, that branch I promised to refer you to: lp:~jtv/launchpad/bug-499405-translationtemplates-buildmanager04:17
al-maisanjtv: thanks, will take a look.04:18
=== abentley1 is now known as abentley
=== abentley1 is now known as abentley
mrevellhi09:26
thumperhey09:52
beunohello09:53
=== abentley1 is now known as abentley
=== matsubara_ is now known as matsubara
=== Peng___ is now known as Peng_
=== mrevell is now known as mrevell-lunch
vilahey guys, is loggerhead down or is it just me ?13:09
vilasee http://bazaar.launchpad.net/~spiv/bzr/per-file-merge-hook-491711/revision/490513:09
beunovila, seems its down, a user just reported ut13:09
beunoin #launchpad13:10
vilarhaa, one more channel to track :-}13:10
vilabeuno: thks, will wait13:12
=== gary_poster_ is now known as gary_poster
=== gary_poster_ is now known as gary_poster
=== salgado is now known as salgado-lunch
deryckMorning, all.14:49
magciusWoah, found this: http://bitbucket.org/RonnyPfannschmidt/anyvc/14:54
magciusIt's another Leaky Abstraction®, but whatever.14:55
=== matsubara is now known as matsubara-lunch
kfogelgmb or anyone: 'make build' failing in a recently-updated branch of db-devel for us -- see http://paste.ubuntu.com/358557/.  Any ideas?  "ImportError: No module named builder.recipe"15:52
=== salgado-lunch is now known as salgado
kfogelmaxb: random build issue, you might have seen before?  'make build' failing in a recently-updated branch of db-devel for us -- see http://paste.ubuntu.com/358557/.  Any ideas?  "ImportError: No module named builder.recipe"16:04
* kfogel some how associates "maxb" with "understands all sorts of config & build problems"16:04
=== matsubara-lunch is now known as matsubara
gmbintellectronica, bdmurray: Here's the top 100 hot bugs that the abortive staging run produced: https://pastebin.canonical.com/26558/16:21
maxbkfogel: hah :-)   /me is grinning.  Not seen that one though16:23
kfogelmaxb: we never did figure it out -- we just routed around :-).  Sigh.16:23
maxbmy vague thought is: relative import which shouldn't be?16:23
=== henninge_ is now known as henninge
kfogeljml: https://code.edge.launchpad.net/~kfogel/launchpad/509194-506514-bug-heat-db-fixes/+merge/17597   easy review, if you feel like, you know, a break from whatever you're doing :-)17:02
=== henninge is now known as henninge-afk
intellectronicalooks like there's a new dependency in db-devel, on the bzr builder plugin. because of that you can no longer build a fresh branch from db-devel. anyone knows anything about this?17:24
intellectronicathis is probably related to the recent build-from-branch work, b.t.w17:27
bdmurraymake schema is failing for me due to sourcecode/mailman should that be there?17:29
bdmurrayI think I found it17:31
allenapgmb: https://code.edge.launchpad.net/~allenap/launchpad/stop-checkwatches-hammering-products-bug-506158/+merge/1760117:45
allenapgmb: https://code.edge.launchpad.net/~allenap/launchpad/stop-checkwatches-hammering-products-bug-506158/+merge/1760217:52
thumperintellectronica: update your sourcecode dir and download cache18:55
=== dpm is now known as dpm-afk
* thumper waves at the devs19:00
bigjoolshello thumper19:19
thumperbigjools: hey19:25
thumperbigjools: it will be 22 degrees C here today and sunny :)19:25
bigjoolsbastards19:25
thumpernow the conf has tarted19:25
thumperstarted that is19:26
bigjoolsthumper: I am not sure how you keep doing 30+ hour flights and remain sane19:26
thumperbigjools: you assume I'm still sane19:27
=== matsubara is now known as matsubara-afk
bigjoolsit's all relative19:27
thumper:)19:27
goundyHi guys19:41
goundySomething new about wiki support in LP guys ?19:41
jamaltaHi, what's the proper way to reload the server so browser code gets re-executed?19:46
jamaltaIs closing and re-running make run the only way?19:46
thumperjamalta: I think so19:55
thumperit is what I do19:55
jamaltathumper: alright thanks.. annoying but at least it works19:56
thumperyeah19:56
thumpercould be worse19:56
jamaltaheh yeah19:56
jamaltathanks for the input19:56
wgrantbigjools: Hm, publisher is broken? Did we delete something that we shouldn't have?20:11
bigjoolswgrant: tell you in a sec20:11
bigjoolsfixing it20:11
=== salgado is now known as salgado-afk
bigjoolswgrant: someone managed to copy from a private to a non-private PPA20:13
bigjoolsand p-a thought it was a delayed copy20:14
wgrantbigjools: Ah, good. So it's not something terrible.20:14
bigjoolsI'm guessing the API was used since he couldn't see it in the ui20:15
wgrantSo the PPA was actually still private?20:15
bigjoolsin what terms?20:17
bigjoolsthe target ppa was not private and it had a packageupload with no signing key20:18
jamaltais there a list of user logins for dev?20:22
jamaltacan't find it in the wiki20:22
wgrantHm, private source builds are sadly broken in trunk.20:31
wgrantI wonder why there are no tests for that.20:31
allenaprockstar: Hello, are you versed in the ways of the IJob?20:38
rockstarallenap, I believe I can be classified as "brown belt"20:38
allenaprockstar: I wondered why the job.reason column is unused?20:38
allenaprockstar: gmb and I are wondering if we can bend it to our will.20:39
rockstarallenap, so I don't know the reason (no pun intended)20:40
wgrantjml: Why was _sendFileToSlave moved from Builder to BuilderSlave?20:40
allenaprockstar: :) Do you think abentley will know? I'd like to check before I either use it or create a new table.20:41
bigjoolswgrant: I feared that a lot of stuff would be broken after last week20:41
bigjoolswhat's bust?20:42
rockstarallenap, abentley is the architect of the job stuff, and I remember him telling me the purpose, but I don't remember the specifics.20:43
allenaprockstar: Okay, I'll email him about it. Thank you for your help :)20:43
mwhudsonallenap: is there anything in commets.sql about it?20:43
allenapmwhudson: Tip top idea. "The reason the job was created, if applicable" (thanks to gmb).20:44
maxb<bigjools> the target ppa was not private and it had a packageupload with no signing key20:53
maxbIsn't that what happens if you copy a synced-from-Debian upload from the ubuntu primary archive to a public PPA? And that works fine.20:53
bigjoolsmaxb: well in this case the problem is that in combination with the private archive it came from, it triggered the bug20:54
maxbok - just saying that public->public with no signing key works just fine20:54
bigjoolsok thanks20:55
jmlwgrant, because AIUI, BuilderSlave is the LP-side representation of the slave20:56
jamaltaWell, since I haven't received a reply I'm assuming it doesn't exist..20:56
jmlwgrant, sendFile seems like a meaningful operation on that object20:57
jamaltamind if I make a page on dev.launchpad.net that lists login for dev accounts?20:57
jmljamalta, sounds like a good idea to me20:57
wgrantjml: Somwhat, but it's marked private, accessed from other places, and there is another implementation of that interface.21:01
jmlwgrant, it was already accessed from places other than Builder21:01
wgrantI think there's a README somewhere in the tree that lists dev redentials.21:01
jmlwgrant, multiple implementations of the BuilderSlave interface?21:02
wgrantlib/canonical/launchpad/pagetests/README.txt21:03
wgrantjml: Yes. There's RecordingSlave, which is actually used by builddmanager.21:03
jamaltawgrant: ah ok21:03
jmlwgrant, then it seems to me that the interface needs to be tweaked a bit21:04
jamaltawgrant: is it ok if i post that ont he wiki? i guess that's where i should have looked, but it will make it easier for people not to come here asking like idiots (aka, me ;)21:04
wgrantjamalta: I dont like duplicatng information. Perhaps reference the file alongside the existing provided credentials on the wiki.21:05
wgrantjml: Probably.21:05
wgrantA lot of interfaces need tweaking :(21:06
jamaltawgrant: understood21:06
gmbjml: Could you take a squizz at https://code.edge.launchpad.net/~gmb/launchpad/add-bugheatjob-table/+merge/17620 when you have  a second please? It's a branch to add a BugHeatJob table to the DB.21:06
jamaltashould i still make a table of the current users and reference the file as well?21:06
jmlgmb, the diff is empty21:07
wgrantjamalta: I think just reference the file.21:07
jamaltawgrant: ok21:07
gmbjml: That's because I'm an idiot.21:07
gmbjml: I've re-pushed with the change actually committed... don't know when the diff will update.21:08
jmlgmb, "soon"21:08
jmlgmb, thanks.21:08
gmbKewl21:08
jamaltawgrant: thanks21:08
jmlgmb, got it21:11
jmlgmb, the approach that the branch jobs take is to have one table for BranchJobs and have an enum type & json metadata21:11
jmlgmb, rather than a new table per type of job21:12
jmlgmb, what do you think about renaming the table to BugJob?21:12
gmbjml: wfm, sure.21:15
gmbjml: Thus: https://pastebin.canonical.com/26732/21:17
gmb?21:17
jamaltarunning a test, i got a "2 import policy violations" notice.. but this isn't from anything i changed. Is it safe to ignore this, or could it be something I broke?21:19
jamaltathe test in question is lib/lp/soyuz/stories/ppa/xx-copy-packages.txt21:19
gmbjml: Fixed in the branch and pushed.21:25
maxbhttps://edge.launchpad.net/~launchpad/+archive/ppa/+build/1452706 https://edge.launchpad.net/~launchpad/+archive/ppa/+build/1452707 ... Could a Canonical person hit the retry button on those? Unfortunately my "uploader" access apparently doesn't imply "retry" access :-/21:30
bigjoolsmaxb: done21:31
maxbthanks21:31
bigjoolsfwiw there's a bug open about that21:31
bigjoolsright, bed time, g'night21:32
gmbjml: I've assigned the review to you for posterity, FWIW.21:32
maxbAnother interesting edge case is that uploader doesn't implied "can delete" - though that one's more up for debate21:32
kfogeljml: (we redid this trivial db patch to be from db-devel instead of devel; this supersedes the earlier merge proposal which you were asleep during anyway :-) ):21:35
kfogelhttps://code.edge.launchpad.net/~kfogel/launchpad/509194-506514-bug-heat-db-fixes/+merge/1762521:35
wgrantWe beokr more than I thought :(21:47
wgrant(that was 'broke' with about 20s lag)21:48
EdwinGrubbssalgado-afk: ping22:09
jmlhello? what?22:17
jmlkfogel, gmb: which reviews do I need to look at?22:17
kfogeljml: one sec22:17
kfogelhttps://code.edge.launchpad.net/~kfogel/launchpad/509194-506514-bug-heat-db-fixes/+merge/1762522:17
kfogeljml: there you go22:17
kfogeljml:  there will be another Really Easy one in about five minutes22:17
jmlI've just done my LCA talk22:18
gmbjml: The one you were looking at before: https://code.edge.launchpad.net/~gmb/launchpad/add-bugheatjob-table/+merge/1762022:18
gmbIf you pleases.22:18
jmlkfogel, Gabriella Coleman's talk was great. thanks for the recommendation.22:18
wgrantIt was really good, yeah.22:19
jmlBjornT is doing his talk right now.22:19
kfogeljml: glad you liked it!  I'll tell her she pleased at least one person :-).22:19
jmlBjornT, sorry for reviewing db patches while you are talking -- these guys are sprinting22:19
kfogelwgrant: make that two people22:19
kfogelwgrant: she has such a different perspective from the coders, it's really enlightening22:19
kfogelheh22:19
wgrantYup.22:19
jmlgmb, approved. You need an approval from stub.22:22
jmldie sample date die die22:23
jmldata.22:23
gmbjml: Thankee.22:23
jmlkfogel, "The heat bin in which this bugtask appears" -- that doesn't seem correct any more, given the current design22:24
jmlkfogel, it's more like a score than a bin, right?22:24
jmlhmm. maybe I misunderstand.22:25
kfogeljml: interesting point.  let me ask others here, who know more about the bin terminology22:25
kfogeljml: I think it's still correct -- that is, it seems there's this other thing going on whereby heat scores may be grouped into various bins, and that's what this is about.  That may be obsolete now (gmb says, as allenap looks slightly sick next to him), but in any case their possible obsolescence is not related to this patch.22:28
jmlkfogel, ok.22:28
jmlkfogel, approved.22:29
jmlkfogel, gmb: stub's pretty prompt w/ db reviews, but I'll poke him later today to have a look at these.22:30
gmbjml: Thanks22:30
kfogeljml: thanks.  We'll send the next one to stub (btw, I'm working with adeuring not gmb -- dunno this gmb guy, never heard of him; his name is suspiciously short).22:31
jmlkfogel, there are two db patches22:31
jmlkfogel, he has a middle name, that makes him extra trustworthy in alabama22:31
kfogeljml: we're about to submit the next one, actually22:32
jmlkfogel, although judging by his accent, I suspect he's from north of the Mason-Dixon line.22:32
kfogeljml: gmb said the "Welcome to Alabama" sign on the highway coming in from Atlanta airport (in Georgia, the next state over) had bullet holes in it.  Yee-haw!22:32
kfogeljml: https://code.edge.launchpad.net/~kfogel/launchpad/505845-affects-me-too-from-dups/+merge/1763322:39
jmlkfogel, looking22:39
kfogeljml: thanks22:40
kfogeljml: afk for a bit, will be back to answer questions after22:41
=== mup_ is now known as mup
=== Ursinha__ is now known as Ursinha
=== Ursinha is now known as Guest1321
=== Guest1321 is now known as Ursinha_
=== Ursinha_ is now known as Ursinha
Ursinhaae22:53
jmlei22:53
Ursinhathis rain won't let me down!!22:53
jmldown from where?22:57
jmlkfogel, reviewed. happy to talk further.22:57
kfogeljml: we're reading now22:58
jmlkfogel, \o/22:58
maxbI've just been trying to update-sourcecode, and I've noticed a couple of things:22:59
maxb(1) it breaks if you have bzr-git installed22:59
maxb(2) to make it work on lucid we'll need to have bzr in the launchpad PPA _or_ make it run with python 2.622:59
jmlmaxb, that first one surprises me.23:00
maxbjml: It's because bzr sees the test git repositories within dulwich as branches, and update-sourcecode tries to delete them as obsolete :-)23:01
mwhudsonaaraaarsadhjk23:01
mwhudsonlet's not use find_branches23:01
jmlmaxb, look, you broke mwhudson23:01
jmlmaxb, I hope your sorry.23:02
jmlyou're. dammit.23:02
Ursinhajml: it's raining like it's the end of the world here23:02
maxbYeah, bzr-<other-vcs> is cool, but I wish it could read your mind to figure out when to run :-)23:02
jmlUrsinha, :(23:02
maxbYou can imagine the interesting edge cases I run into with bzr-svn installed, where my homedir is a svn wc.....23:03
maxbSo.... soliciting thoughts on which is the lesser evil? Add bzr to the set of things we have to rebuild to work with the Launchpad blessed Python version, or run devscripts stuff under a possibly different python to launchpad?23:06
elmo*blink* devscripts has python things in it?23:07
mwhudsonmaxb: why does bzr need to be rebuilt?23:07
maxbBecause update-sourcecode imports bzrlib23:07
elmomaxb: I don't see any python in devscripts?23:08
maxbI mean lib/lp/devscripts/23:08
mwhudsonmaxb: i think update-sourcecode being run with the system /usr/bin/python23:08
mwhudsonwould be fine23:08
maxbuh, lib/devscripts/23:08
mwhudsonpossibly not the best choice of package name i guess23:09
maxbOh, this is going to be pulling on a thread which unravels ridiculously, isn't it :-/23:10
jmlif only we had a vcs that supports renaming23:10
wgrantWasn't devscripts stuff running on 2.5 long before LP was ported?23:10
jmlmaxb, everything on LP worth doing is like that. stick with it.23:10
wgrantSo it shouldn't be too hard.23:10
elmojml: are you crazy?  eventually the whole cloth will come apart if you do that!23:11
maxbI'm just worrying about any deps wanted from eggs for devsscripts23:11
wgrant(disclaimer: Launchpad difficulty ratings may differ wildly from generally accepted standards.)23:11
jmlelmo, but think of the long, useful (dare I say light-weight & flexible) yarn23:12
maxbmm.... do nothing in devscripts looks to be using any launchpad custom code, but some of the launcher scripts in utilities do use it after an "import _pythonpath"23:14
kfogeljml: responded in merge proposal.  we're about to enter sprint wrapup, so I may not be able to respond till later23:15
maxbBut update-sourcecode can't do that because it is run from rocketfuel-get, which is run from rocketfuel-setup23:17
jmlhi ho23:23
jmlI'm back23:23
jmlkfogel, I'll take a look.23:23

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