/srv/irclogs.ubuntu.com/2014/10/02/#juju-gui.txt

jcw4rick_h_: back by any chance?01:13
rick_h_jcw4: what's up?01:17
jcw4quick question01:18
jcw4(and if not it can wait)01:18
rick_h_shoot01:18
jcw4running juju-gui against local dev version of juju01:18
jcw4HACKING.rst is bit-rotted01:18
rick_h_jc:(01:18
jcw4is there a quick howto somewhere?01:18
rick_h_jcw4: that's it. If it's bitrotten we need to fix it01:19
jcw4I have juju local running01:19
jcw4the instructions aren't clear how "make devel" hooks up with the local env01:19
rick_h_jcw4: oh it doesn't :)01:19
jcw4it talks about api-port but doesn't say where exactly01:19
rick_h_jcw4: it uses a fake backend (all JS) in devel/etc01:19
jcw4have to deploy a charm?01:19
rick_h_jcw4: rgr01:19
rick_h_best way to go01:20
jcw4so any dev changes on the gui side have to be packaged and redeployed?01:20
jcw4i.e. no runtime code changes?01:20
jcw4on the juju-gui side01:20
rick_h_jcw4: so what we do is there's a config on the charm for 'juju-gui-source' 01:20
jcw4okay01:20
rick_h_that takes a branch name you can set01:20
rick_h_and it'll download/load from github/etc01:20
rick_h_jcw4: or you can use a tarball at a url, etc01:20
jcw4i see01:21
rick_h_jcw4: so normally we dev against our fake provider, and then when we QA on live env we'll deploy, set the source to our branch, and test stuff out01:21
jcw4hmm, in this case I want to play on the -core side and tweak on the -gui side simultaneously...01:21
jcw4okay01:21
jcw4that helps a lot01:21
rick_h_jcw4: yea, will be interesting. 01:21
jcw4if I can improve the docs I will01:22
rick_h_jcw4: there are configs on the charm to not use the minimized JS and such01:22
jcw4oh01:22
rick_h_but it'll take some poking to get at the source in the build dir/etc01:22
jcw4okay I'll be sure to check that too01:22
rick_h_jcw4: and the normal "make devel" won't work since you won't have a bunch of dev tools01:22
rick_h_so it's kind of hacky01:22
jcw4hehe01:22
jcw4no worries01:22
rick_h_but it's enough to get decent tracebacks/etc just to not have it minified01:22
jcw4now that I know it's not supposed to 'just work' I'll get my boots on01:22
rick_h_jcw4: but yea, config optinos for the source, for enabling the console (console.log/error), and for debug mode should come in handy01:23
rick_h_(config options on the charm)01:23
jcw4in summary - I'll build the charm with options to keep source un minified01:23
jcw4deploy to local juju01:23
jcw4and then hack around in the charm dir to tweak code01:23
rick_h_deploy to local juju, then once it's up change the config options (or set them on deploy using a config.yaml file)01:24
jcw4k01:24
rick_h_just like a normal charm, they're just set via "juju set ..."01:24
jcw4k 01:24
rick_h_see https://jujucharms.com/trusty/juju-gui-9/#configuration01:24
jcw4I saw that but now it makes more sense01:25
jcw4tx rick_h_ 01:25
rick_h_jcw4: cool, np01:25
rick_h_there, get annoyed, write some code :P https://github.com/juju/juju-gui/pull/60101:48
rick_h_huwshimi: if you're up for a GUI review ^ appreciate it 01:48
rick_h_huwshimi: but if you're slammed for luca, that comes first01:48
huwshimirick_h_: OK, I'll take a look if I get a chance01:49
rick_h_and booya, managers coding ftw101:49
rick_h_err !01:49
huwshimirick_h_: A couple of notes about the styles, I'll do a proper review later if I don't run out of time today.01:54
rick_h_huwshimi: thanks01:55
rick_h_huwshimi: I knew you'd call me out on that :P01:55
huwshimirick_h_: There's no excuse :)01:55
rick_h_huwshimi: but but but ... yea01:56
huwshimihehe01:56
rick_h_ok, time to head home night all 01:59
huwshimirick_h_: Night02:00
hatchrick_h_: jcw4 you can hack on the deployed charms code by ssh'ing into the juju-gui instance 03:53
jcw4rick_h_: just figured that out a little bit ago04:04
jcw4thanks!04:04
=== uru_ is now known as urulama
huwshimiLuca__: Hey are you around yet?07:41
=== rogpeppe is now known as rogpeppe1
=== rogpeppe1 is now known as rogpeppe
rick_h_morning10:58
frankbanmorning rick_h_: do you need a review for your card in maintenance? link?11:32
rick_h_frankban: yes please11:35
rick_h_antdillon: what are you doing around here?11:42
antdillonrick_h_, ha couldnt stay away11:42
rick_h_antdillon: I'm going to tell your boss to send you home11:43
rick_h_antdillon: did you have your kid?11:43
antdillonrick_h_, yeah little girl Chloe11:44
rick_h_antdillon: woot! congrats!11:44
antdillonrick_h_, thank you very much11:45
antdillonrick_h_, dont tell my boss ...11:45
rick_h_frankban: can we join our call early please? e.g. now? I want to squeeze it in before I leave for the doc please11:52
frankbanrick_h_: now is ok11:52
rick_h_ty11:52
rick_h_jujugui heading off to the morning doc apt. I'll be back in a bit. 12:07
* frankban lunches12:17
* rick_h_ is back13:32
rick_h_jujugui can I get a second review of https://github.com/juju/juju-gui/pull/601 please and I'd appreciate a second QA as I'm unable to dupe the issue frankban had and curious if others can replicate13:34
frankbanrick_h_: I had a second QA comment, fixing my first one FYI13:34
rick_h_frankban: oh oh oh I see now. You're right, they're not checked based on current value on load. That's bad on my part I'll update in this branch13:35
rick_h_should be a simple read/set checked=checked thing in the template13:35
* rick_h_ misread it at first13:35
frankbanrick_h_: cool thanks13:35
rick_h_jcsackett: around to chat on the feature flag branch?13:45
jcsackettrick_h_: rick_h_ ping you in a few? trying to work my way through a mountain of PRs13:46
rick_h_jcsackett: rgr13:46
jcsackettall small, should be done in just a bit.13:46
rick_h_all good, just wnat to make sure I unblock as soon as I can13:46
rick_h_unblock you that is 13:46
=== kadams54_ is now known as kadams54
jcsackettrick_h_: that took a bit longer than i thought. you free?14:13
rick_h_jcsackett: otp atm14:13
jcsackettnatch. :p14:13
jcsackettwell, i'll start reviewing your branch, then.14:13
rick_h_jcsackett: free now14:18
rick_h_jcsackett: get in while the 10min last :)14:18
jcsackettrick_h_: coming to the standup hangout now.14:18
hatchrunning to grab a coffee14:25
rick_h_jujugui call in 4 kanban please14:57
=== kadams54_ is now known as kadams54
rick_h_frankban: call15:00
bacfrankban: did you get a response from homebrew?  after the initial acceptance updates seemed to be automatic in the past.15:04
frankbanbac: I know, it's like this: https://github.com/Homebrew/homebrew/pull/3286615:05
hatchrick_h_: jcsackett fixed the manual provider issue - just need tests now15:29
hatchDO is so darn fast compared to ec2 lol15:33
jcsacketthatch: whoo!15:45
hatch*sigh* the go.js test file is over 2k lines15:56
luca__rick_h_: I’ve assigned a card to Huw to build the new footer design, is that ok?16:13
hatchjujugui I need a review and qa for a manual provider bug16:20
hatchhttps://github.com/juju/juju-gui/pull/60216:20
hatchwe have a new footer design? luca__ what did I tell you about changing the design?? HUH HUH?? :P16:21
hatchrick_h_: next card of choice for me?16:24
* hatch started a new js framework last night16:27
hatchhmm I sure wish apple had a 4k thunderbolt display - having a hub in the monitor would sure be nice!16:33
hatchfrankban: thanks for the review16:38
hatchfrankban: did you qa?16:38
frankbanhatch: I am bootstrapping a manual env for the first time16:38
hatchOoo :)16:38
hatchon Digital Ocean?16:38
frankbanno, on a vmware machine locally ;-)16:39
hatchoh haha I never even thought of doing that :)16:40
frankbanhatch: it worked pretty fast16:40
frankbannow switching to your branch16:40
rick_h_hatch: can you see if kadams54 or Makyo need help. Priority is to look at getting those up for landing so release is possible tomorrow16:42
rick_h_hmm, luca is gone16:42
kadams54hatch: I'll need QA shortly ;-)16:42
hatchkadams54: lemme know! 16:42
kadams54hatch: code changes are minor, so it's mostly QA.16:43
hatchI'm a boss at qa16:43
hatchI can break anything16:43
hatch:P16:43
hatchjujugui has anyone else been able to add the calendar in Sarah's email? 16:45
kadams54hatch: yeah, just did it.16:47
hatchhmm - you used the url and went to Add Url?16:47
rick_h_hatch: I've got it from before and can't share it it seems16:47
hatchit says that the url is invalid for me16:47
Makyo I got it by pasting the email address in 'Add a coworker's calendar"16:48
hatchahhhh16:48
hatchthere it goes16:48
hatchthanks peeps16:48
kadams54http://cl.ly/image/132H0Y3R0l3V16:48
hatchkadams54: haha that's cool how did you do that?16:49
kadams54cloud.app16:49
kadams54https://www.getcloudapp.com16:49
kadams54I've started sending animated GIF screenshares instead of typing up 6 page tech support e-mails to friends and family. Very handy.16:50
hatchdarn osx only16:50
hatchwe need some of thse for Ubuntu16:51
frankbanhatch: done16:54
hatchawesome thanks16:54
hatchkadams54: do you know if this cloudapp will work in fullscreen vm? So I could use it from osx in my ubuntu vm?17:02
kadams54I don't, sorry.17:03
hatchI have screencloud for Ubuntu but it doesn't do 'video'17:03
hatchwhich would be helpful for some things17:03
kadams54Yeah, definitely. Cloudapp just added the video feature in the last release; I was pretty excited to see it.17:04
Makyojujugui reviews/qa please! https://github.com/juju/juju-gui/pull/60317:19
hatchI'm on it17:19
hatchBOOM, just like that17:19
MakyoThanks.17:19
hatchbut now you have to review mine :P17:19
MakyoFiiiiine17:19
kadams54hatch: you need a second in addition to Makyo?17:21
hatchnope frankban got the review and qa17:21
kadams54I'm running make check right now for my PR :-)17:22
=== urulama is now known as urulama-afk
hatchMakyo: I think you missed the 'clear all' functionality17:25
hatchwait nm'17:25
hatchI am bling17:25
hatchblind even17:25
MakyoOh, just forgot that in the QA instructions17:25
hatchcode looks good - qa'ing17:27
hatchMakyo: now that we have this functionality it's quite odd that you can't expose in the ghost inspector17:30
hatchthoughts?17:30
hatchand done17:32
kadams54hatch: https://github.com/juju/juju-gui/pull/604 is ready for QA17:32
kadams54;-)17:32
Makyohatch, it's a little weird, yeah.  Was going to make that a followup.17:32
Makyo...which you said.  So yeah.17:33
hatch:)17:33
hatchman doing work like this is such a relief post MV release :)17:34
hatchfrequent releases ftw17:34
kadams54Continuous deployment, baby.17:34
hatchlol I wouldn't quite go that far :) we do ship a packaged app afterall17:34
hatchkadams54: lol got to love the branches where the test loc's FAR exceed the code changes17:37
kadams54actual code < test code < time spent on manual QA17:37
kadams54Whee!17:37
kadams54Plus I had a comparison error yesterday that manifested by breaking other, unrelated parts of the code, even though the QA stuff worked fine. Also whee!17:38
hatchkadams54: just a request for the future - if you move code could you add a reviewer comment to point that out so reviewers don't have to stare at the diff for changes when none happened17:40
kadams54Ah yeah, good point, will do.17:41
kadams54The tricky thing is that some of that did change, so I'll annotate with comments17:41
hatchahh yes plz do :)17:41
kadams54Some tests moved into a describe block. New tests were also added to that same describe block.17:41
kadams54OK, new tests annotated. The rest of the test deltas is just reorg.17:45
hatchthanks17:48
Makyojujugui stepping out over lunch to fedex some things.  Back after.17:59
rick_h_jujugui kind of cool, have blog stats from google in image form on the mv blog post http://uploads.mitechie.com/lp/mv-blog-stats.png 18:21
bacrick_h_: it was so good, 700 people came back to read it again118:23
bacs/1/!/18:23
rick_h_lol18:23
bacor that was you 700 times18:23
rick_h_well you know me and my refresh button issues 18:23
hatchman google maps 'create map' is missing a huge feature "add points from starred" :/18:41
benjihi all18:52
benjihatch: I have a YUI question for you.  We have an issue on the Landscape project involving clicking on links handled by event handlers before all of the JS has run, causing either nothing to happen or the wrong thing to happen.  Is there a standard YUI approach to that problem?18:53
hatchwb benji18:57
hatchlong time :)18:57
benji:)18:57
hatchno there isn't a standard problem but there is a common technique to solve it though18:57
hatchthe first step of course is to try and reduce that time :)18:58
hatchthe second is a raw js handler which catches all clicks on A's and then queues them up18:58
hatchonce YUI comes online it re-fires them 18:58
hatchs/problem/solution18:59
hatchbenji: but with that said you should be able to get a running YUI instance within a second19:00
hatchhopefully less than 500ms tbh19:00
hatchso you might have an architecture problem there19:00
benjihatch: good stuff, thanks!19:00
hatchanytime19:01
rick_h_benji: using a combo loader?19:01
benjirick_h_: I honestly don't know.  I've tried to stay away from the JS stuff for a while, but I think I've built my tolerance back up enough to look at it again. ;)19:02
hatchlol19:02
hatchbenji: really try to reduce the load time to be a non-issue first - implementing this can be a hassle. I've done it before :)19:04
hatchgranted that was 4ish years ago hah19:04
kadams54jujugui: heading out, should be back online in two hours.19:05
benjihatch: yeah, I'm a little surprised that it can take as long as it does for everything to get into a working state19:05
hatchyou may have a big blocker in there somewhere19:06
hatchahh machine view is so bawler 19:21
* hatch feels like he has gold teeth and a rap album when he uses it19:21
bachatch: you don't own any rap albums?19:24
hatchmaybe some but none with my name on them19:25
bachas anyone gotten the 'cloud track' calendar to show up on their freedom-hating phone? it eludes me.19:25
hatchbac when you go to "Calendars to display' do you have a button 'Calendars to sync' ?19:28
hatchif so - go there and you can add it to the list19:28
bodie_how do I get my version of the gui charm up on my local provider?  I'm also going to tweak a simple charm to play with a new backend feature.  I assume this info is in HACKING, but I must be missing something19:30
bodie_there's a "local" charmstore you can use, right?19:30
hatchbodie_: use the --repository flag when deploying via the CLI19:31
hatchhttps://juju.ubuntu.com/docs/charms-deploying.html#deploying-from-a-local-repository19:31
bodie_sweet, thanks19:31
jcsackettrick_h_: updates to the kill-mv-env branch sent.19:31
rick_h_jcsackett: rgr19:31
hatchonce you have the GUI up and running you can then drop local charm zip files onto the GUI canvas bodie_19:31
bodie_oh, that's handy.  thanks hatch19:32
hatchno problem19:32
hatchwhat are you hacking on?19:32
bodie_actions19:32
hatchahh cool - I want those19:33
hatch:)  19:33
bodie_what about to hack on the gui charm itself?  I guess you'd use upgrade?19:33
bodie_yeah, we just got the worker / unit bits nailed down finally, and we have the skeleton api landed too19:33
hatchI'm not sure about upgrading the gui charm via the gui :) never tried that19:33
hatchshould work though...heh19:33
bodie_what could possibly go wrong?  :P19:33
bodie_I was thinking more about using the CLI for that, though19:34
hatchsure thing - if you want to hack on the actual GUI code in a live env you can do that too19:35
hatchwithout having to modify the charm19:35
bodie_ah, juju upgrade-charm --repository19:35
bodie_oh19:35
bodie_hm19:35
bodie_I guess that's what HACKING covers, right19:35
bodie_something about that doc just didn't make things clear to me19:36
hatchyeah run `juju help upgrade-charm` for detailed instructions19:36
bodie_ok.  thanks again for the pointers19:37
hatchnp I'll be here for a while yet19:37
hatchif you have any more19:37
hatchhmm in qa'ing kyles branch I'm having an issue with the gui charm not upgrading the ws19:45
jcsacketthatch: lemme know if you want me to test your branch on my env.20:05
* rick_h_ steps out to throttle someone at a t-mobile store20:14
jcsackettrick_h_: when you're back from throttling if you can look at my mv-flag branch so i can move it to landing. :)20:18
hatchjcsackett: hey it has been merged20:48
hatchjcsackett: it would be awesome if you could qa kyles branch with me20:49
hatchhttps://bugs.launchpad.net/juju-gui/+bug/1376901 I found this bug while doing it so I'd like another set of eyes20:49
mupBug #1376901: Destroying a machine with a container on it causes machine view to break <juju-gui:New> <https://launchpad.net/bugs/1376901>20:49
hatchMakyo: https://bugs.launchpad.net/juju-gui/+bug/1375251 requires you to deploy the env using the DO plugin20:54
mupBug #1375251: Juju gui malforms display with missing x/y coordinates on existing env w/ charm upgrade <juju-gui:Triaged> <https://launchpad.net/bugs/1375251>20:54
Makyohatch, yeah, I was just hunting for that.  Have a link handy?20:55
hatchMakyo: from lazyPower http://blog.dasroot.net/juju-digital-ocean-awesome/20:55
hatchawesome tut20:55
* lazyPower hattips20:56
lazyPowerthanks for the feedback20:56
lazyPowerfyi, thats also in the documentation now20:56
lazyPowerjuju.ubuntu.com/docs/provider-digitalocean.html20:56
hatchoh cool20:56
hatchthe video was really awesome but too bad the sound wasn't quite there20:56
lazyPowerhttps://juju.ubuntu.com/docs/config-digitalocean.html   rather20:56
kadams54guihelp: Anyone around to provide a second QA and review on https://github.com/juju/juju-gui/pull/604 ?21:07
hatchyeah definitely needs another qa21:07
kadams54We need more West Coasters ;-)21:14
rick_h_kadams54: will look in a bit21:19
rick_h_jcsackett: will look21:20
jcsacketthatch: i can take a look. be a bit to spin his branch up on ec2.21:22
hatchkadams54: haha we used to 21:23
kadams54jcsackett, rick_h_: thanks21:23
kadams54Switching locations, will be back shortly.21:29
jcsacketthatch: so, i'm not going to be able to qa it. amazon is saying it has insufficient capacity. :p21:38
hatchlol21:38
hatchuse DO21:38
rick_h_jcsackett: maas time!21:41
rick_h_:p21:41
rick_h_jcsackett: :+1: ty!21:43
jcsackettrick_h_: awesome, thanks.22:05
jcsackettrick_h_: the bug is an ec2 bug, alas.22:05
rick_h_jcsackett: rgr22:06
hatchmy internet is super flaky mind if I skip on the aus call rick_h_?22:20
rick_h_hatch: np22:21
rick_h_hatch: thanks for the updates today to get ready for release <322:22
hatchyeah this fix release is going to be really nice22:25
hatchespecially if we can keep up doing releases like this22:25
hatchreally nice to do22:25
rick_h_hopefully22:26
rick_h_though new feature starts next week so it'll slow down a bit, but smaller features and more releases is good, out of MV's shadow22:27
huwshimiMorning22:32
huwshimirick_h_ hatch: Sorry I'm late22:33
rick_h_huwshimi: all good22:33
rick_h_huwshimi: joining late myself22:33
bachi huwshimi23:04
bacurulama-afk: i've submitted a pull request to your CScharmTools branch with working ngram searches.23:04
huwshimibac: Hey!23:05
bachuwshimi: have you not left yet?  better hurry or you'll miss opening cocktails.23:17
huwshimibac: haha, yeah, it's a bit like that...23:22

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