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

rick_h_huwshimi: morning, how goes?00:13
huwshimirick_h_: Hey! Good thanks. Yourself?00:25
rick_h_huwshimi: party party, LUG meeting00:26
rick_h_huwshimi: I wanted to check in on your test stuff. I know I wasn't amazing helpful with Y.later, while it's in the code base I've not used it much so wasn't sure00:27
rick_h_huwshimi: but I know I had looked at how to do testing around the animations and had those couple of ideas for you, not sure if they helped?00:27
huwshimirick_h_: Sounds fun!00:27
huwshimirick_h_: The main problem is that it messes with other tests. I think I'll just go with an extra check to see if the node exists. I know that will only come into play in the tests, but I don't have a better idea :)00:28
rick_h_frankban: thanks for the notes. i'll put that machiner file stuff back. No reason to pull it especially in a pyjuju specific branch11:58
frankbanrick_h_: +111:58
frankbanrick_h_: do you have a chance later to review + intensively QA my branch?12:00
rick_h_frankban: sure thing. 12:00
frankbanrick_h_: thanks!12:00
bacbenji: good news!  we now have two stuck baskets.  progress.13:30
benjiheh13:30
hatchgooood morning13:51
rick_h_mornnig13:53
rick_h_or something13:53
hatchhaha, hows it going?\13:53
rick_h_well, ec2 hates me but other than that...ok13:53
hatchboo ec213:55
rick_h_frankban: I've definitely done something that's taken the test run time waaaay up. the test_deployer test runs forever compared to trunk14:06
frankbanrick_h_: looking again at your new mp14:09
frankbanrick_h_: is the deployed gui slow as well?14:09
rick_h_I'll try it again. I did a manually make deploy once to make sure it was working but didn't time it14:10
rick_h_what should a make deploy run time be about?14:10
frankbanrick_h_: https://codereview.appspot.com/52440043/diff/1/tests/20-functional.test line 61 on the right: force_machine should be 014:11
frankbannot None14:11
frankbanrick_h_: so the functional tests are creating new machines, that takes lots of time14:11
rick_h_ahhhhh!14:11
frankbanrick_h_: I think you can safely delete that line and just pass force_machine=0 to juju_deploy()14:12
rick_h_bah, bad if replacement there. Gotcha14:12
rick_h_rgr14:12
frankbancool14:12
rick_h_thanks, I hated that everything seemed fine but didn't want to land it with these long test runs14:13
rick_h_will do one more test run and will reduced the test time back to 40min if all goes well14:13
frankbanrick_h_: +114:13
rick_h_jujugui afk for a sec while I go pick up new glasses. 14:13
hazmatrick_h_, google glasses ? :-)14:16
rick_h_hazmat: hah, not quite. More like "old-man glasses"14:38
rick_h_hazmat: I'm more a watch guy, waiting for that steel to ship :)14:38
rick_h_man, new glasses adjust period...begin!14:40
hatchshouldn't take toooo long14:41
hatchhazmat interesting bug report re a node api module14:42
rick_h_lady is telling me that "you had a 15 degree axis of rotation change that'll take some getting used to". Not sure wtf that means but it's strange. 14:42
rick_h_hatch: +1 on that as well :)14:42
hatchit's quite a bit of work beyond face value because we actually use a modified api because of the guiserveer 14:43
hatchbut I can see that being fixed14:43
rick_h_right14:43
rick_h_but overall it's an awesome idea14:43
hatchagreed14:43
rick_h_<3 api libs14:43
hatchwe'll just add that to the list of things to do.....lol14:43
hatchrick_h_ I'm going to review huw's branch just fyi14:45
bacjujugui: i've stopped charm/basket ingestion on staging.jujucharms.com so i can analyze the existing failure state without looking at a moving target.  let me know if that causes you any problems.15:08
rick_h_bac: rgr15:08
bacrick_h_: so you've got astigmatism and it's rotating?15:08
rick_h_bac: I have no idea. I've always had just a light correction, -.25 and .5 or something15:09
rick_h_but this year one eye went to a full 1.015:09
rick_h_and never heard of the axis stuff until today15:10
bacrick_h_, benji: it looks like the stuck baskets are caused by entries in basket-queue that are locked and have been there for a while.  my theory is that they were locked when the process was abruptly stopped for some reason, though i cannot prove it.  Q: any reason to not delete the incoming charm and bundle queues at start up?15:11
hatchrick_h_ I asked, it's possible but as the delta increases the likelyhood the dr screwed up does too :)15:11
hazmatrick_h_, pebble steel is ships in two weeks. https://getpebble.com/steel15:12
rick_h_bac: hmmm, the worry is that something falls through the cracks15:12
bacrick_h_: i think the axis is for astigmatism.  other problems (near/far sight) are not rotationally sensitive but astigmatism is.15:12
rick_h_bac: I can't work up a case where it could atm though15:12
rick_h_hazmat: yep, order placed...just now impatient15:12
bacjujugui: <commerce>speaking of wrist-based gadgets, anyone interested in a slightly used fitbit?  the original, not the one with the time display.</commerce>15:13
rick_h_bac: hah, I can't give mine away15:14
bacoh, really?15:14
rick_h_it's on its second person15:14
rick_h_yea, gave it to a friend that tried to give it back and made them give it to a friend15:14
hazmatbac, rick_h_ i'd be interested re fitbit.. you mean the original pod or the wrist thingy?15:14
baci like it, but it was made obsolete by the motion chip on the new iphone15:14
bachazmat: original wrist15:14
hazmathmm.. pass.. doesn't work with a standing desk very well.15:15
bachazmat: the flex15:15
bacrick_h_: i did a comparison to fitbit vs phone.  and the logs were almost identical.  not worth having an extra doohickey if i'm always going to have the phone in my pocket.15:16
rick_h_bac: yea, I got tried of having two 'watches' and it was so inaccurate15:17
rick_h_once told me I took 680 steps in a shower15:17
rick_h_that and does't work with my phone (though they did just update it this past week) so I got rid of it15:17
bacha.  yeah, hammering confuses it too15:18
hazmatis gary out today?15:30
hatchhazmat till friday15:31
hatchanything I can help with?15:31
hatchbrb in 15, relocating15:39
Makyojujugui call in 1015:50
hatchjujugui call in 715:53
hatchjujugui call in 215:58
benjicall in 1:4815:58
Makyo:T15:59
benjirick_h_: come back!16:07
rick_h_benji: oh righ 16:07
rick_h_right16:07
hatch119kph wind right now16:20
benjiyow!16:21
hatchooookk well the 119kph wind gusts have knocked out the power16:26
hatchwell...knocking out...16:27
hatchyay power16:56
rick_h_yay17:03
rick_h_charge those batteries17:03
hatchyou bet, all plugged in now17:10
hatchit's so quiet 17:57
rick_h_ssshhhh17:59
rick_h_I'm trying to juju set juju-gui juju-gui-source="https://github.com/juju/juju-gui.git network-prototype" 18:00
hatch:)18:03
rick_h_hmm, well it errors but it loaded the source18:06
rick_h_https://ec2-50-16-67-171.compute-1.amazonaws.com18:06
rick_h_hatch: should this work?18:06
hatchlooking18:06
rick_h_getting juju.views.NetworkListView is not defined18:06
hatchrick_h_ it's possible that the file which has that view is not getting rolled up18:07
rick_h_k, will call that a "not my problem" problem atm18:07
rick_h_but it worked bwuhahahaha18:07
hatchI believe it was only ever used with make devel18:09
hatchyeah I'd say it's a success for the deployment system though :)18:09
rick_h_https://ec2-50-16-67-171.compute-1.amazonaws.com/juju-ui/version.js is good so I'll roll with it18:09
hatchrick_h_ maybe the source should be two values18:12
hatchso the repo as one and the branch as the other18:12
hatchand maybe taking it one step further, the hash18:12
hatchso that we can 'update' something to a new source version18:12
rick_h_hatch: yea, I started down that path but it's kind of a pita18:15
rick_h_going with space to specify a branch/sha18:15
rick_h_and @123hfehw for a sha18:15
rick_h_so right now I'm testing  juju set juju-gui juju-gui-source="https://github.com/juju/juju-gui.git @c7ddd186"18:15
hatchyeah I bet - I'm just thinking of the story where we want to use a certain branch but then keep it updated when new changes are pushed to it18:15
hatchdo you have to use the @?18:16
rick_h_yes, because it's the diff between a checkout and a checkout -b 18:16
hatchcan you not go `git checkout mybranch` and it'll just checkout the latest revno?18:17
rick_h_not if the branch doens't exist yet :)18:17
rick_h_you need the -b18:17
rick_h_if you have a local branch with that name, sure 18:18
rick_h_you can't do git clone url branch 18:18
rick_h_you have to git clone url && git co -bt branch18:18
hatchwel you CAN but that doesn't actually solve the issue of requiring some indicator that it's a hash18:19
rick_h_huh?18:20
hatchgit clone -b brancho --single-branch <repo>18:20
rick_h_bah, this sucks. because we did a shallow clone you can't checkout a hash anyway. 18:22
rick_h_have to unshallow it first18:22
* rick_h_ checks git version on precise18:22
hatchhmm18:23
hatchit seems like there should be a way to pull down said hash 18:23
rick_h_working on it since git fetch --unshallow is only in trusty :(18:24
hatchrick_h_ could we not say they need to specify a repo and branch name then a hash?18:28
hatchthen we could use the single branch checkout 18:29
hatchwhich should be fast....I think18:29
rick_h_but a hash has nothing to do with a branch. It's just a single git commit in a detached head state18:29
hatchright but if we already have the single branch pulled down we will have that hash18:29
hatchit's not ideal, but in order to support the widest range of systems18:29
hatchI don't know if this idea will work just thinking of alternatives :)18:31
rick_h_yea, sorry testing out here. --depth=20000 gets me caught up and I can git co hash18:32
rick_h_I know we *can* do a lot of things. Just trying to cover initial bases atm18:32
rick_h_been on this card for a week and trying to support everything git does will drive me nuts18:32
hatchwell we don't need to support everything. The BIG thing is a branch18:33
rick_h_right, and branch works18:33
hatchthat's the primary goal, I think a hash is a 'nice to have'18:33
rick_h_now, I can see wanting to go back to an old commit in that branch to see "does xxx work"18:33
hatchyeah for sure18:33
hatchor move forward because you have updated that branch18:33
rick_h_but even that can be done by the dev with "git checkout hash && git co -b test-feature && git push origin test-feature"18:34
hatchjuju needs a 'trigger' config option :) 18:34
hatchdimitern are you still around?18:58
benjirick_h_: I think I tracked down the cause of https://bugs.launchpad.net/charmworld/+bug/1263120.  Let me know when you have a minute to chat about it.  Thanks.19:33
_mup_Bug #1263120: self related services in a bundle fail proof <charmworld:Triaged> <https://launchpad.net/bugs/1263120>19:33
rick_h_benji: sure thing, your hangout or mine?19:34
benjirick_h_: I'll make one.19:34
benjirick_h_: https://plus.google.com/hangouts/_/7acpi7cc7l9l3r0bdfembikccg?hl=en19:35
* Makyo runs out quickly to grab MiniDP->VGA adapter. Kind of important for presentations D:19:45
bacyay, me, i'm sort of back20:22
rick_h_lies!20:23
hatchhah the power keeps dropping out here20:27
hatchit's like there is a switch that gets tripped when the power lines get stretched to far lol20:27
bachatch i think my ISP is on your power grid.  not sure how that happened.20:42
hatchhaha - ours is usually really stable, I guess 100+km winds are around it's limit20:43
hatchbac https://twitter.com/nheagy/status/42351331444696268820:44
bachatch: you should use that three weeks in the summer when the ground isn't frozen to bury those lines20:44
hatchhaha, they are except for the major powerlines...or something like that20:45
bacwow, that's a twisty mess20:45
hatchheh yup20:45
rick_h_yxe?20:46
hatchSaskatoon20:46
hatchairport code20:46
rick_h_umm, ok. No idea wtf the y and x and e came from, but I'll take your word for it20:47
hatchrick_h_ well the Canadian airports start with y. But I have no idea how they decide where the other 2 come from20:49
rick_h_y are you going to canada?20:49
hatchlol20:50
rick_h_it's X-tremely E-...fail20:50
hatchE-lectric20:50
rick_h_I thought it was rather un-electric currently :P20:51
hatchlol20:52
bacjujugui: anon21:21
rick_h_c-ya21:21
lazypowerSo, juju-quickstart21:28
lazypowerthis is awesome!21:29
rick_h_:)21:29
lazypowerit deploys the gui by default too?21:31
lazypoweroh man21:31
rick_h_yea, on node 0, pre-packaged so it installs super fast21:31
rick_h_should auto log you in with your admin secret as well21:31
rick_h_at least that feature landed, not sure on released/what version you're using21:31
lazypowerI'm running trusty, but i digress - this is good stuff 21:32
hatch:)21:32
rick_h_we'll let frankban know you like his baby21:33
lazypoweri file bugs against juju-quickstart right?21:34
rick_h_sure21:34
lazypowerAh, probably better to put it in the merge request21:34
rick_h_lazypower: are you testing out the branch I sent jcastro ?21:35
lazypowerindeed21:35
lazypowerWhen the local machine takes longer than the deployer expects to come online it fails the quickstart21:35
rick_h_lazypower: ah, then yea. Please put them actually over in the code review page link. Look for the 'codereview.reitveld' link in the MP21:35
rick_h_oh, that's a bug and different that the MP21:35
rick_h_file that as a real bug then please21:36
rick_h_mainly I wanted to get some others to help qa that interactive ncurses mode stuff 21:36
rick_h_but yea, it'll hit 1.0 in this branch so want to make sure it's solid pre-1.0 and that seems like a bug to hit. 21:36
lazypowerno problem i'll keep working through the ncurses stuff, i've got creds to do it.21:37
rick_h_cool, appreciate it21:37
lazypowerdude it works with the mouse too?21:41
rick_h_yep21:41
lazypoweri'm a fan21:41
lazypowership it21:41
lazypowergood thing i'm not your final quality gate21:41
lazypowerrick_h_, i've configured everything but openstack. Looks solid to me21:47
lazypowerthumbs up21:47
rick_h_lazypower: cool, thanks21:48
=== rogpeppe2 is now known as rogpeppe
lazypower*note the form validation was an excellent touch21:55
marcoceppirick_h_: it's related to this, https://bugs.launchpad.net/juju-core/+bug/1269519 kapil knows about it21:55
_mup_Bug #1269519: Error on allwatcher api <juju-core:New> <juju-deployer:New> <https://launchpad.net/bugs/1269519>21:55
marcoceppiif you guys find a solution to it, please let me know21:55
rick_h_marcoceppi: k21:56
huwshimiMorning22:11
hatchmornin huwshimi 22:19
huwshimihatch: Hey. Thanks for the review!22:20
hatchnp, hope my comments make sense, I'm also open to pushback on the widget vs view thing :)22:20
huwshimihatch: It all makes sense and seems straight forward22:21
hatchexcellent22:22
huwshimihatch: Are you all settled into home timezone yet?22:30
hatchhuwshimi when I came back I actually went snowboarding in another timezone and just got back late last night haha22:30
huwshimihatch: hah!22:30
hatch3 timezones in 1.5weeks 22:30
hatchtoo much22:30
huwshimiouch22:30
hatchis there a GUI bug about making multiple relations to the same service?22:37
hatchI can't seem to find it22:37
hatchmy search foo might be failing22:37
Makyohatch, ping.22:58
hatchhey22:58
MakyoI think my current card is related to double-dispatch and removing full-screen, but having a hard time digging into it, because it only shows up in prod.22:59
MakyoI don't want to leave it hanging, and am getting nowhere.22:59
MakyoInterested?22:59
MakyoNeed to review frankban's branch before EoD/vacation.22:59
hatchyeah I can pick it up23:00
Makyohatch, cheers.  I've tracked it down to somewhere in subapp/browser/browser.js, one of the state functions (it inits, but doesn't complete)23:01
hatchalright I'll pick it up first thing23:02

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