[01:38] <wallyworld> hml: if you're still here, a review of that oracle-compute -> oracle pr would be awesome https://github.com/juju/juju/pull/7334
[04:17] <axw> wallyworld: can you please take a look at https://github.com/juju/juju/pull/7322 when you have a chance? there's more backed up behind it (aside from the watcher fix)
[04:18] <wallyworld> sure, give me 5
[04:41] <babbageclunk> wallyworld: quick review? The environ interface changes we talked about. https://github.com/juju/juju/pull/7335
[04:41] <babbageclunk> wallyworld: I'll get jam to have a look at them as well, but he's having a weekend atm.
[04:42] <wallyworld> babbageclunk: ok, just finishing another one
[04:42] <babbageclunk> ta
[05:07] <wallyworld> axw: i had a question about Storage and AttachedStorage param values
[05:08] <wallyworld> what happens if these overlap/conflict
[05:10] <wallyworld> babbageclunk: just a minor quibble on wording
[05:10] <babbageclunk> wallyworld: ok, thanks
[05:12] <babbageclunk> wallyworld: yeah, model's better - I wasn't happy with database either.
[05:12] <wallyworld> \o/
[05:25] <burton-aus> wallyworld https://github.com/juju/juju/commit/64fe0a58aacc7eff08cc3684eceb0bd05a4a41b5
[05:25] <burton-aus> wallyworld just found out all oracle ci jobs failed there.
[05:26] <wallyworld> burton-aus: right, we have been asked to rename oracle-compute t oracle. nicolas knows but i accidentially left you off the notification, sorry :-(
[05:28] <burton-aus> wallyworld no worries, I will update the cloud-city for credentials.
[05:28] <wallyworld> burton-aus: thank you, appreciated
[05:36] <wallyworld> babbageclunk: her's the first of several PRs. it's on a new feature branch. maybe you can look first up on monday?
[05:36] <wallyworld> https://github.com/juju/juju/pull/7336
[05:37] <babbageclunk> wallyworld: sure.
[05:37] <babbageclunk> wallyworld: Hey, you killed off crossmodelcommon?
[05:37] <wallyworld> yeah
[05:37] <wallyworld> no longer needed
[05:37] <wallyworld> as there are no longer 2 controller facades
[05:37] <babbageclunk> wallyworld: Doh, I was just putting more things into it.
[05:37] <wallyworld> :-(
[05:38] <wallyworld> everything  is now in just the applicationoffers facade
[05:38] <babbageclunk> I'll just put the functions into the application facade until I need them from elsewhere.
[06:06] <axw> wallyworld: you want to check my responses before I merge?
[06:13] <wallyworld> axw: looking
[06:14] <wallyworld> axw: all good
[06:14] <axw> wallyworld: ta
[07:53] <rogpeppe> axw, babbageclunk: would you be able to take a look at this? it's kinda tangential, but useful. https://github.com/juju/juju/pull/7331
[07:54] <axw> rogpeppe: looking
[07:54] <rogpeppe> axw: ta
[08:01] <axw> rogpeppe: LGTM
[08:01] <rogpeppe> axw: tyvm