/srv/irclogs.ubuntu.com/2013/03/15/#juju-gui.txt

rogpeppeif anyone wants to take a look at the latest allWatcher CL, it's here: https://codereview.appspot.com/7594048/11:34
rick_h_gary_poster: the hacking doc doesn't mention submitting changes to the charm. I'm looking to just self-review/land the changes https://code.launchpad.net/~rharding/charms/precise/juju-gui/subapp_config/+merge/153557 is there a process I need to make sure I follow? e.g. is there a tarmac/test runner for the charm?13:51
benjirick_h_: Gary is out of the office today.  I'm not completely sure what the charm landing procedure is, but I'm sure there are tests and reviews.  Let me look real quick and see if anything jumps out at me.13:55
rick_h_doh gary_poster is out today isn't he13:55
rick_h_benji: thanks, yea there are tests, and instructinos for running locally. 13:55
rick_h_benji: but curious for this small change if it's safe to merge with trunk and push directly or if something mechanical should be looking/doing that?13:55
rick_h_benji: we don't run tarmac on our charm, but it doesn't have the nice tests you guys have so figured it might be here.13:56
benjinope, there is not automation as of yet13:56
rick_h_benji: ah ok cool then. 13:56
benjijujugui: call in 314:27
rick_h_hatch: https://code.launchpad.net/~rharding/juju-gui/subapp_updaet is my changes applied to your subapp branch. You can pull them in from there if you'd like. Sorry for the typo in the name. Can't seem to type today.14:28
hatchahh very cool thanks14:30
rick_h_goodspud: so on icon related notes. Is there someone we can ping to make charmworld/juju-gui leaders by examples and getting something fancy in .svg for our charms?14:42
goodspudrick_h_  I'd love to say yes but our designer hasn't adopted IRC as an acceptable form of communication14:46
goodspudBUT he has just recently discovered his work email14:46
rick_h_goodspud: is there someone I should send an email to?14:46
goodspudgreg.viercant@canonical.com14:46
rick_h_lol14:46
hatchlol!!14:46
rick_h_goodspud: thanks, I shall begin ancient ineffecient communication patterns with said person :)14:46
goodspudHe's got a bit of work to do today but he's going to produce some documentation re icons on Monday14:47
hatchno irc means they can claim the 'email got lost'14:47
hatch:)14:47
goodspudI sit beside him... cc me into it14:47
goodspudrick_h_, you could describe IRC as ancient as well couldn't you? Even older than email?14:48
hatchooo he's got you there14:48
rick_h_is it? I figured email came first. After all I'm not counting finger on an old unix system as irc 14:49
* rick_h_ heads to wikipedia14:49
hatch1982 SMTP14:49
hatch1988 IRC14:49
hatchOOO14:49
hatchsoo close14:49
rick_h_email 1982 and irc 198814:50
rick_h_I win!14:50
goodspudDamn it!14:50
hatchyeah except I totally wiki faster than you14:50
hatch:P14:50
rick_h_must be that fancy canadian internet14:51
goodspudI accept defeat14:51
hatchit's ok as time goes on, the difference becomes negligable14:51
hatchnegligible even14:52
hatchpeople who research dino's even thing 1M years is no big deal14:52
hatch:P14:52
arosalesretrospective this week?14:55
hatchsure thing, when are you thinking?14:56
rick_h_I thought there was none due to the people out.14:56
hatchwell there wasn't - but we could if people had things they wanted to complain about ;)14:57
goodspudarosales, Gary said yesterday that we'd skip the weekly meeting because half of the people are away14:57
arosalesgoodspud: ah, ok 14:57
goodspudBut, up to those who are still around really14:57
* arosales wasn't aware of the yesterday's comments14:57
goodspudI can certainly tell you all where we are at in the design deparment14:57
arosalesno need to meet on my account or anything. I wasn't aware of the logistics.14:58
* arosales puts headset up14:58
goodspudarolsaes, no worries. Just know that everything is alright.14:59
goodspud(hopefully)14:59
hatchrick_h_ I saw you merged in that charm code - I THINK that we are supposed to tag things like that as trivial before merging...I don't know where or how, but I saw Ben do that during the sprint15:00
rick_h_hatch: ah, sorry. 15:01
hatchoh it's no problem, I don't even know if that's true - I just saw him do it15:01
hatchhaha15:01
hatch:)15:01
goodspudrick_h_, regarding your email....15:07
goodspudYou're asking for an icon for the Juju GUI Charm15:07
goodspudAND15:07
goodspudOne for the charm store/charm world?15:08
goodspud(assuming there is not Charm for deploying the Charm Store?)15:08
rick_h_goodspud: so I think for now the primary one is the Gui since it'll be front/center15:09
rick_h_there's talk of people doing their own charm stores at some point, and I'm working on both projects atm so I figured I'd ask for both :)15:10
rick_h_goodspud: but both are charms from canonical and I assume at some point we'd want some sort of icons for things we put out there. 15:10
goodspudrick_h_. OK. We have our branding icon for the Canonical Charm Store (it's the 4 j's in a rosetta pattern). Mark liked it so we've stuck with it15:11
rick_h_goodspud: cool15:11
goodspudrick_h_, but yes we could definitely work on an icon for the Juju GUI charm itself15:12
rick_h_goodspud: awesome! appreciate it. 15:12
goodspudrick_h_, I'll beat our designer into submission and chain him to his desk until he does it15:12
goodspudPS: I'm only saying this because he's not on IRC15:12
hatchlol15:13
rick_h_goodspud: make sure he gets occassional crackers and cheese. We're not cruel people :)15:13
goodspudrick_h_, He's from "Up Norf" so I'l lthrow in some breaded scampi and chips15:13
rick_h_ah, yea I don't speak 'over the pond'. I'll try to get some reading time in this weekend.15:13
hatchit would be awesome if we could trigger our charm tests via the gui15:16
goodspudrick_h_, I've been here 6 years and I'm still getting used to it15:16
hatchgoodspud: where are you originally from?15:31
goodspudhatch, a small country I like to call "Godzone" but the rest of the world calls "New Zealand"15:31
hatchlol15:32
goodspudhatch, the locals call it "Aotearoa"  -  the and of the long white cloud15:32
goodspuder, *land15:32
hatchso it's always cloudy there?15:32
goodspudYep15:33
goodspudExcept at the moment... it's suffering a drought15:33
hatchour license plates say 'land of the living sky' - what it should have said was 'damn cold, stay away'15:33
goodspudhatch, Bwa ha ha ha ha15:34
goodspudOnly in winter surely15:34
hatchyeah only in winter :) In the summer it's actually really nice here15:34
goodspudI've only been to Canada in Winter - bloody freezing15:35
goodspudAnd then it was only Toronto/London (ontario)15:35
hatchoh pfft that's probably the warmest place :P15:35
hatchimho if you realy want to see Canada you need to get out of the big cities15:36
hatchmuch the same as anywhere I suppose15:36
goodspudhatch, that's my opinion of most places I go... which is why I like to go by motorbike when I travel15:38
goodspudBut, I went to Toronto before I had a motorbike15:38
goodspudit did only get down to -1715:39
hatchhaha -17 is cold enough though15:40
hatch:)15:40
hatchbuy a motorbike...that's a great idea!15:40
goodspudFor a moderate climate boy who had only been in the snow skiing once, it was definitely cold15:41
goodspudYeah mang, motorbikes are awesome15:41
goodspudJust don't crash15:41
goodspudBut then don't crash in any vehicle15:41
hatch:) I had 2 gsx-r 1000's but sold the last one a few years ago15:41
hatchI'm scared of the other drivers haha15:41
hatchone almost took me out on the freeway - that's when it went up for sale15:42
hatchfyi all it looks like this node issue was caused by the update15:43
benjiman, I am not feeling 100% today; I hope I'm not coming down with something16:04
hatchuh oh16:04
hatchtime go to pick up some OJ16:04
hatcharg16:37
rogpeppegary_poster: did you see my proposed branch, BTW?16:38
benjirogpeppe: Gary is out of the (virtual) office today.16:40
rogpeppebenji: ah, thanks16:40
* rogpeppe thinks we should probably use /away more16:40
hatchrogpeppe: I have a q which you might know....is it documented somewhere about the relation between CS path and LP repo name?16:42
rogpeppehatch: CS path?16:42
hatchcharmstore16:42
hatchso for example...16:42
rogpeppehatch: ah! hmm, it might be16:42
hatchI can't use this charm because it's not called juju-gui  lp:~hatch/charms/precise/juju-gui-conflict/trunk16:43
hatchI change the path to be juju-gui and it deploys using cs:~hatch/precise/juju-gui16:43
hatchbutI can't use cs:~hatch/precise/juju-gui-conflict16:44
hatchso instead of me fighting with this I was hoping there were some 'rules' :)16:44
rogpeppehatch: i don't know this off the cuff, i'm afraid16:45
rick_h_hatch: time to push to lp:~hatch/charms/precise/juju-gui/conflict 16:45
hatchdarn - I'm moreso curious why I can't specify an absolute path to begin with but I'm sure there is a logical explanation behind it16:45
hatchrick_h_ that won't work, it expects it to end in 'trunk'16:45
hatchat least that's the only way I got it to deploy16:46
rick_h_hatch: hmm, so you're not pushing a local charm to your env?16:46
rick_h_guess not if you're referring to the remote cs:...16:47
hatchyeah I'm actually debugging this CI Stuff so I can't do it locally16:47
hatchmatsubara: it looks like you're having issues staying connected :)16:48
rogpeppehatch: i *think* this might be logic in question: http://paste.ubuntu.com/5617026/16:48
hatchahah!16:49
hatchso if I wan'ted to file a bug about that who where is the logical place to do so?16:49
rogpeppehatch: that's inside juju-core/store/lpad.go16:49
rogpeppehatch: juju-core16:49
hatchgotcha16:49
hatchthanks a bunch16:49
rogpeppehatch: np16:49
hatchI knew 'how' to get it to work but wasn't sure why16:49
hatchrogpeppe: is there an unminified version?16:50
rogpeppehatch: ?16:50
hatchoh I just see 'u', and 'nil' those looked minified to me16:51
hatchor is nil a go keyword?16:52
rogpeppehatch: it is16:52
hatchoh heh16:52
hatch:D16:52
hatchgo looks pretty easy to follow16:52
hatchI'm going to guess those are 'famous last words' :D16:53
rogpeppehatch: it's designed to be easy to follow. that doesn't mean it always is though :-)16:53
rogpeppehatch: more accurately, nil isn't a go keyword, it's a predefined identifier. you have have variables and struct fields named "nil" if you really want :-)16:53
hatchohh16:54
hatchwell I'll be sure to add go to my list of 'languages to learn'16:54
hatchwell at least learn well enough to know what I'm looking at :D16:55
hatchwow it's snowing here, can't even open the backdoor haha16:57
hatchok here is to hoping that I  have solved the node issues16:59
hatch(by reverting to an old version) :-(16:59
hatchalright so it looks like that the PPA that we use for node no longer includes the working version of node17:17
hatchhttps://launchpad.net/~chris-lea/+archive/node.js/17:18
hatchso we can manually build from source17:18
hatchOR wait17:18
hatchanyone have any input?17:18
hatchhttps://github.com/isaacs/npm/issues/2907 here is the issue - looks like it's a regression17:19
benjihatch: I haven't tried to build the gui in a while; are you blocked?17:19
hatchwell it's blocking the CI17:19
hatchissue is that I don't know how long until a fix...could be days...could be weeks17:20
hatchbut our system isn't really setup to build node right now17:20
benjithat issue say that it it fixed in 0.8.1417:20
hatchscroll down ;)17:20
hatchwayyyyyy down17:20
benjiheh17:21
benjidoes it happen on trunk?17:21
hatchyep17:22
hatchit's an issue with that version of npm17:22
rick_h_hmm, can't you lock a previous version in a repo?17:22
hatchrick_h_ I am but the PPA only has node 0.1017:23
hatchlooks like he removed 0.8.1417:23
hatchhttps://launchpad.net/~chris-lea/+archive/node.js/17:23
benjioh, that's evil17:23
benjiactually, that may be the default (or only) PPA behavior17:24
hatchahh17:24
rick_h_benji: yea, now that I think of it I don't often recall tons of back revisions17:24
hatchhttps://launchpad.net/~chris-lea/+archive/node.js-legacy17:24
hatchyay!17:24
rick_h_hatch: there you go. I'd say go that route vs doing our own compiling of fresh packages until npm fixes itself17:25
hatchoh yeah for sure - I REALLY didn't want to set up our system to build it hah17:25
benjithis page shows lots of old builds: https://launchpad.net/~chris-lea/+archive/node.js/+builds?build_text=&build_state=built17:26
benjithe old ones are available for download, so they might be available in the archive17:27
rick_h_just changing the ppa source to the archive should unblock and once the bug is fixed can be switched back.17:28
hatchyup17:28
hatchwe should make our own repo for things and then deploy from there so we always have working versions haha17:32
rick_h_one thing we did in charmworld was to create a download cache of all of our node deps and npm install from local only. 17:34
rick_h_I didn't look at the npm bug close enuogh to see if it was only a problem fetching remote packages or a more general npm issue17:34
hatchyeah that's something we should also do - would probably speed up the installs too17:36
hatchput that on the infinitely long todo list17:36
hatchhaha17:36
benjihatch: how do you provoke the problem?  A make clean-all; make; make test-prod worked for me.17:41
hatchsudo apt-get update; rm -rf /path/to/trunk/node_modules; make clean-all; make devel17:42
hatchshould do it17:43
hatchnode --version needs to output 0.10.017:43
hatchyay install working now - tests running17:52
hatchand it's not even lunch yet17:52
hatchw00t17:52
benjiI can't reproduce the problem.18:00
benji(and by the way, make clean-all should remove the node_modules directory)18:00
rick_h_benji: but npm is a system wide thing. It's like needing to update apt-get itself. it's not in the node_modules18:07
rick_h_benji: just like the node binary itself18:07
benjiah... that's stupid18:08
* benji looks around for where the modules are stored18:08
rick_h_benji: :) well virtualenv does bring it's own pip to install. but at least it's part of the virtualenv18:08
rick_h_which npm and which node should show a system-wide install from taht chris lea ppa18:09
hatchI've solved it18:10
hatchbenji: you need to be running the most recent version of node and npm18:10
hatchfor it to break18:10
hatchso probably don't18:10
hatch:D18:10
benjihatch: cool, what is the solution?18:11
benjiI am runnign the latest from the PPA but I haven't tried the remove-the-node-modules bit yet, so that's prbably why I can't reproduce the problem.18:11
hatchusing node v 0.8.14 and whatever npm comes with it18:13
hatchI'll monitor the situation and when they release the new version I'll upgrade to 0.10 which requires some other small changes18:13
benjiI'm confused.  I'm running node v0.10.0 and npm 1.2.14 with no problems.  It's not something that we have to nail down now, but I'm concerned that other developers (and users) will see breakage.18:22
rick_h_benji: yea, in that bug report there's reference to a 'sudo npm cache clear' command. Wonder if there's a cache in place keeping you happy18:24
benjiI wonder where that cache is ~/.npm perhaps18:25
benjiyay!  I broke it!18:40
benjiit is indeed the cache that was saving me (good thing I put it aside rather than killing it outright)18:40
rick_h_benji: heh, caution is always a good thing18:40
benjihatch: this seems like something the other developers should be aware of, will you write an email to the juju-gui-dev list (whatever it's called) and tell everyone what to do/not do?18:41
benjiOK guys, I'm not feeling well and I'm not getting anything done so I will take a late lunch and stop a bit early.  See you all on Monday.19:00
hatchbenji: I can do that19:07
hatchget better - see you Monday19:07
hatchrick_h_ are you around?19:12
hatchjujugui anyone around able to give me a quick review on a one line diff https://codereview.appspot.com/7759050/19:33
hatchok 2 line diff, 1 line change :)19:33
bachatch: yep19:34
hatchthank yas19:34
bachatch: did19:35
rick_h_hatch: what's up?20:31
hatchyou are no longer needed20:32
hatch;)20:32
hatchunless you want to help me debug my flickr carousel :P20:32
hatchGood news is that the CI is back up and running albeit one obscure test failure20:40
rick_h_26hatch I hate obscure 20:44
rick_h_26cool news that things are trucking at EOW though20:45
hatchyeah but somehow the unit tests are failing when they pass no problem (obviously because you can't commit if they fail)20:45
=== rick_h_26 is now known as rick_h_droid

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!