[12:07] <gary_poster> bac benji frankban http://tinyurl.com/yellowsquad in 3
[12:10] <gary_poster> bac you around?
[12:27] <bac> benji: reminder to tell me about the property rental trick
[13:03] <gary_poster> benji, please make sure your Google Canonical calendar is up to date with your availability for this week ASAP.  The recruiter will be using that to arrange interviews this week
[13:03] <benji> gary_poster: I am available all week, so we're good to go.
[13:04] <gary_poster> fantastic thanks benji
[13:04] <benji> np
[14:10] <benji> bac: is this the best list of reviewable things we were discussing in the meeting? https://code.launchpad.net/~launchpad-reviewers/+activereviews
[14:11] <bac> benji: i *think* so but haven't had a chance to look yet.  the alternative would be +ar on the launchpad suite.  but i think gary said maas wasn't a part of the suite
[14:12] <benji> bac: right, I don't see any MAAS reviews in the suite+ar, but they exist in the -reviewers+ar
[14:12] <bac> cool
[14:12] <benji> however, lpsetup reviwes aren't in -reviewers+ar but are listed in suite+ar
[15:34] <benji> frankban: I am finally done with https://code.launchpad.net/~frankban/lpsetup/bug-1023895-init-repo-no-checkout/+merge/115108
[15:43] <frankban> benji: thanks, I agree with your second comment, that is related to the first. For the first suggestion: replacing ~ with namespace.home_dir is naive. expanduser is not used there to support command run as root, with a --user option. In that case --user can also refer to a non-existent user.
[15:44] <benji> frankban: mmm, I was worried somethign like that was happening but after reading the code I didn't realize it; two thoughts:
[15:44] <benji> 1) a comment explainnig that bit so we don't trip over this again
[15:45] <frankban> benji: but we could replace ~ only when it is at the beginning of the string and followed by '/'
[15:45] <benji> 2) some paranoia in the tilda-substitution... right
[15:46] <frankban> benji: ok
[15:51] <frankban> benji: I can also try to avoid the real ~/.bazaar is used (using sheltoolbox.eviron). in that case, should I remove the BackupFile context manager?
[15:53] <benji> frankban: I think it would be an improvment to use a fake .bazaar, and yeah, it seems BackupFile wouldn't be needed then (and we can always resurrect it if we need it later)
[15:53] <benji> I'm over here now.
[15:53] <frankban> benji: right, thank you
[17:26] <frankban> benji: branch updated, could you take another look at it? no rush, it's my EOD, have a nice evening
[17:26] <benji> frankban: sure; have a good evening
[21:35] <bac> hi gary_poster, benji -- y'all still around?
[21:36] <gary_poster> bac I am kinda, though k just came by with Julia
[21:36] <gary_poster> I was trying to finish something at the time :-)
[21:37] <bac> gary_poster: ok, perhaps we can talk in the morning.  i'm stuck getting install_lpsetup_in_lxc to checkout the branch and i cannot figure out why.  bzr permission problems that usually mean you are root, though i explicitly am not.
[21:38] <gary_poster> bac, key is sometimes shared
[21:38] <gary_poster> which is kinda grody
[21:38] <gary_poster> but maybe we are not anymore and that is the problem?
[21:38] <gary_poster> IIRC...
[21:38] <bac> don't understand.  i should mention, this is actually happening in the host not the lxc
[21:39] <bac> so ssh isn't involved
[21:39] <bac> except for bzr using it
[21:39] <gary_poster> oh! um.  why is this happening in host?
[21:39] <bac> huh, i may try --use-http and see if that helps.
[21:39] <gary_poster> oh
[21:39] <gary_poster> ok
[21:39] <gary_poster> that would be something to fix if so
[21:40] <gary_poster> but if it helps you make progress...
[21:40] <gary_poster> I'm all in favor