/srv/irclogs.ubuntu.com/2009/11/18/#launchpad-dev.txt

maxbwhy do we even *have* time-based rescoring?00:03
maxbIt seems a rather pointless concept given the queues are mostly FIFO anyway00:03
cody-somervillemaxb, they're not FIFO at all00:04
wgrantThe order seems to be based on (score, id)00:05
wgrantSo apart from score, they are FIFO.00:05
wgrantAnd most builds have the same score.00:05
maxbok, that was an oversimplification. But the increments added by the time-based rescorer are smaller than most of the other score-determining factors00:05
wgrant(except for a few which differ slightly on the distro builders, and private PPAs will get a 10000 point bonus)00:06
cody-somervilleI don't think anyone disagrees that scoring needs work00:06
bigjoolsright, the small increments don't make enough of a difference to change the order00:06
cody-somervilleit also stops making a difference after 4 hours of waiting00:09
wgrantThe entire build dispatch algorithm needs to be redone. Trying to fix scores is unlikely to do anything.00:10
* mwhudson lunches (and is setting up a new modem so will be mostly offline)00:15
thumpermwhudson: ping00:21
wgrantHmm, interesting.00:41
wgrantI cannot access +index of some projects. I suspect this is because the development focus branch is private.00:41
thumper??00:42
thumperarse00:42
thumperwgrant: can you please give me an example?00:42
thumperwgrant: and file a bug?00:42
thumperwgrant: I'll fix asap00:42
wgrantthumper: eg. https://launchpad.net/landscape. I don't have an OOPS code.00:42
wgrantI will file a bug.00:42
wgrantthumper: Registry or Code?00:42
thumpercode00:42
wgrantthumper: Bug #48453300:45
mupBug #484533: Cannot access index for project with private development focus branch <Launchpad Bazaar Integration:New> <https://launchpad.net/bugs/484533>00:45
thumperwgrant: thanks00:45
mwhudsonwell that was exciting01:24
mwhudsonplug in new modem, doesn't work, plug in old modem, doesn't work01:24
thumpermwhudson: what's that?01:24
mwhudsonphone isp tech support01:24
thumper:(01:24
mwhudson"it seems your account was suspended because of reports of spam last week"01:25
mwhudson!!!!01:25
wgrantOuch.01:25
thumpermwhudson: so now you have in the new modem?01:26
mwhudsonthumper: no, still the old01:26
mwhudsonand indeed, using 32 gigs of traffic in the first 18 days of the month is a bit unusual01:27
thumperoh01:27
* thumper stabs windmill01:27
thumpermy windmill tests are failing to use the api!!!01:27
thumperoh FFS01:29
* thumper frowns01:30
* thumper frowns more01:31
thumperAppServerLayer not tearing down01:31
thumperstabby stabby01:31
* ajmitch steps away from the knives01:31
mwhudsonafter all that01:32
mwhudsonlunch, finally01:32
thumper✁☹01:42
wgrantthumper: Windmill still killing you(ll)?01:43
thumperyes01:43
thumperworks locally, but not in windmill01:43
bacjml: yes, i've seen those import violations today since going to 2.501:52
thumperrecipe would live at https://launchpad.net/ubuntu/karmic/+source/some-package/+recipe/recipe-name03:39
=== ursula is now known as Ursinha
Ursinhahey stub, when you return, can we chat a bit?03:45
stubI'm here03:46
lifelessthumper: no team/person in there ?03:47
thumperlifeless: not in the url03:48
thumperlifeless: but on the object, yes03:48
thumperlifeless: do you think it is really needed?03:49
lifelessI don't know03:49
mwhudsonhooray03:49
stubUrsinha: ^^03:50
Ursinhahi stub03:51
Ursinhastub, what should I do after you approve my db-patch and give me a patch number?03:52
stubIf I and jml have both approved it, you need to land it on lp:~launchpad-pqm/launchpad/db-devel (with the correct patch number if you haven't changed that on your branch yet).03:53
Ursinhastub, but should I change from pending folder or just rename it?03:53
stubUrsinha: ^^03:53
Ursinha(and  change the line inside of the patch to contain the correct number)03:53
stubOh - it needs to be in database/schema.03:53
stubMost people keep them in there - pending tends to only be used by me.03:54
Ursinhastub, anything else?03:54
stubNup. If the tests pass its fine.03:54
wgrantstub: If I want to get some production data trivially fixed, how do I go about it?03:56
Ursinhastub, I did what you said, and I'm getting this: https://pastebin.canonical.com/24833/03:56
wgrantGet the query approved by the appropriate team and you?03:56
Ursinhastub, what am I doing wrong?03:56
stubwgrant: Sounds good. If it is really trivial I can just do it.03:58
stubUrsinha: The last statement of the DB patch should be "INSERT INTO LaunchpadDatabaseRevision VALUES (2207, 10, 0);".03:59
Ursinhastub, I've changed that03:59
stubUrsinha: Please pastebin the db patch04:00
wgrantstub: There are some thousands of old sourcepackagereleases with a null dsc_format. None of these have been created since 2006. Until now, only the '1.0' format has been accepted, so all the old SPRs should have their dsc_format set to that.04:00
Ursinhastub, it's in the other computer, a minute04:01
stubwgrant: I recall that. Sounds fine to me but this one I'd want soyuz team to agree too, just in case there is some need to differentiate records made before the code that added that field landed.04:01
stubWhich is a stretch, but better safe than fired.04:02
wgrantstub: Indeed. bigjools seemed happy with it last week, but I will get more concrete approval this evening.04:02
wgrantThanks.04:02
stubWe also need a bug opened to set the column NOT NULL and targetted to this cycle.04:03
wgrantShall I prepare and get landed a DB patch for that?04:03
ursulastub, http://paste.ubuntu.com/321244/04:04
stubwgrant: If you want, or I can just append it to my pending db stuff branch.04:04
wgrantstub: That would be handy. Filing a bug now.04:04
wgrantstub: bug #48460204:07
mupBug #484602: SourcePackageRelease.dsc_format should be NOT NULL <Soyuz:New> <https://launchpad.net/bugs/484602>04:07
wgrantstub: I cannot target or anything.04:07
stubI've got it04:08
* mwhudson afk for a few minutes04:25
=== Peng__ is now known as Peng_
mwhudsonis there a word for "tree that you could plausibly try to run debuild in" ?05:36
mwhudsoni.e. source tree, debian/ dir etc05:36
lifelesssource tree05:36
lifelessno, you don't run it in the debian dir, rather in the top dir05:36
lifelessyou could coin a word05:36
lifelessdebianized source tree05:36
lifelessis probably what folk would understand05:37
mwhudson"debianized source tree" works05:37
wgrant'[unpacked] source package' is my usual term.05:37
wgrantBut I like 'debianized source tree'05:37
mwhudson Results 1 - 10 of about 11,800 for "debianized source tree".05:37
spmdebianised perhaps?05:37
lifelessspm: sadly no, I'm not making this up :)05:37
lifelessspm: or I would have used an s05:37
mwhudsonspm: "debian" CERTAINLY did not arrive in the english language via french05:37
lifelessmwhudson: but did deborah and ian ?05:38
spmI knew I should have put the evil/ironic smiley in.... ;-)05:38
mwhudsonlifeless: deborah doesn't sound like it does it?05:38
mwhudsonian i guess might have done05:38
mwhudsonhah, debian policy says debianised05:41
spm*snort*05:41
* spm is reminded of the wikipedia rules on UK vs USA spelling :-D05:41
spivmwhudson: well, if it's *policy* then you have no choice...05:44
mwhudsoni think i shall alternate between the two in this mail and see how well that troll works05:44
spivmwhudson: pick a middle ground: "debianiced"05:44
mwhudsonspiv: that's a strange middle you're standing on05:45
spivmwhudson: it's the sort of middle that makes no-one happy!05:45
mwhudsondebianated05:45
mwhudsondebianorized05:46
spivdebianificated05:46
mwhudsonendebianned05:46
spmmwhudson: bigendebianned ???05:47
mwhudsonspm: that's clearly much too silly05:48
spmsorry05:48
spmwill aim for a more moderate level of silliness next time05:48
spm0xdebianised ??05:49
mwhudsonspiv, lifeless: are you guys on launchpad-dev?05:49
spivYou mean this channel?05:50
lifelessmwhudson: are you human?05:50
mwhudsonspiv: no, the mailing list?05:50
lifelessI think I am05:50
spivProbably.  I certainly get lots of mail about Launchpad...05:50
lifelessis there something we should have seen and replied to?05:50
mwhudsonlifeless: the mail i'm about to send05:51
lifelessmy advice is that if you want $foo to see mail, CC them always.05:51
lifelesson-a-list doesn't guarantee 'pay attention'05:51
mwhudsonyeah, i was thinking that05:52
spmjtv: heyo! can I draw your attention to https://answers.edge.launchpad.net/rosetta/+question/89602 pls. Is this (1) a sane request and (2) If Y, how? :-)05:56
jtvspm: looking...05:56
spmta05:56
jtvIt sounds like there was some mistake... pt and pt_BR are different enough that we treat them as separate languages.05:57
jtvIt's pretty exceptional for us to do that; the differences between country variants of Spanish can be pretty huge but people manage to keep those translations unified.  From that I infer that it really couldn't be done for pt and pt_BR.05:58
jtvspm: I'll follow up.  I've had a fair amount of interaction with these people.05:59
spmjtv: ta05:59
spmjtv: fwiw, from ex-brazilians I've worked with in previous jobs (vs the wonderful Brazilians I work with now! ;-) ) They speak in quite ... derrogatory terms of pt vs pt_BR :-)06:01
mwhudsonmy limited understanding was that they're basically different languages06:01
jtvspm: to make sure I follow... they hate pt, right, not the distinction of pt vs pt_BR?06:02
spmthe descriptive analogy I got was pt to pt_BR is like 12th century english to modern english.06:02
spmjtv: exactly.06:02
jtvThey may have gotten pt translations imported as pt_BR by accident, or vice versa.06:03
mwhudsonspiv, lifeless: mail sent fwiw06:16
spivmwhudson: ta, will take a look06:17
wgrantmwhudson: Does anybody have any idea at all how private branches are going to be done?06:26
mwhudsonwgrant: you don't mean private branches there do you?06:26
mwhudsonwgrant: but i don't think so, no06:26
wgrantmwhudson: I mean BFB from private branches.06:27
mwhudsonwgrant: ah06:27
mwhudsonwgrant: then no06:27
mwhudsoni guess it'06:27
wgrantThe only possibility that is obvious to me is to have them served over internal HTTPS, and have buildd-manager grant and revoke credentials.06:28
mwhudsons worth thinking about a bit so we don't paint ourselves into a corner, but right now BFB at all is looking hard enough to be starting with06:28
mwhudsonoh at that level06:28
mwhudsonyes, something like that06:28
mwhudsondoesn't really have to be https though does it?06:28
wgrantProbably not, as the buildd network seems rather secure. But it still seems like a Good Idea, and that's how it's done with P3As.06:29
wgrantWhat other problems are there with private BFB?06:29
mwhudsoni guess there must be some inspiration to be found in the restricted librarian06:29
wgrantmwhudson: The restricted librarian is not useful here.06:30
mwhudsonwgrant: well, P3Asa at least then06:30
wgrantRight.06:30
mwhudsonwgrant: well, i'm imagining stuff like preventing people building a private branch into a public pps06:31
mwhudsonppa even06:31
mwhudsonanyway it's late and i need to get away from the computer06:31
wgrantIt would be nice to design this properly from the start, so there isn't a crazy situation like there is with P3As, where file retrieval is completely different from public PPAs.06:31
wgrantIndeed.06:31
mwhudsonwgrant: feel free to reply to my mail06:33
mwhudsoni don't know how P3As work, i guess i should find out06:33
wgrantmwhudson: They are accessed from the archive with static buildd credentials over HTTPS. The master gives the slave a set of URLs with credentials embedded.06:35
mwhudsonwgrant: oh, for when a package build-deps on things in the P3A?06:37
wgrantmwhudson: That's separate.06:37
mwhudsonwgrant: then i'm not sure what you'06:38
mwhudsonre talking about, but as my brain has turned to runny cheese and dribbled out of my ears, perhaps that's ok06:38
wgrantmwhudson: For public archives, the slave is given a list of SHA1s and it is up to it to grab them from the librarian.06:38
wgrantFor private archives, the slave is given a list of HTTPS ppa.launchpad.net URLs with embedded credentials.06:38
wgrant(this is for retrieving the source package, not deps)06:39
wgrantBut anyway, late.06:39
mwhudsonoh that06:40
mwhudsonaccessing librarian files by sha1 seemed a little bonkers to me tbh06:40
wgrantIt's ancient Soyuz stuff. It is expected to be bonkers.06:43
spmmaxb: ref https://answers.edge.launchpad.net/launchpad/+question/89097 you didn't actually click on the proffered link did you? :-) And seeing as I'm about to remove that response, did you want me to tidy yours away as well?07:13
=== joey is now known as Guest63272
adeuringgood morning08:13
henningebin/test is segfaulting for me09:50
henningeI did make clean ; make build but it did not help ...09:50
henningemake run works fine btw09:52
allenaphenninge: Can you paste the full error?10:11
allenapMorning adeuring :)10:11
adeuringhi allenap!10:11
henningeallenap: on the command line it is just "segmentation fault"10:12
henningeallenap: I am currently tracing into it10:12
henningeallenap: it happens on "from canonical.testing.layers import *" in testgin/__init__.py10:13
henninge"from canonical.launchpad.ftests import ANONYMOUS, login, logout, is_logged_in" in layers.py10:14
allenaphenninge: Oh, not nice. Have you tried a "bzr clean-tree --ignored --force" for each directory in sourcecode/ ?10:14
henningetracing deeper ...10:14
thumpermake cleaner?10:14
thumperheh10:14
thumpersorry10:14
allenaphenninge: And then a rebuild?10:14
henningeallenap: no, let me try that10:14
allenapthumper: :)10:15
henningeallenap: looking into sourcode I found an invalid symlink for lsprof10:16
allenaphenninge: Mmm, link-external-sourcecode should remove dead symlinks in sourcecode/10:17
wgrantWhy do dead symlinks matter?10:18
wgrantupdate-sourcecode will remove obsolete sourcecode branches?10:18
wgrants/?/./10:18
henningewgrant: what's "update-sourcecode"? Is that run by rocketfuel-get?10:19
wgranthenninge: Yes.10:19
wgranthenninge: It grabs new sourcecode branches (in lp-sourcedeps), updates existing ones, and removes old ones.10:19
henningeI did run it recently but I'll try and run it again10:19
henningewgrant, allenap: unchanged :(10:28
henninge"make build" includes this line:10:29
henningelib/twisted/web/http.py:40: RuntimeWarning: Python C API version mismatch for module _c_urlarg: This Python has API version 1013, module _c_urlarg has version 1012.10:29
allenaphenninge: bac had a similar problem yesterday. IIRC, he did:10:31
allenaphenninge: . ~/.rocketfuel.env10:32
allenaphenninge: cd $LP_SOURCEDEPS_PATH/twisted10:32
allenaphenninge: bzr clean-tree --ignored --force10:32
allenaphenninge: make PYTHON=python2.510:32
allenaphenninge: The Twisted Makefile has python2.4 hardcoded :(10:33
henningeah, I see10:33
henningeallenap: let me try that10:33
gmbI come to learn things.10:34
allenaphenninge: Oops, that should be: . ~/.rocketfuel-env.sh10:34
henningeallenap: ;) saw that10:35
allenapCool.10:35
henningeallenap: after that a "make build" again?10:37
allenaphenninge: I don't think it should be necessary, but it'll do no harm I guess.10:38
henningeallenap: you are my hero today!10:39
henninge:-D10:39
henningenow the import facist is complaining ...10:39
henningeoh, that is after the test10:40
allenaphenninge: Woohoo \o/ :)10:40
henningeallenap: thank you10:40
allenaphenninge: You're welcome.10:41
allenapgmb: Any luck?10:41
gmbWe'll know in a second...10:41
gmballenap: Wewt, etc. Thanks.10:42
allenapgmb: Cool. I'll email the list with this too.10:43
=== matsubara-afk is now known as matsubara
maxbspm: (Re: https://answers.edge.launchpad.net/launchpad/+question/89097) Um, proffered link? I don't remember there being a link. Yes, it would make sense to tidy away my response since the thing I was responding to isn't there any more.11:23
thumpermwhudson: you shouldn't be replying to email at 40m past midnight12:17
thumpermwhudson: it sets a bad precident12:17
thumper:)12:17
sinzuibac: does bug 483607 need updating14:14
mupBug #483607: The person profile page still shows the image and link to set somone's location <Launchpad Registry:In Progress by bac> <https://launchpad.net/bugs/483607>14:14
bacsinzui: done14:15
sinzuisweet. The page looks much better without a empty map section14:16
sinzuibac: EdwinGrubbs: stand-up in 2 minutes14:30
=== jamalta_ is now known as jamalta
=== salgado_ is now known as salgado
jmlwhere do the doctest utilities live?15:06
jmlcanonical/launchpad/testing/pages.py15:07
=== salgado is now known as salgado-lunch
jmlgmb, hello15:09
gmbjml: Hi15:09
jmlgmb, I came across a XXX comment of yours that says we won't need canonical.launchpad.testing.systemdocs.ordered_dict_as_string once we've switched to Python 2.5 because dict ordering is guaranteed when __str__() is called15:10
gmbjml: Right.15:10
gmbI assume that I research that before inserting the XXX...15:10
jmlgmb, I'm trying to find a citation before I kill it.15:10
gmbThought you might be.15:10
gmb:)15:10
jmlgmb, heh15:11
jmlhttp://www.python.org/doc/2.5.4/lib/typesmapping.html doesn't say much about it.15:11
gmbjml: I'm sure I looked it up, but I can't remember where I got the information from.15:11
gmbI should have cited it in the XXX, sorry.15:11
jmlgmb, np.15:11
jmlI'll ask on #twisted15:12
=== abentley1 is now known as abentley
jmlman15:21
jmlthing that sucks about doctests15:21
jmlwe have a proliferation of dict-printing helpers15:21
jmlall of which are subtly different.15:21
jmlhttp://www.python.org/doc/2.5.4/lib/doctest-warnings.html15:26
=== Guest63272 is now known as joey
marsjml, makes you wonder if we should just define a sensible __str__() on the objects we want to print, or maybe even start using pprint inside __str__()?15:36
marsunless you are talking about printing true dictionary objects15:36
jmlmars, actually it makes me wonder why people continue to think doctests are a good idea15:37
marsI though of that too, I just didn't say it :)15:37
jmlmars, we should have useful __repr__() methods for more of our objects definitely15:37
jmlmars, in this particular context, I'm printing a thing of type dict though.15:37
marsah, so no way around that then.15:38
marsjust realized something.  BDD and cucumber lead to the same proliferation, but instead of dict output formatters, they lead to a proliferation of natural language interpreters.15:39
marseither way, you have a bunch of helpers to translate to and from plain text15:39
=== matsubara is now known as matsubara-lunch
marsmight just be necessary for the medium15:39
jmlmeh15:39
jmlI still prefer building a DSL in the programming language15:40
jmlI am bigoted against executable natural language.15:40
bigjoolsjames_w: are package recipes tied to a distro or are they more generic?16:02
=== salgado-lunch is now known as salgado
sinzuibac: are you available for our weekly call?16:37
[reed]gmb: huh, I had never heard of doctest before16:43
[reed]neat16:43
[reed]I don't get to do much python coding in my current job16:43
gmb[reed]: Yeah, we use them a lot. It's a nice way of keeping strong documentation, but it's not always terribly useful. We still love good old-fashioned unit tests for a lot of things.16:44
[reed]hehe16:44
[reed]yeah16:44
gmbSo, things with lots of iterations and lots of nuanced behaviour == unittests16:44
[reed]as much as it is a pain to write tests, they can be so useful :)16:44
gmb[reed]: Before I can land this, you need to have sent in your contributor agreement form. Have you done that?16:44
[reed]gmb: no, I didn't know there was one. If you can point me to where it is, I can probably get that done in the next hour or two, depending on what form I have to send it in.16:45
bacsinzui: i am now.16:46
gmb[reed]: https://dev.launchpad.net/ContributorAgreement16:46
gmb[reed]: Just ping me when you're done.16:46
[reed]literally, that was my first lp bzr branch ever ;)16:46
sinzuibac: calling16:47
[reed]ah, it's just a simple e-mail16:47
[reed]ok, it's a bit scary that in less than 30 min. after I post that bzr branch, the google alert for my name goes off and informs me of it16:48
gmb[reed]: If you don't contribute anything else in six months you get an email that says "we know where you live," too.16:51
[reed]lol16:51
[reed]gmb: ok, read the agreement and sent the e-mail... I cc'd you, as per the instructions.16:53
gmb[reed]: Brilliant, thanks. I'm working on the tests now. Should be able to land it tonight, or tomorrow at the latest.16:53
[reed]cool16:54
sinzuibac: https://edge.launchpad.net/launchpad-registry/3.116:59
al-maisanbeuno: just letting you know: jml is on level 317:01
gary_postermaxb, From following the instructions I was given, I believe you should be able to upload to the PPA.  Let me know how it works.17:02
beunoal-maisan, thanks17:07
salgadomars, what revno of lazr-js are you using in your yui3-final-upgrade branch?17:09
marssalgado, 15317:09
marssalgado, but that can be updated as needed.  Downgrading is a bit tricker.17:10
salgadomars, ok, just wanted to check we'd be using tip17:12
=== matsubara-lunch is now known as matsubara
salgadomars, btw, all that I'm going to integrate is the combo loader -- flacoste's css minifier will come together as a bonus?17:16
marssalgado, yes, I believe it was merged during the sprint.17:17
salgadoyeah, it was17:17
salgadomars, do you have an idea how that's going to be integrated into LP?  if not, I guess flacoste has17:19
marssalgado, I have no idea how.17:20
salgadoflacoste, I'm going to work on integrating the combo loader into LP, and I assume you've got an idea how that should be done, so was hoping to have a pre-impl call with you. ;)17:21
maxbgary_poster: Nothing to upload right now, but the Launchpad web ui claims I can upload. :-)17:23
gary_postermaxb: awesome :-)17:24
maxbI guess I could start populating it for Lucid :-)17:24
maxbBut actually many of the packages are obsolete now we don't need python 2.4 support17:25
maxbI should make a list and propose deleting them at some point17:25
marssalgado, presumably the combo loader will be a stand-alone process like the librarian.  It will also need access to all of our static files, and I don't know how that part would be linked into the system.17:27
maxbIs it just me or has someone broken the page layout on https://dev.launchpad.net/ ?17:29
maxboh, it's just that it wraps amazingly poorly if your browser window isn't wide enough17:30
maxbhm, has someone changed something about the way drop-down select boxes get handled on edge?17:32
maxbI'm seeing a major performance regression with drop-downs taking several seconds to display17:32
marsmaxb, which page?17:32
maxbppa pages17:32
salgadomars, yeah, that sounds reasonable17:33
marsmaxb, you are right.  I wonder why that is.17:33
maxbgary_poster: Also, I appear to have upload rights but not delete-package rights, oddly17:34
maxbmars: Hmm. I see the problem on the ppa root page, and on /+copy-packages, but *not* on /+packages, oddly17:35
marsmaxb, it may be the JavaScript blocking the main browser thread :(17:36
maxbETOOMUCHJAVASCRIPT :-(17:36
marsI can't reproduce it.  It is only on initial page load17:36
marsmaxb, not too much, just doing the wrong thing at the wrong time.17:36
maxboh dear. heisenbugs are not fun. It seems to be consistently affecting me17:36
marsmaxb, the other explanation is something not downloading fast enough.  On my page load, I see a long, long blocking on laucnhpad.js and the arrowBlank images.17:38
maxbsurely that shouldn't be an issue after navigating around a few launchpad pages?17:38
marsif it is cached properly, then no, it should not be an issue.17:39
marsmaxb, since this is a good reproducible case, would you mind filing a bug about it?  And please note the exact page(s) you are visiting, and the browser you are using.17:42
marsmaxb, I'd like to work on front-end performance at some point, and this would be a good point to cut my teeth on :)17:42
maxbHmm, it seems to only occur on PPAs with a non-trivial number of packages17:49
marsI had it happen on a PPA with two packages.  But only the first time.17:50
flacostesalgado: suer, let's have a call18:13
salgadoflacoste, I'm ready18:14
al-maisanhello salgado, do you have a minute? I'd like to ask your advice re. a branch I am currently working on.18:33
salgadohi al-maisan.  what's up?18:34
al-maisanI need to expose a method on the LP API and that new method would use a class that's currently in the browser layer (ProxiedLibraryFileAlias)18:35
al-maisanthe new method will be part of a model class18:36
al-maisanso, not quite sure what the best approach is18:36
al-maisansalgado: using browser-layer code in a model class is obviously less than ideal18:37
salgadoindeed18:37
salgadoal-maisan, do you really need to expose ProxiedLibraryFileAlias or do you just want people to have access to files in the restricted librarian?18:39
al-maisansalgado: to be more concrete, sourceFileUrls() (http://pastebin.ubuntu.com/321744/) will need to use ProxiedLibraryFileAlias18:39
salgadooh, I didn't understand it at first18:40
al-maisansalgado: we can't throw librarian URLs back since these may be *restricted* librarian URLs in case of private PPAs18:40
salgadoI thought you wanted to expose ProxiedLibraryFileAlias, but you just want to expose a method that happens to use ProxiedLibraryFileAlias18:41
al-maisansalgado: right!18:41
al-maisansalgado: bigjools just pointed me to an example where we already use ProxiedLibraryFileAlias in the model layer18:43
salgadooh, and ProxiedLibraryFileAlias.http_url calls get_current_browser_request()18:44
al-maisanbut if there's a better way of doing it I am open to it18:44
al-maisanthe changes_file_url() property in lp/soyuz/model/publishing.py is using it already and it does not seem to be a problem18:46
salgadoal-maisan, it will be if someone uses that property in a script, for example18:46
gary_postermaxb, I asked bigjools about you not having delete permission and he said it was expected/desired behavior.  Only PPA owners can delete.18:49
maxbquirky that I can use copy-packages but not delete-packages18:50
maxbdoesn't really matter, I guess18:50
maxbI guess copy-packages is 'sort of' an upload. If you squint a bit18:50
al-maisansalgado: can we continue later? It's lunch time here..18:51
salgadoal-maisan, so, basically we're making model code depend on the current browser request because we can only expose things from the model in the webservice?18:52
salgadoal-maisan, and I was thinking you were working late!18:52
al-maisansalgado: UDS :)18:52
salgadoal-maisan, sure thing, go enjoy some tex-mex food. ;)18:52
al-maisansalgado: "we can only expose things from the model in the webservice" .. is this a fixed rule?18:54
al-maisan.. or do you know of examples where non-model code was exposed via the LP API?18:54
salgadoal-maisan, no, I think that's just a limitation of our existing webservice18:55
al-maisansalgado: hmm .. I'll try using ProxiedLibraryFileAlias in the method that is to be exposed (although it's less than ideal) and see how it goes..18:56
al-maisandefinitely lunch time now18:56
salgadoal-maisan, it will work fine until somebody comes along and try to use that method in a script18:56
al-maisanright18:56
al-maisanI see18:57
al-maisansalgado: thanks for taking a look!18:57
al-maisanttyl18:57
maxbWoo, just got my "you have been added to launchpad-committers"19:01
maxbThough with none of the ~launchpad admins around, I guess it's not going to be actually have branches for a while yet19:02
* maxb reassigns some branches19:04
marssalgado, around?19:45
salgadomars, yep19:45
marssalgado, in Francis' reply to me on lp-dev, the thread about "Redesigning lazr.restful", he said that you know about common cases for update pages via AJAX?19:46
marssalgado, if so, I was wondering if you could reply to the thread with one of those cases, for the benefit of myself and other readers?19:47
salgadomars, the one I know about is the one he mentioned19:47
salgadoAdding a team member inline19:47
marsok19:48
marsso how does that involve updating multiple parts of the page?19:48
salgadomars, would you like to know what needs to be updated in that case?  or other cases where this would be needed?19:48
marssalgado, both19:48
salgadomars, if the new member is a person, we add it to the 'Recent members' section and update the count of active members19:49
salgadoif it's a team, we update the count of invited members19:49
salgadobut if the new member was a proposed/inactive member, then we need to update two lists and two counts19:50
marsoh?19:50
salgadomoving the member from the 'pending members' section to the 'recent members' one19:50
marsjust thinking, that leads to a problem.  Using francis' scheme, how do you know when to ask for one, two, or many fragments in return?19:51
marssalgado, anyway, please go on19:51
salgadomars, all of this is in one portlet19:51
marsah19:51
salgadoI think it's quite unlikely that an inline action would require changes to more than one fragment/portlet19:53
marssalgado, thank you, I'll reply to the thread19:55
salgadomars, np19:58
jelmerbigjools: yeah, it works now - thanks20:03
mwhudsonjames_w: hi, you there?20:04
mwhudsonjames_w: it seems like the official source package branches for indicator-applet are not in 2a format20:06
mwhudsonjames_w: do you want to fix that or shall i?20:06
* thumper is fetching kickass strong coffee20:17
mwhudsonhm20:20
marsabentley, ping20:20
mwhudsondid kiko make all the importd slaves fall over with those kernel imports?20:20
abentleymars: on a call20:20
marsk20:20
=== matsubara is now known as matsubara-afk
=== salgado is now known as salgado-afk
abentleymars: pong20:39
marsabentley, could you please explain to me how the +fragment URL scheme that you were using works?20:40
abentleymars: Sure, let me refresh myself.20:42
abentleymars: I manually construct the relative URL of the comment fragment, using the comment ID.20:43
abentleymars: Then I use Y.io to retrieve the fragment, and invoke a callback with the response text.20:44
marsabentley, so Y.io("/+bug/1234/+comments/1"), or something?20:44
mupBug #1234: Gina is an unmaintainable mess of command line options, environment variables and shell scripts <Launchpad Foundations:Fix Released by debonzi> <https://launchpad.net/bugs/1234>20:44
marsugh20:45
thumperabentley, mars: just an FYI20:45
thumperI started doing this with showing the merge proposal details on the branch page20:45
thumpery.io worked fine20:45
abentleymars: No, I was doing it only with code review comments.  So it was comments/120:45
thumperbut when I needed to do so on the bugs page20:45
thumperit blew up20:45
marsabentley, ah, but same idea20:45
thumperso I had to use the api to get the formatted code20:45
abentleymars: And then appending +fragment on the end, so "comments/1/+fragment"20:46
marsah20:46
abentleythumper: Did you try constructing an absolute URL?20:46
thumpergetting the fragments rendered all at once through the api is necessary20:46
thumperabentley: didn't work20:46
thumperabentley: there may be other reasons why20:47
thumperbut I couldn't find them20:47
abentleythumper: I'm surprised, because I thought that limitation only applied to AJAX.20:47
thumperabentley: it is ajax20:47
thumperdoesn't Y.io use xmlhttprequest?20:47
abentleythumper: But it returns text, not xml.20:48
* thumper shrugs20:48
* mwhudson blinks20:48
abentleymwhudson: You know the same-origin policy for ajax, right?20:49
mwhudsonwell vaguely20:50
mwhudsonabentley: i think the part that confuses me is what you mean by "ajax"20:50
abentleymwhudson: I mean making asynchronous requests for XML using javascript.20:50
* sinzui saw that remark coming20:51
mwhudsonabentley: but that's not what ajax means in practice these days, is it?20:51
mwhudsonas ~noone actually does the xml bit20:51
marsmwhudson, that may be true, we just never talk about it :)20:52
abentleymwhudson: Okay, I rephrase my comment to "I'm surprised, because I thought that limitation only applied when AJAX actually does XML".20:52
sinzuiAJAJ The greek hero of jam20:52
mwhudsonabentley: ah, i don't think that's true20:53
mwhudsonmaybe i'm wrong!20:53
abentleymwhudson: Apparently with json, it's popular to use an IFRAME hack to circumvent the same-origin policy.20:56
abentleymwhudson: Though according to http://www.json.org/fatfree.html this only gives access to subdomains.20:57
marsthumper, fwiw, same-orgin policy isn't a problem for the JavaScript API.  The whole webservice is exposed via every subdomain's /api URL.20:57
* mwhudson sticks his fingers in his hears and hides under his desk20:57
mwhudsonmars: that's why he had to change his code to actually use the api :-p20:58
mwhudson(at least, that was my understanding)20:58
marsyep :)20:58
sinzuiabentley that is true about the iframe, and YUI allows you to shoot yourself in the foot20:59
marssinzui, how?20:59
sinzuiabentley: I believe there is another reason though to use an iframe, I think it was file uploads20:59
thumpermars: yes, that is how I got it working20:59
abentleymars: Anyhow, I think the idea of allowing us to get the views, and get them all at once, makes a lot of sense.21:00
* thumper is starting to wake up 3 hours after getting up21:01
thumperabentley: +121:01
sinzuimars: I think any site that circumvents the security in my browser and risks handling my data should also have its pages re-written in javascript to say security-holes are here to play, come steal user user's data today21:01
marsheh21:02
ajmitchhow poetic21:02
marswell, there was that nifty page that stole your gmail contact list...21:02
thumperflacoste: ping21:04
flacostehi thumper21:04
flacostecall?21:04
thumperyeah21:04
marsWow, cherrypicks take a long time to run.21:15
mwhudsonyes21:15
abentleymwhudson: call?21:17
mwhudsonabentley: skype ok?21:17
abentleymwhudson: sure.21:17
mwhudsonthen yes21:17
abentleymwhudson: https://pastebin.canonical.com/24854/21:18
mwhudsonabentley: http://twistedmatrix.com/pipermail/twisted-python/2009-April/019482.html21:20
mwhudsonabentley: https://launchpad.net/ampoule21:26
mwhudsonyay skype21:44
abentleymwhudson: Yeah, it's getting to bug me more and more.21:45
mwhudsonabentley: i'll try to get my new router going21:46
abentleymwhudson: thanks.21:46
mwhudsonand then we can sip for great justice or something21:46
marsis the layout of the subscribers portlet on edge broken for everyone, or just me?21:54
marsto pick an arbitrary bug, #23456721:55
mupBug #234567: Gnome toolbars unresponsive after update and no toolbars on reboot. <toolbar> <gnome-panel (Ubuntu):Invalid by desktop-bugs> <https://launchpad.net/bugs/234567>21:55
wgrantmars: Yes.21:55
wgrantmars: It was fairly broken last week.21:55
wgrantmars: But this week it transitioned to really broken.21:55
marswgrant, last week!21:55
wgrantA bug was filed last night.21:55
marswgrant, thanks.  #484848 :)21:58
mupBug #484848: Subscriber icons in the subscribers portlet appear on a separate line from the subscribers' names <bug-page> <ui> <Launchpad Bugs:Triaged> <https://launchpad.net/bugs/484848>21:58
mwhudsoncool bug number22:01
wgrantToo close to 500000 :(22:01
mwhudsonhttps://code.edge.launchpad.net/~vcs-imports/linux/trunk is running really fricking slowly22:13
jmltee hee22:24
jmlmwhudson, hi22:24
mwhudsonjml: hello22:25
* jml waves to the rest of the launchpadders22:25
* jelmer waves22:26
* mwhudson starts working through the stack of replies to his mail of last night22:26
jmlso... when are we doing bzr-svn?22:26
mwhudsonjml: after i've replied to these mails i guess22:27
jmlwoop woop!22:27
mwhudsonjml: or you or jelmer could bang on https://code.edge.launchpad.net/~mwhudson/launchpad/bzr-svn-imports a bit22:27
marsflacoste, cool, thanks for the reply22:27
mwhudsonwhere do the packages that get built by the current soyuz system get signed?22:28
* jml looks22:29
wgrantmwhudson: The sources are signed on the dev's machines.22:29
jmlorrible ack :)22:29
wgrants/'s/s'/22:29
mwhudsonwgrant: sorry22:29
mwhudsonwgrant: s/packages/binaries/22:29
wgrantOops, didn't read the 'built by'22:29
wgrantmwhudson: They're not actually signed themselves.22:29
mwhudsonwell they are by the time they get into the archive?22:30
wgrantmwhudson: They're stuck in the repository, and the repository's Release file (containing lots of hashes) is signed.22:30
mwhudsonoh22:30
mwhudsonok22:30
mwhudsonthis happens during publication then?22:30
wgrantYes.22:31
mwhudsonare the source packages that are uploaded for a ppa published before they are built?22:31
wgrantFor PPAs, not necessarily.22:32
wgrantFor P3As, yes.22:32
wgrant(because P3A builds need to get the source files from the published archive)22:32
mwhudsonok22:32
mwhudsoni think the mist is beginning to lift22:33
mwhudson(or close in, or something)22:33
wgrantNow, sources don't actually have to be signed. But as bigjools says, you probably should if you can.22:33
mwhudsonwgrant: is the 'not necessarily' for PPAs because publication and building are essentially independent?22:33
mwhudsonman, i'm amazed the soyuz guys got P3As to work22:34
mwhudsonmuch respect due there22:34
wgrantmwhudson: Well, they worked, but I was able to penetrate them simply three times, so it's a restricted value of work.22:34
wgrantmwhudson: The source signature is only used to verify that the uploader is who they say they are.22:35
wgrantmwhudson: Once archiveuploader finishes with it, it is no longer important.22:35
wgrant(and archiveuploader already accepts unsigned source packages from trusted sources)22:36
wgrantWhat could perhaps work is getting the builddmaster to sign them once it gets them from the slaves, but that's a bit ew.22:39
thumperrockstar: ping22:55
rockstarthumper, hi.22:55
spmmars: hi; "<maxb> spm: (Re: https://answers.edge.launchpad.net/launchpad/+question/89097) Um, proffered link?" it looked like a sig; but was to.. um.... an male enlargement assistant site. To put things delicately.23:00
maxbhuh. I don't remember seeing a link there. Are the spammers getting clever enough to stick their links in legitimate-ish responses now !?23:07
marsspm, ?23:07
spmmars: gah. sorry. tab completion fail. 1001 apologies.23:08
marsok.  I didn't see the link-spam anyway?  I hate spam.23:08
maxbspm removed a question comment to which I'd responded. Apparently there was a dodgy link there. I'm confused because I interpreted it as a genuine user, and wrote a response to it, without noticing.23:08
spmmaxb: well... one of them at least. the responses they we're making were *vaguely* on topic; just ... odd. but the link was a no-no. and some of the responses they had were pure spam.23:09
wgrantsinzui: Why Jaunty->Lucid rather than Karmic->Lucid?23:09
james_wmwhudson: I have a list somewhere of the packages that aren't in 2a23:09
james_wI haven't done the work to upgrade them all yet, do you have a way to do it easily?23:09
maxbspm: huh. there was only one response from that user at the time I wrote my response.23:09
sinzui wgrant It is a typo23:09
mwhudsonjames_w: well there is a script now23:09
thumperrockstar: it doesn't look link yui 3.0 is in devel23:09
spmmaxb: (mars) they hit about half a dozen different answers across the place23:09
thumpermars: are you landing it?23:09
mwhudsoni don't know if we have an easy way of letting it rip at a bunch of branches23:09
sinzuiwgrant: And my script definitely knows not to do exactly that23:09
rockstarthumper, it's in db-devel23:09
wgrantsinzui: great.23:10
james_wmwhudson: for loop?23:10
sinzuiin fact. I think I need to get my confidence up and ask a LOSA to run it on staging.23:10
mwhudsonjames_w: i guess :-)23:10
spmsinzui: I do recognise I'm scary to ask. just saying. ;-)23:10
marsthumper, waiting for the CP on db-devel, fixing up any gross errors (I've found one so far), then merging back into devel.23:11
mwhudsoni don't know if we have an easy way of scheduling even one branch to be upgraded i guess23:11
mwhudsonrockstar: ^^ ?23:11
marsthumper, maybe early next week.  But you can build on top of the branch now!23:11
sinzuispm: The sample data is small, My script certainly works. as wgrant hinted. The order I fix the packing link is very important.23:12
* sinzui fixes bug title23:12
wgrantsinzui: I don't see why you single out Feisty there.23:12
james_wmwhudson: I don't think trying to do this over conference wifi would be wise23:12
james_wmwhudson: I could pass the list to you?23:12
mwhudsonjames_w: that would be great23:12
mwhudsoni can always do it from devpad if the new sexiness proves uncooperative23:13
spmmwhudson: james_w: coming in this half way thru - I gather this may be appropriate to run on crowberry itself? or?23:13
sinzuiwgrant: the original bug mentioned that, and that is the point we added the initialisation.. My script runs from hoary to lucid23:13
wgrantsinzui: Ah, I see.23:13
rockstarmwhudson, I don't think so.23:14
mwhudsonspm: maybe yes, don't think it matters too much23:14
mwhudsonspm: you can't do it at the fs level because of the ever joyful stacking23:14
rockstarmwhudson, so, what we have REALLY needs to be tested.23:15
spmmwhudson: sweet....23:15
mwhudsonrockstar: well i guess we can test it23:15
mwhudsonrockstar: the way of scheduling a job now is "INSERT INTO BranchUpgradeJob ...." ?23:16
james_wmwhudson: http://paste.ubuntu.com/321956/23:16
james_wthat's the packages23:16
james_winferring the branches from that is left as an exercise for the reader23:16
mwhudsonjames_w: btw23:16
mwhudsonjames_w: what should i do when i find a source package has no branches?23:16
* mwhudson feels a query of doom coming on23:16
rockstarmwhudson, yes, but that's not even all that tested.23:17
mwhudsonrockstar: ok23:17
james_wmwhudson: the branch URLs are predictable fwiw23:17
mwhudsonrockstar: so maybe i won't try to upgrade all of these branches as the first test :-)23:17
rockstarmwhudson, I have UI, but didn't think it wise to land while I was sprinting/at UDS/week 323:17
rockstarmwhudson, please for the love of Thor no.23:17
james_wmwhudson: for packages with no branches, file a bug against "udd" project please23:17
mwhudsonjames_w: one per package?23:17
mwhudsonrockstar: >:)23:18
sinzuispm: I have this script that walks the history of Ubuntu's series and copies the packaging links restore the data we have been losing with each series we create. I want to run this test on stagings database to verify it runs, and to examine the quality of data that it makes. https://pastebin.canonical.com/24860/23:18
mwhudsoni guess a test on staging is a good idea in any case23:18
rockstarmwhudson, yeah, probably.23:18
james_wmwhudson: a bunch per package23:18
spmsinzui: sure. one sec.23:19
rockstarmwhudson, Also, the upgrade won't actually upgrade to 2a just yet.  I didn't realize that until week 4 of last cycle.23:19
james_w~ubuntu-branches/ubuntu/<distroseries>/<sourcepackagename>/<suite>23:19
rockstarWe never updated the format map when we moved to 2.023:19
mwhudsonrockstar: groan23:19
mwhudsonjames_w: sorry, i'm being dense, was that aimed at me?23:20
rockstarmwhudson, yes, but I think we can actually get rid of the mapping now, since a call to upgrade should upgrade properly to 2a23:20
james_wmwhudson: yeah, however, I'm out of battery and so can't expand, sorry :-)23:20
mwhudsonjames_w: the query to find all the official branches for a particular source package is not very hard23:20
rockstari.e. I shouldn't have to make some format first...23:20
james_wmwhudson: that's probably the better way for you to do it then23:20
maxbjames_w: Do we still file bugs for native packages without branches? Or is that a general problem still?23:20
mwhudsonjames_w: the "of doom" part was thinking about requesting upgrades of all of them :)23:21
james_wmaxb: file a bug please23:21
james_wback in a few when I can escape from this corner of the session with no electricity23:21
spmsinzui: quick Q; is this expected to be quick; or slow? do you want it run with any special options or just go for it?23:22
mwhudsonjames_w: https://code.edge.launchpad.net/ubuntu/lucid/+source/bzr-fastimport23:22
sinzuispm: I honestly do not know. I think it could complete in a few minutes because we are looking with 1000s of objects.23:23
spmsinzui: oki; suck and see time.23:23
sinzuispm: when it is done. We will know exactly how may objects we are playing with23:24
spmsinzui: I do love Dev's who wildly overestimate how long their code takes to run. ;-) https://pastebin.canonical.com/24861/23:26
sinzuispm: It better to please than to disappoint. I schedule milestones the same way23:27
spmhahahaha23:27
sinzuithose numbers do look like the pages. I'll look at lucid on staging and see if it makes sense. I suspect I have just create 1200 links to packages that are in PPAs, not main23:28
wgrantThere are some links to PPA packages, but not that many.23:30
sinzuiugh. It copied squeak.  That project is proprietary and should be deactivated23:31
sinzuiThese links look correct. I see some wrong one, but they are wrong because the version changed late in Karmic, so karmic and lucid both need need fixing23:44
jelmerbigjools: Hi23:48
jelmerbigjools: Do you perhaps have a free moment to look at a zope issue?23:49
bigjoolsjelmer: possibly, I am sorting out a production problem, I can try and find you later23:49
jelmerbigjools: thanks - we're in presidente23:51

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