/srv/irclogs.ubuntu.com/2015/07/21/#juju-dev.txt

=== natefinch-afk is now known as natefinch
natefinchwwitzel3, ericsnow: btw, I made a change to the juju status --processes output to 1.) be valid yaml (list of hashtables) and present valid looking data.  Would appreciate some eyes on it: https://docs.google.com/document/d/1PcRQXaerlsACro4y1y5LWD-uvhfHya2CkOcoljyFyCU/edit#heading=h.3ypclmmspp3g00:31
thumpersinzui: is there any way I can watch the current CI run?01:12
sinzuithumper: login to http://juju-ci.vapour.ws:8080/view/Juju%20Revisions/, sort the columns on last success01:13
thumpersinzui: this doesn't tell me which are voting etc01:14
thumperI want to get a jump on any potential failure...01:14
sinzuithumper: the ones in read at the bottom01:14
thumperalso, how can I tell which rev from which branch is being checked?01:15
thumperI know I can click through01:15
thumperbut wondering if there was a summary page01:15
thumperfor the current one01:15
thumperperhaps a nice future feature :)01:15
sinzuithumper: voting isn't Jenkins, it is a ci-directory feature. Jenkins also corrupts the description so it is hard to read the rules for each job01:15
* thumper nods01:16
sinzuithumper: I might break that view. I am trying to add columns that give more incite01:17
thumperi'm linda sad that vivid and wily unit test runs have failures01:17
thumperdamn left hand typing01:17
sinzuithumper: They are close ot passing though. I think wily is fixable in a few hours01:18
thumpereta on current ci run?01:18
sinzuithumper: I cannot make description viewable, but...The weather icon is a condicator cloudy means there is a real chance of failure and black cloud often mean released revisions with failures01:19
* thumper nods01:20
sinzuithumper: about 2 hours. I removed all the utopic jobs last hour. I am trying to make the openstack on maas job run faster01:20
sinzuisomething bad just happened revision results is queued. I think the won tests failued for 1.2401:23
thumperwat?01:24
sinzuiI think the machine needs to be restarted. I am doing that now an requeuing the tests. the failurea re not consistent01:26
ericsnowthumper: FYI, looks like your most recent change in the cmd repo breaks a few tests in juju core02:24
thumperwhat are you referring to?02:24
ericsnowthumper: https://github.com/juju/cmd/pull/1902:25
thumperericsnow: it all works in jes-cli :)02:25
thumperso I know I've fixed it where I'm bringing it in02:25
ericsnowthumper: k02:26
thumperericsnow: do you need the new rev?02:26
ericsnowthumper: just need to know how you fixed jes-cli02:27
thumperwith magic and fairie dust02:27
ericsnow:)02:27
ericsnowthumper: looks like you took care of it in https://github.com/juju/juju/pull/254002:31
* thumper nods02:32
thumpersee... fairie dust02:32
thumpersprinkled over 37 files02:32
natefinchlol02:33
axwthumper: what's the plan for hosted env destruction, in light of fwereade's reply?02:53
thumperaxw: I'm thinking we'll need to do something new03:02
thumperbut I'm thinking through what03:02
* thumper afk taking kids to jiu-jitsu03:03
axwthumper: okey dokey. I think I'll go ahead and clean up storage in destroy-env --force for now, but leave the apiserver-side destruction. we can use what I'm doing if needed though03:03
axwlater03:03
thumperurulama: hey05:56
urulamahey, thumper06:00
thumperurulama: when is good for you?06:00
thumperif you need an hour or so06:00
thumperI can come back06:01
urulamatomorrow same time is ok for you?06:01
thumperyeah...06:01
thumperI guess06:01
urulamamy mother got sick so kids stayed at home an i need to figure out what to do06:01
urulama:(06:01
thumperkk06:01
thumpernp06:01
thumperchat tomorrow06:01
urulamathanks, thumper06:01
wallyworld_jam: saw your email, shared doc with you as i could access06:50
jamthx06:50
jamgot it06:51
wallyworld_axw: if you get a chance before eod, could you look at http://reviews.vapour.ws/r/2224/ for me?07:16
axwwallyworld_: ok07:17
wallyworld_ta07:17
axwsigh, yet another UPS that is not U07:33
axwwallyworld_: LGTM07:40
TheMuedooferlad: regarding our talk about postgresql see here https://twitter.com/rpy/status/622291301002670080/photo/109:32
dooferladTheMue: arg!09:32
TheMuedooferlad: hehe09:32
TheMuedooferlad: dunno how to ever have this idea09:33
bogdanteleagadoes ec2 support agent-metadata-url?09:48
=== frobware is now known as mcd
=== mcd is now known as frobware
mgzbogdanteleaga: yes, it does10:07
bogdanteleagamgz: I'm trying to set agent-metadata-url after the secret key10:07
bogdanteleagamgz: and it still goes to streams.canonical.com10:07
bogdanteleagamgz: am I missing something?10:08
mgzbogdanteleaga: welll... in practice our ci envs use tools-metadata-url but that's just for compat with older versions10:10
bogdanteleagamgz: I've tried that too, it gave a few warnings, but it still didn't work10:10
bogdanteleagamgz: I've been doing this with maas for a while now, but it just doesn't seem to pick it up here10:10
mgzbogdanteleaga: did you bootstrap with --debug? paste the log?10:11
bogdanteleagahttp://paste.ubuntu.com/11913639/10:13
bogdanteleagahttp://paste.ubuntu.com/11913647/10:14
mgzbogdanteleaga: nothing jumps out at me10:23
mgzI'll have to enable try debug with one of our scripts and compare the simplestreams bits more closely10:24
mgzbogdanteleaga: have you done this before for earlier juju versions?10:24
bogdanteleagamgz: nope, I've always did it for maas, but I haven't tried the other providers10:25
mgzwait, why does it say "trying keystone catalog"10:25
mgzthat's not ec2 code10:25
bogdanteleagaheh10:26
mgzbogdanteleaga: have you tried `juju metadata validate-tools`?10:27
bogdanteleagahttp://paste.ubuntu.com/11913726/10:28
bogdanteleagaI'm still not sure wtf is happening with the keystone thing10:29
mgzbogdanteleaga: that was with -e amazon?10:30
bogdanteleaganope but it's set as default10:30
bogdanteleagaand you can see it's trying amazon in the logs10:30
bogdanteleagathe first ones10:30
mgzbogdanteleaga: http://paste.ubuntu.com/1191376410:39
mgzlooks like if there are no matches you'll only see the last fallback, which is streams10:40
mgzso I think I'd start by trying to get validate-tools to see the matching tools10:40
bogdanteleagaI gave you a paste of validate-tools10:45
bogdanteleagait still doesn't try the ones specified in the url10:45
bogdanteleagamgz ^^10:45
mgzbogdanteleaga: compare with my paste10:46
bogdanteleagamgz: I should have a tools-url?10:46
mgzwhen I tried a version that wasn't in the streams I had, it doesn't show my different agent url10:47
mgzbogdanteleaga: no, that's spurious junk in the environments.yaml file I'm using there10:47
mgzbogdanteleaga: so, it may just be that your streams data is somehow now matching when you think it should10:48
bogdanteleagaI might have an idea10:48
bogdanteleagaI'm using devel tools and I don't think it's looking after those by default10:48
mgzright, yeah, you should add agent-stream: devel10:50
bogdanteleagaI'll try to upload them again10:50
mgzno, leaving as devel in the streams data is fine10:51
mgzyou should just add that in your environments.yaml10:51
bogdanteleagaoh yeah, I meant I'll upload them again just in case10:52
mgzthen run the validate-tools again and see if it helps10:52
bogdanteleagathe issue with doing devel in the environments.yaml10:52
bogdanteleagais that the bootstrap does not fail10:52
bogdanteleagaand it uses local tools in case it doesn't find them on simplestreams10:52
bogdanteleagawhich doesn't work on windows10:52
bogdanteleaganah, somethings badly broken10:53
bogdanteleagait shouldn't output anything about keystone10:53
bogdanteleagaI think I'm missing something, I'll try it from the top10:53
perrito666morning10:58
bogdanteleagahey horacio!11:02
mgzbogdanteleaga: http://paste.ubuntu.com/1191387711:02
bogdanteleagamgz: I managed to get it working, it was a problem with how I uploaded them. It still says crap about keystone though11:02
mgzwith debug, you can see it's trying the config location then falling back to the main11:03
bogdanteleagamgz: that's probably gonna be helpful thanks11:03
wallyworldjam: not sure if you have a minute to clarify the hash as part of the naming for resources?11:12
jamwallyworld:  /wave11:12
wallyworldhey11:12
wallyworldjam: might be easier to have a hangout if you had time. if not now, then i can hang around11:14
jamwallyworld: a hangout is probably warranted, I'm currently in another conversation, so I don't want to be too distracted to talk both yet11:16
wallyworldjam: no problem, just ping me if you are free later11:17
perrito666sinzui: isn t https://bugs.launchpad.net/juju-core/+bug/1475724 fixed?11:37
mupBug #1475724: UniterSuite.TestUniterRelations <blocker> <ci> <ppc64el> <regression> <unit-tests> <windows> <juju-core:In Progress> <juju-core jes-cli:Triaged> <https://launchpad.net/bugs/1475724>11:37
mgzperrito666: probably not, but it's not a completely consistent failure so it's hard to day12:05
perrito666mgz: I retracted update-status patch12:08
perrito666so at least That should be fixed (not the underlying problem)12:08
mgzwell... maybe we should mark it as fixed and see12:09
mgzthe batch of failures we have on it from the 16th rev 5b05541 are probably due to that branch landing12:10
mgzat least the trusty one I just looked at was a failure due to having one rather than two update-status calls12:10
perrito666mgz: this is master?12:12
mgzwe currently hve an updated jes-cli being tested that has got passes on tests that were failing pretty reliably12:12
mgzperrito666: yup, and feature branches from master12:12
perrito666mgz: odd, this landed last night12:12
mgzperrito666: I'm looking at the history of failures since the 16th12:13
mgznot a CI run with last night's change12:13
mgzmaster hasn't been run with your revert in yet it seems12:13
* perrito666 is a human popsicle12:13
mgzyou're melting in the sun?12:13
mgzfalling off your stick?12:14
perrito666mgz: argentinian constructions are old, completely made of brick and certainly not prepared for cold12:17
mgzah yes, you are in the other hemisphere12:18
mgzI guess you don't want to hear about how sunny and hot it is here then? :)12:19
perrito666mgz: you live in brittain, you probably call "sunny and hot" the same thing I am calling freezing cold12:28
mgz:P12:28
perrito666mgz: its 4C which must be a typicall hot summer morning for uk, right?12:29
perrito666doesn't go support a way for us to have blobs of text in something other than a const?12:32
perrito666I am reviewing a go file that contains 2 lines of go and a lot of powershell12:33
dooferladperrito666: I am sure that the right way of having lots of powershell is to read in a powershell file12:35
perrito666dooferlad: I am sure of it, the problem is how do you distribute that?12:36
dooferladperrito666: other than distributing the .ps1?12:37
perrito666dooferlad: this is something I wish could be embedded as a resource on compile time12:37
perrito666dooferlad: yes12:37
perrito666dooferlad: the thing is, our software should not depend on packaging particularities12:38
dooferladperrito666: can we really not just distribute the file? That seems kind of wrong.12:38
perrito666dooferlad: dunno, I would say not as we have the same thing for bash at some points12:39
dooferladperrito666: sounds like something worth starting a conversation about. I would hope that after we move to go 1.4 we could use go generate (http://blog.golang.org/generate) to pull in external files into consts12:40
perrito666dooferlad: take a look http://reviews.vapour.ws/r/2199/ I accept suggestions12:45
mgzbogdanteleaga: see last comment in bug 128794913:20
mupBug #1287949: {image,tools}-metadata-url not usable w/ ec2 provider <arm64> <config> <hs-arm64> <ppc64el> <juju-core:Triaged> <https://launchpad.net/bugs/1287949>13:20
mgzbogdanteleaga: poké for hangout13:33
perrito666ericsnow: please get the blessing from fwereade too on my review14:14
ericsnowperrito666: k14:14
ericsnowperrito666: and thanks :)14:18
* fwereade just wrote some disgustingly smug test code, and can't figure out if it's good or bad: http://paste.ubuntu.com/11914787/14:32
mgzfwereade: cuteness depends on map ordering? :)14:40
fwereademgz, the cuteness depends on the order I wrote them in, but not on any runtime properties AFAIAA14:40
perrito666fwereade: cute to read, might be confusing to debug14:41
mgzit's always what the failures would look like that I think about14:42
fwereademgz, I'm not sure the word choice helps or hinders much -- it's deliberately a tangled racy write because that's what I'm testing14:45
bogdanteleagacan anybody check this out? http://paste.ubuntu.com/11914363/14:46
bogdanteleagaI'm trying to get aws with a custom image up14:47
bogdanteleagait seems to detect the custom image metadata and then throw it away and resort to the official one again14:47
mgzfwereade: to be clear, it's fine as written, I'm just chatting about paint14:47
fwereademgz, cool -- I'm still on the fence for some reason14:48
mgzbogdanteleaga: did you add in the unsigned selection hack?14:48
fwereademgz, mainly because I look at it and I can see me admiring my own cleverness and that's a bit unattractive ;)14:48
bogdanteleagamgz: nope, because from what I gathered from the bug it wouldn't work for either tools or images14:49
bogdanteleagamgz: and tools work for me14:49
mgzthen I think you need to add in more logging in the simplestreams code till you understand why it's not selecting an image from the first stream14:51
bogdanteleagait does seem to select it at first14:52
mgzsure, but it's not matching an image, need to work out why14:53
bogdanteleagaif you look at line 7 it does match it, it just fails after that and I think after the failure it tries again with the official sources14:55
bogdanteleagacan't seem to understand why it fails though14:55
mgzbogdanteleaga: the 1.24.3 - 1.24 tip upgrade does seem to be broken by the registry thing15:05
mgzbogdanteleaga: http://data.vapour.ws/juju-ci/products/version-2906/maas-1_8-upgrade-win2012hvr2-amd64/build-62/machine-1.log.gz15:07
wwitzel3katco, ericsnow: ping if you need me, I think this --extend bug is the last hurdle for me *fingers crossed*15:21
ericsnowwwitzel3: cool15:21
katcowwitzel3: ty15:22
mgzbogdanteleaga_: ah, you dropped15:43
mupBug #1473517 changed: juju environment not usable after the upgrade <juju-core:Fix Released by wallyworld> <juju-core 1.22:Fix Committed by wallyworld> <juju-core 1.24:Fix Released by wallyworld> <https://launchpad.net/bugs/1473517>15:44
=== bogdanteleaga_ is now known as bogdanteleaga
mgzbogdanteleaga: http://data.vapour.ws/juju-ci/products/version-2906/maas-1_8-upgrade-win2012hvr2-amd64/build-62/machine-1.log.gz15:55
mgzsinzui, ocr: http://reviews.vapour.ws/r/222815:58
=== dames is now known as thedac
perrito666mgz: youve got to be kidding me right?16:01
perrito666that is 7 pages of reviews16:01
mgzperrito666: you are not intended to review-review it16:01
mgzjust check I haven't screwed up the merge16:01
mgzthe reviews happened in the landings onto the jes-cli branch16:02
perrito666mgz: rb is failing for some files16:02
mgzand ci has done a full run on the branch16:02
mgzperrito666: you probably need to use github, rb is bad with moves16:02
sinzuiperrito666: I added a comment about the testing done16:02
sinzuiperrito666: http://reports.vapour.ws/releases/2906 is blessed and confirms https://bugs.launchpad.net/juju-core/+bug/1475724 is fixed16:03
mupBug #1475724: UniterSuite.TestUniterRelations <blocker> <ci> <ppc64el> <regression> <unit-tests> <windows> <juju-core:In Progress> <juju-core jes-cli:Fix Released> <https://launchpad.net/bugs/1475724>16:03
perrito666how wonderful it would be that the sign on would redirect me to the link I was going16:05
perrito666sinzui: mgz evidence seems to tell it is all ok, but I dont think I could notice a merge misshap in 139 files16:06
sinzui:/16:06
perrito666sinzui: I think we have been careful enough to be able to say this is ok to merge16:10
perrito666bbl, lunch16:11
mupBug #1300692 changed: relation-get fails with return code 2 <charm> <hooks> <relations> <juju-core:Invalid> <https://launchpad.net/bugs/1300692>16:38
fwereadekatco, if you're OCR, can I point you at http://reviews.vapour.ws/r/2229/ and http://reviews.vapour.ws/r/2230/ ?16:41
fwereadekatco, 2229 is a minor interface change, already reviewed elsewhere, but maybe of interest to you16:41
fwereadekatco, 2230 actually uses it16:41
mupBug #1300692 opened: relation-get fails with return code 2 <charm> <hooks> <relations> <juju-core:Invalid> <https://launchpad.net/bugs/1300692>16:44
mupBug #1300692 changed: relation-get fails with return code 2 <charm> <hooks> <relations> <juju-core:Invalid> <https://launchpad.net/bugs/1300692>16:50
katcofwereade: not ocr today... if i have time i'll give it a look later :)16:51
fwereadekatco, ...hmm, apparently it's not wednesday16:51
fwereadewho knew?16:51
fwereadeahem16:51
katcohaha16:52
fwereadeperrito666, ^^16:52
wwitzel3ericsnow: so I've narrowed it down to command.go not properly applying --extend16:52
wwitzel3ericsnow: which is highly probably since we don't have tests for that16:52
ericsnowwwitzel3: k16:52
wwitzel3ericsnow: so I'm added some tests and hopefully that will point me to the problem :)16:54
natefinchkatco: wwitzel3, ericsnow: back16:57
ericsnowwwitzel3: cool16:57
wwitzel3ericsnow: as I'm attempting to make tests for this, I think I am getting the sense why it wasn't testing :P17:02
ericsnowwwitzel3: :)17:02
katcowwitzel3: hey where is that charm at?17:14
=== tvansteenburgh1 is now known as tvansteenburgh
bdxcharmers, core: how do access predefined hooks like relation-changed or relation joined hooks in python when writing a charm? Must I import those charmtools or charmhelpers modules?17:27
marcoceppibdx: I'm not sure I understand what you're asking17:27
bdxmarcoceppi: for example, in the tutorial https://jujucharms.com/docs/stable/authors-charm-writing17:29
bdxhooks are referenced like "db_host=`relation-get private-address`"17:29
bdxmarcoceppi: how can I access similar hooks in python?17:44
marcoceppibdx: I see, those aren't hooks, those are hook-tools17:46
marcoceppihooks aer the relation-joined, relation-changed, config-changed, etc17:46
marcoceppibdx: there's a charm-helpers libaray, charmhelpers.core.hookenv which has all those available as methods17:46
marcoceppibdx: https://pythonhosted.org/charmhelpers/api/charmhelpers.core.hookenv.html17:47
marcoceppibdx: relation_get will be what you're looking for17:47
bdxmarcoceppi: ok, that sweet..... I think I was just a little confused using hooks between bash and python17:58
katcowwitzel3: natefinch: ericsnow: i'm back. what can i help with?17:59
ericsnowkatco: not sure18:00
katcowwitzel3: please lmk when the charm is ready to go. i think we're blocked on that atm18:09
natefinchkatco: I was thinking that ComponentStatus should be a map[string]interface{}  that way a component can just dump a struct into it, and have strongly typed values that automatically get converted in and out.18:13
katconatefinch: ok, noted. let's discuss more after the demo18:14
natefinchkatco: ok18:14
katconatefinch: can you still manage your piece w/o doing that?18:14
natefinchkatco: yeah18:15
mupBug # changed: 1370896, 1430245, 1461354, 1463480, 1465115, 1468653, 1469130, 1469199, 1471332, 1472014, 1472729, 147516318:20
marcoceppiCan I just make a simple request? I'm not sure who thought juju scp would be better over rsync, but it's not, it should be juju rsync ;)18:34
=== Viperz28_ is now known as Viperz28
xwwt_sinzui: Can we delay for 1/2 hr and meet at 3:30?18:57
sinzuixwwt_: sure18:58
=== xwwt_ is now known as xwwt
perrito666btw, I am back19:01
katcowwitzel3: ping?19:08
wwitzel3katco: yep, I'm still trying to figure out why the --extend args are not being properly populated19:40
katcowwitzel3: need any help? running out of time19:41
wwitzel3katco: other people exploring it would be fine, I'm just walking through and re-compiling different logging outputs right now trying to find out how far it is actually getting19:42
natefinchbtw, I'm trying to test the status output, just whipping up a test charm that uses the docker plugin19:43
wwitzel3katco: right now I'm poking around in process/context in the launch command and launch_test19:43
katcoericsnow: natefinch: wwitzel3: let's plan on touching base in moonstone in 15m or so?20:15
ericsnowkatco: k20:15
wwitzel3katco: k20:21
natefinchditto20:22
thumperOMG, is that a bless on jes-cli I see?20:52
cheryljthumper: yeah!20:58
* fwereade cheers at thumper, wonders if he'd like to talk environment lifecycles shortly? am approaching sleepiness but also sort of want to stay up and talk to wallyworld21:04
thumperfwereade: on calls for a while now21:05
thumperfwereade: but yes, at some stage21:05
thumperprobably not right now21:05
fwereadethumper, sgtm, I'm not *especially* in the mood myself right now :)21:06
wallyworldfwereade: i have a meeting in 10, should be short hopefully21:20
fwereadewallyworld, cool, ping me when you're done, I may or may not still be both present and coherent21:25
wallyworldok21:25
wallyworldor we can talk at yuor SOD21:26
wallyworldwe have one scheduled21:26
xwwtsinzui: ping21:32
menn0fwereade: I'm forward porting this https://github.com/juju/juju/pull/2801 but of course everything has changed in master21:59
menn0fwereade: I'm going to remove the insertWithoutEnvironment collection attribute. yell at me if you don't think that's right.22:00
fwereademenn0, that's exactly right, and it's what I did when I backported all that stuff to 1.2422:00
menn0cool22:01
fwereademenn0, I think if you check 1.24 the multiEnvRunner already does the right thing22:01
menn0fwereade: well yes, I removed the always-assert-on-env-life feature from there (that's what 2801 is about)22:02
fwereademenn0, I think I had a point but it escapes me right now22:04
menn0fwereade: np :)22:05
perrito666why is 1475724 fix committed yet still blocks?22:42
_thumper_perrito666: possibly  because the blocker tag is still there and the revision hasn't passed the relevant CI test?22:49
=== _thumper_ is now known as thumper
perrito666thumper: ah makes sense22:49
alexisbsinzui, ^^^22:50
* perrito666 was falling asleep on the keyboard, decided to cook to wake up a bit, now is falling asleep and laptop has garlic smell22:50
sinzuiperrito666: fix commits is not CI's proof that the fix works and doesn't not introduce other regressions.22:51
sinzuiperrito666: alexisb thumper : CI is almost done proving that the fix work and will open master with jes-cli merged22:52
sinzuiJoyent is very ill.22:53
menn0wallyworld: is http://reviews.vapour.ws/r/2225/ a back/forward port?22:55
wallyworldmenn0: forward port, already landed in 1.24, waiting for master to unblock22:56
menn0wallyworld: cool, i'll ignore it unless you say otherwise22:56
wallyworldnp22:56
sinzuiWho wants to be first to break master22:59
perrito666memememme23:00
sinzuiperrito666: thumper alexisb wallyworld : Ci blessed master (wiith jes-cli merged) and it closed the blocking bug. We can merge now23:00
anastasiamac_sinzui: \o/23:00
perrito666mail client with notification ftw, instant bless notice23:01
perrito666:p23:01
sinzuiDon't panic (as I did) about the email with all the failed joyent jobs. Joyent is/was very ill. Machines were not being provisions in our region23:01
wallyworldyay23:01
mupBug #1475724 changed: UniterSuite.TestUniterRelations <blocker> <ci> <ppc64el> <regression> <unit-tests> <windows> <juju-core:Fix Released> <juju-core jes-cli:Fix Released> <https://launchpad.net/bugs/1475724>23:09
thumpersinzui: is that rev going to be 1.25-alpha1?23:33
* wallyworld off to pick up newly repaired coffee machine :-D bbiab23:47

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