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

hazmatanyone noticed significant issues with the 1.21 alpah2 and gui?10:51
frankbanhazmat: I am trying quickstart with the alpha version12:16
frankbanhazmat: on a local run, quickstart never exits because the addresses in the mega-watcher for machines don't include a public scope12:16
frankbanhazmat: e.g. http://pastebin.ubuntu.com/8703311/12:17
frankbanfwereade: ^^^12:19
hazmatfrankban, i filed a bug12:23
frankbanhazmat: thanks, number?12:24
hazmathttps://bugs.launchpad.net/juju-core/+bug/138614312:24
mupBug #1386143: 1.21 alpha 2 broke watch api, no longer reports all services <api> <regression> <juju-core:Confirmed> <https://launchpad.net/bugs/1386143>12:24
hazmatthe txt attachment shows some of the issues.. basically not all services appear in the megawatcher12:25
hazmatas in very broken12:26
frankbanhazmat: ok thanks, I'll add a comment to that bug re the issue with public addresses12:28
frankbanhazmat: updated your bug, confirmed it affects the GUi as well12:37
hazmatfrankban, after i dug into the actual bug, i assumed as much in the initial reporting, allwatcher is just broken.12:39
hazmatfrankban, thanks for confirming12:39
rick_h_frankban: when you're back can you check out https://bugs.launchpad.net/juju-quickstart/+bug/1385407 as well. I'm guessing this is the bug jrwren_ ran into and that is now fixed in core. 12:49
mupBug #1385407: juju-quickstart destroys existing AWS  environment <juju-quickstart:New> <https://launchpad.net/bugs/1385407>12:49
rick_h_frankban: it doesn't mention the juju version in the bug which will be the trick to making sure. But it's how jrwren_ brought down CI so core issue vs quickstart12:50
jrwren_:)12:53
rick_h_jrwren_: if you can find the bug/link to the fix to help frankban as it might be in your own bug history that'd be +112:53
jrwren_linked.13:09
rick_h_ty13:10
hatchjujugui call in 914:51
hatchbecause of osx yosemite issue I barely have an internet connection so I'll need someone to run this one14:52
hatchhey redir14:52
rediryo!14:52
redirhow goes hatch ?14:52
hatchwell osx yosemite has ruined my week - but other than that, it's going ok :)14:53
hatchand yourself:14:53
hatch?14:53
fabricejcsackett: tell me if it's inline with your comments now for bundle vis14:54
fabricejcsackett: please14:54
hatchhey kyle how goes the conversion?15:18
hatchI'm pretty close to getting this thing ready to go - just took a few different approaches before I found one that made sense15:18
hatchkadams54: ^ :)15:19
hatchbrain not working well today I guess hah15:19
kadams54hatch: I'll have the refactor ready for review shortly. Cleaning up lint errors.15:20
hatchcoolio15:20
hatchkadams54: over the weekend I was wondering why we don't mark the service models as faded/highlighted? It makes is about 1M times easier from the canvas side15:23
kadams54I think we should15:23
hatchok good, I am15:23
kadams54Both services and units15:23
hatchheh15:23
kadams54There's some duplication there and we could talk about whether setting on the service should cause the same flag to percolate automatically down to all units within that service.15:24
hatchhmm that's a good idea15:24
hatchI think what I'll do then is wait until your branch lands then go back and update it15:25
hatchelse there are going to be a ton of conflicts15:25
hatchjust ping me whenever you're ready for a review15:32
redirhatch: us health insurance system paper work ruining my day^H^Hweek^H^Hmonth^H^Hquarter^H^H but other than that doing great:)15:54
hatchhaha - wait, the US has health insurance? :P15:54
hatchI thought ya'll were still using leaches :P15:55
redirhatch: the US has a health insurance market. Though it has little to do with health or insurance.15:55
hatchlol15:55
redirhatch: they are leeches. 15:56
hatchhaha15:56
redirhatch: I've actually had a fine experience over the last 10+ years, just horrid paperwork problems since changing providers.15:56
hatchahh - yeah I have no idea how the system works, just fun to poke fun at it :)15:57
kadams54hatch: https://github.com/juju/juju-gui/pull/628 is ready for QA and review.16:02
hatchkadams54: thanks16:02
hatchkadams54: you answsered this last week but I can't remember - can you highlight and fade?16:02
hatchso the icon will be different but faded?16:03
kadams54No, not on the same service.16:03
hatchok16:03
kadams54You can highlight a service and one of its related services can be hidden.16:03
kadams54At which point the highlighted-but-hidden related service needs to be faded to 20%16:04
hatchkadams54: did you make any changes to the moved code? Or was it just a copy/paste16:05
kadams54Copy-n-paste.16:05
hatchcoo16:06
hatchl16:06
kadams54And the new tests reflect the current (wrong) behavior, so there were aspects I didn't even bother changing.16:06
hatchsure - but what about the old tests? 16:06
hatchor were there no old tests? heh16:06
kadams54Er, "didn't even bother testing", not "didn't even bother changing"16:06
kadams54Very few old tests, if any16:06
kadams54None that directly tested16:06
hatchoh jeesh ok thanks16:07
hatchhah -  this is why we need more of these extensions lol16:07
hatchkadams54: review done - I vote to land it once the tests run so we can keep truckin along16:11
kadams54K16:11
hatchjujugui can we get another review on #62816:11
hatchdart virtual dom vs reactjs benchmark http://localvoid.github.io/vdom-benchmark/ looks like dart is faster - at least according to this benchmark16:22
hatchkadams54: shippit?16:30
kadams54hatch: Done.16:31
hatchnice16:31
hatchjust don't want to get held up heh16:31
hatch(although I seem to be holding myself up :?)16:31
hatch:/16:31
hatchthe undocumented show/hide/highlight/fade methods and their differing call signature is driving me batty16:37
kadams54Ah yes… I had a branch where I documented them. Not sure what happened to that.16:44
hatchit's ok I've got it working now just super frustrating because they look the same but are slightly different16:51
hatchkadams54: so I'm just doing https://gist.github.com/hatched/41657f7e8bf45d5dc84d in the handlers16:58
hatchis that the same direction you are going?16:58
kadams54Sure.16:58
kadams54Pretty much16:59
hatchoh you haven't started? :)16:59
hatchI just saw the card in coding16:59
hatchhaha16:59
kadams54I'm eating lunch :-)16:59
hatchoh haha sounds good16:59
hatchare you also going to move the unit updating into the service?16:59
hatchlets have a chat when you're done lunch17:00
hatchjust want to limit the conflicts 17:00
hatchwb Makyo you workin today?17:00
Makyohatch, Yep, I just took a half day since I worked half of Thursday.17:01
hatchahh coolio 17:01
hatchgood trip?17:01
MakyoVery!  I didn't look at a laptop once.17:02
hatchhaha nice those are the best kind17:03
hatchkadams54: this is the approach I settled on for the service topology stuff https://github.com/hatched/juju-gui/compare/juju:develop...hatched:canvas-fade?expand=1 17:08
hatchthe same needs to be done to the relationship side 17:08
hatchbut I think it's easier to reason about this way17:08
hatchthis of course assumes that the service models are updated accordingly17:09
kadams54hatch: OK, give me 5 minutes and then we'll chat?17:11
hatchyeah np17:12
hatchfrankban: https://bugs.launchpad.net/juju-gui/+bug/1383381 this done? can you update the bug17:15
mupBug #1383381: autoplaced units don't show up in machine list <juju-gui:In Progress by frankban> <https://launchpad.net/bugs/1383381>17:15
kadams54hatch: Ok, ready when you are.17:25
hatchkadams54: ok it might be a bit - I'm just debugging a gui issue with a user atm17:25
kadams54np17:25
hatchwill ping when I can17:25
hatchkadams54: ok I can probably do both at once17:28
hatch:)17:28
hatchsee u in standup room17:28
kadams54Yay multitasking17:28
hatchjujugui can anyone confirm if juju uses the lowest possible unit id's for new units or always goes +1 from the last id regardless if that unit still exists?18:16
hatchI am pretty sure it always +1's the last one whether it exists or not18:16
urulamahatch: +118:16
hatchthanks sir18:16
frankbanhatch: how do you want to use this assumption?18:17
hatchfrankban: https://bugs.launchpad.net/juju-gui/+bug/1375918 this context will help you understand :)18:18
mupBug #1375918: units can be created without a service causing cascading failures <juju-gui:Triaged> <https://launchpad.net/bugs/1375918>18:18
hatchfrankban: essentially I'm not sure what the GUi can do but create temporary id's like we do with services18:18
frankbanhatch: so basically no assumption on the next id sounds good, because it's unpredictable18:20
hatchright - but we need unique id's for ghost units18:20
hatchso we'll have to switch this over to using temporary ids like we do for services18:20
hatchquite a bit more work to fix than I had originally thought heh18:21
frankbanhatch: yeah, for the machines I used an internal counter and a "new" prefix, perhaps we can do the same for units?18:21
hatchyeah that's possible 18:21
hatchI'm just so glad he helped me debug the issue heh18:22
frankban:-) good night all, done for the day18:24
hatchnight frankban18:24
kadams54hatch: how do you want to handle related services for something like highlight?18:50
kadams54i.e., should I set any flags on them? Or are you going to handle finding them and styling them appropriately based on the one service having a highlight flag?18:50
hatchkadams54: sorry just cooking lunch19:03
hatchkadams54: yes the topology should be able to just parse the db for what it needs to hide/highlight19:03
hatchso that should be done in the handler imho19:04
kadams54Except that nothing gets set in the DB for related services19:04
kadams54There is no "your relation is highlighted" flag19:04
hatchohh19:06
hatchhmm that's a good point19:06
hatchthat'll have to be done in the relationship update handler?19:06
kadams54I could handle it in browser by creating three sets: the targeted service, the related services, and the unrelated services.19:07
kadams54targeted.highlight = true, unrelated.fade = true, related -> noop19:08
hatchhmm nah then we aren't using the db as the source of truth19:09
hatchok lunching19:09
urulamawhy does it make sense to make search queries with limit=1000?19:12
hatchurulama: because 1001 runs the system out of memory?19:31
hatch;)19:31
hatchkadams54: back - what did you think of my comment?19:31
urulamahatch: :P just checking that it's not some final call :)19:32
hatchhaha I have no idea - 1000 results over the wire would certainly put something in limp mode19:33
kadams54hatch: not sure I follow. Why would we not be using the DB as a source of truth?19:49
hatchwell where would this object be?19:49
hatchtargeted, unrelated etc19:49
kadams54They're just lists of models.19:51
hatchquick chat? I think we are crossing ideas here19:51
kadams54And then I'd iterate through and set the appropriate flags19:51
kadams54I have my 1/1 with Robbie in 10.19:52
hatchit'll be quick I promise :)19:52
kadams54k19:52
kadams54Anyone else had their 1/1 with Robbie yet?20:00
hatchI have20:01
jrwren_I did before the sprint.20:08
hatchkadams54: ok I think I have the relation one done - I'll bench this branch now until yours lands20:49
hatchis there anything else I can help on now?20:49
kadams54hatch: I need some dog food from the grocery store.21:22
hatchwell there is your problem - don't get dog food from a grocery store21:23
hatchunless of course you have a progressive grocery store that sells something other than that made in 'who knows where' garbage21:25
hatchkadams54: it's your EOD now?21:58
rick_h_evening22:32
hatchwelcome back fearless leader22:33
hatchI saw your team lost22:33
hatchnot that we expected anything else ;)22:33
rick_h_:( it was quite brutal22:34
rick_h_more than a loss, a thorough beat down22:34
hatchfor that I have this https://play.google.com/music/m/Tugnjibik2twwzbrudbnvjc5cfi22:34
hatchwell THAT link didn't work22:35
rick_h_worked here22:35
hatchoh good :)22:35
hatchsince I upgraded to yosemite my internet is absolute garbagio22:36
rick_h_can https://plus.google.com/photos?pid=6074616927809511298&oid=116120911388966791792 work for you?22:36
rick_h_heh, not upgraded at all here22:36
hatchjust a black screen22:36
hatch:(22:36
rick_h_hmm, just have your canonical G+22:36
hatchlemme go reboot my router - maybe that'll help22:37
rick_h_hah22:37
hatch__rick_h_: even the email doesn't work22:43
hatch__sure you have the permissions open?22:43
hatch__ahah I got it22:43
rick_h_hatch__: ? I just shared it with your work google plus account22:43
rick_h_you have to go there to get it22:43
hatch__yeah22:43
rick_h_part of stupid G+ getting rid of way to share photos via a url :(22:44
hatch__sweet shot22:44
rick_h_yea, google auto awesome'd one of my shots in the game of brady tossing another TD against my team22:44
hatch__lol22:45
hatch__salt in the wound22:45
rick_h_yea, well the salt was to pay the $$ to fly out there, get awesome seats, and watch a game that wasn't even competitive. 22:46
hatch__yeah I could see that22:49
=== hatch__ is now known as hatch
=== hatch is now known as Guest35185
Guest35185*sigh* stupid freenode22:53

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