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

=== Makyo is now known as Makyo|out
=== _mup__ is now known as _mup_
gary_posterteknico, pair in 5 or so?13:48
teknicogary_poster, yep, thanks13:48
teknicogary_poster, https://plus.google.com/hangouts/_/02bb45411739e441fe107c9f66e2a8cc36ba4ba7?authuser=0&hl=en13:51
benjiHouston, we've had a problem.14:09
benjigary_poster: do you know how to deploy a different backend to the ui stage?14:09
benjiWe need to do so (or revert the login branch) before uistage will work again.14:10
gary_posterbenji, yes.  bac, you around to help or shall I? I'm on a call with Nicola atm14:11
bachi benji, gary_poster: yes i can help14:11
gary_posterthanks bac14:11
benjibac: great, thanks14:11
benjigary_poster: thank you14:11
gary_posternp14:11
bachazmat: ping14:32
gary_posterbenji, it looks like you only get one chance for a password.  If you type in a bad password, then it will reprompt you but it doesn't matter if you type in a good or bad password then: it will fail and you will never be challenged again.14:47
benjigary_poster: yep, that is a bug in the backend; I was just trying to figure out where to file that and whether or not I should dig into it14:47
gary_postergotcha14:48
benji(all channels with hyphens in their names are not the same)14:48
gary_poster:-)14:48
benjigary_poster: since I have your attention, my best guess is to make a card and send an email to the list; any better ideas?14:48
gary_posterbenji gotcha.  I'm talking with Nicola about the next UI login card.  Yeah, please make a bug/card and send an email to list.  I might have things to add but can do it there14:50
benjicool, thanks14:50
gary_posterbenji, join us at https://docs.google.com/a/canonical.com/file/d/0B1IM--9A1RkTaTUwZjVDVWlybFk/edit for a sec maybe so we can pick your brain?14:51
benjigary_poster: k14:51
gary_posterthanks14:51
gary_posterbenji sorry https://plus.google.com/hangouts/_/02bb45411739e441fe107c9f66e2a8cc36ba4ba7?authuser=0&hl=en14:52
=== Makyo|out is now known as Makyo
hazmatbac, pong15:20
bachazmat: on uistaging ~/juju was updating.  it looks like it was changed to be a lightweight checkout and the cron 'bzr pull' was failing.15:21
bachazmat: i manually updated using 'bzr update' and changed the script.  just letting you know.15:21
hazmatbac  cool, thanks15:21
bacs/was updating/ was NOT updating/15:21
hazmatbenji, looking into login issue15:26
benjihazmat: cool; let me know if there is anything I can do15:26
hazmatbenji, have you verified from the console that the correct credentials are being sent on the ws?15:28
hazmatjs console15:28
hazmatbenji, the unit test for login, does an invalid login, and then a valid login.15:29
benjihazmat: not js console, but the backend reports the correct credentials on its stdout15:29
benjihmm15:29
gary_posterbac bcsaller1 benji frankban goodspud Makyo teknico call now (hazmat is sick I think)15:30
hazmatbenji, fwiw i'm referencing juju/rapi/tests/test_context.py15:30
hazmati'll try closer to the transport15:31
Makyohangout troubles..15:31
benjifrankban: there is a global you can set that will disable login attempts (that will only work as long as the backend ddoesn't actually require credentials)15:34
frankbanbenji: cool. however, could you remind me what the password is?15:36
benjifrankban: "admin"15:36
frankbanbenji: ack, ty15:37
benjinp15:37
teknicosorry, phone call at the wrong time15:51
gary_posterteknico, back to https://plus.google.com/hangouts/_/02bb45411739e441fe107c9f66e2a8cc36ba4ba7?authuser=0&hl=en16:13
gary_poster?16:13
teknicogary_poster, yep16:13
robbiewdamn..yet another irc channel I need to be in...time to prune my list17:36
goodspudApologies for missing the meeting today. Alejandra and I were having a very long pow-wow and lost track of time.18:00
bacbcsaller1: your viewport branch has some confusing conflicts18:21
bcsaller1bac: vs trunk you mean? I'm doing the merge now18:37
bacthanks bcsaller1.  yeah, that's what i meant.18:37
bcsaller1bac: I think it was in the tests due to the merge of kapils stuff, did you see something else?18:38
bacbcsaller1: test/index app.js and mega i think18:38
benjiMakyo: do you have a minute to help me?  I am working on hiding the gui's charm in the environment view and I have it hidden when it is first rendered, but it becomes visible if I move any other service because of the .show() method.18:47
benjiPerhaps I should give it a class with "display: none !important" or "opacity: 0 !important".18:48
Makyobenji, sure, I've got a few.18:50
benjiMakyo: what do you think about the ideas above?18:51
Makyobenji, give me a sec to check out show again, it's been a bit.18:51
benjik18:52
Makyobenji, I think either the 'display : none !important' or modify the selection passed to show would both work.  Something like topo.fire("show", vis.filter(function(d) { return d.id !== "juju-gui"; })) or however it works.  However, and maybe bcsaller1 would be the one to talk to about this, maybe it'd be best to just not draw it at all?18:59
benjiMakyo: thanks for the info.  Re. not drawing it: that is certainly a possibilty, but at this point I haven't been able to fine the right point to "cut" to get that behavior because so much code assumes that every service has a DOM representation19:00
MakyoThat'd be in mega.js in updateData.  Like services = services.filter(function(d) { return d.id !== 'juju-gui' }); or whatever.19:00
bcsaller1yeah, I'd filter the input before it got that far, maybe even make an exception at the delta stream to ignore the name and not create a model19:00
Makyobefore this.services = services;19:00
bcsaller1I don't think this is an env view issue, its a data issue19:01
benjiMakyo: unfortunetely that doesn't work, other code assumes that if there is an entry in db.service then there is a parallel DOM node19:01
benjibcsaller1: I've considered that approach but I strongly suspect it will be even messier.  E.g., we also have to purge any reference to the GUI, like relations19:02
MakyoAh, hmm..19:02
Makyorelations are only drawn in the env view if they have two endpoints, but I suppose that'd be an issue elsewhere..19:03
benjiright now the display: none approach is half way working and I /think/ it will end up being cleanest, since this is a display issue addressing it in the display layer may cause the least pain.19:04
Makyobenji, all relations pointing to a hidden service (we had been talking about a class of service that wouldn't be displayed, dunno if that's still the case) would have to be made invisible, as well, and also in the service view, because you can destroy/unexpose a service from there or follow relations to the hidden service.19:14
benjitep19:16
benjis/t/y/19:16
bacbcsaller1: was the essence of your zoom fix just turning off the mousewheel so that the default d3 zoom actions don't resize the rect?19:28
bcsaller1bac: the essence was putting the rect in the proper place, syncing it with the viewport so its the proper size and then (like you mentioned) passing the events properly to vis. The way it was most different was placing the rect as a sibling of the vis so its not scaled with it.19:30
bacgary_poster: what do you want to do with the zoom card?  just have it track ben's viewport one?  delete it?  putting it in review will be confusing.19:38
gary_posterbac, yeah good question.  move it to landing (?)19:39
baci thought about that.  ok, will do19:39
* bac -> dogwalk20:55
* Makyo dogwalks.22:22

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