kelvinliuwallyworld: the review tool change is not ready yet. `confinement 'classic' not allowed with plugs/slots lint-snap-v2_confinement_classic_with_interfaces`00:46
wallyworldkelvinliu: ah, they are behind schedule. i'll email them to ask01:22
kelvinliuwallyworld: ok thx01:22
manadartNeed a forward merge review: https://github.com/juju/juju/pull/1097808:25
nammn_de1manadart 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/1096709:46
nammn_de1btw. manadart: there is no way for a controller to access another controllers collection, right?09:47
nammn_de1at least not without a lot of overhead (?)09:47
stickupkidnammn_de1, nope09:47
nammn_de1stickupkid: thanksss09:47
stickupkidnammn_de1, you'd need to write a LOT of api server/client apis09:47
nammn_de1stickupkid:  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:49
stickupkidnammn_de1, https://github.com/juju/juju/tree/develop/apiserver/facades09:51
nammn_de1stickupkid: I see, somehow overlooked this 😭09:52
hallbackInteresting 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 cases09:52
achilleasamanadart: 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 collections12:13
manadartachilleasa: OK; will look in a bit.12:13
achilleasamanadart: take your time... it's a big one :-(12:14
=== tinwood_ is now known as tinwood
nammn_de1manadart did you mean that by renaming in your pr review? https://github.com/juju/juju/blob/4ad19fc6fc3105d0d94404cedfd9be33db4f34dc/state/modelmigration.go#L861 Wasn;t 100% sure15:07
nammn_de1wanted to make sure before merging15:07
nammn_de1rick_h: what would be the ideal output of this bug be? https://bugs.launchpad.net/juju/+bug/185352915:20
nammn_de1possible options: https://pastebin.canonical.com/p/fVh89BcHQ9/15:20
mupBug #1853529: The lack of a controller is not an error <bitesize> <papercut> <ux> <juju:Triaged> <https://launchpad.net/bugs/1853529>15:20
stickupkidmanadart, when you get 5 minutes https://github.com/juju/description/pull/6716:07
stickupkidi'm going to start on external controller16:07
manadartachilleasa: 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:13
rick_hnammn_de1:  I think option 3, just "No controllers registered"16:15
nammn_de1rick_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:16
manadartnammn_de1: Yes, and rename LatestRemovedModelMigration as well. That can call CompletedMigrationForModel.16:24
stickupkidmanadart, external controllers done, I'm now going to wire them up, before landing the PRs https://github.com/juju/description/pull/6817:42
stickupkidmanadart, I'm unsure how relation id plays into this whole thing, so I may end up removing it before landing #6717:43
achilleasamanadart: 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 place17:45
=== timClicks__ is now known as timClicks
hmltimClicks__: did you get the provider instance types answer yesterday?20:24
timClickshml: not really, but I don't think that we have a lot of capacity available to implement what erik/scania would like20:33
timClickswhich is supporting centos7 images on vsphere20:34
hmltimClicks:  I only know a few, and not how it works for vsphere.20:41
hmltimClicks:  with vsphere it’s in the config file we setup i believe.20:41
hmltimClicks:  the other is aws, were we pull down the data, and parse it for our use.  usually check when we update AZ20:41
timClicksyeah, we build a custom base template image20:42
hmltimClicks: rackspace would be the same as openstack20:42
hmltimClicks: at a very high level, if the vsphere has centos7, why couldn’t we use it in the config file?20:43
timClicksI believe we could, which is why I'm probing around asking questions.20:44
hmltimClicks:  :-D20:46
hmltimClicks:  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:49
timClicksno, it doesn't20:50
timClicksor at least, this is what users see when they try to set up vsphere20:50
hmltimClicks: hrm.. the values are there… but only if you have bootstrapped vsphere.20:50
hmltimClicks: https://github.com/juju/juju/blob/develop/provider/vsphere/config.go#L1720:51
hmltimClicks: juju show-cloud --include-config <vsphere-cloud>20:52
hmlwonder why they aren’t seen?20:53
hmli don’t have a vsphere cloud defined locally to check.20:56
rick_hhml:  timClicks what do we need? I've got one now :)20:58
timClicksrick_h: I just filed a bug20:58
hmlrick_h: run the juju command above please20:58
hmltimClicks: were you bootstraped to vsphere when you ran the commands?20:58
timClicksi didn't, I copied that output from a discourse post from someone else20:59
rick_hhml:  timClicks ok, ran the command...what am I looking for?20:59
hmlrick_h: what does “The available config options specific to vsphere clouds are:”20:59
rick_hoh bummer I don't get to do that since I'm on JAAS21:00
hmltimClicks: okay, i’ll check disposable221:00
rick_hso it doesn't give me cloud config in that way21:00
hmlrick_h: hrm, not cool, can i get a pastebin of what you do see?21:00
hmlI meant I’ll check discourse21:01

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!