/srv/irclogs.ubuntu.com/2016/10/20/#juju-dev.txt

redirredir, yes empty string00:01
redirpart 1 https://github.com/go-amz/amz/pull/72 PTAL00:08
rediranyone have a minute to look at something with me?00:20
* thumper heading up to the cafe00:24
* 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:30
rediralso there's this: https://github.com/reedobrien/juju/tree/lp/1612645%2B1634289_new-aws-regions00:32
redirbut it fails because: cannot write file "provider-state" to control bucket: The specified bucket does not exist00:33
redirso I must have missed something00:33
redirin adding the regions, or something else is wrong00:33
redirlater00:33
=== hml_ is now known as hml
rediranyone have hany thoughts about why the control bucjet doesn't exist?02:19
wallyworldcontext?02:20
wallyworldmenn0: have you had a chance to take another look at that commit with my latest changes to fix import ordering etc?02:21
redirwallyworld: added new ohio region to amz and 1.25 provider and tried to deploy02:22
redirgot the error 6 lines up02:22
wallyworld6 lines up from where?02:22
redirwondering if I missed something obvious,02:22
menn0wallyworld: sorry, I haven't looked yet, otp but will look after02:22
redirthis channel02:22
wallyworldmenn0: np, ty02:22
wallyworldredir: not 6 lines for me, i have other notifications turned on, more like 20 and it's scroll off my screen :-)02:23
redirwallyworld: this channel at 17:33:0702:23
wallyworldredir: did your environment.yaml have a control bucket? it shouldn't02:23
* redir looks02:23
wallyworldalso not 17:33 more me?02:23
wallyworld:-)02:24
redirhehe02:25
redir33:07 after the hours two hours ago02:26
redirno control-bucket in environments.yaml02:26
wallyworldredir: can you provide debug logs? aws is expected to create a controller bucker using a self generated bucket name from memory02:28
redirwallyworld: I'll work on it tomorrow02:29
wallyworldit's been 2 years since i saw that code, so will have to go digging02:29
redirit only fails in US East new region. The AP South one I added works02:29
wallyworldhmmm, interesting02:29
* redir guesses it assumes the wrong things for useast02:29
wallyworldmaybe that new rwgion doesn't have s3 support yet02:29
wallyworldor the s3 endpoints are wrong02:29
wallyworlddoesn't sounds like a juju issue02:30
wallyworldif it's just the one region. more likely a go-amz issue02:30
redirit says it does, but not at s3.a....com it needs to be at s3.us-east-2....com02:30
wallyworldok, ok, so you'll need to add update go-amz to ensure it has the right endpoints02:31
redirwallyworld: right. My guess too, I'll dig in to goamz tomorrow02:31
wallyworldok02:31
redirthanks wallyworld much appreciated. Wanted to make sure it wasn't something simple02:32
wallyworldredir: tis ok, i didn't tell you much except propose a guess or two :-)02:32
redirwallyworld: sometimes it's nice when the rubberduck squeaks:)02:34
wallyworlddepends on how hard you squeeze it :-)02:34
wallyworldnormally I quack02:34
redir:)02:35
menn0wallyworld: ship it02:57
wallyworldmenn0: tyvvvvvvm for looking02:58
menn0wallyworld: opening the diff nearly killed my browser02:58
wallyworldmenn0: another reson why github reviews suck02:58
wallyworldwith rb, you get a paginated list of files02:58
wallyworldi don't get how people navigate gh reviews02:59
wallyworldwithout pulling their hair out02:59
menn0wallyworld: I just had a quick play with the gerrit instance for the chromium project. it's pretty nice, if intimidating at first.03:04
menn0wallyworld: knobs and buttons for everything03:04
menn0wallyworld: but fast and has good keyboard shortcuts for navigation.03:04
wallyworldmenn0: would love to seriously consider that03:04
wallyworldbut people need to get over the NIH syndrome03:05
wallyworldwhich I think will be impossible for our group :-(03:05
menn0wallyworld: I don't know about that. I can see why some people are attracted to GH reviews.03:07
wallyworldi am having trouble. they are really hard to navigate (all one big page), and so much wasted space03:08
wallyworldand no way to manage comments03:08
wallyworldthe main argument seems to be "well it might get better sometime"03:09
wallyworldbut it's crap *now*03:09
thumperw00t03:11
thumperwow03:11
thumperthat looks really out of place now03:11
thumpermy view had this as the last comment:03:11
thumper<perrito666> yay vsphere provider working correctly03:11
thumperthen I scrolled down03:11
thumperheh03:11
rediramz bits ready for review: PTAL https://github.com/go-amz/amz/pull/7203:36
alexisbwallyworld, if you are still around can you provide a review for redir ^^03:40
wallyworldcan do in a minute03:41
alexisbwallyworld, thank you!03:41
redirekanite looks pretty nice. A syslog server that accepts msgs over UDP/TCP [+TLS] parses RFC5424 headers, does full text indexing, and manages retention03:41
wallyworldredir: did you sort the s3 issue?03:42
rediralso sorts by log timestamp rather than the order received03:42
redirwallyworld: yup03:42
wallyworldgreta03:42
wallyworldgreat03:42
redirand will have a juju 1.25 PR when the AMZ bits are merged03:42
redirwallyworld: ^^03:42
rediroh and the ekanite binary is 12MB03:43
wallyworldredir: did you researc that us-east-2 requires the region bools to be true ie bucket lower case etc?03:44
redirwallyworld: that fixed the issue03:44
redirand US East is the only region that doesn't03:44
redirin amz03:44
redirus-east-103:45
wallyworldinteresting03:45
wallyworldredir: lgtm03:46
redirSee https://github.com/reedobrien/amz/blob/2eab8e64ed1675ae196f785bbc2d293e45aaf7a1/aws/aws.go03:46
redireverything else is not false.... so looks like they were lax on the original region03:46
redirand changed hteir minds for everything else03:46
redirwallyworld: my guess it that since bucket can be used in the website url they wanted to make it consisten with case insensitivity...03:47
redirafter the original s3 in 2006 was let out into the wild03:48
wallyworldsounds plausible yeah03:48
redircan we merge amz?03:48
redirdoes all the $$merge$$ stuff work there?03:48
wallyworldshould do03:49
redirk. tx03:54
dimiterndooferlad, frobware, macgreagoir: I'd appreciate a review on https://github.com/juju/juju/pull/6481 fixing bug 1616048 ta!08:21
mupBug #1616048: Create interface for ofono <snapd-interface> <snapd (Ubuntu):In Progress by alfonsosanchezbeato> <https://launchpad.net/bugs/1616048>08:21
dimiternoops - bug 161609808:22
mupBug #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
frobwaredimitern: will do - otp08:22
dimiternfrobware: no rush08:22
dimitern;)08:22
* dimitern steps out for ~1h10:39
dimiternfrobware: ping12:28
dimiternmacgreagoir: hey12:31
dimiternmacgreagoir: got some time for a review? :)12:31
anastasiamacdimitern: 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
dimiternanastasiamac: :D12:33
dimiternanastasiamac: that's a nicer way of looking at it, sure ;)12:33
dimiternanastasiamac: I'd appreciate a review as well hehe12:34
anastasiamacdimitern: 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 :D12:38
dimiternanastasiamac: uuh you should definitely get some rest!12:38
anastasiamacdimitern: \o/ will soon :D thank you for care and concern :)12:39
dimiternanastasiamac: any time ;)12:40
dimiternrick_h_: ping12:45
rick_h_dimitern: pong12:46
dimiternrick_h_: I thought about picking up bug 1580501 tagged 1.25 .. it seems it's only related to 2.0 though12:46
mupBug #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 board12:46
dimiternrick_h_: that's the first one on the board :)12:47
dimiternrick_h_: top left12:47
rick_h_oh wtf, the title of the bug and the card got out of wack12: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 only12:48
rick_h_dimitern: so going to yank that one off the board12:48
dimiternrick_h_: +1 I'll look into bug 1589680 then12:49
rick_h_dimitern: one sec, let me see if I did bad copy/paste voodoo12:49
mupBug #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/156048712:49
mupBug #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 bad12:49
rick_h_dimitern: apologies for the confusion12:50
dimiternrick_h_: right :) np - do you want me to look into that last one instead?12:50
rick_h_dimitern: either one is ok thank you12:50
dimiternrick_h_: ok - 1589680 it is then12:51
* dimitern dusts off his 1.25 branch .. it's been a while12:52
mgzhm, we're not doing staging and such like with 1.25 I presume?12:52
rick_h_mgz: no, we're not12:52
dimiternit's free for all :D12:53
rick_h_frobware: ping, can I grab a sec when you're available?12:55
macgreagoirdimitern: You nean 6481? Sorry, I started and got distracted by my own :-)13:04
rick_h_dimitern: I'd expect that we can just upgrade the lxc in order to find a path forward13: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 patterns13:10
dimiternmacgreagoir: np, if you can have a look at it - great!13:11
dimiternrick_h_: I'd expect setting openstack-origin to cloud-archive:mitaka will trigger adding the repo and the lxc upgrade..13:12
dimiternwill know soon anyway..13:12
rick_h_dimitern: right, but my point is you can do this with the ubuntu charm13:13
rick_h_dimitern: no need for anything openstack13:13
rick_h_dimitern: fewest moving parts/complications the better13:13
dimiternrick_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 trusty13:13
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 work13:14
dimiternyeah: lxc2.0.5-0ubuntu1~ubuntu16.04.1~cloud0 is in cloud-archive:mitaka13:16
dimiternand lxc is not there in c-a:icehouse13:16
chrome0dimitern : afaicr the error was when going from c-a:trusty-liberty -> c-a:trusty-mitaka13:17
chrome0And orig. install was plain trusty, sans c-a13:18
dimiternchrome0: yeah, I'm trying exactly that now :)13:18
chrome0+113:18
dimiternrick_h_: post 2.0 lxc won't work with 1.25 AIUI13:19
rick_h_dimitern: can you hop in the standup real quick?13:19
dimiternowm13:19
frobwarerick_h_: yep, back13:23
rick_h_frobware: meet you in the standup room please?13:24
frobwarerick_h_: omw13:24
SimonKLBis there a good way to find out when a relation is completely removed?13:28
SimonKLBi find it really hard to test adding and removing relations13:28
SimonKLBsimply looking at the departed/broken hooks doesnt seem to be enough to determine the actual status of the relation in juju13:32
rick_h_natefinch: ping for chat13:34
natefinchrick_h_: oops coming13:36
rick_h_frobware: dimitern natefinch ping for standup14:00
mgzdooferlad: yo, free for bothering?14:41
frobwarerick_h_: I think we have a nice conclusion for the openstack bug14:45
frobwarerick_h_: https://bugs.launchpad.net/juju/+bug/1621590/comments/1814:45
mupBug #1621590: openstack provider ignores a properly created bootstrap machine <cpec> <rteam> <juju:In Progress by frobware> <https://launchpad.net/bugs/1621590>14:45
frobwaredimitern: now looking at your PR :)14:47
redirmgz: yt?14:50
dimiternfrobware: sweet! :) thanks!14:50
redirmgz: 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: ping14:51
mgzredir: sure, I'll take a look14:52
dooferladrick_h_: hi14:52
dooferladmgz: sorry, still talking to Mick14:52
rick_h_dooferlad: heads up, assigned another bug your way. It's another config param getting ignored like your current one.14:52
dooferladrick_h_: ok, thanks14:52
rick_h_dooferlad: the possible idea is that it's a more generic problem and that two birds/one stone and such14:52
rick_h_dooferlad: and it was brought up on the cross team call by the stakeholders there that it should be replicatable with canonistack14:52
rick_h_dooferlad: so if getting OS goign is a burden please dump that and run with an already running OS14:53
mgzredir: heh, that's a group I'm not actually owner on, but I can trigger the merge for you14:53
rick_h_dooferlad: bah, LP won't let me change the assigner atm, timing out. https://bugs.launchpad.net/juju/+bug/161423914:53
mupBug #1614239: bootstrap-timeout ignored in --config <landscape> <juju:Triaged by rharding> <https://launchpad.net/bugs/1614239>14:53
mgzredir: merged as 775438014:59
frobwaredimitern: what were the rules on 500 lines of diff? You're dangerously close. :)15:06
dimiternfrobware: I really tried to minimize the changes :/15:07
dimiternfrobware: 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:08
frobwaredimitern: have time to HO?15:10
dimiternfrobware: yeah15:11
dimiternfrobware: I'm in 'core'15:12
frobwaredimitern: omw15:13
mgzdooferlad: so... free now?15:15
redirmgz: thanks15:20
redirmgz you know who the owner is?15:20
mgzredir: canonical-is and niemeyer15:22
mgzI don't know who has access to the canonical-is role account15:22
dooferladmgz: 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
mgzdooferlad: no worries, I'll set something up for tomorrow15:26
dooferladmgz: thanks - much appreciated15:26
* dooferlad goes to hide in a dark room15:26
rediranother PR ready for review; https://github.com/juju/juju/pull/6483 PTAL15:33
redirbbiab after dentist15:33
SimonKLBhello, anyone got some time guiding me on how to test adding and removing relations?15:37
SimonKLBi was using the relation function amulet to test wether or not the relation was sucessfully removed, and that worked, but now it doesnt anymore15:38
SimonKLBwould really appreciate some help!15:38
katcodimitern: 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#L1715:42
redirabentley: was that update for Ohio because of the amz release that happened?15:46
redirthis morning15:46
abentleyredir: No, it was because AWS announced the region on the 17.15:47
rediri see, thanks abentley15:48
rediranyone, PTAL https://github.com/juju/juju/pull/648316:14
hoeniraxw, katco, https://github.com/juju/juju/pull/641416:37
redirls18:11
redirwhoops18:11
natefinchahh 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:45
perrito666looooool18:46
* redir lunches19:02
natefinchmade a new struct that helps you ask the user repeated questions.... called it Pollster.  Figured it was appropriate given current events.19:10
redirAs long as it produces output that isn't actually relevant to anyone, natefinch19:12
redirand diferent results fo rthe same input by different users19:13
natefinchheh19:14
natefinchrick_h_: http://pastebin.ubuntu.com/23355482/19:52
rick_h_natefinch: coolio19:53
natefinchrick_h_: so, one reason to use add-cloud for ec2 etc ..... it's the only way to save a cloud-specific config19:54
natefinchrick_h_: instead of haviong to do bootstrap --config=myconfig.yaml19:54
rick_h_natefinch: yea, but I think we need a different approach for config as a whole19:54
rick_h_natefinch: because config is across all providers and that's more of an "edit cloud" than an add-cloud and such19:54
rick_h_natefinch: so I've punted that for now because adding it here is a hack imo19:54
natefinchrick_h_: that's fine.  I can pare the list of clouds back to just the custom ones19:55
rick_h_natefinch: though I wonder if we can bootstrap by adding an empty config section whenever we write it out19:55
rick_h_natefinch: so that it's ready to be edited ootb19:55
natefinchwell, you don't really need a config section19:57
rick_h_natefinch: I mean that you can specify a config key to the cloud definition and then add overrides/etc to that file19:57
natefinchrick_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:02
natefinchrick_h_: I guess just asking for them comma separated will work for our needs20:13
rick_h_natefinch: yea, I think pick a format and to validate it20:14
redireasy review https://github.com/juju/juju/pull/6483 PTAL20:31
mgzredir: you haven't got anyone to bite yet? I'll take it.20:32
mgzredir: you took the pricing stuff from the webpage, or andrews's new generation thing?20:35
redirmgz the web page:/ Didn't know about a generator20:36
mgzredir: it's what I did last time, I think that's reasonable for 1.2520:36
mgzwe know it's not always up to date but should be good enough20:36
mgzredir: lgtmed20:36
redirmgz: tx20:36
mgz...why did I give andrew an extra s...20:37
redirchannelling Gollem?20:38
mgzpreciousss anddrewsss20:39
natefinch^ nice20:39
natefinchrick_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:40
rick_h_natefinch: k20:43
axwwallyworld: is there no new price list info for us-east-2?23:13
wallyworldaxw: not yet, that bit was not updated in juju23:14
wallyworldso far, just the region and endpoints have been updated so it works23:14
axwwallyworld: yeah, it's just that it only works because of a fall-back. it could be inaccurate23:15
axwanyways, nice that it works :)23:15
wallyworldyeah23:15
axwwallyworld: hm, there is info for ohio in the latest index23:16
axwI'll update23:16
wallyworldta23:16
axw... right after I remember to go to standup23:16
=== hml_ is now known as hml

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