wallyworldtlm: or hpidcock: or kelvinliu: a small constraints PR https://github.com/juju/juju/pull/1202004:20
hpidcockwallyworld: looking04:27
kelvinliuwallyworld: got this pr for adding priority and priority, could u take a look plz https://github.com/juju/juju/pull/12022  thanks04:45
wallyworldi cam04:46
kelvinliualso tested updateStrategy04:46
wallyworldkelvinliu: lgtm, tyvm04:49
kelvinliuwallyworld: as i can see, i tested updatestrategy working as expected, im not sure why cory got that parsing error, probably he wasn't using 2.8+?04:51
wallyworldkelvinliu: yeah, i believe he was using 2.8.1 or something04:51
kelvinliutlm: https://github.com/juju/juju/pull/12015 got this pr for caas firewaller worker for embedded apps, there are a few todos in the pr will be implemented in following prs, could u take a look plz? thanks05:14
wallyworldkelvinliu: lgtm with a question about the api calls05:41
kelvinliuwallyworld: right, I will change the CharmInfo(url string) to CharmInfo(appName string)05:44
kelvinliuthx for reviewing05:46
taohansenI'm trying to wrap my head around Juju: is Juju Charmed Kubernetes? Why would I want Juju over microk8s? Juju can consume microk8s to make a Juju cloud but I don't know specifically why I would want to do this. "A cloud is a target that Juju knows how to manage workloads for. " But Kubernetes is already a tool for managing workloads across pods and nodes.08:02
taohansenIt looks like Charms are equivalent to Helm Charts. Is this a fair assumption?08:02
iceythat's a new one: `Create instance: Invalid devices: Device validation failed "eth1": Invalid value for device option "mtu": Out of MTU range (1280-9202) "9214"`08:24
stickupkidmanadart, fixes the documentation issue https://github.com/juju/juju/pull/1202310:03
manadartstickupkid: Nice.10:04
stickupkidnot like me to miss documentation :|10:04
achilleasastickupkid: yet another one... cursed yaml! https://github.com/juju/bundlechanges/pull/7010:30
achilleasastickupkid: small fix for an NPE in charm that I discovered while testing: https://github.com/juju/charm/pull/32010:46
manadartstickupkid: Small one - https://github.com/juju/juju/pull/12024. achilleasa, having trouble with my Microstack; wanted to get this one up before a reboot. Be back on your review presently.10:55
achilleasamanadart: did you get it to work out of the box with snap?10:56
achilleasaI had to use the edge with --devmode and add an ip to get it to init10:56
stickupkidachilleasa, https://github.com/juju/juju/pull/11253/files#diff-a86032d7aeab77815b81f23cdfb4692110:57
achilleasastickupkid: I also had to "ip add X" to allow it to find rabbit10:58
manadartachilleasa: Yeah, I had the same issue. No change from when it was last working, but I lost the device addresses.11:11
manadartThen I hung on configuring the bootstrap machine...11:12
manadartBack up all fresh, see how it goes.11:12
achilleasamanadart: if you set up spaces and stuff please also try the last step in the QA (moving subnets)11:16
manadartachilleasa: Yeah, I was also going to delete a space. How would that go?11:17
achilleasait should also work because the space-id in the subnets collection will change11:22
achilleasaand it will trigger a refresh of the rules11:22
achilleasastickupkid: just pushed the PR for including expose settings in diff-bundle output. If you 've got some spare time can you take a look at https://github.com/juju/juju/pull/12025?12:44
stickupkidwon't be till after standup tbh12:44
achilleasanot in a rush. just wanted to get all remaining chunks of work up for review today as next week I won't be able to work on this12:45
achilleasathat's the last PR though. Now I need to write a discourse post about it :D12:46
stickupkidmanadart, fixes your issue with acceptance tests https://github.com/juju/juju/pull/1202613:29
jamQuick review of 2.8 into develop ? https://github.com/juju/juju/pull/1202713:44
=== kirkland is now known as Guest82000
stickupkidhml, so I've gone back to juju 2.0 and .Update has never been used to set a charm 4ddfc7da4fa771621e5f2a87492e1c6e6f06867514:35
stickupkidhml, so it was probably used for something, but not sure what tbh14:36
stickupkidhml, maybe future planning?14:36
=== tamas_erdei is now known as terdei
hmlstickupkid:  no idea‚Ķ. maybe it was a juju 1 ism?14:40
stickupkidhml, I just wanted to be sure, but we're all good14:40
hmlstickupkid:  +114:41
hmlstickupkid:  have we done any upgrade juju testing on the charm origins?14:47
stickupkidhml, not for a long time14:47
stickupkidhml, we should do that14:47
hmlstickupkid:  same thoughts14:48
stickupkidhml, in theory https://github.com/juju/juju/blob/develop/state/upgrades.go#L310814:49

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