/srv/irclogs.ubuntu.com/2014/06/18/#juju-gui.txt

hatchinteresting bug https://bugs.launchpad.net/juju-gui/+bug/133124800:12
_mup_Bug #1331248: Uploading a local charm hangs when uploading to ec2 <juju-gui:New> <https://launchpad.net/bugs/1331248>00:12
rick_h_hatch: yea, I'm guessing it's more sync waiting for juju response00:13
rick_h_since your upload speed to juju probably stinks00:13
hatchrick_h_ yeah we need to show some kind of inspector...or spinner...or something :)00:13
rick_h_hatch: yea00:14
hatchdamn to I like the GUI though00:15
hatch:)00:15
hatchusing the CLI is for suckers :P00:15
rick_h_:)00:15
rick_h_cli4lif 00:15
hatchlol00:15
hatchrick_h_ there was a bug about the inspector staying around when you destroy the service.....any idea where that bug is?00:20
rick_h_hatch: thought it was corrected in the last release00:21
hatchI was able to reproduce it in the latest release so I can re-create it if it was destroyed00:22
hatchyeah I'll remake it, I can't find the bug00:22
rick_h_I thought it was just marked fix released00:22
* rick_h_ looks00:22
rick_h_hatch: so are you getting that bug?00:22
hatchyep00:22
rick_h_https://bugs.launchpad.net/juju-gui/+bug/132155800:23
_mup_Bug #1321558: Destroying a service leaves inspector visible <juju-gui:Fix Released by hatch> <https://launchpad.net/bugs/1321558>00:23
hatchon the latest GUI release on precise 00:23
hatchahh ok my issue is slightly different00:23
hatchI'll create a new bug00:23
hatchaaaaa voila https://bugs.launchpad.net/juju-gui/+bug/133125000:26
_mup_Bug #1331250: Clicking to view the inspector of a dying service keeps it around after the service is destroyed <juju-gui:New> <https://launchpad.net/bugs/1331250>00:26
hatch^ rick_h_  so it's different, definitely an edge case, but something that we should address sometime :)00:28
rick_h_hatch: rgr, add it to the backlog and we'll get it up sometime. 00:29
hatchsounds good00:32
hatchhmm changing the port to 80 breaks the ghost charm00:40
hatchvewwwwwy intewwesting00:40
rogpeppehuwshimi: hiya07:55
huwshimirogpeppe: Morning07:56
rogpeppehuwshimi: you're good at Javascript, right?07:56
rogpeppehuwshimi: i just saw a (quite amusing - he's always quite amusing) random question about javascript from a mate on facebook and wondered if you might be able to provide the answer...07:57
rogpeppehuwshimi: http://paste.ubuntu.com/7662476/07:57
rogpeppehuwshimi: any idea?07:58
huwshimirogpeppe: Maybe, I'll take a look in a sec, just on a call :)07:58
rogpeppehuwshimi: ah, np, ta!07:59
huwshimirogpeppe: Instead of doing the substring he could try ".replace('£', '')"08:21
huwshimirogpeppe: So, var origprice = parseInt($(this).find(".origprice").text().replace('£', ''));08:22
huwshimirogpeppe: Would that work?08:22
rogpeppehuwshimi: it may do. on further looking at the issue, i reckon it's probably something before that statement08:23
rogpeppehuwshimi: if it was me, i'd probably just use a regex to replace everything except trailing \ds08:23
rogpeppehuwshimi: apparently (reading further) there was possibly a nbsp there too08:24
rogpeppehuwshimi: perhaps it was getting treated as white space and stripped automatically at some point08:25
rogpeppehuwshimi: it seems he's now solved the issue by not producing the £ in the first place... which seems like a better solution08:26
rogpeppehuwshimi: ta for looking!08:26
huwshimino problems!08:27
frankbanrogpeppe: morning, how are you doing?09:35
rogpeppefrankban: hiya09:36
rogpeppefrankban: not bad, thanks09:36
rogpeppefrankban: you?09:36
frankbanrogpeppe: fine thanks. how is the bundle stuff going?09:37
rogpeppefrankban: i've just been familiarising myself with the store code, mostly09:38
frankbanrogpeppe: impressions?09:38
rogpeppefrankban: seems fine09:38
rogpeppefrankban: shall we have a chat about what features we want for bundles?09:39
frankbanrogpeppe: sure09:39
rogpeppefrankban: standup hangout?09:39
frankbanrogpeppe: sounds good, joining09:40
rick_h_morning all10:56
rick_h_rogpeppe: frankban can you guys bring up in the standup today to find a volunteer to attend the cloud cross team call tomorrow?11:05
rick_h_rogpeppe: frankban there's an email to canonical-tech that you can reply to do get invited in. 11:05
marcoceppihave you guys seen this? It's coming in the Google Chrome release (running dev atm)11:19
marcoceppihttp://i.imgur.com/PW29H2W.png11:19
marcoceppithis is 37.0.2041.4 dev11:20
marcoceppiThere's no way to get around it at the moment11:20
marcoceppiOh, just kidding11:21
marcoceppithere's a link11:21
rick_h_frankban: did you get travel auth?11:24
frankbanrick_h_: yes, already booked the flight11:25
rick_h_frankban: ok awesome 11:25
* frankban lunches11:41
frankbanrogpeppe: I am back12:37
frankbanrogpeppe: please ping me when you are ready12:40
rogpeppefrankban: ping13:16
jcastrohttps://news.ycombinator.com/item?id=790928613:16
jcastrocheck it out folks ^^ 13:17
bacnice jcastro13:32
frankbanrick_h_: time for a quick hangout?13:34
frankbanbac: ^^^13:35
rick_h_frankban: sure thing13:35
frankbanrick_h_, bac: we are on the daily standup hangout13:35
rogpeppebac: cs:bundle/mediawiki-313:57
rogpeppebac: cs:~someone/bundle/mediawiki-4/scalable13:58
hatchmorning all14:28
lazyPowero/ hatch14:31
lazyPowerGuess who's got 2 thumbs and aced his Ubuntu Membership exam this morning? this guy!14:32
hatchw000t congrats! 14:32
hatcharosales hey I worked a bit on the ghost charm last night and ran into an issue putting the charm on port 80 by default - I haven't solved it yet, just FYI in case you try to set it to port 80 :)14:34
lazyPowerhatch: are you rying to serve over port 80 as a normal user?14:34
hatchumm I'm not sure what user it's running as, one sec14:35
Makyohatch, we tried that in Vegas, remember?  Have to run it behind a proxy (either charm or local)14:35
hatchlazyPower ubuntu user14:36
hatchMakyo right, but why doesn't it work....it doesn't make much sense 14:36
hatchit should work without proxy14:36
Makyohatch, ports <1024 are privileged, can only run as root.14:36
hatchbut the GUI can be served over 80?14:36
MakyoBehind a privileged proxy.14:36
hatchohh14:37
hatchwell what the deuce14:37
MakyoYou can proxy it behind nginx in the charm, maybe?14:37
hatchso the review made us change the owner off root then wants us to put it on port 8014:37
hatchlol14:37
MakyoOr lighttpd or something small.14:37
* hatch shakes fist14:37
hatchI'm confused as to why we can put nginx as root but not the blog?14:38
MakyoBecause a lot of very seriously smart people put a lot of thought into how to write something that can run on a privileged port.14:39
MakyoI believe the only privileged part  of most webservers is the listener, everything else runs as an unprivileged user and communicates with that process.14:39
hatchohhhhh so you're calling the ghost authors stupid14:40
hatchnow I see14:40
hatch:P14:40
Makyo:P14:40
MakyoThere's just a difference between the type of dev work you do for a web app and the type of dev work you do for a server.14:40
hatchok well I guess I'll update the review notes to this effect (I'm not setting up nginx off the bat) to get this in the store14:40
hatchthat's what they said about crypto and openssl and look at what happened there 14:41
MakyoIt'll run behind a haproxy charm on 80, that's what I did for the demo.14:41
hatchyeah I'm fine with that - I was just trying to address the review notes14:42
hatchI wish there was an external nginx charm14:42
hatchhttps://jujucharms.com/~hp-discover/trusty/nginx-4/?text=nginx14:43
hatchthere is this one but it's not promoted for some reason14:43
hatchmarcoceppi ^ why don't we have a promoted nginx charm? Is there something wrong with this one?14:44
marcoceppihatch: I just wrote that charm last week and it still needs some work14:44
hatchohhh well then you rock14:44
marcoceppihatch: it also requires you to use subordinates (website, php-website) and it's currently not working wiht more than one sub deployed becauseeeeeee dns is a mythical being in the land of juju deployments14:45
* marcoceppi should update the readme14:46
hatchhaha - yeah I looked into writing an nginx charm and realized that it was outside of my expertise :)14:46
hatchif you need any help testing I'd be happy to give it a go14:46
hatchbut unfortunately I'm not very well versed in the nginx networking stuffs14:46
marcoceppihatch: you can deploy this and cs:~hp-discover/trusty/website if want to see it working together14:47
hatchcool14:47
Makyojujugui call in 1014:50
hatchwow it's that time already14:50
* bac trying. will be there shortly i hope15:00
hatchjujugui call now15:00
arosaleshatch, thanks for the fyi :-)15:07
hatchnp!15:07
hatchMakyo heh now you have a good consistent 1-2s lag :)15:08
bacjujugui: did everyone get their travel auth15:09
MakyoI knowww.  Makes me sad.  I may coffeeshop in the mornings.15:09
kadams54bac: yup.15:09
Makyobac, yep15:09
jcsackettbac: yup.15:09
baci guess i should've asked the inverser15:09
bacs/inverser/inverse/15:09
jcsackettMakyo: the 1-2s lag is charming. makes me feel like we're all on the evening news.15:09
hatchhaha15:09
bacMakyo: your lag makes me self-conscious15:09
hatchno, if that was the case Makyo  would be talking about Justin Beiber or something15:10
bacabout my crap internet, even though i was using the maligned LTE15:10
hatchwe need FTL packet transfer15:11
MakyoThis satellite is actually faster than our old cable, but with much higher ping times.15:11
bacjcsackett: if that's the case i want to be Sylvia Poggioli reporting from Rome15:11
MakyoHahah15:11
* bac admits having consulted wikipedia for spelling...wasn't close15:11
kadams54lol15:11
jcsacketthatch: re ghost review--i'm actually ok with us killing sqlite support based on the "opinionated deployment" idea of charms, and by that same token we shouldn't run on 80 by default b/c you should be deploying a front end proxy with it.15:13
jcsacketthatch: but if it's a super insistent requirement i can probably get the charm to setup nginx on the same machine to work with 80.15:14
jcsackettnot sure *when* i'll have time for that, but it's not actually that hard to do.15:14
hatchjcsackett yeah it's "possible" I'm just going to give some push back and see where they let me land15:14
hatchjcsackett can you comment on the "being ok with dropping sqlite" in the bug so that in the future we know where the convo left off? :)15:15
jcsacketthatch: already did.15:18
jcsackett(i assume you mean the issue you filed on github?)15:18
hatchoh haha15:18
hatchyes15:18
hatchso I bought these crystal stout beer glasses - one broke when I put it in the sink to wash it :/ quality15:51
bacfrankban: can you have a look at this doc-only quickstart branch? https://codereview.appspot.com/10612004316:08
frankbanbac sure16:08
bacfrankban: i 'cleaned up' the rst so it renders better.16:08
bacymmv16:09
bacfrankban: restview from pypi is nice-ish16:09
frankbanyeah16:09
frankbanbac: done16:17
=== cHilDPROdigY1337 is now known as nottrobin
hatchnottrobin no longer a prodigy? :)16:50
nottrobinrofl16:51
nottrobinhatch: that was a misunderstanding16:52
hatchhaha16:52
hatchMakyo here is my final version - I think this will work the best https://gist.github.com/hatched/02ac1b0650ed87877655 keeping it all in ECS16:55
Makyohatch, that looks like it'll work, yeah.16:59
hatchit's about 10% of the lines I HAD writen16:59
hatchlol16:59
hatchless is always more in code haha17:00
rogpeppefrankban: i'm very nearly done for the day now. i will try to put an email together summarising the stuff we've put together tomorrow morning17:00
frankbanrogpeppe: EOD for me too. yeah thanks, I'll ping you tomorrow, have a great evening17:01
jcsackettjuju-gui: is there an easy way to get the gui in a deployed environment to update to the most recent commit of your repo/branch?17:06
jcsackett...or even better, with a local environment, make it use your working directory...17:07
frankbanjcsackett: there is no way to use the working directory. For the first question, you can use the juju-gui-source option, e.g. juju set juju-gui juju-gui-source="https://github.com/frankban/juju-gui.git BRANCHNAME"17:09
jcsackettfrankban: yeah, i did that to set it in the first place, but then if i make changes i can't update it. and i can't hack on the source in the lxc, b/c the changes don't get picked up/loaded into the server.17:09
frankbanjcsackett: I did not try if that works only with git revisions rather than branch names. To be able to hack directly the GUI from inside the LXC, you can "juju set juju-gui juju-gui-debug=true juju-gui-console-enabled=true" and then go to something like /usr/lib/juju-gui/juju-gui/build-debug" or similar17:11
hatchjcsackett set it to develop then back to your repo right away17:11
hatchthen wait17:11
hatch:)17:11
frankbans/only/also17:11
jcsackettso, basically, there's no easy way. :)17:11
hatchalthough I would also be interested if there was a way to put a hash there17:12
jcsackettconsole-enabled is a good tip, thanks frankban 17:12
hatchor if it requires the branch name17:12
hatchbut yes frankban's method of hacking the gui in place is the best :)17:12
jcsackettso, i move that we never say a "real env" bug is small. the development process is somewhat cumbersome. :p17:13
frankbanjcsackett: yes we need to improve that, it would be great to investigate connecting a "make devel" GUI to an existing LXC guiserver17:15
hatchfrankban YES!!!17:18
hatch:)17:18
hatchjujugui looking for a review (no qa needed) https://github.com/juju/juju-gui/pull/39217:44
hatchMakyo ^17:44
kadams54hatch: I can take a look at it if you can answer a YUI question :-)17:46
hatchsure shoot17:47
kadams54Is there a way, when simulating a change event on a <select>, to specify what the new value should be?17:49
hatchselectNode.fire('change', { dataz... });17:49
hatchlike that?17:49
kadams54Maybe. I think I need to change the actual value reported by mySelect.get('value')17:54
hatchhave some code for me to see?17:55
hatchcreate a wip pr17:55
kadams54hatch: will do, after I get done reviewing your PR.17:56
hatchcool17:57
kadams54hatch: are these changes so that we can edit values in the inspector and then have them committed/deployed with the rest of the change set?17:57
hatchyep17:59
hatchits so that the inspector displays the ecs'd values even though they aren't actually saved to the env yet17:59
kadams54hatch: OK, review done. Getting WIP branch up.18:03
hatchthanks18:03
kadams54hatch: https://github.com/juju/juju-gui/pull/39318:06
hatchkadams54 why aren't you using e.newVal and deferring instead to touching the DOM again?18:29
kadams54I figured you'd mention that :-)(18:29
kadams54I've already made the change locally18:30
kadams54Unfortunately e.newVal only applies for line 97; doesn't get me off the hook for line 80.18:31
hatchlooking18:39
hatchok I'll make some comments in the code18:40
hatchkadams54 ok replied18:42
jcastrorick_h_, for quickstart for the docs for osx18:49
hatchjcastro he is out today/tomorrow18:49
hatchkadams54 do my comments make sense?18:49
jcastrook18:49
jcastrohatch, do you know anything wrt. quickstart for osx?18:50
hatchI've seen the discussions in the channel :) What did you need?18:51
jcastrodo you know if it's in brew?18:51
hatchit is!18:51
jcastroso is this legit, changing "brew install juju" to "brew install juju juju-quickstart"?18:51
jcastroI have no idea what the syntax is18:52
hatchhttp://bazaar.launchpad.net/~juju-gui/juju-quickstart/trunk/view/head:/README.rst#L5818:52
hatch:)18:52
jcastroack18:53
hatchjcastro are you working on adding the docs to the docs.u.c ?18:59
hatcher juju.u.c/docs18:59
hatch:)18:59
jcastrofor quickstart?19:00
jcastroyes, in progress19:00
hatchyeah19:00
hatchawesome19:00
jcastroI'll have them done by EOD19:00
hatchnice19:08
kadams54hatch: Yeah, they make sense: this.set('containerType', e.newVal) then just do this.get('containerType') at line 80, right?19:15
hatchyep19:15
hatchsmall improvement, but I think it's better19:15
hatcher...small change I mean19:15
hatchI wonder if we increased the size of our CI vm's if they would be speed up at a reasonable rate19:18
hatchmoar power19:18
hatchI suppose doing the tests in parallel would give the most improvement19:18
=== lazyPower is now known as lazypower-travel
hatchjujugui how do I get changes that show up in the deployer bar to also show up in the summary? 20:39
hatchI am getting set_config calls in the deployer bar, but they are absent from the summary and clicking deploy does not remove them20:40
hatchkadams54 ^ any idea where I should be looking for this?20:40
kadams54hatch: no, sorry…20:41
hatchhmm20:43
hatchok i'll bench for now, revisit in a bit20:43
hatchok now I need to get back to that20:49
hatchheh20:49
hatchahh found it20:51
hatchjcsackett how goes the battle with your bug?21:01
jcsacketthatch: having finally been able to get in a groove with my local env, good.21:05
jcsacketthatch: i ditched the loading circle b/c even in the "there's no service" thing you don't see it for long, and the indicator just doesn't slot into the browser at all.21:06
jcsacketthatch: once this lands, we can file a follow up to see about fixing the inspector view to do it right, with spinny circle, but i want this fixed sooner rather than later.21:06
hatchI don't even remember what the problem was anymore :D21:07
hatchohh now I remember21:07
hatch:)21:07
jcsacketthatch: that's alright, i'll ping you for review and it'll all come back to you. :)21:07
hatchlol21:07
bachatch: damn you're fast21:07
bacon the twitter21:07
hatchbac haha, I have Tweetdeck running with some choice streams :)21:08
baci keep forgetting dumb tumblr autotweets21:08
jcsacketthatch: unfortunately, bug 1331202 makes QA of this difficult.21:09
_mup_Bug #1331202: Incomplete Charm data causes artifacting in the GUI <juju-gui:New> <https://launchpad.net/bugs/1331202>21:09
jcsackettwe're going to want to schedule that bug for the next cycle, i think.21:09
hatchyeah...21:10
hatchagreed21:10
bacbbiab21:10
hatchI also filed a couple bugs last night which I found doing some ghost hacking21:10
jcsackettwhatever caused it happened in the last three days, b/c it wasn't doing that when i first investigated.21:10
hatchyay for dog fooding21:10
hatchjcsackett well there shouldn't be incomplete data in the store21:11
hatchbut the gui should handle it gracefully regardless21:11
jcsacketthatch: it treats nonexistant services as incomplete--produces the same artifact during the error condition i'm handling.21:15
hatchahh yeah it should definitely degrade nicefully 21:15
hatchMakyo these methods to fetch the deployer bar summary are a little funky, is there a reason why we don't loop through the changeset once to build the summary instead of for each record type?21:16
MakyoRushing it out the door for the demo?  They can go if there's a simpler way.21:17
hatchnah it's a follow-up I was jsut wondering if there was a major reason for this21:17
hatchcya21:17
hatchoh nm21:17
hatchI read that wrong lol21:17
MakyoI think it just piled up before ODS :)21:18
hatchok I'll create a follow-up card but in the mean time I'll follow the trend 21:18
Makyo285 failures, could be worse...22:33
MakyoWhoops, 335.  So yeah, I guess that is worse.22:33
hatchhaha22:37
hatchparty...on22:37
hatchMakyo I hope we don't conflict too much :O22:37
MakyoI don't think we did at all, but we'll see.22:38
hatchtravel agent put me in the wrong seats.....hold time at Air Canada.....45m-1:08  - I think I'll try later22:40
hatchlol22:40
hatchI really like places that do the call-you-back stuff22:40
bac_hatch do you see the post I made on G+ about quickstart?22:52
hatchnot yet22:53
hatchlooking22:53
bac_it was over an hour ago. both on my personal and work accts22:53
hatchshared!22:54
bac_annoying that it is so hidey.22:56
bac_thanks22:56
hatchadd a photo to your post so that G+ can pick it up, then it'll stand out more :)22:56
hatchyou can even (somehow) add a special photo which makes G+ show it as a huge image22:57
hatchnot sure how to do that - yet22:57
hatchhttps://plus.google.com/+JorgeCastro/posts/fc6dZSvjBLs vs https://plus.google.com/+JeffPihach/posts/8v5Xw4uf2FN 22:58
hatchthe first one has the special image22:58
huwshimiMorning23:06
hatchmorning huwshimi  hows it going?23:09
huwshimihatch: Good thanks, yourself?23:10
hatchgood good good23:10
huwshimigood23:12
huwshimihatch: So in my new dragenter function can I just do e.preventDefault(); to solve that problem?23:40
hatchhuwshimi yes but you could probably call the _ignore method....just to keep all the preventing stuff in the same place23:41
huwshimihatch: Ah yes23:41
huwshimihatch: Incidentally, it does seem to work fine at the moment.23:41
hatchreally? heh23:42
hatchprobably not cross browserly 23:42
huwshimiyeah, maybe not23:42

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