wallyworldhpidcock: here's that huge PR for rc6 https://github.com/juju/juju/pull/1093703:20
hpidcockon it03:20
hpidcockthis is too big03:20
parlosGood Morning Juju!10:26
=== parlos_ is now known as parlos
rick_hmorning Juju party people12:27
parlos__Good Afternoon rick_h12:35
achilleasamanadart: what do you make of this? https://github.com/juju/juju/blob/develop/provider/maas/maas2_environ_whitebox_test.go#L926-L93912:39
achilleasarick_h: ^^^ is this example realistic?12:42
parlos__If a 'bug' is discovered in a charm, where should that be reported?12:47
rick_hachilleasa:  thinking, trying to make sense of it. it is that the device has multiple nics, and the children are possibly bridged addresses or something?12:48
rick_hparlos__:  charm show cs:xxxx bugs-url is the best start12:49
rick_hparlos__:  or look for the bugs url link on the jaas.ai/xxxxx page12:49
manadartachilleasa: I am up against the limits of my networking knowledge, but I can't see how this can happen. Particularly as the VLAN is on the interface = same for all links...12:54
parlos__rick_h thanks.12:55
achilleasarick_h: manadart I have no idea; strong my maas-fu is not :D12:56
achilleasaaccording to the implementation the last address wins there12:57
manadartachilleasa: You can have multiple subnets on a VLAN, but everything I am reading discourages it.12:59
parlos__Have a nice day!13:00
manadartachilleasa: And Netplan appears not to support virtual interfaces: https://bugs.launchpad.net/ubuntu/+source/nplan/+bug/174320013:06
mupBug #1743200: No support for interface labels <netplan:New> <nplan (Ubuntu):Triaged> <https://launchpad.net/bugs/1743200>13:06
=== dannf` is now known as dannf
achilleasamanadart: can you take a look at https://github.com/juju/juju/pull/10942?14:13
manadartachilleasa: Yep.14:14
nammn_demanadart: can you take a quick/first look into that to see whether I am going in the right direction? https://github.com/juju/juju/pull/10943/files Next step would be to add those to the import step as well15:19
manadartnammn_de: Looking.15:19
=== Trefex is now known as Trefex_
=== Trefex_ is now known as Trefex
nammn_dewhile at it, I would put the export step into another PR after this one got through15:46
nammn_dethe corresponding description PR https://github.com/juju/description/pull/6615:47
nammn_demanadart: thanks! I incorporated your review. description and juju/juju should be ready to be reviewed for real16:39

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