[11:03] hazmat, https://code.launchpad.net/~therve/juju/rapi-uuid/+merge/143662 adds the UUID to the API [11:03] I hope that makes sense, let me know if I miss something [11:10] therve, cool. i've got an in progress branch that requires login.. [11:10] therve, also merge proposal target should be the rollup branch (i resubmitted the mp) [11:11] therve, looks good though [11:11] thanks [11:17] hazmat, thank you! [11:57] gary_poster, benji i don't suppose either of you have that login required communication pastebin link [13:09] hazmat not sure what you are asking for [13:16] gary_poster, just trying to work out if 'login-required' was in the greeting or if there was a separate not auth greeting [13:16] currently we're disclosing provider type and soon uuid in the greeting [13:17] we had a pastebin text doc we passed around as an example [13:17] just trying to locate a ref to it [13:17] oh [13:17] I think I know what you mean [13:18] but benji told me it was pretty different from what ended coming from juju. may be misremembering. I can find, one sec [13:19] hazmat, http://paste.ubuntu.com/1397723/ ? [13:19] gary_poster, thanks [13:19] yeah, that doesn't match reality (at least the reality that existed when I was working on the branch) [13:24] benji, well it wasn't required before... so no. but my current branch makes it required.. just wanted to match more closely to that doc [13:25] sounds good [13:25] bug for these backend changes is bug 1074419 fwiw [13:25] <_mup_> Bug #1074419: Enable simple user & password login to rapi-rollup websocket < https://launchpad.net/bugs/1074419 > [13:34] gary_poster, https://plus.google.com/hangouts/_/7eba46242c127e6b2d38ec9e3db2a8a5898dece9 re rapi [13:34] thanks hazmat. forgot to press Join :-P [14:52] frankban, it strikes me that bac's approach of protecting within the login method is nice in that it also protects from the unlikely but possible situation of someone typing in a password in the login page and submitting it before the connection is ready. I suppose we could not draw the login page in that case...but something else to consider. One other edge case to consider is whether multiple login requests migh [14:52] t be pending. Maybe not an issue, but just came to my mind. [14:53] gary_poster: ack [14:54] thx [14:56] frankban, https://codereview.appspot.com/7138054/ : land as is, thank you [14:57] gary_poster: cool, thanks [14:58] gary_poster: just for curiosity, is there a reason why we don't indent if blocks in our makefile? [14:59] frankban, probably because didn't know we could, within the context of these macros. I think indentation has meaning, and macros don't dedent. If it works though I agree that it would definitely increase readability [15:02] gary_poster: ok, I'll add a slack card to investigate that === _mup__ is now known as _mup_ [15:29] bcsaller, benji frankban goodspud hazmat teknico call in 1 [15:41] http://paste.ubuntu.com/1537964/ [15:41] top 50 charms by download including inflation from gui simulation [15:44] * gary_poster has to run go pick up car from repair. back in 30 [15:48] I've just shared the link in the Canonical juju-gui room to the current set of wireframes for the "vision" piece [15:49] Ale is presenting a reduced set from this to Mark in 10 mins [15:49] "Alejandra" [16:14] I have a review up at https://codereview.appspot.com/7129049 [16:15] on it [16:15] frankban: was I going to review something for you? I don't see anything on the board. [16:15] Nicola beat you to punch [16:16] benji: teknico did that review, thanks [16:16] k [16:33] bcsaller, can you help me with understanding a test behavior? no hurry [16:33] teknico: sure, hangout? [16:34] bcsaller, yeah, https://plus.google.com/hangouts/_/02bb45411739e441fe107c9f66e2a8cc36ba4ba7?authuser=0&hl=en# [16:51] Good news. Mark has given the thumbs up to the concept [16:58] * benji takes an early lunch. [17:23] why is bzr pushing way more stuff than usual? o.O [17:34] congrats goodspud [17:34] benji, land with changes, thank you [19:27] gary_poster: I replied to your review. There are a couple of small discussions therein. [19:28] benji, on call, will look soon [19:28] k [19:54] benji looks great. I'm fine with all of your decisions, but will try to succinctly reply in line to the various questions/points you raised. Wish me luck on the "succinctly" part ;-) [19:54] * benji lends gary_poster a -q [19:54] :-) thanks [19:54] :) [20:03] benji, made simple replies. summary: cool, sounds good, and you should eat chocolate ice cream. [20:04] benji, maybe once you have a new version made you can ping bcsaller and see if he can review. He's the only one we have today. (bcsaller, talking about https://codereview.appspot.com/7069068) [20:05] sounds good [20:06] chocolate is my favorite flavor of ice cream [20:10] :-) [20:32] bcsaller: I will gladly pay you Tuesday for a review of https://codereview.appspot.com/7129049 today. [21:35] benji: I can look it over in a minute [21:36] didn't see this before [23:21] require login for ws branch in review.. https://codereview.appspot.com/6935068/ [23:23] also in card