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

frankbanguihelp: I need two reviews (no QA) for https://github.com/juju/juju-gui/pull/163 . Anyone available? thanks!12:52
rick_h_frankban: looking12:53
frankbanrick_h_: thanks12:53
bacfrankban: sure12:58
frankbanthanks bac 12:58
frankbanbac, rick_h_: thanks for the reviews14:26
frankbanrick_h_: re machineUnitsMap, the idea is to call filterUnits only once, avoiding multiple modelList cycles for each machine14:27
rick_h_frankban: ah, that makes sense then14:27
frankbanrick_h_: cool14:27
rick_h_heh, not enough sleep last night to see it14:27
rick_h_hatch: no luck with upstream kernel14:32
rick_h_and man, kill a cpu makes for crappy performance14:32
hatch:/14:32
hatchguess no 14.04 for me yet then either 14:33
hatch:(14:33
rick_h_nope14:33
hatchdarn I was really looking forward to it14:33
hatchthink they will get it in before release?14:33
rick_h_well, there's a bug. Let's see if it gets fixed.14:33
rick_h_no idea, I'm guessing it's a matter of seeing if it works upstream14:33
hatchcan you link the bug so I can mark myself on it?14:34
rick_h_https://bugs.launchpad.net/ubuntu/+source/linux/+bug/128408514:34
hatchok I guess I can't add myself to it14:39
hatch'affects you' doesn't send emails and I can't add myself to the subscribers list14:39
rick_h_hatch: hmm, you're listed as subscribed14:39
rick_h_Notified of all changes14:39
rick_h_Jeff Pihach14:39
hatchok maybe lp is just being busted14:40
hatchthx14:40
bacrick_h_: hangout?15:01
rick_h_bac or benji can one of you please QA the change from Makyo? My machine just went boom and I need to get it back up. https://codereview.appspot.com/71070051/15:22
benjirick_h_: I'll take a look.15:22
rick_h_thanks15:22
hatchI'm trying to decide if I should land a branch which breaks things or make it even bigger15:47
rick_h_what does it break and what are you doing that's so large?15:48
hatchwell stuff that was in the 'inspector.js' file15:48
hatchhas been moved to mixins15:48
hatchwhich can't be mixed into viewlets15:48
hatchhttps://github.com/hatched/juju-gui/compare/juju:develop...hatched:service-config-view?expand=115:49
rick_h_hatch: on a call but let's chat after the standup?15:49
hatchcurrent diff15:49
hatchsure15:49
rick_h_jujugui call in 1015:50
hatchjujugui call in 215:58
MakyoOne of these days, I'll be able to sign into RV...16:09
MakyoActually, let me see about making a chrome profile with just canonical address.16:09
MakyoThat worked, whew.16:13
hatchlol16:13
rick_h_browser wars! fight the browser 16:14
rick_h_yay calls done! /me sends blood back to his ears16:15
hatchrick_h_ there is another approach - I keep working on this branch making logical commits then when it's done it can be reviewed one commit at a time?16:26
rick_h_hatch: I like the dupe code. We've got a history around that I think. 16:32
rick_h_I don't know I want to go through it one commit at at time16:32
rick_h_thought I guess that's something that might help in review16:33
hatchyeah just an idea16:38
hatchhas anyone ever had mocha just bail after 3 failures? I'm getting 3 failures, fixing them, get three more.... repeat 16:46
benjibac: have you build charmworld from scratch lately?16:49
benjiI'm getting this:16:50
benjiCould not find a version that satisfies the requirement charmworldlib==0.1.4 (from -r requirements.txt (line 7)) (from versions: 0.3.0)16:50
rick_h_benji: hmm is that not in the download-cache?16:51
rick_h_benji: maybe a recent update missed the cache16:51
benjirick_h_: nope, 0.3.0 is the only version of charmworldlib in there, I assume the requirement for 0.1.4 is wrong (since 0.3.0 exists)16:53
rick_h_benji: gotcha, in setup.py ?16:53
benjirequirements.txt16:53
rick_h_has the 0.1.4?16:53
benjiyep16:54
rick_h_hmm, must be a missed update then during bac's recent upgrade. 16:55
benjiMakyo: when the QA instructions say "Get bac's bundle wiki:wiki" what do you mean exactly?17:12
Makyobenji, sorry, one sec.  I got it through pdb, but there's an easier way.17:13
Makyobenji, /api/3/bundle/~bac/wiki/317:14
benjik17:15
Makyodata.services.db.charm should be cs:precise/mysql17:15
benjiMakyo: Internal Server Error17:15
MakyoErk17:15
benji{17:16
benji  "bundle_id": "~bac/wiki/0", 17:16
benji  "type": "no_such_bundle"17:16
benji}17:16
benjiI don't think the ingest worked.  Most, if not all, of the bundles were rejected by proof17:16
MakyoOh, hmm..17:16
MakyoWonder if mine was out of date.  Let me check.17:16
rick_h_in order for the bundles to ingest the charms involved must be loaded17:17
rick_h_so you have to make sure to get the charms that the bundle references ingested first17:17
hatchjujugui I need a review on https://github.com/juju/juju-gui/pull/164 (big diff, big qa)17:17
rick_h_hatch: so you need two reviews then?17:18
MakyoOh, right, hmm.  I guess I already had those loaded.17:18
hatchrick_h_ well....it's so much in flux still that I think I'll hold off on two until it's all done17:18
hatchbut I definitely want to make sure I didn't introduce a regression on the service config view17:18
hatchI also moved the pt1 card to the review lane 17:18
rick_h_hatch: huh? if it's over 400 lines it's a two review branch. How's it 'hold off on two reviews'?17:19
MakyoAmended QA: run enque with --prefix=~charmers/charms/precise --limit=100 first17:19
MakyoThat should get charms enough for at least those bundles.17:19
hatchwell it's mostly mechanical moves17:19
hatchbut sure if I can get two17:19
hatch:)17:19
hatchI'm held up from continuing on until this review is done though17:19
MakyoI can review now17:20
hatchoh ok cool17:20
hatchthanks17:20
MakyoThat's a big diff :P17:25
MakyoI must fortify myself with more tea.17:25
hatchhaha sorry17:25
hatchthere is a lot of mechanical changes17:25
MakyoI do like this better, though.17:32
MakyoViews are cleaner.17:32
hatchmuch17:32
hatchtrying to wrap your head around viewlet > view databinding however.....not so clean17:32
hatch:)17:32
hatchthere are considerable improvements we can make to databinding now that we are using views17:33
hatcheta on that? never17:33
hatchlol17:33
rick_h_:P17:33
hatchhaha17:37
hatchanyone else use evernote on android? their new 'handwriting' thing is pretty cool - now if I Could only find a quality pen/screen-pen combo17:38
=== lazyPower_ is now known as lazyPower
hatchuh oh, I've now sent 666 tweets17:50
benjiMakyo: still ingesting charms/bundles17:58
Makyobenji, Alright. No rush, I have another card in the mean time.17:59
bachatch: well i hope you take a screenshot before you tweet again18:03
hatchshould I?18:04
bacbenji or Makyo (now that we've claimed you as a charmworld dev), would one of you review https://codereview.appspot.com/69980054/ ?  it started out simple in scope and then i got annoyed and did a bunch of drive-bys.18:04
Makyobac, sure18:04
hatchthat's just under 200 tweets per year....That's actually quite a bit for me, I didn't think I tweetered that much 18:06
Makyo*cough*35,983*cough*18:06
MakyoGetting an intelligent telephone was the worst thing for me. :P18:07
MakyoNever realized how much I missed Rietveld.18:08
hatchMakyo twitter says "You follow each other" about us.....that's a little creepy 18:10
MakyoHaha18:10
hatchyou must have two twitter accounts?18:11
MakyoI have or help manage 14 :P18:13
hatchlol18:13
hatchI unfollow people who tweet more than a couple times a day haha18:14
MakyoThat's fair :)18:14
hatchwhich is kind of unfortunate because there is usually some gems but moreoften than not it just clutters up the feed18:14
hatchmaybe one of these days I'll setup tweetdeck or the like18:14
rick_h_hatch: heh, you need to follow some more python folks to get these gems https://twitter.com/zzzeek/status/44163748964074700818:17
hatchrofl18:17
hatchlove it18:17
hatchrick_h_ I didn't get a +1 from you on the review. I've made the requested changes and pushed them up....is it A O K?18:21
rick_h_hatch: sorry, got sidetracked +118:22
hatchhaha ok thanks18:22
hatchdoes anyone have any ideas for cool hardware hacking stuff? I was thinking quadcopter but also looking for other ideas18:23
hatchleap motion and quad copter would be cool18:24
hatchgoing all minority report flying a camo'd ar drone around the block18:26
hatchlol18:27
bacthanks Makyo.  glad i could let you relive your rietveld days.18:47
rick_h_jujugui added a bug card to critical. It looks like we've got a series regression in charm search causing it to send multiple versions of the same charm overloading the db, front end, etc18:49
rick_h_series/serious18:49
rick_h_bah18:49
bacrick_h_: a charmworld problem?18:49
rick_h_bac: yes, the search api is sending back waaaay too much data18:50
bacbetter safe than sorry?18:51
rick_h_:)18:51
rick_h_"we didn't know if you wanted cinder r15, r16, r17, r18..have them all!18:51
bacrick_h_:  i need to add a new card for the charm-proof change.  it was tiny and  i'm ready to propose it.18:53
hatchrick_h_ this is re the bug I filed about a 5.4MB response? 18:53
hatch:D18:53
rick_h_hatch: yep18:53
rick_h_hatch: finally got around to looking and prioritizing them18:54
rick_h_bac: ok cool18:54
rick_h_hatch: and yea, it's not good18:54
hatchhaha no no it is not18:54
bachey if you add a card directly into a full lane it doesn't ask you for justification!18:54
hatchlooking for a <1s response time....got a 20s response time lol18:54
bacmarcoceppi: could you look at this simple change to charm-tools for 'charm proof'? https://code.launchpad.net/~bac/charm-tools/bundle-proof-warnings/+merge/20975419:10
benjiMakyo: the ingest is still running; did yours take this long?19:11
Makyobenji, no, don't think so.  Can you check if you have mysql and mediawiki yet?19:12
benjicharms or bundles?19:12
MakyoIf so, you can kill, bin/dequeue, run the enqueue in the qa instructions, then ingest again and it'll only ingest the bundles./19:12
MakyoCharms.19:12
benjiI have both.19:13
MakyoShould be able to just load bac's bundles, then.19:13
benjithe API URL for the bundle still 404s though because the ingest hasn't gotten to the bundles yet19:13
benjiwhat do you mean by "load bac's bundles"?19:14
hatchrick_h_ thanks for the mention on discourse re the power button lol19:14
rick_h_hatch: :)19:14
Makyobenji, the commands in the QA instructions.  If you empty the ingest queue with dequeue, then enqueue only his bundles with the enqueue command given, it should only ingest those.19:14
hatchlately no powerdowns because I'm using an external keyboard haha19:15
benjiI think that's what I did.  I'll try again.19:15
marcoceppibac lgtm19:29
benjiMakyo: when I view /api/3/bundle/~bac/wiki/3 I still get the internal error, when I view /api/3/bundle/~bac/wiki/0 I still get the "no_such_bundle" message, however when I fetch http://10.0.3.149:2464/api/3/bundle/~bac/wiki/3/wiki/file/bundles.yaml I see a bundle definition with non-fully-specified charm URLs19:32
rick_h_benji: it needs to be ~bac/wiki/3/wiki I thought19:33
rick_h_benji: ending an api call in the revision isn't a legit api endpoing I didn't think19:33
benjirick_h_: that's what I'm saying19:34
rick_h_what does http://10.0.3.149:2464/api/3/bundle/~bac/wiki/3/wiki return?19:34
rick_h_and what are the charm definitions in ^ (if it loads)19:34
rick_h_in the data: key19:34
benjirick_h_: http://paste.ubuntu.com/7045935/19:35
benjino "data" key19:35
rick_h_sorry, services19:36
rick_h_and that's perfect I think. Makyo confirm? ^19:36
rick_h_benji: line 111319:36
MakyoYeah, that looks good.19:37
benjiheh, darn I searched for "data:"  not '"data":'19:37
MakyoOh, yeah, boo JSON!19:37
rick_h_yea, it's json, must be quoted sorry I wasn't more explicit19:37
benjiQA passed then19:37
rick_h_but yea, all the services are legic cs: urls so yay!19:37
rick_h_land that branch and bury it please! :)19:38
MakyoOkay!19:38
rick_h_Makyo: please make sure to update the bug if there's one on that. I don't see one associated but thuoght there was a bug19:38
rick_h_Makyo: maybe it was on the investigate card19:39
hatchjujugui lf a review/qa on the constraints viewlet to view changeover https://github.com/juju/juju-gui/pull/16519:39
rick_h_hatch: what's lf?19:41
hatchlooking for19:41
hatchsheesh, when did you grow up? that's in the same tmeframe as asl19:41
hatch:P19:41
rick_h_heh, yea I avoided all that19:42
hatchI often had to google the ones gary used19:42
hatch:)19:42
hatchYMMV was a new one for me19:42
rick_h_oh <3 that one. I learned and like WCPGW19:43
rick_h_when I joined launchpad19:43
hatchlolwhat's that one?19:43
rick_h_"what could possibly go wrong?"19:44
rick_h_"change all those viewlets to Views, WCPGW"19:44
hatchlol19:44
hatchtruth!19:44
Makyojujugui it's lbox submit to land a charmworld branch, correct?  Or is there a different step?20:17
* Makyo can't remember20:17
bacMakyo: bzr rv-submit20:17
MakyoAh, thanks20:17
bacusing the rvsubmit plugin20:17
bacMakyo: you can't mess up.  if you try 'lbox submit' it will fail due to permissions.  yay.20:18
MakyoOh, good.20:18
bacyeah, unless you fire-n-forget only to discover much later it errored20:18
MakyoWhoops, conflict anyway.20:22
MakyoSomething keeps revving up my fans. Switched back over to chrome to a notice that "Google talk plugin has crash."  Promising.21:07
hatchyeah that thing is a huge battery hog21:16
hatchsometimes it'll just sit in the background burning cycles for fun21:16
* hatch grumbles about cascading test failures21:43
rick_h_no grumbling 21:45
hatchI'm pretty sure it's ok to grumble about this21:46
rick_h_come on, close to EOD, no grumbling :)21:47
hatchI supppooooose21:47
hatchwell I gots no idears why this one is causing the failure21:51
hatchwell THAT was a stupid mistake21:58
* hatch blames my qa'er for missing it21:58
rick_h_what did I miss? 21:58
rick_h_I even found a bug in the last qa21:58
rick_h_I was feeling good21:58
hatchin the viewlet base view extension I had the _eventHandles and changedValues property as prototype properties so they were shared across every viewlet21:59
hatchwhere they should have been instance level properties21:59
rick_h_oh...hmm guess I didn't test out multiple inpsectors/ghosts and noticed 21:59
hatchit's ok I missed it and I've done 3 branches since then lol22:00
rick_h_heh, better now than figuring it out later I guess22:01
hatchyeah that would have been a tough one to find22:02
hatchso in this case....cascading test failures was a GOOD thing22:02
hatchlol22:02
rick_h_wheeee!22:02
hatchI also found out why my wifi sucked so bad sometimes (even after the new router)22:03
hatchthe wifi antenna must be in the screen of MBP's and when the screen is closed the reception is crap (even though it shows full bars)22:03
rick_h_?22:03
rick_h_oh, interesting22:03
rick_h_I think it just sucks myself. Every time I wake it up it takes so long22:03
hatchI'll test it over the next couple days to see if the slow internet issue comes back after opening the lid22:04
rick_h_I was really hoping I'd fall in love with this air because of the lennovo issues but :(22:04
hatchoh yeah - if it's in sleep mode half the time it won't come back hah22:04
hatchthe good news is that it's a software issue - people who install windows don't have that issue22:04
hatchso a on-metal install of Ubuntu may also solve the issues22:04
rick_h_yea, I'm hoping to compare linux on it22:04
rick_h_yea, we'll see :/22:04
rick_h_but the keyboard and wifi issues on the air really lose it poits22:05
hatchI really hope someone fixes that kernel bug22:05
rick_h_points22:05
hatchoh yeah - and the new MBP has the same keyboard - I thought it was the same as the old MBP keyboards....but I was wrong :(22:05
hatchI was hoping for a little more concave keys with some more travel 22:05
rick_h_exactly, they're too soft and mushy22:06
rick_h_and the travel definitely needs a bit more22:06
rick_h_after a few days I go "oh, it's not so bad"22:06
rick_h_then I pull out my x230 and start typing "whoa, this is so great"22:06
rick_h_I do love the air size though. In the backpack, carrying around the house22:07
rick_h_makes me ponder that 13" xps more22:07
hatchyeah - it's a little sharp to carry around a conference lol22:07
rick_h_but I hear the keyboard there isn't great either22:07
rick_h_if the wifi would come back up faster it'd be a great conference machine22:08
rick_h_but you close it, go to the next room, and open it and hang for 5min before you can get into your google doc22:08
hatchoh mine isn't THAT bad22:08
hatchit just sometimes never comes up22:08
hatchelse it's about 30s to a min22:09
rick_h_it comes up eventually, but only after it hangs and shows that 'finding signal' animation for minutes22:09
hatchinteresting...22:11
hatchI wonder if that's because it's a referb?22:11
hatchjujugui lf a review on the ghost inspector viewlet to view conversion https://github.com/juju/juju-gui/pull/16622:11
rick_h_hatch: will look tonight sometime and try to qa better. 22:12
hatchoh poop22:12
hatchI think I screwed up the commits22:12
hatchsec22:12
hatchok fixed22:13
hatchrick_h_ you know your email is the jujugui bot email :)22:13
rick_h_yea, someone had to sign up22:13
hatchohh ok I thought that was by accident instead of some secondary email22:13
rick_h_hmm, does everyone else not get the pull request messages?22:13
hatchnope22:14
hatchwell..22:14
rick_h_oh, hmm. Maybe I should try to change that then22:14
hatchok no we do22:14
hatchsorry I just always miss them heh22:14
hatchrick_h_ I don't think CI is being run anymore on new PR's23:38
hatchhttps://github.com/juju/juju-gui/pull/166 this is still showing 'Waiting to hear...'23:39
hatchohh23:39
hatchCI is hung23:39
hatchheh23:39
hatchStarted 10 hr ago23:39
hatchBuild has been executing for 10 hr23:39
hatchkilled that one....now lets see if it catches up over the next couple hours :)23:40
rick_h_hatch: cool yea. It did that the other day. There was a nework issue between CI and saucelabs and it just hung23:43
hatchohh, are we being throttled by sauce labs maybe?23:44
hatchtoo many runs on an oss account23:44
hatchmaybe we can get the juju logo on https://saucelabs.com/opensauce :)23:45
hatchohhh https://saucelabs.com/docs/status-images 23:48

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