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

=== rogpeppe is now known as rogpeppe1
hatchmorning13:12
rick_h_party 13:12
rick_h_hatch: does that mean I have a +1 for the docs update?13:13
rick_h_or more notes inbound?13:13
hatchoh....woops I meant to add the +1 to the initial comment :)13:13
hatchI don't use the aliases anyways so I'm a horrible person to hold up changes to them :)13:14
rick_h_heh that's ok13:15
rick_h_I want to get all opinions and such13:15
rick_h_but hope this will help the immediate issues for everyone13:15
hatchyep it's a good thing to have in there for sure13:20
lazyPowerCan charms have multiple maintainers? and if so, what is the definition style in metadata.yaml for said multiple maintainership declaration?14:12
hatchlazyPower I'm pretty sure, no14:14
hatchalthough that's definitely something that we should have14:14
marcoceppilazyPower we need to propose to the list, get concensous on how to do it (shouldn't be hard) then have that made known to the teams and have them implement it as needed14:14
lazyPowerfiling a bug14:14
lazyPowerwould that go against -gui or -core? assuming -gui 14:15
hatchwell...14:16
marcoceppilazyPower: charmworld and gui probably14:16
hatchcharmworld needs to be able to parse it14:16
hatchtoo14:16
hatchyeah14:16
lazyPowerack. 14:16
lazyPowerI'll tack that on to both and reference14:16
hatchugh osx is such a steaming pile14:26
hatchyay local charm upgrade works on a real env14:30
hatchrick_h_ this is a much better UI than dragging on the inspector14:31
hatch+114:31
bachey hatch, can you give me some git help?14:33
hatchbac I can try :)14:34
hatchshoot14:34
bachatch: i have a branch 'constraint-export'.  i pushed it to github and the commit shows up in my 'develop' branch up there14:34
bachatch: i want to create a pull request14:34
hatchcan you link me your fork?14:34
hatchjust so I can see the repo structure14:34
bachatch: https://github.com/bac/juju-gui/tree/develop14:35
hatchbac so you need to do a PR from a branch14:36
hatchoh that's not entirely true14:36
hatchyou can specify a commit....but...14:36
hatchwhen you do `git branch` does it show your branch locally?14:36
bachatch: if i try to do a pull request it shows changes from other contributors going back to dec-1214:36
bacso is it not updated to the project tip correctly?14:36
bachatch: yes, git branch shows this branch, develop, and two others14:37
hatchlets have a hangout14:37
back14:37
hatchhttps://plus.google.com/hangouts/_/7acpi3t6o8bq2dmt0gsebvqj9s?hl=en14:37
rick_h_hatch: cool on the upgrade. I sent  luca__ a screenshot of the deploy/upgrade inspector in our call today and they'll look at it in the next week or so. 14:42
rick_h_lazyPower: hatch yea, we'll have to update charmworld/gui to handle it but nothing crazy to support. 14:46
lazyPower+1 i've got an open thread on the mailing list for discussion.14:47
rick_h_lazyPower: yep, tracking14:47
lazyPowerwith a glorious typo in the subject line14:47
* lazyPower facepalms14:47
lazyPowerfrakkin mondays14:47
benjifrankban: what version of ubuntu are you using for your GUI development?14:50
frankbanbenji: saucy14:50
rick_h_trusty here with a saucy lxc for charmworld14:51
benjithanks; I'm still on raring and the latest nodejs package isn't available for it14:51
hatchrick_h_ have a second to help bac with some git issues?14:51
rick_h_hatch: yep14:51
rick_h_hangout above?14:51
hatchhttps://plus.google.com/hangouts/_/7acpi3t6o8bq2dmt0gsebvqj9s?hl=en14:51
frankbanyeah, saucy host + saucy LXC for the GUI14:51
benjirick_h_: yeah, I think I'm going to go straight to trusty14:51
benjiI really didn't want to do an OS upgrade today14:52
rick_h_benji: heh, welcome back...here's a ton of packages!15:15
hatchwb benji 15:15
benji:)15:15
bachey rick_h_, when i pushed it went to my develop branch on github again.  http://paste.ubuntu.com/7028047/15:23
bacrick_h_: should i have had a colon in that command, as shown further down in the hacking doc?15:24
rick_h_bac: can you paste your .git/conf?15:28
rick_h_.git/config15:28
bacrick_h_: http://paste.ubuntu.com/7028077/15:29
rick_h_bac: ok, so in there your merge = refs/heads/develop is setup to push to develop15:29
rick_h_so now the question is why is that set that way. 15:29
rick_h_how did you create hte feature branch?15:29
frankbanguihelp: I need two review + QA for https://github.com/juju/juju-gui/pull/152 anyone available? thanks!15:29
hatchfrankban I'll take one once I finish my tests15:30
hatch~20mins15:30
bacrick_h_: here is all that i did http://paste.ubuntu.com/7028092/15:30
frankbanhatch: thank you15:31
rick_h_bac: k doing a local test in a new directory here as well15:31
rick_h_bac: I dupe the issue here, looking into it15:34
hatchoops15:45
hatchhere we are thinking everyone else is running the commands wrong15:45
hatchwhen the commands were wrong :)15:45
rick_h_hatch: this is different15:46
rick_h_hatch: and at one time worked. I'm wondering if something changed in git in trusty or something15:46
hatchohh that might be it15:46
hatchgit version 1.8.1.215:47
hatch^ that's what I'm using fyi15:47
rick_h_hatch: I've been doig git push origin $newbranch for years now without issue15:47
rick_h_suddently that's going to the remote develop vs tracking a new branch on the remote15:47
hatchhmm i've been always doing it long handed15:47
hatchyeah15:47
rick_h_long hand being git push origin branch:branc15:48
rick_h_h ?15:48
hatchgit push origin upgrade-from-local-charm:refs/heads/branch_to_create15:48
hatchyeah15:48
rick_h_ugh15:48
hatchbut now I use grb so I don't need to do that anymore15:48
hatchjust `grb create foo`15:48
rick_h_until git changes something that breaks grb :P15:48
bacrick_h_: as i said, trusty has git 1.9.015:49
hatchwell it does everything -very- explicitly.....but now that it's no longer maintained - that is a very real possibility haha15:49
rick_h_bac: yes looking. I can dupe, will try to get an update once I figure out what's changed/wrong15:49
rick_h_jujugui call in 1015:50
rick_h_git push origin master or more generally any git push $where $what has been enhanced to use the same ref mapping git push origin would use to determine which ref at the origin is to be updated to the local master. For example, with this configuration:15:51
rick_h_so git push origin xxx got some changes15:51
rick_h_http://blogs.atlassian.com/2014/02/whats-new-git-1-9/15:51
rick_h_bah, so the 'enhancement' brreaks how we set it up to be used lovely15:53
rick_h_bac: sorry, you're the first one to hit 1.9 and find the change 15:53
bacred shirt15:53
bacrick_h_: nice find, though.  now we know what's happened15:54
rick_h_yea, I'll update the docs and mention it on the call15:54
rick_h_you have to be more verbose15:54
rick_h_git push origin branchname:branchname15:54
rick_h_should tab complete at least 15:54
hatchjujugui lf a review/qa on real env https://github.com/juju/juju-gui/pull/15315:54
rick_h_at least it does in zsh15:54
bachurrah for DRY15:55
bachurrah for DRY15:55
rick_h_no kidding15:55
rick_h_love how they call it 'enhanced' and all I can think is "more explicit"15:55
bacrick_h_: so my develop on github is screwed again, so i need to start over?15:55
hatchrick_h_ just fyi - tab complete does not work on osx unless you have installed an extra package15:56
hatchel stupido limitationo15:56
bachatch: do you use the github tool on os x?15:57
rick_h_hatch: sucks to be osc15:57
rick_h_osx15:57
hatchbac I don't... tbh I didn't even know there was one :)15:57
benjihmm, I don't see the daily meeting on my calendar any more15:58
bacthere is and it looks nice.  i haven't played with it much since i don't develop on os x15:58
benjioh, there it is15:58
rick_h_jujugui call in 215:58
hatchI wont' be dev'ing in osx either once rick_h_  confirms they fixed the kernel issue for osx15:58
hatcher apple15:59
rick_h_my ethernet adapter is on the way15:59
hatchyay15:59
bacrick_h_: is there any easier way for me to get github untangled than killing everything and starting over?16:13
rick_h_bac: 'easier' if you have a better feel what you're doing. If you don't I worry it'll be messier16:14
bacer, ok16:14
rick_h_bac: if you want to chase the other way I can walk you through it :)16:15
bacnah16:15
hatchbenji did you do a qa on frankban's branch?16:28
benjihatch: nope; shall I?16:28
hatchI can, I just didn't want both of us to do it :)16:28
benjik16:29
hatchfrankban the qa can be done on lxc right?16:29
frankbanhatch: no, it requires ec2 and juju-core trunk16:29
frankbanhatch: core does not support lxc inside lxc16:29
hatchohh ok16:29
hatchfrankban do you know if 1.17.4 will work?16:30
frankbanhatch: I don't know for sure16:31
hatchok np thanks, I'll get on the qa now16:31
hatchdo you want me to shipit, if everything is good?16:31
frankbanhatch: sure thanks16:32
hatchCurrently Playing: Good Vibrations (feat. Loleatta Holloway) - Marky Mark and the Funky Bunch16:33
hatchaww yeah16:33
hatchlol16:33
hatch90s ftw16:34
rick_h_never been so glad to work from home16:34
hatchrofl16:34
frankban:-)16:36
hatchI can see frankban rockin out to some Marky Mark16:36
hatchrofl16:37
hatchoh man the 90s16:37
MakyoSo. If ever you need to move your VMs, make sure to try and figure out vboxmanage (or at the very least kill VBoxSVC before you edit VirtualBox.xml)16:38
hatchuh oh16:41
hatchdid you break something?16:41
hatchjust in case you want to take your laptop on your river rapid tour http://www.stormtech.ca/wlc-1-stormtech-waterproof-laptop-carrier.html lol16:49
MakyoYeah, but I fixed it :P  Just note that if you have the VBoxSVC running, all of your changes to your config file get overwritten as soon as you start virtualbox again.16:50
hatchthat's interesting16:50
bacjujugui: simple pull request: https://github.com/juju/juju-gui/pull/155 for one review / QA16:50
frankbanbac: on it16:54
frankbanbac: when I export the bundle I see comma separated constraints17:01
hatchMakyo in my branch review rick_h_ asked about a bug with upgrade charm ui listing 'old' charms which don't exist....do we have a bug for that? I thought so but I can't seem to find it17:36
Makyohatch, yes.  Someone made a non-linear version thingy.17:37
MakyoOne sec.17:37
rick_h_hatch: if you find that can you make a card in the maint. backlog for it as well please?17:37
hatchyou bet17:38
rick_h_Makyo: hatch luca is going to look at upgrade UX in the next week or two as well :)17:38
hatchawesome17:40
Makyohatch, https://bugs.launchpad.net/juju-gui/+bug/124692817:40
_mup_Bug #1246928: Do not offer to upgrade/downgrade local charms <juju-gui:Triaged> <https://launchpad.net/bugs/1246928>17:40
hatchfrankban you still around?17:42
hatchMakyo thanks!17:42
frankbanhatch: yes17:42
hatchfrankban when running app.env.deploy('cs:precise/wordpress-21', 'wordpress', null, null, 1, null, '1/lxc/0'); I get an error in the console which is....17:43
hatchRelation added without matching service17:43
hatchmodels.js:81817:43
frankbanhatch: is the service deployed?17:43
hatchit is deploying.....17:44
hatchsec checking status17:44
hatchhttps://gist.github.com/hatched/933038117:44
hatchfor the wordpress machine17:44
frankbanhatch: if so, that's an error in the stream handlers, maybe we'll have to sort the messages by entity. 17:45
hatchok so is that a qa issue for this branch?17:45
frankbanhatch: this is not related to the branch, could you please file a bug with the stacktrace and steps to dupe?17:45
hatchsure17:46
frankbanthanks17:46
hatchfrankban https://bugs.launchpad.net/juju-gui/+bug/1287283 just fyi17:53
_mup_Bug #1287283: deploying service to lxc container throws relation error <juju-gui:New> <https://launchpad.net/bugs/1287283>17:53
hatchfrankban +1'd17:55
frankbanhatch: thanks17:56
bacsorry frankban, a set of changes must've got lost in my github shenanigans.17:58
hatchrick_h_ should this go in the Pool or On deck? 17:58
frankbanbac: np, I only see constraints parsing related code indeed17:58
hatchI'm guessing pool?17:58
rick_h_hatch: stick it in the on deck atm and I'll look otp17:58
hatchnot entirely sure what the difference is 17:58
rick_h_on deck is the limited stuff I pull out of the pool to poker plan on 17:59
hatchok cool, added17:59
rick_h_hatch: so that we don't go thorugh everything ever to do 17:59
hatchdefinitely a good idea with our backlog haha18:00
rick_h_:)18:00
bachey hatch, during my git branch recovery, the patching dropped one set of changes.  i just re-applied them but when i try to push to github it says the branch there is newer and needs to be merged.  that isn't true.18:07
hatchlooking18:08
bachatch: i don't want to get messed up again18:08
hatchbac did you rebase your fix in?18:08
bacyes18:08
hatchok then you need to use force18:08
hatchgit push -f18:08
hatchbut probably best to be explicit18:08
bacok18:09
hatchgit push origin constraint-export -f18:09
hatchassuming your origin is your fork18:09
hatchthere you go18:09
hatch:)18:09
hatchhttps://github.com/juju/juju-gui/pull/155/files18:09
hatchall good?18:09
bacyeah, thanks18:10
bacfrankban: the lost changes are reapplied in https://github.com/juju/juju-gui/pull/155 : model.js and test_model.js are the only two that changed18:11
hatchbac, np, just be VERY careful with -f because we have the ability to force update develop which will blow out everyone elses stuff causing us to fix it :)18:12
frankbanbac: redoing QA18:12
frankbanbac: done18:24
bacthanks frankban.18:25
frankbanyw18:25
jcastrohey bac18:47
jcastroyou had a launchpad link to show all the bundles, have that handy?18:47
rick_h_jcastro: https://code.launchpad.net/charms/bundles ?18:48
jcastrota18:48
hatchrick_h_ I'm going to be writing the integration tests then onto 'real' work again :) is there a Project you would like me on? 1 vs A?18:52
hatchI'm thinking A18:52
rick_h_hatch: yep, souds good. I think it's something you wanted to get involved with19:01
rick_h_hatch: let me know if we need to pre-imp or anything19:02
hatchsure19:02
bacjujugui, need another review for https://github.com/juju/juju-gui/pull/155 since frankban has EODed.  rick_h_ if you have just a sec to weigh in that would be appreciated19:12
rick_h_bac: rgr, looking19:13
rick_h_bac: commented19:20
bacrick_h_: ok, so you get it a +1, but with a request to do something.  is it going to start merging?19:22
rick_h_bac: no, :shipit: is the merge command19:22
rick_h_bac: :+1: is the LGTM 19:23
bacrick_h_: so you want me to un-promise the three tests i touched, right?19:23
rick_h_bac: yes please. 19:24
rick_h_bac: heads up on test failure due to make lint19:36
bacrick_h_: thx.19:36
hatchbac heh yeah, you'll get used to running `make lint && make test-debug` before your final commit :D19:49
rick_h_meh, lander catches it quickly and emails you. not the worst thing19:49
bachatch: oh, i'm used to it19:50
hatchno but then you have to rebase commit bleh bleh bleh19:50
hatchman sometimes I wish our yui modules would just fall over if their local containers didn't include the proper deps19:52
bacso if you make incremental changes during code review, do you rebase before pushing each up?19:52
hatchbac I do, so I don't have a bunch of 'lint' commits in trunk19:52
bacisn't that what messes up github causing you to 'push -f'?19:52
hatchyes19:52
bacit seems you should only rebase before the first push and landing19:53
rick_h_that's not a "mess up" it's ok to push -f19:53
rick_h_it's only effecting your local branch19:53
bacrick_h_: oh, i thought it was like '--overwrite'19:54
bacin bzr19:54
hatchwell it is - if you use -f on anything other than your current branch19:54
rick_h_bac: well in a way it is, but since it's just your local space it's safe. It's one of the reasons we all work in our local forks 20:05
benjiaaaand I'm not back20:21
benjiinstall crashed and took my OS partition AND MY HOME PARTITION20:22
rick_h_benji: doh!20:23
hatchbenji what upgrade was this?20:24
bacsee hatch, this here is just a lie: https://github.com/juju/juju-gui/pull/155/commits20:24
rick_h_bac: but it's all the lie we care about20:25
hatch:) git is really just a database20:25
bacsince proposing the branch i've pushed two subsequent revisions.  now the history, even in the review, is collapsed20:25
benjihatch: upgrading my laptop to trusty20:25
hatchbenji from?20:25
benjiraring20:25
rick_h_benji: fresh install time?20:26
hatchoh, well that's no good, were you able to file a bug?20:26
benjirick_h_: I'm speaking to you from a fresh install now. But I'll have to re-install because it installed to my HOME PARTITION... stupid stupid thing20:26
rick_h_benji: oh20:26
bacrick_h_: would you look to see if i unpromised that mess correctly and gimme a shipit if so?20:26
rick_h_bac: sure thing looking20:27
benjithis will take so long to recover20:27
benjiok, off to install again (or maybe munge paritions since I have nothing left to loose)20:27
benjiif I'm not back by the morning send a search party20:27
rick_h_hah20:28
bacthanks rick_h_.  god that was harder than it should've been.20:33
rick_h_bac: thanks for bearing throughit. Hopefully it'll go smoother since we found the new git issue, you've gone throgh the process, etc20:34
rick_h_have a good day all /me runs away20:44
hatchjujugui lf a quick review of my integration tests20:45
hatchhttps://github.com/juju/juju-gui/pull/15620:45
hatchso far that card was considerably less work than we estimated20:45
hatchlol20:45
hatchhelps that I wrote it once before ;)20:45
rick_h_hatch: will look in a few min22:01
hatchcool thanks22:01
rick_h_and hey, card can take less time :) we'll need it for the longer ones22:02
hatchhaha22:06
hatchI was hoping to just take the extra time off?22:06
hatchlol22:06
rick_h_hatch: hah, that's why we have slack cards22:12
rick_h_hatch: so ignore the 'thinking out loud' on the helper stuff. Let me know if I'm nuts on any of the rest22:23
hatchok cool looking22:24
hatchrick_h_ replied22:26
hatchI'll add some periods and the like then land it22:26
hatchthanks 22:27
rick_h_hatch: k, more meant commend on the argument objects you're building vs the "calling callback" which is ok22:29
hatchohh ok22:30
hatchyeah I can add that22:30
rick_h_it reads like "ok, yep you stub that, set the 4th arg to something, whatever that is. I guess it works."22:31
hatchI found out our elliptical takes D batteries....I'm not even sure where to get those anymore lol22:31
rick_h_harder to give real review feedback without going back/forth a ton22:31
hatchhaha - yeah ok22:31
rick_h_lol22:31
rick_h_I've got enoloops with a case around to fit C and D22:31
hatchoh thats a nice idea, I have some Duracell 'Staycharged' but no cases22:32
hatchI should look for some of those22:32
rick_h_so get http://www.amazon.com/eneloop-Capacity-Pre-Charged-Rechargeable-Batteries/dp/B0058GZWXQ/ref=sr_1_1?s=electronics&ie=UTF8&qid=1393885972&sr=1-1&keywords=eneloop+max 22:33
rick_h_and http://www.amazon.com/eneloop-SEC-DSPACER4PK-Spacers-battery-cells/dp/B002V8C3W2/ref=pd_sim_e_16?ie=UTF8&refRID=0Y7QENAYX6Y1ERTSG2GT22:33
rick_h_they don't last as long since they're AA in the core, but they're rechargable so you charge your thing once a month or whatever22:33
hatchyeah that's cool thanks22:33
hatchI'll see if amazon will sell to the great white north22:34
rick_h_hah22:34
hatchthe last few things I tried to buy from .com they woudln't ship here22:34
rick_h_well just let me know22:34
rick_h_they let ship to you last time22:34
rick_h_let me *22:34
hatchyeah, will do thanks22:34
hatchoo boy 20 test failures removing the multiple inspector support23:00
rick_h_wheee23:13
rick_h_then hopefully some of those tests go away as well23:13

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