[00:37] Running make results in the following error: [00:37] Error: Couldn't find a distribution for 'lazr.restful==0.9.17'. [00:37] any clues? [00:38] cody-somerville: bzr update download-cache maybe? [00:38] I was just going to try that [00:39] mwhudson, thanks. That seems to have fixed it. [00:40] rocketfuel-get will do that for you [00:40] (but i don't use that) [00:40] I don't use it either [00:41] i'm sure some people do :) [00:46] Has anyone floated the idea of making "Policy: You must be a team member to subscribe to the team mailing list. " configurable in the past? [00:48] cody-somerville: I have a simple bash script I use for updating stuff [00:48] cody-somerville: involves cd and bzr pull :) [00:49] ditto :-) [00:49] rocketfuel-* is mostly too magic for my liking [00:52] maxb: my thoughts exactly [00:52] I prefer a simpler magic [00:52] I want to interact with an object in launchpad via python shell. Whats the easiest way to go about that? [00:53] cody-somerville: launchpadlib isn't too bad for interactive use [00:54] * thumper agrees with mwhudson [00:54] this function isn't exposed via the API [00:54] cody-somerville: submit a patch :-) [00:55] nor should it be exposed [00:55] or maybe it could be [00:57] getting launchpadlib going in a python shell is harder than it needs to be [00:57] unless there are helpers that I'm unaware of [00:58] bug 425856. I solicit comments :-) [00:58] Bug #425856: non-team members cannot subscribe to launchpad/team mailing list [00:58] maxb: getting going is a bit of a pita yes [00:58] mwhudson, thumper: basically, I want to cal an SQL helper method to see what the result is [00:58] cody-somerville: the correct interface for that is "a losa" i think? [00:58] if you can run an irc client in you python terminal, that might work :) [00:59] cody-somerville: or do you mean locally? [00:59] SELECT CAN HAZ CHEEZEBURGER FROM GREASYS [00:59] :) [00:59] mwhudson, locally [00:59] cody-somerville: 'make harness' [00:59] yes, make harness [00:59] can I do that when launchpad is running? [01:00] yep [01:03] thanks [01:03] Now, whats the easiest way to get the object for a particular bug filed against a source package in a distribution? [01:04] cody-somerville: using its number [01:04] Is there some zope magic I invoke against the interface or some helper function? [01:05] * thumper looks [01:05] getUtility(IBugSet).get(number) [01:07] thanks [01:07] I suppose I should refer to the doctests [01:07] lots of good examples there [01:08] well [01:08] lots of examples, at least [01:10] * ajmitch saw some lovely doctests in thumper's talk :) [01:10] I didn't read the doctests for that [01:10] I went to lib/lp/bugs/interfaces/bug.py [01:10] oh it was in the testing discussion, wasn't it? [01:10] heh [01:11] you showed off branch.txt [01:11] :( [01:12] jml: it was an example of a bad doctest [01:12] good good :) [01:12] jml: don't worry, he didn't say it was all crap [01:13] branch.txt is very largely crap [01:13] thumper: where can I read about the factory stuff you mentioned earlier? [01:13] ajmitch: look at any tests in the lp.code area [01:13] ajmitch: or look in lp.testing.factory [01:13] ajmitch: we tend to use TestCaseWithFactory from lp.testing [01:13] ok, thanks [01:14] ajmitch: if you are using a doctest, there is a factory glob object [01:14] factories are cool [01:14] but [01:14] ours is really slow [01:14] and it ought to have tests of its own [01:15] lp.testing.factory does look rather large to wade through [01:20] thumper, you know what would be great [01:20] jml: unlimited cosmic power? [01:20] thumper, being able to see all the trunk branches I have commit access to [01:27] thumper, what do you think? [01:27] jml: useful for some people [01:28] jml: what is your main use case? [01:28] thumper, I want to find my projects. [01:28] thumper, I have quite a few of them now. [01:28] jml: I'd much rather have a way to personally associate myself with a project [01:28] jml: that is of more interest to more people [01:29] thumper, I don't get it. You want to enter even more data? [01:29] yes [01:29] if [01:29] I can have a page that just shows me about projects I care about [01:29] kinda like a "follow" this project [01:30] hmm [01:30] I think I'd still want to be able to see the projects I have commit rights to [01:31] wgrant, where is the configuration for lucile kept at? [01:31] cody-somerville: Which bit? [01:32] publisher [01:32] Some in the DB, some in configs/development/launchpad-lazr.conf, some in configs/schema-lazr.conf [01:33] And the occasional bit hardcoded. [01:33] https://dev.launchpad.net/Soyuz/HowToUseSoyuzLocally is my documentation on setting the whole thing up. [01:34] jml: where would you put the link? [01:34] thumper, why not make it part of the personal code page [01:35] could do... [01:35] patches accepted [01:35] jml: I suppose that you could have write access to quite a few projects indirectly through teams? [01:35] ajmitch, yes. [01:35] thumper, I'm actually still waiting on a chance to get my other patches landed. [01:38] thumper, can you please review upload-permission-joy? [01:43] how do I expose a function over the API that takes a list of URLs as a parameter? [01:45] rockstar: how do I assert that an xpath doesn't exist in a windmill test? [01:51] hey wgrant :) [01:51] Hi Ursinha. [01:55] my lp setup still wants to use the 0.9.9 version of lazr.restfulclient when I have the 0.9.10 inside the download-cache folder [01:55] what do I have to do to lp use the newest? [01:55] Merge from devel. [01:56] (or alter versions.cfg manually) [01:58] thanks wgrant :) I'd like to know which file defined that [01:58] You might also have to run bin/buildout again, I guess. [01:58] yay [01:59] Where's the bestest best best place to get help on LP API; right here? [02:03] MTecknology: yep [02:04] thumper: How do I get my hands on it so I can use it? I feel like I'm about to dive into a sea of sharks with this but my company wants two of us to learn all we can about it [02:05] MTecknology: to what end? [02:06] One reason is because we want to modify some core tools we use to utilize the openid+login process of Launchpad so the use of LP is entirely transparent to the user. We're also looking into our own server to host the LP server [02:07] If we can get that, then we're looking into using it for a support ticket system and other things [02:08] MTecknology: the project is launchpadlib, and I believe it has good docs [02:08] next question [02:09] how can I expose a method that returns a dict? [02:09] mapping URLs to IBranch [02:09] jml: Give up, I'm afraid :( [02:09] wgrant, that sucks. [02:09] It does. [02:09] wgrant, I guess a list will work ok in this context though. [02:10] You can return a dict, but launchpadlib will just see it as a dict of URLs to URLs. [02:12] thumper: thanks. I was trying to figure out how we could do everything that's being planned and it hit me that Launchpad can handle every piece of that [02:13] MTecknology: I don't see what the LP API has to do with this. [02:13] jml: I think leonard is working on exposing a dict over ReST [02:13] thumper, that's good. [02:14] * thumper throws hands in the air [02:14] how can XPath have "and" and "or" but not "not"? [02:15] wgrant: I don't really understand what the api is; but I was told to get it so we can see if it will help us [02:17] MTecknology, https://help.launchpad.net/API [02:18] kfogel, hi [02:19] jml: hey [02:20] kfogel, how's things? [02:21] jml: well, I discovered halfway through the day that today was a U.S. holiday, that was interesting :-). How's the sprint going? [02:21] kfogel, good. [02:21] kfogel, less coding than I had hoped, but probably the better for it. [02:22] jml: ah, too bad, but that's what happens when you know too much [02:23] hello [02:23] nice reply there kfogel [02:24] it's thanksgiving isn't it? [02:25] poolie: thanks. I was wondering if it was too harsh; glad you didn't find it so. It's not Thanksgiving yet, btw, but it is Veterans Day in the US. [02:25] poolie: veterans day [02:25] oh right [02:25] thanksgiving is in two weeks [02:26] armistice day is observed here but not a holiday [02:26] * kfogel is glad the whole world knows US holidays, and wonders if it has something to do with all our missiles [02:27] lol [02:27] jml, do you know which commit message ec2 script considers when using the land command? [02:27] Ursinha, the one on the merge proposal page [02:27] kfogel: no, just approved rockstar's holidays, that's all [02:27] Ursinha, you can provide one as an option to ec2 land [02:28] thumper: heh [02:28] Ursinha: but don't add all the [xx] rubbish to that commit message :) [02:28] thumper, oops, why? [02:28] Ursinha: because the land commands adds them [02:28] Ursinha: based on who reviews/approves, bugs et al. [02:28] thumper, makes sense [02:29] Ursinha, --dry-run ftw [02:29] jml, :) [02:29] Speaking of the [xx] rubbish, is there a reference to what it all means anywhere? [02:29] maxb, if it's not on the dev wiki, then probably not. [02:29] maxb, the source code for ec2 land might actually have pointers to docs :) [02:30] not that I've been able to find [02:30] The only one that was unobvious to me was rs. [02:30] maxb, in that case, I won't be able to help you find docs, since I looked for docs when I wrote land :) [02:31] Well, yeah, I think I've figured most of them out by example by now. But [!log] still mystifies me (seen in some sourcecode branch I forget which) [02:32] maxb, don't include it in the news announcement for the release. [02:32] maxb, we used to use it to make preparing announcements easier. We don't any more. [02:43] poolie: nice response -- I didn't mean to guilt-trip you, though! :-) [02:43] not at all [02:43] I am feeling unhappy [02:43] and it is because of software [02:43] again [02:44] If I return a list of branches over the API, but some of those branches are None, then the 'None' objects don't appear in the list. [02:48] AFAICT, this means that helper methods like IBranchSet.getByUrls are impossible [02:53] jml: and there isn't a map? [02:53] thumper, there isn't. [02:55] I'm going to have to think of a different example for my UDS session. [02:56] jml: is there no way to match up the returned branches to the URL(s) originally requested? [02:56] kfogel, not easily. [02:56] kfogel, there are multiple attributes on branch that the URL might be matched on. [02:57] jml: oh [02:57] and some that aren't even attributes [02:59] although matching against the returned bzr_identity, then matching against the returned branch.url and then falling back to matching the returned unique_name against the requested URL path would _probably_ get you most of the way there [03:00] https://bugs.edge.launchpad.net/lazr.restful/+bug/481090 [03:00] Bug #481090: Cannot define a method that returns a dict [03:20] * thumper ndos [03:23] good evening [03:37] afternoon :) [03:54] Can the LP API allow users to manage teams/users if the person using the API has that permission? [03:54] I know the API is used to pull info, not if it can pushing or altering [03:56] nevermind - I guess if I pay attention the first page kinda says I can do that [03:59] and the api documentation says exactly what can be done [04:00] maxb: I'm still looking aroudn for that. I don't refer to documentation much so reading through it is hard work for mew [04:03] https://launchpad.net/+apidoc/ [04:20] maxb: :D [04:21] thanks === mwhudson_ is now known as mwhudson [06:26] hey, people here know stuff [06:26] I'm trying to prepare a USB stick for UDS that I can use to accelerate people having a Launchpad development environment [06:27] my plan is to set up a minimal chroot, run rocketfuel-setup, make sure everything works, and then whack it all in a tarball [06:27] does that sound sane? [06:27] Awesome idea. [06:29] As long as you rerun rocketfuel-setup at the other end, that does sound sane. [06:29] well, I'm also not 100% clear on what people on the other end will need to do to get the chroot. [06:30] Oh, you're going to copy the entire chroot? Hmm. [06:30] that was my original plan [06:30] That'll run into postgres port conflicts and other mess if they already have postgres installed. [06:30] oh yeah [06:30] Better to make rocketfuel-setup make less of a mess, so you don't really need a chroot. [06:31] wgrant, I'm not going to have time to do that before UDS :) [06:31] Plus even if you do have a chroot, you still need to modify /etc/hosts outside the chroot. [06:31] so maybe just having the repo on a USB stick [06:31] + sourcecode [06:32] right. [06:32] and make sure that rf-setup can be run in such a way as to respect the existing file [06:32] s [06:32] Get a clean ~/launchpad, remove devel from it, but keep a copy of rocketfuel-setup in there. [06:32] It will already. [06:33] If you extract the tarball into ~/launchpad as it expects, it will Just Work Very Quickly. === abentley1 is now known as abentley [06:42] jml: what about a vm on a stick? [06:43] thumper, the problem is then people need to know how to get a vm set up [06:44] thumper, it's all about taking the smallest amount of time as possible so I can get on with the rest of the session. [06:44] jml: people need to know how to get a chroot set up too [06:44] jml: just a thought [06:44] thumper, well yeah, that's why I'm not doing it :) [06:45] If you're expecting quite a few people to set it up, it might be a good idea to remove some of the crap from rf-setup this week. [06:45] eg. the SSH config stuff is no longer necessary. [06:46] wgrant, maybe [06:46] wgrant, are there bugs filed about all of this? [06:47] jml: Bug #419028 [06:47] Bug #419028: Don't edit SSH config in rocketfuel-setup [06:48] wgrant, know of anything else that ought to be removed / cleaned up that doesn't require much thinking? [06:49] jml: I really feel that it's unnecessary to install rocketfuel-* in $PATH, but that's a slightly more destructive change. [06:49] I agree on both counts. [06:49] OTOH, it won't affect any existing installations. [06:55] Apart from that, rf-setup isn't too bad. [06:55] launchpad-database-setup, OTOH... [07:06] wgrant, http://bazaar.launchpad.net/~jml/launchpad/rf-setup-cleanup/revision/9863 [07:07] jml: Great. [07:07] That probably makes it noninteractive, too. [07:07] Hm, not quite. [07:07] But close enough. [07:14] * mwhudson_ reviewatrons [07:17] Is there no link back from Loggerhead to the branch in LP? [07:18] mwhudson_: Nice review comment, but your diff highlighter sucks. [07:18] wgrant: there is a link in the top left [07:18] wgrant: "watches pelcome" or something [07:18] Ah, so there is. [07:20] Oh, right, it assumes it's a header. [08:33] good morning [09:10] Morning [09:15] bigjools: I don't see what your SourcePackageFormatSet methods do. [09:15] wgrant: ok I'll explain in a moment === dpm_ is now known as dpm [09:30] * thumper has many branches waiting for review [09:30] * thumper has fixed two oopses today [09:30] my work here is done [09:30] * thumper goes to wash disher [09:31] s/disher/dishes/ [09:31] it's late [09:42] wgrant: ok let's chat [09:42] so, SourcePackageFormatSet would be a utility [09:43] which is security wrapped [09:43] we don't generally use __init__ methods in model classes [09:43] the utility has the job of creating new instances [09:43] so that it returns security-wrapped objects [09:45] I followed https://dev.launchpad.net/Soyuz/HowToUseSoyuzLocally to add builders but now I want to remove them. I've tried with "remove" instead of "add" ( scripts/ftpmaster-tools/manage-chroot.py -s lucid -a i386 remove -f chroot-ubuntu-lucid-i386.tar.bz2 ) but it does nothing. What am I doing wrong? [09:46] (sorry for repeating the question, noone answered yesterday and I haven't found a solution yet) [09:48] pgquiles: that command deals with chroots, not builders [09:48] bigjools: Right, yes. So it would be add(distroseries, format), and then there would be something like get(distroseries, format)? [09:49] pgquiles: What did you expect it to do? [09:49] wgrant: yep, pretty much, although not get(), getBySeriesAndFormat() [09:49] wgrant: get() is normally used for getting by ID [09:49] bigjools: Right. Thanks. [09:49] bigjools: Now, what did you mean about the XXX? [09:49] bigjools: There were some words omitted. [09:50] wgrant: I read it back and it made sense to me, so maybe it's just my twisted English :) Basically, I would say don't worry about it and remove the XXX [09:50] wgrant: I want to remove a builder from launchpad.dev/builders, and I want to remove a chroot because I did manage-chroot.py -s karmic -a i386 add -f chroot-ubuntu-lucid-i386.tar.bz2 (I told LP to add a Lucid chroot for a Karmic series, my bad) [09:51] pgquiles: mark it as manual and it disappears [09:51] bigjools: "I'm trying to think if it would be bad to reject the upload if we detect orphan files, and I can't think of one." <- one what? reason to reject, I presume. [09:51] wgrant: yes [09:51] wgrant: my twisted English, sorry [09:51] pgquiles: You actually want to mark the build as inactive, or something like that. [09:51] Not manual -- that will still show up. [09:52] pgquiles: You're sure that manage-chroot.py remove does nothing? [09:52] bigjools: Thanks. Fixing now. [09:52] I've never had cause to remove a chroot myself. [09:52] wgrant: I don't know if it does anything but builders are still there [09:52] even worse: when I try to add a new chroot, it adds it but it does not show in LP :-/ [09:52] pgquiles: Chroots are not shown in LP. [09:53] wgrant: the launchpad.dev/builders/bob/+edit I mean [09:53] pgquiles: That has nothing to do with any chroot. [09:53] ¿? [09:54] pgquiles: A builder is it a builder. Its builds will use chroots, but it isn't directly related to them. [09:54] according to https://dev.launchpad.net/Soyuz/HowToUseSoyuzLocally you have to add a chroot, then mark Bob the Builder as OK [09:55] Those steps are not related beyond both being prerequisites for a working build system. [09:55] ah yes mark it as inactive to remove it from /builders [09:56] I don't understand anything :-( After I added a chroot, it was shown in the builders list and I could mark it OK and it worked fine [09:56] The existing builder is shown in the builders list. [09:56] Not the chroot. [09:56] what I want to do is remove a chroot (tried with 'remove' but it does nothing) and add a new chroot for jaunty (but after the 'remove' for lucid, it no longer works) [09:57] bigjools: it will not show the builder in /builders but it's still in LP, you can re-enable it again [09:57] wgrant: do you mean the builder is there but with no chroot? [09:57] pgquiles: that's correct [09:57] ok, so, how do I completely remove a builder? [09:59] as in "this builder no longer has a chroot associated, I don't want it to be shown, even as inactive" [10:01] Builders are not associated with chroots. [10:01] You cannot completely remove a builder without removing lots of other sample data. [10:02] mmm so I add chroots to launchpad [10:02] and then I add builders [10:02] and builders will know what chroot to pick when a package needs to be built [10:02] is that it? [10:02] Yes. [10:03] great :-) so builders are smarter than what I thought (and definitely smarter than I :-) ) [10:04] then, to fix my bad "manage-chroot.py -s karmic -a i386 remove -f chroot-ubuntu-lucid-i386.tar.bz2", can I just run "manage-chroot.py -s lucid -a i386 remove -f chroot-ubuntu-lucid-i386.tar.bz2" and that's it? [10:11] bigjools: SourcePackageFormatSet, or should it be SourcePackageFormatSelectionSet? [10:12] wgrant: the latter [10:12] bit of a mouthful :/ [10:12] There's worse. [10:18] one more question: virtualized builders. Are there any docs on how to set up one? (I can't find anything) [10:18] no, there are no docs [10:20] and I guess it's not exactly easy, is it? [10:20] depends on how much you know about virtualisation [10:23] I know virtualization but I've never set up Xen (which is what LP uses IIRC). Can it use something else (VMware, VBox, qemu, KVM) ? [10:24] you can use what you like, it's nothing to do with LP [10:24] great [10:24] exactly what are you setting up LP for? [10:24] for internal use at work [10:25] we are developing our own distribution for military use, based on kubuntu + packages for our software and backports from lucid, debian unstable and some stuff I've packaged and it's not available from debian or ubuntu [10:26] I think I'm the first one who requested to buy Launchpad for on site use but it was not available at the time (2+ years ago) [10:27] given that it's military use and our software is closed source, hosted LP commercial is not an option [10:30] I wonder how long it will be before enough people start wanting to rebrand launchpad that we actually end up with launchpad hosting projects/branches for a rebranding effort of itself [10:32] I find someone internally using an open-source LP to host closed-source stuff extremely distasteful [10:33] bigjools: does Canonical sell a LP appliance, training, whatever? I've tried to BUY that but I was told (by Mark himself) Canonical does not sell that still [10:33] and I cannot upload military stuff to somewhere else, no matter how much security you promise [10:34] bigjools: why? [10:41] The way I see it, using Launchpad in that way takes nothing away from the Launchpad project, and might be the source of bug reports, fixes, and an interested contributor [11:10] I've fixed a missing quote in HowToUseSoyuzLocally and added the explanation on the non-association between builders and chroots, which was not obvious from reading the page [11:11] It was a hasty conversion last week from a Tomboy note; I'm not surprised I missed a quote. [11:11] Was the attachment change this morning just s/serieses/series/? [11:12] wgrant: yes, that was it - maxb told me what was failing [11:16] HowToUseSoyuzLocally says I should add an external dependency on "deb http://archive.ubuntu.com/ubuntu %(series) main restricted universe multiverse" to the PPA but when I try to do that from "add dependencies" in the PPA, LP fails. Is that the right place? Is that the line I should enter in the "Add PPA dependency" text entry? [11:16] "# We use Hoary as the root, as Breezy and Grumpy are broken." ....... Grumpy? :-) [11:16] Grumpy Groundhog [11:16] The long-fabled bleeding edge daily builds Ubuntu. [11:16] Which somehow made it into the sample data. [11:24] mmm one little trick in /etc/hosts and now archive.launchpad.dev goes to archive.ubuntu.com... and it's building! [11:25] pgquiles: If you followed my instructions, you shouldn't need to do that. [11:27] wgrant: I followed your instructions but I don't know how to add the external dependency on archive.ubuntu.com, adding it in the "Add PPA dependencies" page in LP does not work (Zope error) :-? [11:27] pgquiles: See the field at the bottom of the PPA's +admin page. [11:28] wgrant: heh, so that's why you said it was handy [11:28] wgrant: ah! that was it [11:28] wgrant: I thought I should add it to the "Edit PPA dependencies" page [11:28] thanks! [11:29] bigjools: Yeah, it's particularly convenient for testing primary archive stuff. === matsubara-afk is now known as matsubara [11:29] bigjools: If I'm just dealing with PPAs, I just change archivepublisher.base_url to a local mirror. [11:29] yeah [11:31] added to the wiki [11:31] pgquiles: Note that that same thing is mentioned two steps earlier. [11:32] oops, right [11:35] pgquiles: That looks better. [11:56] wgrant: sorry if this is a stupid question but, the section about "dealing with the primary archive", does it refer to a local mirror of archive.ubuntu.com (i. e. archive.launchpad.dev) ? the steps listed there, are they for adding PPA-built packages to archive.launchpad.dev? [11:57] pgquiles: The primary archive is archive.launchpad.dev. But it's nothing about copying in PPA packages. [12:00] so those steps are for building the packages and adding them to archive.launchpad.dev *instead* of a PPA, is that it? [12:00] Yes. [12:01] perfect, thank you [12:01] now looking into setting up virtualized builders! [12:06] bigjools: Want an intermediate diff? [12:06] wgrant: yes please [12:06] wgrant: btw, /var/tmp/zeca is not created in the by rocketfuel_setup/start_soyuz.sh/whatever [12:07] pgquiles: Ah, yes, forgot about that. [12:32] bigjools: What do you think about running "UPDATE sourcepackagerelease SET dsc_format='1.0' WHERE dsc_format IS NULL"? No new broken SPRs have been created since 2006, so it should be a safe one-off fix. === mrevell is now known as mrevell-lunch === danilo_ is now known as danilos === mrevell-lunch is now known as mrevell [13:54] wgrant: yes, sounds sane [14:45] Chex, gary_poster, rockstar, bigjools, danilos, citrus, allenap: LP production meeting in 15 min @ #launchpad-meeting [14:47] danilos: do we have team lead call in 15, as Google lists? [14:51] gary_poster, no, as per email yesterday [14:51] matsubara, thanks [14:51] danilos: oh, must have missed it in the email swamp. Looking... [14:56] found, and thanks === citrus is now known as sinzui === mbarnett` is now known as mbarnett === matsubara is now known as matsubara-lunch === SteveA_ is now known as SteveA === salgado_ is now known as salgado [16:45] salgado: ping [16:45] hi sinzui [16:46] salgado: I am working on bug 471770. There is a conflict between AdminMergeBaseView.doMerge() which sets the dup email address to NEW and PersonSert._mergeMailingListSubscriptions which moves the subscription (with a NEW email) to the new acount [16:46] Bug #471770: mailing list subscription email address are in NEW status [16:47] salgado: I think I should change PersonSert._mergeMailingListSubscriptions to remove all the subscriptions and send an email to the user that he has to re-validate his address *and* resubscribe... [16:49] salgado: but maybe I should change the subscriptions to the preferred email address, or ensure that the the affect email addresses are VALIDATED [16:55] wgrant: ping === matsubara-lunch is now known as matsubara [17:14] sinzui, I think I'd rather remove the subscriptions and send out emails for the user to re-validate the address and subscribe [17:15] salgado: thank you. That is what I will do. [17:58] night all === danilos is now known as danilo-afk [18:29] gmb, ping === dobey_ is now known as dobey === deryck is now known as deryck[lunch] [20:28] any one around who can review? [20:28] * thumper needs some reviewers === matsubara is now known as matsubara-afk === jtv1 is now known as jtv [21:38] pgquiles: Hi. === abentley1 is now known as abentley [22:07]