=== mattuk1972_ is now known as mattuk1972 === _mup__ is now known as _mup_ [11:45] this might not be the right channel… but.. is there some docs about the juju RAPI? [11:52] dguerri, hi [11:54] hi, teknico [11:55] dguerri, not that I know of [11:58] dguerri: you could find some info on setting up the rapi-rollup agent in the documentation included in juju-gui: http://bazaar.launchpad.net/~juju-gui/juju-gui/trunk/view/head:/HACKING [12:03] thanks frankban but I'm thinking about writing a ruby binding for the juju restful API [12:09] dguerri: it's a websocket API, it makes use of good rest principles, but still it's not a rest API. Unfortunately, my I am not aware of a place where the stream is documented. [12:10] mmmh, so it might not be the right interface for a lang binding … [12:12] morning frankban. how goes the effort? [12:19] bac: it's under review, you can glance at https://codereview.appspot.com/6977043/ . Tests results are different based on who runs the suite. [12:20] frankban: oh, that's good. keeps people on their toes [12:20] dguerri: I guess it depends on what you want to achieve [12:20] bac: :-) [12:23] frankban: is lp2kanban busted? i see your defect in done-done has a bug but it isn't populated [12:23] bac: it seems so [12:25] bac: the same for other two cards in "ready to code" [12:25] yep. i'll look into it [12:38] * frankban lunches [13:06] frankban: lp2kanban is undead again [13:11] hey benji. I thought a fun improvement I could tackle today in very limited time might be to update the release docs a bit. if that sounds ok to you and you are available to talk through things with me so I can pick your brain sometime, lemme know [13:11] gary_poster: sounds like a plan [13:11] cool [13:12] ty [13:16] bcsaller, did you see what I meant about the rietveld review? In a bit I'll make my own diff [13:17] bac: cool thanks [13:17] it is not very robust wrt ill configured boards and projects [13:18] bac, what cam we improve for poor configuration [13:18] thank you for fixing btw [13:18] gary_poster: i just added some exception handling around the call to the board processing. so if someone ( green) isn't configured their processing stops but other boards continue [13:19] heh [13:19] ok cool thanks bac. you shot a note to gmb too? [13:19] i've pinged him, waiting for him to return from lunch [13:19] cool [13:20] brits and their ill-timed lunches [13:20] Europeans and their ill-timed lunches! Americans and their ill-timed desire to sleep through till the morning! [13:20] :-) [13:24] Humans and their ill-timed biological needs. [13:54] gary_poster: I was able to fix it, still don't know why it happened before [13:54] bcsaller, cool, reviewing, thanks. [14:01] teknico, you are going to start bug 1074413, I'm guessing [14:01] <_mup_> Bug #1074413: GUI charm should require HTTPS connections to the environment by default

< https://launchpad.net/bugs/1074413 > [14:02] bac, I made room for you on secondary story to start tests ofthe pan/zoom module, based on bcsaller [14:02] 's branch in review [14:02] hazmat if you could land your branch that would really help us keep the flow moving [14:04] gary_poster, eta 1hr [14:04] thanks [14:09] gary_poster, yep [14:10] gary_poster: ok [14:18] gary_poster: there is a card for bug 1083935 but it is blocked by 1077050 and 1077051. only the latter seems to be fixed. unblock it and carry on? [14:18] <_mup_> Bug #1083935: Environment pan/zoom code is not tested sufficiently < https://launchpad.net/bugs/1083935 > [14:34] bac, yes, that's based on plans that have been superceded. You are still blocked on Ben's branch, but since you are working from it, it's ok [14:34] bcsaller, quick checkin? [14:34] yeah [14:45] hi bcsaller, i just grabbed your current branch. when i try to run the tests i get a time out on the first test and it stops. anything different about running tests with your new work? [14:47] bac; left a .only on one of the test that _is_ having issues, reproposed the branch w/o that , sorry [14:48] bcsaller: np, will grab new version [14:50] bcsaller: cool, it now works except for that one test failure [15:11] gary_poster: hum... it seems that our debug server does not contain everything needed. I see this in the js console after switching to staging in a deployed charm: http://pastebin.ubuntu.com/1452506/ [15:12] frankban, ack on call will look soon [15:16] gary_poster: thanks, and... problems also in the production server: http://pastebin.ubuntu.com/1452512/ [15:16] teknico, ^^^ ? [15:17] will look asap [15:17] gary_poster, looking [15:20] frankban, so, what files are missing? [15:24] teknico: I don't know, my understanding is that https, refusing to load external insecure resources, exposes a problem we have: our releases don't contain everything required by the GUI, and the YUI loader tries to fetch libraries from the net (e.g. ellipsis, markdown, timer) [15:24] ah, I see [15:25] sorry I didn't QA enough to catch that [15:25] teknico, previous call ran over, sorry. talk after standup? [15:25] gary_poster, sure [15:25] frankban, teknico we may need to revert the https code to let frankban proceed :-/ [15:26] gary_poster, right. I still did not manage to see tests complete successfully :-/ [15:28] teknico, I am afraid that this is going to be a very serious problem [15:28] bac bcsaller benji frankban goodspud hazmat Makyo teknico call ni 2 [15:28] gary_poster: I am not blocked, and I already merged trunk and resolved conflicts. And, AFAICT, this is a bug of the GUI, not of the charm. However, is uistage broken? [15:28] frankban, ui stage is not broken [15:29] because not served from charm, or from httos [15:29] https [15:29] but this means that the charm is now unusable [15:29] gary_poster: yes [15:30] yes the bugfix goes in the gui [15:30] but meanwhile the charm is broken [15:30] frankban, bcsaller starting at X:31 :-) [15:30] gary_poster: so you are right, better to revert the changes. Even better if someone can confirm what I see. [15:40] teknico, http://bazaar.launchpad.net/~hazmat/juju/rapi-rollup/view/head:/juju/agents/api.py#L153 [15:41] hazmat, great, thanks [15:54] bcsaller, I think I figured out the resize thing. [15:54] Makyo: ha, sweet [15:55] bcsaller, it's a little unfortunate :| Resize events are only fired when bound using Y.on(), which we're not doing with the module event binding because of publish. If I bind it in initializer in mega.js how it used to be bound, it works. [15:56] Makyo: look at the change in the yui bindings in d3-comp though, it should cover that case, thats where I was getting stuck [15:59] bcsaller, Ah, I see it now. Passing window as context gets the event to fire, at least, but then this is set to the YUI window node, rather than the module. Is that the problem? [15:59] the handler won't be able to pull the state it needs to run I think [16:01] hazmat, let me know when you are free to talk about PPAs, login, etc [16:01] goodspud, ack, 5m max [16:01] eta [16:01] cool [16:09] goodspud, juju-ui hangout [16:10] bcsaller, after target = Y;, add handler.context = null; and it works. [16:10] Not...sure why it doesn't like the context in there. [16:11] Makyo: changing the context on a window event breaks it maybe... that is odd [16:16] teknico, would you like to have a call, or postpone to tomorrow, or...? [16:18] bcsaller, does that get us closer to where we need to be? [16:18] Makyo: I think, let me finish testing it [16:19] gary_poster, a call today would be better, I'll get back to you in a few minutes, thanks [16:19] Makyo: yes, thanks alot, that working [16:19] Whew [16:20] Makyo: the test is still failing, but I can track that down now that the event is properly handled [16:22] Makyo: got it, going to push again :) [16:22] gary_poster: re tls, e have another problem: we want, in the GUI to fetch data from http://jujucharms.com/ [16:24] gary_poster: makyo found the issue and I've pushed the updates, should I go ahead and land it now? [16:37] frankban, I guess we can ask webops to enable https://jujucharms.com/ :-) [16:46] gary_poster, re: call, main thing would be making tests work, if you could help me there it'd be great, otherwise we can postpone [16:55] bcsaller, please land, yes [16:55] thank you [16:56] frankban, :-( hazmat do you have control over jujucharms.com? Can you set up a cert and give us https access? [16:57] gary_poster: woot :) [16:58] bcsaller, reminder to please make the bug/card to fix the initial prod test run, or ask me to (happy to) :-) [16:59] teknico, https://codereview.appspot.com/6977043/#msg5 has Francesco's charm debug notes. They are very good. [17:00] gary_poster, yes, I read that email and saved it for future reference :-) [17:00] :-) [17:00] gary_poster, at first sight it did not help with my problem though, I'll have another look [17:01] teknico, pair for a few minutes? [17:01] up to you, only if you want to [17:01] gary_poster, i do .. what do you need? [17:01] gary_poster, sure [17:01] gary_poster, i'm a little confused [17:01] gary_poster, g+ for a few minutes ;-) [17:01] hazmat, GUI over https needs to talk to jujucharms [17:01] ok :-) [17:01] hmm [17:02] it would be good to do so.. yes [17:02] but required.. not sure.. [17:02] does the browser toss up any thing for async ajax request [17:02] if its not https from an https page? [17:02] well not required atm is the questoin [17:02] don't think so, in Chrome anyway. frankban, ^^ ? [17:03] if not then its more a pending task that we can toss in the kanban .. just trying to determine immediacy of action [17:03] I mean, I think it fails silently. frankban has immediate experience [17:03] hmm.. i guess the websocket is a subresource that fails .. so its similiar [17:04] right [17:04] but thats still a bit different since that what's we wanted encrypted [17:04] hazmat: each request to connect to jujucharms is blocked by the browser, including charm info (e.g. one of the service detail tabs). [17:04] frankban, silently blocked, yeah? in chrome? [17:05] I see silent blocks in chrome, generally, and warnings you can bypass in FF [17:05] the issue with the websocket was its encrypted but with an non user auth'd cert [17:05] hmm [17:05] gary_poster: in chrome yes, errors can be seen only in the js console [17:05] frankban, what's the error msg in the console? [17:06] hazmat: similar to this one, but pointing to jujucharms.com: http://pastebin.ubuntu.com/1452506/ [17:06] we're sending cors headers but their use case is the reverse.. [17:07] frankban, gotcha [17:07] ok so its immediate. checking cert prices [17:09] gary_poster: commented the code enabling TLS in the charm, last test run and then I'll land the branch [17:09] thanks frankban [17:18] gary_poster, progress! I see machines running already :-) [17:18] teknico, excellent :-) [17:46] frankban, do you have a plan for attacking the gui https bits we talked about?I might be able to push it forward after your EoD if you do. If not, no worries [17:48] gary_poster: I think a plan could be to found a way to configure the YUI loader so that it never tries to download dependencies from the outside world [17:48] s/to found/to find/ [17:48] frankban, right. That's what thiago tried at one point. That's probably the thing to try first [17:53] bcsaller i started reading your code to understand what you did and then ended up doing a review. [17:53] bac: thanks, its submitted, but I can make changes in the next branch if need be [17:53] * gary_poster restarts then lunches [17:53] bcsaller: ok, that's be good === gary_poster is now known as gary_poster|away [17:56] Happy Yule Time Hugs everyone. Enjoy your holidays. I'm out of here. === gary_poster|away is now known as gary_poster [18:45] benji, when you have a quick sec to explain/remind what the expected release pattern is, please lemme know [19:04] gary_poster: the branch finally landed \o/ . It seems to me that some modules (like gallery-ellipsis or gallery-timer) are not included just because they cannot be found under node_modules/yui. [19:04] yay frankban [19:04] hm [19:05] frankban, so they are not in the compressed file either [19:05] so perhaps we just copy them over and serve them [19:05] I'll try that :-) [19:05] if I have time [19:06] thanks and have a good evening frankban [19:07] gary_poster: it seems so (they are not in all-yui.js). you too [19:19] gary_poster: I am back from lunch and available at any time. [19:21] thanks benji. ony 9 min till next call, so maybe later :-) [19:21] gary_poster: k [20:26] bcsaller, ping [20:27] gary_poster: hey, whats up? [20:28] hey bcsaller. I am doing exploration on the https issues and messing around with things. modules-debug.js gives this warning, which I vaguely recall: [20:28] / The "requires" property should not be used here because the javascript [20:28] / minimizer will not parse it. [20:29] juju-topology requires files here, and then dupes those in its own definition [20:29] Is it OK with you to rip out the ones in modules-debug, since AFAICT they are a dupe, and can't be used? [20:29] (when compressed) [20:31] gary_poster: yeah [20:31] cool thanks bcsaller [20:31] thank you [23:52] * hazmat gary_poster fingers crossed [23:52] whhops [23:52] happy holidays folks