nigelbwallyworld_: \o/01:01
nigelbIt works, except for the anonymous user case, which I'm fix0ring now :)01:01
lifelesscan has review? https://code.launchpad.net/~lifeless/python-oops/extraction/+merge/7179401:17
lifelesscan has review two? https://code.launchpad.net/~lifeless/python-oops-datedir-repo/extraction/+merge/7179501:18
lifelesscan has review three? https://code.launchpad.net/~lifeless/launchpad/useoops/+merge/7179601:18
lifelesswas there any objection to just nuking 2011-08-17 01:12:19 WARNING Found 2 competing templates with translation domain 'javascript': "openerp-javascript in OpenERP Web Client 6.0"; "view-calendar-javascript in OpenERP Web Client 6.0".01:19
lifeless ?01:19
wgrantlifeless: Why has that started spamming us now, anyway?01:21
wgrantlifeless: I wonder if somebody broke asuka's mail capturing.01:21
lifelessprobably puppetisation01:22
lifelessrt 4741301:24
lifelessand set to prior 9001:24
wgrantWould be nice if we were warned when nasty machines like that were puppetised, so we could watch for breakage.01:24
wgrantOnly 90?01:25
wgrantWe may be spamming innocents.01:25
lifeless90 is zomg01:25
wgrantAll Zopeless mail is probably untrapped.01:25
lifelessthe only higher thing I can sanely do is ring the escalation phone at 2:25am01:25
lifelessThis hasn't been picked up on for several days.01:26
lifelessso I think that would be overkill01:26
* lifeless looks around for a revuer01:27
StevenKlifeless: Hm?01:32
lifelessStevenK: oh hai; I have three small reviews up ^01:33
lifelessStevenK: (they really are small. The Genuine Article.)01:33
jtvStevenK, wgrant: mind if I update & restart dogfood?01:43
StevenKjtv: Go ahead01:44
lifelessStevenK: thanks!02:09
* StevenK tries to get his eyes unglazed after reading 4,000 lines of diffs02:10
lifelesstime for a vodka tonic ?02:10
StevenKAt midday?02:10
lifelessevery 4000 lines :P02:11
lifelesscode review drinking game02:11
StevenKOh, I was going to go with "Every time you mark an MP as approve, take a drink. Every time you mark an MP as Needs Fixing, finish the glass."02:12
lifelessah, thank you internet: http://movieboozer.com/2011/07/04/source-code-2011-drinking-game/ close enough02:12
lifelessStevenK: that works too02:13
jtvbtw StevenK, wgrant: I put a script on dogfood that updates the code / dependencies / db permissions, rebuilds the current branch, and restarts the app server.03:01
jtvTakes a few minutes but it's convenient, and can save us the "damn this time a partial update doesn't quite do it" exceptions.03:02
jtv*Unfortunately* it breaks at the "make" where the Makefile tries to rm -rf /var/tmp/mailman, which is root-owned on dogfood.03:02
wgrantA few minutes?03:03
wgrantYou mean a few hours?03:03
jtvAlthough… no, /var/tmp/mailman's been fixed.03:04
jtvwgrant: no, a few minutes.03:04
jtvIt doesn't update the database (though it does update the schema)03:04
jtvOf course if you didn't run it for months on end, it'd probably take a lot longer.  But having it be easy to run would also solve that.  :03:05
jtvI just ran the thing, actually.03:05
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
jtvwgrant: have a look — ~launchpad/bin/upgrade-dogfood-launchpad03:06
wgrantI've only SSHed into the DC once this week, and I'm trying to not do it again :)03:07
jtvOK OK OK03:07
jtvJust try it sometime.  :)03:07
jtvUh-oh.  StevenK's laughing at me.  This can't be good.03:09
nigelbwgrant: Are you still on vacation?03:09
jtvnigelb: if you have to ask, I guess that means he never was.03:10
nigelbjtv: haha. I think he needs to vacation in some small island without internet.03:11
jtvI thought he lived on one?03:11
jtvIt's that blip just off the cost of Asia, isn't it?  It has that airport where you refuel on the way to South America.03:11
StevenKjtv: You can talk03:11
jtvYes, I can.03:12
StevenKjtv: I think Australia has more Internet than Thailand03:12
jtvThat can't be right.  Thailand has more internet than any other country in the world.03:12
jtvMeasured by the number of sites the government censored, at least.03:12
nigelbjtv: is in Thailand?03:12
jtvMore than China.  Come on, who's laughing now?03:13
nigelbI thought it was stub in Thailand.03:13
jtvYes, yes, you're right.  I'm mistaken.03:13
lifelessnigelb: we are allowed > 1 person in a country :)03:19
lifelessjamesh: hi03:20
jameshhi lifeless03:20
lifelessjamesh: did you release that python-gpg update?03:20
nigelblifeless: haha03:20
jameshno I haven't.  I may as well sort that out now.03:20
lifelessjamesh: I can put a nag into cron if thats better for you :P03:21
lifelessjamesh: also how much do you know about wsgi-oops03:22
jameshlifeless: I've only made a few small changes to it.  It started out with the Landscape team trying to extract the LP OOPS system, and then we added some WSGI glue to that03:23
lifelessjamesh: yeah03:24
jameshwhat did you want to know?03:24
lifelessjamesh: I'm redoing that but differently... and migrating LP at the same time rather than never :)03:24
lifelessjamesh: well, the traceback extract grabs an awful lot of data03:24
lifelessjamesh: thats not marshallable in a sane way into the current rfc822 format03:24
lifelessjamesh: but I'm wondering whether its been useful to folk; should I file a wishlist bug on python-oops to port that code across ?03:25
jameshlifeless: back when I was on the LP team, one of the long term goals was to switch to storing OOPS reports as zip files, as a way to make it easy to represent attachments like the traceback and sql statement log.  Obviously that has never happened03:26
jameshlifeless: at this point, if the reports are readable by oops-tools, I don't think many people would care what format they are stored in03:27
lifelessjamesh: right; I'm more talking about the stack-walking variable extraction code than format03:27
lifelessjamesh: porting oops-tools to use lp:python-oops-datedir-repo/serializer_rfc822 is fairly straight forward - and once thats done migration to e.g. bson dicts is trivial.03:28
jameshlifeless: are you talking about wsgi-oops, or the LP oops code here?03:30
lifelessjamesh: wsgi-oops has stack-variable-extraction, LP oops doesn't (it just uses zope.exceptions.format_exception), oops (the new module) doesn't, (It just uses traceback.format_traceback)03:31
jameshI remember the LP oops code used zope's traceback collection API, but the wsgi-oops one seems fairly simple03:31
jameshoh.  I see what you're talking about.03:33
jameshI don't think the locals or globals dictionaries from the traceback frames ever make it into the serialised oops report03:35
lifelessjamesh: http://bazaar.launchpad.net/~canonical-launchpad-branches/python-oops/trunk/view/head:/oops/createhooks.py#L9103:35
lifelessjamesh: right, because the rfc822 format is so limited ;)03:35
jameshI know the LP code used Zope's traceback formatter, which did include some locals in the output03:35
lifelessjamesh: yeah, so the lp errorlog module now installs its own exc_info grabber03:36
lifelesswhich is about the same as the code I just linked to, but usin the zope thingy03:36
jameshlifeless: so, given that we aren't storing this data, perhaps it would be okay to leave it out of the new system you're building.  If it is useful, then perhaps we can work out a way of storing it later.03:38
lifelessjamesh: thanks03:39
lifelessjamesh: I had tentatively concluded that03:39
lifelesstime to register yet another lp project03:39
lifelesspython-oops-wsgi, to avoid confusion :P03:39
jameshlifeless: it might be worth checking up with the Landscape folks to see if they are using a different serialiser that does actually store the data though.03:39
jamesh(or you might have already done that)03:40
lifelessjamesh: I've chatted with them about this project, not about that specific bit of detail.03:40
lifelessit should be easy enough to port the logic later03:40
lifelessmy first goal is a generic, used by LP, and used by a non-LP wsgi app (the gpgverify one by choice), implementation03:41
lifelesswith strict dependency boundaries - for folk shipping it, don't want to bring in the universe03:41
lifelesssecond goal will be updating oops-tools to use the parser module from datedir-repo to replace its homegrown one03:42
lifelessthen we can add a new serialiser to datedir-repo and win.03:42
jameshlifeless: https://launchpad.net/pygpgme/trunk/0.203:56
lifelessjamesh: sweet03:57
lifelessnow to get it uploaded $everywhere :)03:57
lifelessjamesh: http://pypi.python.org/pypi/pygpgme03:57
jameshI'm doing that.03:57
lifelessjamesh: heh, sorry!03:58
lifelessjamesh: awesome04:04
lifelessI'll chase up lp's dep versions a little layer04:04
poolielifeless: hi; how would we go about getting a release of restfulclient and co04:08
* poolie is hoping for something better than 'do it myself'04:08
lifelesspoolie: it is do it yourself04:13
poolielifeless: what i really meant is, can't the lp core team do it?04:13
lifelessmodulo the pypi permissions (we -really- would benefit from a teams module there)04:14
lifelessand I can grant pypi permissions as needed, I think flacoste added me to everything04:14
lifelessthis checklist needs trimming and automation, rather desperately, but it is functional04:15
pooliei feel like i'm already kind of doing at least half the work by fixing the bug04:18
poolieoh well04:19
lifelessI mean, thats what bzr contributors expect - they fix, core team releases on $whatever schedule04:20
lifelessare you asking because a release is long overdue? or its particularly urgent?04:20
pooliethe first04:21
poolielet me check if this is actually true though04:22
poolienot all that long04:25
poolieperhaps the difference is there's no obvious schedule04:26
lifelesshow much merged work is sitting unreleased?04:26
pooliethough, that's also somewhat true for bzr plugins04:26
pooliejust two https://launchpad.net/lazr.restfulclient/+milestone/0.11.304:26
poolieit's not urgent, it just would be a shame to have them sit for a long time04:27
pooliei thought there were more in other lp dependencies04:27
lifelessso one possibility is that the system is working04:29
lifelessanother possibility is that with leonard gone, folk are not releasing-on-land, nor will anyone release-on-schedule04:29
jtv\o/ I think I've figured out what's breaking rocketfuel-get on dogfood.  When it looks for "sourcecode" dirs to re-link, in dogfood's setup it also finds lp-sourcedeps.  Maybe rocketfuel-get should exclude lp-sourcedeps explicitly.04:30
pooliethat's the uncertainty that was in my mind04:31
lifelessI think some discussion on list is in order04:32
lifelessI'm not sure what (or even whether) we have a policy about doing-releases-of-things-we-garden [and perhaps we don't need one... but perhaps we do]04:32
lifelessjtv: isn't lp-sourcedeps normally a level up ?04:34
jtvNormally, yes.04:34
jtvBut these scripts evolve over time.04:35
jtvSo it depends on the age of the LP installation.04:35
jtvAnd in this case, permissions and responsibilities.04:35
jtvIf my hunch is right, fixing up rocketfuel-get is the least invasive.04:35
lifelessfailing buildmaster tests04:49
lifelesscould not stab /var/tmp/buildd/build-slave.pid04:50
StevenKOh, screw you, testrepository04:50
lifelessStevenK: ?04:50
lifelessoh, and for bonus I have a test fail in there too04:51
StevenKtestr failing --list => no output, testr run --failing => I'll just run every single test, lol04:51
lifelessStevenK: heh04:51
StevenKWith no output04:51
lifelessStevenK: bit of a foot-gun event, I guess.04:51
StevenKsteven@liquified:~/launchpad/lp-branches/denorm-bspph% testr run --failing04:51
StevenKrunning=xvfb-run ./bin/test --subunit04:51
lifelessjamesh: do you happen to know how I can reset gnome-gpg to ask for my passphrase again ?!05:09
lifelessand preferrably nuke the silly thing that ever captured it05:10
jameshlifeless: delete the passphrase from the keyring using the "Passwords  and Encryption Keys" app05:10
jameshI haven't used gnome-gpg in years (gnupg-agent seems to do a good job)05:10
lifelessjamesh: that seems to want to change the actual gpg managed passphrase05:11
lifelessjamesh: which isn't what I intend05:12
jameshlifeless: you're looking on the passwords tab, right?05:12
jameshnot the "My Personal Keys" or "Other Keys" tabs05:12
lifelessjamesh: it shows 'passwords: default and 'Passwords: login'05:13
jameshlifeless: right.  expand those and find the entry gnome-gpg added.05:14
lifelessthere isn't one :)05:14
jameshwell, that's where gnome-gpg would be storing the passphrase05:14
lifelessI had to click 'unlock'05:14
lifelessto unlock the first folder, before it would show me its contents.05:14
lifelessui win!05:14
lifelessjamesh: thanks!05:15
jamesh"passwords: default" would be a keyring from before the PAM integrated login keyring05:15
jameshif it doesn't hold anything interesting you would probably be best off deleting it.05:15
lifelessjamesh: I've been running this homedir uninterrupted since, uhm, 200005:15
lifelessjamesh: it appears to have a lot of semi interesting things05:16
jameshthe login keyring is locked and unlocked automatically by pam05:16
jameshthe default one requires manual password entry (assuming it has a password set)05:17
=== almaisan-away is now known as al-maisan
StevenKjtv: O hai. Do you have time for a small review?05:31
jtvYes, okay05:32
StevenKjtv: https://code.launchpad.net/~stevenk/launchpad/denorm-bspph-qualified/+merge/7181205:32
jtvI find it odd that you should come to me with this request, after all my hard work to scare you off.05:32
StevenKjtv: lifeless owed me, but he's buggered off. And you're supposed to be OCR anyway05:33
jtvah yes05:33
jtvAnyway, I see a familiar pattern: it's as if someone has tried to make the SQL harder to figure out.05:34
jtvBy arbitrarily turning some keywords into lower-case, and putting the line breaks in odd places.05:34
jtvCould you please just put the FROM, in all caps, at the beginning of the line?05:35
StevenKjtv: In bugtask?05:35
jtvYes.  And maybe make the other SQL keywords consistently upper-case as well?05:35
wgrantI wish it was practical to write a lint check to force that.05:36
jtvYes!  Oh God, yes!05:36
jtvAnd go away.  You are not here.05:36
wgrantFew reviewers do.05:36
jtvReviewers nowadays are a bunch of pussies.05:36
jtv"Oh yes, I can't read this, but if you get it past the python parser I'll approve it."05:37
StevenKjtv: Anything else?05:37
jtvAFAICT it's all cosmetic.  Am I missing something?05:38
StevenKwgrant: I'd solve the bloody problem and storm-ify the methods, but they make my eyes bleed.05:38
StevenKjtv: It's lead-up to adding a sourcepackagename column to SPPH05:38
lifelessif you can make a lint check for it, you can make an automated fixer for it.05:39
jtvThen go forth and land ye your branches, that they may be merged.05:39
jtvlifeless: you can, but lately I've really come to appreciate the affirmative power of manual formatting: "yes I've thought about this, yes I mean this to be in the source code as it is."05:41
jtvI even rely on it by not inserting proper spacing into temporary debug statements etc, as an extra safeguard against accidentally leaving them in.05:42
lifelessmmm, I can see an argument for that, but I'm not convinced05:44
jtvI'm not making an argument, I'm telling you how it is.  It helps.05:46
wgrantLet's just version control the AST.05:46
lifelessif python was just a little less fragile05:47
lifelessnow where, was I, thats right, housework. Then extracting code to oops-wsgi05:47
* jtv wonders if that is pronounced "oopski"05:48
=== jtv changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: jtv | Critical bugs: 238 - 0:[#######=]:256
=== jtv is now known as jtv-eat
LPCIBotProject db-devel build #814: FAILURE in 5 hr 47 min: https://lpci.wedontsleep.org/job/db-devel/814/06:49
jtvthanks for the surprise review, lifeless :)07:07
lifelessjtv: de nada07:12
rvbaStevenK: Hi! Thanks for the review Steven, I made the corrections you suggested.07:25
adeuringgood morning07:36
jtvmorning adeuring07:40
adeuringhi jtv!07:41
=== al-maisan is now known as almaisan-away
jtvStevenK: you're not still reviewing, are you?  Because I've got this: https://code.launchpad.net/~jtv/launchpad/bug-824553/+merge/7182307:52
bigjoolsmorning all08:05
mrevellGood morning08:07
StevenKrvba: r=me08:15
rvbaStevenK: Thanks!08:16
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away
=== almaisan-away is now known as al-maisan
stublifeless: Was the reason pqm results weigh in at 400k for a single test failure because of the Librarian logs not being cycled between tests, or something else?09:33
lifelessthere is a bug open I think; all that needs to happen is the fixture glue to be tweaked to only attach new rows from the log, rather than all of it.09:33
* stub wonders how the Librarian would cope with that many HUPs09:33
lifelessor truncate the log, or something.09:34
lifelessstub: stress test time!09:34
lifelessstub: one easy way would be to diff the log in memory - read or stat at the start, and etc09:34
stubMaintain an open file, seeking to the end after each test.09:35
stubfsync before the seek if you care.09:35
stubsomehow handle the tests that test Librarian log file rotation :-)09:36
stubHow good is the bzr cherry picking story now? If I merge a revision from db-stable into devel, will it cause a conflict when stable is merged to db-devel?09:39
pooliehi all09:45
poolieany lazr or lplib experts that could look at https://code.launchpad.net/~mbp/udd/819910-service-root/+merge/71832 ?09:45
poolieis there a more kosher way to do it?09:45
lifelessstub: it may; but if the texts are identical it should never.09:47
lifelessand -woo- we have a wsgi layer.09:47
stubSo as long as I don't have to resolve conflicts should be fine.09:47
lifelessand its (at least to me) a lot cleaner than the launchpad-loggerhead on. I may be biased.09:47
cjwatsonjtv: sorry, I know I still owe you an assessment of your custom upload plans, and I promise to figure out what I think today - but any chance you could review https://code.launchpad.net/~cjwatson/meta-lp-deps/fix-target-detection/+merge/71840 for me?  there was a regression when IS tried to deploy my previous meta-lp-deps change last night09:51
jtvcjwatson: funnily enough I was just composing another IRC message for you.  :)09:52
jtvI'm having a look.09:52
lifelesspoolie: whats your cli for pqm called again ?09:52
lifeless(and bugs etc, your nih api client :P)09:52
jmllifeless: hydrazine?09:53
jmllifeless: not to be confused with lptools or ubuntu-dev-tools ("ubuntu-dev" being code for "launchpad" there)09:53
pooliein hydrazine09:54
pooliethe nih api client is 'wrested'09:54
lifelesspoolie: I needed it to do that review for you :)09:54
jtvcjwatson: taking a while for the MP to update.  Did you get a failure email by any chance?09:58
cjwatsonnot yet ...09:58
* cjwatson pokes offlineimap just in case09:59
cjwatsonjtv: it seems to be up to date when I look from here09:59
jtvcjwatson: yup, just got it at last.10:00
jtvcjwatson: not much I can do with this one, but reviewed.10:04
jtvMeanwhile, about those custom uploads...  :)10:04
jtvcjwatson: can you land it yourself?  I'm not even sure I can, tbh10:05
jtvYour MP I mean, not mine.10:05
danilosjtv, heya, I've got a branch up for review: https://code.launchpad.net/~danilo/launchpad/bug-826692/+merge/7183610:06
jtvdanilos: do you now?  Good for you mate!10:07
=== danilos changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: jtv, danilos | Critical bugs: 238 - 0:[#######=]:256
cjwatsonjtv: I can't; it should just involve merging it into lp:meta-lp-deps, which is owned by ~launchpad-committers and which apparently you just push stuff to directly rather than via PQM10:07
danilosjtv, yeah, I am just bragging around, thanks10:07
cjwatsonjudging from the commit history10:07
jtvcjwatson: yes, that's how it happens with the less well-managed branches.  Can't say I'm familiar with this one.10:07
cjwatsonjtv: I can deal with talking to IS about the actual deployment stuff after that10:07
jtvdanilos: let me just help colin out here, and then I'll get to your branch10:08
jtvOr maybe the other OCR… oh.10:08
danilosjtv, sure, thanks10:08
jtvOh wow, I've worked with meta-lp-deps in the distant past.10:09
danilosjtv, yesterday?10:09
cjwatsonjtv: going to try to get my head round your mail now10:09
jtvYour considerable head.10:10
jtvShouldn't be too hard.10:10
stubCan someone else run bin/test.py -vv test_garbo , and tell me if it hangs for over a minute on test_HWSubmissionEmailLinker and test_ObsoleteBugAttachmentPruner ?10:19
danilosstub, sure, need it on latest stable/devel/db-stable/db-devel?10:21
stubdanilos: stable or whatever. I think it is an old problem nobody noticed.10:22
danilosstub, running now, you got me confused for a moment with bin/test.py instead of bin/test :)10:24
jelmerpoolie: btw, can you land my branches on lp:lptools, or should I apply for members?10:25
stubActually, needs to be stable or devel - I think a cherry pick from db-devel -> devel killed it.10:25
stubdanilos: ^^10:25
pooliethere's already a bug for https://code.launchpad.net/~ubuntu-branches/ timing out right?10:26
jtvcjwatson: your change has landed.10:27
pooliejelmer: i'll look10:27
danilosstub, right, I am trying with stable10:27
jelmerpoolie: thanks :)10:27
poolieyou have the power!10:27
danilosstub, I am having some troubles with tests actually failing10:30
stubdanilos: database in use errors?10:31
danilosstub, nope, LaunchpadSilentFailure or something :/10:33
daniloslet me check my dependencies10:33
danilos(it's likely my download-cache is out of date)10:35
cjwatsonjtv: thanks!  and replied (though whether you'll like the reply I'm not sure ...)10:35
jtvdanilos: can't the preloading you're doing be done with load_related?10:35
jtvcjwatson: argh!10:35
jtvdanilos: thinking specifically of do_eager_load (I understand you probably don't want to mess with that other code in detail)10:36
danilosjtv, not that I know of, since I need to do a bunch of "get me this and then stuff that is related to it" (I did start with a bunch of load_related and load_referencing initially)10:36
danilosjtv, i.e. I need double "related" indirection10:36
jtvCan't you pass a result set to load_related?10:36
stubSo on stable, database/schema/trusted.sql no longer has _killall_backends - this removed in db-devel along with code changes. Looks like trusted.sql has ended up in stable without the code changes, which makes the db teardown rather flakey and slow for tests that used the librarian.10:36
jtvdanilos: ahhh, iswym10:37
jtvyou're starting out with a python-constructed nontrivial set of ids, not of objects.10:37
stuboh... nm. it is still in stable. It is in testfuncs.sql, not trusted.sql10:38
stubRunning stable tests with a db-devel database... and it sometimes works!10:39
jtvdanilos: done10:39
danilosjtv, thanks10:39
jtvcjwatson: aren't ddtp-translations tarballs flushed from the librarian after a while?10:42
jtvI mean, it's GC'ed automatically.10:43
jtvAnd AFAIK fairly aggressively.10:43
jtv(Also note that dsync isn't currently used.)10:43
cjwatsonjtv: surely not *that* frequently?  there's usually a ddtp-tarball upload not very long before release10:46
jtvcjwatson: are we talking hours, days..?10:46
danilosstub, ok, finally got it all to a clean state, I don't see the lag: https://pastebin.canonical.com/51339/10:46
stubdanilos: ta10:47
cjwatsonjtv: for natty it was about two weeks (https://lists.ubuntu.com/archives/natty-changes/2011-April/011309.html)10:47
cjwatsonjtv: but we could copy them if they're available and not worry if they aren't, I'd have thought10:48
cjwatsonjtv: dsync> that's a shame10:48
jtvcjwatson: stub will correct me on this if needed but I think Librarian is on a much tighter GC schedule than that.  IIRC wgrant said flat-out that it would be impossible to do the copying for ddtp-translations.10:48
wgrantjtv: ddtp-translations is probably not expired.10:49
cjwatsonjtv: what generates the md5sums index then?10:49
wgrantjtv: Rosetta translations are.10:49
cjwatsonlp_archive@cocoplum:~$ ls -l /srv/launchpad.net/ubuntu-archive/ubuntu/indices/md5sums.gz10:49
cjwatson-rw-r--r-- 1 lp_publish lp_publish 29926782 Aug  8 10:23 /srv/launchpad.net/ubuntu-archive/ubuntu/indices/md5sums.gz10:49
jtvOkay, so we could add ddtp-translations.10:49
cjwatsonthat looks unfortunate10:49
wgrantcjwatson: It was generated by dsync.10:49
cjwatson(I'm fairly sure that there exist mirroring tools which look at indices/md5sums.gz)10:49
stubIf the LibraryFileAlias is linked to another row in the DB, it is not garbage collected. If it is not linked, it will be removed in <72hours IIRC.10:49
wgrantWhich we should really running again.10:49
cjwatsonwgrant: yes, that's my point :)10:49
stubWe can change that timeout easily enough, although we pay for it in disk space.10:50
jtvcjwatson: I have no doubt that it could be improved in lots of ways, but unless there's something decidedly wrong or unsafe I'd much prefer to get a partial solution in now and improve on it later, than drag this out further.  It's already sat around bit-rotting for longer than is usual nowadays.10:51
cjwatsonjtv: that's fair enough, but I would like to see those comments addressed at some point10:51
cjwatsonI wasn't intending these to be blockers, but you did ask for a review :-)10:51
jtvBut since this was originally billed as "if you think there's something you can do about it easily, you know, in a day or so..."10:52
cjwatsonI *think* your date comparison approach is safe, even if IMO not the right thing10:52
jtvWell it was more a "if you know of any reason why this should not land, speak now or forever hold your peace" kind of thing.10:52
cjwatsonperhaps you can turn it into bugs10:52
jtvI was thinking kanban cards, but I can do both.10:53
cjwatsonor that, I don't know your processes :)10:53
cjwatsonjust avoid being forgotten10:53
jtvAn advantage of splitting these off is that other people can work on them, who knows, maybe even at the same time.  :)10:53
jtvGlad to help.  If you'll let me.  :-P10:53
jtvcjwatson: by the way, any idea where in LP these tarball names are already being parsed?10:56
cjwatsonjtv: lib/lp/archivepublisher/{debian_installer,dist_upgrader,ddtp_tarball}.py10:56
jtvGreat, thanks.10:56
poolieif there's anyone on maintenance good at sql perhaps they could look at bug 82793510:57
_mup_Bug #827935: consistent timeout on user branch listing page <Launchpad itself:New> < https://launchpad.net/bugs/827935 >10:57
cjwatsonwhich is what processes those uploads in the first place10:57
cjwatson# This code is mostly owned by Colin Watson and is partly refactored by10:57
cjwatson# Daniel Silverstone who should be the first point of contact for it.10:57
cjwatsoncurrent comments 'r' us10:57
pooliei love how many old names still appear in the lp test data10:58
pooliedaf, daniels,10:58
cjwatson(custom uploads used to be processed entirely by hand.  it was horrendous)10:58
jtvcjwatson: and you're saying that the first parameter in the tarball name is not needed for disambiguation?  It can change in the next upload while still being a complete replacement for the current one with a different prefix to the tarball name?10:59
jtvI've been assuming that architecture _if present in the tarball name_ does disambiguate, i.e. a ppc dist-upgrader does not replace an i386 dist-upgrader with otherwise the same tarball name.11:00
jtvAnd that if there's no architecture, that means "all."11:01
cjwatsonthe upload code disambiguates the tarball type by a field in the .changes file ('raw-debian-installer' etc.) rather than by the tarball prefix.  Ideally that would be preserved somewhere in the database too, and I thought it was11:02
cjwatsonISTR an enum for it11:02
jtvbigjools, do you know about this?  ^^11:03
cjwatsonall the current parsers just ignore the first component11:03
cjwatsonexistence of that suggests to me that the type is stored in the db somewhere11:03
cjwatsonthough I guess that could just be queue rows11:04
bigjoolsit is somewhere11:04
cjwatsonif it isn't stored, then in practice you can probably get away with looking at the first component of the tarball name, since AFAIK they've never changed yet11:04
cjwatsonjtv: TBH I have no idea what would happen if we ever tried to upload architecture-dependent dist-upgrader tarballs11:05
bigjoolsjtv: see PackageUpload.contains_*11:05
cjwatsonjtv: but certainly a powerpc version of the debian-installer images tarball does not replace an i386 one with otherwise the same tarball name11:05
jtvcjwatson: PackageUploadCustomFormat just distinguishes between debian-installer, ddtp-translations, dist-upgrader, rosetta-translations.  Much coarser.11:05
cjwatsonjtv: then I do not know what you mean by "the first parameter in the tarball name"11:05
bigjoolswhich uses PackageUploadCustom.customformat11:06
jtvSo again, not what we're looking for.11:06
jtvThe PackageUploadCustom is already fully accounted for.11:07
bigjoolscan you re-state the question in simple terms then so I don't have to wade through backscroll11:07
cjwatsonthe tarballs are debian-installer-images_VERSION_ARCH.tar.gz, dist-upgrader_VERSION_ARCH.tar.gz, or translations_COMPONENT_VERSION.tar.gz; I assumed you meant the "debian-installer-images", "dist-upgrader", or "translations" part when you said "the first parameter"11:07
jtvOK: if we get two custom uploads of the same type for the same distroseries, what information should I take into account to determine whether the one supersedes the other, or is a separate thing that may also need to be copied?11:08
jtvAnd I understand that this differs by custom type, but I'd like to know the specifics for debian-installer; dist-upgrader; and ddtp-translations.11:08
cjwatsonFor debian-installer-images and dist-upgrader, different version + same arch means it supersedes.  For translations, different version + same component means it supersedes.11:08
cjwatsoni.e. we want the most current version of (debian-installer-images/per-arch, dist-upgrader/per-arch, translations/per-component)11:09
bigjoolsjtv: in lp/archivepublisher/ there are custom file handlers for publishing11:10
jtvThat's not too far off what I have now, except (1) I currently don't have support for ddtp translations and (2) I currently also disambiguate by the "name" part (e.g. "debian-installer-images" vs. "dist-upgrader").11:10
jtvbigjools: yes, colin just pointed those out.11:11
bigjoolsjtv: they are called via PackageUploadCustom.publish...11:11
bigjoolsthe whole custom file thing is hideous11:11
jtvDoes that logic encode anything about what supersedes what?  If so, it may be better to extract and reuse.11:11
cjwatsonyes, to some extent11:12
bigjoolsit does, somewhere, trying to find it11:12
cjwatsone.g. I pointed to the fixCurrentSymlink stuff which does version comparison11:12
cjwatsonjtv: if the name changes, then the version will also change, so there isn't really much point in disambiguating on it11:13
cjwatsonthis is guaranteed because if you don't change the version then you don't get a new directory under /dists/$SERIES/main/$BLAH/11:13
jtvThat's fine then.  It's just that for one of the upload types I ended up not supporting (translations I guess), the name part seemed to be a package name.11:13
cjwatsonno, it's NAME_COMPONENT_VERSION in that case11:14
cjwatsonwhere NAME == "translations" (not enforced/required but in practice true right now), COMPONENT in ("main", "restricted", "universe", "multiverse")11:14
cjwatsonthat's the ddtp-tarball type11:14
cjwatsonrosetta-translations might have a package name there, but we don't care about rosetta-translations for this because they aren't published to dists/11:15
jtvSo only custom type & architecture matter for now, and for ddtp, component will also matter.11:17
jtvAh, I have some examples in the database now.  That helps.11:19
cjwatsonjtv: and ddtp isn't per-architecture, *only* per-component11:19
cjwatsonI expect at least part of the code for this ought to live in the DebianInstaller etc. classes ...11:20
jtvWell I treat "no architecture" as "all" for genericity.11:20
cjwatsonerr, DebianInstallerUpload I mean11:20
cjwatsonthat sounds like an encapsulation bug, if you're having to consider architecture in generic code that suggests that actually the decision ought to be pushed down to the classes that know about the specific upload format11:21
nigelbmrevell: Hi!11:21
jtvThe point is to isolate the format-specific part, so that it only extracts enough information to group the uploads such that of each group, only the latest matters.11:22
henningejtv: here is a fun branch to review for you  ;-) https://code.launchpad.net/~henninge/launchpad/devel-update-testtools-r228/+merge/7165911:22
jtvhenninge: sorry, past eod11:23
mrevellnigelb, Hi, on the phone just now.11:23
=== henninge changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: danilos | Critical bugs: 238 - 0:[#######=]:256
henningejtv: np11:23
henningedanilos:: here is a fun branch to review for you  ;-) https://code.launchpad.net/~henninge/launchpad/devel-update-testtools-r228/+merge/7165911:23
wgrantjtv: Regretting your evil endeavour yet?11:23
nigelbmrevell: cool, can you please poke me once youre done? :)11:23
jtvwgrant: not at all—it turns out not to be impossible at all and my design can stay largely intact.11:24
nigelbSeeing how much wgrant is failing to disappear from here during vacation, I'd suggest a +b from #launchpad-* during vacation time ;)11:24
jtvOoooh good idea11:24
wgrantI only drop in occasionally!11:25
nigelbwhere ocassionaly = ~2 hours11:25
jtv…and then stay for how long, would you say?11:26
jtvbigjools, cjwatson: different concern…  I notice that the uniqueness check for debian-installer uploads distinguishes between uploads with ".0." in them and ones without!11:27
cjwatsonjtv: true, although we haven't used that for years11:28
cjwatsonperhaps we should just ditch that, it is a rather bizarre way to do it11:29
jtvAye, that it is11:29
jtvQuestion is, what should be done about it?11:29
cjwatsonin fact I don't think we've used it since we switched to LP11:29
cjwatsonit dates from the similar dak code11:29
cjwatsonso if it's causing problems I suggest just binning it11:30
* jtv files separate bug11:31
nigelbDoes rocketfuel setup a password for postgres after I install postgres with that?11:31
cjwatsonI mean, in theory I suppose daily builds of d-i would be kind of nice ... but they were a lot more useful in the early days than they are now, and in practice just uploading d-i again whenever I need to works out fine11:32
cjwatsondefinitely not worth awkward infrastructure to support it11:32
danilosjam: hi, do you think you could have a go at https://code.launchpad.net/~danilo/loggerhead/bug-718566/+merge/7184911:33
daniloshenninge, looking at your branch now11:34
jtvnigelb: it configures a trusted TCP login from localhost.11:34
henningedanilos: thanks11:34
nigelbjtv: so, can I safely do sudo passwd postgres and not break my lp install?11:34
jtvnigelb: oh, a system password for the postgres user?  I don't see any reason why it should have a password at all.11:35
jtvnigelb: sorry, I thought you were talking about a password for logging into the postgres databases.11:35
jtvSystem users generally shouldn't have passwords.  No good for anyone but attackers.  :)11:35
nigelbI'm just following instructions for hacking on harvest, which I think maybe a bit overzealous11:36
daniloshenninge, that was an easy one, r=me :)11:37
jamdanilos: commented11:38
henningedanilos: indeed. Thanks ;-)11:38
danilosjam: thanks, I'll try it out on a LP branch (I suppose you are suggesting doing it locally)11:39
jamnote that the *first* request is going to be uncached as well11:40
jamsince it has to do the work11:40
jammy feeling is that it is already fast enough that we don't need the cache at all anymore11:40
danilosjam: yeah, I can't really tell the difference between the two (trunk and the branch with removed changes), I'll do something more scientific from the terminal :)11:43
jamdanilos: probably try to find a revision with a particularly large number of changes11:46
jamlike when you merge into db-devel or whatever it is called now11:46
danilosjam: right, I'll do that, and use "ab" to get actual data11:46
danilosjam: fwiw, I got the following exception with trunk when benchmarking: https://pastebin.canonical.com/51341/11:53
jamdanilos: I've seen ScopeReplacer issues when running a lot of multithreaded requests as the "first" requests on a new loggerhead startup11:54
jamdanilos: so you can try starting loggerhead, making a single request11:54
jamthen running ab11:54
danilosjam: right, thanks11:54
jamScopeReplacer happens at 'import' times, and it is known not to be threadsafe11:54
danilosah, makes sense, should I file a bug about it somewhere?11:55
=== henninge is now known as henninge-lunch
LPCIBotProject devel build #979: FAILURE in 5 hr 23 min: https://lpci.wedontsleep.org/job/devel/979/12:13
jamdanilos: there is a bug already12:15
danilosjam: cool, I've added benchmarking results with a summary on https://code.launchpad.net/~danilo/loggerhead/bug-718566/+merge/7184912:16
danilosjam: tell me if you think I should land this considering the "mild" performance degradation12:16
jamdanilos: going from the Launchpad discussion. Which is "things MUST complete in <9s (or 5s, ymmv), and SHOULD complete in <1s". We are still well underneath satisfying the 5s rule, and even the 1s rule.12:18
jambtw, I wouldn't expect /changes to differ12:18
jamlet me grab a URL, just a sec12:18
danilosjam: sure, I wasn't sure if /changes used this cache as well12:19
jamit is the +revlog URLs that would be affected, I think12:20
jam /changes doesn't show you the file changes until you expand something12:20
danilosjam: ok, I'll test it directly as well12:24
jamdanilos: updated the mp with my comment12:24
jamI think I have a URL for you to try12:25
jam(posted to the mp)12:25
jtvcjwatson: see email.  All that needed immediate fixing then was that the "name" part of those tarball filenames is irrelevant, right?  I just fixed that.12:25
jtv(And cross-referenced to all those bugs I filed, *pant*)12:25
danilosjam: right, I was already benchmarking that URL12:25
jamdanilos: I'm expected that to regress approximately the same as /revision, which would still put it in the "fully acceptable" realm.12:26
danilosjam, that's right, 325ms vs 267ms for the trunk12:27
danilos(trunk with cache populated, already accessed the page from the browser first)12:27
jamdanilos: right. And ultimately a cache will always be faster, so what is "enough". And I think using SQUID to cache these pages is far superior to trying to do a file-change cache.12:28
jamyou might also try one more URL12:28
jamsomething with a small change12:28
jamwhich should be much less sensitive12:28
danilosjam: yep, I'll do that too12:29
rvbadanilos: Hi, can I add this MP to your queue? https://code.launchpad.net/~rvb/launchpad/desc-header-827893/+merge/7184212:45
danilosjam: anyway, smaller revision is actually faster in the few runs I did (259ms vs 264ms), so all good12:47
danilosrvba, a queue, there's a queue? :)12:47
jamdanilos: good deal12:47
rvbadanilos: :)12:47
cjwatsonjtv: yep12:48
cjwatsonjtv: thanks for the attention to detail :-)12:48
danilosrvba, nice, simple and clean, r=me (and thanks for the formatting fixes as well)12:49
rvbadanilos: Thanks!12:49
danilosjam: now, how do I go land loggerhead changes? just merge into trunk (I see nobody is using [r=...] tags)12:51
jamdanilos: yeah, merge it to trunk, comimt12:51
jamthen make a proposal against Launchpad to use the new loggerhead code.12:51
jamutilities/source_deps I believe12:51
StevenKjam: ^12:54
deryckMorning, all.13:04
StevenKstub: If you're still around, can you please re-rubber stamp https://code.launchpad.net/~stevenk/launchpad/denorm-bspph/+merge/71811 ? I've had to resubmit due to the addition of a pre-req branch.13:12
=== al-maisan is now known as almaisan-away
deryckabentley, let's chat about this shortly:  https://code.launchpad.net/~deryck/launchpad/bug-heat-wonky-recent-activity-check-688364/+merge/7175613:47
deryckabentley, and thanks! :)13:47
abentleyderyck: np13:47
=== henninge-lunch is now known as henninge
deryckabentley, I'm free now, if you have a minute to chat about that MP.14:04
deryckabentley, mumble 1 o 1?14:04
abentleyderyck: sure14:04
=== danilos changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 238 - 0:[#######=]:256
=== almaisan-away is now known as al-maisan
=== abentley changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: abentley | Critical bugs: 238 - 0:[#######=]:256
=== al-maisan is now known as almaisan-away
LPCIBotProject devel build #980: STILL FAILING in 2 hr 25 min: https://lpci.wedontsleep.org/job/devel/980/14:38
* barry sees lots of timeouts today14:54
rvbaderyck: Hi, do you have a few minutes for a small JS question?15:08
LPCIBotProject devel build #981: STILL FAILING in 31 min: https://lpci.wedontsleep.org/job/devel/981/15:10
barryhas anybody else noticed a high number of timeouts today?15:11
nigelbmrevell: are you free-er?15:11
benjiIs anyone else seeing "ImportError: No module named html5browser" on devel?15:12
benjithe last buildbot run that wasn't killed by a KeyboardInterrupt was clean and I don't see any bugs filed about it15:12
barryhere are two urls that are consistently timing out for me today, but were fine yesterday:15:13
_mup_Bug #807950: zeitgeist-daemon crashed with LookupError in remove_from_connection(): <_zeitgeist.engine.remote.RemoteInterface at /org/gnome/zeitgeist/log/activity at 0xb74ee2cc> is not exported at a location matching (None,None) <apport-crash> <bugpattern-needed> <i386> <oneiric> <running-unity> <unity-2d> <zeitgeist (Ubuntu):Triaged> <zeitgeist (Ubuntu Oneiric):Triaged> < https://launchpad.net/bugs/807950 >15:13
mrevellHi nigelb, sure. How can I help?15:13
nigelbmrevell: I'm looking to see if you have suggestions on how the subscription could be improved15:14
nigelbI found it non-intuitive15:14
deryckrvba, sure.  voice or chat?15:14
nigelbfixing it would be trivial, but the UX is probably of more concern here than the actual patch15:14
benjithe ImportError can be reproduced with "bin/test -c -t test_archivesubscribers_index"15:14
nigelb*comparitively trivial15:14
rvbaderyck: great, I think chat is ok ... I'll paste my understanding of the problem ..15:15
timrccalling addArchiveDependency() on PPA's (belonging to my LP user ~timrchavez) via the web service is resulting in OOPses in the web UI15:15
rvbaderyck: It's weird problem with FormOverlay. Basically, when the overlay is closed (for instance 'cancelled') it stays in the page (which is normal) but since it's only hidden using "visibility: hidden" it prevents me from clicking on the radio button that is underneath it.15:15
rvbaHere is the real example:15:15
rvbaIf you open up a row on this page https://dogfood.launchpad.net/ubuntu/oneiric/+localpackagediffs then changed the "Ignored" status using the radio buttons, you'll get an overlay to enter a comment (to describe *why* you changed the ignored status). Cancel the overlay and you will see that now you can't click on the radio buttons any more.15:15
rvbaI'm not the only one to have this problem it seems: http://stackoverflow.com/questions/1751861/do-controls-in-firefox-receive-mouse-events-when-their-css-visible-property-is-fa15:15
rvbaThis fixes it: http://paste.ubuntu.com/668072/15:16
rvbaderyck: But obviously I don't want to mess with that without the advice of a JS expert ;)15:16
mrevellnigelb, Subscription to what? Sorry, I've not really been following this channel much today.15:16
deryckrvba, reading back, thinking.....15:16
nigelbmrevell: Oh, I should have been clearer. I meant bug subscription, more specifically, subscribing  myself to a bug15:17
timrcHere is a script that should allow you to reproduce the problem (replace the PPA with a PPA you own): https://pastebin.canonical.com/51356/15:17
deryckrvba, yes, display none is what I would prefer.  I always wondered why we used visibility anyway.15:19
deryckrvba, I couldn't think of this exact issue, but I knew there was something with invisibility vs display like this.15:19
bigjoolstimrc: an OOPS code is more useful15:20
timrcbigjools, certainly, OOPS-2055AU6815:20
rvbaLooks much saner to me to use "display: none" too ... I'll do the change then ... I hope it won't break other overlays ;)15:21
bigjoolstimrc: FTR you can't add 2 dependencies on Ubuntu like that15:21
bigjoolsjust the latter one will do15:21
timrcbigjools, it was originally a test to verify that... however15:21
timrcbigjools, the first call to add the dep is what blows these ppas up15:21
rvbaderyck: thanks for looking into it.15:21
timrcbigjools, just did it again on a ppa owned by ~oem-archive user, OOPS-2055AR5615:21
bigjoolsI need to wait for the oops to sync15:22
deryckrvba, np15:22
timrcbigjools, our goal is to try to figure out how to get this option via the web service, http://i.imgur.com/ewHzM.png15:22
timrcbigjools, maybe you know off hand :)?15:22
timrcnot specifying any component gives us the second option15:23
bigjoolstimrc: it defaults to that, no?15:23
timrcbigjools, when we call addArchiveDepedency without a component specified, it defaults to "Use all the same components..." I believe15:23
bigjoolstimrc: don't call addArchiveDependency at all, it defaults to use everything15:24
timrcbigjools, but need to chang the Ubuntu Dependency...15:25
timrcbigjools, we need to set the equivalent of "Basic" from "Default" in the web ui15:25
bigjoolstimrc: ok your image paste didn't make that clear15:26
timrcbigjools, sorry :)15:27
bigjoolsnot sure you can change this over the API tbh15:27
timrcbigjools, would you consider this a bug then?15:28
bigjoolstimrc: let me investigate a little15:28
mrevellnigelb, We have a bug report ... bug 816105 ... I agree it's not easy to find the "subscribe me" option right now15:28
_mup_Bug #816105: UI for subscribing to a bug is confusing <bugs> <ui> <Launchpad itself:Triaged> < https://launchpad.net/bugs/816105 >15:28
mrevellnigelb, Was there something else you found non-intuitive?15:28
nigelbmrevell: the other bits are great. I'd love to fix the bug if there's a consensus on what needs to be done.15:29
mrevellnigelb, I'm about to go into another phone call ... I'd love to read your thoughts in a comment on that bug and then talk to you again later int he week, if that's okay.15:30
nigelbmrevell: that sounds good.15:30
mrevellgreat :)15:30
timrcTwo bugs, 1) using addArchiveDependency() is causing OOPses (which seems like a regression); and 2) Cannot express "Use all components availalbe" via the web service for addArchiveDependency()15:30
timrcbigjools, ^15:30
cody-somervillebigjools, https://lp-oops.canonical.com/?oopsid=OOPS-2055AZ5115:35
adeuringabentley: fancy a "double review"? https://code.launchpad.net/~adeuring/lazr.batchnavigator/lazr.batchnavigator-slicing-only-in-factory/+merge/71913 and https://code.launchpad.net/~adeuring/lazr.batchnavigator/lazr.batchnavigator-slicing-only-in-factory/+merge/7191315:44
abentleyadeuring: sure, I'll have a look.15:45
adeuringabentley: one link should be https://code.launchpad.net/~adeuring/launchpad/bug-739052-6/+merge/7191215:45
bigjoolstimrc: grar, ok please stop adding those, we need SQL to clean the DB15:50
bigjoolstimrc: can you get me a complete list of PPAs where you've done this and I will wipe all their dependencies15:50
timrcbigjools, lol, okay15:51
timrcbigjools, https://pastebin.canonical.com/51366/15:54
allenapmatsubara: Are you the person to talk to about qa-tagger?15:55
matsubaraallenap, I'd be happy to talk to you about it. Anyone in the maintenance teams is encouraged to fix bugs there too.15:58
bigjoolstimrc, cody-somerville: FYI: https://bugs.launchpad.net/launchpad/+bug/82813315:58
_mup_Bug #828133: Archive:+edit-dependencies OOPS <oops> <ppa> <Launchpad itself:Triaged> < https://launchpad.net/bugs/828133 >15:58
allenapmatsubara: Ah, okay. Well, I'm wondering if it has missed a bug. The branches in https://bugs.launchpad.net/launchpad/+bug/824499 seem to have landed in stable but qa-tagger has not updated the bug. I'm trying to figure out why.15:59
_mup_Bug #824499: "All derived distros" option for publish-ftpmaster <derivation> <Launchpad itself:In Progress by jtv> < https://launchpad.net/bugs/824499 >15:59
timrcbigjools, this bug was Fixed Release, so it seems to imply the interface should be usable for adding Ubuntu deps... https://bugs.launchpad.net/launchpad/+bug/776449, no?16:00
_mup_Bug #776449: Set Ubuntu dependencies for PPA via API <api> <bad-commit-13157> <escalated> <not-pie-critical> <oem-services> <ppa> <qa-ok> <Launchpad itself:Fix Released by abentley> < https://launchpad.net/bugs/776449 >16:00
allenapmatsubara: Ah, I think I've figured it out. The wrong bug number was used in the commit message.16:00
allenapSorry for the noise.16:01
bigjoolstimrc: I forgot about that16:01
bigjoolstimrc: I guess it has a bug16:01
timrcbigjools, I should note that this was working (at least setting the Dep) up until recently (not sure when it went bad)... what wasn't working was the ability to achieve the first Components option in the web ui from the web service16:02
matsubaranp allenap16:03
abentleyadeuring: r=me16:09
adeuringabentley: thanks!16:09
timrcbigjools, here's the other bug16:17
_mup_Bug #828145: Cannot add an archive dependency that uses all Ubuntu components available via the web service <oem-services> <Launchpad itself:New> < https://launchpad.net/bugs/828145 >16:17
bigjoolstimrc: I think you misunderstand16:18
* bigjools writes on bug16:19
timrcbigjools, probably :)16:19
timrcbigjools, all I know is addArchiveDependency() will change the what's displayed in that view16:19
timrcbigjools, so I expect to be able to do everything via the API that I can do in that view :)... is that too much to expect?16:19
bigjoolstimrc: yeah :)16:20
bigjoolsthe addArchiveDependency does not allow you to change the ubuntu dependencies16:20
bigjoolsjust add new ones16:20
timrcbigjools, when I create a PPA and use addArchiveDependency() to add an ubuntu dependency, it alters what's displayed in "Ubuntu dependencies"...16:21
bigjoolstimrc: you're trying to change the ubuntu dependency to just "main" aren't you?16:21
bigjoolstimrc: ah ok16:21
bigjoolsright, I remember how it works internally now16:21
timrcbigjools, here's our actual method... this will change the selection to the "Basic" Ubuntu dependency16:22
timrcbigjools, http://paste.ubuntu.com/668400/16:23
timrcbigjools, however, it assumes the wrong Components choice16:23
bigjoolstimrc: see my comments on the bug16:23
timrcbigjools, ah16:24
timrcbigjools, works like a charm16:26
timrccody-somerville, honestly a little surprised you didn't catch that... tsk16:26
mrevellHey, sinzui, have you seen bug 827902?16:27
_mup_Bug #827902: Private teams not able to subscribe to bugs or blueprints for email updates <disclosure> <privacy> <teams> <Launchpad itself:Triaged> < https://launchpad.net/bugs/827902 >16:27
bigjoolsabentley: you know you did that change so that PPA dependencies can be set on the API?  Can you have a look at this bug that it's causing:  https://bugs.launchpad.net/launchpad/+bug/82813316:38
_mup_Bug #828133: Archive:+edit-dependencies OOPS <oem-services> <oops> <ppa> <Launchpad itself:Triaged> < https://launchpad.net/bugs/828133 >16:38
abentleybigjools: I don't think I have any special insight.16:41
bigjoolsabentley: it's extremely weird16:41
bigjoolsthe data looks sane16:42
abentleybigjools: It looks like a lp.soyuz.model.component.Component is the input to the getTerm.  Is that correct?16:45
bigjoolsabentley: should be, it's worked for ages with no changes16:46
=== beuno is now known as beuno-lunch
bigjoolsshame Component doesn't have a __repr__ :/16:49
bigjoolsoh it does16:50
bigjoolsso it doesn't get used when the security proxy is in the way16:50
cody-somervilletimrc, catch what?16:55
bigjoolsabentley: I see what's wrong16:56
sinzuimrevell, I have. I added the disclosure tag16:56
bigjoolsabentley: the API allows you to add data that the form cannot cope with16:56
bigjoolstimrc: set that component to "multiverse" and it'll be OK.16:58
timrccody-somerville, that by specifying 'multiverse' as the component, for example, it pulls in 'main', 'restricted', and 'universe' as well16:58
bigjoolsthe form code for the page only knows about multiverse and None :)16:58
=== almaisan-away is now known as al-maisan
=== deryck is now known as deryck[lunch]
=== beuno-lunch is now known as beuno
=== deryck[lunch] is now known as deryck
abentleyderyck: Why aren't comments 8 and 9 showing? https://bugs.qastaging.launchpad.net/launchpad/+bug/100018:28
_mup_Bug #1000: There are too many bug reports in Malone <lp-foundations> <Launchpad itself:Invalid> <Ubuntu:Invalid> < https://launchpad.net/bugs/1000 >18:28
* deryck looks18:29
abentleyderyck: These are comments I created using the 'macintosh' charset, but I wouldn't expect that to hide them...18:29
deryckabentley, hmmm, I have no idea.  I wouldn't expect them to be hidden either.18:30
deryckabentley, I can get to them directly, too.  Weird.18:31
=== al-maisan is now known as almaisan-away
abentleyderyck: They are missing the Subject line, unlike 7 which was the same except it used the 'mac-roman' charset.18:32
deryckabentley, ah, good catch.  curious that a missing subject line on an email generated comment would cause the comment to hide.18:33
abentleyderyck: The Subject was present on the email, though.18:34
deryckabentley, right, I followed.  I assumed we tripped over it somehow and didn't store it in the db.18:35
abentleyderyck: Yeah, I suspect.18:36
deryckabentley, seems like another bug that the comment won't display without a subject.18:36
abentleyderyck: I doubt it's just any missing subject.18:37
abentleyderyck: I bet it's related to the charset.18:37
abentleyderyck: I've sent a latin-1 email with absolutely no subject, to see what happens.18:39
deryckabentley, good idea18:39
abentleyderyck: subject-free comment is not hidden.19:05
deryckabentley, ok, good to know.  Thanks for confirming.19:07
abentleyderyck: I am marking bug 820039 qa-ok, and filing a new critical bug for "bug comments with macintosh encoding not displayed".19:08
_mup_Bug #820039: process-mail.py fails with a LookupError: unknown encoding: macintosh error <mail> <oops> <qa-needstesting> <Launchpad itself:Fix Committed by abentley> < https://launchpad.net/bugs/820039 >19:08
deryckabentley, sounds good.19:08
benjiabentley: do you have a minute for a review? https://code.launchpad.net/~benji/launchpad/bug-162868-2/+merge/7194220:01
abentleybenji: sure.20:01
benjiabentley: here's the soundtrack: http://www.youtube.com/watch?v=0O2aH4XLbto&feature=channel_video_title20:01
abentleybenji: The change looks reasonable.  The docstring on Macro is sort-of obsoleted by the branch.20:08
abentleybenji: it talks about the present state, rather than the state we'll have once the branch lands.20:08
benjiabentley: hmm, you might be right; I'll take a second look20:08
abentleybenji: e.g. "that pattern also has the side effect of making the template URL traversable from any object" could be "without this class, that pattern would make the template URL traversable..."20:09
abentleybenji: r=me.20:11
benjiabentley: sounds good, I'll change that;  I'd like to keep the docstring though, I like that it provides both a justification for why the class exists and describes how it should be used.  I often wish someone had written something like that when reading code.20:11
abentleybenji: yes, I agree.20:11
abentleybenji: You might want to update this page: https://dev.launchpad.net/Web/TemplateCodeReuse20:13
benjiabentley: ooh, thanks20:14
sinzuijcsackett, mumble?20:45
jcsackettsinzui: sure, one moment.20:46
=== gary-lunch is now known as gary_poster
lifelessallenap: hi21:11
lifelessallenap: I've commented on the review; I have to run now to a hospital visit; I hope I haven't been too brief with my description.21:14
lifelessallenap: if I have, please ask here or there and we can refine our understanding of whats going on21:14
lifelessok, popping out to my monthly allergy injection, at l'hopital, back in ~ 3hr21:35
=== almaisan-away is now known as al-maisan
=== al-maisan is now known as almaisan-away

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