/srv/irclogs.ubuntu.com/2013/01/17/#juju-gui.txt

thervehazmat, https://code.launchpad.net/~therve/juju/rapi-uuid/+merge/143662 adds the UUID to the API11:03
therveI hope that makes sense, let me know if I miss something11:03
hazmattherve, cool. i've got an in progress branch that requires login..11:10
hazmattherve, also merge proposal target should be the rollup branch (i resubmitted the mp)11:10
hazmattherve, looks good though11:11
hazmatthanks11:11
thervehazmat, thank you!11:17
hazmatgary_poster, benji i don't suppose either of you have that login required communication pastebin link11:57
gary_posterhazmat not sure what you are asking for13:09
hazmatgary_poster, just trying to work out if 'login-required' was in the greeting or if there was a separate not auth greeting13:16
hazmatcurrently we're disclosing provider type and soon uuid in the greeting13:16
hazmatwe had a pastebin text doc we passed around as an example13:17
hazmatjust trying to locate a ref to it13:17
gary_posteroh13:17
gary_posterI think I know what you mean13:17
gary_posterbut benji told me it was pretty different from what ended coming from juju.  may be misremembering.  I can find, one sec13:18
gary_posterhazmat, http://paste.ubuntu.com/1397723/ ?13:19
hazmatgary_poster, thanks13:19
benjiyeah, that doesn't match reality (at least the reality that existed when I was working on the branch)13:19
hazmatbenji, well it wasn't required before... so no. but my current branch makes it required.. just wanted to match more closely to that doc13:24
benjisounds good13:25
gary_posterbug for these backend changes is bug 1074419 fwiw13:25
_mup_Bug #1074419: Enable simple user & password login to rapi-rollup websocket <deploy-story> <juju-gui:Triaged by hazmat> < https://launchpad.net/bugs/1074419 >13:25
hazmatgary_poster, https://plus.google.com/hangouts/_/7eba46242c127e6b2d38ec9e3db2a8a5898dece9 re rapi13:34
gary_posterthanks hazmat.  forgot to press Join :-P13:34
gary_posterfrankban, 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 migh14:52
gary_postert be pending.  Maybe not an issue, but just came to my mind.14:52
frankbangary_poster: ack14:53
gary_posterthx14:54
gary_posterfrankban, https://codereview.appspot.com/7138054/ : land as is, thank you14:56
frankbangary_poster: cool, thanks14:57
frankbangary_poster: just for curiosity, is there a reason why we don't indent if blocks in our makefile?14:58
gary_posterfrankban, 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 readability14:59
frankbangary_poster: ok, I'll add a slack card to investigate that15:02
=== _mup__ is now known as _mup_
gary_posterbcsaller, benji frankban goodspud hazmat teknico call in 115:29
hazmat http://paste.ubuntu.com/1537964/15:41
hazmattop 50 charms by download including inflation from gui simulation15:41
* gary_poster has to run go pick up car from repair. back in 3015:44
goodspudI've just shared the link in the Canonical juju-gui room to the current set of wireframes for the "vision" piece15:48
goodspudAle is presenting a reduced set from this to Mark in 10 mins15:49
goodspud"Alejandra"15:49
benjiI have a review up at https://codereview.appspot.com/712904916:14
gary_posteron it16:15
benjifrankban: was I going to review something for you?  I don't see anything on the board.16:15
gary_posterNicola beat you to punch16:15
frankbanbenji: teknico did that review, thanks16:16
benjik16:16
teknicobcsaller, can you help me with understanding a test behavior? no hurry16:33
bcsallerteknico: sure, hangout?16:33
teknicobcsaller, yeah, https://plus.google.com/hangouts/_/02bb45411739e441fe107c9f66e2a8cc36ba4ba7?authuser=0&hl=en#16:34
goodspudGood news. Mark has given the thumbs up to the concept16:51
* benji takes an early lunch.16:58
teknicowhy is bzr pushing way more stuff than usual? o.O17:23
gary_postercongrats goodspud 17:34
gary_posterbenji, land with changes, thank you17:34
benjigary_poster: I replied to your review.  There are a couple of small discussions therein.19:27
gary_posterbenji, on call, will look soon19:28
benjik19:28
gary_posterbenji 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 -q19:54
gary_poster:-) thanks19:54
benji:)19:54
gary_posterbenji, made simple replies.  summary: cool, sounds good, and you should eat chocolate ice cream.20:03
gary_posterbenji, 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:04
benjisounds good20:05
benjichocolate is my favorite flavor of ice cream20:06
gary_poster:-)20:10
benjibcsaller: I will gladly pay you Tuesday for a review of https://codereview.appspot.com/7129049 today.20:32
bcsallerbenji: I can look it over in a minute21:35
bcsallerdidn't see this before 21:36
hazmatrequire login for ws branch in review.. https://codereview.appspot.com/6935068/23:21
hazmatalso in card23:23

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