[00:16] veebers: any idea what's up in http://ci.jujucharms.com/job/github-merge-goose/56/console? /tmp/jenkins7796601857268642679.sh: line 23: /var/lib/jenkins/juju-ci-tools/git_gate.py: No such file or directory [00:16] veebers: more to the point, what's the proper way to fix? [00:28] axw: hey I'll have a look now [00:32] axw: that node was aggressively cleaned up, a little too aggressively. I'm righting it now [00:39] axw: right, that's now running and succeeded. Sorry for the noise [00:41] axw: pr updated back to using map[string]string. verified with a float and a bool [01:07] veebers: thanks [01:08] veebers: is it a manual step to get the juju-ci-tools bits on there? [01:14] axw: for the initial setup yes, but then there is a script that keeps things up to date. This slave in particular had some manual intevention recently (and is a bit of a snow flake) [01:15] veebers: okey dokey, thanks [01:32] axw: thanks for review. i wanted the "preserveType" value to, by default, retain existing beheaviour, ie the yaml unmarshalling. "preserveType" means do not mess with the values' types by unmarshalling [01:34] "setDecodedYAMLValues" would need to be explictly set to true for all the current use cases [01:34] wallyworld: that seems to be the opposite of what hte doc comment says hto [01:34] the comment makes sense to me i guess. maybe i can reword it [01:35] i can explain what true/false means [01:35] wallyworld: "... sets whether name values should be converted to a type that is inferred from their string value" <- they're inferred by default, so it's setting them to *not* be inferred [01:35] wallyworld: perhaps "SetPreserveStringValue" [01:36] they are inferred by the default - the comment just says that the value of the attr will controll the behaviour of it. "SetPreserveStringValue" works for me [02:26] wallyworld: take a look at https://github.com/juju/juju/pull/8210 please? [02:30] wallyworld: you’re popular, though babbageclunk got there first, no rush, but if you could review please: https://github.com/juju/juju/pull/8211 [02:30] g’night [03:54] babbageclunk: sure, looking [03:54] wallyworld: thanks [03:54] sorry i missed the ping [03:54] axw: here's the next PR in the series https://github.com/juju/juju/pull/8212 [03:59] axw: lgtm. i can see we will be asked to allow this to be set on an existing controller. regardless, have you considered upgrade steps? will thingsbreak if we run edge on a 2.2.6 controller [04:00] i guess not because everything has a default [04:00] but worth checking [04:00] oops [04:00] babbageclunk: ^^^^^ [04:00] sorry, wrong person [04:00] you mean me right/ [04:00] poor andrew, i didn't mean to insult him :-P [04:01] ha [04:01] :-D [04:01] I've thought through all of the new ones to make sure they work if there's no value, but I should try it too. [04:05] wallyworld: time for a quick chat about the problem I've been talking to blahdeblah about? [04:05] in 1:1? [04:05] sure, give me 1 minute [06:48] axw: thanks for review. about to test the final piece to make it all work, and then need to add more tests, but really close now [06:49] wallyworld: cool :) [06:49] balloons: in case you get a bunch of failures in CI, I just deleted all the resource groups. there were a bunch of old ones, and new jobs were failing because the resource quota was hit for public IPs [06:50] axw: there's a vspherebug that is confusing - it tries to connect to an /sdk and the ip addr is something not sure of where it came from [06:50] balloons: in case it helps for automated cleanups, I did this: az group list --query "[?starts_with(name, 'juju-')].name" -o tsv | xargs -L1 az group delete -y --no-wait --name [06:50] did you see that float past? have you see anything similar before? [06:50] wallyworld: I haven't seen the bug [06:51] https://bugs.launchpad.net/bugs/1737868 [06:51] Bug #1737868: Can't bootstrap with vsphere 6.0 [06:52] wallyworld: I believe govmomi tacks "/sdk" on the end [06:52] I'll take a look and see if anything jumps out [06:52] ok, ty [08:04] axw: here's the last bit if you had time https://github.com/juju/juju/pull/8214 i'm still going through manual testing, will doube check everything before landing [08:04] got to make dinner, bbiab === frankban|afk is now known as frankban === chrome0_ is now known as chrome0 [11:39] anyone have ideas on how we could test https://github.com/juju/juju/pull/8216 ? [14:08] Is there a method somewhere that filters a host/port collection by network space subnets? [14:14] NVM network/address.go [14:27] axw, thanks for the cleanups [17:09] how's it going all [17:09] https://bugs.launchpad.net/juju/+bug/1736022 is killing me [17:09] Bug #1736022: failed to bridge devices: bridge activaction error: bridge activation failed: Killed old client process [17:10] I'm willing to just hand over my maas to you guys so you can pound on it and get what you need to fix it [17:12] if anyone has a few extra minutes to look into this^ further with me it would be greatly appreciated [17:12] thx thx [17:22] oooh shoot [17:23] I think my bug #1736022 [17:23] Bug #1736022: failed to bridge devices: bridge activaction error: bridge activation failed: Killed old client process [17:23] is really [17:23] https://bugs.launchpad.net/juju/+bug/1737640 [17:23] Bug #1737640: /usr/sbin/fanctl: arithmetic expression: expecting primary | unconfigured interfaces cause ifup failures

smb> [19:53] hml: I'm going to be a few minutes late, dropping kiddo in town [19:53] thumper: ack [20:05] hml: here now [21:33] wallyworld: ping [21:50] thumper: hey [21:50] wallyworld: hey [21:51] quick chat? [21:51] sure, give me 2 minutes [21:53] thumper: can we bail tonight? we're getting snowed in and I need to finish this round of shoveling [21:53] rick_h: sure [21:53] rick_h: have a good holiday break [21:53] thumper: 1:1? [21:53] thumper: I don't have anything atm, just doing stuff as folks start to disappear for the year [21:53] rick_h: I'm off after tomorrow [21:53] thumper: you too [21:53] wallyworld: ack [21:53] thumper: yea, common these days :P [22:51] so wallyworld, shall we meet? I have just 2 things to ask about [22:52] balloons: sure, wanna do it now? [22:52] wallyworld, sure