[03:01] jujugui new release is out [03:01] http://jujugui.wordpress.com/2014/11/04/juju-gui-1-2-3-released/ [03:01] urulama: you really here? === kadams54 is now known as kadams54-away [03:21] rick_h_: Nice! [03:23] hazmat: bug fix release out, hopefully helps. ^ [03:25] rick_h_: Is there a flag to view the added services panel? [03:25] huwshimi: /:flags:/as [03:25] huwshimi: it's still a couple of cards from landing, hopefully done later this week [03:25] rick_h_: Oh, I should have looked at the full changelog :) [03:25] :) [03:27] rick_h_: And how do you actually show the add services panel? :) [03:31] huwshimi: deploy something so you have added services? [03:31] huwshimi: and then under the search bar is a button that opens it [03:33] huwshimi: find it? [03:33] rick_h_: Oh, I was trying on jujucharms.com [03:34] huwshimi: oh yea, comingsoon [03:34] huwshimi: since we're trying to bring up the new site didn't bother updating that url yet [03:34] yeah :) [03:34] I'm not a person [03:35] not a person? [03:36] rick_h_: I have no brain? I am not functioning :) [03:36] wait, that wasn't a question [03:36] or was it? [03:38] it was, I didn't follow the 'I'm not a person' line [03:38] but gotcha now, some funky saying :P === kadams54 is now known as kadams54-away [08:26] rick_h_, sweet [08:48] so another gui bug, resolve/retry doesn't go through commit === rogpeppe1 is now known as rogpeppe [14:48] hazmat: hmm, yea that was intentional. The thought was that doing a resolve/retry isn't really a part of a 'changeset' you'd be trying to put together [14:48] hazmat: more of a debugging command that you want to happen [15:06] Makyo: u in yet? [15:10] kadams54: so could you do the added services ui buttons respecting the service visibility state? [15:11] hatch: yeah. [15:11] I don't think that will conflict with my stuff [15:11] right now I'm trying to figure out why ghost services get selected differently by d3 [15:11] :/ [15:12] hatch: I don't see a card for that? [15:12] hmm I was wrong they are getting selected the same... [15:12] oh I don't know if there is one [15:12] lookin [15:13] If not, I can create one [15:13] now there is [15:14] guihelp: anyone know what the diff is between the agent_state_info and agent_state_data fields on a unit? [15:15] ahh now I see the issue - css properties override the dom attributes with svg's [15:15] kadams54: it might be left overs from the pyjuju > juju-core conversion [15:15] why u ask? [15:16] I'd grabbed a card from the maintenance bugs. The GUI is falling over (uncaught exception) when dealing with endpoint errors. [15:17] oh - no reproduction steps [15:17] hah [15:17] The code currently checks unit.agent_state_data when, at least in this situation, that field is not set. From reading code comments, it seems like it should actually check unit.agent_state_info. [15:18] I don't think that's the case [15:18] without repro steps how are we to know if the bug is fixed? [15:18] I'd reply asking for steps on repro [15:19] hatch: it's not that simple [15:19] That's fine, but there's enough info that I can write a test to reproduce the error condition. [15:19] hatch: that bug was from a running env without real steps. There was something when a unit was removed was the comment [15:19] It'll be a 'try to catch this known error by being more defensive' [15:19] well it appears that there was a failure with a unit being removed causing it's agent data to be cleared out [15:19] that shouldn't happen - so the issue is somewhere up the stack [15:19] hatch: right, and that broke the gui [15:20] agent_state_info was still set [15:20] hatch: sure, juju probably did something bad [15:20] We could have looked in there to see that there was an error in the relation [15:20] but we need to try not to break if we can help it. [15:20] Which is what this code is trying to do [15:21] what I'm concerned about is just covering over the issue instead of resolving the root cause [15:21] hatch: understood, but it's not a simple reprdocing issue and there was no work around to help debug further [15:21] *Our* issue is bad error handling. [15:22] We can address that, which may help in uncovering/addressing the root cause. [15:22] +1 [15:22] sure - but lets step back through the sequence of events which would have caused agent_state_data to be undefined [15:22] So I'm reasonably confident that 1) I can write a unit test that reproduces the error condition and will allow us to say that the bug is fixed and 2) that we ought to be checking both agent_state_data AND agent_state_info fields for the name of the relation. [15:23] hatch: sure, walk the code and see what could have emptied it [15:23] what I'd first like to do is figure out the difference between info and data [15:23] I am pretty sure that data is the new field added in juju-core [15:23] for things like relation issues and the like [15:24] Except in this case data isn't there. And info is. And info has the relation issue reported in it. [15:25] right [15:25] but they were different formats [15:25] data was added (I believe) to give us more detailed easier to parse information about the units [15:25] Which is great, if it's there. [15:26] What I'm saying is that our error handling code ought to check data, and if it's not there, fall back to info. [15:26] Instead of just making assumptions that data will always be there. [15:26] that's fine to do, but before you do that first understand what info and data are and why data was removed when info wasn't [15:26] I don't think that's necessary to write good error handling code. [15:27] I think it would be good as a follow-up task. [15:28] so you know that info contains the exact same information as data is checking for? [15:28] But good error handling code ought not make any assumptions. Check data first, then info, and if neither is there, do the best you can… but don't trigger a null error. [15:28] unit.agent_state_data.hook.indexOf(endpoint[1].name + '-' + 'relation') === 0; [15:28] It's checking to see if the relation name is in the hook [15:28] We already know agent_state is "error" [15:29] We also know, from juju status, that this is what was in info: 'hook failed: "homer-relation-joined"' [15:29] So if that bit of code had just checked for the relation name in both fields (in a manner that didn't assume either field existed), it would've worked fine. [15:33] right - but there was a reason why we didn't use that and instead added the data attribute [15:33] I'm just saying - do all the error checking you want, but make sure you understand why data was no longer available [15:34] * hatch pinging Makyo hatch to Makyo, calling all Makyo's [15:35] hatch: as best I can tell, we don't directly set or clear data in our code, which means that it was likely cleared/not set in juju-core. [15:35] ok if that's the case then we need to file a bug with juju-core [15:36] it's removing the data prematurely [15:36] I agree, but we also need to not fall down when juju-core does something wrong. [15:38] we just need to pop this image up http://i0.kym-cdn.com/photos/images/newsfeed/000/774/489/829.jpg [15:40] :-) [15:43] ok I'm going to ignore the not fading ghost services thing and get what I have ready to land - getting ghost services to fade/hide requires changing the d3 fade bits to use css instead [15:43] ^ rick_h_ [15:43] I fixed it so that it no longer throws an error, but it still doesn't fade/hide (ghosts) [15:44] hatch: rgr [15:44] follow-up can do that stuff - but got to get this thing landed so it can be qa'd really hard by peeps :) [15:45] :) [15:51] jujugui call in 10 kanban please [16:00] Makyo: call [16:01] jcsackett: call [16:18] rogpeppe, fabrice: that's interesting about the lack of options. I'm always hearing about how Europeans have more options because they don't have the duopoly we have here. In the US, for the most part, you can get your internet from the phone company or the cable company (if you can get it at all). There's very little competition. [16:18] same here except in big towns [16:19] kadams54: i have provider options but the only connectivity is copper around here [16:19] kadams54: http://www.speedtest.net/my-result/3883355136 [16:19] kadams54: I have a few provider but they all go to the same physical lines [16:19] http://www.speedtest.net/result/3883355136.png [16:19] kadams54: tehy don't bring a lot of different cable to individual house in France [16:20] rogpeppe: Yeah, copper is also the only option for a lot of places in the US. Not sure if fiber would be an option where I live, but it's moot because I already know it's out of my price range. [16:20] that is quite a bit crapper than usual [16:20] * rogpeppe goes to complain to the ISP [16:20] kadams54: but if I was on fiber it will be the same price 30 euros [16:21] kadams54: did I mention fiber is cheaper than copper here? ;) [16:21] (because it damn well should be) [16:21] hatch: yeah, I kinda hate you. [16:21] Every time you complain about the fiber being put in. [16:21] well it's slow as crap right now [16:21] 600kbps [16:21] :P [16:22] they say they are fixing it, but I'm just glad they aren't digging in my yard anymore [16:22] I am actualy surprised that more places don't have crown owned telco's [16:23] crown being govt (not sure if that terminology crosses borders) :) [16:23] Why's that? [16:24] As an American, the only thing worse (to me) than the current duopoly would be a government-owned monopoly. [16:24] because a crown corp can run at a low margin because it's goal is to provide a quality service, not fatten the wallets of some billionares [16:24] so (in theory) you would get better service for less $ [16:25] You have a high opinion of the ethics of bureaucrats and politicians. [16:25] ;-) [16:25] our telco is run as it's own company (owned by the govt) [16:25] and it's hugely profitable [16:25] and we have 2 other competitors here [16:25] which basically means the 'big corps' are screwing everyone [16:26] because this is proof that you can provide a quality service with low population over large distances [16:26] I think we barely have 1M people [16:28] * kadams54 scratches his head. [16:28] I like how you say everyone's being screwed but then turn around and say "quality service" in the next statement :-) [16:28] hatch, standup? [16:29] right - I am not on a big corp telco [16:29] Makyo: sure joining [16:32] interesting, it might bit my wi-fi playing up. i tried plugging in directly and got a much better result http://www.speedtest.net/result/3883389667.png [16:32] wow, indeed [16:32] Though that up is still painful [16:32] it's supposed to be 802.11n too. [16:33] You know what that means… [16:33] Time to go get an 802.11ac access point :-) [16:33] there's definitely something playing up in one of my ether switches recently - one my two wi-fi networks refuses to a NAS box in the attic that's attached to the other [16:46] Makyo: so that got it to work, unfortunately it exposed another bug :/ lol [16:46] hatch, sigh, programming 9.9 [16:48] Makyo: I don't understand your new-age-emoji [16:49] what ever happened to the classics??????? [16:49] It's eye-rolling. Come on :P [16:49] really?? [16:49] I thought it was crying [16:49] Yeah, like the . is the nose and the 9s are the eyes [16:49] man I would suck as a teen now [16:49] 9.6 [16:49] Hahaha [16:49] stoned [16:49] that one is stoned right? [16:49] Haha, sure :) [16:50] henceforth be known as The Denver [16:50] Which makes a "Denver omelet" kind of spectacular [16:51] haha [17:23] kadams54: after merging in develop again I'm having issues with the unhighlight event order of events for making sureonly one thing can be highlighted at a time [17:24] I think it will need to be changed to work as a method instead of an event - or the event will need to carry with it the token that it's supposed to skip [17:24] I swear this code never ends [17:25] it appears to only be out of sync with ghost services?!? [17:26] yeah - looks like the time it takes to parse the service list is enough to cause the events to get out of order [17:35] hatch: I'd like to try a patch [17:36] hatch: http://pastie.org/private/spwbpol1p5aqbjovclha [17:36] That will eliminate all of the unecessary events. [18:01] kadams54: cool Ill give it a go [18:01] thx === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [18:35] hatch: did that patch help? [18:35] not sure - had started lunch, broke for a qa, now lunching again :) [18:50] jujugui tiny branch for review/qa https://github.com/juju/juju-gui/pull/639 [18:50] Makyo: looking [18:56] Makyo: looks good [19:23] kadams54: unfortunately that patch didn't work - the service highlights but the token unhighlights leaving the service highlighted [19:23] very odd [19:23] OK… give me a few minutes and then maybe we can pair? [19:25] I was just going to leave it until I can get this branch landed [19:26] need to get this branch in then we can follow it up with fixes [19:26] get people using it and such [19:27] Yeah, that works too [19:27] Can you be sure to include that patch in your branch? [19:27] Even if it doesn't address this issue, it still needs to be in there. [19:30] * Makyo zoom to appointment, back in a bit [19:30] yep can do [19:36] jujugui: looking for reviews on https://github.com/juju/juju-gui/pull/640 - not sure how possible QA is. [19:36] hatch: thanks [19:51] well THAT was a violent crash [19:53] Eh? [19:54] vm took down the whole computer === kadams54 is now known as kadams54-away [20:50] http://www.codingame.com/home/platinum-rift coding challenge battle heh [20:58] mhall119: hey can employees of Canonical participate in the scopes callenge? [21:02] hatch: not this time, no [21:03] mhall119: booo [21:03] hatch: but don't let that stop you from writing a scope :) [21:03] haha [21:03] I just wish it opened faster [21:15] Makyo: I'm investigating the outline issue you noticed. turns out something was changed in this branch that's causing this line to not work [21:15] curr_node.attr({'xlink:href': new_href}); [21:15] new_href is the proper path but the icon isn't being updated [21:18] oh I think I found it [21:28] Makyo: d3 question https://github.com/hatched/juju-gui/blob/canvas-fade/app/views/topology/service.js#L1494 [21:29] I need to determine while looping through these selections what to set the icon to [21:31] so I was thinking selection.each(function(d) ... ) buuut it doesn't look like I can get from the object 'd' to the element to set it [21:31] I'm sure I'm missing something obvious though [21:32] ohhh `this` is the element [21:32] I knew it was something obvious :) [21:34] selection.each(function(d) { d3.select(this).select('.service-block-image'); }); [22:02] Morning [22:02] mornin! [22:03] hatch: Hey :) === kadams54-away is now known as kadams54 [22:15] jeesh playing wakamole with these tests [22:16] I've literally had 5 failing tets for 4 hours and been fixing them constantly === kadams54 is now known as kadams54-away [22:20] * Makyo returns. [22:20] Problem solved, hatch ? [22:21] yeah that one was 2 problems ago [22:21] :/ [22:21] these d3 tests use so many kludges that it's trying to figure out wth is going on first [22:21] Yeah :/ [22:23] can a subordinate relation not go bad? [22:24] right now I'm working on https://github.com/juju/juju-gui/blob/develop/test/test_environment_view.js#L1165 [22:24] this is failing because iRemainUnchanged isn't in the path [22:25] but the path is correct without that [22:25] so I think that this test is just wrong [22:25] but I am not sure what it's actually testing :/ [22:25] Makyo: maybe you can shed some light on this one ^ [22:31] Makyo: it was actually you who added it in response to some review comments c858ac02 [22:32] https://github.com/juju/juju-gui/commit/c858ac02 [22:33] not entirely sure why that change was made [22:33] https://github.com/juju/juju-gui/commit/f37d4787a1675d13e9dc4cc4cd42e206b04a9c6b [22:34] not sure how to get to the PR from there (unless this was a reitveld one) ? [22:39] after a bunch of testing it doesn't appear to be necessary any longer [22:39] removing [22:45] holy I'm now down to 4 tests failing lol === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 === kadams54 is now known as kadams54-away === kadams54-away is now known as kadams54 [22:57] jujugui: still looking for reviews on https://github.com/juju/juju-gui/pull/640 [22:58] Wooo~