/srv/irclogs.ubuntu.com/2013/08/29/#juju-gui.txt

=== rogpeppe1 is now known as rogpeppe
=== schwuk_away is now known as schwuk
hatchshhhhhhhh13:05
TheMuefrankban: ping13:19
frankbanTheMue: hi13:20
TheMuefrankban: hello, it's me again13:21
TheMuefrankban: I've rolled back the changes regarding ServiceSet due to the API incompatability13:22
TheMuefrankban: but william told me that you're thinking about an API change make larger settings/unsettings at once13:23
TheMuefrankban: am i right? in that case we could also use this new API call to allow the explicit unsetting13:23
frankbanTheMue: as gary_poster mentioned, for the time being we can live with that incompatibility, meaning the GUI users will set string options to an empty string, or receive an error for bool/int/float fields13:24
TheMuefrankban: the problem is that other parts of the software use the API too, so it should not get incompatible13:26
frankbanTheMue: you mean, other parts of juju-core?13:26
TheMuefrankban: yep13:27
frankbanfwereade_: re large settings/unsettings at once are you referring to the ServiceUpdate for multiple services?13:28
frankbanTheMue: anyway, I believe you are right, we can take care of options unsetting when we improve the service API calls13:36
TheMuefrankban: yes, so we should stay in contact then13:38
frankbanTheMue: sure13:38
TheMuefrankban: any ideas about it already or do we start on a green field?13:38
TheMuefrankban: and i have to find a good we to allow setting string options to empty strings by cli without changing the api behavior13:39
frankbanTheMue: I think William already has some ideas for ServiceUpdate and ServiceDeploy, and some design decision is required13:39
adeuringabentley, sinzui: could one of you have a look at this MP: https://code.launchpad.net/~adeuring/charmworld/spurious-failures/+merge/182901?13:39
TheMuefrankban: ok, will ask William when he's back13:40
abentleyadeuring: sure.13:42
frankbanrick_h: could you please take a look at https://codereview.appspot.com/13386043 ? it's the chrome dev fix.13:55
hatchMakyo: review done13:57
Makyohatch, thanks.13:58
Makyohatch, none of the others are sprited, can we defer that to a separate branch?13:58
MakyoA sprite-ageddon, if you will.13:59
MakyoWill respond to comments instead of here, sorry.14:00
fwereade_frankban, hey, sorry, I don't recall what the progress was on ServiceUpdate14:05
fwereade_frankban, I had a feeling you were doing something with it14:06
hatchMakyo: yep np14:10
rick_hfrankban: sure thing14:10
frankbanfwereade_: ServiceUpdate is implemented for a single service. And for now it takes SettingsStrings as a map[string]string, so there is no way  to unset an option. TheMue proposed a new API call for that. Anyway, I had the crazy idea of changing SettingsStrings to be a map[string]*string, not sure that applies well.14:10
frankbanrick_h: thanks14:10
frankbanfwereade_: from the GUI perspective, rather than unsetting an option, we could set the default value for that option.14:12
TheMuefrankban: unsetting an option is one part of the game, but setting a string option to an empty string another one14:14
TheMuefrankban: and this shouldn't been done with ServiceSet, because here option="" is interpreted as unsetting it. and this should not be changed14:16
frankbanTheMue: yes I know, I was just saying that, from the GUI perspective, a default value in the juju-core db == no value. And that we can react without too may problems to possible changes in how Service(Set|Update) are implemented.14:18
TheMuefrankban: ok14:19
hatchrick_h: you around?14:22
rick_hhatch: yep14:22
rick_hwith that lovely post-dentist "I've been touched in strange places" feeling :P14:22
hatchjust a very small FYI on your zIndex branch - the sidebar now shows above the 'modal' blackout when removing relations and the dialogue pops up14:23
hatchI hate modals so I would remove the modal14:23
hatchbuuuuuut14:23
rick_hhatch: ok, well, they should be adjusted. I tried to set a 'standard' for where z-indexes should lie and they can be adjusted clearly now.14:23
rick_hhatch: I added a doc chunk to the top of the stylesheet.less walking through it14:24
rick_hshould be a two-liner14:24
hatchahh cool cool14:24
hatchI just noticed it in QA but it was definitely not blocking lol14:25
rick_hhatch: k, yea my bad. I tried to look over anything with a manual z-index to make sure it looked safe, but was hard to tell 14:25
hatchyeah that's dynamic I think14:25
rick_hfrankban: looks good, small note on the doc comment14:26
hatchworking from the laptop today....boy is the console ugly14:26
* hatch can't wait for rick_h to fix it for me14:26
gary_posterbac, sorry for missing time.  May I move our call to 2 PM14:26
rick_hhatch: console is ugly?14:27
rick_hoh, your tmux/zsh issues?14:27
hatchyeah I tried to customize it....and broke it14:27
gary_posteroh bac not here right14:27
gary_posterforgot14:27
frankbangary_poster: one review+qa is also valid for charm branches, correct?14:27
frankbanrick_h: thank you!14:28
gary_posterfrankban, sure.  we haven't agreed to it yet, but sure, give it a try and report back. ;-)14:28
frankbanok thanks :-)14:28
frankbangary_poster: and, if it's not clear, I am +1 on that proposal, including possible team reviews of parts of the code, as discussed in the juju ml14:29
gary_posterfrankban, cool :-)14:29
rick_hfrankban: thank you for fixing it up. My use of chrome dev can continue!14:31
hatchnice14:31
frankban:-)14:32
rick_hhatch: ooh, new YUI and Template landed14:33
hatchok so at our rate in about 3 months I'll get to upgrading it :P14:33
frankbanrick_h: juju-core + lxc + juju switch is really cool. and, for when you need pyjuju, a virtualenv + virtualenv wrapper + a single symlink work well ("workon pyjuju" and you are done)14:34
sinzuihi bac: Would you have time today to review https://code.launchpad.net/~sinzui/charmworld/support-gui-charm-urls/+merge/18292414:47
hatchbrb running to drop the car off at the shop, back in 1514:58
Makyojujugui call in 9 kanban now15:51
gary_posterjujugui, I'll attend http://summit.ubuntu.com/uds-1308/meeting/21899/servercloud-s-juju-new-user-ux/ which conflicts with our standup. :-/  I feel like I need to be at both.  Makyo, would you mind running and taking some basic notes of anything important not covered by the kanban?15:56
Makyogary_poster, sure.15:56
gary_posterthank you Makyo 15:56
Makyojujugui call in 215:58
sinzuiabentley, benji : Would you have time today to review https://code.launchpad.net/~sinzui/charmworld/support-gui-charm-urls/+merge/18292416:12
hatchjujugui can I get a review/qa https://codereview.appspot.com/13249045/16:17
hatchbcsaller: did you QA?16:22
hatchthanks for the review16:22
bcsallerI didn't16:22
hatchyou should :D16:24
rick_hhatch: I can qa, looking at now16:28
hatchcool thanks16:28
hatchrick_h: I was thinking the same thing as your perf comment and figured that it was virtually insignificant either way16:36
rick_hhatch: yea, I guess I just wanted to raise it. 16:38
benjijcsackett: your next dev environment: http://www.youtube.com/watch?feature=player_detailpage&v=8SkdfdXWYaI#t=54516:41
hatchbenji: that's real cool16:43
=== schwuk is now known as schwuk_away
jcsackettbenji: that is awesome.16:59
benji:)16:59
hatchrick_h: so a couple days of use with Sense 5 and I can tell you that it has some cool features but I muuuuuuch prefer stock android17:00
rick_hhatch: yea, I'm all nexus/stock any more17:01
hatchI'm hoping I can get the Zoe camera app apk for the stock OS17:01
hatchbut I won't do that until I'm back home17:01
hatchdont' want problems while away hah17:02
hatchI wish tracking numbers had a delivery time17:14
hatchyuss finally someone else who agrees with me that the search is broken :P17:21
hatchour power is growing17:21
hatchgary_poster: I was thinking of tackling 1214087 but the comments are a little confusing17:23
hatchhttps://bugs.launchpad.net/juju-gui/+bug/121408717:23
_mup_Bug #1214087: GUI fails to deploy keystone <juju-gui:Triaged> <keystone (Juju Charms Collection):New> <https://launchpad.net/bugs/1214087>17:23
hatchit reads to me that it's a charm problem but then you want us to only send changed config values?17:24
gary_posterhatch, yes17:30
gary_posterhatch, you see why?17:30
hatchyep17:30
hatchso should this ticket be renamed then?17:30
gary_posterhatch, fwiw, openstack is quite arguably our most important Juju story right now17:30
hatchand I'm guessing keystone is part of that?17:31
hatchheh17:31
* hatch googles keystone17:31
gary_poster:-) yeah17:31
hatchoh yeah that is a pretty important part17:31
gary_poster:-)17:31
hatchok I'll add that to the config save17:32
gary_posterhatch I wouldn't rename bug because both should happen: we can only send changes, because that is reasonable; and keystone charm should not be so persnickety :-)17:32
gary_postercool hatch.  ghost config is more important than post-deploy config if you have to choose17:32
gary_posterthank you17:33
hatchhttps://bugs.launchpad.net/charms/+source/keystone/+bugs odd this picks up the wrong status for our ticket17:34
hatchgary_poster: in attempting to reproduce the issue using rapi I get an error that it can't understand the constraints...not the config error17:41
hatchI'd like to reproduce the error first so I know that it's fixed or not17:42
gary_posterhatch ack on call.  could be that the "undefined" thing is a problem17:42
hatchok I'll peek into that first17:42
hatchok the error is `juju.errors.ConstraintError: Could not interpret u'mem' constraint: need more than 1 value to unpack`17:57
hatchI should mention `u` prefixes every json key value17:58
hatchthe data being sent is correct18:01
rick_hhatch: that's a python thing, ignore it18:03
rick_hhatch: basically saying it was a string 'mem'18:03
rick_hu'mem' means unicode string18:03
hatchohh ok18:04
hatchthe syntax looks to match the syntax in the python tests18:04
hatchrick_h: looks like we can't deploy anything on rapi in the new inspector18:05
hatchbecause of the constraints18:05
hatchdo you have time to pair on this?18:06
rick_hhatch: sure thing18:06
* rick_h can catch up18:06
hatchcool guichat18:06
hatchgary_poster: here is the constraints issue https://bugs.launchpad.net/juju-gui/+bug/1218555 working on it now18:30
_mup_Bug #1218555: Constraints are sent in the incorrect format with new inspector to rapi <juju-gui:Triaged by hatch> <https://launchpad.net/bugs/1218555>18:30
hatch$50 to get the unlock code from my carrier.....18:44
MakyoMain laptop battery: ~40 mins. Backup laptop battery: ~20 mins.  I hope the flight to London involves power outlets :|18:44
* hatch searches for a cheaper code18:44
hatchMakyo: :'(18:44
gary_posterack and thanks hatch18:45
* gary_poster not really here for a bit, because of early morning/late day Thursday18:45
sinzuibenji, abentley . I deployed your revisions to staging. Mine works, but I did not notice the bread crumbs changed: http://staging.jujucharms.com/~gnuoy/precise/apache2-118:56
benjiI... don't think I know anything about bread crumbs.18:57
abentleysinzui: having bad internet weather here.18:57
sinzuibenji, abentley. Maybe I should put "revision" in the breadcrumbs.18:57
sinzuiabentley, floods again?18:58
abentleysinzui: No, just random slowness.  Sunny today, actually.18:58
benjithe "Apache2 1" at the end is suspect18:58
sinzuibenji, my current branch will reverse the breadcrumbs and put them in the page title, I don't think revision is important, though we might want to keep it because it distinguishes it from the versionless charm18:59
sinzuiBC: Home/ Gnuoy/ Precise/ Apache2  revision119:00
benjiyeah, I think including the revision helps people know that they're looking at a particular revision which would seem to be important19:00
sinzuiTitle: Apache2 revision 1 : Precise  : Gnuoy : Juju Charms.19:01
sinzuiOh, we still say "Charm Browser" in the heading. That must say Juju Charm Browser or just Juju Charms19:02
rick_hbcsaller: I saw Makyo reviewed. So if we can't dedupe the code then is it cool to leave current feedback as more tests required then? The code on things like sub relation indicators, exposed indicators, positioning tests, x/y annotation verification, etc need to be added?19:09
bcsallerrick_h: thats cool, yeah. In the end more of this will be broken out into smaller functions and the level of reuse can go up, and there are somethings like sizes that should become configuration rather than hardcoded values 19:18
rick_hbcsaller: k, I don't want to hold up things and wanted to double check what the plan is from here then. 19:20
abentleyorangesquad or benji: Could you please review https://code.launchpad.net/~abentley/charmworld/prefix-bundles/+merge/182992?19:21
bcsallerrick_h: Honestly I expect that the drawing code in there will undergo quite a bit of change once UX sees it, there is very little info on what people want, this is more a 'show capabilities' branch for now19:21
benjiabentley: I'll take a look19:22
abentleybenji: Thanks.19:23
abentleybenji: What trailing slash do you mean?19:51
benjiabentley: the import_filter is a prefix of the branch path, right?19:52
abentleybenji: right.19:53
benjiabentley: therefore, a filter of "XXX/foo" would match "XXX/foo/bar" and (erroniously) "XXX/foozzle", right?19:54
abentleybenji: I wouldn't consider the second erroneous.19:54
benjihmm, maybe I misunderstood the purpose of _filtered_payloads.  I thought it was meant to match on path segments, not just on a character-by-character basis19:55
abentleybenji: No, it is just a prefix.19:57
benjiabentley: so, would it ever make sense to use it to match part of a path (like my second example)?19:57
abentleybenji: In reality, if it starts with "f", it has to be "feisty", anyhow.19:57
benjiok, I guess it's not worth worrying about then19:58
=== sinzui is now known as sinzui_walking
hatchso hows everyones afternoon?22:04
rick_hthe shed base is reinforced, shelves setup, and more crap shoved in it. now to cool off in the basement, phew22:05
hatchgota love some good time in the yard22:05
hatchor garage22:05
hatch:)22:05
rick_hyea, sprinkler folks moved the head under the shed out of the way so can put things back together22:05
rick_hyea, I can't wait until winter to get back into the woodshop22:06
rick_hI'm getting an aching for some woodworking. Need more hours in the day22:06
hatchyeah - winter is when I can work on the cars22:06
hatchsummer should be the time, but who has the time!22:07
rick_hright, as much fan as camping has been, looking forward to finishing up our last trip in Oct and winterizing it22:07
hatchI thought you park it indoors?22:07
rick_hyea, but it's not heated. 22:07
rick_hjust a large warehouse style building22:07
rick_hso protected from the elements, but no promise it won't get cold in there22:08
hatchahh gotcha22:08
rick_hhatch: is your branch up for review? /me missed that go by22:11
hatchI am just lboxing now22:12
hatchfrom my laptop though so it'll take 4 evaaaa22:12
rick_hah, ok. 22:12
hatchrick_h: I'd like it if you could QA this because it needs a QA on all three backends but I realize it's past your EOD https://codereview.appspot.com/13252045/ so tomorrow would be fine too22:19
huwshimiMorning22:28
hatchmornin huwshimi22:31
gary_posterhi huwshimi.  my wife is still at dr.  I'll ping you when I am available.  hopefully soonish22:58
huwshimigary_poster: No problems, any time.22:59
gary_posterbcsaller, did you ever get your necessary reviews?23:15
huwshimigary_poster: Happy to postpone if it's getting to late for you. We'll be in the same room on Monday :)23:15
bcsallergary_poster: one LGTM and now we are down to one with requests for tests paralleling the service module23:15
gary_posterhuwshimi, heh, true, but I think we should talk.  can do it now.  you ready?23:15
huwshimigary_poster: Yep23:16
gary_posterbcsaller, oh ok, so two reviews but one response in progress, cool thx23:16
hatchgary_poster: whenever you have a moment - do we have a mockup for the way the unit relation view is supposed to look like? Or just make it look decent?23:38
gary_posterhatch, no mockup.  decent, based loosely on existing design (which is engineer design, but whatever :-) )23:39
hatchhey, I've always said engineers develop the best UI because it's the most efficient lol23:39

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