[00:00] pg upstream would probably do pg.bouncer if at all; namespace packages are a pita to package [00:55] hmm [00:55] probably want to make the rabbit fixture port allocation logic reusable [02:10] Bah. [02:10] The critical deficit is resolved already. [02:13] wgrant? [02:13] We were at 239 criticals two checks ago, 237 when I checked this morning, and now 239 :) [02:33] hi [02:33] what's the status of the magic diff updating? [02:36] magic? [02:46] db rebulid is good for the soul: * 266 Time Outs [02:56] woo [02:56] 16384 | test_db1 | 2801 | 16385 | user1 | select * from pg_stat_activity; | f | 2011-07-18 02:55:50.531462+00 | 2011-07-18 02:55:50.531462+00 | 2011-07-18 02:55:31.384875+00 | | -1 [02:56] machine setup pgbouncer. [02:56] now we're bootstrapped, I can move onto more interesting tests. [02:58] thumper: Waiting on Red Squad being free to fulfill lifeless' deployment requirements. [03:04] * thumper nods [03:05] Mostly needs OOPS integration and metrics now, I think.; === poolie1 is now known as poolie [03:15] Hey, I just had a play with bzr-git for importing the GDB 7.3 release branch into Launchpad. [03:16] It works well, and can track more than one branch - you need to create a remote tracking branch first, but that's cheap [03:18] michaelh1: rocking! [03:18] hopefully soon lp will be able to track multiple branches [03:18] yeah, so to my naive eyes it has the support that LP needs. [03:19] yes, it does [03:25] A simple review if someone wants it: https://code.launchpad.net/~huwshimi/launchpad/email-tag-order-645962/+merge/68207 [03:26] michaelh1: A remote tracking branch? [03:27] huwshimi: Ah, that has bugged me for ages! [03:27] wgrant: Yeah, I finally got sick of it this morning :) [03:27] wgrant: yeah, bzr-git converts any local branches to bzr branches in refs/branch-name. It won't do remote branches, but you can set up a local branch that tracks the remote branch [03:28] michaelh1: Oh, doing an import like that, I see. [03:28] That's not how Launchpad does it. [03:28] huwshimi: Excellent. Approved. [03:28] wgrant: Thanks mate! [03:37] i was wondering in the shower if lp ought to show 'recently frequently duplicated bugs' as suggestions in +filebug [03:37] and whether they would be better guesses, for some users, than matching on words in the suggestion [03:37] or in fact hot bugs would be close to the same thing === almaisan-away is now known as al-maisan [03:40] well, the dup search should factor that in perhaps [03:40] OTOH many devs (me included) think offering dup detection is often misguided [03:44] because the person filing the bug is not in a good position to know if it's a dupe? [03:45] yup [03:45] and superficial things like the backtrace are often not indicative of dupe-or-not [03:45] i agree [03:45] sometimes they are, one-size doesn't fit all [03:46] yep [03:46] That's not very Launchpad of you. [03:48] wgrant: ok, so you didn't like pgbouncer as a module name [03:52] lifeless: Correct. [03:52] Do I have suggestions beyond terribleness like pgbouncerfixture? Nope. [03:54] Hmm. [03:54] Let's play "make qastaging update faster". [03:56] wgrant: so, in the absence of suggestions, I'm going with pgbouncer... we can do a rename dance later if needed [03:56] Perhaps. [03:56] could someone tell me briefly how lp's debian publication metadata importer works? [03:56] Perhaps we should make namespace packages not suck. [03:56] or point me to a wiki page or code [03:56] poolie: grep around for gina [03:56] poolie: There's no wiki page that I know of. [03:56] how does it get the data? [03:56] poolie: It is very unLaunchpaddy. [03:56] is that a good thing? :) [03:57] poolie: debmirror runs on iron to create a local mirror. gina then walks the indices and imports everything it sees. [03:57] This means it fails at tracking removals etc. [03:58] (for optional context, https://lists.ubuntu.com/archives/ubuntu-distributed-devel/2011-July/000872.html) [03:59] poolie: Sounds like our imports are badly timed. [04:00] I believe we import at the same frequency, but perhaps too offset. [04:00] well [04:00] between the ubuntu imports and debian [04:00] 52 2,8,14,20 * * * sh /srv/debian-import.launchpad.net/scripts/mirror-update.sh >> /srv/debian-import.launchpad.net/production-logs/mirror-update.log 2>&1 [04:00] folk have been known to upload the same version to both [04:00] 05 3,9,15,21 * * * LPCONFIG=gina /srv/debian-import.launchpad.net/production/launchpad/scripts/gina.py lenny squeeze sid experimental wheezy >> /srv/debian- import.launchpad.net/production-logs/gina.log 2>&1 [04:01] there seems to be a bug that codehosting oopses like https://lp-oops.canonical.com/oops.py/?oopsid=2021BZR207511 don't actually include a traceback [04:02] or is that just a one off? [04:05] many others do [04:05] I don't know why that doesn't. [04:05] oh well, bug 812106 [04:05] <_mup_> Bug #812106: codehosting oops doesn't include traceback < https://launchpad.net/bugs/812106 > [04:06] can fix or investigate it if it ever actually gets in the way [04:07] poolie: please use the code not the URL for oopses - otherwise they get gc'd [04:07] huh [04:07] they get gc'd when there's no open bug that mentions it? [04:09] yes [04:09] ok lunch [05:00] ok, EODing [05:00] till stub is around === al-maisan is now known as almaisan-away [05:03] wgrant: qas is WADLing [05:03] I sometimes wonder if the name was deliberate. [05:04] Haha [05:06] * StevenK kicks vocabs more [05:22] StevenK: You don't happen to feel like qa/donotcareing bug #808680, do you? [05:22] <_mup_> Bug #808680: The initializeddistroseries job fails with "permission denied for relation distributionjob" < https://launchpad.net/bugs/808680 > [05:22] It's all that's blocking a 48 revision deployment. [05:22] qa-untestable [05:23] It only affects multiparent initialisation, so I'm tempted to donotcare it. [05:23] Same [05:23] Doit [05:23] * wgrant doesit. [05:23] * wgrant makes tea while qa-tagger updates. [05:23] wgrant: Do you need a hand putting the deployment request together? [05:23] 48? qatagger says 32 [05:24] 15 ready to deploy, one blocker, 32 behind it. [05:25] The deployment report should say that [05:25] patch it up [05:25] its one of our codebases [05:27] Oh, sure. In the *oddles* of spare time I have :-P [05:28] Maybe I will decide I've had enough lazr-js for the day and try to fix it. [05:30] Besides, we can't. 1) We are not on maintaince. 2) Even if we were, criticals take priority. [05:31] lifeless: Are you in lpqateam? [05:31] Yes. [05:32] lifeless: Could you maybe change the crontab to regenerate lpqateam.canonical.com at 02,12,22,32,42,52, rather than a minute before qa-tagger? [05:36] Just what I wanted. A 131 line traceback [05:39] Oh goddammit. [05:40] There is a third revision involved in the mess. [05:40] I think it's just missing CSS, though. [05:41] If you say there is another rollback ... [05:52] With r13421 reverted, collapsibles created by activate_collapsibles (eg. "Extra options" on +filebug, "Configuration progress" on Product:+index) are no longer green. [05:52] Not sure we care for a day or so. [05:53] wgrant: File a critical/high, asign to Danilo, move on? [06:13] hi stub [06:13] stub: I have a present for you [06:14] Like your cat has a present for you? [06:14] better [06:15] https://launchpad.net/python-pgbouncer [06:20] stub: I wanted to simulate the 'cannot connect behaviour for the builddmaster [06:21] stub: and realised I had no idea what that meant pgbouncer wise, nor any way of programmatically testing it [06:21] I will have the answers to your questions today. [06:22] stub: so the new module bring up pgbouncer and should (as stands, or with minor tweaks - e.g. a reload() method to run pgbouncer -R) let us drive it programattically [06:22] There are two possibilities to what will happen - the connection is accepted by pgbouncer and is blocked, or the connection gets rejected. Depends on how much control pgbouncer gives me. [06:23] stub: FWICT dropping the user definitions (assuming we're using 'trusted') would block during connection [06:23] stub: that seems to be the only knob I could find that doesn't talk to the backend [06:26] lifeless: http://pgbouncer.projects.postgresql.org/doc/usage.html , in particular PROCESS CONTROLLING COMMANDS [06:27] I've tested PAUSE and it seems to do what we want, with inbound connections just blocking. I need to test SUSPEND in case we think it better to reject connections rather than queue them up for when things come back online. [06:28] stub: Oh, suspend rejects connections? :/ [06:28] This is easy and reliable for me to control, as I just open a database connection to the internal 'pgbouncer' database and issue commands with psycopg2 or psql - no need to edit or swap config files in place. [06:29] wgrant: I don't know about suspend. if we don't want to reject, we use PAUSE. [06:29] stub: I think it better to reject [06:29] stub: unless we can guarantee < 5 second application times (because 5 seconds is the hard timeout for new pages) [06:30] wgrant: Blocking until the system is back up isn't necessarily the best choice - eg. it might be preferable for appservers to display a 'down for maintenance' message rather than queuing up requests until HAProxy displays a 'down for maintenance' message [06:30] Right. [06:31] lifeless: Yup. Anyway - I'm working on the script today on staging. I should also be able to reliably do 'sudo /etc/init.d/pgbouncer stop' or just kill the pgbouncer processes. That will also reject connections :-) [06:31] stub: indeed :P [06:31] I think this whole no-readonly thing is pretty hideous, but if we're going that direction then immediate notification is better than one based on a timeout. [06:31] lifeless: If you are playing now, just 'sudo pgctlcluster 8.4 main stop' and see what exception you get trying to connect to a database that is not there at all. [06:31] stub: anyhow, we can make the pgbouncer python module support the same thing, so we can test how the appserver etc handle it [06:32] stub: without pgbouncer its OperationalError: connection failure ... [06:32] stub: I can now play with pgbouncer of course :> [06:32] Which will be a subclass of psycopg2.Exception (Error?) - probably best to just catch that on connection attempts. [06:33] mmm perhaps; probably needs a code read if we want to be broad [06:34] Well, we don't really care *why* the connection failed. Unless you want to get into socket exceptions for things like no-route-to-host vs. nothing listening, but there are so many choices there we won't do full coverage. [06:35] indeed [06:35] uhm [06:35] esp. as they will be coloured by how the firewall rules are (mis)configured [06:35] I don't know if we do don't. [06:35] will all depend on the most (fast-and-reliable) way to stop pgbouncer letting stuff through [06:35] I think treating all connection failures the same, and monitoring when scripts don't run for extended periods of time, covers us. [06:36] The monitoring will catch the stuffups such as users not existing or dud firewall rules etc. [06:36] sure [06:37] I guess I have a small worry that we'll be down for 30 seconds or something with a maintenance page before anyone starts actually clicking to there being a problem when we didn't plan to go down === henninge changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: henninge | Critical bugs: 237 - 0:[######=_]:256 [06:45] anyhow, this is minor and contingent on the actual implementation in pgbouncer anyhow [06:47] Morning folks. Any objects to my running process-accepted on dogfood? [06:47] jtv: None, but I think that bug is already QAed... [06:48] You would know — but did you actually Q/A the new feature, or just test that it didn't affect the features we currently rely on? [06:48] --derived seems to behave as specified. [06:48] That's great — thanks! [06:49] (I think it's probably dangerous, but we'll see) [06:49] gulp [06:49] Care to explain that? [06:50] If it's dangerous, it may be good to be aware of it before we "see." [06:50] Given that the primary archive publisher is not meant to be bulletproof and unexploitable, automatically publishing every Ubuntu derivative on one machine seems... potentially unwise. [06:50] What with the whole running arbitrary shell scripts and stuff thing. [06:50] With paths configurable in the web UI, albeit only by ~admins at this stage. [06:50] Arbitrary? [06:52] eg. things like Baltix are derivatives of Ubuntu, but probably don't want to go near bilimbi. [06:56] lifeless: re the postmortem: why not just pop back to the last safe version, rather than landing a reversion? [06:56] or rather than landing a reversion first [06:57] poolie: I'm talking about things we haven't deployed [06:57] poolie: If we do that then the deployment pipeline stalls for longer than 5 hours. [06:57] ? [06:57] ah, so it failed to deploy [06:58] no [06:58] failed qa [06:58] It failed QA, was not the head rev so had to be reverted. [06:58] Was fixed, deployed, broke production, rolled back, reverted. Another 5 hours. [06:59] was it deployed or not? [06:59] The second time. [06:59] IIRC it failed QA the first time. [06:59] poolie: when it does get through to prod, we do just switch to the previously deployed version, but we still also land a rollback (async from fixing prod) [07:00] ok i see [07:00] switching prod versions takes an hour, and then we're blocked on all deploys till the rollback gets through on trunk *and* we're fully qaed. [07:00] its a Real Big Deal, but fortunately fairly rare [07:00] the rollback is to get trunk back to a state on which other changes can be landed [07:01] lifeless: Rolling production back to a previous rev is quick. [07:01] But still a big pain. [07:01] wgrant: takes a while to walk the server processes [07:02] wgrant: we don't have to stage code, buts its still kinda slow [07:02] i guess splitting the test suite here could be good to [07:02] to say that changes to bzr probably don't need all the malone pagetests run [07:02] though, doing that manually would probably be tedious [07:02] ok [07:02] poolie: Why don't they? [07:03] You would be unpleasantly surprised :) [07:03] there is a good cost:benefit to running them? [07:04] so is switching production versions "quick" or "an hour" or an hour is quick by lp deployment standards? [07:04] its a bit less than an hour [07:04] poolie: It's quicker than an hour, but an hour is quick. [07:04] buts it not quick [07:04] :) [07:06] so the timeline was: [07:07] committed to devel; rolled out to qas; failed qa; attempted fix; fix rolled out to qas; passed qas(?); rolled out to lpnet; broke production; rolled back on production; reversion merged [07:07] is that correct? [07:07] no [07:08] committed to devel, rolled to qa, failed qa, rolled back, recommitted with fix to devel, rolled to qa, passed qa, deployed, broke, production switched to prior version and deploys frozen, rolled back on devel, recommitted with fix to devel, ... [07:09] deployed to qa, deployed to prod [07:09] ok [07:13] so it seems like catching things on qastaging is only a little cheaper than catching them after that? [07:13] it's still a lot of work but at least there is no production outage? [07:15] right [07:16] and we can deploy up to the bad commit [07:16] which (all too frequently) is useful - its a sign we're closing the queue too slowly [07:25] what was that python docs -> web service jml used? [07:25] rtfm something [07:25] readthedocs? [07:26] Yeah [07:26] yep thanks [07:26] poolie: the shortform, rtfd is more entertaining :) [07:27] oh, that was built in 24 hours o.O [07:55] good morning [07:59] hi adeuring! [07:59] hi jtv! [08:02] morning all [08:09] Morning [08:09] hi bigjools, hi mrevell! [08:10] allo allo ukians [08:11] any ever seen lp.services.job.tests.test_runner.TestTwistedJobRunner.test_memory_hog_job fail in ec2? [08:11] anyone* === jam1 is now known as jam [08:11] I happened to have made changes to IJob in my branch but it passes locally [08:12] I remember seeing _some_ failures in that area… hang on, I'll search. === almaisan-away is now known as al-maisan [08:15] poolie: Are you still around? [08:15] oui [08:16] poolie: bazaar-experts has had its celebrityness removed in prod, do you think it's worthwhile to delete the team in a few days? [08:16] i do [08:16] thanks for cleaning that up [08:16] bigjools: not much luck so far, and I'm back more than a month. [08:16] you might want to reply on the thread if it's not already clear [08:16] jtv: oh well [08:16] could be very spurious. I'm re-landing. [08:20] danilos: did you see bug 810309? [08:20] <_mup_> Bug #810309: Translator credits appear duplicated < https://launchpad.net/bugs/810309 > [08:24] bigjools: I need to make some config settings for generate-contents-files; wgrant thankfully discovered something I had neglected to take care of. Are dogfood and ftpmaster-publish the right, and complete, configs to edit? [08:25] jtv: should be [08:25] OK, then up for review it goes. [08:25] what did he see? [08:25] I didn't set the content_archive_root setting, which points the new script to where the (now legacy) ubuntu-contents dir is. [08:25] ah that [08:26] Apart from that it all looks good and a tad less awful than the shell script :) [08:26] Once we've started using the new script, we can retire that item again; I have a separate bug filed about that. [08:26] Thanks. :) [08:26] Plus it actually works. [08:27] jtv: Any reason not to start using the script as soon as it's on cocoplum, given that the old one doesn't work any more? [08:27] Far as I know, only this one last point. [08:28] (Of course there's usually _another_ one last point after that, but we can hope :) [08:28] heh. [08:28] I'm trying to take it all in stages and be meticulous about filing kanban cards; seriously scared about missing something. [08:30] StevenK: remind me tomorrow and I'll nuke the team [08:37] lifeless: I think tomorrow is too early, but if you can do it, excellent [08:37] I own it now [08:42] jtv, I didn't so far [08:43] jtv, this used to be broken in the past but I believe we fixed it [08:43] jtv, perhaps we only did so for translator-credits messages, and not for KDE messages [08:44] I thought that difference was well-isolated though. [08:44] poolie: rtfd.org [08:44] got it thanks [08:45] danilos: I wonder if the sharing has anything to do with it. It sounds like somebody changed the message's sharing status. [08:48] jtv, well, it seems the "Shared" message is shown even if the message is read-only (there's a read-only icon), so there are a few problems there [08:48] jtv, without looking at the code, I can't really say much else :) [08:50] It'd be nice if we could get a maintenance squad on it soon, but I couldn't bring myself to mark it Critical. [08:50] I just closed like 9 criticals, so nobody will notice if you bump it :) [08:51] That's quite a pace… the counter in the topic says 237; is that open ones or closed ones? === al-maisan is now known as almaisan-away [08:52] I opted not to update the topic again today; it was back up to 239 before the deploy, but should be down to 230 now. [08:52] We'll see soon. [08:52] 231, sorry. [08:52] Since I just filed a new one. [08:54] Still, probably best to resist the temptation to generate Criticals at a higher rate. :) [08:54] :( === almaisan-away is now known as al-maisan [08:57] StevenK: whats the reason for waiting ? [08:59] lifeless: That we can't revert to a pre-celebrity branch if we need to [09:11] well, could always recreate the team pretty quickly [09:11] but sure [09:11] happy to wait === wgrant changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: henninge | Critical bugs: 233 - 0:[######=_]:256 [09:23] StevenK, wgrant: on mawson, I am now planning to (1) put ubuntu-contents back into /srv/launchpad.net, and (2) run generate-contents-files to see that it moves the directory back to where it is now. Any objections? [09:28] is IDS in use yet ? [09:28] Informix Dynamic Server? [09:29] initialise distro series [09:30] It should be somewhere around the phase where it becomes accessible to colin only. [09:31] (AIUI this is all behind the feature flag for the derived distros UI, which is not being enabled for the general public yet) [09:32] wgrant: 233/256, what is 256? [09:32] In progress or fix released? [09:33] countdown [09:33] countdown? [09:33] to 0 [09:34] did the countdown reset from a particular date? === al-maisan is now known as almaisan-away [09:35] Essentially, why has the numerator changed while the denominator doesn't seem to change [09:35] its not a fraction [09:36] lifeless: Yes, IDS is in use [09:36] its a scalar and a meter [09:37] argh, right. [09:37] Sorry. [09:37] lifeless: Currently, it's only used for Ubuntu moving to the next release. [09:38] jtv: Sorry, was eating. Sounds good. [09:38] I figured if nobody's responding, nobody's doing anything vital. :) [09:39] It's currently running. [09:40] Heh [09:40] /srv/launchpad.net/ubuntu-contents is gone, which can only be a good sign. [09:44] * danilos tries to get expanders landed again... [09:46] danilos: the landing isn't the hard bit :) [09:46] lifeless, well, sometimes it is [09:46] danilos: sure, I mean in this case [09:47] The landing is easy. Having it *stay* landed is hard :-) [09:47] lifeless, well, the fix would have been landed sometime on Friday if there was not a test testing for the presence of the img tag [09:47] lifeless, and no reversion would have had to happen :) [09:47] ah true [09:48] of course, running lp.soyuz tests wasn't enough, so it was 5h of testing, and me being away from any computers over the weekend that made it impossible to go through :) [09:48] anyway, this code has gone through ec2/buildbot so many times that it has been well tested by now :) [10:05] lifeless: do your comments on https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations-schema/+merge/67992 amount to an approval? [10:07] preferable for stub to do so [10:07] if he was on leave, they would. [10:07] ah [10:08] he did approve the version in https://code.launchpad.net/~cjwatson/launchpad/multiarch-translations/+merge/67640 (admittedly before I renamed the column) [10:08] but then I split out that change into a separate branch for the new schema changes process === almaisan-away is now known as al-maisan === jtv is now known as jtv-eat [10:12] huwshimi, Hi, do you perhaps have a minute to discuss CSS sprites issue (bug 812044)? [10:12] <_mup_> Bug #812044: Expanded suggestions on +filebug leak branch icon < https://launchpad.net/bugs/812044 > [10:14] jtv-eat: if that Colin means me, I hope it'll actually be ~ubuntu-release rather than me personally [10:14] otherwise opening p-series will be blocked on me, rather than in practice normally being done by me :-) [10:26] danilos: Hey, sure === al-maisan is now known as almaisan-away === henninge is now known as henninge-lunch === henninge-lunch is now known as henninge [10:56] huwshimi, if you've got any ideas, please add them to the bug === matsubara-afk is now known as matsubara [11:08] danilos: Where can I see this bug in action? [11:08] danilos: Is it in someone's branch? [11:09] huwshimi, local LP instance (fresh devel), go to bugs.launchpad.dev/firefox/+filebug search for "a" [11:11] danilos: Thanks === jtv-eat is now known as jtv [11:19] cjwatson: don't worry too much — the "we'll let Colin see it" situation was always meant as temporary, for testing. [11:23] ok [11:26] Project devel build #895: FAILURE in 5 hr 57 min: https://lpci.wedontsleep.org/job/devel/895/ === henninge is now known as henninge-lunch [11:31] Can anyone help me figure out my bzr config for lp-production-configs? [11:32] I'm getting this error from PQM: "Exception processing merge: unknown url type 'bzr+ssh://bazaar.launchpad.net/~launchpad-pqm/launchpad/stable'" [11:32] No idea where either the "launchpad" (as opposed to lp-production-configs) and the "stable" come from. [11:34] jtv: use one of the debugging options on pqm-submit to figure out what email it's sending. (dry-run or verbose or something) [11:34] thanks [11:34] yup, it's --dry-run [11:35] star-merge bzr+ssh://bazaar.launchpad.net/~jtv/lp-production-configs/config-810986 bzr+ssh://bazaar.launchpad.net/~launchpad-pqm/lp-production-configs/trunk [11:35] That's what PQM produces. === almaisan-away is now known as al-maisan [11:52] jtv: --submit-branch FTW [11:53] bigjools: but then why doesn't the submit_branch setting in my locations.conf do the same? [12:00] jtv: is the branch in the location you think it is? [12:01] I was wondering whether that mattered, given the incorrect path in the error message. [12:01] I'm trying though. [12:01] "bzr info bzr+ssh://.../config-810986" [12:01] Yup, that works. [12:02] Now trunk: that works as well (apart from some garbage output from bzr) === benji changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: henninge, benji | Critical bugs: 233 - 0:[######=_]:256 === henninge-lunch is now known as henninge [12:15] Ahh, worked this time. [13:30] deryck: henninge: microphonde does not today... let try... [13:32] abentley, ping for standup === gary-lunch is now known as gary_poster [13:33] can't even open the audio assistent... [13:33] no luck... [13:34] adeuring, we're moving to skype, so I'll call you there for the standup/. [13:34] deryck: ok, just two minutes to strat another machine where skype is installed [13:35] adeuring, ah, ok. np. [13:38] adeuring, we'll start and dial you in if we can. [13:38] deryck: I'm online on sykpe [13:39] adeuring, you seem to drop immediately when abentley invites. [13:39] adeuring: Skype says you're not online and it doesn't work when I invite you, either. [13:39] let me restart skype... [13:39] done [13:40] adeuring, not working, sorry. [13:40] adeuring: still not working. [13:40] adeuring, we'll come back to you on IRC. [13:40] ok [14:03] benji: Up for a short JavaScript review? https://code.launchpad.net/~allenap/launchpad/localpackagediffs-filter-by-package-set-bug-809786-refactor/+merge/68233 [14:04] allenap: sure [14:04] Thanks. [14:17] allenap: approved [14:17] benji: Thanks :) [14:18] allenap: Invoking Y.use in a loop isn't something I've seen before. (Not that it's bad, it just surprised me.) [14:18] benji: I hope it works ;) [14:29] henninge: I've merged from stable, but lp:~abentley/launchpad/json-serialization still has failures: http://pastebin.ubuntu.com/646546/ [14:30] hm [14:31] let me look a little closer [14:53] henninge, benji: anyone wants to pick up https://code.launchpad.net/~danilo/launchpad/bug-812044/+merge/68263 (60 lines of diff) === henninge changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: benji | Critical bugs: 233 - 0:[######=_]:256 [14:54] danilos: I am not, sorry. ;) [14:54] abentley: look at this anonymously: https://answers.qastaging.launchpad.net/ [14:54] henninge, you bastard! :) [14:55] benji, hi, a very short branch up for review if you can find the time to look at it [14:55] danilos, benji: nm me, I'll take it [14:56] k [14:56] henninge: look at the mouseover text of why no "joe@example.com in question" [14:57] abentley: I know ... [14:57] abentley: I wonder why your test passes (or were you fixing that?) [14:57] henninge: What test do you mean? [14:58] abentley: the one from your paste [14:58] henninge: That test fails. [14:58] henninge: for me, anyhow. [14:58] henninge: You mean why does it pass on trunk? [14:58] abentley: oh, sorry, but other tests pass that should fail [14:59] henninge: I have ignored email obfuscation because I knew you were working on that. [14:59] in question-obfuscation.txt [15:00] danilos: why did you not fix the jslint warnings? [15:01] henninge, it's very old code, I just didn't bother, I can though [15:01] henninge, I also wanted to avoid having to QA the file bug form extensively as well [15:01] I think you should, unless it somehow relies on =='s casting behavior [15:04] henninge, sure, I can do it [15:05] danilos: thanks, otherwise it looks fine to me. r=me [15:05] henninge, cool, thanks [15:21] abentley: the test passes in devel. What is different in your branch? [15:22] * henninge pulls stable [15:23] henninge: In my branch, the anonymous user can see all cache objects, and the jsoncache rendering is done in LaunchpadView.getCacheJSON [15:26] henninge: my bad, the anonymous user is still restricted to just the context: https://pastebin.canonical.com/49879/ [15:31] mrevell: hi, you around? [15:34] abentley: I am glad that's figured out [15:35] nigelb, I'm on the phone at the moment === al-maisan is now known as almaisan-away [15:35] henninge: nothing is figured out. I still don't know why the email address is not being obfuscated. [15:35] abentley: oh [15:35] misread you [15:36] henninge: where did you do the obfuscation? [15:37] abentley: in the marshaller [15:37] abentley: maybe your code does not go through that? [15:38] henninge: Does the marshaller not provide ResourceJSONEncoder? [15:38] abentley: don't think so (I never heard of it) [15:39] abentley: the marshaller is for a specific field type [15:39] henninge: The ResourceJSonEncoder is provided by lazr.restful.tales. [15:49] henninge: Okay, so where is the marshaller and how is it used? [16:01] abentley: lib/lp/app/webservice/marshallers.py [16:01] abentley: zcml/override-includes/ws-marshaller-configure.zcml [16:02] abentley: I am sorry, I gotta run now. [16:02] henninge: okay. [16:04] abentley: the lazr-restful code does the apdapter lookup [16:05] abentley: adeuring should know all about that, too === deryck is now known as deryck[lunch] === matsubara is now known as matsubara-lunch === beuno is now known as beuno-lunch [17:02] Yippie, build fixed! [17:02] Project devel build #896: FIXED in 5 hr 35 min: https://lpci.wedontsleep.org/job/devel/896/ === deryck[lunch] is now known as deryck === salgado is now known as salgado-lunch === beuno-lunch is now known as beuno [17:46] sinzui: time to chat? [17:50] benji: any time for a review? https://code.launchpad.net/~jcsackett/launchpad/form-macros-frighten-me/+merge/68277 [17:50] jcsackett: sure thing [17:50] thanks! [17:54] jcsackett: I take it that there's some sort of quoting issue that neccesitated using \x22 for the double-quote character on line 42 of the diff. [17:55] benji: i believe so. almost all that code is as it was written before, just moved out of the template and into a js file. [17:55] we'll let sleeping escape sequences lie [17:59] jcsackett: add done [17:59] thanks, benji. [18:00] my pleasure [18:10] jcsackett, sorry, I did not see you message. I can chat now if you like [18:10] sinzui: sure. be on mumble in a moment. [18:16] gary_poster: For anonymous access to the web site, is request.principal None or does it provide IUnauthenicatedPrincipal? === matsubara-lunch is now known as matsubara === salgado-lunch is now known as salgado [18:27] abentley, should provide [18:27] abentley, sorry just saw this :-/ [18:28] gary_poster: np. === almaisan-away is now known as al-maisan === benji is now known as Guest43852 === al-maisan is now known as almaisan-away [18:33] gary_poster: in doctests such as lib/lp/answers/stories/question-obfuscation.txt, it appears request.user may be None. Is that a bug? [18:34] abentley, I've seen that stuff in out doctests before. I believe it is a bug in our test infrastructure, yes. [18:34] abentley, that, or the test is representing an incomplete state. [18:35] that's doubtful to be intentional though [18:36] Should (view.user is None) == IUnauthenticatedPrincipal.providedBy(request.principal) ? [18:37] mm... [18:38] abentley, Zope itself does not have a contract for the view other than __call__. It has a convention of 'request' and 'context' but they are not required from the perspective of the publishing or security machinery. I'm not sure what in Launchpad sets up view.user; I expect LaunchpadView. I'll look there for a moment [18:39] abentley, user in LaunchpadView is getUtility(ILaunchBag).user [18:40] I'll look there now [18:41] abentley, there are multiple implementations of the ILaunchBag, including at least three test stubs, one of them apparently intended to be general purpose (lib/canonical/launchpad/tests/test_helpers.py) [18:42] abentley, that one defaults to user=None [18:43] abentley, so that is at least one case in which tests would have a user of None. I'll look at the "real" implementation now... [18:44] abentley, yes, it looks like the launchbag's user can be None, it the principal cannot be adapted to IPerson [18:44] and I expect we do not have an IPerson implementation for anonymous [18:44] abentley, I'm stopping looking now. :-) [18:44] gary_poster: I'm actually experiencing the case where request.user is None, so it doesn't provide IUnauthenicatedUser, which is causing it to be treated as an authenticated user. [18:45] gary_poster: thanks for having a dive into it. [18:46] abentley np. As I said, I suspect that's a test infrastructure unpleasantness. I'm pretty sure I could find the Zope code that stuffs an unauthenticated principal on the request if you said that would be helpful. [18:48] gary_poster: I'm not sure what the right way to proceed is. Is IUnauthenticatedPrincipal.providedBy(request.principal) the best way to detect whether the user is authenticated? I guess I could ask the LaunchBag instead... [18:49] abentley, within the context of Launchpad, and Launchpad tests, the LaunchBag would be the path of least resistance, I think, given what you've seen. [18:49] and not a horrible one--plenty or precedent [18:49] of [18:53] gary_poster: Okay, thanks very much. [18:53] np [18:58] flacoste, ping [19:15] good morning :) [19:19] Hiya lifeless. :) [19:20] hi jkakar [19:26] hi sinzui [19:26] flacoste, do you have a few minutes to mumble about bug linking/dependencies? [19:31] sinzui: i'm on the phone already, will be a long one, i'll ping you afterward [19:31] thank you === matsubara is now known as matsubara-afk === Ursinha is now known as Ursinha-nom [20:27] sinzui: is that the call you wanted me on ? [20:29] lifeless, no. I think I know you you and OEM think my team is doing dependencies. There are TWO bug linking features planned in the next 3 months. It is very confusing and they obviously overlap [20:29] win [20:37] I guess I should revert that sendbranchmail thing. === wgrant changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: benji | Critical bugs: 237 - 0:[######=_]:256 [20:39] conflicts :( [20:49] allenap: I guess we should not deploy anything between r13460 and r13464? [20:54] lifeless: skype? [20:55] ringing you [20:55] (yes) [20:55] lifeless: yeah, skype crashed [20:55] \o/ === salgado is now known as salgado-afk [20:56] lifeless: can you try again? [21:22] sinzui: can i skype you in with lifeless? [21:22] sinzui: is your skype working ? [21:23] * sinzui thinks so === statik` is now known as statik [21:56] wgrant: have you spent the week i've been on leaving reverting the same branch over and over again? === benji changed the topic of #launchpad-dev to: https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 237 - 0:[######=_]:256 [22:03] mwhudson: No, several different ones :( [22:04] I've never reverted any one more than three times, though. [22:04] there's a first time for everything i guess [22:04] And they only sometimes get ontoproduction and break it in a different way each time :) [22:05] i guess that's a bit less terrible than breaking production the same way each time [22:05] They do that sometimes too. [22:24] Project db-devel build #730: FAILURE in 5 hr 37 min: https://lpci.wedontsleep.org/job/db-devel/730/ === Ursinha-nom is now known as Ursinha [22:40] Project devel build #897: FAILURE in 5 hr 37 min: https://lpci.wedontsleep.org/job/devel/897/ [23:01] StevenK, mumble? [23:13] hullo sinzui [23:13] ahoy oy poolie [23:15] wgrant: hi, following on from yesterday's question [23:15] do you think it would be feasible to change lp so that it always exposes debian publications before it does the derived ubuntu packages? [23:16] poolie: No. [23:16] It's impossible to determine. [23:17] And any approximation would just delay and anger people. [23:18] ... because it will mostly end up delaying the ubuntu package record, and that's often not needed and annoying? [23:18] Yes. [23:18] I'm not sure anybody would consider making Launchpad slower to be a feature. [23:18] :) [23:19] if we want to take the course of delaying the ubuntu package, we could do that only in udd where it's needed [23:19] poolie: We should do two things: [23:19] - Get the Debian import to be more up to date. [23:19] - Fix UDD. [23:20] so, more up to date means running it more often? [23:20] Or perhaps merely at more appropriate times. [23:20] But more often is always good. [23:20] No reason not to. [23:20] it's not outrageously expensive or slow? [23:20] debmirror should be trivial if the indices haven't changed, and if they have changed then we want to reimport. [23:20] right [23:21] And iron doesn't do anything useful. [23:21] do you know how frequent it would be now, or should i ask a losa what's in the cron job [23:21] I pasted the crontab lines yesterday. [23:21] * wgrant finds. [23:21] 52 2,8,14,20 * * * sh /srv/debian-import.launchpad.net/scripts/mirror-update.sh >> /srv/debian-import.launchpad.net/production-logs/mirror-update.log 2>&1 [23:21] 05 3,9,15,21 * * * LPCONFIG=gina /srv/debian-import.launchpad.net/production/launchpad/scripts/gina.py lenny squeeze sid experimental wheezy >> /srv/debian-import.launchpad.net/production-logs/gina.log 2>&1 [23:22] where did you find them? [23:23] lp-production-crontabs, I daresay [23:23] Yes. [23:23] iron-launchpad in lp:lp-production-crontabs [23:24] perhaps eventually it would be good to have something that just runs continuously rather than from cron [23:24] or, continuously with a short pause in between [23:25] Given what it has to do, that's no better than a */1 cron job. [23:26] i guess so [23:26] because they all have lock files to prevent concurrent runs? [23:26] No, but they would if we bumped it to */1. [23:27] The two scripts would run as a single frequent job, with locking, and only executing gina if the mirror changed. [23:27] so if we set it to /1 at the moment they would get into trouble, but if we added a lock file they would cope? [23:27] Right. [23:27] poolie: also because folk directly upload sometimes [23:27] ? [23:28] Occasionally. [23:28] poolie: ubuntu having a version before debian is a normal, if rare, situation [23:28] right [23:28] changing how we import debian won't avoid it [23:32] well [23:32] reducing the latency will reduce the number of confusing situations [23:32] it can't totally avoid it [23:36] ok bug 812597 [23:36] <_mup_> Bug #812597: gina imports from debian arrive too slowly < https://launchpad.net/bugs/812597 > [23:37] poolie: Are you sure that's the case? [23:38] poolie: The Debian archive is only published every six hours, whereas Ubuntu is published every hour. [23:39] *And* we do the possibly of syncing from Debian incoming if we wish. [23:39] s/do /do have / [23:51] http://webnumbr.com/launchpad-critical-bugs === 64MAAWTQK is now known as wallyworld === lifeless changed the topic of #launchpad-dev to: Performance Tuesday | https://dev.launchpad.net/ | On call reviewer: - | Critical bugs: 237 - 0:[######=_]:256