/srv/irclogs.ubuntu.com/2014/01/28/#juju-gui.txt

=== _mup__ is now known as _mup_
marcoceppigary_poster: ping13:45
gary_postermarcoceppi: pong13:46
marcoceppigary_poster: hey, we're sprinting and using maas + juju + gui13:46
marcoceppigary_poster: when deploying a simple mysql + mediawiki bundle relations aren't being created13:46
marcoceppirepeated on multiple pieces of hardware for multiple bundles for multiple deployments13:46
marcoceppiwhat do you need from me for debugging13:46
gary_posterand works on ec2 I assume13:47
marcoceppinot tested against ec213:47
marcoceppijust against various heaps of hardware, but the physical add-relation command works from the gui and juju13:47
marcoceppion juju 1.17.013:47
gary_posterk, thank you.  frankban: can you direct marcoceppi to the bundle deploy log in the charm?  (marcoceppi, looking for docs if he's not around)13:48
rick_h_using the bundle the relations are added by the deployer and not the gui. We'd have to trace/look at the websocket log13:48
gary_posterwe have good logs on charm13:48
rick_h_heh or that sounds even better13:48
frankbangary_poster, marcoceppi: gui server logs can be found in /var/log/upstart/guiserver.log13:49
gary_posterlogs to send: /var/lib/juju/units/[NAME OF UNIT]  /var/log/upstart/guiserver.log13:50
gary_posterthanks frankban, and HACKING doc was perfect :-)13:50
gary_postermarcoceppi: also, if you are curious https://<juju-gui-url>/gui-server-info might be interesting13:50
frankbangary_poster: yes: https://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/view/head:/HACKING.md#L22213:51
frankbanmarcoceppi: ^^^13:51
gary_posterfrankban: is it worth mentioning builtin-server-logging=debug also?13:51
frankbangary_poster: it is already mentioned in the HACKING file13:52
frankbanline 23313:52
gary_posterfrankban: I know, that's where I was stealing it.  I meant do you think it would help to ask marco to change that config for our debugging13:52
marcoceppigary_poster: frankban http://paste.ubuntu.com/6832109/13:53
marcoceppinothing really interesting in guiserver.log13:54
frankbangary_poster: oic. it is useful to inspect the Websocket requests/response from the GUI to the guiserver13:54
frankbanmarcoceppi, gary_poster: lines 154-157 seem interesting13:55
rick_h_frankban: +1 seems Juju didn't like something in the data 13:56
marcoceppiOH I KNOW THIS BUG13:56
rick_h_marcoceppi: have the bundle file up? does it have any constraints in it?13:56
marcoceppiwe need 1.17.113:56
marcoceppiit's the statewatcher bug13:56
frankbanmarcoceppi, gary_poster: line 158 also means you should see an error notification in the GUI13:56
marcoceppiin 1.17.1 for jujuclinet/deployer13:56
gary_posterInteresting in a MY GOSH COULD WE PLEASE HAVE SOME DECENT INFO FROM THE DEPLOYER TO LOG ;-)13:56
frankban:-)13:56
rick_h_I thought we patched things to not send an "EnvError" like that? Didn't bac add that?13:58
frankbanmarcoceppi: yes I remember Kapil mentioned the watcher unexpectedly closed bug13:58
frankbanrick_h_: 'Error': u'state watcher was stopped' looks like a good message to me13:59
gary_posteryeah that's the improved version, IIRC, rick_h_.  we used to send a stringified json thing as the message,  Now we send 'state watcher was stopped'13:59
rick_h_frankban: oh, duh. that's the 'type'13:59
* rick_h_ hits more coffee13:59
gary_poster:-)13:59
rick_h_yay for logs go frankban 14:00
gary_poster:-)14:00
frankban:-)14:00
rick_h_jujugui have to run up to the kids day care for a few min. afk. 14:15
gary_posterack14:15
hazmatfrankban, thats resolved14:15
hazmatfrankban, in trunk14:15
frankbanmarcoceppi: ^^^14:15
frankbanthanks hazmat 14:15
hazmatfrankban, the py jujuclient isn't async, so if it was used for a little while, it wouldnt be able to process pings, rogpeppe2 yanked ping timeouts on the server.14:16
marcoceppifrankban: how do I get trunk?14:16
hazmatmarcoceppi, go get launchpad.net/juju-core ?14:16
hazmat-u -v14:16
marcoceppihazmat: oh, I know that, I thought he meant trunk of juju-gui14:16
marcoceppiwe're started compiling14:16
frankbancool14:17
hazmatfrankban, speaking of interesting info to log, any progress on feedback/validation errors branch?14:17
gary_postermarcoceppi: I don't *think* you want trunk of gui?14:17
hazmatfrankban, i'd like to push a new deployer release before friday, if not it can make the release after.14:18
hazmatlots of goodies got merged last week14:18
frankbanhazmat: working on the server side putcharm API now, improving feedback is likely to be my next card14:18
marcoceppiI was confusing what frankban was suggesting14:19
hazmatfrankban, fwiw, i pushed a new jujuclient with those apis14:19
frankbanhazmat: looking14:19
frankbanhazmat: ic, add_local_charm right? seems great14:21
hazmatyup, that's the one14:21
hazmatfrankban, the size param is required btw14:21
hazmatfor file objects14:21
hazmatthere's some broken code in stdlib httplib that looks like it would handle file objects, but doesn't correctly14:21
frankbanhazmat: in our case, that request is generated by the GUI (javascript+xhr) and the server goal is to just act as a proxy to the real endpoint14:22
hazmatunderstood14:23
jcastrohey frankban and hazmat14:24
jcastrocan we talk about that bug where the deployer bails if a unit doesn't come up properly?14:24
hazmatjcastro, sure... its actually intended behavior.. but i can see where that wouldn't be ideal for the gui.14:25
jcastroreally? 14:25
* jcastro is confused14:25
jcastroso if a unit doesn't come up it's supposed to error out?14:25
hazmatjcastro, well deployer's original use case was automated testing and hacking on ostack charms, if it didn't come up right, it was a fail... there's options on deployer to try and auto-resolve errors as well, per some lscape use cases around ha stack14:26
jcastrooh I see!14:26
hazmatjcastro, yeah14:26
hazmatjcastro, but for the gui, that doesn't feel like its really appropriate14:27
jcastrothe bundle use case for bundler would be something like "19 out of 22 instances have come up" and the relationships and bundle would be deployed, but you'd have to manually resolve the issues14:27
hazmatfrankban, gary_poster, ^ any comment?14:27
hazmatjcastro, agreed14:27
jcastroI think it's more for "bundles" than just the gui14:27
frankbanhazmat, jcastro : I'd suggest  a deployer option to switch behavior, I don't remember how difficult it is14:27
gary_poster+114:27
hazmatfrankban, should be straightforward.. its a short-circuit to wait_for_units14:27
gary_posterI wonder if default should be behavior jcastro describes14:28
jcastroif it's deploying a bundle do it the resilient way, and maybe if not doing a bundle it reverts to "test mode"14:28
hazmatgary_poster, for gui, i think so. for cli.. a switch14:28
hazmatnon default switch that is14:28
hazmatjcastro, there isn't a deployer distinction to bundle vs not bundle14:28
jcastroyeah as long as the default is what the users want, which IMO would be "get the thing up and running"14:28
gary_posterhazmat: for cli, simply because of history14:29
frankbanhazmat: that would be great, considering that the importer is instantiated with the defaults by the gui server14:29
jcastrofor everyone else --bundle-test-bail-out or whatever14:29
hazmatjcastro, would you mind filing a bug?14:29
jcastroyeah there's a bug, I can't find it right now, digging14:29
gary_posterhazmat: I think the "continue on error" case would be preferred for users, where "stop on error" should be opt in for devs14:29
gary_posterif we are moving the tool to be more user centric14:30
gary_posterbut if the vision is for deployer to continue to be dev focused then I agree14:30
hazmatgary_poster, well.. not entirely.. ops wants stop on error to14:30
hazmatand qa as well14:31
hazmatjcastro, not seeing the bug flipping through the open list14:32
jcastroWell if I am deploying an HA bundle and some units don't come up but the service gets up, then I am up and running and just manually fire up more nodes14:32
jcastroinstead of being dead stop 14:33
gary_posterhazmat: huh, ok14:33
gary_posteryeah, I am seeing jcastro's position, but <shrug>14:33
jcastrohazmat, so for example, the mongodb charm has a race (which yes, should be fixed)14:34
benjirick_h_: I'm having a problem with a test, do you see any reason this assertion would fail? http://paste.ubuntu.com/6832325/14:34
jcastrobut the entire bundle is undeployable because out of 13 upcoming units, one will bail14:34
jcastrohttps://bugs.launchpad.net/charms/+source/juju-gui/+bug/125230114:35
_mup_Bug #1252301: guiserver reports second bundle as failing after the first fails <juju-gui (Juju Charms Collection):Triaged> <https://launchpad.net/bugs/1252301>14:35
jcastrohere's the bug14:35
benjirick_h_: the error is: AssertionError: expected null to equal 'sidebar'14:36
rick_h_benji: looking14:36
jcastrohazmat, my other POV is that instances are disposable entities that may or may not come up depending on the provider so deployer should plan for failure14:37
jcastrofor example if a unit goes away juju won't die, it'll just fire up another one and relate it14:37
gary_posterfrankban: mm, I added deployer as "also affects" to jorge's bug.  was that right?14:38
jcastroI did that yesterday but I might not have gui bug powers14:38
frankbangary_poster: it seems right to me. hazmat: IIUC that bug describes a side effect of what we are discussing: IIRC if a pre-existing unit is in an error state, wait_for_unit raises an error14:40
rick_h_benji: would have to find the code that app uses on fire14:40
benjirick_h_: will you unpack that a bit?  I don't quite understand.14:41
rick_h_benji: you're firing an even on app, which must be caught and then triggers a change event on subapp? the viewNavigate is getting no viewmode in the change object14:41
rick_h_benji: hangout?14:41
benjirick_h_: sure; how about the daily hangout?14:41
rick_h_benji: rgr14:41
jcastrofrankban, quickstart -i is a thing of beauty by the way14:44
frankbanjcastro: great! :-)14:45
hatchI wonder why my +1's on G+ never show up on the posts14:56
rick_h_hatch: it was an unworthy +1, Google has said so14:57
hatchhaha, it's happened quite a bit in the past week or so, I get the notification that people have +1'd a post but then the post doesn't show it14:58
rick_h_'eventually consistant'14:59
hatchtooth paste, coffee, oranges.....flavours that do not mix well14:59
hazmatfrankban, yeah. the wait_for_units should ideally be self-contained at least to services being deployed.15:08
jcastrohazmat, is that bug sufficient or do you need anything else from me?15:29
hazmatjcastro, its not quite the same, but i'll add a note15:36
bacjcsackett: do you plan a deploy of your charmworld change to production?15:44
jcsackettbac: we're actually only interested in triggering tests for charms from staging right now, b/c charm testing has been completely sorted out and we can tweak things on staging easily.15:46
bacjcsackett: perfect15:46
jcsackettbac: as long as charm_testing_* variables aren't set in the charm config, testing shouldn't be triggered on production at all.15:46
jcsackettonce everything is locked down we'll turn it off on staging and file an RT to have those config vals set on prod.15:47
Makyojujugui call in 1015:50
benjisigh; I really wanted this done by the call15:50
rick_h_benji: no luck?15:50
benjirick_h_: well, I did something that is not entirely evil, but I'd rather it have been simpler15:51
benjinow I just need a couple more tests and it should be reviewable15:51
rick_h_hatch: the hard part will be all the manual work to turn viewlets into views :P15:53
rick_h_curses, he ran away on me!15:53
hatch"did you turn it off and back on again" 15:56
hatchlooks like osx has the same issue resolution that windows xp has15:56
* gary_poster has to switch computers15:57
MakyoOops, jujugui call in 115:59
rick_h_bac: stand up please16:01
rick_h_hatch: the hard part will be all the manual work to turn viewlets into views :P16:01
frankbanjujugui: I am getting "You're not allowed to join this video call."16:01
hatchrick_h_ oh no that should be pretty mechanical16:03
hatchwith some changes to the viewlet manager it can even be done in stages16:03
hatchrick_h_ lemme find that plugin example16:09
Makyogary_poster, want that I should be on that call?16:10
Makyo(I see the invite, just missed the discussion around it)16:10
rick_h_hatch: yea, I mean I found people asking a bunch of questions and the usualy answer is the global before/after calls but nothning that could hoook/load into it16:10
hatchrick_h_ I think I was going to monkey patch the methods which call the before/after calls to allow middleware to be injected16:15
hatchI'm just looking at istanbul16:16
hatchit integrates with mocha, just want to see how they do it16:16
rick_h_hatch: rgr, ok. Yea that's my plan as well16:20
rick_h_and I'm looking at beforeEach/afterEach actually16:20
hatchwhy is everything they do documented so poorly lol16:22
rick_h_hatch: yea, I didn't come away from this search liking mocha any more than before tbh16:22
rick_h_you got my hopes up with a plugin architecture16:22
hatchthere is a setup() method, but no documentation on what params it actually takes :/16:22
=== rogpeppe2 is now known as rogpeppe
hatchhmm mocha actually works pretty interesting under the hood16:24
hatchrick_h_ ok here is probably the best place to inject https://github.com/visionmedia/mocha/blob/master/lib/suite.js16:25
hatchthat modifies the actual test suite16:25
hatchit's where it defines the before/after etc methods16:26
rick_h_hatch: rgr16:26
hatchfrankban have some time in a couple minutes to give me a run through of quickstart?17:16
bacgary_poster, frankban: how does that discussion on the auth call affect the current quickstart effort?  are the .jenv files not going to be named based on the local env name?17:16
gary_posterbac, they will continue to be named based on the local env name AIUI17:17
bacgary_poster: ok, so quickstart has that info but the charm does not.  that's the difficulty, correct?17:18
gary_posterbac, right17:18
bacgary_poster: i feared they were tending to making the jenv files opaquely named17:18
gary_posternot aiui17:18
bacgotcha17:18
frankbanhatch: sure, ready when you want17:20
hatchfrankban https://plus.google.com/hangouts/_/76cpiisbms6bilnp0eh9n94nuc?hl=en17:24
benjirick_h_: I'm going to take lunch now; would you be able to pair with me some time after lunch on these tests?  There are no tests for some of the code I am changing and building up enough state to get a simple test to run is killing me.17:32
hatchbenji I wouldn't do what you're trying to do17:35
hatchI would implement a unit test for either side then a selenium test for the integration test17:36
hatch(assuming you're doing the minimize-charm-browser-when-inspector-charm-details-opens work)17:36
benjihatch: that's what I'm trying to do.  I want to call a function and test that it fires an event.17:36
benjithe function in question is not called in the test suite17:36
hatchhmm ok, well when you get back from lunch if rick_h_  isn't available I can also give you a hand with it17:37
benjithanks17:38
hatchI may end up liking python after doing this quickstart work17:39
hatchshhh don't tell rick_h_ 17:39
benjiheh17:40
rick_h_benji: sure thing17:42
rick_h_hatch: :P17:42
hatchhaha17:43
frankbanguihelp I need two reviews + QA for https://codereview.appspot.com/57820043 . Anyone available? Thank you!18:12
frankbanit's the server side putcharm18:12
MakyoNew modem \o/18:13
hazmatfrankban, out of curiosity what do you like tornado?18:17
hazmater.. s/what/how18:17
hatchit's good, a little windy and tough to control, but it clears a nice path18:20
frankbanhazmat: yes I do, it worked pretty well for the charm needs18:20
Makyohatch, listen: no18:20
hatchMakyo lol18:20
hatchOH CMON that was funny18:20
frankbanneed to go now, see you!18:20
hazmatfrankban, more modern/better than twisted? more explicit than gevent?18:20
gary_posterfrankban: will review/get reviews for that.  ttyl18:21
benjirick_h_ and hatch: you can play paper-rock-scissors now to decide who helps me18:43
hatchhaha18:43
hatchbenji sure I'll help18:43
rick_h_benji: I got it :)18:43
hatchgot the code pushed up to gh?18:43
rick_h_:P18:43
hatchlol18:43
rick_h_benji: standup hangout?18:44
benjirick_h_: sure18:44
hatchok rick_h_ go ahead I'll cede....this time....18:44
rick_h_hatch: you can come along but beware :P18:44
rick_h_I'll show you how the browser is setup to already deal with extra views while we're at it hah!18:44
hatchwell maybe I'll listen in18:44
hatchhaha, I'm actually thinking of a slight refactor to the side bar 18:45
hatchso we would have two different approaches18:45
rick_h_think all you want, we'll meet and do battle18:45
rick_h_code wars!18:45
benjihttp://i1.kym-cdn.com/photos/images/newsfeed/000/296/322/c6f.gif18:45
rick_h_benji: hatch https://plus.google.com/hangouts/_/calendar/Z2FyeS5wb3N0ZXJAY2Fub25pY2FsLmNvbQ.j0rk5d371ph8331ijtf48t2uj0?authuser=118:46
hatchbenji lol18:46
benji"You're not allowed to join this video call."18:46
rick_h_benji: :(18:46
hatchtry again18:46
rick_h_ok, I'll create one18:46
hatchit's been doing that18:46
rick_h_or hatch got in18:46
rick_h_benji: ah, might be a work vs non-work account thing18:47
rick_h_I can create one for non-work accounts18:47
rick_h_hatch: benji https://plus.google.com/hangouts/_/76cpjjso2tiuhku81mmnp9md8g?hl=en18:47
=== _mup__ is now known as _mup_
hatchdoes anyone have any favourite python learning resources? 19:25
hatchI'm mostly interested in the ecosystem19:26
rick_h_hatch: hmm, I mean I went through a ton of books and such when learning python to get out of php. I don't know I've got a single/couple favs. 19:27
rick_h_hatch: but more than happy to help answer any ecosystem questions/etc19:27
hatchwell there is stuff like pip, virtual envs, __whatevers__19:29
hatchso less about the syntax of the language and more about how applications are actually structured19:29
gary_posterhey arosales, we'd like to get quickstart in main in trusty.  Is that possible?  Who would you suggest we talk to?19:33
rick_h_hatch: yea, there's still some personal preference stuff in there that builds up with experience. There's a couple of books on 'good practices'19:34
arosalesgary_poster, I would start with https://wiki.ubuntu.com/MainInclusionProcess19:34
arosalesand get the bug filled out with the required points listed in the wiki19:35
gary_posterok thank you arosales.19:35
arosalesgary_poster, after that it is makeing sure the bug has ubuntu core devs subscribed and following up with the core devs if the bug isn't be reviwed in time for feature freeze.19:37
gary_posterarosales: do we have a juju and/or ecosystems contact for that in particular?19:37
arosalesgary_poster, not specifically. Best bet is to subscribe ubunt-mir team to the bug so it shows up in the archive admins queue19:39
gary_posterok19:39
hatchso is there a python api docs somewhere?20:15
hatchtried here http://docs.python.org/2/library/20:15
hatchbut it doesn't have the 'call' method for some reason20:15
hatchoh nm20:16
hatchI was using the docs wrong20:16
hatchheh20:16
hatchcarrryon20:16
hatchbrain....wants....to.....type.......'var'20:24
rick_h_hatch: lol20:25
rick_h_yea, and the ; and the function vs def20:25
hatchhaha it's hard!20:25
hatchI wish you could indent one more time on def's20:26
hatchthe fn name is hard to scan for20:26
hatchcould maybe be my syntax highlighting though20:26
hatchrick_h_ which syntax is preferred in the python world? https://gist.github.com/hatched/867574320:33
hatchor does it really matter20:33
rick_h_hatch: replied20:36
rick_h_bah, lost my formatting20:36
rick_h_hatch: there we go20:37
hatchyou guys are really functional eh? :)20:37
rick_h_testable :)20:37
hatchcool I like it better your way20:37
rick_h_so then if bootstrap_requires_sudo: cmd.insert(0, 'sudo')20:38
hatchthanks20:38
rick_h_np20:38
benjirick_h_: how do you get github to not eat indentation?20:40
rick_h_benji: markdown, add 4 spaces first20:40
benjiah!20:40
rick_h_so it's a code block20:40
hatchI think you can also do ``` ```20:40
hatchmaybe that depends on the md type though20:40
* hatch dislikes that there are dif implementations of md20:40
benjirick_h_ (and hatch): https://gist.github.com/hatched/8675743/#comment-99508620:41
rick_h_benji: +120:41
rick_h_benji: wasn't sure on line length in there20:41
hatchpython and js appear to have very similar operations20:41
rick_h_I'd rather have the if block vs dealing with newline in there20:41
rick_h_OMG I think it finally works!20:42
* rick_h_ does a happy dance20:42
hatch?20:42
hatchlol20:42
rick_h_hatch: https://github.com/juju/juju-gui/pull/8920:43
hatchkewl lookin20:44
rick_h_and mocha can go choke on a carrot atm 20:45
rick_h_pita to get this hacked into play the way they bootstrap everything for you on load so you can't intercept crap20:45
hatchohh lol I Was reviewing it20:47
hatchtill I saw the huge commented out section20:47
hatchhaha /me fail20:47
rick_h_oh no man, it's WIP. I've got to find every instance of makeContainer and pass a this in as the first argument20:47
rick_h_and remove all the cleanup instances20:47
rick_h_and figure out how to clean it up with more comments/wtfs20:47
hatchyou should probably land them as separate branches20:48
hatchor at least separate commits20:48
rick_h_definitely commits, but hard to do as different branches20:49
rick_h_if I don't remove the cleanup the container won't exist. If I don't update the context argument then the calls will all fail with wrong args20:49
hatchright20:50
hatchhmm20:50
hatchrick_h_ why the addCleanup method? can't they just push to jujuTestCleanup?20:52
rick_h_hatch: could, but I like the clean api better. 20:52
rick_h_hatch: "call this function with a cleanup" and you dont' have to know jujuTestCleanup/etc20:52
rick_h_just addCleanup20:52
bacfrankban: (hoping you see this tomorrow morning): please review the admin-secret change  https://codereview.appspot.com/5790004320:52
hatchmaybe _jujuTestCleanup = {}20:52
hatcher20:52
rick_h_meh, I guess it's nitpicking but I liked it better20:52
hatch[]20:52
hatchwhichever implementers choice 20:53
hatchdefining the function is a perf hit is the only negative20:53
rick_h_originall it was _jujuTestCleanup as a hidden var and then realized wtf...no one's going to use _juju...20:53
hatchhaha no the idea behind using _jujuTestCleanup is to signify to other devs that you shouldn't interact with it directly20:54
rick_h_:P it's been a long day20:54
hatchlol20:54
hatchalso you don't need bind() as you have it in your WIP (although I suspect that's not staying anyways)20:55
hatchbind() returns a function that's bound to the context 20:55
rick_h_yea, I was hitting some issues there. The scope changes around a bit20:55
rick_h_right20:55
hatchyeah with call() you have it right20:56
gary_posterbac, fwiw I think jenv files started in 1.16 (per your MP)20:56
hatch+1  20:56
hatchI like it20:56
rick_h_I was running into the various bits are in a context of mocha.suite, Hook, etc20:56
rick_h_getting it straight is a pita20:56
hatchyeah, this is pretty much exactly how I was going to do it so yay! lol20:56
rick_h_lol, ok well will clean it up and get it going up for review in the morning if I can get my sed/awk fu on to mass update the makeContainer calls20:57
rick_h_though I wonder what will happen to makeContainer calls not in the before hook but the test itself :/20:57
hatchit should add to the cleanup and clean it up in the after20:58
rick_h_hatch: right, but will the context of 'this' have access 20:58
rick_h_hatch: because the this in there is the Hook object, not the test fun20:59
hatchohh...hmm20:59
rick_h_hatch: I'll get it worked out. anyway, small happpy dance20:59
hatchyeah - maybe we can assign a global context object that the utils methods can use20:59
gary_posterMakyo or hatch, either of you up for being the second reviewer of frankban's https://codereview.appspot.com/57820043 ?  I'm trying to do qa and review21:01
hatchsure21:01
Makyoty hatch21:01
MakyoRushing on tests21:01
gary_posterthank you21:01
hatchgary_poster can you ping me after you're done yours so I can learn from the comments (if any)21:01
gary_postersure hatch.right now I'm flailing at trying to update juju core :-P21:02
hatchhaha, I'm failing at getting mine to stop using the custom built one lol21:02
gary_poster:-)21:03
hatchI borked my path, there needs to be a $PATH api21:03
hatchso you could splice into it and stuff haha21:03
hatcharray methods on $PATH21:03
benjirick_h_: here's my WIP if you have a minute to look and comment or formulate thoughts for a discussion tomarrow about how to split the events up plus anything else you see, I would appreciate it21:04
rick_h_benji: rgr21:04
rick_h_benji: link? or issue pushing?21:14
hatchrick_h_ you should just KNOW21:15
hatchlol21:15
benjirick_h_: https://github.com/benji-york/juju-gui/compare/juju:develop...benji-york:auto-open-close?expand=121:16
hatch:) legit checkpoint commits21:16
hatchI dont' think I've ever seen someone write 'checkpoint' lol21:16
rick_h_benji: k, I'm a little confused will look at it. 21:26
hatchrick_h_ it must be that app is a bubble target (somehow) of the topo21:39
hatchmight have to grep for that one21:39
gary_posterhatch, that local charm DnD is looking pretty cool :-)21:53
* gary_poster excited to see it hooked up21:53
gary_posterhatch, I completed review of https://codereview.appspot.com/57820043/ .  Not the most scintillating review ever :-)21:54
hatchyeah it's pretty nice, and it looks like core will be fixing the folder in a zip issue too21:54
hatchhaha ok thanks21:54
hatchI'm guessing @gen.coroutine is part of a lib?21:55
hatchI didn't think python had real coroutines 21:55
rick_h_hatch: that's part of tornado I think21:57
rick_h_the @gen stuff21:57
rick_h_hatch: http://www.tornadoweb.org/en/branch2.4/gen.html21:57
hatchahhh21:57
hatchasync is hard :P21:58
rick_h_darn it, my pebble isn't in the first orders :(22:01
hatchthe steeeel one?22:01
huwshimiMorning22:01
hatchhey huwshimi 22:02
hatchhuwshimi did you want us to prioritize upgrading the node dep? (if possible) are you able to do it?22:02
huwshimihatch: I can give it a go, but I have no idea how to do it.22:03
hatchhuwshimi do you know how to deploy a local charm and all that business? 22:03
rick_h_hatch: yea, ordered day of annoucement but guess I'm still in a second batch 22:03
hatchrick_h_ that's a good thing - everyone else gets the first broken run22:04
hatch:)22:04
huwshimihatch: I've done it before (to LXC).22:04
hatchhuwshimi yeah our issues were when it was uploaded to ec222:04
hatchso you pull it down, update the python deploy files, update the package.json, shrinkwrap, then push it up somewhere then deploy from that repo22:05
huwshimihatch: Have you done it before? How long a task might it be?22:09
hatchhuwshimi for you, probably a day, just because of learning the process, testing takes a while to spin up ec2 machines etc22:10
huwshimihatch: Sounds horrible22:11
hatchlol22:11
huwshimihatch: Sounds like the worst kind of monkey testing.22:11
hatchwell it's taken me a day to do a 15 line diff in a python script because of learning haha22:11
hatchso it takes a while to get up to speed sometimes 22:11
rick_h_https://github.com/blog/1767-redesigned-conversations 22:19
hatchstill can't reply to a comment :/22:20
* gary_poster runs22:20
gary_posterbye all22:20
=== gary_poster is now known as gary_poster|away
rick_h_hatch: does the Environment really have no destructor?22:31
rick_h_hatch: or am I blind?22:31
hatchrick_h_ the environment view?22:31
rick_h_hatch: rgr22:31
hatchnope it doesn't22:32
hatchnot sure why, it never has 22:32
hatchrick_h_ it's not really an issue because it's never destroyed haha22:32
hatchbut we should have one for testing22:32
rick_h_hatch: heh, except for tests22:33
rick_h_hatch: ok, thanks for sanity checking22:33
hatchrick_h_ so I'm writing tests and I need to stub out the 'call' method in utils.py 22:43
hatch_, version, _ = call('juju version') 22:44
hatchhere is the 'real' code22:44
rick_h_hatch: k, that's what the Mock library is for and you should see some @patch stuff in there22:44
rick_h_or "with patch"22:44
hatchok looking22:44
hatchok got it22:45
hatchhttps://pypi.python.org/pypi/mock22:46
hatchthat one?22:46
rick_h_yep22:46
hatch""Mock is very easy to use"" HAH maybe if you know Python22:48
hatch:P22:48
rick_h_yea, kind of the basis of mocking like that. You have to learn a bit on how it works22:48
hatchI'll get er figured22:49
rick_h_k, I'll be in/out if you need a hand. Dinner making time22:49
hatchyeah it's close to EOD time for me too and I need to go shovel a parking spot because the graders came by22:49
hatchand graded a nice 3ft pile of snow into the street spot :)22:50
hatchthis might be a working tonight thing22:50
rick_h_wheeee22:50
rick_h_yea, same here trying to get this event chain right22:50
rick_h_was so happy leaving the inspector be lately22:51
hatchhaha. I want to convert the viewlets to views and then modify the viewlet manager to work with views22:51
hatchthere is just not enough time  in the day22:51
hatchor night22:51
rick_h_yea, I know. 22:51
rick_h_there's a good path, just needs some stabbing to find the tender bits22:52
hatchhaha look at this pic http://jalopnik.com/there-have-been-274-car-crashes-in-austin-today-151085362222:52
hatchhow the heck does that even happen!22:52
hatchice? snow? 100MPH should be safe right?22:53
Makyo-Only- 29% of the tests fail now!  Yay!22:58
hatchlol what are you working on?23:13
huwshimihatch: I think he's working on breaking everything23:14
hatchhaha23:14
huwshimihatch: Only 71% to go.23:17
MakyoAaaand down from 264 to 9 failures in one swell foop.23:18
MakyoMy develop was a little stale :T23:19
MakyoThe downside being I've not figured out how to rebase out merge commits yet.23:19
hatchMakyo yeah I'm not entirely sure about those either23:20
MakyoMaybe cherrypick, dunno.  Will burn that bridge when I get to it.23:20
MakyoAaaand down to 6 (with a new bug, d'oh)23:23
MakyoThis would be a one-line addition to rick_h_'s plugin, I think, but I haven't fully investigated.  Will bring it up tomorrow.23:26

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