[00:01] <redir> redir, yes empty string
[00:08] <redir> part 1 https://github.com/go-amz/amz/pull/72 PTAL
[00:20] <redir> anyone have a minute to look at something with me?
[00:24]  * thumper heading up to the cafe
[00:30]  * redir heads to GOSF to look at some talks. Most specifically one about https://github.com/ekanite/ekanite because... What if we just used a raft log to persist state? Where'd the logs go?
[00:32] <redir> also there's this: https://github.com/reedobrien/juju/tree/lp/1612645%2B1634289_new-aws-regions
[00:33] <redir> but it fails because: cannot write file "provider-state" to control bucket: The specified bucket does not exist
[00:33] <redir> so I must have missed something
[00:33] <redir> in adding the regions, or something else is wrong
[00:33] <redir> later
[02:19] <redir> anyone have hany thoughts about why the control bucjet doesn't exist?
[02:20] <wallyworld> context?
[02:21] <wallyworld> menn0: have you had a chance to take another look at that commit with my latest changes to fix import ordering etc?
[02:22] <redir> wallyworld: added new ohio region to amz and 1.25 provider and tried to deploy
[02:22] <redir> got the error 6 lines up
[02:22] <wallyworld> 6 lines up from where?
[02:22] <redir> wondering if I missed something obvious,
[02:22] <menn0> wallyworld: sorry, I haven't looked yet, otp but will look after
[02:22] <redir> this channel
[02:22] <wallyworld> menn0: np, ty
[02:23] <wallyworld> redir: not 6 lines for me, i have other notifications turned on, more like 20 and it's scroll off my screen :-)
[02:23] <redir> wallyworld: this channel at 17:33:07
[02:23] <wallyworld> redir: did your environment.yaml have a control bucket? it shouldn't
[02:23]  * redir looks
[02:23] <wallyworld> also not 17:33 more me?
[02:24] <wallyworld> :-)
[02:25] <redir> hehe
[02:26] <redir> 33:07 after the hours two hours ago
[02:26] <redir> no control-bucket in environments.yaml
[02:28] <wallyworld> redir: can you provide debug logs? aws is expected to create a controller bucker using a self generated bucket name from memory
[02:29] <redir> wallyworld: I'll work on it tomorrow
[02:29] <wallyworld> it's been 2 years since i saw that code, so will have to go digging
[02:29] <redir> it only fails in US East new region. The AP South one I added works
[02:29] <wallyworld> hmmm, interesting
[02:29]  * redir guesses it assumes the wrong things for useast
[02:29] <wallyworld> maybe that new rwgion doesn't have s3 support yet
[02:29] <wallyworld> or the s3 endpoints are wrong
[02:30] <wallyworld> doesn't sounds like a juju issue
[02:30] <wallyworld> if it's just the one region. more likely a go-amz issue
[02:30] <redir> it says it does, but not at s3.a....com it needs to be at s3.us-east-2....com
[02:31] <wallyworld> ok, ok, so you'll need to add update go-amz to ensure it has the right endpoints
[02:31] <redir> wallyworld: right. My guess too, I'll dig in to goamz tomorrow
[02:31] <wallyworld> ok
[02:32] <redir> thanks wallyworld much appreciated. Wanted to make sure it wasn't something simple
[02:32] <wallyworld> redir: tis ok, i didn't tell you much except propose a guess or two :-)
[02:34] <redir> wallyworld: sometimes it's nice when the rubberduck squeaks:)
[02:34] <wallyworld> depends on how hard you squeeze it :-)
[02:34] <wallyworld> normally I quack
[02:35] <redir> :)
[02:57] <menn0> wallyworld: ship it
[02:58] <wallyworld> menn0: tyvvvvvvm for looking
[02:58] <menn0> wallyworld: opening the diff nearly killed my browser
[02:58] <wallyworld> menn0: another reson why github reviews suck
[02:58] <wallyworld> with rb, you get a paginated list of files
[02:59] <wallyworld> i don't get how people navigate gh reviews
[02:59] <wallyworld> without pulling their hair out
[03:04] <menn0> wallyworld: I just had a quick play with the gerrit instance for the chromium project. it's pretty nice, if intimidating at first.
[03:04] <menn0> wallyworld: knobs and buttons for everything
[03:04] <menn0> wallyworld: but fast and has good keyboard shortcuts for navigation.
[03:04] <wallyworld> menn0: would love to seriously consider that
[03:05] <wallyworld> but people need to get over the NIH syndrome
[03:05] <wallyworld> which I think will be impossible for our group :-(
[03:07] <menn0> wallyworld: I don't know about that. I can see why some people are attracted to GH reviews.
[03:08] <wallyworld> i am having trouble. they are really hard to navigate (all one big page), and so much wasted space
[03:08] <wallyworld> and no way to manage comments
[03:09] <wallyworld> the main argument seems to be "well it might get better sometime"
[03:09] <wallyworld> but it's crap *now*
[03:11] <thumper> w00t
[03:11] <thumper> wow
[03:11] <thumper> that looks really out of place now
[03:11] <thumper> my view had this as the last comment:
 yay vsphere provider working correctly
[03:11] <thumper> then I scrolled down
[03:11] <thumper> heh
[03:36] <redir> amz bits ready for review: PTAL https://github.com/go-amz/amz/pull/72
[03:40] <alexisb> wallyworld, if you are still around can you provide a review for redir ^^
[03:41] <wallyworld> can do in a minute
[03:41] <alexisb> wallyworld, thank you!
[03:41] <redir> ekanite looks pretty nice. A syslog server that accepts msgs over UDP/TCP [+TLS] parses RFC5424 headers, does full text indexing, and manages retention
[03:42] <wallyworld> redir: did you sort the s3 issue?
[03:42] <redir> also sorts by log timestamp rather than the order received
[03:42] <redir> wallyworld: yup
[03:42] <wallyworld> greta
[03:42] <wallyworld> great
[03:42] <redir> and will have a juju 1.25 PR when the AMZ bits are merged
[03:42] <redir> wallyworld: ^^
[03:43] <redir> oh and the ekanite binary is 12MB
[03:44] <wallyworld> redir: did you researc that us-east-2 requires the region bools to be true ie bucket lower case etc?
[03:44] <redir> wallyworld: that fixed the issue
[03:44] <redir> and US East is the only region that doesn't
[03:44] <redir> in amz
[03:45] <redir> us-east-1
[03:45] <wallyworld> interesting
[03:46] <wallyworld> redir: lgtm
[03:46] <redir> See https://github.com/reedobrien/amz/blob/2eab8e64ed1675ae196f785bbc2d293e45aaf7a1/aws/aws.go
[03:46] <redir> everything else is not false.... so looks like they were lax on the original region
[03:46] <redir> and changed hteir minds for everything else
[03:47] <redir> wallyworld: my guess it that since bucket can be used in the website url they wanted to make it consisten with case insensitivity...
[03:48] <redir> after the original s3 in 2006 was let out into the wild
[03:48] <wallyworld> sounds plausible yeah
[03:48] <redir> can we merge amz?
[03:48] <redir> does all the $$merge$$ stuff work there?
[03:49] <wallyworld> should do
[03:54] <redir> k. tx
[08:21] <dimitern> dooferlad, frobware, macgreagoir: I'd appreciate a review on https://github.com/juju/juju/pull/6481 fixing bug 1616048 ta!
[08:21] <mup> Bug #1616048: Create interface for ofono <snapd-interface> <snapd (Ubuntu):In Progress by alfonsosanchezbeato> <https://launchpad.net/bugs/1616048>
[08:22] <dimitern> oops - bug 1616098
[08:22] <mup> Bug #1616098: Juju 2.0 uses random IP for 'PUBLIC-ADDRESS' with MAAS 2.0 <4010> <cpec> <juju:In Progress by dimitern> <https://launchpad.net/bugs/1616098>
[08:22] <frobware> dimitern: will do - otp
[08:22] <dimitern> frobware: no rush
[08:22] <dimitern> ;)
[10:39]  * dimitern steps out for ~1h
[12:28] <dimitern> frobware: ping
[12:31] <dimitern> macgreagoir: hey
[12:31] <dimitern> macgreagoir: got some time for a review? :)
[12:33] <anastasiamac> dimitern: I think ur PR do not have a check run coz they r awesome :) m sure that's what ballons coded in the script... "if dimitern > don't check" :)
[12:33] <dimitern> anastasiamac: :D
[12:33] <dimitern> anastasiamac: that's a nicer way of looking at it, sure ;)
[12:34] <dimitern> anastasiamac: I'd appreciate a review as well hehe
[12:38] <anastasiamac> dimitern: i'd love to review but m furiously working to remove myself from keyboard: it's 11pm \o/ if it can wait til my morning -m happy to do it then :D
[12:38] <dimitern> anastasiamac: uuh you should definitely get some rest!
[12:39] <anastasiamac> dimitern: \o/ will soon :D thank you for care and concern :)
[12:40] <dimitern> anastasiamac: any time ;)
[12:45] <dimitern> rick_h_: ping
[12:46] <rick_h_> dimitern: pong
[12:46] <dimitern> rick_h_: I thought about picking up bug 1580501 tagged 1.25 .. it seems it's only related to 2.0 though
[12:46] <mup> Bug #1580501: cloudimg-base-url parameters not in Juju2 anymore <4010> <cpe-sa> <orangebox> <sts> <juju:Triaged> <https://launchpad.net/bugs/1580501>
[12:46] <rick_h_> dimitern: please stick with cards on the board
[12:47] <dimitern> rick_h_: that's the first one on the board :)
[12:47] <dimitern> rick_h_: top left
[12:48] <rick_h_> oh wtf, the title of the bug and the card got out of wack
[12:48] <rick_h_> dimitern: ugh, ok this is a can of worms because this falls under work planned for the lxd image cache stuff, and yes is 2.0 only
[12:48] <rick_h_> dimitern: so going to yank that one off the board
[12:49] <dimitern> rick_h_: +1 I'll look into bug 1589680 then
[12:49] <rick_h_> dimitern: one sec, let me see if I did bad copy/paste voodoo
[12:49] <mup> Bug #1589680: Upgrading to cloud-archive:mitaka breaks lxc creation <canonical-bootstack> <juju-core:Triaged> <juju-core 1.25:Triaged by rharding> <https://launchpad.net/bugs/1589680>
[12:49] <rick_h_> dimitern: https://bugs.launchpad.net/juju-core/+bug/1560487
[12:49] <mup> Bug #1560487: local provider fails to create lxc container from template <canonical-is> <local-provider> <juju-core:Won't Fix> <juju-core 1.25:Triaged by alexis-bruemmer> <OPNFV:New> <https://launchpad.net/bugs/1560487>
[12:49] <rick_h_> dimitern: I pasted the wrong bug number to the card, my bad
[12:50] <rick_h_> dimitern: apologies for the confusion
[12:50] <dimitern> rick_h_: right :) np - do you want me to look into that last one instead?
[12:50] <rick_h_> dimitern: either one is ok thank you
[12:51] <dimitern> rick_h_: ok - 1589680 it is then
[12:52]  * dimitern dusts off his 1.25 branch .. it's been a while
[12:52] <mgz> hm, we're not doing staging and such like with 1.25 I presume?
[12:52] <rick_h_> mgz: no, we're not
[12:53] <dimitern> it's free for all :D
[12:55] <rick_h_> frobware: ping, can I grab a sec when you're available?
[13:04] <macgreagoir> dimitern: You nean 6481? Sorry, I started and got distracted by my own :-)
[13:10] <rick_h_> dimitern: I'd expect that we can just upgrade the lxc in order to find a path forward
[13:10] <rick_h_> dimitern: so if we deploy something on trusty/default lxc and it's pre-2.0, then manually add the PPA/upgrade lxc on there...how would Juju know to use/follow newer lxc patterns
[13:11] <dimitern> macgreagoir: np, if you can have a look at it - great!
[13:12] <dimitern> rick_h_: I'd expect setting openstack-origin to cloud-archive:mitaka will trigger adding the repo and the lxc upgrade..
[13:12] <dimitern> will know soon anyway..
[13:13] <rick_h_> dimitern: right, but my point is you can do this with the ubuntu charm
[13:13] <rick_h_> dimitern: no need for anything openstack
[13:13] <rick_h_> dimitern: fewest moving parts/complications the better
[13:13] <dimitern> rick_h_: yeah, I guess so.. double checking c-a:m's lxc version..
[13:13] <rick_h_> dimitern: just check the version in trusty vs the lxc ppa for trusty
[13:14] <rick_h_> dimitern: if you can get a pre-2.0 in default trusty and then get post 2.0 via the ppa you can "do the upgrade" and chase down what we need to help update to make it work
[13:16] <dimitern> yeah: lxc	2.0.5-0ubuntu1~ubuntu16.04.1~cloud0 is in cloud-archive:mitaka
[13:16] <dimitern> and lxc is not there in c-a:icehouse
[13:17] <chrome0> dimitern : afaicr the error was when going from c-a:trusty-liberty -> c-a:trusty-mitaka
[13:18] <chrome0> And orig. install was plain trusty, sans c-a
[13:18] <dimitern> chrome0: yeah, I'm trying exactly that now :)
[13:18] <chrome0> +1
[13:19] <dimitern> rick_h_: post 2.0 lxc won't work with 1.25 AIUI
[13:19] <rick_h_> dimitern: can you hop in the standup real quick?
[13:19] <dimitern> owm
[13:23] <frobware> rick_h_: yep, back
[13:24] <rick_h_> frobware: meet you in the standup room please?
[13:24] <frobware> rick_h_: omw
[13:28] <SimonKLB> is there a good way to find out when a relation is completely removed?
[13:28] <SimonKLB> i find it really hard to test adding and removing relations
[13:32] <SimonKLB> simply looking at the departed/broken hooks doesnt seem to be enough to determine the actual status of the relation in juju
[13:34] <rick_h_> natefinch: ping for chat
[13:36] <natefinch> rick_h_: oops coming
[14:00] <rick_h_> frobware: dimitern natefinch ping for standup
[14:41] <mgz> dooferlad: yo, free for bothering?
[14:45] <frobware> rick_h_: I think we have a nice conclusion for the openstack bug
[14:45] <frobware> rick_h_: https://bugs.launchpad.net/juju/+bug/1621590/comments/18
[14:45] <mup> Bug #1621590: openstack provider ignores a properly created bootstrap machine <cpec> <rteam> <juju:In Progress by frobware> <https://launchpad.net/bugs/1621590>
[14:47] <frobware> dimitern: now looking at your PR :)
[14:50] <redir> mgz: yt?
[14:50] <dimitern> frobware: sweet! :) thanks!
[14:51] <redir> mgz: can you $$merge$$ this https://github.com/go-amz/amz/pull/72 or add me to the right group, or tell me who to ask to get added to the right group? Please:)
[14:51] <rick_h_> dooferlad: ping
[14:52] <mgz> redir: sure, I'll take a look
[14:52] <dooferlad> rick_h_: hi
[14:52] <dooferlad> mgz: sorry, still talking to Mick
[14:52] <rick_h_> dooferlad: heads up, assigned another bug your way. It's another config param getting ignored like your current one.
[14:52] <dooferlad> rick_h_: ok, thanks
[14:52] <rick_h_> dooferlad: the possible idea is that it's a more generic problem and that two birds/one stone and such
[14:52] <rick_h_> dooferlad: and it was brought up on the cross team call by the stakeholders there that it should be replicatable with canonistack
[14:53] <rick_h_> dooferlad: so if getting OS goign is a burden please dump that and run with an already running OS
[14:53] <mgz> redir: heh, that's a group I'm not actually owner on, but I can trigger the merge for you
[14:53] <rick_h_> dooferlad: bah, LP won't let me change the assigner atm, timing out. https://bugs.launchpad.net/juju/+bug/1614239
[14:53] <mup> Bug #1614239: bootstrap-timeout ignored in --config <landscape> <juju:Triaged by rharding> <https://launchpad.net/bugs/1614239>
[14:59] <mgz> redir: merged as 7754380
[15:06] <frobware> dimitern: what were the rules on 500 lines of diff? You're dangerously close. :)
[15:07] <dimitern> frobware: I really tried to minimize the changes :/
[15:08] <dimitern> frobware: the only "luxuries" are a few added tests, but the refactoring of the common code / tests *did* reduce the original diff by a 100 lines :)
[15:10] <frobware> dimitern: have time to HO?
[15:11] <dimitern> frobware: yeah
[15:12] <dimitern> frobware: I'm in 'core'
[15:13] <frobware> dimitern: omw
[15:15] <mgz> dooferlad: so... free now?
[15:20] <redir> mgz: thanks
[15:20] <redir> mgz you know who the owner is?
[15:22] <mgz> redir: canonical-is and niemeyer
[15:22] <mgz> I don't know who has access to the canonical-is role account
[15:26] <dooferlad> mgz: sorry - have a headache so bad I want to be sick. Can we talk tomorrow? Perhaps invite me to a meeting so we have a slot booked?
[15:26] <mgz> dooferlad: no worries, I'll set something up for tomorrow
[15:26] <dooferlad> mgz: thanks - much appreciated
[15:26]  * dooferlad goes to hide in a dark room
[15:33] <redir> another PR ready for review; https://github.com/juju/juju/pull/6483 PTAL
[15:33] <redir> bbiab after dentist
[15:37] <SimonKLB> hello, anyone got some time guiding me on how to test adding and removing relations?
[15:38] <SimonKLB> i was using the relation function amulet to test wether or not the relation was sucessfully removed, and that worked, but now it doesnt anymore
[15:38] <SimonKLB> would really appreciate some help!
[15:42] <katco> dimitern: frobware: hey, is this robust? i.e. will it do the right thing in complicated network setups? https://github.com/juju/juju/blob/staging/environs/manual/addresses.go#L17
[15:46] <redir> abentley: was that update for Ohio because of the amz release that happened?
[15:46] <redir> this morning
[15:47] <abentley> redir: No, it was because AWS announced the region on the 17.
[15:48] <redir> i see, thanks abentley
[16:14] <redir> anyone, PTAL https://github.com/juju/juju/pull/6483
[16:37] <hoenir> axw, katco, https://github.com/juju/juju/pull/6414
[18:11] <redir> ls
[18:11] <redir> whoops
[18:45] <natefinch> ahh the glorious feeling when you've gotten down in the weeds typing code and you finally get it back to the state where it'll gofmt.
[18:46] <perrito666> looooool
[19:02]  * redir lunches
[19:10] <natefinch> made a new struct that helps you ask the user repeated questions.... called it Pollster.  Figured it was appropriate given current events.
[19:12] <redir> As long as it produces output that isn't actually relevant to anyone, natefinch
[19:13] <redir> and diferent results fo rthe same input by different users
[19:14] <natefinch> heh
[19:52] <natefinch> rick_h_: http://pastebin.ubuntu.com/23355482/
[19:53] <rick_h_> natefinch: coolio
[19:54] <natefinch> rick_h_: so, one reason to use add-cloud for ec2 etc ..... it's the only way to save a cloud-specific config
[19:54] <natefinch> rick_h_: instead of haviong to do bootstrap --config=myconfig.yaml
[19:54] <rick_h_> natefinch: yea, but I think we need a different approach for config as a whole
[19:54] <rick_h_> natefinch: because config is across all providers and that's more of an "edit cloud" than an add-cloud and such
[19:54] <rick_h_> natefinch: so I've punted that for now because adding it here is a hack imo
[19:55] <natefinch> rick_h_: that's fine.  I can pare the list of clouds back to just the custom ones
[19:55] <rick_h_> natefinch: though I wonder if we can bootstrap by adding an empty config section whenever we write it out
[19:55] <rick_h_> natefinch: so that it's ready to be edited ootb
[19:57] <natefinch> well, you don't really need a config section
[19:57] <rick_h_> natefinch: I mean that you can specify a config key to the cloud definition and then add overrides/etc to that file
[20:02] <natefinch> rick_h_: one problem I'm not quite sure how to solve from a UX standpoint is how to query for "choose M of N", like for authentication types, you can choose 1-3 of 3 choices.
[20:13] <natefinch> rick_h_: I guess just asking for them comma separated will work for our needs
[20:14] <rick_h_> natefinch: yea, I think pick a format and to validate it
[20:31] <redir> easy review https://github.com/juju/juju/pull/6483 PTAL
[20:32] <mgz> redir: you haven't got anyone to bite yet? I'll take it.
[20:35] <mgz> redir: you took the pricing stuff from the webpage, or andrews's new generation thing?
[20:36] <redir> mgz the web page:/ Didn't know about a generator
[20:36] <mgz> redir: it's what I did last time, I think that's reasonable for 1.25
[20:36] <mgz> we know it's not always up to date but should be good enough
[20:36] <mgz> redir: lgtmed
[20:36] <redir> mgz: tx
[20:37] <mgz> ...why did I give andrew an extra s...
[20:38] <redir> channelling Gollem?
[20:39] <mgz> preciousss anddrewsss
[20:39] <natefinch> ^ nice
[20:40] <natefinch> rick_h_: forgot about a school meeting I have in the morning at 9am.  Probably will miss standup, though I might get lucky and it won't take forever. Doing well with the add-cloud stuff, though.
[20:43] <rick_h_> natefinch: k
[23:13] <axw> wallyworld: is there no new price list info for us-east-2?
[23:14] <wallyworld> axw: not yet, that bit was not updated in juju
[23:14] <wallyworld> so far, just the region and endpoints have been updated so it works
[23:15] <axw> wallyworld: yeah, it's just that it only works because of a fall-back. it could be inaccurate
[23:15] <axw> anyways, nice that it works :)
[23:15] <wallyworld> yeah
[23:16] <axw> wallyworld: hm, there is info for ohio in the latest index
[23:16] <axw> I'll update
[23:16] <wallyworld> ta
[23:16] <axw> ... right after I remember to go to standup