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

=== kadams54 is now known as kadams54-away
=== kadams54 is now known as kadams54-away
rogpeppemornin' all07:36
mhiltonmorning rogpeppe, everybody07:42
rogpeppemhilton: hiya07:43
mhiltonrogpeppe: If you find time it'd be great to get a review of https://github.com/juju/charmstore/pull/21907:44
rogpeppemhilton: looking07:44
rogpeppemhilton: reviewed07:49
mhiltonrogpeppe: many thanks07:49
frankbanrick_h_: gui charm branch is going well, almost done, going afk, will propose after lunch12:10
rick_h_frankban: <3 ty much12:25
rick_h_uiteam taking over guimaas to prep for demos for the UOS session today12:54
* frankban afk13:22
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
frankbanuiteam I need two reviews + QA for https://codereview.appspot.com/174170043 (GUI charm/python). Anyone, thanks!13:57
rick_h_frankban: will do but will probably be after standup13:57
frankbanrick_h_: thanks np, I'll ping Jeff too13:57
rick_h_frankban: so we'll shepard it through probably after your EOD to land13:57
rick_h_frankban: ty for helping with that!13:57
frankbanrick_h_: doe sthe next charm release include a GUI one? anyway, I am available for releasing the charm later or on my monday morning, and now good luck with UDS13:59
rick_h_frankban: yes it will. There's one card left before a new gui release with added services13:59
rick_h_I'll be trying to pull it all together and want to make sure whatever release we do to prodstack on monday includes it13:59
frankbancool13:59
rick_h_frankban: ty, I'll let you know how far we get today13:59
* frankban afk again, trying to finish lunch14:16
rick_h_uiteam I've got a demo of gui running on the develop branch. If you plan on borking trunk please don't :) (I'm pretty sure I'm safe)14:28
hatchwell it appears that my internet is shot today :/14:43
rick_h_hatch: kadams54 found a bug in prepping for the demo with hiding units on machine view 14:45
rick_h_hatch: kadams54 will leave this up on guimaas for debugging and such after the session14:45
kadams54Hit me14:45
rick_h_kadams54: hatch so one of the services is in error, with a relation in error14:45
rick_h_when I hit hide, it doesn't hide14:45
rick_h_if I hide a second service, both the new one and the first one that failed to hide at first now hides14:46
kadams54Ah, so something with the error status is borking the hide action.14:46
rick_h_probably14:46
kadams54Well, I guess that's better than hitting hide on the second service and having it do something completely random :-)14:47
rick_h_yea14:47
frankbanhatch: morning, https://codereview.appspot.com/17417004314:50
hatchrick_h_: bleh ok :/14:53
hatchfrankban: awesome will take a look14:53
hatchtoday it appears I'm on dialup lol14:53
rick_h_lol14:53
rick_h_you can watch the animated gif of our UOS session14:53
rick_h_uiteam anyone want to be in the actual hangout for the chat? the etherpad for notes is at http://pad.ubuntu.com/uos-1411-whats-new-and-upcoming-in-the-work-of-juju-ui-engineering and the irc room is #ubuntu-uds-devops-114:54
kadams54rick_h_: what's the hangout URL?14:56
rick_h_kadams54: no idea yet, working on it14:56
hatchrick_h_: lol yeah I'll watch the stream in B&W via rabbit ears on my old crt 14:57
hatchfrankban: so this get() method that's on the config - is that provided by the json serializer?14:58
lazyPowerhey gui team - i'm  helping you get setup14:59
lazyPowerhas anyone made a hangouts on air link for the broadcast yet?14:59
rick_h_lazyPower: no, I thought a sessoin chair or someone was going to14:59
frankbanhatch: it's a default method on all python dicts15:00
rick_h_lazyPower: do I need to create just a hangout on my canonical account and get you a link or how does this need to work to be on air?15:00
lazyPowerdoing it now15:00
rick_h_lazyPower: ty15:00
lazyPowerhttps://plus.google.com/hangouts/_/hoaevent/AP36tYek4Fu7CwFd9SQQ9f4CrPfzUOyHQ_E-pq1YS2zjwGaUPtyYBw?authuser=0&hl=en15:00
frankbanhatch: python -c 'help({}.get)'15:00
hatchohh intersting15:01
hatchthanks15:01
hatchmuch nicer than all my conditionals15:01
frankbanhatch: FWIW, to take a look at all dict methods: python -c 'help({})' and that works for all python objects15:05
jcsackettdammit, hangouts won't let me stay on.15:06
jcsackettrick_h_: rather than hopping in and out i'll just stay off. sorry.15:07
hatchthe video isn't loading for me, might just be my connection though15:08
hatchhttp://summit.ubuntu.com/uos-1411/meeting/22387/whats-new-and-upcoming-in-the-work-of-juju-ui-engineering/15:08
hatchkadams54: are you on the AS bug?15:15
kadams54Yeah, I will after the demo.15:16
hatchalright cool15:18
hatchfrankban: why lambda vs function?15:23
hatchread some docs, dont' really see when it's useful15:24
frankbanhatch: simple function, just one line, mostly a personal preference, sometimes I find the slightly more expressive...15:25
frankbanhatch: no hard rules there15:25
hatchgot it15:25
hatchI see you added a port validator15:25
hatchthat's pretty nice15:25
hatchfrankban: code looks good just pulling the branch down15:32
hatchbut at 80kbps it's taking a while :)15:32
frankban:-/ thanks for looking at it15:33
hatchno thanks for picking up my slack :)15:33
=== tvansteenburgh1 is now known as tvansteenburgh
hatchfrankban: +1's15:46
frankbanhatch: cool, landing it15:47
rick_h_uiteam call in 7 kanban please15:53
rick_h_hatch: kadams54 ok, talk is over so you guys can have the guimaas for debugging if that helps15:58
rick_h_some issues with the demo on a live env it looks like15:58
rick_h_uiteam gui has the guimaas for a bit for new feature debugging15:58
kadams54rick_h_: Thanks - I'll dig into the bugs after standup.15:58
kadams54rick_h_: What's the publicly-exposed URL for the guimaas?16:58
rick_h_kadams54: check out https://docs.google.com/a/canonical.com/document/d/1v3Bs7JU_mYrwRFbXWcQzNRje_EYz0nqqz9yaR4Nadro/edit17:05
rick_h_kadams54: http://maas.jujugui.org/MAAS is the maas side, not sure which item you're looking for 17:06
kadams54rick_h_: thanks17:07
kadams54rick_h_, hatch: I think I found the bug that came up during MAAS18:03
kadams54er, during demo.18:03
hatchoh?18:03
hatchspillit18:03
kadams54Yeah, a change I made in fixing the last bug caused this one. The machine change event wasn't being fired for fades, so the token in MV wasn't being re-rendered.18:04
kadams54I found that while working on decreasing change event noise18:04
kadams54So it's actually already fixed.18:05
kadams54I think.18:05
kadams54:-)18:05
hatchlol18:07
hatchawesome18:07
hatchyou added a test for it, right? :)18:07
kadams54yes18:08
hatchbut the bug that was in the demo was the service token not being hidden18:09
hatchnothing to do with the mv18:09
kadams54ah, true18:15
kadams54dangit18:15
rick_h_kadams54: that was in develop, is this fix in your current branch?18:54
=== kadams54 is now known as kadams54-away
=== kadams54-away is now known as kadams54
kadams54rick_h_: Yes, though as hatch pointed out, the problems you saw were in service view, not machine view.19:14
kadams54So I think we still have bugs19:14
kadams54rick_h_: I'm having problems SSHing into MAAS. I'm assuming you took the SSH keys off Launchpad?19:34
rick_h_kadams54: yes, just ssh-import-id19:35
kadams54I have three keys on launchpad… not familiar with ssh-import-id… did it grab all three, or just one?19:36
rick_h_kadams54: what's your lp username?19:38
kadams54kadams5419:38
kadams54https://launchpad.net/~kadams54/+sshkeys19:38
rick_h_kadams54: ok all three imported try again19:38
kadams54Hmm, still no go. What username should I be using? kadams54?19:40
rick_h_ubuntu@19:40
kadams54http://pastie.org/private/f5gr8xhesfyw7qiosjiew19:43
kadams54rick_h_: Wow, OK, I think I may have badly misunderstood something due to my sshuttle newbie-ness. I was expecting it to use the SSH keys to login, so when I'd get the password prompt above I assumed something was wrong.19:45
kadams54But then I tried the password specified in the Google Docs and that seems to have worked?19:46
kadams54At least, sshuttle reports I'm connected19:46
hatchI always read that as ss-shuttle :)19:48
hatchrick_h_: so this api layer - we rely on a separate module to make the requests so really it's just a thin wrapper around that modules public method19:53
rick_h_hatch: right, that's my thought. 19:54
hatchso my question is, how far does this rabbit hole go? Do we  want to create a new 'network' layer?19:54
hatchif we want to offer this api to general users we'll need to offer a stack19:54
rick_h_hatch: sec, let's chat in 3min19:54
hatchsure just ping whenever19:54
rick_h_hatch: meet you in the standup19:56
hatchjoining19:57
hatchrick_h_: just having an issue joining19:58
hatchcouple mins19:58
rick_h_kadams54: you're in?19:59
kadams54rick_h_: Maybe? Just realized that sshuttle was dying after I started it up; had to take it out of daemon mode to see the message.20:04
kadams54Apparently I need to reboot in order to use it :-\20:04
rick_h_kadams54: reboot your machine or the maas controller?20:06
kadams54Reboot my machine. It's a workaround for a bug in MacOS X 10.720:06
kadams54I also have an AWS env up and running that I'm currently using the debug a potential fix20:07
kadams54So I'm going to hold off on rebooting for a bit.20:07
rick_h_k20:07
kadams54I was able to replicate the bug on relations with errors20:07
kadams54hatch: It's a problem in findUnrelatedServices20:07
hatchahh 20:08
kadams54uiteam: OK, general juju question here. I had juju-gui, wordpress, and mysql deployed in my AWS env. The JS code was throwing an uncaught exception when trying to process a wordpress loadbalancer relation that looked like it was meant to be connected to nginx.20:19
kadams54But I had no nginx deployed… so why would the relation even exist?20:20
jcsackettkadams54: to my knowledge, it shouldn't.20:20
jcsackettis your env still up?20:20
kadams54jcsackett: I'm switching juju-gui-source on it right now, so no, but it will be available again shortly.20:21
jcsackettkadams54: that's alright, i want you to look at something via cli anyway, if you can.20:21
jcsackettkadams54: juju status wordpress20:21
jcsackettcheck the relations on that--then we'll know if the gui is processing something weirdly, or if your juju env has a weird thing.20:22
kadams54    relations:20:25
kadams54      db:20:25
kadams54      - mysql20:25
kadams54      loadbalancer:20:25
kadams54      - wordpress20:25
kadams54https://ec2-54-68-167-143.us-west-2.compute.amazonaws.com/services/:flags:/as/20:26
hatchkadams54: I think wordpress has a peer relation20:27
kadams54What's a peer relation? I've heard that term before, but never really understood what it was.20:27
hatchwell essentially it's a relation to something else installed in the same charm20:28
hatchkadams54: they don't have lines anywhere because it's part of the same service20:29
kadams54Ah, OK, that would make sense20:30
hatchPeer relations are a different matter, in that peer relations are the mechanism by which service units share information internally. It's not unusual for peer service units to maintain convenient caches of distributed information in their own peer settings, and this inevitably involves generating settings keys at runtime.20:30
hatch^ kadams5420:30
hatchfrom the docs20:30
hatchI don't know how to properly use one, I just know what it is haha20:31
kadams54uiteam: OK, https://github.com/juju/juju-gui/pull/657 needs reviews and QA to fix this bug.20:31
hatchlol20:31
kadams54It's a very small PR, but needs a real env for QA.20:31
hatchkadams54: can you add a comment why it might not exist in the code :)20:32
jcsacketthuh, so peer relations are generated automatically? that's peculiar.20:32
kadams54hatch: Done.20:33
hatchkadams54: so when you real real env, would lxc work?20:33
kadams54Yeah?20:33
kadams54Shoot, now that I think about it, sandbox ought to have the same problem.20:34
kadams54As long as you deploy a service that has a peer relationship.20:34
hatchbut I don't think it did20:34
hatchI've used wordpress a lot....20:34
hatchso is this actually the fix? hehe20:35
jcsackettif it doesn't come up in sandbox or lxc, history says it's a timing issue.20:36
hatchkadams54: I'm spinning up an lxc env to test this out20:42
kadams54NOOOOOOOO!20:43
kadams54Just kidding.20:43
hatchlol20:43
hatchmaybe last bug?20:43
* hatch ducks20:43
hatchlol20:43
hatchhaha20:43
hatchhatch ducks20:43
hatchquack quack20:43
* kadams54 suppresses the urge to punch ducks.20:44
hatchkadams54: so I am getting an error in develop when even trying to use the gui because of a hook failure in mysql20:50
hatchdid you get the same bug?20:50
rick_h_kadams54: hatch what's up? 20:51
hatchso on lxc I deployed juju-gui wordpress and mysql via the cli20:51
hatchswitched to the guy and related wordpress and mysql20:52
hatchnow the gui fails with a hook error in utils20:52
hatch(the mysql service failed on start)20:52
hatchI am not sure why mysql failed on start....but I hope that someone else can reproduce this20:53
hatchbecause I can't actually use the gui heh20:53
rick_h_ok so this is something different than the show/hide thing?20:54
hatchI can't even get that far20:54
hatchthe gui is unresponsive on load to the canvas20:54
rick_h_heh, what did you break? :P20:54
hatchI'll take a screenshot of the traceback 20:54
rick_h_hatch: so this is even before using the develop branch20:54
hatchthis is using the local charm version20:55
hatchthis is because agent_state_data is undefined on utils.js:141820:58
hatch^ kadams54 didn't you do something about agent_state_data resiliency recently?20:58
kadams54Sure seems like I did, didn't I?21:04
kadams54https://github.com/juju/juju-gui/pull/64021:04
kadams54hatch: ^21:04
hatchlooks like we need to also add some here21:05
hatchalthough I'd like to know why there is no agent_state_data though21:05
hatchwas that juju-core bug ever created?21:05
kadams54Odd21:05
kadams54You shouldn't be getting that error if you have the patch in that PR21:06
kadams54Are you sure your local charm has the latest code, including that fix?21:06
kadams54hatch: And no, the juju-core bug was not created21:07
kadams54At least, not that I know of21:07
kadams54uiteam: Jenkins merge builds seem to be hung. The current build's been going for 1 day and 1 hour.21:08
kadams54http://ci.jujugui.org:8080/job/juju-gui-merge/782/21:08
hatchkadams54: no the bug I'm seeing is not related to your branch/fix - it's in the relation line decoration 21:08
hatchcode21:08
hatchkadams54: you can just kill the build21:08
hatchalthough you may need to log into the box to then kill the node process21:09
kadams54hatch: you said utils.js:1418, right?21:09
hatchcorrect21:09
hatchnot in develop though21:09
hatchin release21:09
kadams54hatch: Yeah, that's the code I changed.21:09
hatchohh so develop has this fixed?21:09
kadams54Yes.21:10
hatchohh ok cool heh I'll try swapping the branch21:10
kadams54hatch: I've never gotten on the jenkins box… not sure how?21:10
hatchkadams54: ok you are correct develop does have it fixed :)21:16
* hatch shame21:16
kadams54hatch: yay!21:25
hatchlol21:25
hatchkadams54: I have juju-gui wordpress-mysql21:42
hatchif I highlight mysql then fade wordpress juju-gui shows up 21:42
hatchwasn't this what your branch fixed?21:42
kadams54Yes.21:44
kadams54Looking to see if there's a regression.21:44
kadams54Oh, ah ha21:46
kadams54I did fix it21:46
kadams54On the PR that had its merge build hang for a day.21:46
kadams54So the fix still hasn't landed on develop21:46
kadams54Grr.21:46
hatchhmm ok - is that merge running now?21:46
kadams54yes21:46
hatchok cool well once it's done and you update your pr can you ping me for the qa?21:47
kadams54Once it lands successfully, I'll rebase it into the PR.21:47
kadams54Yup21:47
hatchmy env is all set up for it now so it'll be quick21:47
kadams54hatch: PR updated22:04
hatchpulling down thx22:04
rick_h_kadams54: hatch so once things land is the guimaas still setup?22:04
rick_h_e.g. can we change the source to master and then back to develop and QA the exact same env?22:04
kadams54rick_h_: Not sure I follow…22:05
rick_h_kadams54: so taking the gui in guimaas and setteing the source tree back to stable right now22:06
kadams54Yup22:06
rick_h_kadams54: when we think a fix is landed, I'll update it back to develop and see if it works 22:06
kadams54Yeah, sounds good.22:06
kadams54Which reminds me, since I rebooted, I ought to be able to sshuttle in22:06
kadams54Hurrah! I'm on the MAAS box!22:08
hatch+122:09
hatchwe should really sort the added services tokens22:10
hatchthey are always in a different order22:10
hatch:)22:10
kadams54uiteam: need one more (tiny) review on: https://github.com/juju/juju-gui/pull/65722:11
MakyoGot it22:12
kadams54Makyo: thanks!22:17
jcsackettuiteam: i need two reviews, one QA on https://github.com/CanonicalLtd/blues_browser-charm/pull/1322:38
jcsackettlxc env is fine for QA.22:38
kadams54rick_h_: Branch is landed. Update guimaas at your convenience.23:35

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