/srv/irclogs.ubuntu.com/2012/12/04/#juju-gui.txt

=== Makyo is now known as Makyo|out
=== rogpeppe1 is now known as rogpeppe
=== mattuk1972_ is now known as mattuk1972
gary_posterteknico, how is your card going?13:14
gary_posterbenji, should I review your branch or wait until you have resolved the issue Makyo raised?13:17
gary_posterwould it be useful to see if I can duplicate his issue?13:18
benjigary_poster: I don't have a strong opinion either way.13:18
benjiit couldn't hurt :)13:18
gary_posterbenji, ok, I'll see if I dupe.  If I do, I'll wait on its resolution before a full review.13:18
benjik13:19
hazmatgary_poster, meeting this am is a one time evt13:23
gary_posterbenji, duped.  Also, make debug, at least, seems worse in that it will repeatedly try to make files that should already have been made, such as sprite file.  trunk is quiet.13:24
benjihmm13:24
benjiok, thanks 13:24
gary_posterhazmat ack thanks.  10:30 will normally work fine, just not this week?13:25
hazmatgary_poster, just not today13:27
gary_postercool hazmat thanks.  will adjust, and reply to mail13:27
bachi mattuk1972 would you mind taking a look at the visuals for the charm store filter widget to ensure it looks like you expect?14:10
mattuk1972bad, sure in a meeting - will be out in a few14:26
mattuk1972bac,14:26
bacbenji: second review done.14:50
benjibac: thanks14:50
mattuk1972bad, could i see it without the 1px stroke? it seems too much here -also the text and tick need some padding to bring them in a bit. basically the same as the text has before you click in it to open the chooser.14:58
bacmattuk1972: you can play with the CSS in your browser, as we did before15:01
bacmattuk1972: i had trouble placing that tick mark.  i know it is poorly placed but i cannot figure out how to move it in.  the CSS that should do the trick is ineffective.15:02
mattuk1972bac, yep it doesn't seem to line up with the text either15:03
mattuk1972should i fond another way to show selected?15:03
bacnope15:03
bacmattuk1972: my "nope" reply was to your previous message.  as to finding another way of marking the selected item that may be a good idea but i'd wait until i've exhausted all attempts at getting the check mark to place correctly.15:10
goodspudgary_poster, could you please send me a new invite to our meetings 15:14
goodspudAnd how do I sign up to our mailing lists (like juju-gui peeps)?15:16
gary_postergoodspud, mailing list: please join https://lists.ubuntu.com/mailman/listinfo/juju-gui15:17
gary_posterwe are trying to dump peeps15:17
gary_postergoodspud, reinvited you to mtg15:18
goodspudThanks gary_poster15:50
gary_posterwelcome15:50
goodspud... although i haven't received the meeting invite email yet....15:52
gary_postergoodspud, don't know.  deleted and added you.  what do you need?  should be on your calendar15:56
goodspudNope, not on my calendar. I figured I'd get an email with the details... stoopid technology15:57
gary_posterbac bcsaller benji frankban hazmat Makyo mattuk1972 teknico call in 2, unless hazmat and my call goes over...15:58
gary_postergoodspud, I used nick.boettcher@canonical.com15:58
gary_posteranother?15:58
goodspudThat should work... perhaps the mail server is super slow. I'll have some patience15:59
gary_posterok15:59
gary_posterlooks like call is going over...?15:59
gary_posteryup16:00
gary_postersorry16:00
gary_posterdon't know how much yet16:00
gary_posterbac are you willing to run the meeting today?16:01
bacgary_poster: doing so16:03
gary_posterthank you16:04
bacMakyo i'll review your endpoints card.  i added my tag16:11
Makyobac, thanks.16:11
hazmatgary_poster, reap the whirlwind :-)16:12
gary_posterbcsaller, ready in 10 if you want to pair16:12
benjibcsaller: if you want to see what I was talking about in the standup, take a peek at line 50 of lib/server.js in trunk16:13
gary_posterhazmat it seems like a whirlwind certainly :-)16:13
bcsallerbenji: I'll take a look16:13
* gary_poster restarts16:14
=== gary_poster is now known as gary_poster|away
bcsallerbenji: that is insane, its under assets as well which isn't even the right place, so rather than fix a relative path issue there was a dynamic workaround it looks like16:15
goodspudbcsaller, just as an aside, what part of the world are you in at the moment?16:18
bcsallergoodspud: refactoring the frontpage view to better support the upcoming features and to allow more of us to work on it at once 16:19
bcsallergoodspud: unless you mean geography, in which case NYC16:21
goodspudbcsaller, geographically.... but good to know what you are working on anyway16:22
goodspud:)16:22
goodspudbscaller, just seemed like you'd moved from San Fran... it was bright and sunny out your window16:22
bcsallergoodspud: today it is. I'm only here for a while, maybe another couple months16:23
=== gary_poster|away is now known as gary_poster
gary_posterbcsaller, I lied.  Decided to make some gazpacho for a bit.  back soon.  will ping16:26
bcsallergary_poster: np16:26
bacumm, the cold soup of spain16:28
* bac heads to tapas joint16:29
teknicogary_poster, I'd like some, thanks :-)16:32
gary_poster:-)16:51
* bcsaller grabs some food, back in a few17:05
benjiMakyo: my branch should be much saner now, I would love it if you would try it again17:37
benjibac: I suspect my branch does most of what you want for your card; I would be happy to discuss it if you like17:50
bacbenji: i figured it might17:50
benjigary_poster: my branch is ready for your review (https://codereview.appspot.com/6865054/)17:51
gary_posterbenji, don't you already have two reviewers?  I'm OK with it if they are, though I'm happy to look if you'd liek me to.17:51
gary_posteror even like17:51
benjigary_poster: yep I have two, so unless you are super excited about reviewing it, no need17:52
bacbenji: is your branch merged?17:52
benjibac: if lbox does it, then yes... I thought it did17:52
benji"No new revisions or tags to push."17:53
gary_posterbenji, not super excited :-)17:53
benji:)17:53
Makyobenji, I'll forever wonder about that regex, always, in the back of my mind..18:00
benjiInflicting obscure psycological pain is my greatest joy in life.18:00
bacbenji: the rule for $(BUILD_ASSETS_DIR)/images fires everytime 'make build' is used, whether it needs to or not.  i can't figure out why18:03
* benji looks18:03
benjibac: did you uncomment the debug helper at the top of the Makefile?  It can help a little.18:04
bacbenji: nope.18:04
benjibac: I don't see that behavior, do you want to pair on it?18:04
bacsure18:05
benjibac: you get tb ready and I'll make a hangout18:05
benjibac: https://plus.google.com/hangouts/_/15f7b6bcb0736024fb1834d49ca7c6acf56a4080?authuser=0&hl=en-US18:07
hazmathmm.. the charm readme covers testing, but really it should be talking usage for a user18:17
hazmatalso 18:20
hazmatno mechanism to change config after deploy18:20
Makyohazmat, should I create a config-changed hook card?18:27
hazmatMakyo, sounds good18:27
Makyohazmat, can do a README card, too, while I'm at it.18:27
Makyohazmat, for the README, should the testing information be moved to a different file, or just kept at the end of the current one?18:32
gary_posterThe config changed is a good idea, but also a nice to have IMO: I would make a crad but put a low priority.  README for users is a normal priority one18:34
hazmatMakyo, separate file. 18:36
hazmatgary_poster, requiring config at deploy time is a bad semantic for a charm, its more than a nice to have imo18:37
hazmatMakyo, i'd suggest DEVELOPER.txt18:37
hazmatgary_poster, we have several folks wanting to do a demo of it.. the deploy process is already convulted enough, having to switch out defaults at deploy time is kinda of painful to..18:38
hazmatthis is my current instructions on deploy fwiw http://pastebin.ubuntu.com/1410842/18:39
gary_posterTechnically, it's a nice to have.  It will still work fine without it, and that's the way I've worked with other charms--with a config file.  But eh.18:39
hazmatfor a staging env its also echo / cat to config.yaml and pass along for deploy.. much nicer if we can juju set staging=true18:40
hazmatgary_poster, i consider charms requiring a config file to generally be broken18:40
hazmater.. badly done let's say18:40
* gary_poster not going further with it :-)18:40
hazmatit means they take a bunch of config options which they don't accept18:41
* hazmat takes a shallow breath and moves on18:42
MakyoAnyone else unable to get to the board?20:52
bacMakyo: yes, i cannot connect to the server either20:53
bacMakyo: https://leankitkanban.com is gone too20:54
bacfrom twitter: LeanKit ‏@LeanKit20:57
bacWe have an unexpected outage at the moment. The team is diagnosing. Resolution ASAP.20:57
Makyobac, ah, thanks!  Not urgent, just realized I should move cards.20:57
bacall of this happened an hour after they announced their new Chief Marketing Officer.  coincidence?20:57
MakyoMaybe they hit the big red Door Open button in the server room?21:00
benjiheh21:02
hazmathaving odd issues with the charm http://pastebin.ubuntu.com/1411132/21:03
benjihazmat: I touched those rules recently, but I didn't have any problem with them; looking at it again to be sure21:05
* bac gives in to frantic dog. bbiab.21:08
benjihazmat: I can't reproduce the problem21:11
hazmatbenji, try it with jitsu deploy-to 021:21
benjihazmat: try what?  the charm?  I was just running the make file; I haven't deployed the charm before21:21
hazmatbenji, the charm yeah..21:38
Makyohazmat, the GUI dir is chowned in hooks/start, but make is run in hooks/install.  I've not run into the problem with juju deploy, but maybe jitsu deploy-to is calling it differently?21:38
hazmatMakyo, the hooks get called the same.. the deploy is the same .. its just different on the front end cli21:38
Makyohazmat, ah, alright.  Curious that it's a permission denied thing, since it's in the same hook that's creating the directory.21:41
hazmathmm.. these hooks are idempotent21:52
hazmater. aren't21:53
hazmatcan't rerun it to resolve, as it will barf on the checkout21:53
hazmatfull charm log fwiw.. bottom is a resolved --retry ... http://paste.ubuntu.com/1411248/ 21:54
hazmatMakyo, good point.. i think i foobar'd it running by hand.. it looks like its needs root..21:57
hazmataha.. the actual error.. 21:57
hazmathttp://paste.ubuntu.com/1411255/21:57
MakyoThough the 'ln -sf ....yui' bit succeeded this time.21:58
hazmatMakyo, yeah.. i was running it as a normal user instead of as root with the first pastebin22:00
Makyohazmat, oops, I see.22:00
MakyoOh, huh.  ln succeeds if the target exists, and creates a bad link.22:02
MakyoSuppose that...sort of makes sense.  Can make the link, create the file later.22:02
=== gary_poster|away is now known as gary_poster
gary_posterbcsaller, worth checking up now, or do it tomorrow?\22:04
bcsallergary_poster: lets do it tomorrow22:04
gary_postercool bcsaller.  have a good evening22:04
bcsalleryou too22:05
hazmatbenji, i think we need to revert that last commit..22:13
hazmatgary_poster, it works on 26222:13
gary_posterhey benji.  I asked hazmat to revert.  Please make a card for getting that landed again and move it through as "critical"22:15
hazmatbenji, fwiw the error is http://paste.ubuntu.com/1411255/22:15
* gary_poster stepping away22:16
gary_posterhave a good night22:16
benjigary_poster: will do22:16
bacthanks for the review benji22:18
benjihazmat: if you feel so inclined, I would appreciate testing the pre-reverted code with just this bit reverted in the Makefile: http://paste.ubuntu.com/1411289/22:19
hazmatbenji trying out 22:24
hazmatunrelated.. the makefile doing a bzr file check outside of a target is actually quite painful when there is no local bzr repo.. the charm uses a lightweight co.. and its a multi-second pause22:26
hazmatand of course wouldn't play nice with a tarball22:26
hazmatbenji, those diff chunks don't match up22:44
hazmathmm... or i already applied..22:45
hazmatyeah.. no diff to revno 26422:46
hazmatbenji, perhaps you mean the inverse of the diff you pasted22:49
hazmatno help with the inverse either.. reverting22:55

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