[00:27] <hml> anastasiamac: thank you!  silly white space.  :-)
[00:28] <anastasiamac> hml: nps - i was surprised it didn't pick up merge from my comment ;) but awesome that it's picked up now!
[01:31] <hml> wallyworld: my window is thinking
[02:32] <axw> 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] <anastasiamac> axw: lgtm
[02:34] <axw> anastasiamac: ta
[03:31] <menn0> thumper, jam: tech board?
[03:33] <thumper> coming
[03:38] <jam> menn0: omw
[04:02]  * thumper groans
[04:05]  * thumper sads
[04:08] <thumper> fuck
[04:08] <thumper> machine instance status for migration is broken
[04:16] <mwhudson> https://github.com/juju/juju/pull/7170 btw, do i need to do something to get it noticed?
[04:29] <babbageclunk> 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] <wallyworld> babbageclunk: tyvm, i'll look at your question, feel free to ping when yu'r eback
[04:31] <wallyworld> mwhudson: you type $$merge$$ to get the bot to pick it up once approved, i'll take a look
[05:28] <wallyworld> babbageclunk: i've pushed changes, you were right in your assertion that write access should not have been removed
[05:32] <wallyworld> 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] <axw> wallyworld: left a couple of comments
[06:12] <axw> wallyworld: (you can't do a review when you're scoped to one commit , apparently)
[06:12] <wallyworld> ok, ty
[06:20] <wallyworld> axw: yeah, that ErrPerm check should have been NotFound()
[06:22] <axw> wallyworld: ok. might be a good idea to add a unit test for that.
[06:22] <wallyworld> axw: it's only for the case where getting the stored permission fails, we don't tend to test those secanrios
[06:26] <wallyworld> although i realise i need to update feature tests, i'll cover it in there
[07:55] <lalit> hello
[07:55] <lalit> i want to write amulet test case for my charm
[07:55] <lalit> is there any way to pass resource to this charm From amulet framework
[08:06] <anastasiamac> lalit: best channel to ask would be on #juju :)
[08:06] <lalit> Thanks
[12:24] <jam> 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
[20:23]  * mwhudson frantically unstars lots of juju repos on github /cc wallyworld
[20:29] <mwhudson> sorry, "unwatch"es
[20:33] <thumper> :)
[21:12] <wallyworld> mwhudson: sorry. i added you to the juju team so you could land your own patches if you wanted to
[21:43] <mwhudson> wallyworld: no worries :)
[21:49] <wallyworld> balloons: veebers: fyi, pr just landed, so i'll keep an eye on CI tests today to ensure it is fixed
[21:49] <veebers> wallyworld: awesome
[21:53] <babbageclunk> 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] <wallyworld> babbageclunk: sort of. the workers get a connection to the correct model facade which for now is the same controller
[21:55] <babbageclunk> 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] <babbageclunk> wallyworld: or is that just wishful thinking?
[21:56] <wallyworld> 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] <wallyworld> babbageclunk: for the current work though, we assume not routable right?
[21:57] <wallyworld> so we don't need to implement any api calls yet
[21:58] <wallyworld> well, any api calls that do real work
[21:58] <babbageclunk> 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] <wallyworld> i'd be happy for now to stub that out one layer higher
[21:58] <babbageclunk> wallyworld: ?
[21:59] <wallyworld> just to get something worker faster
[21:59] <babbageclunk> wallyworld: I don't understand what you mean by "stub that out one layer higher".
[21:59] <wallyworld> maybe a hangout is easier
[21:59] <babbageclunk> yup
[22:07] <bdx> is the beta controller under high load right now?
[23:34] <anastasiamac> menn0: axw: migration with storage (block) on openstack failure... any advice? https://bugs.launchpad.net/juju/+bug/1677225
[23:34] <mup> 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 <juju:New> <https://launchpad.net/bugs/1677225>
[23:38] <menn0> anastasiamac: i'm looking btw
[23:39] <menn0> babbageclunk: ^^^
[23:39] <menn0> babbageclunk: looks like a problem with AdoptResources
[23:39] <babbageclunk> menn0: looking too
[23:42] <anastasiamac> babbageclunk: \o/
[23:47] <babbageclunk> 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] <anastasiamac> 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> 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] <babbageclunk> duh, anastasiamac
[23:50] <babbageclunk> ^
[23:51] <babbageclunk> I keep @ing myself like a damn fool.
[23:51] <anastasiamac> i was wondering if u were just making anote to urself :)
[23:52] <anastasiamac> babbageclunk: to me the approach sounds fine, but do run it by axw and menno :D
[23:52] <wpk> Roses are red, violets are blue, I am schizophrenic, and so I am too
[23:53] <anastasiamac> wpk: that
[23:53] <anastasiamac> wpk: 's very philosophical.. is evrything k?
[23:55] <babbageclunk> wpk: at least it rhymes.
[23:55] <babbageclunk> wpk: been watching Legion?
[23:55] <hml> it’s the middle of the night for wpk,… :-)
[23:56] <hml> must be ramblings?
[23:56] <babbageclunk> anastasiamac: I added a comment with some explanation.
[23:56] <babbageclunk> hml: maybe he's sleep-ircing.
[23:57] <hml> babbageclunk: ha!
[23:57] <babbageclunk> Sometimes I do that with Telegram
[23:59] <anastasiamac> babbageclunk: menn0: tyvm! awesome comment(s) :)