=== goodspud_ is now known as goodspud | ||
bac | hi teknico, i'm having trouble deploying the juju-gui charm to an LXC. do you know of any oddities i should be aware of? | 12:43 |
---|---|---|
teknico | bac, hi, I know that it's very, very slow, while on ec2 it's only very slow :-) | 12:44 |
teknico | bac, and the other thing is | 12:45 |
teknico | bac, you need to bootstrap manually and then pass the --no-bootstrap option, in my experience | 12:45 |
bac | teknico: no "go to bed and wake up with it still pending" slow, i assume | 12:45 |
teknico | bac, I remember times of more than half an hour for lxc, and twenty minutes for ec2 | 12:46 |
bac | teknico: i'm not talking about running the tests. i'm talking about bringing up a live environment in lxc. i.e. juju bootstrap && juju deploy local:juju-gui | 12:46 |
teknico | bac, oh, ok | 12:46 |
bac | that's all that should be required, no? | 12:47 |
teknico | bac, yes, but I'm not sure I managed to make it work on lxc, actually | 12:48 |
bac | hmm | 12:48 |
bac | well, my AWS bill was only 18 cents last month. i guess i can splurge. | 12:48 |
* bac wonders what he could even do for $0.18 | 12:48 | |
* teknico lunchtimes | 12:53 | |
gary_poster | goodspud, fwiw I put my comments all over that "user journeys" spreadsheet. Did you see 'em? | 13:14 |
goodspud | gary_poster, I certainly did and thank you very much | 13:14 |
gary_poster | cool, welcome goodspud. looking at the login sketches, thank you | 13:15 |
goodspud | gary_poster, and while you do that I'm going to feed my belly | 13:16 |
gary_poster | have fun :-) | 13:16 |
bac | gary_poster: thanks for the leave policy email. very sensible. | 15:02 |
gary_poster | cool bac | 15:05 |
gary_poster | bac bcsaller benji goodspud hazmat jovan2 (do you still want to be pinged for this, Jovan?) Makyo teknico call in 2 | 15:28 |
gary_poster | benji ping | 15:32 |
benji | gary_poster: pong | 15:32 |
gary_poster | call! | 15:32 |
benji | oh! | 15:32 |
Makyo | Hmm, most recent propose didn't go as planned. Looking into it. | 16:10 |
gary_poster | :-) | 16:10 |
gary_poster | benji, going to have early lunch. did you want to talk about the pastebin really quickly? | 16:10 |
benji | gary_poster: sure | 16:11 |
Makyo | Oh boo, had pushed to an existing branch, so it used an existing mp. | 16:11 |
benji | gary_poster: juju-ui is open | 16:11 |
gary_poster | sounds good benji | 16:11 |
Makyo | Apologies for the spam; going to try that propose one more time. | 16:27 |
hazmat | Makyo, different branch ? | 16:32 |
hazmat | console looks like it was already merged | 16:32 |
hazmat | ah static | 16:32 |
Makyo | hazmat, Yes, console was already merged. Should've pushed to static | 16:32 |
Makyo | I had some checkout problems with the console branch, so it was named funny. Sorry about that :( | 16:33 |
hazmat | no worries | 16:35 |
benji | this is a very cool help system for keybindings in web apps: http://hellopat.github.com/mousetip/examples/ | 16:40 |
Makyo | Oh, that is cool | 16:47 |
teknico | gary_poster, benji, I'm finally done :-) | 19:48 |
teknico | there have been quite a few more changes, some additional reviewing is probably warranted | 19:48 |
benji | teknico: I'll take a look now | 19:48 |
teknico | benji, thanks | 19:48 |
benji | my pleasure | 19:49 |
teknico | I'll go now, I'll have a look tomorrow (my) morning and hopefully land the thing :-) | 19:49 |
benji | gary_poster: do you object to the "release" target being renamed to "dist", it is the "standard" name: https://www.gnu.org/prep/standards/html_node/Standard-Targets.html | 20:12 |
gary_poster | benji not in the least | 20:32 |
benji | cool | 20:32 |
Makyo | Running into the make error we were seeing a day or so ago, in that, starting from a clean repository (not just `make clean`), app/assets/javascripts/d3.v2.min.js does not exist, so bin/merge-files fails. | 21:35 |
Makyo | This is on lp:~teknico/juju-gui/bug-1078910-serve-app-statically (for the nginx charm) | 21:36 |
Makyo | I'll email teknico and do some investigation, but I'm running a little behind on the nginx charm for that reason. | 21:37 |
gary_poster | benji ^^^ | 21:47 |
gary_poster | you were just talking about what Makyo is talking about | 21:48 |
gary_poster | Makyo, or wait | 21:48 |
Makyo | I have a fix, trying to get a patch. | 21:48 |
* benji looks up to sky^Wscrollback. | 21:48 | |
gary_poster | benji, maybe that was bac, sorry :-P | 21:48 |
gary_poster | Makyo, awesome! | 21:48 |
gary_poster | Is it a race condition? | 21:48 |
Makyo | gary_poster, I think a wrong dependency. My fix might be heavy-handed, though. Will include with email to teknico | 21:49 |
benji | it almost certainly is not a race, unless someone is specifying -j when they run make | 21:49 |
gary_poster | benji, it's a node build though | 21:49 |
gary_poster | IOW, the node command might fork | 21:50 |
benji | ah; yeah I don't know what node does | 21:50 |
gary_poster | Makyo, I don't think it is teknico's branch | 21:50 |
Makyo | What's the diff flag for patch style output? :S | 21:50 |
Makyo | gary_poster, Hmm. Let me check with a clean branch of trunk, then. | 21:50 |
gary_poster | Makyo, bzr diff does a nice job I think, and bzr patch consumes it | 21:51 |
Makyo | bzr diff doesn't work on a checkout, rather than a branch. | 21:51 |
gary_poster | Makyo I mean I think the problem pre-exists the branch | 21:51 |
Makyo | Ah, okay | 21:51 |
benji | Makyo: you're looking for -u, I believe | 21:52 |
Makyo | benji, thanks | 21:52 |
gary_poster | Makyo, IOW, I think if you could get a fix infor that pronto that would make a lot opf people happy. If it is quick and small maybe some combination of benji, bac, and I could get it reviewed quickly | 21:52 |
gary_poster | ...a fix in for... | 21:53 |
Makyo | http://pastebin.ubuntu.com/1415682/ | 21:53 |
* gary_poster looks | 21:53 | |
Makyo | gary_poster, will check in a fresh branch, just a sec. | 21:53 |
Makyo | The javascript-libraries target ln's stuff from node to app/assets/javascripts | 21:53 |
Makyo | Which should be a dependency for bin/merge-files | 21:54 |
bac | gary_poster: yeah, it was me | 21:57 |
gary_poster | Makyo, your change looks reasonable, but I don't see how it could actually fix things: javascript-libraries actually resolves to the same thing | 21:58 |
gary_poster | as far as that file is concerned | 21:58 |
gary_poster | well, it fixes things in that the dependency is not declared | 21:59 |
gary_poster | no, it is, right above | 21:59 |
gary_poster | Makyo, bac, the other approach I was thinking of was removing the dependency chain for that collection | 22:00 |
gary_poster | and making javascript-libraries just do everything that it needs to | 22:01 |
Makyo | gary_poster, I believe it javascript-libraries depends on app/assets/javascripts/d3.v2.min.js, which doesn't exist, whereas previously, the build target was relying on the node_modules file, which bin/merge-files doesn't see. | 22:01 |
Makyo | gary_poster, But, basically, so long as app/assets/javascripts/(d3*|yui) exist, merge-files should be happy, from what I can see :) | 22:01 |
Makyo | I'm still figuring out make, I feel a little behind. | 22:02 |
gary_poster | Makyo, don't follow. It's my EoD but would love to see this resolved. Quick hangout, with bac if he wants to, in juju-ui? | 22:02 |
Makyo | Yep | 22:02 |
* Makyo walks dog before he gets eaten. | 22:16 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!