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

hatchoh....yeah that whole thing is a typo00:00
hatchlol00:00
huwshimioops, that was a comment delete error00:00
huwshimiwindowNode.simulate('keydown', { keyCode: 104, shiftKey: true, altKey: true });00:00
hatchI can't say that that's the proper syntax (I'm pretty sure it is though) but it should work00:01
hatchare you saying that when you add an extra modifier it stops working?00:01
huwshimiNevermind, I'm an idiot00:02
hatchwell I wasn't gona say anything....00:03
hatchbut since you brought it up.....00:03
hatch:P00:03
huwshimi:)00:03
hatchok now I want my new watch.....patience is not a virtue 00:08
rick_h_hatch: heh, just closed my tab checking my xps build status00:09
hatchstill in China is it?00:09
hatchI remember when I bought my last computer form Dell it took a month and a half heh00:15
hatchgranted it wasn't one of those 'fast ship' ones00:16
rick_h_yea, still 'in production'00:16
hatchthink it'll get here before london?00:21
huwshimiI give up on ever hoping my tests will pass our CI00:21
hatchlol00:22
hatchlemme take a look00:22
hatchhuwshimi that looks like it's the intermittent failure00:23
huwshimiIt's actually getting a bit ridiculous at this point.00:23
huwshimihatch: Yeah, I realise that's the problem, it's just a hassle that that happens every time00:23
hatchyeah I'm not going to disagree there00:24
rick_h_yea, I mean it's an issue, but try watching juju-cores landings :) 00:46
rick_h_I actually feel better about our CI lately. It really rarely fails 2 times in a row00:47
rick_h_and really I think it fails 1 in every 7 ish branches00:47
rick_h_which is still :( 00:47
hatchwe'll get there00:51
hatchin time...in time00:51
rogpeppehuwshimi: mornin'05:33
huwshimirogpeppe: Woah, a very early morning to you too :)05:33
rogpeppehuwshimi: :-)05:34
rbasakrick_h_: hey! Got some time to chat about juju-quickstart and status in Ubuntu?12:01
* rbasak isn't sure who else would need to be in that conversation12:01
rick_h_rbasak: on a call atm but should be free shortly12:01
rbasakrick_h_: OK, thanks!12:01
rick_h_rbasak: free as a bird!12:12
rbasakrick_h_: ten minutes? I just grabbed some food!12:13
rick_h_rbasak: ok, sounds good12:13
rbasakrick_h_: https://wiki.ubuntu.com/StableReleaseUpdates#Procedure12:37
rbasakhttps://bugs.launchpad.net/ubuntu/+source/juju-quickstart/+bug/131132112:39
_mup_Bug #1311321: ascii can't decode error in 14.04 server install <juju-quickstart:Fix Released by frankban> <juju-quickstart (Ubuntu):New> <juju-quickstart (Ubuntu Trusty):New> <https://launchpad.net/bugs/1311321>12:39
rbasakhttps://bugs.launchpad.net/ubuntu/+source/juju-quickstart/+bug/130967812:39
_mup_Bug #1309678: a value is required for the control bucket field <juju-quickstart:Fix Released by bac> <juju-quickstart (Ubuntu):New> <https://launchpad.net/bugs/1309678>12:39
rbasakhttps://bugs.launchpad.net/juju-core/+bug/130653712:39
_mup_Bug #1306537: LXC local provider fails to provision precise instances from a trusty host <deploy> <local-provider> <lxc> <juju-core:Fix Released by wallyworld> <juju-core 1.18:Fix Released by wallyworld> <juju-quickstart:Fix Released by frankban> <juju-quickstart (Ubuntu):New> <juju-quickstart (Ubuntu Trusty):New> <https://launchpad.net/bugs/1306537>12:39
rbasakrick_h_: https://bugs.launchpad.net/ubuntu/+source/juju-quickstart/+bug/132501312:41
_mup_Bug #1325013: juju-quickstart has no dep8 tests so can break when juju-core changes <juju-quickstart (Ubuntu):Triaged> <https://launchpad.net/bugs/1325013>12:41
=== urulama is now known as uru-food
rbasakrick_h_: https://bugs.launchpad.net/juju/+bug/108124712:55
_mup_Bug #1081247: maas provider releases all nodes it did not allocate [does not play well with others] <maas-provider> <verification-done> <pyjuju:Invalid> <juju-core:Fix Released by julian-edwards> <juju-core 1.16:Fix Released by rogpeppe> <MAAS:Invalid> <juju-core (Ubuntu):Fix Released> <juju-core (Ubuntu Saucy):Fix Released> <juju-core (Ubuntu Trusty):Fix Released> <https://launchpad.net/bugs/1081247>12:55
* rick_h_ runs for coffee and breakfast. early meetings ftw13:06
=== uru-food is now known as urulama
=== rogpeppe2 is now known as rogpeppe
jcastrohey rick_h_14:06
rick_h_jcastro: otp what's up?14:06
jcastrooh good, you can just listen then.14:07
jcastroso you know how the cluttered search sidebar is my favorite? 14:07
rick_h_jcastro: yep14:07
jcastroI am now realizing that every time we accept a charm into trusty, we double the horribleness14:07
rick_h_shortcut to hide it has landed14:07
rick_h_heh yea14:07
jcastrofor example, try "elasticsearch" 14:07
lazypowerthats strange. Trusty ES shows up under other, and not the recommended charms.14:09
rick_h_jcastro: ok, phone calls over. 14:48
rick_h_jcastro: yea, we try to not confuse users and show one 'recommended' charm14:48
rick_h_jcastro: and the precise one is 'more popular' and wins14:49
jcastrorick_h_, I really want the full screen search back14:49
jcastrothis isn't going to get any better as the number of charms goes up14:49
=== makyo_ is now known as Makyo
rick_h_jcastro: understood, wip to help14:50
rick_h_jujugui call in 214:58
rick_h_rogpeppe: ^ antdillon 15:00
MakyoWas expecting "to your scattered bodies go", which I think pegs me as a nerd.15:15
rick_h_rogpeppe: available for a hangout?15:19
rogpepperick_h_: yup15:20
rogpepperick_h_: got one to hand?15:20
rick_h_rogpeppe: https://plus.google.com/hangouts/_/gwumte2sj2qezub4v54iy37dvqa?authuser=1&hl=en15:20
hatchrick_h_ so I've been messing with this hide sidebar branch and I think we have to expand its functionality a bit.15:33
rick_h_hatch: ruh roh15:33
hatchif the user has hidden the sidebar and then clicks a service the sidebar should open again15:33
hatchelse it just looks broken15:34
rick_h_hatch: but but but, that's the point15:34
hatchbecause the url changes but nothing shows up15:34
rick_h_it's just a manual hack for demo/etc15:34
rick_h_it's not a usable experience15:34
rick_h_at least for the moment, file a bug but it's going to be down the list tbh15:34
hatchright, but I'm not sure what demo is possible without a sidebar when they click a service15:34
hatchok that's fine15:34
hatchthx15:34
rick_h_hatch: right, for the demo, they need to manually show/hide the sidebar 15:34
rick_h_like manually picking the slides or the terminal during a presentation15:34
rick_h_you have to swap between then on your own15:35
rick_h_hatch: but agree, it's a definite nice to have to have it not suck15:35
jcastrorick_h_, heya15:36
rick_h_jcastro: party15:36
jcastrowhat are the chances you have a sec to talk bundles with carla/ant?15:36
rick_h_jcastro: changes are good for the next 2hr15:36
hatchrick_h_ ok cool, as long as people know that they have hidden the sidebar so anything which requires that will also be hidden heh15:43
hatchother than that his branch is gtg so I'll land it15:44
bacso, my ISP saw we had a lot of traffic going to port 6667 so they concluded we had a TROJAN and helpfully blocked the port for us.16:05
hatchrick_h_ we are now getting the intermittent ci failures on the merge job too16:05
bacfreenode == 666716:05
hatchlooks like we need to put some time into fixing it now16:05
hatchbac lol - lots of traffic? freenode?16:06
rick_h_hatch: k, otp will lok16:07
rick_h_look16:07
hatchhttps://saucelabs.com/jobs/3eeb9e9d9dba479cbf5621b069399007 16:15
rick_h_hatch: that's the common issue on that crap stuff16:15
hatchright - but now it's also happening on the merge ci16:16
hatchit never used to happen on that one 16:16
hatchit appears the frequency of the failures has gone up16:16
hatchand it's spreading.....like a virus....a test virus16:16
hatch:)16:16
hatchyeah it's happened twice in a row now on two different branches16:17
rick_h_ok, but that's bad luck16:17
rick_h_it's happened before16:17
rick_h_hatch: ok, off the phone16:18
rick_h_hatch: yea, that sucks. We've had that kind of mass sauce tests hate us before and it usually makes a day suck and passes16:19
rick_h_hatch: but yea, we definitely want to fix that as part of our code/clenaup work post machine view16:19
hatchright, it's just, how much time is wasted re-running ci builds to see if they pass16:19
rick_h_hatch: right, if today it takes 4 tests runs, that's 2hrs. Fixing notifications is several days of work noe 2hr16:20
rick_h_so it takes having this happen for a week in a row to be a blocker like that imo16:20
rick_h_and it's not like this solves all CI failures, this notification one in IE is the most regular of them for sure. 16:20
rick_h_and it'll be first on the list of fixing up test stuff post machine view16:21
hatchthis is in ff now16:21
hatchtoo16:21
rick_h_ok16:21
hatchit's only this test16:21
hatchbut I don't see anything wrong with it, or the previous ones16:21
rick_h_right, and I spent a day trying to update that test and found it's a fundamental issue in how notifications are written and the fix means rewriting that code16:22
hatchblarg16:22
hatchnotifications are one of the last original code things16:22
hatchheh16:22
rick_h_I took a day to try to fix it and the hack is more than a day of work, and the workarounds are uglies than this failures16:22
rick_h_the only right thing to do is to change it to be less timeout heavy and work properly in an event world 16:22
rick_h_which is a base level rewrite of how it works16:23
rick_h_hatch: sorry it's sucking today. We will make it better, but not today. 16:23
hatchI'm just curious why it happened all of a sudden16:23
hatchsomething must have changed16:23
rick_h_it's a timing thing16:23
rick_h_the sauce machine is slower than usual, the network, and it fails16:24
rick_h_it's depenent on the stack and it's a big stack of pita and that stack is now 10 inches tall every single day16:24
rick_h_it's 9.8" one day, and 10.2" another day16:24
hatchhaha16:25
rick_h_and if it gets to 10.3" the tests tell us to go screw ourselves and die16:25
hatchI wonder if we went on a 'private' sauce instance if it would be faster16:26
hatchthey don't say there is any difference in performance16:26
rick_h_hatch: right, it's something I want to bring up. 16:26
rick_h_hatch: when we get theblues going and hopefully are doing more sauce-like tests I want to try to get us a real account16:26
rick_h_welcome BradCrittenden!16:27
hatchyeah cool16:27
rick_h_hatch: at that point it'll be easier to justify and make a case it's a cross project thing16:27
hatchright16:27
rick_h_hatch: but yea, not looked into it yet. Just talked with them at pycon16:27
hatchhaha that louis suarez got banned for 9 games16:27
hatchahh16:27
rick_h_ok, I have to refill my coffee, afk for a min16:27
hatchlazypower are you collecting the steam summer adventure cards? I need two of the ones you have ;)16:32
rbasakrick_h_: can I suggest that quickstart prompts for ssh key creation before installing packages? Then the user can get over the interactive part without having to wait first.16:36
hatch+116:37
rbasak(otherwise I come back to a test that I started earlier to find that it's waiting on a prompt)16:37
hatchrbasak I'd recommend filing a bug16:37
rbasakI'd be happy to if it sounds like it's doable16:38
hatchI have no idea, but it would improve the experience so I'm all for at least investigating it16:39
rick_h_rbasak: yea please file a bug and we'll investigate16:40
hatchthey are running the fiber through my neighbourhood right now16:40
rbasakFiled https://bugs.launchpad.net/juju-quickstart/+bug/1334757 - thanks. I don't have permission to set Importance: Wishlist.16:41
_mup_Bug #1334757: ssh key generation prompt happens after a long wait, mismatching the user's expectation that it will all be ready when he comes back <juju-quickstart:New> <https://launchpad.net/bugs/1334757>16:41
rick_h_rogpeppe: had a talked with design and eco on bundles and there's a couple of notes for the bundle spec. I'll update the spec after lunch and then file a couple of bugs16:41
hatchrbasak thanks a lot16:42
rick_h_rogpeppe: but we wanted to support categories and description as optional items to the spec 16:42
rogpepperick_h_: how is "description" different from README ?16:42
rick_h_rogpeppe: and want to talk about pulling vcs info during a publish, so it's part of the publish process, but we'll have some metadata that the store will want to keep 16:42
rogpepperick_h_: (personally i'd be happy ditching README)16:42
rick_h_rogpeppe: the description is a paragraph while the readme is a full blown document going through optios, what it does, how it works, minute details16:42
rogpepperick_h_: ok16:43
hatchMakyo when you have a second I'd like to have a chat about implementing the 'removal' story in the ecs when there are hierarchical changes to be made 16:43
rick_h_rogpeppe: e.g. https://jujucharms.com/bundle/mongodb/5/cluster/#readme is the readme, but the desciption might be "Deploys a 3-shard mongodb cluster"16:43
rick_h_rogpeppe: anyway, will follow up after lunch in the right places but wanted to let you know. 16:44
* rick_h_ goes for food16:44
bacrick_h_: i'm trying to launch against azure.  i'm getting 'affinity group conflict' so i assume there is a machine associated with 'bactest' that is still alive, though juju doesn't know about it.  on their dashboard i cannot figure out which machines are which.  would you happen to know how?16:44
rick_h_bac: I just match the names in the dashboard with the juju status outpuyt16:44
rick_h_output, using the dns names16:44
bacrick_h_: right but i now have no juju output16:44
Makyohatch, sure, now's good.16:45
rick_h_bac: :/ ok looking16:45
hatchMakyo ok going to the standup room16:45
rick_h_bac: go to the monitor tab and move the timeline to 24hrs and view which machine has data for things like disk going back the right timeframe?16:46
bacoi16:47
rick_h_bac: can you join the other irc channel from this morning please?16:48
bacrick_h_: i got a list of the real ones16:48
rick_h_bac: awesome, let me know if you need a hand16:50
* rick_h_ goes for lunch 16:50
* rogpeppe is done for the day.17:24
rogpeppeg'night all17:24
rick_h_see ya rogpeppe 17:31
rbasakjuju-quickstart 1.4.0 uploaded to Utopic. It should land in the next hour or so.17:56
rick_h_rbasak: <317:57
rbasakThank you for making it easy :)17:57
rbasakI didn't need to change the packaging at all.17:57
rick_h_bac and frankban rock on that 18:00
Makyojujugui need add'l reviews/qa on https://github.com/juju/juju-gui/pull/405 - switching to new upgrade UX. I suspect I'll have conflicts on my next branch, so I'd like this to land ASAP.18:01
hatchon it18:02
Makyohatch, thanks.18:02
hatchjcsackett what's round 16?18:08
rick_h_hatch: soccer18:14
hatchoh....18:14
hatchI saw 5 minutes of a game yesterday and saw a guy flop18:15
hatchhe flopped, sat down holding his knee, when no one cared, he got up and started playing again...18:16
hatchstupid game18:16
hatchMakyo code is good18:26
hatchdo you need another qa as well?18:26
Makyohatch, I should be good, the next few branches will be additional QAs.18:27
lazypowerhatch: got a gui question for ya - http://askubuntu.com/questions/488575/juju-configuring-a-service-by-clicking-on-charm-with-stand-alone-implementatio18:27
hatchMakyo ok cool18:28
hatch+118:28
hatchlazypower looking18:28
hatchlazypower ok I'll reply thx18:29
lazypowerits easy points, thought you'd like that one18:32
=== BradCrittenden is now known as bac
hatchreally wish people would ask one question at a time18:38
hatch:)18:38
hatchlazypower and answered18:38
hatchwoot I broke 900 points on AU18:44
hatchlook at me go18:44
lazypowerupvoted18:47
hatchman it's so loud here19:41
hatchthey are directional drilling or something....19:41
* rick_h_ steps a way for a bit. 19:48
rick_h_see you all at the AU call tonight19:48
hatchrick_h_ when you return - have you seen this error before? http://ci.jujugui.org:8080/job/juju-gui-merge/433/console usually it happens once then works the next time but this keeps happening on this branch and there isn't any information on the failure other than that it failed20:01
hatchtrying again, 4th time is the charm!20:05
rick_h_hatch: that's github's api not responding nice20:19
hatchoh ok 20:19
rick_h_hatch: we should add some error checking in the result of the api response to the merge command20:19
rick_h_if github has a timeout/etc you get stuff like that20:19
hatchif broke try again20:19
rick_h_yea, pretty much20:19
rick_h_nothing to do but ask github again 20:19
rick_h_pretty please do this merge for me20:19
* rick_h_ sneaks back into the shadows20:20
hatch"hey cool, it works"20:36
hatchOoooo20:38
hatchafter all that work I ended up solving it in 7 LOC's20:39
hatch*sigh*20:39
bacrick_h_: those bugs for the SRU have been updated.  i'm not sure i convinced myself on the 'regression potential' for the last one.20:40
Makyojujugui quick PR for review/QA https://github.com/juju/juju-gui/pull/40622:05
hatchsure22:05
MakyoFixes #132898122:05
_mup_Bug #1328981: Reloading the inspector on upgrade of a charm creates blank sidebar <juju-gui:Triaged> <https://launchpad.net/bugs/1328981>22:05
hatchMakyo so is clicking this change version button supposed to block out the entire inspector with no way to close it? :)22:12
Makyohatch, Did in the previous version, this should fix that.22:13
MakyoCan you check your cache?  This has been an issue in the past.22:13
MakyoAlso, report the URLs that you see in the process.  Should clear sectionA, then repopulate with inspector/<service>22:13
hatchMakyo I mean, I click Change version then a list shows up22:14
hatchwhich I can't close22:14
Makyohatch, yes, you didn't QA the previous branch, but that was step 1, step 2 is styling/etc.22:14
MakyoWhich includes setting active tab to null, etc.22:15
hatchok when I click to upgrade it changes the url to http://192.168.33.10:8888/precise/mysql-31/#/precise/mysql-3122:15
hatchso it shows the charm details22:15
hatchbut it doesn't always do this....hmm22:15
MakyoAre you clicking the word 'upgrade'? there's two links there, just like there was in the previous impl, it just hasn't been styled yet.22:18
Makyothe 'upgrade' link does not have an href and should not change your URL.22:18
hatchoh lol oops22:18
MakyoSorry, I promise styling is coming next.22:19
Makyo7 days was just too long for that branch.22:19
hatchlol np22:19
hatchok I see I can't test this on sandbox22:19
MakyoI can.  Not sure what's up on your end.22:19
hatchwell it doesn't say to reload the inspector22:20
hatchit just reloads22:20
hatchwithout issue22:20
Makyohatch, hmm, I noticed this with mysql too.  Can you try wordpress while I see what's up? I have the feeling it's a databinding issue.22:20
hatchsure trying wp22:21
Makyo(That button for reloading the inspector is databound to an attr on the service.22:21
Makyo)22:21
Makyojujugui .au weekly call in 9, for those going.22:21
hatch"error interacting with charmworld api"22:21
hatchCannot set charm on a service with units in error without the force flag.22:21
hatchinteresting22:21
hatch^ that last one was because simulator22:21
hatchinteresting non the less22:22
hatchMakyo wordpress also 'just changes' 22:22
Makyohatch, yeah, that's a core thing.  SetCharm api endpoint doesn't work when the service has units in error, reproduced in sandbox.22:22
MakyoButts.22:22
MakyoLet me prowl some more.22:22
MakyoDatabinding is a little too magical.22:22
hatchok lemme know22:23
MakyoLooks like it has something to do with the delta stream.  service.charmChanged doesn't get set to true unless things come back in the delta in the proper order.22:27
MakyoWhich is a little dumb.22:27
MakyoI'll poke more after the call.22:27
rick_h_mm, no huw22:31
rick_h_there he is22:32
huwshimiMorning22:32
rick_h_hatch: coming along?22:33
rick_h_huwshimi: call?22:33
hatchhey huwshimi  yep joining call22:33
huwshimiJoining22:33
huwshimibrb changing location23:03
hatch13s relocation? :)23:09
hatchoh :)23:10
hatchjujugui lf a review and qa for https://github.com/juju/juju-gui/pull/407 plz and thx23:22

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