/srv/irclogs.ubuntu.com/2013/12/02/#juju-dev.txt

=== bradm_ is now known as bradm
axwhey thumper, have a good week away?01:12
thumperaxw: yeah, I did01:12
thumperquite relaxing01:12
axwcool01:13
axwgood week for it ;)01:13
axwit was a little bit crazy last week, but all turned out well01:13
thumpercool01:14
thumperI'm tyring to get the team's prioritized list01:15
thumperso we can focus on what to do next01:15
thumperI'm finishing off the kvm support01:15
axwthumper: I'm pretty close to having synchronous bootstrap done I think; I've got some updates to do per jam's suggestion of toning down the output01:17
* thumper nods01:17
thumpercool01:17
axwthumper: waiting on some feedback from fwereade regarding state changes for API-based destroy-environment01:17
axwwhen that's done I can finish off destroy-env for the manual provider01:17
thumperthat'll be good to finish off01:17
thumperaxw: I think it would be great to create an "ask ubuntu" question around "how do I use juju with digital ocean", and answer yourself demonstrating the manual bootstrap / manual provisioning01:18
axwthumper: I also looked into manual provisioning into non-manual env. There's one particularly difficult bit, which is that agents/units/etc. address each other with private addresses, which isn't likely to work with an external machine01:18
thumperhmm...01:19
* thumper nods01:19
axwthumper: SGTM, tho Mark Baker (I think it was him) wanted me to get in touch before publicising widely01:19
thumperaxw: what about crazy shit like running a sshuttle01:19
thumperon the bootstrap node01:19
axwyeah something like that may be necessary01:19
axwtho I'm starting to think that it's a lost cause, and we should just handle it with cross-environment01:20
axwhaven't gotten too deep into it yet01:20
thumperhmm...01:24
axwwallyworld_: you can just do "defer os.Chdir(cwd)" - no need to change it now though02:06
wallyworld_ah bollock02:06
wallyworld_s02:06
wallyworld_the tests didn't break regardless, just thought i'd be complete02:07
wallyworld_i'll fix as a driveby02:07
axwcool02:09
wallyworld_thumper: initial framework https://code.launchpad.net/~wallyworld/juju-core/ssh-keys-plugin/+merge/19731003:30
thumperwallyworld_: ack03:34
* thumper nips to the supermarket for dinner stuff...03:39
jam1axw: "agents address each other with private addresses", in the case of AWS I *think* we use the DNS name, which appropriately resolves private/public based on where you make the request. But I'll agree that you don't guarantee routing. Assuming a flat network for agent communication probably isn't going to change this cycle, but we may get there eventually.04:19
axwjam1: it may not be a difficult problem to solve, but I first came up against this when the agents were deployed with stateserver/apiserver set to the private address04:21
axwthat was on EC204:21
jam1axw: yeah, code has been changing in that area. It would appear something is resolving that address and passing around the IP, rather than the DNS name.04:26
=== jam1 is now known as jam
axwjam: I don't think it was the IP - it was an internal DNS name04:28
axwcouldn't be resolved outside EC204:28
axwbbs, getting lunch04:28
axwjam: still working out the kinks, but synchronous bootstrap will look something like this in the near future: http://paste.ubuntu.com/6507781/05:15
jamaxw_: looks pretty good.06:04
=== axw_ is now known as axw
axwthe apt stuff should be at the beginning obviously :)06:06
dimiternmorning07:11
rogpeppemornin' all08:43
jamjamespage: poke about mongo/mongoexport/mongodump/etc09:15
jamespagejam: morning09:15
jamjamespage: I hope you had a good weekned09:16
jamweekend09:16
jamespageyes thanks - how was yours?09:16
jampretty good. It was Thanksgiving into UAE National Day, and Expo 2020 celebration, so the weekend gaps were all a bit silly. My son will end up with a 5-day weekend09:16
jamespagejam: nice09:18
jamjamespage: so I responded to your questions about what tools we need to include. I'd like to know more from your end what the costs of it are so that we can find a good balance point.09:18
jammorning fwereade, I had a question for you when you're available09:19
fwereadejam, heyhey09:19
jamfwereade: namely, the 14.04 priorities/schedule stuff09:19
fwereadejam, I couldn't see anything by mramm, so I just copied them straight into the old schedule doc09:20
fwereadejam, https://docs.google.com/a/canonical.com/spreadsheet/ccc?key=0At5cjYKYHu9odDJTenFhOGE2OE16SERZajE5XzZlRVE&usp=drive_web#gid=209:20
jamfwereade: yay, I was hoping they would end up there09:21
jamespagejam: just reading09:25
jamk09:25
jamespagejam: OK - I see what you are saying - I'll respond on list so everyone can see09:26
=== axw__ is now known as axw
axwfwereade: re https://codereview.appspot.com/28880043/diff/20001/state/state_test.go#newcode567, there's code in AddServce/AddMachine* to assert environment is alive09:55
axwand accompanying tests09:55
axwactually that is the test - maybe I misunderstood you09:55
fwereadeaxw, yeah -- but Ididn't see a test for what happens if the env was alive as the txn was being calculated, but became not-alive before the txn was applied09:56
fwereadeaxw, look in export_test.go -- inparticular, SetBeforeHook, I think -- for a mechanism that lets you change state at that point09:57
fwereadeaxw, there are a few other tests that use it... but not many, the capability is relatively recent09:58
axwfwereade: thanks. just so I understand- you mean to check what happens between the initial environment.Life() check, and when the transaction is executed?09:58
fwereadeaxw, yeah,exactly09:59
axwokey dokey, I will add that in10:00
fwereadeaxw, cheers10:00
axwfwereade: and this one: https://codereview.appspot.com/28880043/diff/40001/state/environ.go#newcode84  -- I thought I was following your earlier advice ("I kinda hate this, but we're stuck with it without a schemachange for a total-service-refcount somewhere..."), but I suppose I've misunderstood you10:00
axwjam: I'd appreciate another look at https://codereview.appspot.com/30190043/ if you have some time to spare later10:05
fwereadeaxw, indeed, I was a bit worried I might have misunderstood myself there10:05
axwheh :)10:05
fwereadeaxw, I don't *think* I ever said that you'd have to remove everything before an env could be destroyed, but I might have said something else so unclearly that the distinction was minimal10:06
fwereadeaxw, it's always been manual machines that are the real problem10:06
axwindeed10:07
axwthey are not much like what's preexisting10:07
fwereadeaxw, yeah10:07
fwereadeaxw, I think we're massaging them into shape though10:07
axwfwereade: how about I just drop that comment and stick with the cleanup. did you have any thoughts on cleaning up machine docs?10:07
axwi.e. destroying machines as environment->dying10:08
fwereadeaxw, I feel that bit should be reasonably short-circuitable10:08
fwereadeaxw, ie just kill all the instances and be done with it10:09
fwereadeaxw, the only reason not to is, again, the manual machines10:09
axwyeah, except for manual machines10:09
fwereadejinx ;p10:09
axwfwereade: so, one option is to have it schedule a new cleanup for machines that *doesn't* imply --force10:10
axwand that will wait for all units to clean up (as a result of services being cleaned up)10:10
fwereadeaxw, so yeah I could maybe be convinced that *if* there are manual machines we should destroy all the others10:10
fwereadeaxw, well, there's not actually much point waiting10:10
fwereadeaxw, I am starting to think that actually destroy-machine should always act like --force anyway10:11
axwfwereade: I was just thinking about leaving units in a weird state, but maybe we don't care?10:11
axwdoesn't matter for non-manual of course10:11
fwereadeaxw, if you don't want a machine, you don't want it, and if the whole thing's being decommissioned there's no point caring about the units in a weird state10:11
fwereadeaxw, yeah, doesn't apply to manual machines ;p10:12
axwyeah... except if you want to reuse that machine10:12
axwheh10:12
axwok, I'll play with that some more tomorrow10:12
fwereadeaxw, destroy-machine implies no reuse, I think10:13
fwereadeaxw, whereas manual... often implies reuse10:13
axwfwereade: at worst, users can destroy-machine the ones they care about10:14
axwthen wait for things to clean up10:14
axwthen do destroy-env10:14
fwereadejam, seeding a thought: can we get rudimentary versions into the API by 1.18? can chat about it more after standup10:14
axwassuming destroy-machine doesn't change to always --force10:14
fwereadeaxw, yeah, indeed10:14
fwereadeaxw, manual machines again are the reason not to always --force10:15
fwereadeblasted things ;)10:15
* fwereade has to pop out a mo, bbs10:15
* axw has to go10:15
axwadios10:15
jamhave a good evenning axw10:15
axwthanks jam10:17
jamTheMue: rogpeppe, mgz, standup? https://plus.google.com/hangouts/_/calendar/am9obi5tZWluZWxAY2Fub25pY2FsLmNvbQ.mf0d8r5pfb44m16v9b2n5i29ig10:47
jamwallyworld_: ^^10:48
TheMuejam: i'm already each day in the qa timeout10:48
jamTheMue: absolutely, just didn't know if you were coming to ours, so you're welcome if you want10:49
TheMuejam: so i think it's better to take part on thursdays, there's more than status10:49
TheMuejam: right now i'm in an interesting testing, maybe tomorrow10:49
jamTheMue: have a good day, then.10:52
TheMuejam: thank, u210:54
TheMuejam: short info about current status, built a "Tailer" for filtered tailing of any ReadSeeker and writing to a Writer. looks good, right now testing it.10:57
TheMuejam: it's part of the debug-log api10:57
* TheMue => lunch11:22
natefinchjam: I can try reaching out to the github.com/canonical guy through linked in... we're like 3rd degree relations. Seems like he keeps it up to date there.13:00
jamnatefinch: go for it.I'll forward you the email I tried sending earlier13:00
natefinchjam: cool13:00
* rogpeppe3 goes for lunch13:25
=== gary_pos` is now known as gary_poster
sinzuijam I still see my blocking bug stuck in triaged: https://launchpad.net/juju-core/+milestone/1.16.4 Is bug 1253643 a duplicate of bug 125246914:15
_mup_Bug #1253643: juju destroy-machine is incompatible in trunk vs 1.16 <compatibility> <regression> <juju-core:Fix Committed by jameinel> <juju-core 1.16:Fix Committed by jameinel> <https://launchpad.net/bugs/1253643>14:15
_mup_Bug #1252469: API incompatability: ERROR no such request "DestroyMachines" on Client <terminate-machine> <juju-core:Triaged> <https://launchpad.net/bugs/1252469>14:15
jamsinzui: that gets bumped to 1.16.5 because we moved the DestroyMachines code out of 1.16.414:16
jamI'll fix it14:16
sinzuijam, then the bug is closed :) I will start motions for the release of 1.16.414:16
jamsinzui: right, the *key* thing is NEC is already using something (1.16.4.1) which we really want to make 1.16.4, and then make the next release 1.16.514:17
jamI forgot to check the milestone page, as another of the bugs gets bumped as well14:17
sinzuiuhg14:17
jamsinzui: did we end up re-landing bug #1227952 ?14:18
_mup_Bug #1227952: juju get give a "panic: index out of range" error <regression> <goyaml:Fix Committed by dave-cheney> <juju-core:Fix Committed by dave-cheney> <juju-core 1.16:Fix Committed by sinzui> <https://launchpad.net/bugs/1227952>14:18
sinzuirelanding? I don't know. I can check the version14:19
sinzuiin deps14:19
jamsinzui: I pivoted to remove everything in lp:juju-core/1.16 back to 1.16.3 so that we could land the things that were critical to NEC, and then get the non-critical stuff out in the next one14:20
jamI think bug #1227952 needs to target 1.16.514:20
_mup_Bug #1227952: juju get give a "panic: index out of range" error <regression> <goyaml:Fix Committed by dave-cheney> <juju-core:Fix Committed by dave-cheney> <juju-core 1.16:Fix Committed by sinzui> <https://launchpad.net/bugs/1227952>14:20
sinzuijam, ah, the missing info from the Wednesday conversation. okay. A fine strategy14:21
jamsinzui: as soon as you give me the go ahead I have lp:~jameinel/juju-core/preparation-for-1.16.5 to bring back all the stuff14:21
sinzuiunderstood14:22
=== gary_poster is now known as gary_poster|away
=== gary_poster|away is now known as gary_poster
benjimarcoceppi: It is a work-in-progress, but I would like your initial impressions of lp:~benji/+junk/prooflib-first-cut15:51
marcoceppibenji: I thought proof was already free of all sys.exit calls already?15:51
benjimarcoceppi: I haven't looked for sys.exit specifically yet, so far I have concentrated on project structure15:52
marcoceppibenji: why seperate this from charm-tools?15:52
marcoceppiI'm confused as to the goal15:53
marcoceppibenji: care to jump on a hangout?15:53
benjithe goal is to create a library that third-parties can consume15:53
benjiI have a call in a couple of minutes, but I can do it after that (say 30 minutes or so from now)15:53
marcoceppibenji: I'm confused as to why charm-tools can't be that library?15:54
marcoceppisounds good!15:54
mattywstupid question: I'm trying to query the mongodb that gets started when I run make check - but I can't work out where to get the right creds to start a mongo shell connection15:58
rogpeppe3mattyw: what's "make check"?16:05
=== rogpeppe3 is now known as rogpeppe
rogpeppemgz: ping16:05
mattywrogpeppe, running all the tests16:14
mgzrogpeppe: pong sorry, not paying attention17:10
rogpeppemgz: np, am currently in a call17:11
mgzyell after if you still need me17:11
mgzwill be around at least another couple of hours17:11
rogpeppeniemeyer: ping17:49
niemeyerrogpeppe: pongus17:49
rogpeppeniemeyer: i wondered if you might have a moment to join us in a hangout - i'm trying to resolve some issues after restoring a mongodb17:50
niemeyerSure thing17:51
niemeyerLink?17:51
rogpeppeniemeyer: https://plus.google.com/hangouts/_/calendar/am9obi5tZWluZWxAY2Fub25pY2FsLmNvbQ.mf0d8r5pfb44m16v9b2n5i29ig?authuser=117:51
natefinchFixing tests with <-time.After(x)  ...bad or truly terrible?18:03
hatchHey all, is there a whitelist of characters for service names in core? I need to set up proper validation in the GUI https://bugs.launchpad.net/juju-gui/+bug/125257818:05
_mup_Bug #1252578: GUI shows invalid service name as valid when deploying ghost <juju-gui:Triaged> <https://launchpad.net/bugs/1252578>18:05
hatchfound the regex18:10
=== BradCrittenden is now known as ba
=== ba is now known as bac
sinzuiabentley, i am have a bad day tearing down envs. Local required manual deletions of lxc dirs and symlinks. I got two timeouts destroying azure19:18
abentleysinzui: Ugh.19:18
abentleysinzui: I am starting a test run on the new instances. (juju-ci-2 env)19:19
sinzuifab19:19
abentleysinzui: Is it possible that you recently destroyed an aws environment?19:45
sinzuiabentley, in the last 15 minutes I destroyed test-aws19:46
sinzuiand test-hp19:46
sinzuiabentley, I don't think these can collide19:46
abentleysinzui: This log is baffling: http://162.213.35.54:8080/job/aws-upgrade-deploy/71/console19:47
abentleysinzui: It bootstraps successfully, and later reports it's not bootstrapped.19:47
sinzuihmm, maybe the control-buckets are the same.19:47
abentleysinzui: Most likely explanation is it was torn down after bootstrapping by one of us or our pet machines.19:47
abentleysinzui: Ends with yakitori?19:48
sinzuiabentley, yep19:48
abentleysinzui: I think that's the explanation, then.19:49
sinzuiabentley, can you instrument CI destroying aws and undermining my acceptance test?19:49
sinzuiwell19:49
sinzuiI can teardown now and find some more japanese food19:49
abentleyI didn't understand the request.  But I can certainly change the control bucket.19:50
sinzuiabentley, I was wondering if you wanted to make ci destroy aws and then I would see if the env went missing19:50
abentleysinzui: Okay.19:51
abentleysinzui: Done.19:51
sinzuiabentley, that was it19:52
abentleysinzui: Okay, changing control bucket here.19:52
sinzuiand I will change mine too19:52
abentleysinzui: 1.16.4 on the local provider no longer ooms, but the deploy goes funny: http://162.213.35.54:8080/job/local-upgrade-deploy/75/console19:56
abentleysinzui: So the upgrade went fine, but the deploy seems to use the 1.16.3 tools instead of the 1.16.4.19:57
sinzuiabentley, we could use --show-log or --debug to see more info about what was selected19:58
marcoceppiWho knows the most about the openstack provider?19:58
abentleysinzui: Okay.  I can say from the existing log that 1.16.3.1 was selected on that run.19:58
natefinchmarcoceppi: I wouldn't say I know a lot about the openstack provider, but maybe I can help?20:00
sinzuiabentley, looking at before the destroy step, we can see that 1.16.4.1 was selected, but I don't see any agents upgraded during the wait phase20:01
abentleysinzui: As the agents reach the expected value, they disappear from the list.  So when 0 disappears from "<local> 1.16.3.1: 1, 2, mysql/0, wordpress/0", we can infer that it was upgraded.20:02
abentleysinzui: When /var/lib/jenkins/ci-cd-scripts2/wait_for_agent_update.py exits without an error, that indicates that all the agents have upgraded.  I can change it to print that out.20:04
sinzuihmm, why are the two upgrade commands different in this log20:04
sinzuiabentley, this command assume that 1.16.4 is at the location specified in the tools url: juju upgrade-juju -e local --version 1.16.4.120:05
thumpermorning20:05
sinzui^ I think that might assume the bucket was populate from the previous effort20:06
abentleysinzui: They are for different reasons.  the first tests upgrades.20:06
thumpermramm: back in the land of stars and stripes?20:06
mrammyes20:06
abentleysinzui: The second is to force the agents to match the client.  It was intended for the case where  the agent is newer than the client.20:06
sinzuiabentley, sure, but --version requires a binary at the tools url20:07
abentleysinzui: sure, but shouldn't 1.16.4 deploy its binary to the tools url?20:08
sinzuiabentley, only --upload-tools will put the tool in the location. I think --version just pulls from that location20:09
natefinchmorning thumper20:09
thumpero/ natefinch20:09
thumpermramm: our 1:1 has moved from 9am to 11am for me due to summer time here and not there20:09
thumperwe can move it earlier if you like20:09
mrammok20:09
mrammright now I have an interview20:10
abentleysinzui: So how does 1.16.3 get into the tools url?  Remember we're talking local provider here.20:10
mrammthumper: but I can move it forward for next week20:10
thumpermramm: sure20:10
sinzuiabentley, local-bucket > streams > aws20:11
abentleysinzui: So do we need to specify a testing tools-url for local-provider then?20:12
sinzuiabentley, I think so, I have in the past with limited success.20:12
* sinzui ponders trying lxc with with aws testing20:12
abentleysinzui: In the old environment, I have http://10.0.3.1:8040/tools20:14
thumpersinzui: fyi, we are going to want to test the local provider with lxc and kvm shortly20:18
sinzuithumper, I was planning that for 1.18, but given the topic of jamespages's reply to the 1.16.4 announcement, I think we need to quickly increment the minor numbers and release 1.18 today20:21
* thumper tilts his head20:21
thumpersinzui: must have been a personal reply20:21
thumperdidn't see it20:21
sinzuithumper, do you have a reply to "Adding new modes to the provisioner and new plugins feels outside the scope of stable updates to me"20:21
thumperwhat is the rationale there?20:21
thumperI guess that makes sense20:22
thumperare we needing a release?20:22
sinzuiI learned that jam had planned an alternate 1.16.4 from myself. I made the release with jam's plan, but james thinks we should be doing a 1.18 with these changes20:23
sinzuiabentley, thumper I am tempted to create a stable series and branch from 1.16 and just release stables from it with selective merges20:24
abentleysinzui: Does this shed any light? http://162.213.35.54:8080/job/local-upgrade-deploy/76/console20:24
abentleysinzui: That sounds like it could work, but I think it would be better if the dev team was doing that.  Or better yet, landing fixes to stable and merging them into devel.20:27
sinzuiabentley, I am sure that strategy would avoid the porting pain we have had in the last 30 days20:28
abentleysinzui: agreed.20:28
thumpersinzui: what is the push to get the plugins out?20:28
thumperis it external need?20:29
thumperif that is the driver,20:29
thumperthen +1 on a 1.18 from me20:29
thumperif it is to just get testing20:29
thumperwhy not 1.17?20:29
thumperwe haven't done 1.17 yet have we?20:29
sinzuithumper, no way, 1.17 is really broken. we haven't see hp-cloud work in weeks20:30
thumperreally?20:30
thumperWTF?20:30
thumperbroken how?20:30
sinzuithumper, we can do a dev release with 2071 from Nov 4.20:30
thumperwhy is it so broken?20:31
thumpershouldn't that be a "stop the line" type issue?20:31
sinzuiwe cannot tell. exactly. Charms don't deploy on it for testing20:31
* thumper shakes his head20:31
thumperwhat about canonistack?20:31
thumperis that working?20:31
sinzui2071 always works, 2072 always fails. canonistack passes20:31
sinzuiabentley, about the log. I am still not surprised. I think 1.16.3 was pulled from aws because --upload-tools was not used.20:34
thumperthe only think that leaps to mind with that has to do with installing stuff in cloud-init leaving apt in a weird state20:34
thumperthat revision is nothing openstack specific20:34
thumperso we are left to look at the differences in the actual cloud images20:34
sinzuiabentley, let me see if I can get control of my local and try with aws testing location20:34
thumpersinzui: got logs?20:34
sinzuisure do20:35
sinzuithumper, https://bugs.launchpad.net/juju-core/+bug/125524220:35
_mup_Bug #1255242: upgrade-juju on HP cloud broken in devel <ci> <hp-cloud> <regression> <upgrade-juju> <juju-core:Triaged> <https://launchpad.net/bugs/1255242>20:35
sinzuithumper, abentley ^ about this bug. I ponder if Hp got too fast for juju. I see similar errors on Hp when I am making juju go as fast as possible. When I wait a few minutes for the services to come up before add-relation, I get a clean deployment20:37
abentleysinzui: I can change the script to always supply --upload-tools for the local provider, if that's what we want.20:37
thumpersinzui: weird20:38
sinzuiabentley, I think that might be the case. let me finish my test with with local + aws testing20:38
abentleysinzui: Sure.  BTW, here's an example of our automatic downgrade to 1.16.3 to match a 1.16.3 client: http://162.213.35.54:8080/job/aws-upgrade-deploy/73/console20:39
sinzuiabentley, use use --upload-tools for the second deploy phase. my aws testing hacks didn't help20:47
abentleysinzui: Can I also use it for the first deploy phase?  I use the same script for both deploys.20:48
sinzuiabentley, if the first juju is stable and the second is proposed, then it is okay20:49
abentleysinzui: Okay, I will use it for both deploys.20:50
abentleysinzui: Okay, I applied --upload-tools to bootstrap, but for some reason, 1.16.3.1 was selected again.  Did you mean I should apply it to upgrade-juju?21:09
sinzuiabentley, for the second case these is a disaster. 1.16.4 should only upload tools for itself.21:10
sinzuiabentley, the test is to verify propose stable can bootstrap itsel21:11
sinzuif21:11
sinzuiabentley, The local upgrade and bootstrap just works for me.21:16
sinzuiabentley, http://pastebin.ubuntu.com/6511274/21:16
abentleysinzui: I am not installing the deb, so that I don't have to worry about version conflicts.21:17
sinzuiabentley, but is this a case where we are using the extract juju? Since we didn't install juju, the 1.16.3 tools are all that is available21:17
abentleysinzui: Instead, I am just using the binary directly.21:17
sinzuibingo21:18
sinzuithis is tricky21:19
abentleysinzui: So there's a ./usr/lib/juju-1.16.4/bin/jujud in a directory in the workspace.  Any way to convince juju to use that?21:19
sinzuiGOPATH?21:19
abentleyI don't know enough about what GOPATH means.  Is it for resources as well as executables ?21:21
sinzuiGOPATH=./usr/lib/juju-1.16.4/ indicates where to find bins and srcs21:22
* sinzui can try this now in fact21:22
sinzuiabentley, I think GOPATH works are in thunderbirds21:27
abentleyin thunderbirds?21:27
natefinchthat's the signal!  Delta team, go go!21:28
natefinch>.>21:28
natefinch<.<21:28
sinzuiabentley, "thunders are go" http://pastebin.ubuntu.com/6511330/21:29
thumper:)21:29
* sinzui likes supermarionation21:30
natefinchthis does not surprise me.21:30
* abentley liked Team America: World Police, but hasn't seen much of the original stuff.21:31
sinzuimy son has a youtube subscription to follow Captain Scarlet21:32
abentleysinzui: Did I do it wrong? http://162.213.35.54:8080/job/local-upgrade-deploy/81/console21:42
natefinchthumper: my tests for mongo HA require two thread.Sleep() equivalents (<-time.After(time.Second))... it's  because I'm starting and stopping mongo servers, and the code that does it is asynchronous.... so sometimes mongo hasn't finished starting yet.  Thoughts?  I could spend some time making the mongo start function synchronous.. but it's just a test mechanism, so I'm not sure how much time to put into it.21:44
thumpernatefinch: I'd suggest making the start synchronous, it shouldn't be too hard no?21:45
thumperwe do this now with the upstart start method21:45
thumperso we go: start, are you started?21:45
sinzuiabentley, I explicitly call the juju bin instead of PATH resolution21:45
thumperwait a bit, and ask again21:45
natefinchthumper: yeah, that's what I was thinking of doing.  Fair enough.21:45
thumperwe have short attempts21:45
sinzuiabentley, but I see you put the bin as the first element in PATH21:46
abentleysinzui: Also, I run 'which' to make sure I have the right one.21:46
thumperbut I think making it synchronous is the most obvious thing, hide the waiting from the caller, make the tests simple to read and understand21:46
thumperI don't think you ever regret making tests better and more understandable21:46
sinzuiabentley, is is possible sudo got root's PATH21:46
thumperwithin reason21:46
abentleysinzui: I use -E to preserve the environment.21:47
sinzuiabentley, Doesn't work for me21:50
sinzuiGOPATH=~/Work/juju-core_1.16.4 PATH=~/Work/juju-core_1.16.4/bin:$PATH sudo -E juju --version21:50
sinzui1.16.3-trusty-amd6421:50
sinzuiabentley, but this works because I removed all of the historic PATH:21:51
sinzui$ GOPATH=~/Work/juju-core_1.16.4 PATH=~/Work/juju-core_1.16.4/bin juju --version21:51
sinzui1.16.4-trusty-amd6421:51
abentleysinzui: Weird.21:51
sinzuiyeah21:52
thumper-E doesn't pass in PATH21:52
thumperIIRC21:52
thumperI use "sudo $(which juju) --version"21:53
thumpermramm: I'm in the hangout... just hanging out22:01
bigjoolshello.  I have a dead machine but juju still thinks it's there.  How can I remove it?  terminate-service/destroy-machine all fail to work because they seem to want to contact the agent on the dead machine.22:45
davecheneybigjools: i don't thnk you can at the moment22:59
bigjoolsdavecheney: so my env is fucked?23:00
davecheneyyou could try building 1.17-trunk from source23:00
bigjoolsok23:00
davecheneybigjools: we don't tell the customres their environment is fucked23:00
davecheneybut, yes23:00
bigjools:)23:00
bigjoolsI think that when you see a "hook failed" message it ought to suggest running "juju resolved"23:01
bigjoolssomeone who will remain nameless decided to use "nova delete" instead23:01
davecheneythat's a paddlin'23:01
=== gary_poster is now known as gary_poster|away
wallyworld_thumper: wrt https://codereview.appspot.com/35800044/, william had some issues. i've responded. i feel like there's value in this work. do we need to discuss?23:48

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