[00:46] <kelvinliu> wallyworld: the review tool change is not ready yet. `confinement 'classic' not allowed with plugs/slots lint-snap-v2_confinement_classic_with_interfaces`
[01:22] <wallyworld> kelvinliu: ah, they are behind schedule. i'll email them to ask
[01:22] <kelvinliu> wallyworld: ok thx
[08:25] <manadart> Need a forward merge review: https://github.com/juju/juju/pull/10978
[09:46] <nammn_de1> manadart stickupkid as you guys already gave it a quick look, its ready for a review. Mostly little cleanup and updated the go doc  https://github.com/juju/juju/pull/10967
[09:47] <nammn_de1> btw. manadart: there is no way for a controller to access another controllers collection, right?
[09:47] <nammn_de1> at least not without a lot of overhead (?)
[09:47] <stickupkid> nammn_de1, nope
[09:47] <nammn_de1> stickupkid: thanksss
[09:47] <stickupkid> nammn_de1, you'd need to write a LOT of api server/client apis
[09:49] <nammn_de1> stickupkid:  is there a quick way to find out from the package naming which runs on a controller and which on the units on the not controller? Or  something similiar?
[09:51] <stickupkid> nammn_de1, https://github.com/juju/juju/tree/develop/apiserver/facades
[09:52] <nammn_de1> stickupkid: I see, somehow overlooked this 😭
[09:52] <hallback> Interesting to hear about the CentOS + VMware issue, I know some people that are struggling with that, or at least would like to use it. I've used MAAS + my own CentOS images to be able to use Juju with CentOS until I figured out how to do it with LXD instead, but VMware would provide more use cases
[12:13] <achilleasa> manadart: this is the final instancepoller PR https://github.com/juju/juju/pull/10979. I have also noted my findings about the dups in the linklayer devices/addresses collections
[12:13] <manadart> achilleasa: OK; will look in a bit.
[12:14] <achilleasa> manadart: take your time... it's a big one :-(
[15:07] <nammn_de1> manadart did you mean that by renaming in your pr review? https://github.com/juju/juju/blob/4ad19fc6fc3105d0d94404cedfd9be33db4f34dc/state/modelmigration.go#L861 Wasn;t 100% sure
[15:07] <nammn_de1> wanted to make sure before merging
[15:20] <nammn_de1> rick_h: what would be the ideal output of this bug be? https://bugs.launchpad.net/juju/+bug/1853529
[15:20] <nammn_de1> possible options: https://pastebin.canonical.com/p/fVh89BcHQ9/
[15:20] <mup> Bug #1853529: The lack of a controller is not an error <bitesize> <papercut> <ux> <juju:Triaged> <https://launchpad.net/bugs/1853529>
[16:07] <stickupkid> manadart, when you get 5 minutes https://github.com/juju/description/pull/67
[16:07] <stickupkid> i'm going to start on external controller
[16:13] <manadart> achilleasa: Still NFI what is up with that. This fixes the error https://pastebin.canonical.com/p/kqVJGHFjJS/ but the test still times out waiting for the address.
[16:15] <rick_h> nammn_de1:  I think option 3, just "No controllers registered"
[16:16] <nammn_de1> rick_h: but this would mean for all the other commands then as well? E.g.
[16:16] <nammn_de1>   juju offers, juju status, juju models are all returning the same message. Do we want to change them as well?
[16:24] <manadart> nammn_de1: Yes, and rename LatestRemovedModelMigration as well. That can call CompletedMigrationForModel.
[17:42] <stickupkid> manadart, external controllers done, I'm now going to wire them up, before landing the PRs https://github.com/juju/description/pull/68
[17:43] <stickupkid> manadart, I'm unsure how relation id plays into this whole thing, so I may end up removing it before landing #67
[17:45] <achilleasa> manadart: that's odd... I will spend some time in the morn commenting out things from the linked commit to see if I can find the block that makes it fail in the first place
[20:24] <hml> timClicks__: did you get the provider instance types answer yesterday?
[20:33] <timClicks> hml: not really, but I don't think that we have a lot of capacity available to implement what erik/scania would like
[20:34] <timClicks> which is supporting centos7 images on vsphere
[20:41] <hml> timClicks:  I only know a few, and not how it works for vsphere.
[20:41] <hml> timClicks:  with vsphere it’s in the config file we setup i believe.
[20:41] <hml> timClicks:  the other is aws, were we pull down the data, and parse it for our use.  usually check when we update AZ
[20:42] <timClicks> yeah, we build a custom base template image
[20:42] <hml> timClicks: rackspace would be the same as openstack
[20:42] <timClicks> sure
[20:43] <hml> timClicks: at a very high level, if the vsphere has centos7, why couldn’t we use it in the config file?
[20:44] <timClicks> I believe we could, which is why I'm probing around asking questions.
[20:46] <hml> timClicks:  :-D
[20:49] <hml> timClicks:  related to the bug, the providers do add model config keys, at leaste the openstack provider does.  does vsphere not add the ones in your bug?
[20:50] <timClicks> no, it doesn't
[20:50] <timClicks> or at least, this is what users see when they try to set up vsphere
[20:50] <hml> timClicks: hrm.. the values are there… but only if you have bootstrapped vsphere.
[20:51] <hml> timClicks: https://github.com/juju/juju/blob/develop/provider/vsphere/config.go#L17
[20:52] <hml> timClicks: juju show-cloud --include-config <vsphere-cloud>
[20:53] <hml> wonder why they aren’t seen?
[20:56] <hml> i don’t have a vsphere cloud defined locally to check.
[20:58] <rick_h> hml:  timClicks what do we need? I've got one now :)
[20:58] <timClicks> rick_h: I just filed a bug
[20:58] <hml> rick_h: run the juju command above please
[20:58] <hml> timClicks: were you bootstraped to vsphere when you ran the commands?
[20:59] <timClicks> i didn't, I copied that output from a discourse post from someone else
[20:59] <rick_h> hml:  timClicks ok, ran the command...what am I looking for?
[20:59] <hml> rick_h: what does “The available config options specific to vsphere clouds are:”
[20:59] <hml> list
[21:00] <rick_h> oh bummer I don't get to do that since I'm on JAAS
[21:00] <hml> timClicks: okay, i’ll check disposable2
[21:00] <rick_h> so it doesn't give me cloud config in that way
[21:00] <hml> rick_h: hrm, not cool, can i get a pastebin of what you do see?
[21:01] <hml> I meant I’ll check discourse