| === Makyo is now known as Makyo|out | ||
| benji | gary_poster: I would like to verify that the loading message branch is doing the right things. We can have a call to discuss if you want. | 13:22 |
|---|---|---|
| gary_poster | benji, I was trying to reply. I had a security breach in my accounts yesterday, which made me actually start to use LastPass, which is slowing me down a bit as I try to understand it. All I wrote was this: | 13:23 |
| gary_poster | app/index.html:62: Trying to connect to the Juju environment.</span> | 13:24 |
| gary_poster | Initially this is about loading JS. It's only about connecting once the app has | 13:24 |
| gary_poster | started. The login shouldn't show until we've connected. | 13:24 |
| gary_poster | Does that make sense? Happy to call. | 13:24 |
| benji | gary_poster: I think I took that wording from the UI mockup. I'm happy to change it. How about "Loading the Juju GUI."? | 13:25 |
| gary_poster | +1 benji. We can then use moxckup's wording once we are connecting | 13:26 |
| benji | gary_poster: I don't undestand that bit. Should there be another box that pops up with a connecting message? | 13:26 |
| gary_poster | benji, I suggest changing text | 13:27 |
| bac | gary_poster: -1 on breach. +1 on lastpass though! | 13:27 |
| gary_poster | :-) yeah | 13:27 |
| benji | yeah, sorry to hear that; I hope it has been easy to clean up | 13:27 |
| gary_poster | so far not bad. Takes time to get everything back using the new passwords, and I have so many google passwords, and so many places they are remembered that it is the most annoying bit | 13:28 |
| benji | gary_poster: by the way, I found a spinner generator and made a spinner that looks like the one in the mockup... <wink> <wink>, <nod> <nod> | 13:29 |
| gary_poster | benji, heh, was that the one in the branch and I should have actually looked at it? was about to do that when I went down a LastPass rabbit hole | 13:29 |
| gary_poster | will look | 13:29 |
| benji | yep, it's in there | 13:30 |
| benji | I need coffee. I'll be back in a sec. | 13:30 |
| gary_poster | benji, spinner is not like js. spinner js is also 1/6 the K weight if gif | 13:38 |
| gary_poster | of | 13:38 |
| gary_poster | benji, also, in non-chrome browsers, should only show spinner message once "your browser is not supported" message has been clicked | 13:39 |
| bac | gary_poster: i could not reproduce that routing bug so i'm glad to see your comment on it. i've marked it invalid. | 13:43 |
| benji | gary_poster: good info, thanks | 13:43 |
| gary_poster | thanks bac, benji | 13:43 |
| bac | gary_poster: preferences on next card i should grab? | 13:47 |
| gary_poster | bac, if you don't have a Go Juju dev environment, that would be good to make. Otherwise I like bug 1117888 | 13:48 |
| gary_poster | or bug 1119412 | 13:49 |
| hatch | good morning | 13:49 |
| gary_poster | morning hatch | 13:49 |
| gary_poster | did you start that styling bug you mentioned or did it end up being a rabbit hole? | 13:49 |
| gary_poster | yo _mup_. You going to translate those bug numbers? | 13:50 |
| hatch | I 'started' on it but didn't get far, once the coffee hits I'll be resuming | 13:50 |
| gary_poster | :-) ok hatch. If you want to pair I can help or someone else can | 13:50 |
| hatch | did I put the task on the kanban in the right place? | 13:50 |
| gary_poster | nah, it is a slack task, and you are working on it, so it goes in Slack -> Coding -> Active, hatch. I moved it | 13:51 |
| hatch | got it | 13:51 |
| hatch | bazaar kind of feels like svn gone distributed with the way it handles branches | 13:52 |
| gary_poster | hatch yeah. there are a bunch of ways to use it though. This is the easiest entrance, I think. There are other approaches that are more like git, though they are a bit clunky IMO in the bzr world | 13:53 |
| gary_poster | We can explore at a sprint | 13:53 |
| gary_poster | BTW, speaking of sprint, everyone, hazmat sez that location is still not set, so that's why we have not had a grand ticket buying exercise | 13:54 |
| gary_poster | jujugui ^ | 13:55 |
| gary_poster | ooh, I could use that for my daily call pings! | 13:55 |
| hazmat | gary_poster, location close to set (atlanta), but haven't gotten final confirmation yet. trying to figure out accomodations. | 13:55 |
| gary_poster | cool hazmat thanks | 13:55 |
| hatch | hmm I'll have to figure out what to do for cell data while traveling | 13:56 |
| hatch | I'd need a second job to pay for it if I didn't find a plan or buy prepaid heh | 13:56 |
| hazmat | hatch, prepaid sim card from tmobile, att should do it | 13:56 |
| hatch | $5/MB | 13:56 |
| hazmat | assuming unlocked phone | 13:57 |
| hatch | yeah it's not....but I'm debating picking up a nexus 4, (or unlocking this one) | 13:57 |
| hazmat | hatch, there are some co-located branch add ons (ie multiple branches in the same dir) .. cobzr is pretty popular in the juju-core aka backend team. | 13:57 |
| benji | gary_poster: isn't it "guihelp"? | 13:58 |
| bac | benji: yeah. i ding on both. | 13:58 |
| hazmat | benji, we should have both.. one is meant for #juju | 13:58 |
| gary_poster | benji, yeah, I could do both. jujugui is our ping for customer service help in #juju. guihelp is our ping for getting attention locally. By using jujugui I am making sure that we have the customer service one connected for everyone :-) . I also realized a while ago that we could only have jujugui, so in my mind that's how I think of it | 13:59 |
| benji | I hilight on one and make a sound on the other. Making a sound on both seems a bit interruptful for a 10-person team | 13:59 |
| hazmat | juju+gui there gui+help here.. | 13:59 |
| hazmat | yeah.. collapse to one would work fine | 14:00 |
| benji | +1 on fewer if we are going to treat them as synonyms anyway | 14:00 |
| gary_poster | cool, I'll mention it on call | 14:01 |
| hatch | do we track time on tickets? | 14:01 |
| gary_poster | no hatch | 14:01 |
| * benji practices Movie Voice: "Coming to a theater this summer... a space thriller like nothing you've seen before: No Hatch" | 14:02 | |
| gary_poster | :-) | 14:02 |
| hatch | clearly that would be an awesome movie | 14:04 |
| benji | :) | 14:06 |
| hatch | so waht is the proper procedure for working with the less file? Do I run make after every change or can I just run less manually? | 14:08 |
| hatch | I see the make includes recess | 14:08 |
| gary_poster | hatch, make devel will automatically notice changes and regenerate | 14:09 |
| gary_poster | so edit and reload | 14:10 |
| === teknico_ is now known as teknico | ||
| frankban | gary_poster: it seems we have a regression, both in trunk and uistage: "Uncaught TypeError: Cannot call method 'get' of null" while trying to destroy a service (from the svc menu or from the svc detail) | 14:21 |
| gary_poster | frankban, :-/ ok. bac, ^^ do you have bandwidth? | 14:22 |
| gary_poster | thanks frankban | 14:22 |
| bac | gary_poster: yes. i don't understand the context of your question though. | 14:32 |
| bac | gary_poster: oh, you mean "brad, look at this regression please." | 14:33 |
| gary_poster | bac, yes :-) | 14:33 |
| bac | gotcha | 14:33 |
| * bac looks | 14:33 | |
| gary_poster | thanks | 14:33 |
| hatch | need.....more.....ram!!!! | 14:54 |
| hatch | I remember upgrading to 4MB | 14:54 |
| hatch | now 8GB isn't enough :) | 14:54 |
| bac | frankban, gary_poster: that regression has been around for 10 days or so. :( | 14:58 |
| gary_poster | bac, :-/ | 14:58 |
| frankban | bac: :-( | 14:59 |
| gary_poster | bac, easy to fix? | 14:59 |
| gary_poster | easy to test? :-) | 14:59 |
| bac | gary_poster: dunno, just traced it back to the broken revision so now i'm trying to see what it did | 14:59 |
| Makyo|out | frankban, gary_poster bac - Current landing branch may fix. | 14:59 |
| === Makyo|out is now known as Makyo | ||
| hatch | I was just going to say...when you guys find things like this do you write a test for it? :) | 14:59 |
| Makyo | Whoops. | 14:59 |
| gary_poster | ack cool Makyo | 14:59 |
| gary_poster | yes hatch :-) | 15:00 |
| bac | Makyo: so a deleted service is still in the set of services but has no model. does that sound like something you just fixed? | 15:00 |
| Makyo | bac, yeah. | 15:00 |
| frankban | cool | 15:00 |
| bac | great | 15:00 |
| Makyo | bac, but by accident. Will do a quick test branch after lbox finishes. | 15:00 |
| bac | Makyo: do you want to write a test for it or would you like me to? | 15:01 |
| Makyo | bac, if you'd be willing, that'd be excellent :) | 15:02 |
| teknico | hatch, how many VMs do you have in there? :-) | 15:02 |
| bac | Makyo: sure | 15:02 |
| hatch | teknico: just Ubuntu and Win 8 but I can't boot the win 8 one up because OSX Is using too much ram | 15:02 |
| hatch | "Used 7.98GB" - since I have paging turned off I better not use those extra 20MB or something is going to crash ;) | 15:04 |
| gary_poster | heh | 15:04 |
| goodspud | gary_poster, do you have 5 mins for me to "quickly" present the landscape integration designs I've just emailed you? | 15:05 |
| Makyo | bac, branch merged. | 15:05 |
| bac | Makyo: thx | 15:05 |
| gary_poster | goodspud, sure! was about to approve via email, with a mention that we will need "Unit Landscape" too | 15:05 |
| gary_poster | going to jujugui | 15:05 |
| hazmat | goodspud, the attachments didn't make it.. can you send a link | 15:14 |
| goodspud | hazmat, give me five and I'll upload them to Drive... | 15:15 |
| gary_poster | jujugui call in 2 | 15:28 |
| goodspud | hazmat, emailed you a link. The images have some further exploration of the interface but focus on the Landscape integration aspects | 15:29 |
| hazmat | goodspud, thanks | 15:29 |
| gary_poster | ^^ that's the new version of bac bcsaller_ benji frankban goodspud hatch hazmat Makyo teknico announcement :-) | 15:29 |
| gary_poster | so please set up the ping if you have not already | 15:29 |
| hatch | oh it's dinging | 15:30 |
| gary_poster | hatch bac Makyo starting without you, come on by | 15:31 |
| benji | Makyo: data point: I don't see the weirdness on trunk that you guys see. | 15:50 |
| Makyo | benji, thanks. How is your workspace set up? I have mine set up like http://doc.bazaar.canonical.com/beta/en/user-guide/organizing_your_workspace.html#feature-branches | 15:51 |
| benji | Makyo: yep, mine is set up like that | 15:53 |
| Makyo | Hmm, alright. Just wondering given that our makefile touches bzr | 15:56 |
| teknico | hazmat, what juju IRC channel did you mention again (for asking about MongoDb), and on what server? | 15:59 |
| teknico | I guess it's #juju here on freenode, right? | 16:01 |
| hazmat | teknico, #juju-dev | 16:02 |
| teknico | I thought you mentioned that one, but could not find it | 16:02 |
| teknico | ok, I'll try joining anyway :-) | 16:03 |
| hazmat | gary_poster, auth stuff was merged fwiw | 16:06 |
| gary_poster | cool hazmat | 16:06 |
| Makyo | bcsaller_, a new checkout of trunk works with make devel. Tried it in a separate folder, though. Will try replacing my existing one and see if that helps. Not sure what would've gotten stale, though. | 16:21 |
| bcsaller_ | Makyo: me either | 16:21 |
| Makyo | bcsaller_, also works just deleting and rebranching trunk in existing repo tree. | 16:25 |
| Makyo | ^^^ guihelp - anyone else that had that issue with running trunk | 16:25 |
| hazmat | Makyo, does make clean && make devel work in old trunk? | 16:25 |
| Makyo | hazmat, no :/ | 16:25 |
| bcsaller_ | hazmat: that didn't work for me | 16:26 |
| bcsaller_ | I'd like to understand the cause here | 16:26 |
| hazmat | meld the two directories | 16:26 |
| hazmat | re meld fwiw - if there isn't a sig delta there | 16:27 |
| hazmat | whoops | 16:27 |
| hazmat | -> http://meld.sf.net | 16:27 |
| hazmat | bcsaller_, Makyo what's the symptom? | 16:28 |
| hazmat | back button broken? | 16:28 |
| bcsaller_ | hazmat: no, Makyo posted the link before, an error and broken rendering | 16:28 |
| bcsaller_ | the site just won't function | 16:28 |
| teknico | make devel on trunk seems to be working fine here | 16:29 |
| * hazmat joins the line stoppage | 16:29 | |
| hazmat | anyone know this error.. http://paste.ubuntu.com/1639863/ looks like a nodejs inotify issue around the css watch? | 16:30 |
| bcsaller_ | hazmat: yes, I started seeing that as well | 16:31 |
| hazmat | that feels like a newer dep issue | 16:31 |
| * hazmat does a make clean-all | 16:32 | |
| bcsaller_ | I *think* its not releasing the watch handle properly, but it could be that we just crossed some ulimit number on recent builds | 16:32 |
| hazmat | no reason we should be hitting that | 16:32 |
| hazmat | we don't have that many files to compile.. | 16:32 |
| gary_poster | hazmat, hatch points out for landscape work that session storage will be per tab, while session cookie would work across tabs. This won't make a huge difference but could be nice for some stories, like gui -> landscape -> new tab for gui. Only downside is we have to think a bit harder about security. https should make most common concerns fine though. I don't see any issues with shared domain bits. thoughts? | 16:33 |
| hazmat | gary_poster, cookie from who? | 16:34 |
| gary_poster | hazmat stash cookie with auth bits | 16:34 |
| gary_poster | from ourselves | 16:35 |
| hazmat | gary_poster, is there an api that let's the browser set its own cookies? | 16:35 |
| gary_poster | y hazmat | 16:35 |
| hazmat | oh. setCooke | 16:35 |
| gary_poster | also hatch tells me sessionStorage in spec is per top-level domain | 16:36 |
| gary_poster | cookie is per full domain | 16:36 |
| hatch | http://dev.w3.org/html5/webstorage/#the-sessionstorage-attribute | 16:36 |
| gary_poster | full domain works better for us (e.g. aws) | 16:36 |
| hatch | ""Each top-level browsing context has a unique set of session storage areas, one for each origin."""" | 16:36 |
| hatch | YUI has a cookie module as well if we choose to go this route http://yuilibrary.com/yui/docs/cookie/ | 16:37 |
| hazmat | top level browsing context doesn't reference domains afaics | 16:38 |
| Makyo | bcsaller_, make debug and make prod seem to be working in oldtrunk, so perhaps something with a stale file as served by node | 16:38 |
| hazmat | gary_poster, hatch yeah.. the separate per document nature of session storage defeats the purpose of it. cookies or localStorage both work.. re cookies, the only concern is non-https gui usage.. | 16:41 |
| hazmat | but we're really trying to stop that usage for any real world usage in preference of minimally a self-signed cert | 16:41 |
| gary_poster | agreed hazmat. | 16:41 |
| hazmat | local storage means client password potentially on disk | 16:42 |
| hazmat | gary_poster, so if i go to a new tab against the same domain do i get back my own cookie? | 16:42 |
| gary_poster | y hazmat | 16:43 |
| * hazmat explores a bit more | 16:46 | |
| hazmat | gary_poster, only concern is that it seems quite strange to send the cookie out with plaintext creds, at min we'd have to require ssl unless we're in.. if we have things like js error collection or analytics, it could easily accidentally sniff those, ditto is possible for logs on the static resource server | 16:51 |
| hazmat | gary_poster, the per domain nature of storage isn't nesc an issue, we can namespace creds using env uuid | 16:52 |
| gary_poster | hazmat, per domain: we may have misread. agreed either way not an issue | 16:52 |
| hazmat | gary_poster, i think the per domain is accurate wrt to localstorage | 16:52 |
| gary_poster | if js runs on page, both approaches are flawed | 16:52 |
| gary_poster | I mean, foreign js | 16:53 |
| hatch | yeah any js from the same domain will have access to both | 16:53 |
| hazmat | gary_poster, not nesc.. but std browser env stuff can get pushed out.. | 16:53 |
| gary_poster | didn't follow last sentence hazmat sorry | 16:54 |
| hazmat | gary_poster, hatch re third party js, the common case is either analytics or error tracking js, typically from different domains | 16:54 |
| hatch | hazmat: so your worried that the user will access the page via http and the data will be sent clear over the wire? | 16:54 |
| hatch | that's valid | 16:54 |
| hatch | sessionstorage won't have that issue because we explicitly need to send it | 16:55 |
| hazmat | gary_poster, things like cookies are common access for analytics, or error collection, storage is not and easily audited | 16:56 |
| gary_poster | ack hazmat | 16:56 |
| gary_poster | agreed | 16:56 |
| hazmat | hatch, we're trying to minimize http, i hope we can get farther along that road. but say we start hosting to the gui on cdn for better speed access, now we're at the mercy of a third party to complete the loop on security | 16:57 |
| hatch | yep good point | 16:57 |
| hatch | so then we should pop landscape open in a new window | 16:57 |
| gary_poster | well, we will send the password in the clear over the websocket in that case | 16:57 |
| gary_poster | we need https/wss | 16:58 |
| hazmat | gary_poster, cdn is independent of http/https.. we own the websocket code.. and we can promote https there | 16:58 |
| hazmat | gary_poster, even if the cdn is https, we'd still be dependent on the third party to close the security loop, because they could simply log the cookies, even by accident | 16:58 |
| gary_poster | true | 16:59 |
| gary_poster | hazmat, convinced by sessionstorage. hatch? | 16:59 |
| hatch | yep sessionstorage looks like the best approach | 16:59 |
| gary_poster | cool, thanks both | 16:59 |
| hazmat | i was wondering if we could get away with localStorage to avoid the tab issue.. but realistically we don't want the user to have multiple gui tabs open.. its a sig. waste.. | 17:01 |
| hatch | gary_poster: remember that permission denied issue I had yesterday - I think it's because npm was installed via sudo :/ | 17:01 |
| gary_poster | agreed hazmat. so you agree tab/window for opening launchpad? we can use same name so we don't open billions on repeated use | 17:02 |
| gary_poster | s/launchpad/landscape/ | 17:02 |
| gary_poster | hatch ^^ :-P | 17:02 |
| hatch | haha | 17:02 |
| gary_poster | hatch, npm is supposed to be installed in system | 17:03 |
| hazmat | gary_poster, we'd always get two.. we can't control the name on the first one | 17:03 |
| gary_poster | via apt | 17:03 |
| gary_poster | true :-( | 17:03 |
| hazmat | hatch, via chris-lea node js ppa | 17:03 |
| gary_poster | (as in hacking doc, as you were looking at) | 17:04 |
| hatch | yeah I blindly copied that line | 17:04 |
| hatch | which installs npm under sudo | 17:04 |
| hatch | so every time I run make devel | 17:04 |
| gary_poster | that line is fine | 17:04 |
| hatch | I need sudo | 17:04 |
| gary_poster | :-/ | 17:04 |
| gary_poster | not me. package just changed... | 17:04 |
| hatch | so I need to 'un sudo' npm :) | 17:05 |
| gary_poster | hatch, is this same for you? | 17:05 |
| gary_poster | $ ls -l /usr/bin/npm | 17:05 |
| gary_poster | lrwxrwxrwx 1 root root 21 Nov 5 14:52 /usr/bin/npm -> /etc/alternatives/npm | 17:05 |
| gary_poster | and | 17:05 |
| gary_poster | $ which npm | 17:06 |
| gary_poster | /usr/bin/npm | 17:06 |
| hatch | yep | 17:06 |
| gary_poster | uh. | 17:06 |
| hatch | I'm logged in as my user though | 17:06 |
| gary_poster | yeah, supposed to be | 17:06 |
| hatch | I can run npm without sudo | 17:06 |
| gary_poster | yeah, good, that's what I'd expect | 17:07 |
| hatch | but running make devel throws an npm error about crypto something | 17:07 |
| hazmat | hatch, can you pastebin the error | 17:07 |
| gary_poster | hatch, pastebin? http://pastebin.ubuntu.com/ commonly used fwiw but whatever works | 17:07 |
| hazmat | hatch, incidentally there's a nifty cli for pastebinit.. apt-get install pastebinit | 17:07 |
| hazmat | you can pipe a file into or pass a file as arg | 17:08 |
| hatch | oh that's cool, does it give you an option to delete these after? :) | 17:08 |
| gary_poster | who wants to delet anything? In perpetuity, and beyond! | 17:08 |
| hatch | http://pastebin.ubuntu.com/1639976/ | 17:10 |
| gary_poster | hatch ls -l output at top of branch? | 17:11 |
| hatch | http://pastebin.ubuntu.com/1639979/ | 17:12 |
| hatch | I'll create a new branch and run make devel again to get the logs | 17:13 |
| hazmat | hatch, you probably sudo make at some point.. | 17:14 |
| hazmat | it seems like your perms aren't quite right | 17:14 |
| gary_poster | hatch, not clear. you already tried make clean, I think, yesterday? would be interesting to find what is owned by root or whomever. You could try sudo chown -R pihach:jpihach . | 17:14 |
| hazmat | i'd suspect some root owned files.. you could also adjust via chown -R <youruser_name>: checkout_root_dir | 17:14 |
| gary_poster | :-) | 17:15 |
| hatch | https://gist.github.com/hatched/171fd627c3f757d4111d | 17:15 |
| * hazmat wanders back to the line stoppage | 17:15 | |
| hatch | that's thrown right at `npm install` | 17:16 |
| hazmat | hatch, yeah.. definitely ran npm as root accidentaly.. | 17:16 |
| hazmat | hatch, chown -R jpihach: ~/.npm | 17:16 |
| hazmat | and then run again | 17:16 |
| hazmat | you don't need to prefix make with sudo | 17:16 |
| hazmat | it will ask for sudo if it needs it | 17:16 |
| hatch | ahh there we go - on osx I had to install npm without sudo for it to work properly | 17:17 |
| hatch | thanks | 17:17 |
| gary_poster | hatch, you are using makefile on OS X directly, not in vm? | 17:18 |
| hatch | no no I'm in VM now, I was referencing what I previously had to do on osx | 17:18 |
| gary_poster | oic | 17:18 |
| gary_poster | fwiw, make devel is working fine for me | 17:21 |
| hatch | looks like I need 'shelltoolbox' for lbox propose -cr | 17:21 |
| hatch | going to have to add that to the HACKING doc | 17:21 |
| hatch | ImportError: No module named shelltoolbox | 17:21 |
| gary_poster | thanks hatch. python-shelltoolbox. Actually, one sec | 17:22 |
| hazmat | bcsaller_, the emfiles on watch thing is concerning | 17:22 |
| hazmat | Makyo, bcsaller_, basically trunk make-devel is busted then afaics | 17:23 |
| gary_poster | wfm :-/ | 17:23 |
| hazmat | gary_poster, make clean-all && make devel works? | 17:23 |
| bcsaller_ | hazmat: yes, I can try to track it down later | 17:23 |
| gary_poster | hazmat was fresh checkout. can try that too | 17:23 |
| Makyo | hazmat, bcsaller_, it works for me with a fresh checkout, yeah. Exploring more. | 17:24 |
| gary_poster | hatch, yes, go for python-checkout | 17:24 |
| hazmat | so the other local state we have is in ~/.npm | 17:24 |
| gary_poster | ah | 17:24 |
| gary_poster | python-shelltoolbox hatch | 17:24 |
| hatch | both? ok | 17:24 |
| gary_poster | line 53 of HACKING hatch. no only python-shelltoolbox | 17:24 |
| hazmat | hatch, later | 17:24 |
| gary_poster | sorry | 17:24 |
| hatch | ok gota find PPA for that | 17:27 |
| hatch | before I install.... https://code.launchpad.net/~yellow/+archive/ppa/ | 17:28 |
| hatch | that's the proper python-shelltoolbox ? | 17:28 |
| gary_poster | hatch, no, use juju ppa | 17:28 |
| gary_poster | hatch sudo add-apt-repository ppa:juju/pkgs | 17:28 |
| hazmat | that belongs in the hacking doc as well | 17:28 |
| gary_poster | you should use that version of juju anyway | 17:28 |
| gary_poster | agreed | 17:28 |
| gary_poster | hatch ^^ | 17:29 |
| gary_poster | hazmat, bcsaller_ , Makyo, fwiw, all of following worked: fresh checkout, make devel, make clean-all, make devel . Agreed that clean-all apparently misses some local state that is corrupt. My trunk has been around for awhile and is ok | 17:30 |
| gary_poster | I mean I have a separate, older one | 17:30 |
| gary_poster | I think | 17:30 |
| gary_poster | doublechecking | 17:30 |
| * hazmat tries fresh checkout | 17:30 | |
| hazmat | my trunk is definitely feeling a bit busted | 17:30 |
| hazmat | i still suspect we're getting different deps and possible caching in ~/.npm due to inexact version specs | 17:31 |
| gary_poster | would prefer exact version specs msyelf | 17:31 |
| hazmat | still working to verify though | 17:32 |
| bcsaller_ | gary_poster: we might have found it, I think its app/templates.js | 17:32 |
| bcsaller_ | some old version was still being used and wasn't sync'd with the current code, it should have been regenerated on server startup though and wasn't, even after a make clean | 17:33 |
| hazmat | gary_poster, we don't control transitive deps though | 17:33 |
| hazmat | although i would hope that's not the issue | 17:33 |
| hazmat | bcsaller_, did that also resolve the em watch error? | 17:33 |
| gary_poster | ugh, true hazmat. we could if we specified all of them | 17:33 |
| gary_poster | yuck | 17:33 |
| bcsaller_ | hazmat: I don't get that everytime, only if the systems been running for a while | 17:34 |
| hazmat | bcsaller_, aha, thanks | 17:34 |
| bcsaller_ | hazmat: which is why I think there is a descriptor leak | 17:34 |
| * hazmat stops barking at the wrong tree | 17:34 | |
| hatch | wow propose takes a long time | 17:35 |
| hazmat | bcsaller_, this is my first gui dev in a bit, so it could be a leak unrelated | 17:35 |
| hatch | need more powa!! | 17:35 |
| hazmat | hatch, i always run lbox commands with -v | 17:35 |
| hatch | oh that's a good idea | 17:35 |
| Makyo | Yeah, tracked the error from console out to stale templates. | 17:36 |
| Makyo | Should clean get generated files like templates.js? | 17:36 |
| gary_poster | Cool bcsaller_, Makyo thanks | 17:36 |
| gary_poster | Makyo, definitely | 17:36 |
| gary_poster | I thought it cleared out all build-* | 17:36 |
| gary_poster | which ought to do the trick? | 17:36 |
| Makyo | templates.js is stored in app/ | 17:36 |
| gary_poster | Makyo, that's old | 17:37 |
| gary_poster | should be anyway | 17:37 |
| gary_poster | I think... | 17:37 |
| gary_poster | checking | 17:37 |
| gary_poster | yes Makyo that's old | 17:37 |
| gary_poster | app/templates.js should not exist | 17:37 |
| gary_poster | nothing builds it any more there AFAIK | 17:37 |
| Makyo | gary_poster, yeah, but it did in mine and bcsaller_'s old trunks. | 17:37 |
| gary_poster | Makyo, used to be there | 17:37 |
| gary_poster | so new builds should never have this error | 17:38 |
| bcsaller_ | and the system is still configured to read that file? | 17:38 |
| gary_poster | we could try cleaning out old cruft too | 17:38 |
| gary_poster | I suspect that this was some build transition mistake | 17:38 |
| Makyo | Still loaded by modules-debug.js | 17:38 |
| Makyo | Line 116 | 17:39 |
| gary_poster | Makyo but built in build- dirs | 17:39 |
| gary_poster | Makyo, or served dynamically when in devel mode IIRC | 17:40 |
| hatch | OK can I get review 2x https://codereview.appspot.com/7312084 | 17:40 |
| gary_poster | I can't make my freaking index.html reste | 17:40 |
| gary_poster | reset | 17:40 |
| * hatch hopes he did it right | 17:40 | |
| gary_poster | I did a make appcache-force repeatedly, and cleared chrome cache repeatedly, but old index.html still there | 17:41 |
| gary_poster | Makyo forgetting anything? Any other ideas ^^ ? | 17:41 |
| Makyo | gary_poster, Yeah, I see that in the lib/templates.js script, but it doesn't jive with what devel's serving, because there's a catchall rule for /juju-ui/*-> app/{file} | 17:41 |
| Makyo | gary_poster, Hmm, just a sec. | 17:42 |
| hazmat | bcsaller_, aha.. re em watch error.. its the inotify limit.. http://stackoverflow.com/questions/10086336/multiple-node-js-always-error-watch-emfile | 17:45 |
| hazmat | its 128 by default it seems | 17:45 |
| hazmat | fixed with echo 8704 | sudo tee /proc/sys/fs/inotify/max_user_instances | 17:45 |
| Makyo | hazmat, bcsaller_: oh man, sorry, I should've looked at your pastebin earlier. I ran into that a while back, too, fixed similarly. Sorry :/ | 17:46 |
| bcsaller_ | Makyo: heh, I increased my limit too already, but I'm still not sure why we are hitting it, maybe we crossed 128 files in the dir or something | 17:47 |
| Makyo | gary_poster, I added a letter and it showed up with make appcache-force, then right click in the "Network" section of the console, clear browser cache. | 17:47 |
| gary_poster | trying | 17:47 |
| Makyo | Then it takes a second to reload and the status bar says 'Waiting on AppCache...' | 17:48 |
| gary_poster | no luck Makyo | 17:48 |
| gary_poster | I'll try restart chromium... :-/ | 17:49 |
| gary_poster | nope | 17:51 |
| Makyo | gary_poster, In the network tab, for localhost, does it say (from cache) in the size column? | 17:54 |
| gary_poster | Makyo, yes | 17:55 |
| gary_poster | and timestamp of manifest is definitely changing | 17:55 |
| gary_poster | and I'm definitely clearing cache :-) | 17:56 |
| Makyo | Snagging Chromium. Did you get yours from apt, gary_poster ? | 17:59 |
| gary_poster | y Makyo thanks | 17:59 |
| gary_poster | on call but will return soon | 17:59 |
| gary_poster | Makyo back | 18:12 |
| gary_poster | You install chrome from google rather than chromium from apt usually? | 18:13 |
| Makyo | gary_poster, personally, yeah. Just hadn't gotten around to installing chromium yet. | 18:15 |
| Makyo | gary_poster, ctrl+shift+r help any? That made it work on my end. | 18:15 |
| Makyo | changed index, appcache-force, clear browser cache - plain refresh didn't show a change, but ctrl-shift-r did. Different behavior from Chrome. | 18:16 |
| gary_poster | no had tried that. will retry. I could also try touching index maybe... | 18:16 |
| gary_poster | touching index.html was the trick Makyo . Thanks and sorry to bother | 18:18 |
| Makyo | gary_poster, no worries, good to know! | 18:18 |
| gary_poster | hatch, if you link branch to bug that is nice. Can do in command line with bzr commit --fixes=lp:1122706, or in Launchpad from bug or branch. I just linked for you | 18:19 |
| gary_poster | It helps people and kanban automation find related artifacts | 18:20 |
| hatch | ok great, I just made note of that | 18:21 |
| hatch | am I doing these requests properly? | 18:21 |
| hatch | they sure take a long time for small changes - are we able to group fixes in a branch? | 18:21 |
| Makyo | hatch, lbox taking a long time? | 18:23 |
| Makyo | hatch, if so, the reason is it's running tests and lint. | 18:23 |
| gary_poster | I think he is referring to all of the bug card branch stuff | 18:23 |
| gary_poster | hatch you can combine for very small things | 18:23 |
| hatch | ok :) | 18:23 |
| Makyo | Ah, alright. | 18:24 |
| Makyo | Okay, really need to finish moving stuff outside, back in a few. | 18:24 |
| gary_poster | hatch, "land as is" for your branch for me. You need one other person. I would have either combined the HACKING changes into the same branch or just changed it without a card or bug, since it would take no time at all. | 18:28 |
| gary_poster | What you are doing is even nicer, to be clear | 18:28 |
| gary_poster | but for your own annoyance, I mean | 18:28 |
| hatch | ok sure thing | 18:29 |
| hatch | ok finally cleaned up enough ram to boot up this windows 8 vm | 18:30 |
| hatch | *please don't crash...please don't crash* | 18:30 |
| hatch | ;) | 18:30 |
| hatch | anyone here use 'Fitbit' ? | 18:30 |
| hatch | do we have a Canonical fitbit team? :) | 18:31 |
| gary_poster | heh, I don't, heard good things about it though | 18:31 |
| hatch | it's proof of how lazy I can be some days | 18:32 |
| hatch | :) | 18:32 |
| gary_poster | lazily active? | 18:32 |
| gary_poster | actively lazy? | 18:32 |
| hatch | I'm going to go with actively lazy | 18:33 |
| gary_poster | :-) | 18:33 |
| hatch | this is a longshot but has anyone here know how to access one VM from another via the network using parallels? | 18:43 |
| gary_poster | hatch, speaking generally, put ubuntu vm in bridged networking mode | 18:45 |
| gary_poster | you might need the win one to be bridged too | 18:45 |
| gary_poster | then find out the ip of ubuntu | 18:46 |
| gary_poster | and use that to connect from win to ubuntu | 18:46 |
| gary_poster | for the gui, you will want to change a file: app/config-prod.js for make prod and app/config-debug.js otherwise | 18:47 |
| gary_poster | change socket_url: 'ws://localhost:8081/ws' from localhost to the ubuntu ip | 18:47 |
| hatch | yeah I'm in bridged and have the IP but no luck - I probably need to punch a hole in ubuntu to open up the port | 18:47 |
| gary_poster | then restart | 18:47 |
| gary_poster | parallels more likely to be firewalled than ubuntu, is my guess | 18:48 |
| gary_poster | IOW, look for firewall on that side | 18:48 |
| gary_poster | first :-) | 18:49 |
| hatch | alrighty thanks | 18:49 |
| gary_poster | bac, you happen to have any experience with this ^^ ? | 18:49 |
| bac | gary_poster: yep. reading back. | 18:49 |
| hatch | basically I want to access the gui server on ubuntu | 18:50 |
| bac | gary_poster, hatch: the steps gary outlined are the ones i've used successfully. never had any problems with firewall on the ubuntu vm | 18:50 |
| hatch | so all I am doing in windows is opening up the browser | 18:50 |
| hazmat | hatch, yes re fibit | 18:51 |
| hazmat | hatch, although mine busted.. so not anymore.. waiting for a new compeling device atm | 18:51 |
| bac | hatch: so ubuntu vm and windows vm both have real DHCP addresses | 18:51 |
| hazmat | hatch, there's a few folks with them | 18:51 |
| hatch | bac: yeah I shut down the windows vm to first get access from OSX (host) to ubuntu (vm) | 18:52 |
| hatch | simply going {ubuntu's ip}:8888 doesnt work | 18:52 |
| bac | does it do anything? | 18:52 |
| bac | or there is just no answer? | 18:52 |
| hatch | unable to connect | 18:53 |
| hatch | ok changed the networking from bridged and then back to it | 18:54 |
| hatch | and now I have the browser not supported warning | 18:54 |
| hatch | great | 18:54 |
| bac | hatch: ah, cool | 18:54 |
| hatch | must have been a parallels glitch | 18:54 |
| hatch | thanks | 18:55 |
| hatch | hazmat: maybe I'll fire a msg to see if there is a 'Canonical group' on there | 18:55 |
| hatch | I see there is a mailing list where people complain about things so that's probably the best place to ask | 18:57 |
| hatch | ;) | 18:57 |
| gary_poster | heh, yeah warthogs is the place for that | 18:57 |
| hatch | excellent now have juju in windows 8 | 19:01 |
| hatch | time to get some work done! | 19:01 |
| hatch | oh websocket error....forgot to change the config | 19:02 |
| hatch | woops | 19:02 |
| gary_poster | hatch, our CI tests are not liking IE or Firefox much :-( http://pastebin.ubuntu.com/1640314/ | 19:03 |
| gary_poster | well, soon-to-be-CI tests, for now merely selenium saucelabs tests :-P | 19:04 |
| gary_poster | this is from frankban's branch in review: https://codereview.appspot.com/7307104/ | 19:04 |
| hatch | well now I know what happens when you try to use ram you don't have | 19:06 |
| hatch | can you repost that link gary_poster? :) | 19:06 |
| gary_poster | heh | 19:06 |
| gary_poster | <gary_poster> hatch, our CI tests are not liking IE or Firefox much :-( http://pastebin.ubuntu.com/1640314/ | 19:06 |
| gary_poster | <gary_poster> well, soon-to-be-CI tests, for now merely selenium saucelabs tests :-P | 19:06 |
| gary_poster | <gary_poster> this is from frankban's branch in review: https://codereview.appspot.com/7307104/ | 19:06 |
| hatch | it looks like it can't even connect | 19:10 |
| gary_poster | mm, actually, the "Your browser is not fully supported" breaks our selenium tests on FF andf chrome :-/ | 19:13 |
| hatch | heh I was just going to check that | 19:14 |
| gary_poster | and ie10 does not accept the security cert :-( | 19:14 |
| gary_poster | benji ^^ nothing to act on right now, but fwiw | 19:15 |
| hatch | security...who needs it! | 19:15 |
| benji | darn | 19:15 |
| gary_poster | :-) well, selenium is supposed to be able to tell browsers to accept self-signed | 19:16 |
| gary_poster | it works for chrom and ff | 19:16 |
| gary_poster | but not IE apparently :-/ | 19:16 |
| gary_poster | benji, for browser check we can just add some automation for that, yeah? | 19:16 |
| gary_poster | don't know what to do about ie though | 19:16 |
| hatch | curious - why do we stop the execution of the app to tell them that it's not supported? | 19:17 |
| benji | yeah, we could attack that in several ways. One candidate would be to add a ?shut-up-about-my-browser-already flag to the initial request URL | 19:17 |
| gary_poster | http://sauceio.com/index.php/2012/10/announcing-sauce-support-for-windows-8-and-ie-10/ | 19:17 |
| gary_poster | "We’re still working out some of the kinks, such as making these compatible with our standard support for self-signed certificates." | 19:18 |
| benji | well, that certainly explains that | 19:19 |
| gary_poster | :-/ | 19:19 |
| * gary_poster needs some lunch | 19:19 | |
| gary_poster | biab | 19:19 |
| hatch | config-debug.js is loaded with `make devel` ? | 19:20 |
| hatch | using bzr is there a way I can find who wrote a line? | 19:36 |
| hatch | gary_poster: is there someone who could give me a run through of the application code? | 19:39 |
| hatch | I'm mostly interested in topology/service.js | 19:39 |
| hatch | but an application overview would probably be better :) | 19:40 |
| BradCrittenden | hatch: second review done on your alert style branch | 19:42 |
| === BradCrittenden is now known as bac | ||
| hatch | oh great | 19:42 |
| hatch | so I'm free to `lbox submit` that now? | 19:43 |
| bac | hatch: yes since there are no requested changes. | 19:44 |
| bac | hatch: your other branch is in review but i don't see a merge proposal | 19:44 |
| * bac suspects lp2kanban is on strike | 19:44 | |
| hatch | https://code.launchpad.net/~hatch/juju-gui/1123291-update-hacking | 19:45 |
| hatch | the 'ready for review' section? | 19:45 |
| hatch | oh you're saying the kanban thing is missing it | 19:45 |
| hatch | :) | 19:45 |
| === arosales1 is now known as arosales | ||
| hatch | can someone point me to the documentation for `getClientRect()` as used in app/view/topology/service.js #828 var width = label.one('text').getClientRect().width + 10; | 20:09 |
| hatch | I know of getClientRects() but not getClientRect() | 20:10 |
| bcsaller_ | hatch: its a simple utility method from svg-layout in assets/javascript | 20:11 |
| hatch | oh pfft I guess I could have ack'd the dir | 20:11 |
| hatch | :) my bad | 20:11 |
| bcsaller_ | np | 20:11 |
| bac | lp2kanban working again | 20:27 |
| bac | dogwalk. bbiab | 20:38 |
| gary_poster | bac, land with changes | 21:17 |
| bac | gary_poster: thanks | 21:24 |
| gary_poster | welcome | 21:24 |
| bac | hatch is your HACKING branch in review or are you still working on it? | 21:33 |
| hatch | review - it shows as such in lp | 21:33 |
| hatch | but maybe didn't get picked up in kanban | 21:33 |
| bac | https://bugs.launchpad.net/juju-gui/+bug/1123291 | 21:34 |
| bac | hatch: that's what i was looking at ^^ | 21:34 |
| bac | ah, i see your branch now. it isn't linked to the bug. | 21:35 |
| hatch | sorry - linked | 21:37 |
| bac | hatch: it the branch is linked to the bug then we have a cronscript (lp2kanban) that is supposed to do magic like link the kanban card to the rietveld review, etc. without the linkage stuff is hard to find. | 21:37 |
| bac | np | 21:37 |
| hatch | ohh ok now I understand | 21:38 |
| hatch | I suppose how else is the script supposed to know :D | 21:38 |
| gary_poster | benji, do you have a suggested way to qa the "loading" message? | 21:49 |
| benji | gary_poster: step one: wait a couple of minutes while a clean up the branch a bit more ;) | 21:49 |
| gary_poster | benji lol ok | 21:49 |
| benji | gary_poster: step two through N: this is what I did: I tested devel, debug, and prod, against Chrome and FF to ensure the behavior was expected in each case | 21:50 |
| gary_poster | benji, how did you make it so that js took forever to load? hacked code? | 21:51 |
| benji | gary_poster: between my machine being slow and the app taking a while to initialize, I didn't have to simulate anything | 21:51 |
| gary_poster | heh | 21:52 |
| gary_poster | ok | 21:52 |
| gary_poster | benji, ping me when I should look again | 21:52 |
| gary_poster | thanks | 21:52 |
| * benji checks lbox's progress | 21:52 | |
| benji | gary_poster: it's ready: https://codereview.appspot.com/7314082 | 21:53 |
| benji | gary_poster: also, if you want to simulate a slow network the "tc" command can do that | 21:53 |
| gary_poster | 'yeah I thought there was some trick like that | 21:53 |
| gary_poster | benji http://www.commandlinefu.com/commands/view/11475/simulate-slow-network-connection-locally ? | 21:54 |
| hatch | before I do - I'd like to confirm that I can freely delete the folder which was successfully 'lbox submit' 'ed | 21:54 |
| hatch | ? | 21:54 |
| gary_poster | hatch yes | 21:55 |
| hatch | eggcelent | 21:55 |
| hatch | new alert list is on ui stage | 21:55 |
| benji | gary_poster: exactly | 21:55 |
| gary_poster | cool thanks benji | 21:56 |
| benji | gary_poster: if you really want to torture the networking stack you can add "loss 50%" and half the packets will be dropped on the floor | 21:56 |
| bac | benji: two reviews done | 21:56 |
| gary_poster | heh | 21:56 |
| benji | thanks bac | 21:56 |
| gary_poster | I have news on the golang thing but have not had time to write it up | 21:57 |
| bac | benji: bah, who needs 'tc' when you have claro internet? | 21:57 |
| benji | heh | 21:58 |
| gary_poster | summary: there are some things we can start with that are useful. They are too few for all of us. There are a few other things we can do that are less useful but that can get our feet wet. With those two combined we should all have something to dig into tomorrow | 21:58 |
| hatch | to get this alert box QA'd by the designers do I need to do anything? or will they see it in the kanban list? | 21:58 |
| bac | buenos noches mi compadres | 21:58 |
| gary_poster | bac, I hope you enjoy your nachos also | 21:58 |
| * benji translates for non-spanish speakers: "Good nachos, in my compartments" | 21:59 | |
| bac | ummm, cheese product | 21:59 |
| gary_poster | lol, thanks benji | 21:59 |
| * Makyo dogwalkinates | 21:59 | |
| gary_poster | hatch what you have is fine. if they haven't looked at it by daily call, ask them about it then or I will | 21:59 |
| hatch | alrighty | 21:59 |
| gary_poster | sometimes wirth sending them (nick, greg) an email | 21:59 |
| gary_poster | worth | 21:59 |
| bac | i saw the lady at "El Hamburger" load the bag of yellow mess into the nacho cheez dispenser the other day. yuk. | 21:59 |
| gary_poster | heh | 22:00 |
| bac | gary_poster: that typo was almost a pascal reference | 22:00 |
| * gary_poster tries to remember pascal and fails | 22:00 | |
| gary_poster | blaise was a nice fellow... | 22:00 |
| benji | I made a bet with him once... still don't know if I won. | 22:01 |
| gary_poster | heh | 22:01 |
| gary_poster | benji spinner missing from make prod | 22:04 |
| gary_poster | http://localhost:8888/juju-ui/assets/javascripts/spin.min.js is 404 | 22:04 |
| benji | darn | 22:04 |
| benji | I'll have to wrestle with the Makefile some more then. | 22:04 |
| gary_poster | make devel looking nice so far | 22:05 |
| gary_poster | wow, that's a nice slow fake internet I have here | 22:05 |
| gary_poster | benji, make devel is also failing, in an intriguing way | 22:06 |
| gary_poster | it keeps on trying to connect to the ws | 22:06 |
| gary_poster | succeeding | 22:06 |
| gary_poster | and then letting go | 22:06 |
| gary_poster | I see this in my improv output | 22:07 |
| gary_poster | the browser shows "Loading the Juju GUI" forever | 22:07 |
| gary_poster | benji, that's with the tc info I gave you, and a large improv. Trying a faster connection... | 22:10 |
| gary_poster | yeah that was the problem | 22:11 |
| hatch | bac: I'll integrate your comments on the HACKING doc | 22:16 |
| hatch | should I reply on codereview? | 22:17 |
| gary_poster | hatch, yes. As a rule, respond to each request in the review. "I did everything you asked" is good enough, as long as it is accurate. :-) | 22:18 |
| hatch | sounds good! | 22:22 |
| hatch | what's the best approach to `git reset --hard HEAD` with bzr? (that resets the working branch to HEAD throwing out all changes) | 22:24 |
| gary_poster | If you are using lightweight checkouts, which are a bit like git and a component of cobzr and other similar approaches, there are some parallels. Otherwise...you just make a new branch. I bet no one told you about init-repo | 22:27 |
| gary_poster | if you say bzr init-repo . | 22:28 |
| gary_poster | that turns the directory into a cache | 22:28 |
| gary_poster | then you bzr branch within that directory | 22:28 |
| gary_poster | you can delete those branches | 22:28 |
| gary_poster | and when you branch the same source, virtually everything is still around | 22:28 |
| gary_poster | so very fast | 22:29 |
| gary_poster | trunk does not work for me in FF :-( | 22:29 |
| gary_poster | sigh | 22:29 |
| gary_poster | It's almost like we need CI! | 22:29 |
| gary_poster | hatch, ^^ | 22:31 |
| hatch | soo `mkdir juju-gui; cd juju-gui; bzr init-repo; bzr branch lp:juju-gui;` | 22:31 |
| Makyo | hatch, or: `bzr init-repo juju-gui; cd juju-gui; bzr branch lp:juju-gui trunk; bzr branch trunk mine/<featurebranch>` | 22:32 |
| Makyo | Forgot a `mkdir mine` in there. | 22:33 |
| Makyo | Then you can pull down others' branches to, say, makyo/feature1, gary/feature2, etc. | 22:33 |
| hatch | ahh interesting - alright let me try that | 22:34 |
| Makyo | hatch, http://doc.bazaar.canonical.com/beta/en/user-guide/organizing_your_workspace.html#feature-branches is I think how a lot of us work. | 22:38 |
| Makyo | (minus the send/push steps, what with lbox) | 22:38 |
| hatch | gary_poster: Makyo: wow that's fast compared to the way I was doing it before lol | 22:41 |
| gary_poster | hatch, hacking branch: land with changes | 22:41 |
| gary_poster | :-) | 22:41 |
| Makyo | hatch, Yeah, it pulls down most of the tree to trunk, then branching is local. | 22:41 |
| hatch | gary_poster: sure thing I'll add those changes tonight | 22:43 |
| gary_poster | thanks | 22:43 |
| gary_poster | FF is ok. was some weird FF state. restarted and fine | 22:45 |
| hatch | I get every email from codereview x2 - is this normal? | 22:45 |
| Makyo | hatch, Yeah. | 22:45 |
| hatch | ok just checking :) | 22:46 |
| gary_poster | hatch sorry, lbox artifact. rietveld talks to you and then launchpad talks to you. maybe you can quiet LP down. haven't tried | 22:46 |
| hatch | it's alright - I was moreso curious if I did something wrong | 22:47 |
| gary_poster | benji land with changes, though I have some important notes. make prod works fine: it is a stale makefile thing | 23:08 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!