/srv/irclogs.ubuntu.com/2014/08/21/#juju-gui.txt

rick_h__huwshimi: reviewed00:03
rick_h__huwshimi: thanks!00:03
huwshimirick_h__: Thankyou00:03
* huwshimi looking00:03
rick_h__urulama: go to bed!01:48
rick_h__:P01:48
frankbanrogpeppe1: monring, I updated the branch with your (great) suggestions, would you like to take anothe rlook?08:24
frankbananother even08:24
rogpeppe1frankban: looking08:24
rogpeppe1frankban: morning too, BTW!08:24
frankbanthanks08:24
urulamaoh, hey there frankban, rogpeppe108:25
frankbanmorning urulama 08:25
rogpeppe1frankban, urulama: is your Go install from a PPA? I'd like to sort out the godeps problem that was mentioned on the list last night.08:25
rogpeppe1urulama: hiya!08:25
frankbanlooking08:26
urulamarogpeppe1: i can create new VM with go from PPA if needed08:26
rogpeppe1urulama: i'd be interested to see what happens when you do that, so yes please08:26
urulamarogpeppe1: made a note for the task, will do after lunch. if needed sooner, np08:26
frankbanrogpeppe1: 2:1.2.1-2ubuntu1 from trusty08:27
frankbanis there a ppa for 1.3>08:27
frankban?08:27
urulamafrankban: btw, Go on homebrew was bumped to 1.3.1 ...08:27
rogpeppe1frankban: have you run gofmt on your branch?08:29
frankbanrogpeppe1: sublime usually gofmt each time I save08:32
rogpeppe1frankban: hmm, i guess that comments mean that fields don't line up. interesting.08:32
frankbanrogpeppe1: comments in a struct definition? yeah 08:33
urulamafrankban: gosublime package?08:33
frankbanurulama: yes08:33
frankbanurulama: it's pretty cool, the only downside is that I started not caring about indentation/white spaces also when writing python or javascript, feeling sad each time I realize saving does not fix everything for me08:34
urulamafrankban: good to know, tnx. I use the same package.08:35
rogpeppe1frankban: re-reviewed. thanks for making the changes.09:26
frankbanrogpeppe1: thanks for the re-review09:54
rogpeppe1frankban: np09:54
urulamarogpeppe1, frankban: we can make a search query on a name for a charm over Charm.Meta.Name, but there is no such equivalent with bundles like Bundle.BundleData.Name, only through BaseURL10:25
urulamarogpeppe1, frankban: the question is - did i miss something with the bundle name?10:25
urulamarogpeppe1, frankban: or we actually don't have one?10:25
frankbanurulama: we don't have a bundle name as a separate concept, we have the bundle URL10:27
rogpeppe1urulama: frankly, having the charm name inside its metadata was a mistake10:28
rogpeppe1urulama: but that's what we've got, so we leave it there10:29
rogpeppe1urulama: the actual name of the charm is defined by its id, as frankban says10:29
urulamarogpeppe1: ok, so the proper way would be to not use Charm.Meta.name at all, but just IDs in both cases10:31
rogpeppe1urulama: yes10:31
urulamarogpeppe1: got my answer, tnx10:31
urulamarogpeppe1: so, if i upload a charm trusty/mywordpress, but that has a name theirWordpress, and when i search for myWordpress, the resulting charm would show the name theirWordpress in the GUI?10:33
rogpeppe1urulama: i'm not sure. it depends on the GUI logic10:34
rogpeppe1urulama: it *shouldn't* do10:34
urulamarogpeppe1: :D10:34
rogpeppe1urulama: one possibility is that we could refuse uploads of charms where the metadata name doesn't match the actual name10:35
urulamarogpeppe1: was thinking the same thing, but then, it would be better for not requiring the name at all and just add it in code10:35
urulamaif we need to have Charm.Meta.Name nonempty10:36
rogpeppe1urulama: i don't think we can fill in Charm.Meta.Name ourselves, unfortunately10:36
rogpeppe1urulama: because i think it's important to have the charm metadata be an exact reflection of what you get if you fetch the metadata.yaml file from the archive10:37
rogpeppe1urulama: and we can't change that10:37
rick_h__morning wheee10:47
* urulama lunches11:23
frankbanjrwren_: morning, how is it going with the quickstart branches?12:39
jrwren_frankban: on the back burner.12:50
frankbanjrwren_: ok12:50
rick_h__frankban: jrwren_ let's check with urulama if it's ok to make that the goal for friday though please. 12:53
rick_h__frankban: jrwren_ urulama actually feature freeze is today12:53
rick_h__frankban: jrwren_ urulama can we get those done up and a release out?12:53
rick_h__sorry, I should have looked at that earlier this week12:54
jrwren_rick_h__: quickstart?  sure.12:54
rick_h__urulama: is that ok with you if I steal jrwren_ to get that updated today and out?12:54
* urulama starts singing "Aaaaall by myseeeeeelf ...."12:55
jrwren_lol12:55
urulamarick_h__, jrwren_: sure, np12:55
rick_h__urulama: ty much12:55
urulamarick_h__: frankban as well, right. So with rogpeppe1 out tomorrow, i'll go back to QA part of the story. It's all fine.12:56
rick_h__urulama: it's all good, I think getting the search docs/etc together is +1 and it sounds like there's some back/forth going on12:57
rick_h__urulama: so I'd not change up plans really on that front?12:57
rick_h__or maybe I'm misunderstanding. 12:57
urulamarick_h__: i'm always an optimist that i'll have time to code :D12:58
rick_h__hah12:58
frankbanrick_h__: I was thinking about white listing: are we still ok with the plan given that 1) it does not prevent people to use the hooks/ dir as a mp3 repo and 2) in the future, we possibly have the same problem with resources?12:58
rick_h__frankban: let's punt on whitelisting until we know how the zip access holds up under pressure12:59
rick_h__frankban: I think that's the main driver we had before that we don't have now, disk space of uncompressed files laying around12:59
rick_h__frankban: we can reevaluate it as a whole closer to release time once we've used it more.13:00
frankbanrick_h__: so, IIUC no white listing for now?13:01
rick_h__frankban: correct13:01
frankbanrick_h__: +1 cool13:01
rick_h__you've convinved me :)13:01
rick_h__well you and urulama 13:01
frankban:-)13:02
jrwren_:( whitelist sounded so fun.13:08
* rick_h__ goes to start the morning now that morning calls are through. biab13:08
jrwren_I'm not sure what to do with this: https://code.launchpad.net/~evarlast/juju-quickstart/support-lxc-clone/+merge/22725013:18
jrwren_frankban: ^13:18
frankbanjrwren_: what's the problem?13:19
jrwren_nevermind. launchpad is foreign.13:19
jrwren_frankban: updated: https://code.launchpad.net/~evarlast/juju-quickstart/support-lxc-clone/+merge/227250  its marked approve already. I don't know what next step is.13:27
frankbanjrwren_: looking13:28
frankbanjrwren_: looking at https://juju.ubuntu.com/docs/config-LXC.html#fast-lxc-creation it seems that on certain platforms lxc-clone defaults to true13:30
jrwren_frankban: which is why I didn't specify originally.13:32
frankbanjrwren_: can't we just provide that information in quickstart? "LXC clone is enabled by default for Trusty and above, and disabled for earlier Ubuntu releases."13:34
jrwren_frankban: whatever you want.13:34
frankbanjrwren_: thank you13:34
frankbanjrwren_: other than that, you can go ahead and land it. we use lbox for quickstart, so I'd suggest to just "lbox propose" and then "lbox submit" the branch13:35
jrwren_ok13:36
bacrick_h__: i'm in 1:1 hangout14:00
rick_h__bac: doh sorry14:01
hatchlazyPower: hey do you have a link to the talk by Jamie Windsor?14:40
jcsackettjujugui: can someone confirm a bug for me? want to make sure i haven't got something weirdly local going on.14:46
hatchsure14:47
lazyPowerhatch: https://www.youtube.com/watch?v=hYt0E84kYUI14:47
jcsacketthatch: cool. on develop, with mv flag, deploy a service and immediately commit (so autodeploy).14:47
rick_h__it shouldn't do that now, kadams updated to stop it14:47
rick_h__jcsackett: we need to do the follow up to block the 'confirm' and add a UI there14:48
rick_h__jcsackett: I think that's what he's working on currently14:48
jcsackettrick_h__: ok, so we know we have an issue?14:48
hatchlazyPower:  cool thanks14:48
jcsackettb/c right now i see an uncommitted unit hang around.14:48
rick_h__jcsackett: sorry, I guess what's the issue? 14:48
rick_h__jcsackett: we know and kadams has a card in progress around completing work there14:49
hatchhaha I was waiting for the actual issue description (and it never game :'( )14:49
jcsackettrick_h__: ok, didn't realize summary and UX changes included that.14:49
rick_h__jcsackett: so yes, after trying it I see that it doesn't deploy 14:49
rick_h__jcsackett: so this is expected14:49
jcsackettrick_h__: dig.14:49
hatchohh 14:49
hatchyeah14:49
rick_h__jcsackett: it's WIP I guess14:49
hatchexpected14:49
hatch:)14:49
jcsackettrick_h__: all good, just making sure. it complicated QA locally for my branch, and then i saw it on develop, and then i was like "huh"?14:50
rick_h__jcsackett: understood14:50
rick_h__sorry for the distraction :)14:50
rick_h__distraction around your QA that is14:50
jcsackettrick_h__: all good. i'm just happy i didn't a) cause a bug and b) somehow screw up my develop branch.14:50
jcsackettsince, y'know, it wouldn't *remotely* be the first time i'd done that.14:51
rick_h__jujugui call in 10 please kanban14:51
hatchrick_h__:  can huws branch land?14:53
rick_h__hatch: I don't think so, the callback of the items seems broken?14:53
rick_h__hatch: I wanted to chat with you about that this morning, after the standup?:14:53
bacjujugui: regrets, i have another meeting and won't make the standup14:54
rick_h__bac: summary of your stuff, is you came, you saw, you debugged?14:54
rick_h__bac: what's the progress of the slack update?14:54
hatchrick_h__:  oh ok, it worked perfect after he made the changes from my review14:55
bacrick_h__: i have submitted it for review but gotten no bites yet.  will ping tom.14:55
rick_h__hatch: yea, I'm :/ on the way it's working but maybe I'm not following something14:55
hatchI'll pull it down and take a look14:55
rick_h__bac: rgr ty for the update14:55
rick_h__jujugui call in 1 or now, go go go14:59
rick_h__antdillon: around for call?15:00
rick_h__kadams54: there he is15:01
kadams54joining…15:01
kadams54Chrome apparently needs a restart15:02
hatchurulama: Parallels 10 is now available :)15:13
urulamahatch: already in use ;)15:14
hatchurulama:  haha nice - I'm skeptical, I'll wait a bit lol15:22
hatchalthough the upgrade from 8 to 9 was painless15:22
urulamajujugui: the inital search doc was shared with everyone. it's a live doc. jrwren_ and bac please take a look next week, as we might start working on search part15:22
* bac looks15:23
urulamahatch: at least 14.04 works out of the box now :D15:24
bacjujugui: azure sent out email about rolling reboots tomorrow and the next day.  may affect our CI. (not sure if everyone gets those mails.)15:24
urulamahatch: and it does fill slightly faster15:24
baclet me know if it goes wonky15:24
urulamahatch: feel even15:24
hatchurulama: the only real problem I'm having with 14.04 is that the audio is really quiet - also I can't quite get my keybindings correct but that's probably just me being crazy15:25
jrwren_frankban: updated: https://code.launchpad.net/~evarlast/juju-quickstart/which-juju/+merge/227238 & https://codereview.appspot.com/13277004315:25
frankbanjrwren_: looking15:26
urulamahatch: +1 on keybindings15:27
urulamajujgui: cu tomorrow15:40
=== urulama is now known as uru-afk
frankbanjrwren_: reviewed, https://codereview.appspot.com/132770043/ . please ping me if something is not clear. I am trying to proceed with the quickstart release so I did not have time to further investigate16:07
jrwren_frankban: thanks.16:08
frankbanjrwren_: also note that Rietveld allows you to reply in line and/or mark requested changes as done while working on the branch. This really helps keeping track of what's done16:08
frankbanjrwren_: then, when you are ready, you can just "lbox propose" again16:09
* rick_h__ takes lunch time16:21
hatchkadams54:  you have the worst internet :)16:43
kadams54?16:43
hatchmight be time to upgrade to dialup16:43
hatchyou ping timeout all the time16:44
kadams54Ah16:44
hatchlazyPower:  is there a juju plugin to allow `juju export` to work on lxc?16:49
hatchsay I want to run juju-gui in a local deployment but allow someone external on the net to access it (assuming they can access my primary machine)16:50
rick_h__hatch: you'd have to map that IP out to the internet16:50
hatchrick_h__:  yeah I was hoping for a juju plugin to do it for me - and assume that the outside could access a specified port on my machine16:51
rick_h__hatch: it has to do your router/etc so not sure how it would do that16:51
hatchrick_h__:  well assume my router will allow acces to my machine on port 8888 16:52
hatchI need to get the gui from the lxc instance to be available on my host machines port 888816:52
hatchthat should be scritable 16:52
frankbanguihelp, I just noticed we no longer support saucy, is it correct16:55
frankban?16:55
rick_h__frankban: yes16:56
hatchheh that will cause an issue for our dev vagrant image :)16:56
hatchwhich I'm pretty sure is saucy 16:56
rick_h__EOL July 17, 201416:56
frankbanrick_h__: are we supposed to QA quickstrat on utopic?16:56
rick_h__https://wiki.ubuntu.com/Releases16:56
frankbanquickstart even16:56
rick_h__frankban: hmm, we should as this is what we're heading to16:56
rick_h__frankban: and OSC16:56
rick_h__OSX16:56
rick_h__anyone running utopic? BradCrittenden?16:57
frankbanrick_h__: yeah, osx is on the list. we should change the QA instructions and readme for quickstart: we need to remove saucy and add utopic16:57
rick_h__frankban: adding a card for it now16:58
frankbanrick_h__: I prepared quickstart for release with two quick branches and packages for v1.4.2 are being built now. It's my EOD so I'd like to pass the release to someone else (in the case it's not ok for me to continue tomorrow morning).17:00
rick_h__frankban: rgr, are the release process docs there? I'll get BradCrittenden and jrwren_ to help track things up. 17:01
rick_h__frankban: thanks for getting that going and have a good night!17:01
frankbanrick_h__: what we need to do is 1) wait for the packages to be published (https://code.launchpad.net/~juju-gui/+archive/ubuntu/quickstart-beta/+packages)17:01
frankban2) QA on trusty and utopic (pre release QA is described in the HACKING file)17:02
frankban3) copy the PPA packages to juju/stable PPA 4) "make release" -> PyPI17:02
frankban5) make and test the homebrew release17:03
frankban6) add a release tag to trunk and have a drink17:03
frankbanrick_h__: ^^^17:03
rick_h__woot thanks frankban 17:03
lazyPowerhatch: its possible but not with teh default setup17:03
lazyPoweri wrote a blog post about this17:03
frankbanrick_h__: thank you17:04
lazyPowerhatch: http://blog.dasroot.net/making-juju-visible-on-your-lan/17:04
hatchlazyPower:  thanks 17:04
hatchkind of sucks17:04
hatch:)17:04
lazyPoweryes, yes it does17:04
hatchI.....hate......networking17:05
lazyPowerand that article was written against 12.04 - so it may have changed a bit. 17:05
lazyPowerI haven't revisited using LXC as my primary since i setup a VMAAS cluster17:05
lazyPowerbut the networking was similar.  1 ETH device for my communication, 1 ETH device as a bridge adapter for the VMAAS cluster. same basic concept being applied.17:05
* rogpeppe1 is done for the day.17:09
rogpeppe1happy weekends all, see y'all on tuesday!17:09
hatchjcsackett:  ok, lots of comments heh,17:10
hatchmind responding then I'll take another look 17:10
hatchlazyPower: couldn't a juju plugin just use sshuttle to forward the services ip and port to some supplied port on the host?17:11
hatchI know that's not probably how plugins are supposed to work, buuuuuut17:11
lazyPowerhatch: contributions are welcome ;)17:11
hatchdoes that sound like it's a valid solution though?17:11
lazyPoweri dont think sshuttle works like that17:12
lazyPowerit might, but i'm not terriblyf amiliar with using it as a proxy binding for services like that - i'd think you'd have better luck using UFW rulesets17:12
lazyPowerer, IPTables rather17:12
lazyPowerbut in any case, that gets hairy, and its not going to work 100% of the time, as networking is a complicated monkey to just assume a particular use case and run with.17:13
hatchhmm, I used to frequently use `ssh` to create a tunnel between an lxc instance and the host machine for testing17:13
hatchI never tried to get that out of the host machine though...17:13
lazyPowerthe only good way i can see to do it, is to create a bridge adapter and handle host routing that way.17:14
lazyPowerand its not bullet proof17:14
hatch*sigh* networking shouldn't be this hard heh17:15
BradCrittendenrick_h__: i have a utopic vm but don't run it daily.  can spin up for testing17:30
=== BradCrittenden is now known as bac
rick_h__bac: appreciate it, qa'ing the quickstart packages on trusty now17:36
rick_h__bac: even just a quick lxc setup for utopic might work17:36
bacrick_h__: perhaps, but i've got this vm just sitting here...17:39
rick_h__bac: ok cool17:39
jcsacketthatch: was grabbing lunch. looking at your comments now.17:42
kadams54hatch: https://github.com/kadams54/juju-gui/blob/develop/app/views/viewlets/service-overview.js#L46717:43
kadams54I'm guessing this is why my changeState event isn't bubbling up to browser.js?17:43
hatchkadams54:  looking17:43
hatchkadams54:  does it look like events fired from the deployer bar would hit the browser?17:44
hatchbrowser.js17:44
hatchI'm not familiar with where it's instantiated 17:44
kadams54The DeployerBarView is created by app.js17:44
kadams54Which would make it a peer to browser.js?17:45
hatchno 17:45
kadams54Not really sure how subapps work17:45
hatchbrowser.js is a totally different app 17:46
hatchlemme take alook17:46
jcsackettdo we actually need to maintain the subapps idea? are we ever going to have more, and do we gain much by having browser separated like that anymore?17:46
hatchjcsackett:  yeah - we don't have to rewrite it now :) in fact I was going to propose an idea where we have more 'subapps' but at a much higher level17:47
hatchhaven't thought it through yet17:47
rick_h__jcsackett: the reasons on that were a couple fold I can tell you about in our call later17:47
jcsackettrick_h__: cool.17:47
hatchkadams54:  in _renderDeployerBarView, after we instantiate the deployer bar add `this.deployerBar.addTarget(this);` 17:47
hatchthat 'might' make it work17:47
hatchat least give it a try17:48
kadams54k17:48
hatchif not I can guide you through a real fix17:48
hatchwhy is the machine-view-panel in /widgets? lol17:50
rick_h__because we agreed Y.Widget does not == Widget in a generic sense :P17:50
hatchif mv is a widget then......daayyyyymmmmnnnn17:51
rick_h__well, it didn't start out like this :P17:52
rick_h__you forget this stuff started 5 months ago 17:52
rick_h__it's bee a long long ride17:52
rick_h__patches welcome hah17:52
hatchlol17:53
hatchoh man bare metal tokens are so messed up17:54
hatchkadams54:  this is quite the bug you found....17:56
kadams54*sigh*17:56
hatchI might have to defer17:56
hatchit's actually an issue with how bare metal containers are rendered17:57
hatchit wasn't an issue before because we just cleared out the model17:57
hatchso it was all re-rendered17:57
rick_h__huh? I thought this was around 'auto place' in the summary?17:57
rick_h__what's it got to do with bare metal containers?17:57
hatchrick_h__:  I'm referring to the bug he found in my branch17:57
rick_h__oh17:57
hatchkadams54:  would you be ok if I punted on that bug, landed this and then fixed it as an immediate follow-up?  Because it's actually an existing condition17:59
kadams54Sure.17:59
hatchthanks I'll update the PR accordingly17:59
hatchcard created18:00
rick_h__kadams54: call time18:02
hatchMakyo:  because of this issue I created two cards at the bottom of Project 1 wrt the config changed stuff, feel free to take them as I will be pre-disposed for today likely18:04
hatchkadams54:  did that fix work?18:07
kadams54hatch: Yeah, it worked, thanks.18:10
hatchgreat18:11
hatchjcsackett:  replies to your replies :)18:14
hatchI'm going to grab some lunch now18:14
jcsacketthatch: so, i see it modifying the model to set the deleted flat, but otherwise just creating records.18:19
jcsacketthatch: i can maybe see an argument about the `delete unit.machine` bit, but not the whole method.18:21
* bac afk for a bit18:39
hatchback19:09
rick_h__go get him jcsackett!19:09
* jcsackett laughs19:10
jcsacketthatch: you up for a quick chat?19:11
rick_h__guihelp anyone know how to 'copy packages' to another ppa?19:11
hatchjcsackett:  so anywhere in the application which decides it wants to delete a machine has to then duplicate the code in that method elsewhere? 19:11
hatchyeah sure, just lemme get the dogs inside19:11
jcsacketthatch: cool.19:11
jcsacketthatch: standup room, when you're ready. :)19:12
hatchomw19:12
jcsacketti will confess, when rick_h__ jumped in i was worried i was about to be huwed. :p19:26
hatchhahahahaha19:27
hatchhuwed19:27
rick_h__lol19:27
hatchthat's so a thing new19:27
hatchnow*19:27
* kadams54 feels clueless19:27
kadams54What's a "huwing"?19:27
hatchwhen you do your branch19:28
hatchget it reviewed and make changes19:28
hatchthen rick_h__ comes in and changes the implementation 19:28
hatchso maybe "huwing" should be rewriting code19:29
hatchlol19:29
rick_h__:P19:30
hatchkadams54: is your card in review the one in coding?19:30
rick_h__I can't help it I have opinions that occassionly are useful19:30
kadams54Not any more :-)19:30
hatchhaha thx19:30
kadams54guihelp: Looking for reviews on https://github.com/juju/juju-gui/pull/50619:31
hatchkadams54:  doing it19:31
kadams54Awesome19:31
jcsackettkadams54: i'll be your second review if i can put you as the second for mine (once i get done with being reviewed by hatch)19:40
kadams54jcsackett: Deal.19:40
jcsackettawesome. :)19:40
hatchkadams54:  done - one comment you can look into 19:46
jcsacketthatch: so, unhappy moment--if i remove the units in the ECS, i have no way of getting at them to add them to unplaced units in the machine view. :/19:48
hatchjcsackett:  shouldn't you be able to call the 'render unplaced units' method and it'll update the list?19:49
jcsackett...maybe.19:50
jcsacketti'm dubious, but that might work.19:50
hatch_renderUnplacedUnits is what it's called19:52
hatchit might need to be upgraded to check if it's already rendered it though...19:53
hatchagile......19:53
hatchlol19:53
* jcsackett groans19:59
rick_h__ruh roh19:59
jcsackettwell, it seems to need some sort of magic to let a unittoken be back in it once it's palced.19:59
jcsackettso, the problem is that once you assign the unit, it's no longer in this.get(unitTokens), which is what the renderUnplaced thing uses.20:01
hatchhttp://i.dailymail.co.uk/i/pix/2012/10/27/article-0-15B4ABD0000005DC-0_634x376.jpg20:02
jcsackett /me laughs20:03
hatchlol love that pic20:04
jcsackettyeah, when we place a unit, we remove it from unit tokens; so i'm going to need to a) figure out why delete unit.machine isn't picked up as a change to a unit for that event, and b, wire that handler up to add it back to the unitTokens.20:04
jcsackettso...this is probably not landing today.20:04
hatchjcsackett:  well....20:04
hatchso maybe what you do is leave it the way you had it, with a bug about this functionality - because that should really be rendering the unplaced units by parsing the model20:05
rick_h__jcsackett: sounds like you're on a good path there.20:05
jcsackettrick_h__: the "not landing today" path or the "fuck it, land it this way with a bug" path? :p20:05
rick_h__if you need to pause a card to fix something first there's nothing wrong with that20:05
rick_h__I sometimes think we need to do that more than the 'land with issues and I pinky swear to make it better next' thing20:06
jcsackettrick_h__: i don't know that this is really fixing something else, per se. i think its within scope for this card.20:06
jcsackettas long as we're all comfortable with this card hanging out for another day.20:06
jcsackett:p20:06
rick_h__jcsackett: right, I meant you're on the right path with the event wiring20:06
jcsacketti am, to be clear, b/c this does seem better.20:06
jcsackettyeah.20:06
rick_h__jcsackett: +1, good fix using the right model ftw20:06
jcsackettbut, i need to not look at it for a bit. :p kadams54, your PR ready for a second review?20:06
kadams54jcsackett: yup, have at it. And yours?20:07
rick_h__jcsackett: it's close to EOD, put it away for the night20:07
hatchjcsackett:  ok awesome....thanks20:07
rick_h__speaking of, going to step away until calls start back up in an hour20:08
rick_h__biab20:08
jcsackettkadams54: tomorrow for mine. :p20:08
hatchok now what was I doing before all these reviews....20:08
hatchbesides jammin to Hardwell 20:09
jcsacketthatch: it's always fun to regain state after reviews, isn't it? :p20:09
bacrick_h__: i'm doing the quickstart QA on utopic now.  sorry for the delay20:09
hatchjcsackett:  haha yep20:09
rick_h__bac: ok, let me know if you find anything asap.20:09
hatchYou know you live in the prairies when this tweet is a thing....... https://twitter.com/SKGov/status/50254555638923673720:25
kadams54I've been waiting for that crop report!20:28
kadams54Good to hear the farmers are busy desiccating.20:29
hatchhaha20:36
hatchthere is actually a 'crop report' on one of the radio stations as a show20:36
hatchI mean.....99% of the land here that's not covered by trees is farm land sooooo it stands to reason it would be an important part of the economy20:36
hatchstill comical 20:36
hatchI think I'm going to become an ophthalmologist - one is in the news for billing $2.2M last year......20:40
hatchaverage is $1.02M/year20:42
jrwren_hatch: what kind of crops over there?20:46
hatchwhat's your poison? ;)20:46
hatchmostly grains20:46
jrwren_hatch: barley. Can you get me a deal?20:47
hatchlol maybe20:47
hatchbarley is hardcore20:47
jrwren_I'm hoping for like $20/bushel20:47
hatchumm I think it's like $3 right now.....20:48
hatchso yeah....$20...deal!20:48
jrwren_$3/bushel?!?!20:48
hatchOct Barley: unchanged at $2.683 USD or $2.975 CDN20:49
jrwren_zomg, I should so buy.20:49
jrwren_do you know what they charge at the brew store?20:50
jrwren_its like $50!!!20:50
hatchhaha, bagging it into a fancy bag is expensive! 20:50
jrwren_so is malting20:50
hatchprobably not $47 expensive mind you20:51
hatchyou can buy a metric tonne for $140 lol20:52
jrwren_this is going to be me, but with barley: http://thedailywtf.com/Articles/Special-Delivery.aspx20:53
hatchhahaha20:53
hatch instead of virtually trading 28,000 tons of coal, Brad had somehow ended up with 28,000 tons of real coal.20:55
hatchhahahahaha20:55
bacthat story does look fishy21:07
bacBrad may be a doofus, but "1".lower() != "true"21:08
bacwhat am i missing?21:08
jcsackettbac: the story was debunked a few years ago along with a bunch of other dailywtf stories.21:11
bacjcsackett: but but but, they could've at least made it internally consistent21:11
jcsackettbac, actually, i think it is.21:11
bachow's that?21:11
bacso the confirm for physical delivery came back as a '1'21:12
jcsackettright, and that .lower() doesn't equal "true".21:12
jcsackettso if you're setting a flag to stop, you'll se false, and go "ok, no physical delivery, we're good"21:13
jcsackettright?21:13
bacright, so on the originator's side physicalDelivery is False, so ... so they think it is consistent.  gotcha21:13
jcsacketti mean, it's still a load of crap, but it's consistent crap. :p21:14
bac:)21:14
bacand, once again, brad is the butt of the joke.  it all started with "Rocky Horror"21:14
=== uru-afk is now known as urulama
urulamarick_h__: ping?21:24
rick_h__urulama: pong21:25
urulamarick_h__: hey there21:25
bacrick_h__: i'm still working my way through the QA.  everything is fine so far.21:30
rick_h__bac: ty21:31
bacrick_h__: i declare quickstar 1.4.2 yar on utopic21:52
rick_h__bac: cool because I got it released and smoser uploaded it a while ago. Thanks for safety checking it out21:54
rick_h__I really appreciate the check21:54
bacnp21:54
* rick_h__ needs to start to add these release milestones to his calendar21:54
huwshimiMorning22:22
rick_h__morning huwshimi 22:27
hatchmorning huwshimi22:30
huwshimihatch: Call?22:33
huwshimirick_h__, hatch: How do I pass the correct 'this' to a callback? http://paste.ubuntu.com/8110037/23:44
hatchhuwshimi:  you want the context from the token?23:45
hatchmyFunction.bind(this)23:45
huwshimihatch: Yeah23:45
hatchfunction() {}.bind(this)23:45
huwshimihatch: So this would work? this._handleMoveIconClick.bind(this)23:46
hatchthat will create a bound version of that function and return it23:46
hatchyep23:46
hatchif you want to stub that method however for tests23:46
hatchyou'll have to do it BEFORE that bind is called23:46
hatchbecause bind() actually returns another method which is bound to a specific context23:46
huwshimihatch: Brilliant, that worked :)23:46
hatchif that makes sense....23:46
hatch:)23:46
huwshimimakes sense23:47

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