[01:38] 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] 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] sure, give me 5 [04:41] wallyworld: quick review? The environ interface changes we talked about. https://github.com/juju/juju/pull/7335 [04:41] wallyworld: I'll get jam to have a look at them as well, but he's having a weekend atm. [04:42] babbageclunk: ok, just finishing another one [04:42] ta [05:07] axw: i had a question about Storage and AttachedStorage param values [05:08] what happens if these overlap/conflict [05:10] babbageclunk: just a minor quibble on wording [05:10] wallyworld: ok, thanks [05:12] wallyworld: yeah, model's better - I wasn't happy with database either. [05:12] \o/ [05:25] wallyworld https://github.com/juju/juju/commit/64fe0a58aacc7eff08cc3684eceb0bd05a4a41b5 [05:25] wallyworld just found out all oracle ci jobs failed there. [05:26] 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] wallyworld no worries, I will update the cloud-city for credentials. [05:28] burton-aus: thank you, appreciated [05:36] 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] https://github.com/juju/juju/pull/7336 [05:37] wallyworld: sure. [05:37] wallyworld: Hey, you killed off crossmodelcommon? [05:37] yeah [05:37] no longer needed [05:37] as there are no longer 2 controller facades [05:37] wallyworld: Doh, I was just putting more things into it. [05:37] :-( [05:38] everything is now in just the applicationoffers facade [05:38] I'll just put the functions into the application facade until I need them from elsewhere. [06:06] wallyworld: you want to check my responses before I merge? [06:13] axw: looking [06:14] axw: all good [06:14] wallyworld: ta [07:53] 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] rogpeppe: looking [07:54] axw: ta [08:01] rogpeppe: LGTM [08:01] axw: tyvm === frankban|afk is now known as frankban === frankban is now known as frankban|afk