[00:27] anastasiamac: thank you! silly white space. :-) [00:28] hml: nps - i was surprised it didn't pick up merge from my comment ;) but awesome that it's picked up now! [01:31] wallyworld: my window is thinking [02:32] anastasiamac: would you mind reviewing https://github.com/juju/juju/pull/7169, removing the feature flag from feature branch [02:32] * anastasiamac looking [02:34] axw: lgtm [02:34] anastasiamac: ta [03:31] thumper, jam: tech board? [03:33] coming [03:38] menn0: omw [04:02] * thumper groans [04:05] * thumper sads [04:08] fuck [04:08] machine instance status for migration is broken [04:16] https://github.com/juju/juju/pull/7170 btw, do i need to do something to get it noticed? [04:29] wallyworld: reviewed, finally - sorry! I'm a bit confused about one of the permission changes, but I need to rush out now (preschool AGM!). Talk about it after? Or just explain my confusion on the PR I guess. [04:30] babbageclunk: tyvm, i'll look at your question, feel free to ping when yu'r eback [04:31] mwhudson: you type $$merge$$ to get the bot to pick it up once approved, i'll take a look [05:28] babbageclunk: i've pushed changes, you were right in your assertion that write access should not have been removed [05:32] axw: whenever you are free, here's a PR which adds access permission to juju find/show endpoints CLI output https://github.com/juju/juju/pull/7171 (second commit) [06:11] wallyworld: left a couple of comments [06:12] wallyworld: (you can't do a review when you're scoped to one commit , apparently) [06:12] ok, ty [06:20] axw: yeah, that ErrPerm check should have been NotFound() [06:22] wallyworld: ok. might be a good idea to add a unit test for that. [06:22] axw: it's only for the case where getting the stored permission fails, we don't tend to test those secanrios [06:26] although i realise i need to update feature tests, i'll cover it in there [07:55] hello [07:55] i want to write amulet test case for my charm [07:55] is there any way to pass resource to this charm From amulet framework [08:06] lalit: best channel to ask would be on #juju :) [08:06] Thanks === akhavr1 is now known as akhavr === akhavr1 is now known as akhavr === akhavr1 is now known as akhavr [12:24] https://github.com/juju/juju/pull/7172 for someone who wants to review a python script that generates javascript so that people can share their mongo data with us === frankban is now known as frankban|afk === akhavr1 is now known as akhavr === akhavr1 is now known as akhavr [20:23] * mwhudson frantically unstars lots of juju repos on github /cc wallyworld [20:29] sorry, "unwatch"es [20:33] :) [21:12] mwhudson: sorry. i added you to the juju team so you could land your own patches if you wanted to [21:43] wallyworld: no worries :) [21:49] balloons: veebers: fyi, pr just landed, so i'll keep an eye on CI tests today to ensure it is fixed [21:49] wallyworld: awesome [21:53] wallyworld: just because we're assuming same controller for CMR, that doesn't mean we can assume same DB, right? (otherwise that'd break the JAAS case you're talking about) [21:54] babbageclunk: sort of. the workers get a connection to the correct model facade which for now is the same controller [21:55] wallyworld: it'd be good if we could have enough information about the other model/spaces/bindings locally that we could make the routability decision inside state without client code needing to hit a different API to get the required info. [21:56] wallyworld: or is that just wishful thinking? [21:56] we need to make a connection to another api anyway to set up the relation, so i don't think it's too bad to require a connection initially to get the required info [21:57] babbageclunk: for the current work though, we assume not routable right? [21:57] so we don't need to implement any api calls yet [21:58] well, any api calls that do real work [21:58] wallyworld: yeah, but if we're going to need to pass in some information about the other end of the relation then I at least need to define the type. [21:58] i'd be happy for now to stub that out one layer higher [21:58] wallyworld: ? [21:59] just to get something worker faster [21:59] wallyworld: I don't understand what you mean by "stub that out one layer higher". [21:59] maybe a hangout is easier [21:59] yup [22:07] is the beta controller under high load right now? [23:34] menn0: axw: migration with storage (block) on openstack failure... any advice? https://bugs.launchpad.net/juju/+bug/1677225 [23:34] Bug #1677225: Migrating model between controllers in conjure-up/novalxd openstack not complete after 3 hours stuck on transferring ownership of cloud resources to target controller [23:38] anastasiamac: i'm looking btw [23:39] babbageclunk: ^^^ [23:39] babbageclunk: looks like a problem with AdoptResources [23:39] menn0: looking too [23:42] babbageclunk: \o/ [23:47] menn0, anastasiamac: ok, sounds like one of the volume calls in AdoptResources is failing in a nova-lxd setup? So it needs to gracefully handle the not-supported error that comes back. [23:49] babbageclunk: yeah, mayb we just log it... i mean what is our expectation of recovery if this error occurs? my understanding is that there is nothing juju nor user can do.. right? [23:50] babbageclunk: well, if we get this then we just update the instance tags and skip doing volumes (which is fine because there won't be any), then we can continue with the other migration phases. [23:50] duh, anastasiamac [23:50] ^ [23:51] I keep @ing myself like a damn fool. [23:51] i was wondering if u were just making anote to urself :) [23:52] babbageclunk: to me the approach sounds fine, but do run it by axw and menno :D [23:52] Roses are red, violets are blue, I am schizophrenic, and so I am too [23:53] wpk: that [23:53] wpk: 's very philosophical.. is evrything k? [23:55] wpk: at least it rhymes. [23:55] wpk: been watching Legion? [23:55] it’s the middle of the night for wpk,… :-) [23:56] must be ramblings? [23:56] anastasiamac: I added a comment with some explanation. [23:56] hml: maybe he's sleep-ircing. [23:57] babbageclunk: ha! [23:57] Sometimes I do that with Telegram [23:59] babbageclunk: menn0: tyvm! awesome comment(s) :)