[14:26] <frankban> uiteam I need two reviews for https://codereview.appspot.com/195690043 (quickstart/python)
[14:26] <frankban> thanks!
[14:36] <hatch> on it frankban 
[14:37] <frankban> hatch: thanks
[14:37] <frankban> hatch: have you seen my email re juju connect stuff?
[14:38] <hatch> not yet
[14:38] <hatch> i will
[15:34] <hatch> frankban: review and qa done
[15:34] <hatch> looks good
[15:35] <hatch> frankban: I am interested to know why you wrapped those two methods?
[16:26] <hatch> frankban: so why did you wrap those functions in a functools.wrap decorator? can you not return a function in python normally?
[16:28] <frankban> hatch: functools.wrap preserves the original name and documentation of the decorated function
[16:29] <hatch> frankban: ahh
[16:31] <frankban> hatch: FYI those values are stored in func.__name__ and func.__doc__ 
[16:39] <hatch> interesting that those aren't included by default
[17:12] <hatch> any idea why the gui charm needs firefox in its sysdeps?
[17:16]  * hatch cry's "I just want to run the linter"
[17:16] <mbruzek> hatch ping
[17:16] <hatch> sup
[17:17] <hatch> uiteam, anyone know why I wouldn't have the correct version of pip for the guicharm linter? 
[17:17] <frankban> hatch: firefox is required by functional tests using selenium
[17:17] <frankban> hatch: not sure if I understand your second question
[17:18] <hatch> # Ensure the correct version of pip has been installed
[17:18] <hatch> tests/.venv/bin/pip --version | grep "pip 1.[4-9]" || exit 1
[17:18] <hatch> make: *** [setup] Error 1
[17:18] <hatch> ^ frankban 
[17:18] <mbruzek> hatch: I am trying to figure out the charm store uris
[17:18] <frankban> hatch: what pip version do you have?
[17:18] <hatch> 1.5.4
[17:19] <hatch> pip 1.5.4 from /usr/lib/python2.7/dist-packages (python 2.7)
[17:19] <mbruzek> hatch: Kapil has a launchpad url lp:~hazmat/charms/bundles/kubernetes/bundle
[17:19]  * hazmat ears are ringing
[17:19] <frankban> hatch: tests/.venv/bin/pip --version 
[17:19] <mbruzek> hatch: and quickstart maps to that  juju quickstart bundle:~hazmat/kubernetes/kubernetes
[17:19] <mbruzek> hello haz
[17:20] <mbruzek> hello hazmat
[17:20] <hazmat> greetings
[17:20] <hatch> frankban: ummm 6.0.7?
[17:20] <frankban> hatch: really?
[17:20] <hatch> mbruzek: if you want to deploy a bundle using quickstart or the depployer then you'll have to use the old yaml format
[17:20] <mbruzek> hatch: trying to figure out how quickstart maps bundle:~hazmat/kubernetes/kubernetes to lp:~hazmat/charms/bundles/kuberentes/bundles
[17:21] <hatch> pip 6.0.7 from ......../develop-trunk/tests/.venv/local/lib/python2.7/site-packages (python 2.7)
[17:21] <hatch> mbruzek: it guesses
[17:21] <hazmat> mbruzek: on lp distro is charms, series is bundle is the nutshell
[17:21] <mbruzek> hatch trying to figure out the mapping so I can write readmes for other bundles
[17:21] <frankban> hatch: I am rebuilding the devenv for the char, and checking again
[17:22] <hatch> mbruzek: all of that is changing soon - makyo_ is working on updating the deployer to support the new syntax
[17:23] <hazmat> hatch: thats not url syntax is it? i thought it was mostly just syntatic incompatible format changes
[17:23] <hazmat> in the bundle itself
[17:23] <mbruzek> hatch So I am writing a readme for how quickstart works now.  I just want to know what format will work with our bundle here:  p:~kubernetes/charms/trusty/kubernetes-bundle/trunk
[17:23] <hatch> no the url changed as well
[17:23] <hatch> mbruzek: if you want to use that you'll have to point to the old yaml file directly
[17:24] <hatch> the new bundle yaml format is not supported by the deployer
[17:24] <mbruzek> hatch: I don't want to point to that url, I like the bundle: uri, I just need to know how it works.
[17:24] <frankban> mbruzek: quickstart still uses the old charmworld for bundles, and the supported syntax is described in quickstart --help
[17:24] <mbruzek> frankban: thank you.  Will read up on that
[17:24] <hatch> for some reason the deployer wasn't updated before the bundle syntax was
[17:24] <frankban> mbruzek: it will be updated soon to support the new bundle URLs
[17:25] <hatch> frankban: why do we want to use such an old version of pip? 5 major versions older?
[17:27] <mbruzek> hatch: frankban: I see the help text now, it seems that quickstart supports 5 formats.  Which one would you recommend?  I see things are changing, what would be the format that would be recommended that works now and in to the future?  
[17:27] <hatch> I think those two are mutually exclusive :) 
[17:27] <mbruzek> doh
[17:28] <hatch> juju quickstart bundle:mediawiki/single
[17:28] <hatch> we want this format to work
[17:28] <hatch> but currently it doesn't
[17:29] <frankban> hatch: I think in the future we want "juju quickstart bundle/mediawiki-single"
[17:29] <frankban> mbruzek: ^^^
[17:29] <hatch> oh jeeze that's a new format to me
[17:29] <mbruzek> hatch: frankban:  Our bundle currently lives here:  lp:~kubernetes/charms/trusty/kubernetes-bundle/trunk  
[17:29] <hatch> I think I'll just write code - give up on what format is what lol
[17:29] <frankban> hatch: in the new charm store, the "bundle" string is not a schema, is a series
[17:29] <mbruzek> It is not yet promulgated.  Is that in a wrong spot?  Should we change that?
[17:31] <frankban> hatch: it seems that pip quickly switched from version 1.5.6 to 6.0 :-|
[17:31] <hatch> lol
[17:31] <frankban> hatch: so yes that check should be changed, feel free to do that as part of your branch
[17:31] <hatch> frankban: what was it there for? 
[17:32] <hatch> There must be a reason why we checked at all
[17:32] <frankban> hatch: I did not add that code, so not sure, perhaps some backward incompatible changes with previous pip versions
[17:32] <hatch> so think I can just remove the check entirely?
[17:34] <frankban> hatch: I'd suggest to keep the check, just make it more permissive
[17:34] <hatch> so just greater than 1.4?
[17:34] <frankban> hatch: +1
[17:51] <hatch> frankban: do you know of a way to structure the grep for that? I am having a heck of a time nesting the conditionals in the makefile
[17:52] <hatch> ifndef apparently doesn't work for me
[17:52] <hatch> oh makefiles
[17:54] <frankban> hatch: grep -E "pip (1.[4-9]|6)" ?
[17:55] <hatch> negative, tried that :)
[17:56] <hatch> here is what I am attempting to do in the makefile (and failing) https://gist.github.com/hatched/83544f1c201ee1a71a6e
[17:56] <hatch> for some reason ifndef is not found?
[18:00] <frankban> hatch: what I suggested works here
[18:00] <frankban> % echo "pip 6.3" | grep -E "pip (1.[4-9]|6)"                                                                                                                                                                                               (trunk) ubuntu
[18:00] <frankban> pip 6.3
[18:01] <hatch> yeah it works from the echo but not in practice
[18:15] <hatch>  tests/.venv/bin/pip --version | grep -E '1\.[4-9]\.|6\.[0-9]\.[0-9]' || exit 1
[18:15] <hatch> bleh lol
[18:17] <mbruzek> hatch: I tried to deploy using quickstart using a raw html bundle, but none of the charm store charms showed up.  I ran this with --debug and the pastebin is here. http://paste.ubuntu.com/10076915/  Can you help me
[18:17] <mbruzek> ?
[18:18] <hatch> looking
[18:18] <hatch> what was the command you used?
[18:19] <hatch> btw thanks for writing these docs :)
[18:25] <hatch> mbruzek: still there?
[18:25] <mbruzek> yes
[18:25] <hatch> what was the command you used?
[18:25] <mbruzek> juju quickstart https://raw.githubusercontent.com/whitmo/bundle-kubernetes/master/bundles.yaml --debug 2>&1 | tee quickstart_problem.txt
[18:26] <mbruzek> The bundle is legit, it works with deployer and references Charm store uris
[18:26] <mbruzek> I am trying to figure out why none of the charms are loaded
[18:29] <hatch> hmm yeah you're right it looks good
[18:29] <frankban> mbruzek: quickstart states that the bundle deployment request has been accepted
[18:29] <mbruzek> fankban I can show you the juju status that tells me otherwise
[18:29] <mbruzek> or the gui if you don't believe status
[18:29]  * mbruzek *grins*
[18:29] <hatch> mbruzek: have you tried deploying those services directly? 
[18:29] <frankban> mbruzek: what's the content of https://15.125.118.3/gui-server-info
[18:29] <frankban> ?
[18:30] <mbruzek> {"uptime": 3435, "deployer": [{"Status": "started", "Queue": 0, "DeploymentId": 0, "Time": 1423157614}, {"Status": "scheduled", "Queue": 1, "DeploymentId": 1, "Time": 1423159220}, {"Status": "scheduled", "Queue": 2, "DeploymentId": 2, "Time": 1423159241}, {"Status": "scheduled", "Queue": 3, "DeploymentId": 3, "Time": 1423159325}], "apiversion": "go", "sandbox": false, "version": "0.4.2", "debug": false, "apiurl": "wss://10.0.0.166:17070"}
[18:31] <frankban> mbruzek: so you have pending bundle deployments in the queue, they are executed in order
[18:31] <mbruzek> frankban: I guess I tried this bundle several times now.
[18:31] <mbruzek> I can destroy and start over again, but I don't think I would get a different result.
[18:32] <mbruzek> hatch: I have deployed this bundle through juju-deployer when have the bundle on my local machine.
[18:32] <frankban> mbruzek: let's check the guiserver logs: they are stored in the gui unit in /var/log/upstart/guiserver.log
[18:33] <mbruzek> on machine 1?
[18:33] <frankban> mbruzek: wherever the gui unit is placed: juju ssh juju-gui/0
[18:34] <mbruzek> frankban: OK found it, is that safe to pastebin?  It does not show my keys or other stuff?
[18:35] <frankban> mbruzek: I think so, but let's use the canonical pastebin
[18:40] <mbruzek> frankban: working on it
[18:40] <frankban> mbruzek: ty
[18:45] <mbruzek> frankban: had to paste it by hand, no upload feature on pastebin.canonical.com
[18:45] <mbruzek> https://pastebin.canonical.com/125051/
[18:49] <frankban> mbruzek: uhm... I don't see any errors in the logs. it seems that the deployer is still working on your first bundle as usual, except it's not deploying anything
[18:50] <frankban> mbruzek: you can try, from that machine, "sudo service guiserver restart", then "juju set juju-gui builtin-server-logging=debug" and then try to deploy the bundle again (once) and see if the logs show something
[18:50] <mbruzek> I see a python exception in queues.py is that nothing?
[18:53] <frankban> mbruzek: not sure if that's related to the bundle deployment
[18:54] <frankban> mbruzek: but it's well over my EOD now and I have to go, so could you please try what I described and file a bug against the GUI charm with the info attached?
[18:55] <mbruzek> will do,  thanks for your assistance frankban.
[18:55] <mbruzek> Have a good day
[18:55] <mbruzek> night
[18:55] <frankban> night and ty
[20:31] <hatch> uiteam do we know which branch I should push my guicharm changes to? lp:charms/trusty/juju-gui  or  lp:~juju-gui/charms/trusty/juju-gui/trunk 
[20:31] <hatch> and by push I mean propose
[20:46] <hatch> uiteam lf a review for a smallish diff https://code.launchpad.net/~hatch/charms/trusty/juju-gui/add-uuid-configjs/+merge/248826
[22:22] <rick_h_> hatch: juju-gui/charms/trusty/juju-gui/trunk
[22:23] <hatch> oh hai!
[22:23] <hatch> yeah I have it up for review
[22:23] <hatch> :)
[22:24] <rick_h_> ah cool
[22:25] <hatch> tornado is bonkers
[22:25] <hatch> just so you  know :P
[22:25] <rick_h_> :)
[22:25] <rick_h_> async python ftw
[22:25] <rick_h_> hatch: just think of it like nodejs
[22:25] <hatch> ehh I'm still thinking an async language might be better :)
[22:26] <hatch> makyo_ and I are going to have to get schooled by frankban tomorrow :)
[22:27] <rick_h_> heh, docs aren't bad
[22:27] <hatch> nah it's just trying to find the correct spot to put things in the guiserver is the issue
[22:27] <rick_h_> gotcha
[22:28] <hatch> having a yeild in the middle of a function then never returning to it really threw me for a loop
[22:32] <huwshimi> Morning
[22:32] <hatch> morning huwshimi 
[22:32] <rick_h_> morn
[22:32] <hatch> I fixed some of the rendering performance issues in leankit by modifying their stylesheets :) 
[22:33] <hatch> there are cascading shadows and transparencies so it's no wonder you get screen tearing on slower machines
[22:43] <hatch> I found this cool site that shows benchmark comparisons for various languages. http://benchmarksgame.alioth.debian.org/u64q/compare.php?lang=dart&lang2=go