/srv/irclogs.ubuntu.com/2012/07/18/#launchpad-dev.txt

StevenKwgrant: I would have expected http://pastebin.ubuntu.com/1097605/ to fail on the last assert.00:54
wgrantStevenK: Hm, it may just happen to work atm.01:12
wgrantBut we still need the proper fix so that someone can always recover access and we can remove the default sub.01:12
StevenKwgrant: I was going to add an null APG for USERDATA on private team creation01:13
wgrantUSERDATA on what?01:14
wgrantWe should have (pillar, information type) and (team,) APs, I suspect01:14
wgrantI don't see a need for an information type for private team APs01:15
StevenKwgrant: APs don't support teams01:16
StevenKThat I can see01:16
wgrantStevenK: That's correct.01:17
wgrantBut there's no pillar involved01:18
wgrantSo they have to be made to support teams.01:18
wgrantCurrently APs have a key of (pillar, information_type), but they need to also support (team,)01:18
wgrantPrivate junk branches are probably only allowed for private teams, and they get an AP of (team,) regardless of their information_type.01:19
wgrantThe AP is immutable and has a single APG: the team itself.01:19
=== nigelb_ is now known as nigelb
* StevenK glares at wallyworld_, and picks lp.code.branchmergeproposal.nominate apart02:14
wallyworld_StevenK: oh no, what have i done this time?02:15
StevenKwallyworld_: Written lies which weren't lies when you wrote them.02:15
wallyworld_oh, is that all :-)02:20
StevenKwallyworld_: Actually, why is there a confirm_reviewer and a confirm_reviewer_nomination function?02:22
wallyworld_StevenK: no idea, let me read the code02:22
wallyworld_StevenK: one happens client side with known information when the mp page was loaded, the other as a result of mp submission02:25
wallyworld_since the user could enter info that invalidates the initial check02:26
wallyworld_StevenK: confirm_reviewer_nomination is a popup when submit is clicked02:29
wallyworld_the other is a picker validation plugin02:29
wallyworld_like is used for assigning a new person to a bug02:29
wallyworld_it happens inside the picker when a reviewer is chosen02:29
=== Pendulum_ is now known as Pendulum
=== gmb` is now known as gmb
=== matsubara-afk is now known as matsubara
=== frankban changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: frankban* (rvba) | Firefighting: - | Critical bugs: 4.0*10^2
czajkowskicjwatson: do you need me to do some sort of annoucment for you ?09:00
cjwatsonczajkowski: oh, right, let me find a TL to rubber-stamp^Wapprove my plan09:07
jelmerczajkowski: unfortunately we can't import import nburrus/rgbdemo since it contains submodules09:07
czajkowskicjwatson: heading to stand up now09:07
jelmer(also, good morning :)09:08
cjwatsonHm, I think I need to wait for the US to wake up09:12
cjwatsonczajkowski: I was thinking of a post on the Launchpad blog something like this: http://paste.ubuntu.com/1097990/.  It could use tightening up of language, moving the image to a more sensible/permanent place, and so on.09:35
cjwatson(And this is as yet unapproved so don't post it yet :-) )09:36
czajkowskicjwatson: hehe ok, well I can review it later with mrevell on our 1-2-1 if you want09:36
czajkowskior do you need higher beings to ack it :)09:36
cjwatsonbigjools was cautiously optimistic about the general plan but didn't feel plugged in enough to current Launchpad to ack the feature flag changes, so I'll need to grab a team lead for that.  He suggested announcing the changes.  Reviewing the text with mrevell would be good.09:38
cjwatsonI went into a fair amount of detail as I thought this might be a little-known area that could benefit from a bit of light shone on it, but maybe I overdid it :)09:39
czajkowskicjwatson: I just want to highlight other people are helping getting launchpad updated in areas to encourage others to do so :)09:40
czajkowskicjwatson: jam is away this week so poke gary_poster or deryck09:40
cjwatsonRight.  Like I say, waiting for the US :)09:42
czajkowskiok :)09:43
=== matsubara is now known as matsubara-afk
gary_postercjwatson, I'm here early because I very much need to do some more prep work for some interviews that start in 1:19.  So, I've read your proposed blog post (sounds good) and I can spend another 3 or 4 minutes thinking about it.  deryck will be around in a while.  I won't be available again for five or six hours.11:42
gary_posterSo, can I do anything else for you?11:42
rick_h_gary_poster: tell Jim I said hi, you've got him in a bit of awe :)11:47
gary_posterrick_h_, lol ok11:47
cjwatsongary_poster: I may have missed you.  If not, the main thing I would like is approval to make a couple of feature flag changes: soyuz.copypackageppa.enabled on for everyone (AIUI this was disabled because of the lack of failed copy notifications, which is now fixed) and soyuz.derived_series.max_synchronous_syncs to 0 for launchpad-beta-testers so that we can see if async copies work for them11:53
cjwatsonsoyuz.derived_series.max_synchronous_syncs is currently set to something implausibly enormous in order to effectively disable asynchronous copies.11:53
cjwatsonBut I've been working on that area and they're working fine for me on dogfood.11:53
cjwatsonSo I think the next stage is to see if they work for other people.  The ~libreoffice people ask me for manual copies roughly once a week at the moment so I expect they'll provide a test case soon, if nobody else does.11:54
cjwatsonsoyuz.copypackageppa.enabled will also permit the Archive.copyPackage API method to work when the target is a PPA.11:55
cjwatsonUnrelatedly, what's the best Stormy SQLish way of saying "if there is a row matching condition A, return it; otherwise, if there is a row matching condition B, return it"?11:58
cjwatsonCoalesce looks sort of right but I'm not sure.11:59
wgrantcjwatson: What are you trying to do?12:01
wgrantCOALESCE isn't for whole rows, usually12:01
cjwatsonI'm trying to do a version of the target => bug task search in Bug.setStatus that works for multiple bugs.12:02
cjwatsonCurrently it's doing several consecutive queries in Python.12:03
cjwatsonWell, depending on the type of the target.12:03
cjwatson(I know this is only a tiny bit of the close_bugs_for_sourcepackagerelease problem, but I'm working myself up to it.)12:04
wgrantHmmm12:04
cjwatsonSo, say, for an IProductSeries target that method needs to find a product series task if there is one and otherwise fall back to a product task if there is one.12:05
wgrantI'm not sure it's massively worth doing that, and it's going to be a bit of a challenge.12:05
wgrantNot worth it because the ObjectModifiedEvent is going to cause O(tasks) write queries anyway12:05
wgrantIt's not really going to be practical to do all the task lookups in one query. I'd divide them up by type, doing the fallback in secondary queries as it does now12:08
wgrantBut batched12:08
wgrantSo look up tasks for all the targets. If there are productserieses left over, look up tasks for their products. If there are sourcepackages left over, look up tasks for their DSPs.12:09
cjwatsonWhy batched (within each lookup, I assume you mean)?  I was kind of hoping it would be possible to get the query count low enough that it wouldn't be necessary12:09
cjwatsonI guess the number of bugs is technically unbounded12:10
wgrantRight12:11
wgrantThough12:11
wgrantThe task lookup in setStatus right now should be one query12:11
wgrantAnd only the first the first call on a bug12:11
wgrantSubsequent calls should be 0 queries12:11
cjwatsonOh, because the first .target fetches all the columns?12:12
cjwatsonMaybe I should just bulk-load the bugs then.12:12
wgrantAh, yeah, it uses target, so it resolves the whole thing. The .target call will be up to 2 queries for every task on the bug, but still only the first time.12:12
wgrantHowever12:13
wgrantThat getBugTask thing is insane.12:13
wgrantWe have perfectly good ways to filter tasks by target12:13
wgrantWithout making hundreds of queries12:13
cjwatsonMaybe I'm just being stupid but I couldn't find an obvious one that quite fit.12:14
wgrantcjwatson: You can use a normal bug search.12:17
wgrantcjwatson: See eg. Bug.userCanView12:17
wgrantparams = BugTaskSearchParams(bug=self.id)12:18
wgrantparams.setTarget(sometarget)12:18
wgrantgetUtility(IBugTaskSet).search(params).one()12:18
wgrantor so12:18
wgrantYou'll probably want _noprejoins=True to avoid issuing more than one query12:20
cjwatsonBit of a problem with privacy there.12:26
wgrantcjwatson: It'll default to not filter by privacy at all12:27
wgrantUm12:27
wgrantI lie12:28
cjwatsonIt defaults to public bugs only12:28
wgrantIndeed12:28
wgrantcjwatson: That's really the ideal way to do it, but the mandatory privacy filter is a bit annoying.12:30
cjwatsonThere's the evil circumvention of finding an admin user and passing that in ...12:31
wgrantYeah, but no.12:31
cjwatsonQuite.12:31
cjwatsondisable_privacy=True I guess12:31
cjwatsonWell, privacy=False12:31
wgrantSomething like that.12:32
wgrantfilter_private=True maybe12:32
wgrantProbably better to fix task search than use bug_target_to_key manually, anyway.12:32
deryckadeuring, https://plus.google.com/hangouts/_/84943eb8075e033904f027277d6b4e951afc2038?authuser=0&hl=en13:32
=== jcsackett_ is now known as jcsackett
cjwatsonfrankban: Could I have a review of https://code.launchpad.net/~cjwatson/launchpad/garbo-archivepermission-duplicates/+merge/115554 ?14:52
frankbansure cjwatson14:52
cjwatsonI've checked that SQL fragment by hand on dogfood.14:52
cjwatson(Just as well since I'd never have got it right non-experimentally.)14:53
rick_h_frankban: review your way if you get a sec https://code.launchpad.net/~rharding/launchpad/reportbug/+merge/11556215:43
=== matsubara-afk is now known as matsubara
rick_h_frankban: added a link to the first branch for some context15:44
frankbanrick_h_: on it15:44
rick_h_ty much15:45
=== salgado is now known as salgado-lunch
=== Ursinha` is now known as Ursinha
=== frankban changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: - | Firefighting: - | Critical bugs: 4.0*10^2
=== abentley changed the topic of #launchpad-dev to: http://dev.launchpad.net/ | On call reviewer: abentley | Firefighting: - | Critical bugs: 4.0*10^2
abentleyderyck: I'm the OCR.  Could you please review https://code.launchpad.net/~abentley/launchpad/broken-xmlrpc-lookup/+merge/115582 ?16:12
deryckabentley, sure.16:16
abentleyderyck: Thanks.16:16
deryckabentley, np16:17
cjwatsonderyck: Would you mind having a look at my comments to gary_poster at http://irclogs.ubuntu.com/2012/07/18/%23launchpad-dev.html#t11:53 (and, for background, my conversation with bigjools at http://irclogs.ubuntu.com/2012/07/17/%23launchpad-dev.html#t23:31)?  I'm looking for a TL to approve a couple of feature flag changes there.16:21
deryckcjwatson, ah, sure.  Give me just a few minutes and I can look into all that.16:24
rvbarick_h_: hey, I can't say I understand every detail of it and I see this is fixing a critical bug so I don't want to hold you too long but I'm wondering about the lines 119-120 of the diff for the branch ~rharding/launchpad/reportbug.  I'm wondering how that doesn't imply a change in the related test code...?16:33
rick_h_rvba: let me load it back up sec16:39
rick_h_rvba: no, the sharee test code didn't test the navigation html at all relying on the underlying code to test/do it16:39
rick_h_rvba: thought about adding a test, but wasn't sure if it was worth the LoC to test what's tested already.16:40
rvbarick_h_: all right, I guess that's ok if the code gets run by the tests somehow.  I was just checking.  Thanks for the explanation.16:41
rick_h_np, I thought the same thing tbh so understand the concern16:41
rvbaCool.16:42
deryckabentley, looks good, r=me16:43
abentleyderyck: ty16:44
derycknp16:45
deryckcjwatson, in terms of the flags, you have my approval to change them.16:46
cjwatsonderyck: Thanks.  I'll get that onto LPS shortly.17:03
deryckcjwatson, np!17:06
* deryck reboots, brb17:06
=== deryck is now known as deryck[lunch]
=== salgado-lunch is now known as salgado
czajkowskicjwatson: do you want it added to the blog then?17:43
cjwatsonczajkowski: Yeah, was just waiting for the FF change to happen before asking you.  http://paste.ubuntu.com/1098642/ is slightly updated text; is there somewhere more permanent you could put the image there?17:45
cjwatsonczajkowski: I've also logged into wp-admin, and mrevell says you can post in my name once I've done that17:45
czajkowskihe briefly explained this to me today17:46
czajkowskineed to find you on the list17:46
czajkowskicjwatson: set to authour so lets see how this work17:47
czajkowskiwill fix the image up once it's in the blog17:48
rick_h_abentley: so care to be the 3rd person to review? https://code.launchpad.net/~rharding/launchpad/yuiv3/+merge/11559217:52
rick_h_abentley: will not land until after veried everyone has the updated launchpad-deps package17:52
cjwatsonczajkowski: Good to go17:54
czajkowskicjwatson: hmm I don't see it17:55
czajkowskiwill have a munch of dinner and investigate17:56
cjwatsonczajkowski: See what?17:56
czajkowskicjwatson: ah thought you'd added it to the blog and in draft17:56
cjwatsonOh, do you mean you gave me permissions?17:56
czajkowskiwill sort it out after munchables17:56
cjwatsonI misunderstood.  I'll write up a draft.17:57
abentleyrick_h_: okay18:02
cjwatsonczajkowski: http://blog.launchpad.net/?p=3627&preview=true18:07
abentleyrick_h_: 2083 is apparently the max URL length in ie8.18:10
rick_h_71ah ok will update that.18:10
czajkowskicjwatson: lovely18:12
czajkowskiposting to the places now18:12
czajkowskihttp://blog.launchpad.net/general/beta-test-asynchronous-ppa-package-copies18:12
abentleyrick_h_: r=me18:13
rick_h_71ty much abentley especially for the length catch.18:14
cjwatsonczajkowski: Thanks.  Belatedly fixed up the image a bit to not have bits of "demo" over the background18:14
abentleyrick_h_71: np.18:14
czajkowskiheh18:15
czajkowskilooks good18:15
czajkowskithanks cjwatson18:15
cjwatsonczajkowski: I'll do launchpad-users@ if you like18:15
czajkowskicjwatson: great18:16
cjwatsonhttps://lists.launchpad.net/launchpad-users/msg06483.html18:20
micahgcjwatson: would it help if I stress tested copy-packages now?18:53
cjwatsonmicahg: Yes please18:55
cjwatsonmicahg: Though remember that if you copy source only then that's expensive in terms of build time18:57
micahgcjwatson: no, I have a bunch of Mozilla stuff I can copy to a PPA that's a no-op for users (12 sources + ~500+ binaries)18:57
cjwatsonSounds like a good plan18:58
cjwatsonYou'll be "charged" for it in terms of quota, of course18:58
micahgwell, almost a no op, the PPA packages file will bloat a little, but not that much18:58
cjwatsonI don't think there's any hardlinking on disk18:58
micahgRequested sync of 11 packages.18:59
micahgPlease allow some time for these to be processed.18:59
micahgnice :)18:59
micahgooh, and the PPA shows what's waiting18:59
cjwatsonURL?18:59
micahghttps://launchpad.net/~mozillateam/+archive/thunderbird-stable/+packages18:59
micahgok, gotta run, will check later to see if everything worked19:00
cjwatsonEither I was too slow or that's shown to the owner only, I forget19:00
cjwatsonWould make some sense for it to be the latter19:00
micahggone now19:00
cjwatsonOh, there's a bunch of Pending items there, which are probably yours19:01
micahgyeah, all looking good19:01
micahg(aside from not enforcing quota, but that was previous behavior as well)19:01
cjwatsonMm, possibly worth fixing19:02
cjwatsonBut otherwise sounds like flawless victory so far19:02
SpamapShm, seems diffs linked to from +queue are getting the wrong host: 'lplibrarian-private-download.internal:8000'20:15
lifelessyes20:16
lifelessthere is a bug20:16
lifelesscolin put up a patch yesterday IIRC20:16
cjwatson_SpamapS,lifeless: it's fixed on devel - just needs me to do a bit of QA, then a deployment20:48
=== cjwatson_ is now known as cjwatson
cjwatsonI guess I should see if it's possible to delete the busted diffs and have them regenerated, afterwards20:49
SpamapSexcellent. :)20:49
cjwatsonSpamapS: And passed QA, so hopefully deploying tomorrow.22:36
StevenKcjwatson: I'll be doing QA after my stand up, and I'm tempted to put up a deployment afterwards.22:42
cjwatsonI was considering setting status=PENDING for everything in SELECT COUNT(pd) FROM packagediff pd, sourcepackagerelease spr, archive a, libraryfilealias lfa WHERE pd.to_source = spr.id AND spr.upload_archive = a.id AND a.private IS FALSE AND pd.diff_content = lfa.id AND lfa.restricted IS TRUE;22:58
cjwatsonEr, with syntax, but YKWIM22:58
cjwatsonI think that would clear up the breakage.  There are no such rows on dogfood right now, which is a good sign that at least it probably isn't overbroad22:59
lifelesscjwatson: why noy just toggle the restricted flag for them ?23:01
cjwatsonWouldn't the actual diff need to be re-uploaded to the public librarian too?23:03
cjwatsonThat's certainly how it works when copying packages from private to public archives.23:03
cjwatsonSeems easier to just get it to redo the diff - the bug hasn't been present for all that long so I expect it would be quick enough.23:04
wgrant_cjwatson: It's all a lie23:08
wgrant_cjwatson: The reupload is entirely unnecessary23:08
wgrant_I forgot we even still did that23:08
=== wgrant_ is now known as wgrant
cjwatsonSeriously?  And I spent actual effort understanding that.23:09
wgrantThere was perhaps an idea years ago that the two librarians would be separate23:09
cjwatsonAnd possibly even extending it.23:09
wgrantBut that never eventuated23:09
wgrantHm? The majority of the code you touched was for working out the files23:09
wgrantYou still need to work out the files23:09
wgrantYou just need to toggle the flag rather than reuploading.23:09
cjwatsonYeah, I can't remember now.  Plausible.23:09
cjwatsonSo OK, in that case flipping the flag is trivial enough.23:10
huwshimiCan I just ignore these ec2 test failures for AttributeError: type object 'InformationType' has no attribute 'EMBARGOEDSECURITY'?23:31
wgranthuwshimi: Yes, wallyworld fixed those last night after two conflicting branches landed.23:31
huwshimiAh23:31

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