=== Makyo is now known as Makyo|out
danialjoseWhat is the password to access Juju Web UI demo?07:27
danialjosei found it "admin"07:29
=== _mup__ is now known as _mup_
=== frankban_ is now known as frankban
hazmatnice article on css3 transitions http://blog.alexmaccaw.com/css-transitions13:00
gary_posterteknico, did my branch work for you?13:04
teknicogary_poster, working on it with frankban13:04
teknicogary_poster, impressive work, thanks, needed only a few tweaks13:04
gary_postergreat thanks teknico 13:05
teknicogary_poster, we're now struggling with erratic tests13:05
gary_posterteknico, gotcha :-/13:05
teknicogary_poster, will finish fixing them after lunch13:05
teknicogary_poster, and write a few new ones13:05
gary_posterteknico, cool.  Maybe bcsaller will get his branch landed in that time which includes hazmat's fixes for some of the erratic tests13:05
frankbangary_poster: cool13:06
teknicogary_poster, where can we see those fixes?13:06
bcsallergary_poster: I hope so, when I did the merge things started dying all over the place, but only in the tests. pre-merge both branches worked fine. I tried to track it down last night again and am working on it now13:06
gary_posterteknico, frankban. in isolation here: https://code.launchpad.net/~hazmat/juju-gui/reliable-test/+merge/14119713:07
teknicogary_poster, thanks13:07
gary_posterBen has additional test fixes as he outlines in review comments in his viewport branch https://code.launchpad.net/~bcsaller/juju-gui/viewport/+merge/14213813:07
teknicoand now yoga :-)13:07
gary_poster:-) enjoy13:07
frankbangary_poster: do you have time for a quick hangout before my lunch on my current slack task?13:07
gary_postersure frankban 13:08
gary_posteryeah, cool13:08
benjiteknico: we were discussing ways to keep unused EC2 instances in their place, Amazon just anounced this: http://docs.amazonwebservices.com/AmazonCloudWatch/latest/DeveloperGuide/UsingAlarmActions.html13:26
gary_postercool benji.  frankban and teknico just went to lunch.13:27
gary_posterfrankban, I added the cards we discussed to the slack lane13:27
gary_posterthey could probably go in the main project lane tbh13:27
gary_posterbac, approved with mostly trivial comments13:58
bacgary_poster: thanks13:58
gary_posterbcsaller, do you have anyone helping you with the test failures?  If not, would you like someone to help?  If so, do you already have someone in mind (e.g. Matt) or should bac or I try to help?14:02
bcsallergary_poster: another pair of eyes would be great, I can14:05
bcsallerI can't tell whats happening yet, but it looks like login view isn't loaded for some portion of them making fail, but that doesn't appear to be all14:06
bcsallerI'll push what I have14:06
bacgary_poster: good idea on providing example entity names.  i'm unsure how environments are named, though.14:07
gary_posterbcsaller, ok.  I wonder if the changes that I made and teknico and frankban are polishing (esp. tests) will make things better or worse14:08
gary_posterbac, ok. do what you can.  If you can figure it out it will make upcoming branches that much simpler14:08
bcsallergary_poster: trunk runs fine today, as did mine before the merge, so I can't tell offhand. The errors coming out are not clear to me though, things are undefined that shouldn't be but its often not clear what and seldom clear why14:10
gary_poster:-/ yeah, ok14:10
gary_posterbac, I suspect a quick question to Kapil will set you on the right path quickly.  Probably worth bothering him.14:11
bachazmat: the annotation commands (get, update, remove) take an entity name which may be the environment.  i suspect that is the name in your environments.yaml used when bootstrapping.  correct?14:16
hazmatbac no its.. a magic  value.. 'environment' i believe14:18
bachazmat: ah, ok14:19
goodspudAwesome that we have authentication on the demo site.14:19
goodspudWell done14:19
goodspudCan anybody share the password for it though? :)14:20
gary_posterbcsaller, make test-debug fails very very quickly while make test-prod is a bit more gradual.14:20
hazmatbac, magic key is 'env'14:20
hazmatgoodspud, admin14:20
gary_postergoodspud, pretty version is being coded now.  password is "admin"14:20
goodspudCheers chaps14:20
gary_postergoodspud, pretty version includes help text telling demo users what the password is :-)14:20
goodspudVery secure :)14:21
teknicobenji, thanks, got the email from AWS with the timely new feature :-)14:21
hazmatbac, there maybe a caveat in how it comes back in the delta stream.. since there isn't an environment object/representation there14:24
bachazmat: i just pushed changes to my branch.  would you mind doing a review to see if i missed something?  https://codereview.appspot.com/706406014:39
hazmatbac, added comment re doc string on update_annotations14:42
bacthanks hazmat14:43
gary_posterbcsaller, I have hack workaround that you probably also have.  I don't understand what is going wrong with the loader. :-/ simply explicitly saying that app.js requires juju-view-login makes it work (line 8 of http://pastebin.ubuntu.com/1513203/).  I also need line 45 in order to get the tests to pass in "make test-debug".  This should not be necessary either. :-( changes to test/index.html are unncessary but nice c14:47
gary_posterleanup.  change to test_login makes linter happy14:47
gary_posterbcsaller, with that diff I then have seven or eight test failures, all of which seem more specific to the viewport changes14:47
bcsallergary_poster: thanks, I'd seen something similar, but it still seemed intermittent for me, I'll try what you have and track down the rest.14:48
gary_posterbcsaller, five of the remaining failures are in pan zoom module tests; charm normalization looks like another loader weirdness; and juju environment view has two.14:50
=== Makyo|out is now known as Makyo
gary_posterbcsaller, this fixes charm normalization problem for me: http://pastebin.ubuntu.com/1513237/14:52
gary_posterbcsaller, if you would like details on the other failures in environment and pan/zoom, let me know14:52
gary_posterI will leave them otherwise14:53
gary_posternice changes bac, thank you14:55
gary_posterbac typo14:55
gary_posterIf no keys are passed, all annoations14:56
gary_poster-> annotations14:56
bcsallergary_poster: I think we are on the same page now, took a while before the merge even worked for me. Tracking down the real issues now, thanks again 14:58
gary_postercool, welcome15:00
gary_posterbac it would be trivial-ish to do bug 1096273 now, I think.  and for bug 1096270...hazmat, what's your timeline for making the delta stream annotation changes?15:11
_mup_Bug #1096273: on drag end set annotation <juju-gui:Triaged> < https://launchpad.net/bugs/1096273 >15:11
_mup_Bug #1096270: propagate delta stream annotation changes as events <juju-gui:Triaged> < https://launchpad.net/bugs/1096270 >15:11
* bac looks15:12
bacgary_poster: agreed.  grabbed card.15:15
hazmatgary_poster, i can turn that around today15:20
gary_postercool thanks hazmat15:20
hazmatbac was there a use case for the env annotation? looking over the delta stream code that's not currently coming back there, but i can add it as part of the other change under a fake object/parent key 'env'15:21
hazmatlandscape has uses for it though15:22
hazmatenough of a reason to add it15:22
bachazmat: no, i was just trying to make clear the function documentation i saw in context.py15:22
hazmatgary_poster, bac so w/ the change annotations in the delta stream will now be under a sub object/dict 'annotations'15:25
gary_postersounds good hazmat15:26
hazmatgary_poster,  except the environment annotations which will be under a dict under the key ... 'env' ? there isn't a corresponding object to place them under15:27
hazmatalternatively 'annotations' as a top level key15:27
gary_posterhazmat, don't have a strong feeling about it.  Lean towards annotations top level key15:27
hazmatgary_poster, cool, works for me15:28
gary_posterbac bcsaller benji frankban goodspud hazmat Makyo teknico call in 115:29
hazmatbcsaller, i had an unpushed rev on that reliable-test branch.. just pushed might help15:33
bcsallerhazmat: I think things are mostly resolved wrt that, but I'll check15:34
benjibcsaller: my headphones reproduce the bass notes in your typing with high fidelity ;)15:34
bcsallerbenji: I think I'm muted now, maybe before15:35
benjibcsaller: I muted you.  That was my note softening the blow. 15:35
bcsallerhas anyone seen the test runner show all checks but still list a failure count?15:57
gary_posterno, that sounds like an exciting new failure mode :-P15:58
bcsallergary_poster: its pushed if you want to see, you can click on 'failures' to filter the results and nothing appears15:58
bcsallerbut still has a count 15:58
gary_posterok will take a look15:59
bcsallerI'll put more coffee on, brb15:59
gary_posterI am about to be on call also15:59
gary_posterduped bcsaller 16:01
bcsallergary_poster: that helps, know I not totally insane16:02
gary_posterI didn't say that bcsaller ;-)16:02
bcsallerputting .only on some of the tests, something in the pan zoom triggers this16:02
bcsallerha ha 16:02
thervehey there!16:18
gary_posterhey therve :-)16:18
thervehi gary_poster!16:18
therveI'm having an issue trying to deploy the charm16:18
therveit fails with "Unable to locate package python-charmhelpers" 16:18
therveany idea where that comes from?16:18
gary_postertherve, yes.  is this on precise?  Juju PPA should have it, and ppa:charmers/charm-helpers .  That said, concerned that we don't see that problem and would love to know why16:20
thervegary_poster, it is on precise16:21
gary_posterweird :-(16:21
gary_posterfrankban, you seen anything like that lately? ^^^16:21
thervethe ppa is not on the deployed image16:21
thervemaybe my lxc image is out of date or something?16:22
gary_postertherve, oh.  lxc juju doesn't work well in our experience I'm afraid.  We test with ec216:22
gary_posterhaven't seen that failure mode though16:22
thervelxc is so fast and easy :/16:23
gary_posterI know16:23
thervebut I'll look on ec2 :)16:23
gary_posterbac benji frankban teknico, big news: we are no longer responsible for LP at all, and OCR is over.  OCR may return in some new different form with different goals (knowledge sharing across cloud engineering).  But anyway, don't do LP OCR, and take it off your calendar.16:33
* benji wipes a single tear from his eye.16:34
* teknico does not try to contain exultation *at all* :-D16:34
teknico(sorry guys :-) )16:36
bcsallergary_poster: its all fixed and pushed. When you have a sec can you confirm that for me16:43
gary_posterbcsaller, confirmed, yay!16:49
bcsallerthank you :)16:49
bcsallerits really odd, an async test failure in afterEach can stop other tests from running, so an error can silently cut the total number of tests short and that can be easy to miss, also there is no stack trace, just that failure count 16:50
gary_posterugh, weird16:51
bcsallergary_poster: its working now though16:52
gary_posterunderstood bcsaller :-) just an unpleasant failure state16:52
bcsallergary_poster:  should I repropose or submit16:53
gary_postersubmit unless you actuvely want a re-review bcsaller.  your call16:54
bcsallergary_poster: if anything comes up at this point its another branch/bug, going to submit16:55
gary_poster+1 bcsaller 16:55
thervegary_poster, I'm seeing the same thing on ec2...16:57
thervedoing "juju deploy cs:~juju-gui/precise/juju-gui"16:58
gary_posterteknico, I'm going to go have some lunch.  shoot me a mail with details if you want me to run with the branch after your EoD.  Would be great to get it landed, and I'd be happy to help it happen.  Ben's branch is about to land, which should be the last blocker16:58
gary_postertherve, ok will investigate now16:58
teknicogary_poster, will do16:58
MakyoBack in a few, something's wrong with Unity.16:59
gary_postertherve, I have a hypothesis.  I think we have "juju-origin: ppa" in our environments.yaml, and you don't.  Can you confirm that you do not?17:10
thervegary_poster, that's correct. Let me try that17:10
gary_postertherve, that's still something we want to fix very soon, but if that unblocks you for now, that would be great17:11
thervegary_poster, seems to work, thank you!17:14
gary_postertherve, great.  Thanks for bringing this to our attention.  Looks like it will be mildly annoying to address, but important. :-)17:14
benjiwe should move lib/views/stylesheet.less to a saner loction; it just took me three or four minutes to find it18:06
benjiI am seeing a couple of different odd behaviours around adding relations in the environment view (trunk), is that a known issue?18:14
Makyobenji, Specifics?18:14
benjiMakyo: When I click "Build Relation" I see an orange line drawn horizontally off to the left.  The line does not follow the cursor.18:15
Makyobenji, okay.  Mind filing a bug?  Will get to it after this recess thing.18:16
Makyo(which sounds more fun than it is)18:16
gary_posterbcsaller, when I go to https://codereview.appspot.com/7062056/diff/1/app/views/topology/service.js?column_width=80 to try and review your most recent branch I see "Upload in progress."18:16
benjiMakyo: will do.18:16
gary_posterIs that what you see?18:16
Makyobenji, thanks.18:16
bcsallergary_poster: I do, I'll re-propose18:20
gary_posterthanks bcsaller 18:21
benjiMakyo: https://bugs.launchpad.net/juju-gui/+bug/109787118:21
_mup_Bug #1097871: Pending relation line does not follow cursor. <juju-gui:Triaged> < https://launchpad.net/bugs/1097871 >18:21
gary_posterCharm review available for problem encountered above: https://codereview.appspot.com/7073051.18:22
gary_posterFiling a bug now18:23
bcsallergary_poster: oddly, clicking on the 'D' services entry shows the diff, its an issue with a delete/move18:23
gary_posterah, ok bcsaller.  I can try to work around18:24
bcsallerRietveld bug of the day18:24
gary_posterbcsaller approved to land as is18:31
bcsallergary_poster: skip the second review cause its simple?18:31
gary_posterbcsaller, fine by me.18:32
* Makyo skips out of Linter Londerland and grabs lunch.18:53
hazmatgary_poster, nice catch re ppa19:42
gary_posterthanks hazmat 19:42
benjido charm URLs from the local repo have this form?  "local:foo"?20:07
benjido they have version numbers too, like the charm store URLs?20:07
benjii.e., local:foo-720:07
gary_posterI'm not sure benji, sorry--but yes, I think so.21:39
gary_posterbenji, please make sure you hand off before you depart21:39
benjigary_poster: good; I ran with the assumption21:39
benjigary_poster: will do; hmm I guess that is now21:40
gary_posteryeah :-)21:40
benjithe only thing I have left are some integration tests, which are prooving very tricky21:40
benjiI'll write it up and send it to the list21:40
gary_posterMakyo, if you are up for it could you review https://codereview.appspot.com/7060066/ and possibly also https://codereview.appspot.com/7073051?21:41
Makyogary_poster, yep.21:41
gary_posterI'll look at the linter londerland branch21:41
gary_posterthank you21:41
benjiEver since I followed on twitter the pro gamer I met on the plane ride back from Copenhagen Twitter has been suggesting exclusively Danish tweets to me.21:42
* benji notices the tortured structure of that sentence.21:42
benjigary_poster: see you Monday21:59
gary_postersee you Monday benji21:59
gary_posterMakyo, approved with changes.  Thank you for that branch.  Will be cool to have.22:11
gary_posterI like the changes you want to make in the future too--turning on more options22:11
Makyogary_poster, cool,. thanks!  Gotta walk the dog quick,but will do so when I get back.22:11
gary_postercool Makyo.  ttyl22:12
hazmatbenji, take care22:13
benjithanks hazmat 23:24

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