[00:00] https://code.launchpad.net/~dave-cheney/juju-core/170-update-to-mgo-latest/+merge/217839 [00:00] can someone hulk smash this revision [00:00] it will never apply, for reasons obvious from the error in the MP [00:05] mgz: turns out Will had already fixed this test but hadn't landed it in trunk. I've just marked the review as approved (with him sitting next to me). [00:12] mgz: looks like what you need? lp:~hduran-8/goose/json_encoded_errors [00:22] menn0: good job [00:22] perrito666: loooooking [00:27] mgz: those changes landed and the tests pass now on my machine [00:27] mgz: hooray :) [00:30] knock, knock, knock, mgz [00:30] knock, knock, knock, mgz [00:30] perrito666: just got the net back [00:31] comment #1: we want the fallback 500 to include the unknown error string [00:31] perrito666: looks okay past that [00:32] -ify is a bit funny [00:33] mgz: JSON, JSONEncode ? [00:33] AsJSON? [00:34] naming bikeshedding anyone? [00:34] that sounds a bit like fantastic twins :p but I can fix that [00:46] how do I lbox propose something and mark that it depends on another proposal? [00:53] perrito666: it's a flag on propose, do --help [01:02] tedious but easy review please: https://codereview.appspot.com/95920043 [01:03] menn0: you sorted out the appspot login thing? [01:09] mgz: https://codereview.appspot.com/97870044 [05:10] hello all, we're getting a bit hung up on the lifecycle stuff since we're trying to implement a new collection of docs for Action [05:10] is there a technique for getting a unique sequential ID for a new document from txn? === negronjl-afk is now known as negronjl [14:09] hazmat: see where that websocket library now supports py3/2? [15:25] stokachu, awesome! [15:25] stokachu, don't worry someone will rewrite deployer in go [15:26] hazmat: haha [15:26] deployer's logic for waiting on services before relating should go into the juju-core code [15:27] like a 'services pending relation' queue [15:30] jamespage: is there a utopic ami ? [15:30] or cloud image that I can test with ? [15:35] davecheney, http://cloud-images.ubuntu.com/utopic/20140429/ [15:41] jamespage: thanks [15:41] i guess they aren't published to ec2 [15:41] i'm addicted to the ec2 links on the cloud images page [15:46] davecheney, maybe not yet [15:46] davecheney, utlemming would know when [16:18] mgz: strangely, the merge failed https://code.launchpad.net/~hduran-8/goose/testservice_errors/+merge/217818 [16:24] perrito666, vladk|offline, mgz, a quick review about showing networks in status? https://codereview.appspot.com/91000043 [16:28] dimitern: lgtm I added a comment there [16:29] perrito666, cheers! === vladk|offline is now known as vladk [16:35] o/ jcw4 [16:35] \o [16:36] bodie_: I'll start adding the actionids to the unit [16:36] sweet [16:36] i'm hitting up rick for the hangout [16:36] and investigating how the unit would pull the actions off and execute them [16:36] bodie_: cool [16:59] perrito666: that's odd [17:13] https://codereview.appspot.com/96950043 [17:17] dimitern: did a post-landing review [17:24] mgz, cheers [17:43] perrito666: lp:~gz/juju-core/os_listservers_errors_1222413 [17:59] mgz: can you read me? if so pass me again the link, my irc proxy swallowed it [18:00] perrito666: lp:~gz/juju-core/os_listservers_errors_1222413 [18:06] mgz, https://codereview.appspot.com/90970045 updated status as suggested [18:09] perrito666: https://codereview.appspot.com/91010044 [18:10] mgz: rvd [18:11] review please (Python script): https://codereview.appspot.com/94010044 [18:27] mgz: ok, finally got your branch : [18:27] :p [18:28] :D === negronjl is now known as negronjl-afk [18:57] perrito666: landed first stage of your goose branches [18:57] mgz: cool [18:57] mgz: http://4.bp.blogspot.com/-YK_lIV2xxPs/T6iQnXqubbI/AAAAAAAAAUo/SY3DCN5mJzs/s1600/droopy-i-am-so-happy.jpg [19:08] mgz: hey around your code, I cannot return json encoded error, I ether do a sendJSON with the encoded error and return the error or I return the actual error and you call yourself the json encoding [19:10] perrito666: my code? [19:11] or the bit in service_http.go? [19:11] do a sendJSON for now if that's what you need [19:11] the bit on service_http.go [19:11] we want unified error handling later, but shortest path to sane error response is right for now [19:20] mgz: I broke the universe https://pastebin.canonical.com/109483/ [19:20] :p [19:20] will see whats up later, gotta change chairs now [19:24] perrito666: ...what did you do... [19:25] perrito666: looks like your cast is not working [19:25] mgz: nah, its the sendJSON part that is broken [19:26] that I broke that is === vladk is now known as vladk|offline [20:36] perrito666: having any joy? === vladk|offline is now known as vladk [22:13] bodie_: ping [22:26] how do i prevent bootstrap destroying the env if it fails? === Ursinha is now known as Ursinha-afk [22:37] bodie_: bits3rpent - I probably can't make the standup... fyi === Ursinha-afk is now known as Ursinha