/srv/irclogs.ubuntu.com/2012/05/28/#launchpad-dev.txt

wallyworld_StevenK: remind me - are we eventually going to allow info type -> Proprietary via the API or will that always be verboten?00:27
StevenKwallyworld_: We will allow it under certain conditions.00:28
wallyworld_same conditions as for when using the web ui no doubt00:28
StevenKIt's just denied for all with a strange error message because we don't want to deal with Proprietary bugs yet.00:28
wallyworld_yep, just wanted to double check, thanks00:28
StevenKwallyworld_: I refuse to fix 1005325. We decided we didn't want to support Private over the API when I exported transitionToInformationType()00:32
wallyworld_StevenK: i was thinking more that the error message is confusing since we don't support user data or proprierary yet00:34
wallyworld_but i guess we can ignore that for now00:34
StevenKwallyworld_: We do support User Data and Proprietary via the API. User Data will work, and show up as Private in the UI, and Proprietary will throw the terse error.00:36
wallyworld_StevenK: i also was thinking that Private was to be supported as a pseudonym for User Data but as you remind me, not over the api00:37
wgrantRight, it's just to avoid user confusion in the UI for now00:37
wallyworld_ok. i'll mark the bug as invalid00:37
wgrantAPI users already have to use tasks etc.00:37
wgrantThey can deal with a terminology difference, or continue using flags as they do already.00:38
wgrantwallyworld_: I wonder if the trailing underline is related to the odd display issues in Opera00:39
wgrantA box appears after the picker activators, like it's trying to render the nbsp as a character00:39
wgrantIt's very odd.00:39
wallyworld_wgrant: not sure. could be by the sounds of it00:39
wallyworld_worth fixing i think00:40
wgrantThe tree used there is a bit interesting.00:41
wgrantDue to some old problems with KHTML and other browsers not showing them at all if the <a> didn't have some content.00:41
wgrantAnd stuff like that.00:41
=== Ursinha is now known as Ursinha-zz
=== Ursinha-zz is now known as Ursinha-zzz
wgrantlifeless: Are you sure the avahi instructions work?01:20
wgrantlifeless: I'm pretty sure it'll only work if you're lucky and the container's avahi user has a UID not matching anything running any other processes on the system.01:20
wgrantlifeless: Because the avahi initscripts set a user-wide limit of ~2 processes01:21
wgrant(it certainly didn't work two months ago if you tried to run avahi in two containers where it had the same UID, or if the UID matched the host's)01:22
lifelessthe old instructions would work either01:22
lifelessthey assume persistent ip, which is in no way guaranteed01:22
lifelessbah, english robert, english01:23
wgrantIn practice they're reasonably persistent.01:23
wgrantAnd far better than avahi which in a lot of cases won't work at all01:23
wgrantAnd when it does it's inconsistent at best.01:23
lifelessdid you record what you dug up ?01:24
wgranthttp://siphon9.net/loune/2011/02/avahi-setrlimit-nproc-and-lxc/01:24
lifelessdid you file a bug?01:25
lifelessspecifically, the uid counting across boundaries is unsafe - avahi can break e.g. postgresql in a container, that way.01:26
wgrantRight, but user namespaces will make all this go away :)01:27
wgrantAnd they're meant to be happening for quantal, AIUI01:27
lifelessindeed01:27
wgrantSo it's really not an avahi bug01:28
wgrantThe nproc limit is probably a reasonable measure.01:28
wgrantIt's just the lack of user namespacing that's breaking everything.01:28
lifelessyes01:29
wgrantlifeless: Bug #1005330 -- any objections/opinions before I implement it in 4 lines?01:29
_mup_Bug #1005330: Applications can't easily map an SSO account to a Launchpad one <api> <easy> <openid> <Launchpad itself:In Progress by wgrant> < https://launchpad.net/bugs/1005330 >01:29
lifelessI dispute 'most correct way'01:31
wgrantlifeless: 'most reliable and only vaguely official way'01:31
lifelessmulti-openid support, let alone browserid, is inherently hostile to using the ubuntu sso response at all :)01:31
wgrantSure01:32
wgrantBut there's currently no other reliable way :)01:32
wgrantStevenK: Are you QAing the information_type thingy?01:35
StevenKwgrant: I've just finished off the SPPH+PU QA, so that is up next, yes.01:35
lifelessindeed01:36
lifelessuhm01:36
wgrantStevenK: Great.01:37
wgrantlifeless: Yeah, anonymous is fine.01:39
wgrantIt'll crash and burn if it returns a team, since that's meant to be impossible.01:39
wgrantNo private people01:39
wgrantAnd identifiers aren't secret01:39
wgrantSo there's no problem01:39
wgrantlifeless: Is that your uhm?01:41
wgrantOr is there something else?01:41
lifelessit would be nice to ensure the team thing specifically01:41
lifelesse.g. via a test or something01:41
wgrantMaybe01:42
wgrantI guess we can add a test now, since the DB doesn't forbid it01:42
wgrantWill be a bit awful, but is easy01:42
bigjoolsStevenK: now that you're setting pu on spph, there's probably a load of locations in the code that can be optimised to use it instead of joining/querying01:49
StevenKbigjools: I should probably circle back and populate it for older SPPHs too01:50
wgranteg. changesfile01:50
bigjoolsStevenK: yes, hell yes01:50
wgrantWhich currently is a hideous inaccurate query to guess the PU01:50
StevenKbigjools: You say after objecting to it in the first place :-)01:50
StevenKHmmm, where is bazaar.qastaging.launchpad.net01:51
bigjoolsStevenK: I still object01:51
bigjoolsbut since you've done it you can make it useful01:51
StevenKbigjools: Right, fair enough01:51
bigjoolsStevenK: another lovely garbo job? :)01:52
StevenKYup01:52
StevenKWhich will end being my fifth for the year or something01:52
bigjoolswe need a bit of code that rejects bugs with the word "should" in the title02:07
lifeless:)02:09
* dobey files a bug that launchpad "shouldn't" do that :)02:51
StevenKwallyworld_: Can you delete the card for 1005325 too?03:04
wallyworld_yep03:04
wgrantlifeless: https://lp-oops.canonical.com/oops.py/?oopsid=OOPS-b903a8b9cb0e54ccb8b4660ed77bb0f003:33
wgrantlifeless: Several implausibly long SQL statements on the same store on xmlrpc-private -- more evidence that we still have GIL issues with them03:33
lifelesswgrant: we could partition the farms, or we could kill the xmlrpc internal socket03:37
lifelesswgrant: (e.g. moving into the main socket and vhosting it up)03:38
wgrantlifeless: Yeah, I think the latter is probably best, but that gets interesting.03:38
lifelessor nuke all the internal users03:39
=== Ursinha-zzz is now known as Ursinha
StevenKwgrant: https://code.launchpad.net/~stevenk/launchpad/set-spph-packageupload/+merge/10758306:32
wgrantStevenK: That's incorrect.06:33
StevenKwgrant: Which part?06:33
wgrantStevenK: That finds any PU that references the source06:33
wgrantStevenK: That should, in fact, crash sometimes06:33
wgrantBecause delayed coppies06:33
wgrantFor a delayed copy there will be multiple PUSes for the SPR06:34
wgrantThere's no foolproof way to do this.06:34
StevenKBleh06:34
wgrantYou sort of have to match the archive, suite and overrides.06:34
wgrantSo if (SPPH.archive, SPPH.distroseries, SPPH.pocket) match the PU, and (SPPH.component, SPPH.section) match the SPR, it's probably the one you want.06:35
wgrantThat will get the real uploads.06:35
wgrantWon't handle PCJs or delayed copies -- they're probably less feasible and useful to populate, though06:35
StevenKwgrant: http://pastebin.ubuntu.com/1010807/ with appropiate tests?06:40
wgrantYour indentation is criminal again.06:40
wgrantThat's roughly correct, but it relies on my reasoning being flawless, which from 30s of thought it probably isn't.06:41
* StevenK picks on DF06:45
StevenKwgrant: SELECT spph.id AS spph, pu.id AS pu FROM PackageUpload AS pu, SourcePackagePublishingHistory AS spph, SourcePackageRelease AS spr, PackageUploadSource AS pus WHERE spph.sourcepackagerelease = pus.sourcepackagerelease AND spph.sourcepackagerelease = spr.id AND pus.packageupload = pu.id AND pu.archive = spph.archive AND pu.distroseries = spph.distroseries AND pu.pocket = spph.pocket AND spph.component = spr.component AND spph.section =06:50
StevenKwallyworld_: ^ And you *want* to learn Soyuz? :-)06:51
wallyworld_not unless there's lots of beer involved06:52
bigjoolsah the soyuz query06:52
StevenKwallyworld_: There has to be, one way or the other.06:54
wallyworld_yep, either at the start or at the end :-)06:54
StevenKBoth has benefits, depending on which area of Soyuz06:55
czajkowskimorning06:55
bigjoolsthere has been a lot of beer, caffeine and pizza involved in writing soyuz06:56
StevenKYou forgot swearing06:56
StevenKAnd Portuglish06:56
bigjoolsand combinations thereof06:59
StevenKwgrant: http://pastebin.ubuntu.com/1010839/ modulo working and fixing tests07:24
wgrantStevenK: That's probably OK.07:26
=== jml` is now known as jml
=== bac changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
=== benji changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: benji | Firefighting: - | Critical bugs: 3.47*10^2
=== salgado is now known as salgado-brb
mgzis there a bug entry for +bug/NNNN/+thing pages not having cross project redirects like +bug/NNNN pages do?14:52
mgzcan't find one with search14:52
mgzexample case:14:52
mgz* user reports bug14:53
mgz* triager changes bug project14:53
mgz* user tries to add comment, gets 40414:53
cjwatsonI think my brain is atrophying; I just typoed getUniqueString as makeDistroSeries14:55
mgzwell, they have a similar rythym14:56
jmlthose keys are right next to each other!14:57
mgz...and that was a creative misspelling14:58
cjwatsonI think what has happened is that I've internalised about half of LaunchpadObjectFactory and my hindbrain just produces a random method from it on request15:00
james_wbenji, thanks for the review of https://code.launchpad.net/~james-w/launchpad/suppress-notifications-for-all/+merge/107560 it seems that most of the changes you suggest were fixed in trunk in the meantime. I've made the one leftover change now.18:45
james_wbenji, would you land that branch for me please?18:45
benjijames_w: sure thing18:46
james_wbenji, thanks18:46
james_wbenji, not taking the day off today?18:46
benjijames_w: I swapped it.18:46
james_wah, ok18:46
benjiI did all my memorializing last week.18:47
lifelessbac: around ?18:51
bachi lifeless18:51
lifelessbac: I want to suggest you don't land your branch18:52
lifelessthe one with test ids being allowed to be more duplicative18:52
bacwhich one?18:52
lifelessbug 682771 and bug 68277218:52
_mup_Bug #682771: test runner permits duplicate test ids <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/682771 >18:52
_mup_Bug #682772: doctest construction generates duplicate test ids <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/682772 >18:52
lifelessI had forgotten about these bugs, but they cover the situation18:52
* bac looks18:52
bachmm18:53
baclifeless: so doctest constructors like create_interface_test_suite would need to change18:56
* bac intercepts ec218:56
lifelessbac: testtools.clone_test_with_new_id can be used to assign a unique id18:57
lifelessbac: create the test (duplicative id); then clone it to a new id, and add the resulting new-id test into the suite, let the original get gc'd.18:58
baclifeless: are you suggesting doing that in filter_tests or in the various test_doc files?18:59
lifelessbac: I was assuming local-to-the-duplicativeness19:00
lifelessbac: with a sanity-check in the global test_suite() constructor to enforce uniqueness19:00
baclifeless: that's what i thought but i read your statement the other way.19:00
lifelessI think bug 682771 will be fixed if the e.g. ./bin/test --list-tests errors when the are duplicates, and bug 682772 will be fixed when the current duplicates are fixed up19:02
_mup_Bug #682771: test runner permits duplicate test ids <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/682771 >19:02
_mup_Bug #682772: doctest construction generates duplicate test ids <lp-foundations> <Launchpad itself:Triaged> < https://launchpad.net/bugs/682772 >19:02
lifelessyou could do just 682772, but doing 682771 at the same time would give confidence that nothing has cracked through the slips19:02
lifelessjelmer: ping (subunit)19:07
czajkowskilifeless: he;s offline today on hols but has been peeping into irc19:10
lifelessczajkowski: thanks19:11
czajkowskilifeless:do you have a few mins spare so I can check stuff wiht you if you don't mind?19:21
jelmerlifeless: pong19:21
jelmerlifeless: subunit is building in the daily PPA these days19:22
lifelessjelmer: cool - and the thing gary saw, where the built binaries use python3, but the python3 module isn't installed20:01
lifelessczajkowski: sure, shoot20:01
czajkowskilifeless: doing the oops summaries as digo is gone and just a bit um confused.20:04
czajkowskihttps://devpad.canonical.com/~lpqateam/oops-summaries/production-2012-05-26.html20:04
czajkowskilooking at PortConnectionError: DTPFactory timeout  as an example20:04
lifelessgo on20:06
czajkowskisp when I search for bugs with that title they don't exist, so I checked the oops and they aren't reported20:06
czajkowskiso do I created the bug based on that summary20:06
czajkowskisorry laggy this evening I suspect housemate is downloading20:07
lifelesswhat do you mean 'checked the oops and they aren't reported'20:08
czajkowskiwell the way it was explained was I search for the bug summary and then compare the oops that have been reported20:08
jelmerlifeless: I hadn't seen that one, looking at it now...20:09
czajkowskiif they're not report and link the oops to the bug20:09
lifelessczajkowski: ok, so yes you search, if there is no open bug, then file one; if there is an open bug, don't bother linking the new oopses in unless the bug hasn't been touched in months - that would just be make-work.20:12
lifelessczajkowski: the reports can be dug back through by developers, theres no need to keep touching the bugs20:12
czajkowskilifeless: ah ok that wasn't explained20:12
czajkowskigood to know20:12
czajkowskiand thank you20:12
lifelessalso I wouldn't worry about being exhaustive20:13
lifelessget the top 4-5 in timeouts, 4-5 in exceptions20:13
lifelessby frequency20:13
lifelessthats enough for folk to be getting on with.20:13
jelmerlifeless: hmm, it looks like it defaults to python3 just on ubuntu and to python2 on debian20:13
jelmerlifeless: I'll dig deeper tomorrow20:13
czajkowskilifeless: cheers much appreciated20:13
lifelessjelmer: yah, so this is the thing gary was complaining about :)20:13
lifelessjelmer: thanks!20:14
jelmerlifeless: IIRC he was complaining about it not building, which is fixed20:14
jelmerI'm now looking at bug 100391020:14
_mup_Bug #1003910: subunit2junitxml failed on quantal: ImportError: No module named subunit.filters <amd64> <apport-bug> <qa-daily-testing> <quantal> <rls-mgr-p-tracking> <running-unity> <subunit (Ubuntu):Triaged> < https://launchpad.net/bugs/1003910 >20:14
lifelessjelmer: bug 98793820:16
_mup_Bug #987938: subunit trunk packaging breaks subunit-* commands <subunit:In Progress by jelmer> < https://launchpad.net/bugs/987938 >20:16
lifeless$ subunit-filter20:16
lifelessTraceback (most recent call last):20:16
lifeless  File "/usr/bin/subunit-filter", line 33, in <module>20:16
lifeless    from subunit import (20:16
lifelessImportError: No module named subunit20:16
=== benji changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 3.47*10^2
cjwatsonI'm thoroughly lost.  Is there any documentation anywhere on how to debug the case where you've done export_as_webservice_entry on a class but it's stubbornly refusing to show up in the generated WADL?22:10
lifelessyou exported it for devel, and are checking devel ?22:21
cjwatsonYes22:22
cjwatsonAt least I think so otherwise I'm going to feel silly, but I've ntuple-checked22:22
cjwatsonlifeless: e.g. http://paste.ubuntu.com/1012244/22:25
cjwatson(as a reduced experiment)22:25
lifelesscjwatson: hmm, I'm not sure, its something i have diligently stayed on only passing terms with22:28
cjwatsonOh, yeesh, I just found lib/lp/soyuz/interfaces/webservice.py22:31
cjwatsonDRY much22:31
cjwatsonAdding the new interface to the obvious two places there fixed it22:33
lifelessdo you mean ~DRY ?22:35
cjwatsonI meant sarcasm22:35
lifeless:P22:36
cjwatson(The right answer is probably in fact not to export that interface, but it's nice to know how to do it)22:37
lifelessmeep23:27
lifelesshttp://pam-face-authentication.org/23:27
mwhudson_cjwatson: if you want to get really depressed about the api, there's always https://bugs.launchpad.net/launchpad/+bug/95136523:28
StevenKI have fear.23:28
_mup_Bug #951365: Accessing milestone on project group via webservice api returns has_bugs object <api> <milestones> <oem-services> <projectgroups> <regression> <Launchpad itself:Triaged> < https://launchpad.net/bugs/951365 >23:28
cjwatsonmwhudson_: nah, I'm trying to extend it, not get depressed about it :)23:33
lifelesssigh firefox, please become an OS23:43
lifelessthe you might not use a full core when in the background23:43
bigjoolsit has to read email before it can become an OS23:47

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