timClicksDocs review available for anyone with a rubber stamp https://github.com/juju/docs/pull/350703:03
timClicksActually, I'm just going to merge it in as it only brings the repo up-to-date with what's already there03:04
timClicksthumper: do you have any thoughts on the use cases for LXD should be described? with LXD clustering, I feel like saying that it's only for local development is selling it short04:03
thumperyes local development only would be selling it short04:13
thumperone use case I had thought about was using LXD cluseter to install maas04:14
thumperusing LXD clusters, you have a very lightweight cloudish thing04:14
stickupkidmanadart_, achilleasa : CR https://github.com/juju/juju/pull/1176209:02
manadart_stickupkid: I'm looking.09:10
stickupkidmanadart_, quick HO?09:44
stickupkidmanadart_, hopefully that'll fix everything in that PR10:03
manadart_stickupkid: I already approved it.10:03
achilleasamanadart_: where was that code that you were showing me? My grep-fu only seems to match a call to Subnets() in client/subnets/cache.go with an UnknownId arg10:13
manadart_achilleasa: environes/networking, environs/space ?10:14
manadart_`ReloadSpaces` is in environs/space/spaces.go. That's where the fallback to subnet discovery is.10:15
achilleasaah just found it :D10:15
achilleasaso the thing is that I can't seem to be able to get all CIDRs from the lxd api10:16
flxfoohi all,10:47
flxfooI would like to add some little things in `cloudinit-userdata` model config, but I have this error in return: The update path 'settings.' contains an empty field name, which is not allowed.10:48
achilleasastickupkid: or manadart_ simple PR for a bug https://github.com/juju/juju/pull/1176311:35
manadart_achilleasa: Will do it in a sec.11:41
achilleasamanadart_: not in a rush. If you have a few min later let's jump back into the HO to chat subnet discovery again bec I think I'm stuck11:43
manadart_achilleasa: Swap you: https://github.com/juju/juju/pull/1176411:56
=== arif-ali_ is now known as arif-ali
stickupkidhml, i'm unsure we would want a mutex in the firewaller that gates every openstack call14:52
achilleasamanadart_: clear to land 1176414:53
stickupkidhml, esp. when ensureGroups could take forever depending on the api calls14:53
achilleasacan I get a CR on https://github.com/juju/juju/pull/11765?14:54
stickupkidhml, we should be resilent against things not being there imo14:54
hmlstickupkid: rgr.  in that case.  is it appropriate for us to be removing security group rules like this?  that aren’t in our list or egress rules?14:55
stickupkidlet me check that next14:55
stickupkidhml, this worries me though https://github.com/juju/juju/pull/11756/files#diff-cc8fa9cbee78e4f349622a6a1d6d9f24R57014:55
manadart_achilleasa: Ta.14:59
achilleasastatus tests seem to be flaky15:35
stickupkidhml, I think what the code is doing is fine, we just need to handle the issue that we may end up deleting a sg that's already gone15:46
hmlstickupkid: k,  i’ll review shortly15:52
achilleasahml: can you also add https://github.com/juju/juju/pull/11765 to the review queue? it's a really small patch16:26
hmlachilleasa:  sure16:27
achilleasapetevg: you might also want to QA this before we land it ^^^16:27
petevgachilleasa: roger that.16:31
bdxhit this just now https://bugs.launchpad.net/juju/+bug/188533922:30
mupBug #1885339: unit not removed from relation <juju:New> <https://launchpad.net/bugs/1885339>22:30
bdxI remove a unit, but it never leaves the relation22:31
bdxI'm thinking this can't be intended22:31
bdxis it?22:31

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