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

rick_h_Makyo: that error happens once in a blue moon and seems saucelabs related00:31
frankbanguihelp: anyone available for reviewing/QAing https://codereview.appspot.com/83880044 (quickstart)? thanks!13:22
bacfrankban: i will shortly13:32
frankbanbac: thanks13:32
hatchmorning13:56
rick_h_morning hatch 13:57
hatchhey rick_h_ when you did the nr_cpus=1 did you just put that in the list of the commands when editing the startup?13:57
rick_h_hatch: have to put it in the right place13:57
rick_h_where it says splash and such13:57
hatchoh ok, I thought that might have been the case - I coudlnt' find any docs online about it13:58
rick_h_change those to noslash noxxxx and add nr_cpus=113:58
rick_h_nosplash that is13:58
hatchok thanks - man I just want this darn thing on Ubuntu lol13:59
rick_h_lol yea 14:00
rick_h_thanks for following up and trying14:00
rick_h_I've been knocked out dead early this week 14:00
hatchhttps://bugzilla.kernel.org/show_bug.cgi?id=6063514:00
hatchI am using redfind already....not sure how else to do an efi boot14:00
rick_h_yea, I think we need to make that clear14:01
rick_h_the bug linked might not be valid because he's missed we're using refind already14:01
hatchI wish I knew -anything- about kernels lol14:01
kadams54Anyone know why MachineList is a LazyModelList and not just a ModelList?14:05
kadams54Doesn't seem like the performance advantage of LazyModelList would be needed here14:06
kadams54And I'm depending on change events firing on the machines in the list in order to keep the MachineView up-to-date, which LazyModelList makes less likely.14:07
hatchkadams54 there could be thousands of machines14:09
hatchs/could/will14:09
kadams54hatch: scratching my head over YUI stuff - could I have you take a quick look?14:17
hatch2 mins just gota grab my breaky14:18
kadams54np14:18
hatchohh kay link?14:20
kadams54http://pastie.org/private/hu1ljam1js0lhdtxozxdbw14:21
hatchyep14:21
hatchthat's right14:21
hatch:)14:21
kadams54When I create a Machine and add it to MachineList, the raw object that I get back out of the list has a null displayName14:21
hatchtry foos.create()14:22
hatchare the returned values actually strings? or is that just in your paste?14:24
kadams54yes14:24
kadams54actually strings14:24
kadams54Paste is taken directly from web inspector console14:25
hatchdoes create help?14:25
kadams54No14:25
hatchhmm ok lemme look at the lml code14:25
kadams54http://pastie.org/private/qgxsrjhx5canncqqdqn7ug14:25
hatchok I'm going to fire up a jsfiddle here14:26
hatchor jsbin for that matter14:26
rick_h_kadams54: foos.items(0).get('displayName')14:26
hatchit a pojo14:26
* rick_h_ is guessing displayName is an ATTR?14:27
rick_h_oh nvm then :)14:27
rick_h_bad guess14:27
hatchso'ok!14:27
kadams54LazyModelList stores the objects as POJOs instead of full Models for performance reasons14:27
frankbankadams54: the displayName is added to machine objects by MachineList.add14:27
frankbankadams54: it calls _setDefaultsAndCalculatedValues14:27
kadams54Ah14:27
kadams54And that's probably overwriting my "0" with a calculated value14:28
hatchoh there we go :) I was just writing a repro with raw YUI14:28
hatchheh :)14:28
kadams54Which is why we get a "null" string14:28
kadams54Instead of null14:28
hatchthanks for stepping in frankban14:28
hatchbleh lifestyle changes taste so bad...14:29
hatch#dietingisforsuckers14:29
kadams54heh14:29
kadams54Thanks frankban - that was the missing piece14:30
frankbanwelcome14:31
Makyojujugui call in 1014:50
rick_h_I think Makyo has his alarm set for 11min vs 10 :P14:50
Makyo10 minutes by my phone, just whenever that goes off.14:52
Makyo(but now I'm totally doing that.  Thanks rick_h_ :D )14:52
luca__rick_h_: Makyo hatch can we have a call once your off your call? :)14:58
hatchwe can14:59
rick_h_luca__: rgr14:59
luca__thanks14:59
frankbanbac: http://bazaar.launchpad.net/~go-bot/juju-core/trunk/view/head:/doc/bazaar-usage.txt15:01
bacthanks frankban15:04
rick_h_luca__: https://plus.google.com/hangouts/_/canonical.com/daily-standup15:11
luca__just saving my doc15:12
* frankban biab15:22
hatch__brb15:54
frankbanrick_h_: so, should we keep joyent support out for now? I know our exception is for fixing quickstart on newer juju-core. I wonder if that's to be intended as exclusive i.e. only the buf fix, no other developments/changes added in the meanwhile15:58
kadams54I was just getting ready to submit a pull request, but after rebasing to latest on develop, I can't get to the machine view anymore15:58
rick_h_kadams54: hit the machine view tab in the nav15:59
kadams54Yeah, I did.15:59
kadams54Nothing15:59
rick_h_kadams54: oh hmm, that works in comingsoon15:59
rick_h_http://comingsoon.jujucharms.com/:flags:/il/mv/15:59
rick_h_hit machine view button?15:59
rick_h_kadams54: if it doesn't work locally make clean-all maybe? Or check for errors in console/etc16:00
kadams54Will do16:00
kadams54*sigh*16:01
kadams54Error in my code16:01
rick_h_frankban: can you write up an email about the joyent provider now landing and our wish to have that in quickstart to james page and robie and copy me? 16:01
rick_h_frankban: we'll ask if it'll be possible, if not. Let's hold on landing that branch until the bugs land first16:01
rick_h_so we can release just what we promised them, a bugfix release16:01
rick_h_frankban: and if so, then we can follow up with the joyent under whatever rules we can get that in under16:02
rick_h_frankban: and if not, then landing joyent in quickstart will be the start of the new ppa releases, but not in trusty16:02
frankbanrick_h_: sounds good16:02
kadams54guihelp https://github.com/juju/juju-gui/pull/222 is ready for review and QA (list machines in machine view)16:10
Makyokadams54, on it16:11
kadams54thanks16:14
rick_h_jujugui going to go afk for some lunchable. biab16:21
hatchenjoy16:21
frankbankadams54: so, do we display both top level machines and containers in that part of the machine view?16:22
kadams54Not entirely sure. There's a machine column and a container column, so I'd guess so?16:23
kadams54My card was for listing the machines in the machine column, so the container column is still non-functional16:23
frankbankadams54: if I read your diff correctly, you are listing all the machines (db.machines.toArray()). That would include also the containers. It is possible to only list top level machines by using machines.filterByParent(null)16:26
kadams54Ah, did not realize the collection contained both machines and containers…16:27
kadams54Is there a reason containers aren't in a separate collection?16:27
frankbankadams54: it's not a separate entity in juju-core itself, and also they share almost all the properties. it's a tree, where a phisical machine can host containers and containers can host other containers etc.16:28
hatchrick_h_ “Arguing with an engineer is like wrestling with a pig in mud – you realise far too late it is enjoying it”  rofl16:38
kadams54frankban: got it. Will modify the code accordingly.16:45
frankbankadams54: cool thanks16:45
kadams54hatch: Nice. I'll have to remember that one.16:46
hatchI want a tattoo of that haha16:46
kadams54Your wife might argue with you about that.16:46
hatchwhat if it's in Japanese? Chinese? Then I'd be part of the cool kids 16:47
hatchI'd just tell everyone it means "Love"16:47
hatchlol16:47
hatchMakyo I just booted up vagrant and it's creating a new image.....16:49
hatchany idea how I would point it to the old one again?16:49
MakyoUm.16:49
MakyoI think it's in a .vagrant or something folder in ~?16:50
rick_h_hatch: :)16:51
frankbanrick_h_: had a chat with robie in #juju, joyent is out, the other parts of the plan sound great to him. I think landing joyent later next week is fine.16:52
rick_h_frankban: rgr thanks16:52
hatchrick_h_ want to have a quick discussion about this branch17:00
rick_h_hatch: sure link17:10
hatchhttps://plus.google.com/hangouts/_/7ecpj9soau6943l97kuv7qnknc?authuser=1&hl=en17:10
bachi frankban, when bootstrapping joyent, after creating an environment with quickstart, i get http://paste.ubuntu.com/7199671/17:23
frankbanbac: uhm, is this the 1.18 branch?17:24
baci think so.  let me double check17:24
bacfrankban: yes, it is using the version i built using the 1.18 branch.  however, 'juju version' shows 1.19.0-trusty-amd6417:25
bacfrankban: i merged 1.18 into my lw checkout of juju-core trunk.  is that correct?17:26
frankbanbac: I don't think so, you should just branch the 1.18 branch and switch to that. I guess trunk includes other changes17:28
frankbanbac: my 1.18 branch is 1.17.8-saucy-amd64, bzr revno 225717:28
bacfrankban: i'm unsure how to do that with lightweight checkouts.  usually i do a 'bzr switch -b', which gives me a copy of trunk.  how would i get a checkout of 1.18 in place?17:29
frankbanbac: also, before compiling "godeps -u dependencies.tsv" can be a good sanity check (go get launchpad.net/godeps)17:30
hatchpoor bws-icon17:32
hatchdelete!17:32
frankbanbac: I followed the bzr docs in trunk, and so I created a shared repo outside the $GOPATH, in ~/devel/juju-core. There I can just bzr branch some-branch. at that point you can "bzr switch {target}" from the juju-core in $GOPATH/src/...17:32
bacfrankban: ah, gotcha.  i'm used to just working in the other workspace17:33
frankbanbac: yeah the configuration in that doc is cool because you have both the structure go requires ($GOPATH/src/launchpad.net/juju-core) and a real shared repo to work on, given that the former is bound to the real repo as a lightweight checkout. So, basically, $GOPATH/src/launchpad.net/juju-core is just what we usually call "sandbox"17:35
bacfrankban: right.  i just didn't know you could manually insert a different branch in ~/src/juju-core17:36
frankbantime to go, have a nice evening17:36
bacfrankban: wait!17:37
* frankban waits17:37
bacfrankban: for joyent are the sdc user/keys the same ones for manta?17:37
frankbanbac: yes17:37
bacok.  perhaps we can make that clearer in the field descriptions17:37
bacor maybe make the manta ones optional and force them to be the same if blank17:38
frankbanbac: yes, that can be an improvement. FWIW my envs.yamls looks like http://pastebin.ubuntu.com/7199750/17:39
bacfrankban: if you're still around have you seen this before http://paste.ubuntu.com/7199773/17:46
kadams54Makyo, frankban: FYI, updated https://github.com/juju/juju-gui/pull/222 to only pull top-level machines. Also realized I had one commit that didn't get pushed.17:47
frankbanbac: no, but I guess your previous attempt left something there, and juju-core got confused by the binary change.17:47
Makyokadams54, cool, thanks.  Was having trouble on this comp, switched to Air to find vagrant out of date.  Will switch back.17:47
bacfrankban: huh.  ok, i'll poke around.  thanks.  won't bug you again.  good night.17:48
frankbanbac:  you can try "juju destroy-environment joyent -y --force" and check nothing is in ~/.juju/environments/17:48
rick_h_kadams54: tossed a few comments in for fun 17:48
rick_h_kadams54: which might be out of date now it sounds like17:48
=== lazyPower is now known as lazyPower-travel
frankbanbac: cool, there is no rush in landing this branch, so if you are blocked we can take a look at that together tomorrow17:52
kadams54rick_h_: yeah, saw those. I caught the constructor one early enough, but will update the tests as requested on the next pass through the code (after another other reviews).17:53
rick_h_kadams54: cool17:53
rick_h_kadams54: remind me in the call I'll mention how the initializer and some other 'magic' methods work17:53
hatchthink C classes17:54
hatchthat's how initializers work :P\\17:54
kadams54hatch: I haven't programmed in C since college, over a decade ago :-)17:56
rick_h_hatch: right but other things like render and such can be cascaded (at least for widgets/etc)17:56
hatchbase derrived classes yeah17:56
hatchkadams54 haha yeah same here17:56
kadams54Yeah, I was thinking that constructors needed to cascade when I wrote that17:56
kadams54Then I remembered this is YUI, not plain JS.17:57
rick_h_there's a good yuiconf talk on video for how base works 17:57
kadams54And this was an initialize method, not a constructor17:57
kadams54Hit me with the link17:57
hatchrick_h_ kadams54 https://docs.google.com/file/d/0By84yWo_5i6LYXNsdnhHV2Z2TUU  aww yeah C++17:59
hatchsee that one has a 3.5" floppy on it!18:00
hatchlol18:00
rick_h_kadams54: https://www.youtube.com/watch?v=_zhQIfT7g58&index=6&list=PLF49A40DA7A11F08A ftw18:18
kadams54Woot18:19
hatchohh yeah I remember that talk18:19
rick_h_one of my fav yui talks18:19
hatchyeah luke is great18:19
hatchat smugmug now18:19
bacrick_h_: just a heads up, manage.jujucharms.com/heartbeat currently shows an API2 failure.  it is a false positive caused by us currently not featuring any charms, just bundles.18:54
MakyoNo plumber yet, but we did get our new doors delivered.  Bonus.18:57
hatchMakyo might be time to fire this guy18:59
hatch:)18:59
bacMakyo: so you've had this leak at the meter for weeks now?18:59
MakyoYep.18:59
bacyowza18:59
MakyoI just got a water bill, actually.18:59
Makyo$286.0419:00
Makyohttps://www.dropbox.com/s/jregju31mdzuuhc/IMG_20140403_130049.jpg19:01
hatchouch!19:02
hatchrick_h_ https://github.com/juju/juju-gui/pull/223 WIP 19:06
hatchthis doesn't have the tests fixed but I'd like some third party input19:06
Makyohatch, ping when you have a second.19:18
MakyoWant to get a good ECS card.19:19
hatchhmm I'm just cooking lunch atm19:19
hatchcan we do it in 45mins or are you not doing anything now?19:20
MakyoWill do lunch too, finish this review.19:20
Makyo45 mins is fine19:20
hatchsounds good19:20
bacrick_h_: i'm stuck working with the juju guys trying to get the joyent provider to work.  that's kind of like QA, isn't it?  :)19:35
rick_h_bac: hah, sorry catching up19:47
rick_h_bac: oh lol for your qa day, yea we'll call it19:49
hatchMakyo whenever you're ready, rick_h_  you may want to be part of this discussion as well20:02
Makyohatch, sure20:03
rick_h_hatch: rgr20:03
hatchcreating room20:03
hatchhttps://plus.google.com/hangouts/_/7acpjdgv54v5pnjt0fc9v0snjs?authuser=1&hl=en20:03
hatchrick_h_ btw there have been some developments wrt the kernel bits - tonight I'm going to give them a try20:17
rick_h_hatch: thanks, let me know how it goes20:17
hatchapparently both of us have been installing Ubuntu incorrectly 20:17
hatchwe need to use EFI mode instead of legacy20:18
rick_h_yea, not sure how that happened 20:18
hatchyeah I'll definitely keep notes20:18
rick_h_didn't really see an 'efi' mode during install from the usb cd20:18
hatchyeah neither did I...20:18
hatchw00t down to 9 failures20:40
rick_h_hatch: that sounds darn right reasonable :)20:45
hatch:shipit: amiright?20:45
rick_h_and lies20:45
hatchoo the tests found a real bug20:47
hatchlookatthat20:47
rick_h_#so-thats-how-it-works :)20:48
hatchhaha20:49
rick_h_as much I tests are a pita, I so love them when they catch a bug before it lands20:49
hatchsomhow I deleted three tests and fixed one and still have 8 failures....20:57
=== tvansteenburgh is now known as tvan-afk
hatchrick_h_ in test_browser_app.js browser subapp display tree there is a `hits` object21:36
hatchwhy do they default to true?21:37
hatchThat seems backwards?21:37
hatchI feel like I'm missing something21:37
rick_h_hatch: k21:37
rick_h_hatch: they default to true?21:38
rick_h_hatch: there's a 'resetHits() method21:38
hatchohh ok I broke something21:38
hatchright21:38
rick_h_that gets called in setup and manually for multi dispatch tests21:38
hatchright21:38
hatchsorry21:38
hatchhmm21:38
hatchtest failures when things work in the real world...not fun21:41
rick_h_hatch: let me know if you want another set of eyes21:41
hatchthanks but I just skipped the final two because they would need state21:48
hatchrick_h_ ok pushed....would be awesome if you could find some time to qa/review21:55
hatchkeeping in mind the follow-up will be to properly implement the state21:55
rick_h_hatch: rgr21:55
hatchthanks21:55
rick_h_yea, realize the whole 'test and build state' card here didn't get there because of the viewmode removal21:55
rick_h_but smaller branches == better21:55
hatchyeah agreed21:56
hatchI did remove over 1000 lines in total21:57
hatchthat's awesome lol21:57
rick_h_yea, I mean this is good, just time consuming 21:57
rick_h_between this and inspector we're paying a price for the better code hopefully21:57
rick_h_but let's try not to find anything else to rewrite :)21:58
hatchlol is there anythign else?21:58
rick_h_shush, you'll send back luck our way21:58
hatchhaha21:59
rick_h_hehe...except I'm not kidding 21:59
huwshimiMorning22:01
rick_h_morning huwshimi 22:01
huwshimirick_h_: Has our ci broken?22:03
rick_h_huwshimi: not that I'm aware of. What's up?22:03
rick_h_huwshimi: you had a legit failing test. You replied that it could be removed, but never updated and removed the test22:03
huwshimirick_h_: Well, all the current MPs have builds that have started but not completed.22:03
rick_h_huwshimi: I was going to ask what happened yesterday and if there was something that got stuck22:03
rick_h_huwshimi: oh heh, yea that's broken22:04
rick_h_sec, have to stop22:04
rick_h_huwshimi: should be good to go hopefully22:05
huwshimiAh great22:05
rick_h_hatch: replies in bound, will start qa22:15
hatchthanks I'll take a look22:15
huwshimirick_h_: What should I tackle today? I see kyle is already working on the machine panel tokens...22:15
rick_h_huwshimi: he's working on the data, but not the tokens22:16
rick_h_bah, he's gone. I thought he would get that landed22:16
rick_h_huwshimi: what were you working on yesterday? Is that done then?22:16
huwshimirick_h_: Ah, I thought there were two cards about tokens and now there's only one so I assumed he must have done it. I must have mis-remembered.22:16
rick_h_huwshimi: ah well his card says 'using deployed unit token' but it's not ready yet so he's just dumping out text22:17
rick_h_huwshimi: so I'd say to start looking at the token I suppose. 22:18
rick_h_huwshimi: https://drive.google.com/a/canonical.com/?urp=https://docs.google.com/&pli=1#folders/0B_l975nRB6BuM2p4eTZMSXpWRTQ22:19
rick_h_we can go over them during hte call22:19
huwshimirick_h_: That'd be great, thanks!22:20
rick_h_hatch: qa notes added22:28
rick_h_hatch: I ack that not all are in/for this branch. Just noting things that are broken. 22:29
hatchok cool22:29
hatchlooking22:29
hatchline reflow?22:30
rick_h_otp22:31
hatchI actually prefer when there isn't one long line and then another word on the next line :)22:31
rick_h_hatch: oh yea, well the limit is 79, not 20 and 1522:31
hatchlimit.....not required :P22:31
rick_h_it looks odd imo and breads readability and I bet one or two of those will fit on one line now22:32
rick_h_fighting on everything :P22:32
hatchhah no it's just going to take me a while to reflow all of these22:34
rick_h_open in vim, run gq22:34
rick_h_it's one command :)22:34
hatchthat doesn't require a plugin?22:35
rick_h_no, I don't believe it does22:35
hatchhmm ok I'll have to download vim22:35
rick_h_http://www.methods.co.nz/asciidoc/chunked/ch36.html22:35
rick_h_w...t...f doesn't it exist in your virtualmachine?22:35
hatchor did you just want me to do those two lines22:35
rick_h_ssh, vim xxxx.js22:35
hatchnot the entire file?22:35
rick_h_the cases of those comments 22:35
hatchoh ok, then done22:35
hatch:)22:35
rick_h_some of them jump out as looking stupid so I noted them in review22:36
hatchthat's an interesting functionality of vim22:37
hatchto reflow a line22:37
rick_h_very handy when editing blocks of text/comments and emails22:37
hatchyeah if you don't use a real client :P22:38
hatchrick_h_ so changes are made - I'm not clear on your qa notes, did you want me to fix those things?22:47
rick_h_hatch: https://plus.google.com/hangouts/_/calendar/cmljay5oYXJkaW5nQGNhbm9uaWNhbC5jb20.dd77sn7kjl6unba21lutdr0p70?authuser=122:51
hatchis there an aus call?23:07
hatchor did I miss it?23:07
hatchachievement: Ubuntu 14.04 on MBP23:38
rick_h_hatch: missed it, it before you and I talked23:46
huwshimihatch: Did you get all the touch packages installed as well? No idea why I have them...23:50
hatchhuwshimi I don't know what that is23:51
hatch:)23:52
hatchevery few minutes it pops up with a system error report23:52
hatchso it's not really usable 23:52
huwshimihatch: You don't know what Ubuntu Touch is?23:52
hatchand I can't actually shut it down outside of the terminal either23:52
hatchoh the Ubuntu Touch packages23:52
hatchI don't think I got those23:52
huwshimiSo strange...23:53
hatchI have to find where I file bugs23:53
hatchthe inability to shut down is kind of a big one23:53
huwshimihah23:54
hatchswitching to wifi23:55
rick_h_sudo shutdown -h now23:56
rick_h_or per zsh alias 'shut'23:56
rick_h_kadams54: hey, around?23:56
kadams54I am now :-)_23:56
rick_h_kadams54: how's your branch? Is it ready to land? huwshimi is doing some work with the token part that will build on it23:56
rick_h_kadams54: I had thought your branch might get in before he got running23:57
kadams54Ah, yeah… let me get that pushed out23:57
rick_h_kadams54: ty much23:57
rick_h_huwshimi: ^23:57
kadams54np23:57
rick_h_huwshimi: what did you need to do? Just install the packages mentioned in the bug report?23:58
huwshimikadams54, rick_h_: Brilliant, thankyou!23:58
huwshimirick_h_: Somehow when I upgraded to Trusty it installed all the Ubuntu Touch packages as well.23:59
rick_h_huwshimi: hah23:59
hatch__huwshimi does it do anything different with those packages?23:59

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