[08:39] <davecheney> niemeyer: http://code.google.com/p/go/source/browse/ssh/tcpip_func_test.go?repo=crypto
[08:40] <davecheney> ^ http with custom Dial()
[08:40] <davecheney> possibly something similar could be done with mgo
[08:40] <niemeyer> davecheney: Sweet, thanks
[09:38] <niemeyer> :w
[10:20] <TheMue> Aram: your current proposal has changes to the confignode that williams last one has also
[10:20] <Aram> TheMue: william merged my changes before I merged them into trunk because he needed them.
[10:20] <Aram> but they are the same changes
[10:21] <TheMue> Aram: ok, so it's coordinated
[10:21] <TheMue> Aram: you are also very good identifiable on the team picture :D
[10:22] <Aram> heh
[10:22] <TheMue> the man with the hat
[11:29] <TheMue> lunchtime
[11:29] <davecheney> TheMue: well spotted
[12:07] <niemeyer> TheMue: Heya
[12:07] <niemeyer> TheMue: Welcome back
[12:07] <TheMue> niemeyer: hello
[12:08] <TheMue> niemeyer: the go fmt in the review because imho the "true" is one position to far left
[12:08] <TheMue> niemeyer: maybe it's a display failure in the web
[12:08] <niemeyer> TheMue: It looks in the right place to me
[12:09] <TheMue> niemeyer: then just forget it, it seems i need glasses ;)
[12:12] <Aram> TheMue: lbox now refuses to work if you didn't go fmt beforehand
[12:12] <Aram> lunchy
[12:12] <TheMue> Aram: yes, i've seen it. i already wondered
[12:13] <TheMue> ooops, too late :D
[12:14] <niemeyer> TheMue: Btw, please update it :)
[12:14] <niemeyer> (lbox, that is))
[12:14]  * niemeyer > lunch too
[12:15] <TheMue> niemeyer: has been my first job as i read your mail while i've been on the island :P
[12:15] <TheMue> enjoy your meal
[13:01] <wrtp> davecheney: https://codereview.appspot.com/6553049
[13:03]  * davecheney wrtp looks
[13:04] <TheMue> davecheney: it's a larger one
[13:12] <TheMue> wrtp: you've got a first review
[13:13] <wrtp> TheMue: thanks
[13:14] <TheMue> wrtp: i like the naming as 'state service' instead of the used product
[13:14] <TheMue> state server
[13:14] <wrtp> TheMue: thanks
[13:36] <canesin> Hi all, I would like to know if there is any advance in making juju working with spot instaces
[13:38] <wrtp> davecheney: PTAL
[13:40] <davecheney> kk
[14:10] <wrtp> davecheney: PTAL
[14:13] <niemeyer> juju-db start/running, process 6389
[14:13] <niemeyer> jujud-machine start/running, process 6397
[14:13] <niemeyer> jujud-provisioning start/running, process 6409
[14:13] <niemeyer> $ ps auxw | grep mongo
[14:13] <niemeyer> root      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/db
[14:13] <niemeyer> Just sayin'!
[14:14] <davecheney> wd
[14:15] <davecheney> wrtp: LGTM
[14:46] <niemeyer> environs/cloudinit fixes: https://codereview.appspot.com/6543050
[14:46] <niemeyer> ls
[14:46] <niemeyer> See.. :(
[14:46] <niemeyer> Focus is totally broken
[15:08] <wrtp> niemeyer, davecheney: fairly trivial: https://codereview.appspot.com/6549049
[15:09] <davecheney> wrtp: LGTM
[15:12] <davecheney> ubuntu@server-16690:~/src/launchpad.net/juju-core$ go build ./...
[15:12] <davecheney> # launchpad.net/juju-core/worker/firewaller
[15:12] <davecheney> worker/firewaller/firewaller.go:330: undefined: state.PortsWatcher
[15:12] <davecheney> worker/firewaller/firewaller.go:388: undefined: state.FlagWatcher
[15:12] <davecheney> # launchpad.net/juju-core/worker/uniter/relation
[15:12] <davecheney> worker/uniter/relation/hookqueue.go:24: undefined: state.RelationUnitsChange
[15:12] <davecheney> worker/uniter/relation/hookqueue.go:166: undefined: state.RelationUnitsChange
[15:12] <davecheney> that is all that is left
[15:43] <niemeyer> davecheney: I'm firewaller, and Aram is working on adding stubs to the relation watchers while William doesn't have them
[15:45] <davecheney> kk
[15:46] <niemeyer> https://code.launchpad.net/juju-core/+activereviews
[16:25] <davecheney> i am fixing provisioner_test
[16:25] <davecheney> sorry, i sohld have spoken up a few minutes ago
[16:27] <fwereade> niemeyer, https://codereview.appspot.com/6553051 has EnsureDying/EnsureDead; going back to RelationUnitsWatcher now (at last :()
[16:36] <davecheney> http://paste.ubuntu.com/1218959/
[16:36] <davecheney> environconfigwatcher always blows up if the underlying config never set
[17:01] <fwereade> niemeyer, https://codereview.appspot.com/6553051/diff/1/state/service_test.go#newcode249
[17:04] <davecheney> Aram: func (w *EnvironConfigWatcher) loop() (err error) { settingsWatcher := w.st.watchConfig("e") defer settingsWatcher.Stop() changes := settingsWatcher.Changes() configNode := <-changes
[17:04] <davecheney> configNode := <-changes
[17:20] <Aram> davecheney: https://codereview.appspot.com/6546055
[17:21] <davecheney> Aram: i guess that works, I don't think I am qualified to review
[17:46] <fwereade> niemeyer, reproposed https://codereview.appspot.com/6553051
[17:51] <niemeyer> firewaller is up for review! https://codereview.appspot.com/6548051/
[18:06] <davecheney> Aram: lp:~niemeyer/juju-core/new-state-firewaller
[18:07] <davecheney> do that same hack to invalidateEnvironment
[18:22] <wrtp> davecheney, niemeyer: https://codereview.appspot.com/6554061
[18:23] <niemeyer> ls
[18:23] <wrtp> charm	   container   juju    state	trivial  version
[18:23] <wrtp> cloudinit  downloader  log     store	upstart  worker
[18:23] <wrtp> cmd	   environs    schema  testing	util
[18:23] <wrtp> %
[18:29] <davecheney> wow - i didn't know I could do that
[18:29] <davecheney> lucky(~/src/launchpad.net/juju-core/worker/provisioner) % bzr switch lp:~niemeyer/juju-core/new-state-firewaller
[18:29] <davecheney> Updated to revision 555.
[18:29] <davecheney> Switched to branch: bzr+ssh://bazaar.launchpad.net/~niemeyer/juju-core/new-state-firewaller/
[18:31] <davecheney> ^ ok, don't do this
[18:31] <davecheney> it
[18:31] <davecheney> it'll screw up your cobzr
[18:32] <davecheney> lucky(~/src/launchpad.net/juju-core) % bzr switch 115-testing-testing
[18:32] <davecheney> cobzr: .bzr/branch/location points to bzr+ssh://bazaar.launchpad.net/~niemeyer/juju-core/new-state-firewaller
[18:41] <davecheney> Aram: lp;~dave-cheney/juju-core/001-provisioning-worker/+merge/125794
[18:41] <davecheney> ^ this branch compiles, but errors
[18:41] <davecheney> panics
[19:13] <wrtp> Aram: http://paste.ubuntu.com/1219229/
[19:16] <wrtp> Aram: https://codereview.appspot.com/6547058/