[00:04] <bdx> well, checked my creds, vpc, subnet ids ... everything looks good, but I just can't deploy anything it seems
[00:04] <bdx> possibly I am hitting an aws quota I think
[00:05] <bdx> https://pastebin.com/QgZjHbyV
[00:36] <bdx> wallyworld_: http://paste.ubuntu.com/25531025/
[00:36] <bdx> I tried to be as direct as possible
[00:38] <bdx> let me know if you can't grok whats going on, but basically I deploy ubuntu in each of the models at the end, showing the config of the models you can see the config is similar for both models, but I can't get anything to deploy in one of them.... and for that matter, for any new model I create ... as of like less than an hour ago
[00:43] <bdx> those 2 models were created with the same creds .... so strange that the same command completes in one but fails in the other
[02:28] <wallyworld_> bdx: all those allocating machines - that means that either they have not been created by the cloud, or they have not been able to start the juju agent to phone home. you'd need to look at cloud and juju logs to dig in a bit
[02:28] <wallyworld_> axw: thanks for reviews
[02:28] <axw> wallyworld_: np
[02:52] <babbageclunk> wallyworld_: bugger, sorry!
[02:59] <babbageclunk> wallyworld_: Trying to run juju from a snap, I keep getting this: http://paste.ubuntu.com/25531660/
[03:00] <babbageclunk> The snap was installed with --classic.
[03:00] <babbageclunk> wallyworld_: any ideas?
[03:08] <wallyworld_> babbageclunk: nfi, haven't seen that before
[03:08] <wallyworld_> veebers: looks like lnding is borked http://ci.jujucharms.com/blue/organizations/jenkins/github-merge-juju/detail/github-merge-juju/243/pipeline
[03:18] <babbageclunk> wallyworld_: just needed to chown ~/snap - not sure how it ended up owned by root.
[03:18] <wallyworld_> hmm
[03:51] <veebers> wallyworld_: I just re-ran it, lets see what happens
[04:18] <babbageclunk> thumper: hey, should I be doing official builds on develop or 2.2? develop right?
[04:37] <thumper> babbageclunk: develop
[04:37] <babbageclunk> thumper: thanks, that's what I'd guessed but figured I'd check early just in case I needed to move it.
[04:37]  * thumper nods
[07:55] <axw> wallyworld_: are you free to review a little-ish PR? https://github.com/juju/juju/pull/7855
[07:55] <axw> there's more coming soon
[08:54] <wallyworld_> axw: sure, just have to go get lachie from school, will do it as soon as i'm back
[08:55] <wallyworld_> sorry i missed the ping before
[08:55] <axw> wallyworld_: it can wait till tomorrow, I'll just put up the other PR rebased on top
[08:55] <wallyworld_> ok, i'll look at both
[23:13] <thumper> It looks like I have added a potential bomb in the processing of bundle options...
[23:13] <thumper> only in the --bundle-config file, but it is there none the less
[23:13]  * thumper will fix...
[23:14] <thumper> oh no...
[23:14] <thumper> it's all good
[23:14] <thumper> phew
[23:14] <thumper> bundle config is processed before bundle.Verify
[23:14] <thumper> and bundle.Verify ensures the options are valid for the charm option types
[23:14] <thumper> yay
[23:15] <thumper> well fuck
[23:15]  * thumper is stunned
[23:16] <veebers> thumper: this really is an emotional rollercoaster
[23:16] <thumper> :)
[23:16] <thumper> https://github.com/juju/charm/blob/v6-unstable/config.go#L62
[23:16] <thumper> this function returns a value and an error
[23:16] <thumper> but if it gets a type it doesn't understand, does it return an error? no it panics
[23:16] <thumper> FFS
[23:17] <thumper> why?
[23:17] <thumper> at least I haven't made it worse,
[23:17] <veebers> thumper: so the comments are lying "// returns an error if it cannot be parsed to the correct type."
[23:18] <thumper> yeah
[23:18] <thumper> comments lie
[23:18] <thumper> this code is all 4-6 years old
[23:22] <babbageclunk> thumper: gross.
[23:24] <babbageclunk> thumper: maybe the author decided to panic because of the defer option.error call? The wrapping it does doesn't make sense for an invalid type. (Not that that's a good reason.)
[23:29] <thumper> babbageclunk: you could format the error so it still makes sense...
[23:29] <thumper> geez
[23:29]  * thumper adds a bug
[23:29] <babbageclunk> thumper: oh sure