/srv/irclogs.ubuntu.com/2010/02/23/#launchpad-dev.txt

lifelesswell you might want to be sure that this happens before the probe00:00
lifelessfor instance00:00
lifelessbut I cam think of things that will make you cry00:00
mwhudsonhm yes, that would be a bit backwards00:00
mwhudsonlifeless: i think i've run out of tears in this area, but try me00:00
lifelessfor instance, I can setup a bzr:// server serving a git branch over VFS only00:00
lifelesswhich will cause your local obejct to be a RemoteBzrBranch00:01
mwhudsonah true00:01
lifelessbut the conversion to happen on your machine, not mine00:01
lifelessditto svn00:01
lifelessso this is a little complex and worth thinking about some more.00:02
lifelessI think we should get rid of your new-process need in tests in the first instance.00:02
lifelessand fix the vfs/git after that, to get rid of the short term hack.00:02
lifelessfor the 'deployed services need to be robust in xyz case' stuff we've been discussing, thats a longer term topic, right?00:02
maxbwgrant: No, I believe you. I'm adjusting lp-deps now.00:03
wgrantmaxb: Thanks. Going to drop the slony deps while you're there?00:03
mwhudsonlifeless: yes, sure00:03
lifelessok00:04
lifelesswell I'll volunteer to do the bzr-git change00:04
lifelessifyou make a bug for it00:04
wgrantmaxb: Please also depend on python2.5-egenix-mxtools.00:04
wgrantmaxb: It contains the mx package.00:05
mwhudsonlifeless: of course, using an open hook to limit the urls we access leads to a vaguely similar concern00:05
mwhudsonas uninstalling hooks isn't support either, aiui00:05
lifelessmwhudson: hooks are reset by the test framework00:06
mwhudsonah ok00:06
lifelessyou start in a new test with no hooks installed at all00:06
lifelesscalling self.run_bzr installed the 'find command hooks'00:06
lifelessthats about it00:07
wgrant200:13
wgrantGah.00:13
mwhudsonlifeless: ok, i filed a few bugs, https://bugs.edge.launchpad.net/bzr-git/+bug/526133 is the bzr-git one00:13
mupBug #526133: need to be able to unload bzr-git <Bazaar Git Plugin:New> <https://launchpad.net/bugs/526133>00:13
pooliekfogel: this is also true for the help homepage00:19
maxbwgrant: gah, oops. failed to see that and dputted already00:20
kfogelpoolie: hmrm.  I wonder if there's a policy here, and if so, where it's documented.00:21
wgrantmaxb: Nyeh. It's not really important.00:21
wgrantIt'll just break in reasonably obscure ways if the PPA gets outdated again.00:21
maxbheh00:21
maxbit's annoying that 'import psycopg2' doesn't break00:22
poolieand you can't change that either, karl?00:22
maxboh, yes it does, if you use the right version of python00:23
wgrantmaxb: Yeah.00:23
wgrantmaxb: Have you tried a full test run on Lucid?00:23
wgrantI've one running now, and there are a couple of the usual Soyuz failures.00:24
wgrantBut otherwise it's looking good.00:24
mwhudsonstaging is down :(00:24
mwhudsonand sysadmins are in europe00:26
wgrantmwhudson: It should be back in 15 minutes, no>?00:26
wgrantMaybe 10.00:26
mwhudsoni've not caught it up yet today00:26
mwhudsonadmittedly i haven't tried all that hard00:26
wgrantI suspect it went for an update at 23:44Z00:26
mwhudsonoh perhaps, i guess db-stable got updated around then00:27
maxbwgrant: I tried, and it was all hideous. I gave up and decided to lower my sights to python2.6 on karmic to start with00:28
wgrantI sort of wish it didn't pull.00:28
maxbExcept then things broke more, and I spent some time fixing python2.5 on karmic. Got that branch landed today00:28
wgrantmaxb: That bzr plugins thing?00:28
maxbye00:28
maxbs00:28
wgrantOnly one failure so far, and that's just extra hash type on apt-ftparchive output.00:29
wgrantSo this is looking pretty good.00:29
wgrantMuch better than Karmic was, at least.00:30
wgrantmwhudson: There aren't going to be any more recipe model changes, are there?00:31
mwhudsonwgrant: not completely sure00:31
mwhudsonwgrant: nothing fundamental aiui, apart from adding derived recipes00:31
mwhudsonwgrant: there might be changes to do with multi-distroseries00:32
mwhudsonyay staging is back00:32
wgrantSince the backend in trunk now works, it seems almost feasible to add API exports.00:32
wgrantmwhudson: Early!00:32
maxbwgrant: "so far" :-)00:36
* mwhudson finds an old no-codeimportworker-db-access branch, merges 2000 revs of trunk into it00:36
maxbimpressive00:39
pooliethumper: hi, quick call?00:39
thumperpoolie: yeah, sure00:40
* maxb waits for the publisher :-/00:41
maxbjelmer: Once the publisher gets around to it, the launchpad PPA should be fully installable on Lucid again00:43
wgrantmaxb: I suppose you had to wait a cycle for the Lucid one to publish, then copy to other series, then wait for it to publish again?00:43
maxbno, I'm waiting for my second upload of lp-deps :-/00:44
wgrantAh.00:44
wgrantI really should look at killing process-accepted.py for the normal case.00:44
wgrantSince it's only needed for binaries with custom uploads.00:44
wgrantAnything without custom uploads can be realised immediately on upload, allowing a copy as soon as it's built.00:45
wgrantWe already skip it for sources.00:45
maxbbtw, I have a fix locally for importfascist on py2.6, your testrun will probably run into it at some point00:46
wgrantmaxb: I'm running it on 2.5, not 2.6 yet.00:46
maxboh I see, the inverse to me, I tried 2.6/karmic00:47
wgrantI figure that we will need it running on Lucid in a month or so, so it probably deserves a full test run.00:47
maxbhrm, installing lp-deps has now started me a slony daemon :-/00:48
wgrantmaxb: You saw the talk about that on the list?00:48
maxbyes - and: ah, no, it claimed it did, but it was lying to me00:49
wgrantHeh.00:49
wgrantmwhudson: There is a way to tell ec2test to take a standard Ubuntu AMI, Launchpadify it and run the test suite, isn't there?00:55
mwhudsonwgrant: yess00:55
mwhudsonwgrant: however the code in trunk assumes you can log in as root, not ubuntu00:56
wgrantI wondered if that was the case.00:56
wgrantBut I'm sure I used it on a Karmic daily. Maybe I hacked it.00:56
mwhudsonwgrant: lp:~mwhudson/launchpad/ec-log-in-as-ubuntu should work with a newer image00:57
mwhudsoni should look at getting that merged i guess00:57
wgrantProbably.00:57
wgrantOfficial images are good.00:58
* mwhudson writes it on a list00:59
=== jamalta is now known as jamalta-afk
* thumper afk for a few minutes collecting the girls01:53
thumperanyone else not getting emails from EC2?02:59
thumperit seems that I'm not getting any02:59
wgrantI received a failure of my own a couple of days ago.03:00
wgrantI tried to ec2test something this morning, but the instance hung before it had finished booting.03:00
thumpermwhudson: launchpad-code has no New bugs :)03:00
thumperwgrant: :(03:00
mwhudsonthumper: congrats03:00
wgrant(btw, the successor to the branch you tried to land yesterday landed over night)03:01
mwhudsonthumper: i'm getting them03:01
thumperhmm03:01
* thumper checks spam box03:01
mwhudsonbugger03:04
thumpernope, not there03:05
wgrantthumper: So you're reproducibly not getting emails from it?03:06
thumperwgrant: haven't had an email from ec2 in several days03:06
thumperwgrant: can you send an email to me at tim.penhey@canonical.com?03:07
thumperwgrant: it could be being caught there03:07
* thumper taps fingers03:07
thumperparent teacher interviews in 12 minutes03:07
mwhudsonthumper: the incremental kernel import failed :( https://code.staging.launchpad.net/~kiko/linux/2.6.3103:07
thumpermwhudson: just saw the email03:09
thumpermwhudson: perhaps ordering on the revisions it is choosing to import?03:09
wgrantthumper: Sent.03:09
thumpermwhudson: just a wild arse guess03:09
thumperwgrant: ta03:09
thumperwgrant: the email from mwhudson was addressed to that too and arrived fine03:09
mwhudsonthumper: well, the import list we're slicing into is topo sorted, so that shouldn't be a problem03:10
thumperwgrant: got your email :(03:10
thumperwgrant: so it isn't that03:10
thumperit seems ec2 hates me03:10
* thumper stopping for a few hours,03:10
thumperback later03:10
mwhudsonthumper: i guess run ec2 non-headless with --postmortem and log in and poke around after the tests are done03:11
mwhudsongararar i'd forgotten how annoying working with the internal xmlrpc server was04:28
* mwhudson EODs04:29
* wgrant has never had the pleasure.04:29
mwhudsonthe main reason it sucks is that requests are not authenticated04:30
mwhudsonso either you have to weaken permissions or use removeSecurityProxy all over04:30
mwhudsonoh and faults are a crummy way to report errors04:30
wgrantOh, nice.04:30
wgrantYeah, I know that much from buildd work.04:31
mwhudsoni should look (again) at using the PermissiveSecurityPolicy for xmlrpc server requests i guess...04:31
wgrantThat seems like a really good idea.04:31
mwhudsonoh guess what04:33
mwhudsonthe security policy is global, not per-request04:33
mwhudsoni think04:33
wgrantAh ha ha.04:35
mwhudsonmmmmmmmmm04:36
mwhudsonmaybe not actually04:36
mwhudsonaaaa04:37
* mwhudson stares into the zope vortex04:37
wgrantNooooooooo.04:38
* wgrant throws xx-resetpassword-of-sso-account.txt out the window.04:39
mwhudsonwell, that seems to not be really true04:46
mwhudsonbut still overriding the policy per request does seem quite hard04:46
wgrantIs update-sourcecode broken for anyone else? It can't find bzrlib.branch.06:48
wgrantAh, probably because it's running with 2.5, and it's meant to use the system's bzr.06:49
jelmermwhudson: ping06:53
lifelesswgrant: system bzr should be installed for all python versions06:56
wgrantlifeless: Yes, and Python 2.5 doesn't exist.06:57
wgrantLucid hates it and wants it to die.06:58
wgrantOdd that it's only broken recently, though.06:58
=== henninge_ is now known as henninge
adeuringgood morning08:24
wgrantbigjools: Thanks for getting that landed.09:08
bigjoolswgrant: np09:59
wgrantbigjools: Has somebody poked you about the eaten binary?10:03
bigjoolsumm no10:03
wgrantbigjools: create-resources in lucid was eaten today by the arch-indep reverted override bug.10:04
wgrantIt wants to be revived.10:04
bigjoolswhich bug is that?10:05
wgrantIf you take an arch-indep binary, override it, then override it back within one publishing cycle, it gets eaten.10:06
bigjoolsoh that10:06
wgrantIt's very difficult to fix, so it hasn't been yet.10:06
wgrant(I think somebody should just fix change-override.py to die if it's going to get into that situation, really)10:06
bigjoolswe could record overrides as an intermediate step and refuse further overrides10:07
wgrantWe could either reject further conflicting overrides, or just mutate the existing Pending publishing record.10:07
wgrantAlthough I don't trust the publisher to be sufficiently respectful that that would be safe.10:08
bigjoolsit needs time for analysis and given the frequency of occurrence it's not exactly a priority for us, unfortunately10:09
wgrantIndeed.10:10
bigjoolsI am deep in a refactor for process-accepted10:10
bigjoolsconverting it to a LaunchpadScript sounded a good idea at the time10:10
wgrantAh.10:10
wgrantI didn't think it was otherwise much work.10:10
wgrantAre you going with per-copy-archive selection, or just all copy archives?10:10
bigjoolsit should not be but I seem to be suffering from pebcak10:10
bigjoolsall10:10
wgrantOK. So it was about a four line change :P10:11
bigjoolsyes, it should have been ....10:11
bigjoolsbut see if you can find any existing tests that I can add a 4 line change to10:11
bigjoolsthe testing is shambolic10:11
bigjoolswhich is why I am refactoring10:11
wgrantHa ha ha.10:11
bigjoolsit'll make testing easier10:12
bigjoolsPopen on a script is not a test, it's a sledgehammer10:12
wgrantI'd like to discuss PPA download stats at some point when you're free. I got something reasonably working and tested over the weekend.10:14
bigjoolsoh awesome10:14
wgrantOnly for binaries, since it is completely unclear how index counts will work.10:14
bigjoolsok let me finish this code first10:16
wgrantCertainly.10:16
deryckMorning, everyone.11:00
bigjoolshowdy deryck11:08
adeuringwhat happened to the bug heat icons? I don't see anything about bug heat in trunk or in db-devel...11:27
wgrantadeuring: It looks quite present in both to me.11:30
deryckadeuring, what do you mean?  You can't find how the icons are used in code?  or on LP itself?11:30
adeuringderyck: no, I don't see anything in the browser....11:30
adeuringOr am I looking at the wrong pages?11:31
intellectronicaadeuring: maybe that's the same problem joey had the other day, with icons not displaying?11:31
adeuringlike bugs.launchpad.dev/ubuntu11:31
adeuringintellectronica: may be, I don't know about joey's problem...11:31
wgrantThe hot bugs list doesn't show them.11:31
wgrantNormal bug listings and bug pages do, though.11:31
* adeuring is completely confused...11:32
intellectronicaadeuring: try any normal bug listing, they should be there11:32
deryckadeuring, yeah, the actual hot bugs list doesn't have them.  I think it should, though we've had some disagreement about this.11:32
deryckintellectronica, what do you think about this ^^?  i.e. adding them to the hot bugs list?11:32
intellectronicaderyck: the problem is that for any project with a reasonable amount of bugs, all the bugs on the hot bugs list will show four flames11:33
intellectronicaand with no new information, shame about the space11:33
intellectronicathen again, there's an argument from consistency11:33
wgrantDoes the hot bugs lists have a purpose?11:33
adeuringconsufison reolved...11:33
deryckintellectronica, yeah, I agree with the problem statement.  But it is a confusing UI, since they appear everywhere else.11:34
deryckwgrant, show the 10 hottest bugs.11:34
wgrantderyck: Why?11:34
wgrantderyck: What does that achieve that showing +bugs ordered by hotness doesn't?11:34
wgrantApart from requiring an extra page load to get more than 10 bugs.11:35
deryckwgrant, as an "overview" bugs page it shows you an overview of the hottest bugs.11:35
intellectronicawgrant: i think it's only really useful for users who aren't heavily involved in the project they're viewing. i almost never look at this list.11:35
deryckwgrant, everything on that page is a link to something else, no?11:35
wgrantderyck: How is https://bugs.edge.launchpad.net/ubuntu more useful than https://bugs.edge.launchpad.net/ubuntu/+bugs?orderby=-heat?11:37
deryckwgrant, I'm not arguing it is. :-)  But in "theory" there is room for more info on the top-level page, should we want to add it.  i.e. a "Recent books" top 10, etc.11:38
wgrantderyck: Possibly.11:39
wgrantBut this suffers from the same problem as lots of other features.11:39
wgrantit gets added in a completely useless fashion.11:39
wgrantAngers users.11:39
wgrantAnd then maybe eventually gets fixed a year later.11:39
wgrant(see also bug heat)11:39
deryckwgrant, first, I don't get the sense anyone is angry about this or that it's useless.  And second, we're not done, and I assure you, we won't leave this work until people are happy.  Can we make *every* Ubuntu dev or LP user happy?  No.11:40
intellectronicawgrant: one way in which it is more useful is that it renders faster11:40
wgrantintellectronica: I considered that.11:40
wgrantderyck: It irritates everybody on my team.11:41
wgrantbecause it means an extra click to get where they want to go.11:41
intellectronicawgrant: i find it surprising that people on your team even go to that page. i never do, and i was under the impression that most serious users don't.11:41
wgrantA feature looks a lot less bad if it's deployed fantastic rather than useless and fixed gradually later.11:41
wgrantintellectronica: The Bugs tab and breadcrumb go there.11:42
deryckwgrant, and we're trying to fix this.  BjornT is working on a solution to make this possible.  Right now, with monthly releases and incremental work in small branches, it's actually hard to do this.11:42
intellectronicawgrant: we're actually moving towards doing more work separated from edge before landing a feature11:42
wgrantThat would be very good.11:43
wgrantBug heat had the potential to be a pretty exciting feature. But it was rolled out too early, and is now just a set of four gray unexplained flames wasting space. Even when it becomes awesome later, it will not have the same effect.11:44
deryckwgrant, will have the effect of "wow, bug heat!"  No.  Will it be a nice, useful feature?  yes.  So what's the harm?11:45
deryckwgrant, (I'm being a bit sarcastic to make my point.  I agree we could do better at this.  But I think you're being a bit hyperbolic and unfair about he harm here.)11:45
intellectronicawgrant: to some extent there's agreement about that, which is why a lot of effort is being put now into providing the infrastructure for working like that, but i also have a lot to say in favour of working in the open, and letting features evolve with input from users11:45
derycks/he harm/the harm/11:45
wgrantintellectronica: Deploying it to edge is fine and great for that.11:46
wgrantderyck: Quite a few people really don't like Launchpad very much.11:47
wgrantImpressions like that do matter, sadly.11:47
intellectronicaquite a few people don't like coffee too11:47
deryckwgrant, I agree we could make a better impression.  We're trying to fix this.11:48
wgrantSo, I've just had a look at MergeWorkflowDraft. And it indeed looks like a much more flexible setup.11:51
=== matsubara-afk is now known as matsubara
=== salgado changed the topic of #launchpad-dev to: Launchpad Development Channel | Week 3 of 10.02 | PQM is open | https://dev.launchpad.net/ | Get the code: https://dev.launchpad.net/Getting | On-call review in irc://irc.freenode.net/#launchpad-reviews | Use http://paste.ubuntu.com/ for pastes | Channel logs: http://irclogs.ubuntu
=== daniloff is now known as danilos
=== pgquiles__ is now known as pgquiles
james_wis it intentional for a single file in lazr.enum to be LGPL with (or later), when the rest of lazr seems to just be LGPL?14:14
kfogeljames_w: seems unlikely to be intentional, unless that file has special provenance...14:17
james_whttp://bazaar.launchpad.net/~lazr-developers/lazr.enum/trunk/annotate/head:/src/lazr/enum/_enum.py14:17
james_wno other declared copyright holders and no statement of code taken from elsewhere14:18
=== jamalta-afk is now known as jamalta
kfogeljames_w: looks completely like an accident to me.  Check with barry maybe?14:25
james_wit looks like leonardr started lazr.enum14:25
kfogeljames_w: oh, maybe leonardr would be better14:25
leonardrjames_w: i'm looking14:25
leonardrjames_w: i believe that code was refactored out of launchpad14:26
wgrantjtv: Shouldn't start-soyuz.sh be Makefile targets, and the result of make-ubuntu-sane.py be added to the dev sampledata?14:27
jtvhi wgrant!  The latter is exactly what I'm looking at now.14:28
jtvThe big question is how many tests it'll break.14:28
adiroibansinzui: hi. do you remember the form next_url bug when you suggest using HTTP_REFERER ? The is a problem when the form is changing the object name and so the URL is changed. I have this code but I am not happy with it http://paste.ubuntu.com/382280/ ?14:28
leonardrkfogel: flacoste is the final authority but i think that's just an oversight and it can be changed14:28
wgrantNone, because they don't use the dev sampledata.14:28
sinzuihmm14:29
wgrantThey use the other sampledata.14:29
adiroibando you know a better solution?14:29
jtvwgrant: gah!  Of course.14:29
leonardrkfogel: see the README. it still says "Enumerated types are used primarily in two distinct places in the Launchpad code"14:29
* sinzui looks14:29
jtvwgrant: my brain doesn't seem to be working this week14:29
flacostekfogel, leonardr, james_w: it's an error, everything should be LGPLv3 only (no or later)14:29
kfogelflacoste: thx14:30
wgrantjtv: I've not been maintaining the script since I don't use the sampledata any more.14:30
jtvwgrant: why is that?14:30
kfogeljames_w: feel free to fix up the README as leonardr points out too :-).14:30
sinzuiadiroiban: take a look at .lp.registry.browser.product.ProductReviewLicenseView.next_url14:30
wgrantjtv: Too much bad idea in Ubuntu.14:30
jtvwgrant: EPARSE14:30
james_wkfogel: I wasn't planning to fix the license statement :-)14:30
wgrantjtv: bad data14:31
jtvwgrant: so what do you use?  Just dogfood?14:31
adiroibansinzui: but you can not change the product name from ProductReviewLicenseView14:32
adiroibanso the form action is not causing any URL change14:32
sinzuiadiroiban: sorry, I misunderstood your issue14:32
wgrantjtv: I have a similar script which uses the factory to populate an empty DB with celebrities and Ubuntu stuff.14:32
sinzuiadiroiban: yes, that is not pretty code, but at a glance, it looks correct.14:34
adiroibansinzui: yes. is is ugly. the correct one is here http://paste.ubuntu.com/382283/14:34
adiroibansinzui: I can not think at a prettier solution14:35
adiroiban:(14:35
jtvwgrant: that sounds interesting... though I don't want to get into a full-scale sample data rewrite, of course.  :)14:36
wgrantjtv: No. That sounds messy.14:36
wgrantI think you should be able to get away with just running that script and making newsampledata.14:37
sinzuiadiroiban: there is an old bug in launchpad-foundations about broken next_urls  caused by a rename. Launchpad does not handle the situation well in some circumstances. Most engineers never experience the problem because most objects cannot be renamed.14:37
jtvwgrant: yes, that's what I was thinking too.14:37
wgrantAlthough deleting all the primary publications probably isn't advisable, since it will break other parts of LP.14:38
wgrant(eg. bug filing on packages)14:38
adiroibansinzui: hm... is this bug 34228 ?14:40
mupBug #34228: Implement a "teleportation" or "jump-to" feature <feature> <Launchpad Foundations:Triaged> <https://launchpad.net/bugs/34228>14:40
jtvwgrant: it's really only just a status change, isn't it?14:40
wgrantjtv: Yes, but that might be sufficient to break things.14:41
jtvwgrant: I'd prefer to delete the old ones and set up some new, entirely up-to-date ones.14:42
wgrantAnd it will certainly be sufficient to render parts of the UI not easily testable.14:42
jtvThe catch is, I have no idea what's required.14:42
wgrantSo, the reason I delete them is that they reference librarian files without data.14:42
jtvwgrant: can we provide those?14:43
wgrantjtv: I suppose so.14:43
wgrantThat would let us have a reasonable set of default publications for the UI, and also be able to actually publish the archive if required.14:44
bigjoolsone day, death to sampledata apart from major celebs14:44
sinzuiadiroiban: no, the one am thinking of contains a step by step scenario of changing a person or product name and the user gets a 404.14:44
wgrantbigjools: I've already declared death to sampledata except for the bits of my script that I elect to run... it works pretty well.14:45
adiroibansinzui: i tried searching launchpad-foundations after „rename” or „next_url” ... but no luch14:45
adiroibanluck14:45
wgrantNo 5-year-old impossibly broken objects!14:45
bigjoolsheh14:45
bigjoolssome of the original sample data is.... special14:46
jtvwgrant: so... what would we need to provide?14:52
wgrantjtv: A tarball of the library files for the published sources and binaries.14:53
deryckadeuring, there are some issues with your icon branch, I believe.  hold on submitting to ec2 til I check something.14:54
wgrantBut it may just be easier to tell people to delete everything if they want to do anything Soyuzy.14:54
adeuringderyck: ok. What are the problems?14:54
jtvwgrant: I'm such a non-Soyuz person that what you just said doesn't mean much to me.14:54
deryckadeuring, there were new color icons as well and the icons were prepared to use a single image for each degree of heat.14:54
deryckadeuring, and these should be done via sprites, too.14:55
adeuringderyck: Ah, i see.14:55
wgrantjtv: So, publish-distro.py will whine and crash if there are published packages for which the corresponding librarian files are missing.14:55
deryckadeuring, I'll update the MP with comments and info.  Sorry to not have caught you earlier about these.14:55
adeuringI thought that the five images were just another way to show how things should look14:56
jtvwgrant: what needs to be in those files?14:56
deryckadeuring, no, those are actually the images to use.14:56
adeuringok14:56
wgrantjtv: Source and binary packages.14:56
jtvwgrant: how painful would it be to get those into the Librarian out of the box?14:57
wgrantjtv: Out of the box sounds branch-bloating. But providing a tarball that can be extracted into /var/tmp/fatsam for interested would be easy enough.14:58
jtvwgrant: true14:59
jtvwgrant: but is that better than having make-ubuntu-sane.py in utilities?15:00
wgrantjtv: I do not know.15:00
wgrantOne-size-fits-all sample data cannot really work.15:00
deryckadeuring, MP is updated now.  Sorry to send you back to work. :-)15:02
adeuringnp15:02
jtvwgrant: I was hoping it might, but I don't want to make it the goal of the month either.  :)15:02
* wgrant mumbles something about 2am, and sleeps.15:03
jtvwgrant: I was wondering about that...  in fact that's why I didn't ask you about it today  :)15:03
wgrantHeh.15:03
jtvgood night, and thanks.  :)15:03
wgrantnp15:04
deryckadeuring, so the alt and title attributes are required for heat icons and you note.  Sorry for my mistake...15:17
deryckadeuring, alt can be something like "3 out of 4 heat flames" I think.  I'm open to better suggestions there.15:17
deryckadeuring, and the title attribute is bug 51321915:18
mupBug #513219: flames have no tooltip <story-bug-heat> <Launchpad Bugs:Triaged> <https://launchpad.net/bugs/513219>15:18
adeuringderyck: OK, that works. But I though it would make sense to use something "super hot bug", "very hot bug" etc15:18
deryckadeuring, yeah, that works well too.15:18
deryckadeuring, and you could fix bug 513219 the same time as this one. :-)15:18
mupBug #513219: flames have no tooltip <story-bug-heat> <Launchpad Bugs:Triaged> <https://launchpad.net/bugs/513219>15:19
adeuringderyck: yure. But again, any suggestion what to display tehre ;)?15:19
deryckadeuring, I think this could be the heat number itself for now.  Something like: "Heat: 420"15:19
adeuringderyck: thnkas, that's a nice idea!15:20
deryckadeuring, when I finish my branch, we can include the context max_heat, so "Heat: 420 out of 10,000"15:20
adeuringok15:20
deryckadeuring, I think the tooltip could include "base on X subscribers, X dupes, X affected users" (or something similar)  But I worry that gets noisy.15:21
derycks/base/based15:21
adeuringderyck: yeah, let's keep it simple ;)15:21
deryckadeuring, agreed :-)15:22
derycksome have asked for this, though.15:22
kfogeladeuring: ready.  You might want to have my branch at hand (lp:~kfogel/launchpad/515584-use-zope-form) while we're talking.15:57
* adeuring is pulling the branch-...15:59
adeuringkfogel: shall i call you?16:01
kfogeladeuring: sure.  skype?16:01
adeuringkfogel: yes16:01
kfogeladeuring: lib/lp/bugs/browser/bugtarget.py16:02
kfogeladeuring: class BugsPatchesSortFormView(LaunchpadFormView):16:02
kfogel"""Browser view class for sorting bugtasks with patch attachments."""16:02
kfogellib/canonical/launchpad/webapp/launchpadform.py16:03
=== deryck is now known as deryck[lunch]
kfogelintellectronica: ayt?16:13
intellectronicakfogel: what does ayt mean?16:13
intellectronicaand in what language?16:13
intellectronicaare you there?16:13
=== salgado is now known as salgado-lunch
kfogelintellectronica: "are you there", yes.  And the real question is: do you have time for a quick call about lp:~kfogel/launchpad/515584-use-zope-form? :-)16:14
intellectronicakfogel: sure16:14
intellectronicakfogel: give me 5 minutes, though, to get skype going16:14
kfogelintellectronica: sure, ping me when ready16:14
=== matsubara is now known as matsubara-lunch
intellectronicakfogel: ready when you are16:18
kfogelintellectronica: ok, one sec to gt headphones on16:18
kfogelintellectronica: dailing you16:19
kfogellp:~kfogel/launchpad/515584-use-zope-form16:20
kfogelhttp://paste.ubuntu.com/382345/16:22
kfogelintellectronica: search for this in the file: def patchTaskOrderings(self)16:23
kfogellib/lp/bugs/templates/bugtarget-patches.pt16:23
kfogellib/lp/answers/interfaces/faqcollection.py:class FAQSort(EnumeratedType):16:27
=== Ursinha is now known as Ursinha-food
=== beuno is now known as beuno-lunch
=== deryck[lunch] is now known as deryck
kfogelintellectronica: I'm looking in the code for an example of a form that works the way you described.  If I grep for "(EnumeratedType" there are about 20 hits throughout the code (http://paste.ubuntu.com/382372/).  But I don't see new variables being created with those enumerated types.  Instead, the enum classes (or values in them) are used directly.  For example, in class IBranchListingFilter, see this line:16:46
kfogel    sort_by = Choice(16:46
kfogel        title=_('ordered by'), vocabulary=BranchListingSort,16:46
kfogel        default=BranchListingSort.LIFECYCLE)16:46
kfogelintellectronica: "BranchListingSort" is a class that inherits from EnumeratedType.16:46
intellectronicakfogel: yes, that's the way it's used16:47
kfogelintellectronica: ok16:47
intellectronicakfogel: EnumeratedType isa zope vocabulary16:47
kfogelintellectronica: *nod*16:47
=== jamalta is now known as jamalta-afk
kfogelokay, intellectronica -- please sanity check my writeup here (I mainly want to make sure that the example I chose is a textbook example of what we were talking about): http://paste.ubuntu.com/382378/16:56
intellectronicakfogel: yes, that's perfect16:58
kfogelintellectronica: thanks!16:58
=== matsubara-lunch is now known as matsubara
kfogelderyck: ping for 5-10 min phone chat when you have a chance17:12
kfogelre 515584 and some general prioritization questions17:13
deryckkfogel, ok.  Give me 5 minutes to get to stopping point, if that's cool.17:13
=== salgado-lunch is now known as salgado
kfogelderyck: yup17:15
abentleywgrant, around?17:22
deryckkfogel, I can chat now if you like.  Skype?  Phone?17:24
kfogelderyck: +1 (347) 591-773817:25
deryckkfogel, ok, calling now.17:25
=== beuno-lunch is now known as beuno
kfogelderyck: no you're not.  you're typing :-)17:25
deryckone handed17:26
kfogelwow17:26
kfogelhttps://bugs.edge.launchpad.net/malone/+bug/51558417:26
mupBug #515584: BugsPatchesView:batchedPatchTasks() should use a Zope form, instead of validating form values manually <story-patch-report> <Launchpad Bugs:In Progress by kfogel> <https://launchpad.net/bugs/515584>17:26
kfogelhttps://bugs.edge.launchpad.net/malone/+bug/32213017:26
mupBug #322130: Convert IHasBugs.searchTask(order_by) to use a real enumeration <api> <tech-debt> <Launchpad Bugs:Confirmed> <https://launchpad.net/bugs/322130>17:26
thekorn^ ha! - it would be great if the last one could be fixed ;)17:34
kfogelintellectronica: can I submit lp:~kfogel/launchpad/515584-fix-DRY-violation to you for review?  Note it has a very trivial UI effect -- the sort orderings have better names now -- so I'd list you as the UI reviewer too, though I'm not even sure it needs UI reviw.18:03
intellectronicakfogel: i'm just on my way out, so it's probably better if you try and find someone else. if you don't, submit it to me anyway and i'll review it when i'm back later in the evening18:07
kfogelintellectronica: np, thanxs18:07
kfogelgmb: mind if I submit lp:~kfogel/launchpad/515584-fix-DRY-violation to you for review?  Small change.18:07
bachi abentley.  i want to lp-land a branch for someone else.  if i use 'bzr lp-land <url-of-MP>' will it work, even if i don't have a local copy of the branch or a copy on LP?  should it just get all of that info from the MP?18:59
abentleybac, I haven't tried it yet, so I think it would fail.19:00
bacabentley: indeed it does, but with a bzr error19:00
bacit would be handy if it worked19:00
abentleybac, I agree.19:01
bacabentley: i'm getting: bzr: ERROR: exceptions.AttributeError: 'Entry' object has no attribute 'source_branch'19:06
=== gary_poster is now known as gary-lunch
abentleybac, that sounds pretty desperate.  source_branch would be on the merge proposal.  What's the traceback?19:08
bacabentley: i got no traceback but have a crash report19:16
bacwhich i haven't looked at yet19:17
abentleybac, rerun with -Derror.19:17
bacok19:17
bacabentley: still no traceback19:18
bacah, it's in the crash report19:19
bacabentley: http://paste.ubuntu.com/382450/19:19
abentleybac, that's very strange-- the object that's supposed to be a merge proposal has no source branch.19:20
bacabentley: OOPS -- i pasted in the URL to the branch not the MP!19:20
bacabentley: by that i mean, in the lp-land command i'd inadvertantly used the URL of the branch not the URL of the MP19:22
abentleybac, right, I'd forgotten about that option.19:22
abentleybac, we should probably add a -d option to specify the branch.19:23
=== gary-lunch is now known as gary_poster
mwhudsonjelmer: hi?20:52
mwhudsonargh argh argh git isn't installed on any data centre machiens20:52
jelmermwhudson, hello20:52
mwhudsonjelmer: you can probably guess what i want to talk about20:52
* mwhudson fires up an ec2 instance to test stuff on20:57
mwhudsonjelmer: :(20:58
jelmermwhudson: Cricket? ;-)20:59
mwhudsonheh20:59
mwhudsonno20:59
jelmermwhudson: I saw the URL you pasted. My guess is that we are somehow updating the git map before we've added the revisions to bzr21:00
mwhudsonjelmer: oh, so the git map contains all the objects in the git repo, but we only imported some of them21:01
mwhudsonjelmer: oh, something else that might have happened21:01
mwhudsonwe imported 1000 revisions21:01
jelmermwhudson, I'm just speculating21:01
mwhudsonbut the branch only contains 93021:01
jelmerbut that's what I would guess based on the error21:01
jelmermwhudson, oh21:01
jelmermwhudson, that would explain it as well21:01
mwhudsonpresumably because not all of the imported revisions are in the ancestry of the last one we imported21:02
jelmerperhaps we imported those revisions but they didn't up in the repo for some reason?21:02
mwhudsonwell, sure21:02
mwhudsonwe only move the branch around between import attemps with 'push'21:02
jelmermwhudson, does the repo contain 1k revisions even if the branch doesn't?21:02
mwhudsoni doubt it21:02
mwhudsoni can't access the repo directly in any case21:03
mwhudsonENOLOSA21:03
mwhudsonjelmer: A way to fix this, though perhaps not a totally sensible one21:04
mwhudsonis to, in import_git_objects, compute revision_ids as now, then take revision_ids[limit] as the new desired head and recompute revision_ids21:06
mwhudsonalternatively, we can try harder to preserve all revisions in the repo between import attempts21:06
=== salgado is now known as salgado-afk
lifelessrockstar: what are you looking at ?21:08
mwhudsonjelmer: thanks for the hint at least!21:09
mwhudsonbah21:13
pooliehello mwhudson, lifeless, jelmer21:13
lifelessmwhudson: this is incremental fetching?21:13
mwhudsonlifeless: yar21:14
lifelessmwhudson: you can use targetb.repository.fetch(sourceb.repository)21:14
lifelessmwhudson: that will copy all heads21:14
mwhudsonlifeless: oh that transfer all revs?21:14
lifelessnot just referenced ones21:14
mwhudsonlifeless: iinteresting21:14
lifelessjust call that after branch.push21:14
mwhudsonsweet21:15
lifelesshi poolie21:15
mwhudsonand pull i guess?21:15
mwhudsonor sprout, in this case, i think21:15
mwhudsonlifeless: what does BzrDir.sprout return? is it the branch?21:20
* mwhudson must try not to fix every oddness in one go21:22
=== matsubara is now known as matsubara-afk
lifelessmwhudson: yes and pull21:23
lifelesssprout returns the new bzr as per its docstring21:23
lifelessI'm pretty sure [without checking code, that is]21:24
mwhudsonlifeless: the docstring is not super clear about this21:24
lifelessmwhudson: if this works around it for you, file a bug on bzr-git, you shouldn't need to do this, and other folk doing imports may run into it21:24
mwhudson(i did read it first)21:24
lifelessmwhudson: ok; let me check then21:24
mwhudsonlifeless: um, noone else will be using the bzr-git feature for this21:24
mwhudson(as i wrote it last week)21:25
mwhudsonand there's no command line interface21:25
lifelessit returns the new bzrdir21:25
lifelessbzrdir.sprout, that is21:25
mwhudsoncool, that makes sense21:25
mwhudsonyeah21:25
thumperUrsinha-food: ping21:28
Ursinha-foodthumper: pong21:28
=== Ursinha-food is now known as Ursinha
thumperUrsinha-food: http://dev.launchpad.net/CodeTeamTestPlans/10.02?action=diff&amp;rev1=39&amp;rev2=4021:28
mwhudsonthumper: should be able to fix incremental imports easily enough \o/21:29
thumperUrsinha: just got a whole heap of old commits added to the test plan21:29
thumpermwhudson: awesome21:29
Ursinhaargh21:29
Ursinha*sigh*21:29
Ursinhathanks thumper21:30
thumperUrsinha: np21:30
Ursinhathumper: this is because I've implemented the changes to add only edge/staging items to the testplans but the files that contain the last revision numbers were overwritten21:31
thumperoops21:31
rockstarlifeless, I'm not sure what you mean.21:45
thumperrockstar: are you back?21:46
rockstarthumper, I am.21:46
thumperrockstar: talk now?21:46
rockstarthumper, 5 minutes?  My car is running in the driveway right now, and I need it to run just a little longer before I shut it off.21:46
thumperok21:46
mwhudsongosh launchpad takes a long time to import21:54
thumpermwhudson: from what?21:56
thumperrockstar: I have a call with flacoste in 5 minutes21:56
mwhudsonthumper: just the time between starting ./bin/test and tests actually start running21:56
mwhudsonit'21:56
mwhudsonit21:56
thumperah21:57
mwhudsonit's like 4 seconds21:57
mwhudsonand the test i'm running takes 0.1...21:57
rockstarthumper, okay, I'll grab you afterwards.21:59
thumperrockstar: take a look at the code first then :)22:00
rockstarthumper, doing that now.22:00
thumperflacoste: ready?22:00
flacostethumper: yep22:00
thumperabentley: is there an RT for the staging email issue?22:07
abentleythumper, yes, but I don't know the number.  Chex said he had an open ticket.22:08
flacosteabentley, thumper: RT #3760122:12
mupBug #37601: unable to burn dvd .iso <gnomebaker (Ubuntu):Fix Released by motu> <https://launchpad.net/bugs/37601>22:12
kfogelintellectronica: thanks.  incorporating your suggestions and resubmitting for review in a bit22:29
intellectronicacool22:29
maxbI wish the testsuite logged somewhere useful from its background components :-(22:39
maxbwgrant: Reckon we should start a lucid status wiki page?22:40
wgrantmaxb: I was thinking that last night, but there are only three tests plus one or two utilities.22:54
wgrant(plus that bootstrapping setuptools/distribute issue which only sometimes shows up)22:55
wgrant(question-target.txt, test_ftparchive, and update-sourcecode are the problems I know of)22:57
wgrantmaxb: Did you dsee that xx-resetpassword-of-sso-account.txt is gone?23:34
* mwhudson lunches23:37
wgrantI can't create a user any more, can I?23:46
lifelesswgrant: why not?23:46
* wgrant sighs at offloading of functionality to external proprietary applications.23:47
wgrantlifeless: LP uses OpenID now, so I can't create new users locally for testing.23:47
* wgrant gets out the factory.23:47
lifelesswgrant: so, you should setup an authorising party, and point your test lp at it for auth :>23:47
wgrantlifeless: It's producer-locked, sadly.23:48
wgrantAnd the test producer doesn't have functionality to create a user.23:48
lifelessyes, but you can change that :)23:48
wgrantThat functionality is reserved for that quaity piece of software that is c-i-p.23:48
abentleywgrant, launchpad does not yet use OpenID AFAIK.23:49
lifelesseven in your local, test lp ?23:49
wgrantabentley: As of 7 hours ago it does. Which is good.23:49
abentleywgrant, in stable or db-stable?23:51
wgrantabentley: devel. Might not be in stable yet.23:51

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