wallyworld | hpidcock: here's that huge PR for rc6 https://github.com/juju/juju/pull/10937 | 03:20 |
---|---|---|
hpidcock | on it | 03:20 |
hpidcock | uh | 03:20 |
hpidcock | this is too big | 03:20 |
wallyworld | ty | 03:23 |
hpidcock | np | 03:23 |
parlos | Good Morning Juju! | 10:26 |
=== parlos_ is now known as parlos | ||
rick_h | morning Juju party people | 12:27 |
parlos__ | Good Afternoon rick_h | 12:35 |
achilleasa | manadart: what do you make of this? https://github.com/juju/juju/blob/develop/provider/maas/maas2_environ_whitebox_test.go#L926-L939 | 12:39 |
achilleasa | rick_h: ^^^ is this example realistic? | 12:42 |
parlos__ | If a 'bug' is discovered in a charm, where should that be reported? | 12:47 |
rick_h | achilleasa: 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_h | parlos__: charm show cs:xxxx bugs-url is the best start | 12:49 |
rick_h | parlos__: or look for the bugs url link on the jaas.ai/xxxxx page | 12:49 |
manadart | achilleasa: 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 |
achilleasa | rick_h: manadart I have no idea; strong my maas-fu is not :D | 12:56 |
achilleasa | according to the implementation the last address wins there | 12:57 |
manadart | achilleasa: You can have multiple subnets on a VLAN, but everything I am reading discourages it. | 12:59 |
parlos__ | Have a nice day! | 13:00 |
manadart | achilleasa: And Netplan appears not to support virtual interfaces: https://bugs.launchpad.net/ubuntu/+source/nplan/+bug/1743200 | 13:06 |
mup | Bug #1743200: No support for interface labels <netplan:New> <nplan (Ubuntu):Triaged> <https://launchpad.net/bugs/1743200> | 13:06 |
=== dannf` is now known as dannf | ||
achilleasa | manadart: can you take a look at https://github.com/juju/juju/pull/10942? | 14:13 |
manadart | achilleasa: Yep. | 14:14 |
nammn_de | manadart: 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 well | 15:19 |
manadart | nammn_de: Looking. | 15:19 |
=== Trefex is now known as Trefex_ | ||
=== Trefex_ is now known as Trefex | ||
nammn_de | while at it, I would put the export step into another PR after this one got through | 15:46 |
nammn_de | the corresponding description PR https://github.com/juju/description/pull/66 | 15:47 |
nammn_de | manadart: thanks! I incorporated your review. description and juju/juju should be ready to be reviewed for real | 16:39 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!