[09:27] achilleasa: Forward merge of the gateway patch - https://github.com/juju/juju/pull/11817 [09:28] manadart_: looking [11:01] achilleasa, manadart_ CR https://github.com/juju/description/pull/83 [11:09] stickupkid: got a question around typed nils [11:09] achilleasa, no idea, i copied and pasted [11:09] I did wonder about that [11:13] I don't mind removing it, it's just seems convention, similarly to the HORRID underscore usage in the variable names [11:13] @achilleasa, ^ [11:45] hml, this wasn't as painful as I was expecting https://github.com/juju/juju/pull/11818 [11:46] stickupkid: nice. [11:46] stickupkid: i have one up for you when you’re back [11:46] stickupkid: https://github.com/juju/juju/pull/11816 [13:35] achilleasa: Real small address additions - https://github.com/juju/juju/pull/11819 [13:51] hml, so the reason for the storeurl is for json and yaml (real) output [13:51] hml, as a link back to original content [13:51] hml, this isn't the download url [13:53] stickupkid: okay on one, you lost me on 3. HO? [13:53] sure [14:34] achilleasa: https://github.com/juju/juju/pull/11820 merges that one forward. [14:35] hml: can you point me to an example for using a pre-upgrade step? [14:36] hml, if we don't find the series in metadata yaml, wat do? [14:36] achilleasa: sure, let me find it [14:40] achilleasa: https://github.com/juju/juju/pull/11542 - it worked due to environs depending on your reasons [14:43] hml: thanks! it's exactly what I need as the pre-upgrade step is provider-specific [14:44] achilleasa: sweet! [14:49] hml, considering this metadata doesn't contain series information... https://api.snapcraft.io/v2/charms/find [14:50] stickupkid: it’s not real metadata. had the same issue with info. put it in test only [14:50] hml: on the other hand I could make it a normal upgrade step I guess; we should probably refresh the subnets each time we upgrade anyway... [14:56] hml, updated my PR [14:56] stickupkid: ack