/srv/irclogs.ubuntu.com/2012/12/06/#juju-gui.txt

=== goodspud_ is now known as goodspud
bachi 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
teknicobac, hi, I know that it's very, very slow, while on ec2 it's only very slow :-)12:44
teknicobac, and the other thing is12:45
teknicobac, you need to bootstrap manually and then pass the --no-bootstrap option, in my experience12:45
bacteknico: no "go to bed and wake up with it still pending" slow, i assume12:45
teknicobac, I remember times of more than half an hour for lxc, and twenty minutes for ec212:46
bacteknico: 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-gui12:46
teknicobac, oh, ok12:46
bacthat's all that should be required, no?12:47
teknicobac, yes, but I'm not sure I managed to make it work on lxc, actually12:48
bachmm12:48
bacwell, 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.1812:48
* teknico lunchtimes12:53
gary_postergoodspud, fwiw I put my comments all over that "user journeys" spreadsheet.  Did you see 'em?13:14
goodspudgary_poster, I certainly did and thank you very much13:14
gary_postercool, welcome goodspud.  looking at the login sketches, thank you13:15
goodspudgary_poster, and while you do that I'm going to feed my belly13:16
gary_posterhave fun :-)13:16
bacgary_poster: thanks for the leave policy email.  very sensible.15:02
gary_postercool bac15:05
gary_posterbac bcsaller benji goodspud hazmat jovan2 (do you still want to be pinged for this, Jovan?) Makyo teknico call in 215:28
gary_posterbenji ping15:32
benjigary_poster: pong15:32
gary_postercall!15:32
benjioh!15:32
MakyoHmm, most recent propose didn't go as planned.  Looking into it.16:10
gary_poster:-)16:10
gary_posterbenji, going to have early lunch.  did you want to talk about the pastebin really quickly?16:10
benjigary_poster: sure16:11
MakyoOh boo, had pushed to an existing branch, so it used an existing mp.16:11
benjigary_poster: juju-ui is open16:11
gary_postersounds good benji16:11
MakyoApologies for the spam; going to try that propose one more time.16:27
hazmatMakyo,  different branch ?16:32
hazmatconsole looks like it was already merged16:32
hazmatah static16:32
Makyohazmat, Yes, console was already merged.  Should've pushed to static16:32
MakyoI had some checkout problems with the console branch, so it was named funny.  Sorry about that :(16:33
hazmatno worries16:35
benjithis is a very cool help system for keybindings in web apps: http://hellopat.github.com/mousetip/examples/16:40
MakyoOh, that is cool16:47
teknicogary_poster, benji, I'm finally done :-)19:48
teknicothere have been quite a few more changes, some additional reviewing is probably warranted19:48
benjiteknico: I'll take a look now19:48
teknicobenji, thanks19:48
benjimy pleasure19:49
teknicoI'll go now, I'll have a look tomorrow (my) morning and hopefully land the thing :-)19:49
benjigary_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.html20:12
gary_posterbenji not in the least20:32
benjicool20:32
MakyoRunning 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
MakyoThis is on lp:~teknico/juju-gui/bug-1078910-serve-app-statically (for the nginx charm)21:36
MakyoI'll email teknico and do some investigation, but I'm running a little behind on the nginx charm for that reason.21:37
gary_posterbenji ^^^21:47
gary_posteryou were just talking about what Makyo is talking about21:48
gary_posterMakyo, or wait21:48
MakyoI have a fix, trying to get a patch.21:48
* benji looks up to sky^Wscrollback.21:48
gary_posterbenji, maybe that was bac, sorry :-P21:48
gary_posterMakyo, awesome!21:48
gary_posterIs it a race condition?21:48
Makyogary_poster, I think a wrong dependency.  My fix might be heavy-handed, though.  Will include with email to teknico21:49
benjiit almost certainly is not a race, unless someone is specifying -j when they run make21:49
gary_posterbenji, it's a node build though21:49
gary_posterIOW, the node command might fork21:50
benjiah; yeah I don't know what node does21:50
gary_posterMakyo, I don't think it is teknico's branch21:50
MakyoWhat's the diff flag for patch style output? :S21:50
Makyogary_poster, Hmm.  Let me check with a clean branch of trunk, then.21:50
gary_posterMakyo, bzr diff does a nice job I think, and bzr patch consumes it21:51
Makyobzr diff doesn't work on a checkout, rather than a branch.21:51
gary_posterMakyo I mean I think the problem pre-exists the branch21:51
MakyoAh, okay21:51
benjiMakyo: you're looking for -u, I believe21:52
Makyobenji, thanks21:52
gary_posterMakyo, 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 quickly21:52
gary_poster...a fix in for...21:53
Makyohttp://pastebin.ubuntu.com/1415682/21:53
* gary_poster looks21:53
Makyogary_poster, will check in a fresh branch, just a sec.21:53
MakyoThe javascript-libraries target ln's stuff from node to app/assets/javascripts21:53
MakyoWhich should be a dependency for bin/merge-files21:54
bacgary_poster: yeah, it was me21:57
gary_posterMakyo, your change looks reasonable, but I don't see how it could actually fix things: javascript-libraries actually resolves to the same thing21:58
gary_posteras far as that file is concerned21:58
gary_posterwell, it fixes things in that the dependency is not declared21:59
gary_posterno, it is, right above21:59
gary_posterMakyo, bac, the other approach I was thinking of was removing the dependency chain for that collection22:00
gary_posterand making javascript-libraries just do everything that it needs to22:01
Makyogary_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
Makyogary_poster, But, basically, so long as app/assets/javascripts/(d3*|yui) exist, merge-files should be happy, from what I can see :)22:01
MakyoI'm still figuring out make, I feel a little behind.22:02
gary_posterMakyo, 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
MakyoYep22: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!