/srv/irclogs.ubuntu.com/2012/09/21/#juju-dev.txt

davecheneyniemeyer: http://code.google.com/p/go/source/browse/ssh/tcpip_func_test.go?repo=crypto08:39
davecheney^ http with custom Dial()08:40
davecheneypossibly something similar could be done with mgo08:40
niemeyerdavecheney: Sweet, thanks08:40
niemeyer:w09:38
TheMueAram: your current proposal has changes to the confignode that williams last one has also10:20
AramTheMue: william merged my changes before I merged them into trunk because he needed them.10:20
Arambut they are the same changes10:20
TheMueAram: ok, so it's coordinated10:21
TheMueAram: you are also very good identifiable on the team picture :D10:21
Aramheh10:22
TheMuethe man with the hat10:22
TheMuelunchtime11:29
davecheneyTheMue: well spotted11:29
niemeyerTheMue: Heya12:07
niemeyerTheMue: Welcome back12:07
TheMueniemeyer: hello12:07
TheMueniemeyer: the go fmt in the review because imho the "true" is one position to far left12:08
TheMueniemeyer: maybe it's a display failure in the web12:08
niemeyerTheMue: It looks in the right place to me12:08
TheMueniemeyer: then just forget it, it seems i need glasses ;)12:09
AramTheMue: lbox now refuses to work if you didn't go fmt beforehand12:12
Aramlunchy12:12
TheMueAram: yes, i've seen it. i already wondered12:12
TheMueooops, too late :D12:13
niemeyerTheMue: Btw, please update it :)12:14
niemeyer(lbox, that is))12:14
* niemeyer > lunch too12:14
TheMueniemeyer: has been my first job as i read your mail while i've been on the island :P12:15
TheMueenjoy your meal12:15
wrtpdavecheney: https://codereview.appspot.com/655304913:01
* davecheney wrtp looks13:03
TheMuedavecheney: it's a larger one13:04
TheMuewrtp: you've got a first review13:12
wrtpTheMue: thanks13:13
TheMuewrtp: i like the naming as 'state service' instead of the used product13:14
TheMuestate server13:14
wrtpTheMue: thanks13:14
canesinHi all, I would like to know if there is any advance in making juju working with spot instaces13:36
wrtpdavecheney: PTAL13:38
davecheneykk13:40
wrtpdavecheney: PTAL14:10
niemeyerjuju-db start/running, process 638914:13
niemeyerjujud-machine start/running, process 639714:13
niemeyerjujud-provisioning start/running, process 640914:13
niemeyer$ ps auxw | grep mongo14:13
niemeyerroot      6389  0.6  1.8 127396 30916 ?        Ssl  14:12   0:00 /opt/mongo/bin/mongod --port 37017 --bind_ip 127.0.0.1 --dbpath=/var/lib/juju/db14:13
niemeyerJust sayin'!14:13
davecheneywd14:14
davecheneywrtp: LGTM14:15
niemeyerenvirons/cloudinit fixes: https://codereview.appspot.com/654305014:46
niemeyerls14:46
niemeyerSee.. :(14:46
niemeyerFocus is totally broken14:46
wrtpniemeyer, davecheney: fairly trivial: https://codereview.appspot.com/654904915:08
davecheneywrtp: LGTM15:09
davecheneyubuntu@server-16690:~/src/launchpad.net/juju-core$ go build ./...15:12
davecheney# launchpad.net/juju-core/worker/firewaller15:12
davecheneyworker/firewaller/firewaller.go:330: undefined: state.PortsWatcher15:12
davecheneyworker/firewaller/firewaller.go:388: undefined: state.FlagWatcher15:12
davecheney# launchpad.net/juju-core/worker/uniter/relation15:12
davecheneyworker/uniter/relation/hookqueue.go:24: undefined: state.RelationUnitsChange15:12
davecheneyworker/uniter/relation/hookqueue.go:166: undefined: state.RelationUnitsChange15:12
davecheneythat is all that is left15:12
niemeyerdavecheney: I'm firewaller, and Aram is working on adding stubs to the relation watchers while William doesn't have them15:43
davecheneykk15:45
niemeyerhttps://code.launchpad.net/juju-core/+activereviews15:46
davecheneyi am fixing provisioner_test16:25
davecheneysorry, i sohld have spoken up a few minutes ago16:25
fwereadeniemeyer, https://codereview.appspot.com/6553051 has EnsureDying/EnsureDead; going back to RelationUnitsWatcher now (at last :()16:27
davecheneyhttp://paste.ubuntu.com/1218959/16:36
davecheneyenvironconfigwatcher always blows up if the underlying config never set16:36
fwereadeniemeyer, https://codereview.appspot.com/6553051/diff/1/state/service_test.go#newcode24917:01
davecheneyAram: func (w *EnvironConfigWatcher) loop() (err error) { settingsWatcher := w.st.watchConfig("e") defer settingsWatcher.Stop() changes := settingsWatcher.Changes() configNode := <-changes17:04
davecheneyconfigNode := <-changes17:04
Aramdavecheney: https://codereview.appspot.com/654605517:20
davecheneyAram: i guess that works, I don't think I am qualified to review17:21
fwereadeniemeyer, reproposed https://codereview.appspot.com/655305117:46
niemeyerfirewaller is up for review! https://codereview.appspot.com/6548051/17:51
davecheneyAram: lp:~niemeyer/juju-core/new-state-firewaller18:06
davecheneydo that same hack to invalidateEnvironment18:07
wrtpdavecheney, niemeyer: https://codereview.appspot.com/655406118:22
niemeyerls18:23
wrtpcharm   container   juju    statetrivial  version18:23
wrtpcloudinit  downloader  log     storeupstart  worker18:23
wrtpcmd   environs    schema  testingutil18:23
wrtp%18:23
davecheneywow - i didn't know I could do that18:29
davecheneylucky(~/src/launchpad.net/juju-core/worker/provisioner) % bzr switch lp:~niemeyer/juju-core/new-state-firewaller18:29
davecheneyUpdated to revision 555.18:29
davecheneySwitched to branch: bzr+ssh://bazaar.launchpad.net/~niemeyer/juju-core/new-state-firewaller/18:29
davecheney^ ok, don't do this18:31
davecheneyit18:31
davecheneyit'll screw up your cobzr18:31
davecheneylucky(~/src/launchpad.net/juju-core) % bzr switch 115-testing-testing18:32
davecheneycobzr: .bzr/branch/location points to bzr+ssh://bazaar.launchpad.net/~niemeyer/juju-core/new-state-firewaller18:32
davecheneyAram: lp;~dave-cheney/juju-core/001-provisioning-worker/+merge/12579418:41
davecheney^ this branch compiles, but errors18:41
davecheneypanics18:41
wrtpAram: http://paste.ubuntu.com/1219229/19:13
wrtpAram: https://codereview.appspot.com/6547058/19:16

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