| rick_h_ | curses!!!! test pass with .only, fail when run with everything else. | 00:15 |
|---|---|---|
| gary_poster | :-/ | 00:16 |
| rick_h_ | it's around date formatting...how can that get corrupt? Man. | 00:19 |
| gary_poster | yuck | 00:19 |
| rick_h_ | sorry, just ranting. | 00:20 |
| rick_h_ | was hitting -cr thinking I was done for the night :P | 00:20 |
| gary_poster | I understand the feeling :-) | 00:20 |
| rick_h_ | oh well, will look tomorrow with fresh eyes. party on | 00:21 |
| gary_poster | :-) ttyl | 00:23 |
| === jcsackett_ is now known as jcsackett | ||
| gary_poster | fwiw, unexposed issue has gone away somehow, and I fixed Landscape issue. yay. | 01:29 |
| gary_poster | nytol | 01:38 |
| frankban | rogpeppe: hi, re CharmInfo error, from a first investigation, it seems to be a GUI error: the GUI calls CharmInfo before the charm is added to state, i.e. before the charm is deployed. makes sense? | 11:17 |
| rogpeppe | frankban: that makes sense, but... what charm is it trying to get? | 11:18 |
| rogpeppe | frankban: if the charm isn't added to state, why is the GUI trying to get info on it? | 11:19 |
| frankban | rogpeppe: because, I suspect, of a bug in the GUI. For example, https://ec2-50-17-53-143.compute-1.amazonaws.com | 11:20 |
| rogpeppe | frankban: that all seems to work ok for me | 11:21 |
| rogpeppe | frankban: BTW, unrelated remark, shouldn't those service icons show the number of deployed units for each service? | 11:22 |
| rogpeppe | frankban: oh, i see, it only shows that when i hover the mouse over it | 11:22 |
| frankban | rogpeppe: if you click on top-right "Charms" and then, e.g. "cassandra", and then "Deploy", a charminfo is requested for cs:precise/cassandra-2 | 11:22 |
| frankban | (before the charm is deployed) | 11:22 |
| rogpeppe | frankban: ah, i see! | 11:23 |
| rogpeppe | frankban: that seems to argue that you might want a separate PutCharm command | 11:23 |
| rogpeppe | frankban: then you can get the API to download a charm and add it to the state without necessarily deploying it | 11:24 |
| frankban | rogpeppe: I have to investigate further, who is doing that call, what;s the reason? AFAIK we also have the ability to retrieve charm info from the store | 11:25 |
| rogpeppe | frankban: yeah, actually, i was just thinking that | 11:25 |
| rogpeppe | frankban: well, i'm glad it's not a juju-core bug :-) | 11:25 |
| frankban | rogpeppe: well, me too :-) | 11:26 |
| rogpeppe | frankban: nice to see the GUI working, BTW, and with double-clicking on services working too! | 11:26 |
| frankban | rogpeppe: yes, very cool | 11:26 |
| * frankban lunches | 11:27 | |
| rogpeppe | frankban: i've just pointed people to have a look at your gui instance | 11:27 |
| frankban | rogpeppe: ok, I will use that for a debugging session after lunch, so it will be around for a while | 11:28 |
| rogpeppe | frankban: hmm, it's a bit broken. | 11:29 |
| * teknico lunches | 12:04 | |
| gary_poster | rogpeppe, frankban if we are talking about a service from the Juju environment, we should get the charm from the Juju environment IMO. We have a variety of future stories about proxied charm stores | 12:04 |
| rogpeppe | gary_poster: it sounds like this charm info call is happening before the service is deployed. currently there's no way to get charm info from the juju env unless an instance of that charm is deployed | 12:05 |
| frankban | gary_poster: I confirm the GUI calls CharmInfo before the charm is deployed (after "deploy" is clicked but before "confirm") | 12:06 |
| gary_poster | rogpeppe, ack; just saying that if it is supposed to be getting information about a service, we should ideally make sure that we are asking the environment about it. ok frankban, thanks | 12:07 |
| gary_poster | (rogpeppe, we actually have separate stores for the charm info we obtain from the store and the charm info we obtain from the environment) | 12:08 |
| rogpeppe | gary_poster: interesting. | 12:08 |
| gary_poster | perhaps paranoid, or too future looking | 12:08 |
| gary_poster | my fault if so :-) | 12:08 |
| gary_poster | teknico, you probably know about the relation.ids that have spaces in them right? We may need to normalize them | 12:11 |
| gary_poster | well, perhaps we can do that in our d3 code | 12:12 |
| gary_poster | probably a better place | 12:12 |
| rogpeppe | gary_poster: so why is deploy calling CharmInfo before the charm has been deployed? surely it should wait for the deploy to be successful, *then* get info on the charm from the environment? | 12:19 |
| gary_poster | rogpeppe, 1 sec, writing diagnosis for bug 1167295 :-) | 12:20 |
| _mup_ | Bug #1167295: graphical link not updated properly (juju core) <juju-gui:Triaged> < https://launchpad.net/bugs/1167295 > | 12:20 |
| teknico | gary_poster, I don't! I got nutin' to do with 'em! I don't even *like* them! frankban made them, I saw him! he's the culprit! take him, not me! | 12:22 |
| gary_poster | teknico, :-) | 12:22 |
| teknico | (and I'm lunching anyway, ttyl :-) ) | 12:22 |
| gary_poster | ttyl | 12:23 |
| frankban | gary_poster: aha! the relation keys used by juju-core have spaces on them,e.g. "haproxy:reverseproxy wordpress:website". if we use the key as is in the DOM classes this could be the problem | 12:23 |
| gary_poster | frankban, right :-) I just put that in the bug | 12:24 |
| gary_poster | (s) | 12:24 |
| gary_poster | I was unable to deploy dokuwiki | 12:25 |
| gary_poster | no helpful error in gui | 12:25 |
| frankban | gary_poster, rogpeppe: maybe an helpful one in the juju logs: http://pastebin.ubuntu.com/5695209/ | 12:26 |
| gary_poster | ooh, we still convert the ghost to a real service even on error | 12:26 |
| gary_poster | oh... | 12:27 |
| gary_poster | it actually is in the go environment | 12:27 |
| gary_poster | so not a problem on gui side | 12:27 |
| rogpeppe | frankban: interesting | 12:27 |
| frankban | gary_poster: if you didn't see the "no options to set" growl notification, that's another bug | 12:27 |
| gary_poster | ah, I didn't | 12:27 |
| gary_poster | Well, I saw *a* growl notification | 12:28 |
| gary_poster | which probably correlates to that error | 12:29 |
| rogpeppe | frankban: looks like a bug to me. i don't think "no options to set" should be an error | 12:29 |
| gary_poster | but I didn't see the message in growl or details | 12:29 |
| gary_poster | frankban, I can't dupe the CharmInfo thing. | 12:30 |
| rick_h_ | w...t...f... | 12:30 |
| rick_h_ | so my view module requires the datatype-date datatype-date-format. cool | 12:31 |
| rick_h_ | but when running tests, it's not loading the en-US module | 12:31 |
| rick_h_ | unless I require those modules also in my test's .use() | 12:31 |
| rick_h_ | then it's peachy | 12:31 |
| * rick_h_ shakes fist at strange test/setup stuff | 12:32 | |
| gary_poster | rick_h_, in devel and prod both? prod that might make sense. Well, no, not really. :-/ | 12:32 |
| frankban | gary_poster: I think you duped it: http://pastebin.ubuntu.com/5695226/ | 12:32 |
| frankban | gary_poster: that problem is not visible in the GUI | 12:33 |
| rick_h_ | gary_poster: yea, both. I only noticed that a later test module loaded new date-format files and that the test suite had some date modules in .use() | 12:33 |
| gary_poster | frankban, ah! not even with a JS exception | 12:33 |
| gary_poster | rick_h_, gotcha. :-( dunno. I don't see how what we are doing is all that weird, except for instantiating a Y rather than running within the Y's function. I wonder if that would make a diff | 12:34 |
| rick_h_ | gary_poster: sorry, just more mean the whole single file stuff I guess. Because when I did my tests with a .only all the modules loaded fine and it worked | 12:35 |
| rick_h_ | but only when I removed the .only did I have Y problems then it seems. and they were pretty invisible because it just wasn't loading the locale stuff it needed to parse the dates, but the date format module did load so it ran but then bloew up mid-date-parse | 12:35 |
| rick_h_ | just feels like it's half magic to get the right incantation to get bits to play together. | 12:36 |
| gary_poster | rick_h_, I agree, and I think we all would. Do you have any suggestions? The only one I have is one that I talked about a long time ago and that is probably too late to be practical: switch to the YUI test runner | 12:36 |
| rick_h_ | all good, tests pass I can -cr and get this landed. Just lost a couple of hours so frustrated atm. | 12:37 |
| gary_poster | rick_h_, it's all good until the next time... :-) | 12:37 |
| rick_h_ | gary_poster: yea, no. The only suggestion is to isolate tests into own files so that the tests stand alone, but that's probably a lot of work | 12:37 |
| rick_h_ | while running tests in LP was a pita it was nice that you had a small known world while you were working. | 12:38 |
| rick_h_ | gary_poster: heh no until next time. Sorry, I'm coming across as whiny to pulling up short. :) | 12:39 |
| gary_poster | rick_h_, not at all, you are trying to be nice about being frutrated :-) | 12:40 |
| gary_poster | frustrated | 12:40 |
| gary_poster | rick_h_, if we divided up all the tests into their own test runner (that's what you mean, right?) then how would you aggregate? | 12:40 |
| rick_h_ | gary_poster: yea, in small projects I've just opened them all. I've always meant to look at yeti since it can do some agg and run in diff browsers I believe | 12:42 |
| gary_poster | rick_h_, ah ok. yeah, that would require us to be using the yui test runner/syntax I think :-/ | 12:43 |
| rick_h_ | https://github.com/yui/yeti just uses globbing to file all the .html files and launch browsers to cycle through them | 12:43 |
| rick_h_ | gary_poster: right, which would be a ton of work from where things are. | 12:43 |
| gary_poster | right :-/ | 12:43 |
| rick_h_ | gary_poster: so yea, I'm complaining without a great solution from where things are atm | 12:43 |
| gary_poster | s'ok, doesn't mean problem isn't real or worth talking about | 12:44 |
| rick_h_ | if I get a sec I might look out for something that works around the current mocha stuff. I'm not as familiar with the other tools around mocha off the top of my head | 12:44 |
| gary_poster | cool, thank you rick_h_ | 12:45 |
| rick_h_ | jcsackett: hatch review call if you guys get a chance after some coffee and the like. https://codereview.appspot.com/8621043/ | 12:58 |
| frankban | rogpeppe: filed bug 1167344 | 13:04 |
| _mup_ | Bug #1167344: Deployment via the juju-core API fails with a "no options to set" error in some cases <juju-core:New> <juju-gui:Triaged> < https://launchpad.net/bugs/1167344 > | 13:04 |
| rogpeppe | frankban: thanks | 13:05 |
| rogpeppe | frankban: and there's another associated bug i've just seen | 13:05 |
| rogpeppe | frankban: the service is still created when that happens | 13:05 |
| rogpeppe | frankban: i'm just doing a substantial cleanup in that area | 13:05 |
| frankban | rogpeppe: the bug includes the output of juju status for that reason | 13:06 |
| frankban | rogpeppe: cool, thanks | 13:06 |
| gary_poster | frankban, if I create a relation on your instance will that cramp your style? I want to try something | 13:08 |
| frankban | gary_poster: go ahead, no problem | 13:08 |
| gary_poster | thank you | 13:09 |
| bac | teknico: swap reviews? | 13:17 |
| bac | https://codereview.appspot.com/8623043 | 13:18 |
| teknico | bac, sure! | 13:18 |
| teknico | "cramp your style", now that's a new one :-) | 13:18 |
| gary_poster | rogpeppe, do you know offhand what the constraints are for valid names of services, interfaces and relations in juju? | 13:23 |
| gary_poster | teknico, want me to review your destroy_service, or wait till you add test Francesco mentioned? | 13:31 |
| gary_poster | either way is fine with me | 13:32 |
| teknico | gary_poster, probably better wait, also I think bac is reviewing it too | 13:32 |
| gary_poster | oh ok teknico | 13:32 |
| gary_poster | thx | 13:32 |
| teknico | gary_poster, I'll put the card back in coding | 13:32 |
| gary_poster | ok | 13:33 |
| bac | teknico: done | 13:35 |
| teknico | bac, me too | 13:36 |
| bac | teknico: hah on the misspelling. i meant to look it up. thanks. | 13:37 |
| teknico | bac, I'm actually kind of appalled by the usage stats :-) | 13:38 |
| rogpeppe | gary_poster: i'll have a look; there are some regexps in state | 13:39 |
| rogpeppe | gary_poster: a service name must match this regexp: [a-z][a-z0-9]*(-[a-z0-9]*[a-z][a-z0-9]*)* | 13:40 |
| teknico | frankban, did you mean that the successful service destroy test should check that the callback is actually invoked? | 13:40 |
| gary_poster | thanks rogpeppe! handling all the various allowable character sets in different parts of the system is...exciting (CSS + HTML + Juju) :-) | 13:41 |
| frankban | teknico: yes, with the arguments we expect | 13:41 |
| gary_poster | if interfaces and relations have similar constraints then juju will be the easiest | 13:41 |
| teknico | frankban, right, will do | 13:41 |
| rogpeppe | gary_poster: i'm not sure that there are any current restrictions around interface and relation names | 13:42 |
| gary_poster | darn | 13:42 |
| gary_poster | ok thanks rogpeppe | 13:42 |
| gary_poster | rogpeppe, +1 on adding some, as restrictive as possible ;-) | 13:42 |
| rogpeppe | gary_poster: i'll moot the possibility in juju-dev | 13:42 |
| gary_poster | service name regex would be great, for instance | 13:42 |
| gary_poster | thanks | 13:42 |
| gary_poster | excluding colons and spaces would nice at a minimum rogpeppe | 13:43 |
| bac | gary_poster: if you're looking to do a review: https://codereview.appspot.com/8623043 | 13:44 |
| gary_poster | mostly looking to solve a bug, but I need room on the boatrd to actually work on it ;-) | 13:44 |
| gary_poster | so happy to review | 13:44 |
| bac | gary_poster: that's the way it works... | 13:45 |
| rogpeppe | gary_poster: +1 | 13:45 |
| gary_poster | cool rogpeppe | 13:45 |
| gary_poster | bac you reviewing frankban's bug 1166727? | 13:46 |
| _mup_ | Bug #1166727: Fix Status/StatusInfo handling in the go delta stream <juju-gui:In Progress by frankban> < https://launchpad.net/bugs/1166727 > | 13:46 |
| bac | gary_poster: no | 13:47 |
| bac | gary_poster: i can | 13:47 |
| gary_poster | bac cool please do, I'm not done with yours yet :-) | 13:47 |
| bac | gary_poster: i will. ok, sure | 13:47 |
| gary_poster | thx | 13:47 |
| frankban | bac: thank you | 13:47 |
| bac | frankban: done | 13:51 |
| frankban | bac: thanks | 13:53 |
| gary_poster | bac done. | 14:02 |
| bac | thx | 14:02 |
| bac | gary_poster: nice: http://jujugui.wordpress.com/2013/04/10/test-tip-whenever-you-instantiate-our-yui-app/ | 14:03 |
| gary_poster | thanks bac | 14:03 |
| hatch | gary_poster: any chance I could get a review on https://codereview.appspot.com/8578043 I'd like to land it as is then spin up another to finish off the final tasks | 14:07 |
| hatch | if possible that is | 14:07 |
| gary_poster | ok hatch, on it | 14:07 |
| hatch | thanks | 14:07 |
| gary_poster | hatch, you prefer typeof over Y.Lang.isString for speed? (fine by me, just learning your rationale) | 14:08 |
| hatch | gary_poster: just a preference really - all Y.Lang.isString does is do a typeof | 14:09 |
| hatch | http://yuilibrary.com/yui/docs/api/files/yui_js_yui-lang.js.html#l175 | 14:09 |
| gary_poster | ok cool hatch | 14:09 |
| jcsackett | rick_h_ and hatch: can i get a look at https://codereview.appspot.com/8600043/ | 14:10 |
| rick_h_ | jcsackett: sure, otp but will look at in a sec | 14:10 |
| jcsackett | rick_h_: no rush. | 14:10 |
| gary_poster | rogpeppe, what's your expected schedule for turning on authentication, btw? | 14:11 |
| rogpeppe | gary_poster: [sorry, was in a call] | 14:30 |
| rogpeppe | gary_poster: i should have a branch proposed by eod | 14:30 |
| gary_poster | rogpeppe, yay :-) | 14:30 |
| rogpeppe | gary_poster: BTW, do you actually need to see a unit's resolved mode in the API? | 14:30 |
| * gary_poster looking | 14:32 | |
| gary_poster | rogpeppe, we allow resolving (though this doesn't appear to be working/showing up on go side) | 14:34 |
| rick_h_ | jcsackett: comments replied to | 14:34 |
| rogpeppe | gary_poster: ok, but do you need to see the resolved mode that's been set? | 14:34 |
| rogpeppe | gary_poster: i suspect not | 14:34 |
| rick_h_ | jcsackett: loading your review now | 14:34 |
| gary_poster | rogpeppe, I don't think so, no | 14:34 |
| rogpeppe | gary_poster: in particular i'm thinking of removing the Resolved field from UnitInfo | 14:34 |
| gary_poster | looking, three deep in stack now ;-) | 14:35 |
| gary_poster | rogpeppe, no we do not need it AFAICT thank you | 14:36 |
| rogpeppe | gary_poster: cool, thanks | 14:37 |
| jcsackett | rick_h_: when you asked me to add "new" to json and sidebar you meant "popular", right? b/c "new" is already there. | 14:49 |
| rick_h_ | jcsackett: right, sorry the request was more "please look at the wireframes and have all three charm-containers supported" | 14:50 |
| rick_h_ | jcsackett: so featured, popular, and new | 14:50 |
| jcsackett | rick_h_: that's what i figured, just double checking. | 14:50 |
| rick_h_ | thanks | 14:50 |
| frankban | gary_poster: do you still need my GUI instance? if so, no problem, if not, I'll destroy it and create another one | 14:53 |
| gary_poster | frankban, destroy away, thank you | 14:53 |
| frankban | gary_poster: ok thanks | 14:53 |
| hatch | anyone else need any more reviews? | 14:56 |
| rick_h_ | hatch: see you in a bit :) | 14:56 |
| hatch | hah | 14:56 |
| hatch | gary_poster: any closer to letting me know if I can merge what I have? :) | 14:56 |
| gary_poster | hatch, I just gave you a ton of comments | 14:56 |
| gary_poster | well, not really a ton | 14:56 |
| hatch | oh heh | 14:56 |
| gary_poster | but some ;-) | 14:57 |
| hatch | slow internets there | 14:57 |
| gary_poster | no it was essentially simultaneous to your comment, hatch ;-) | 14:57 |
| hatch | oh lol gotcha - ok will check again | 14:57 |
| gary_poster | so hatch, your goal is to land and then immediately follow up? I'm ok with that. will mark as such | 14:57 |
| gary_poster | hatch, "LGTM to land with immediate follow-ons for other changes as discussed." | 14:59 |
| hatch | ok thanks - yeah I want to get this landed because it's getting a little long in the tooth :) | 15:02 |
| hatch | I'll make your recommended changes in this branch and then followup with the inferred interface stuff | 15:03 |
| gary_poster | cool sounds great | 15:03 |
| hatch | anyone else available for a review on https://codereview.appspot.com/8578043 ? | 15:04 |
| rick_h_ | hatch: I can peek at it in a few min if that works | 15:05 |
| hatch | yep it'll take me a bit to make the changes from Gary's review | 15:05 |
| rick_h_ | hatch: k, hit me up after that then and I'll peek | 15:05 |
| hatch | yesterday I educated someone on Google hangouts for meetings - they were blown away by how simple it was | 15:07 |
| hatch | they said "with this, why do people need to go into an office?" haha | 15:08 |
| hatch | gary_poster: always keeping me on my toes with acronyms :) | 15:15 |
| hatch | YMMV - your mileage may vary ? | 15:16 |
| gary_poster | hatch, yes, sorry :-) | 15:16 |
| hatch | haha np - eventually we'll get english communications down to one big acronym EWGECDTOBA | 15:17 |
| hatch | :D | 15:17 |
| benji | It appears console.log doesn't work at all in firefox. Is that a known issue? | 15:25 |
| hatch | hmm I haven't seen that before | 15:26 |
| hatch | in the console if you type `console.log` does it give you a fn? | 15:26 |
| rick_h_ | benji: huh? using firebug or in the normal FF debugger? | 15:26 |
| benji | hatch: it does (return a function) | 15:27 |
| hatch | ok so then console.log still exists | 15:27 |
| hatch | question now is why is it not outputting | 15:28 |
| benji | rick_h_: normal debugger, we sometimes replace the console object with our own (to smooth over cross-browser issues and to disable logging in production builds) and that is what I suspect as the issue | 15:28 |
| hatch | ahhh | 15:28 |
| hatch | that could be it | 15:28 |
| benji | hatch: nope, it turns out to be a PEBKAC | 15:30 |
| * benji tries to get hatch to learn all the acronyms in one day. | 15:30 | |
| hatch | oh I know that one | 15:30 |
| hatch | I've used that one | 15:30 |
| hatch | lol | 15:30 |
| benji | :) | 15:30 |
| benji | it turns out that FF behavior has changed in say the last year since I tried to debug anything with it and the place that console.log messages go is defferent than what I remember, so I was looking in the wrong place | 15:31 |
| rick_h_ | yea, they've been working hard on their dev tools stack | 15:32 |
| benji | silly me, looking at the "console" to see the results of "console.log" | 15:32 |
| rick_h_ | saw they finally released a network panel in the nightly today | 15:32 |
| hatch | rick_h_ yeah but I found out the other day I have NO idea how to open the full debugger lol | 15:32 |
| hatch | all I can get is a thick blue bar across the bottom | 15:33 |
| rick_h_ | well the pain point is they're all different tools | 15:33 |
| * rick_h_ hugs chrome dev channel for best dev tools around | 15:33 | |
| hatch | yeah I can't get the Safari one | 15:34 |
| hatch | it makes less sense to use than the IE one | 15:34 |
| hatch | if maybe just barely | 15:34 |
| hatch | lol | 15:34 |
| rick_h_ | what's sad is the safari stuff is based on the same webkit stuff as chrome | 15:34 |
| rick_h_ | they've managed to mess it up | 15:35 |
| hatch | Yeah it may be super powerful but if it's not intuitive it's a horrible tool | 15:36 |
| hatch | we have to learn so much new stuff every day, a new dev console is not high on the list | 15:36 |
| teknico | benji, is your "More picky yuidoc linter." in Slack Review really under review? or does it refer to the already merged lp:~benji/juju-gui/yuidoc-lint-comment-style branch? | 15:38 |
| benji | teknico: it's merged, I'll move it; thanks | 15:38 |
| teknico | gary_poster, the build in jenkins after I landed my branch failed, apparently for unrelated reasons, do you want to restart it? | 15:54 |
| gary_poster | teknico, done (though you can too, you know :-) | 15:55 |
| teknico | gary_poster, yeah, I was reading up the docs and I don't think I yet have the whole VPN setup | 15:56 |
| gary_poster | jujugui: experiment: please try updating kanban board now to be up to date with your status it it is not already. I will review in two minutes. I will try to then only ask about things I don't think I understand or speciifically want to know more about | 15:56 |
| gary_poster | s/it it/if it/ | 15:56 |
| teknico | (done already, also bothered other people already about it :-) ) | 15:58 |
| gary_poster | :-) thx | 15:58 |
| gary_poster | hatch, duh, I meant map, thanks for correcting my thinko | 15:58 |
| hatch | rick_h_ https://codereview.appspot.com/8578043/ so this is an intermediary branch so you should see some TODO | 15:58 |
| gary_poster | jujugui call in 2 | 15:58 |
| hatch | gary_poster: haha np - I had to read it a few times before I assured myself it was a typo :) | 15:58 |
| gary_poster | yeah, sorry | 15:59 |
| rogpeppe | frankban: have you got a gui instance up that i can check something on? | 16:14 |
| frankban | rogpeppe: https://ec2-54-224-91-223.compute-1.amazonaws.com/ | 16:15 |
| rogpeppe | frankban: ta! | 16:15 |
| rogpeppe | frankban: how does the YAML config file upload thing work? | 16:16 |
| rogpeppe | frankban: i'm wondering if it's per-service or for the whole environment | 16:17 |
| rogpeppe | gary_poster, teknico, bac: ^ | 16:17 |
| gary_poster | rogpeppe, on call will be with you soon | 16:28 |
| rogpeppe | gary_poster: np | 16:28 |
| gary_poster | bcsaller, ready for you :-) | 16:28 |
| hatch | rick_h_ able to check out that review now? | 16:28 |
| rick_h_ | jcsackett: can you peek at it again. Had to make some bigger changes but think it's all better now. https://codereview.appspot.com/8621043 | 16:28 |
| rick_h_ | hatch: will do | 16:28 |
| hatch | thanks | 16:29 |
| hatch | rick_h_ oh looks like ben already did | 16:29 |
| gary_poster | rogpeppe, per service AFAIK | 16:30 |
| rick_h_ | hatch: yea, just noticed that | 16:30 |
| gary_poster | I'm not sure what it would be for environment | 16:30 |
| rick_h_ | woot! off the hook lol | 16:30 |
| gary_poster | rogpeppe, I know that we do upload service yaml | 16:30 |
| rogpeppe | gary_poster: apparently the YAML file should contain a map of service names and a set of config attrs for each service | 16:30 |
| hatch | rick_h_ lol | 16:30 |
| hatch | yay landing that branch....finally | 16:31 |
| hatch | lol | 16:31 |
| hatch | even if it is bittersweet | 16:31 |
| gary_poster | rogpeppe, um. that doesn't sound familiar but on call again will return soon | 16:31 |
| hatch | :) | 16:31 |
| rogpeppe | gary_poster: that's what i've just been told. i haven't looked at the python though. | 16:32 |
| hatch | oh darn there is a conflict | 16:32 |
| === matsubara is now known as matsubara-lunch | ||
| === deryck is now known as deryck[lunch] | ||
| hatch | ugh | 16:42 |
| jcsackett | hatch: can you look at https://codereview.appspot.com/8600043/ ? after sorting out your own woes, of course. :-P | 16:46 |
| hatch | certainly | 16:46 |
| hatch | almost done | 16:46 |
| hatch | me hopes | 16:46 |
| hatch | :) | 16:46 |
| hatch | I love it when merges put methods inside of methods | 16:47 |
| hatch | lol | 16:47 |
| hatch | jcsackett: all done, thanks for the cleanup | 16:50 |
| gary_poster | rogpeppe, the service name map does not make a huge amount of sense to me still given the usage--in the command line and the GUI you specify a config file for a service, at least in pyjuju--but yes. This is an example of something we were using last year, for instance: http://bazaar.launchpad.net/~yellow/charms/oneiric/buildbot-master/trunk/view/head:/examples/lpbuildbot.yaml | 16:53 |
| gary_poster | we would use that with | 16:53 |
| gary_poster | juju set --config=[FILE] buildbot-master IIRC | 16:53 |
| gary_poster | juju set --config=[FILE] buildbot-master | 16:54 |
| gary_poster | IIRC | 16:54 |
| gary_poster | which makes little sense | 16:54 |
| rogpeppe | gary_poster: i agree | 16:54 |
| rogpeppe | gary_poster: but there's the compatibility issue | 16:54 |
| gary_poster | certainly that's what the help suggests (juju set [options] <service>) | 16:54 |
| rogpeppe | gary_poster: yeah, the help is no help :-) | 16:55 |
| gary_poster | :-) | 16:55 |
| frankban | rogpeppe: looking at the delta arriving to that ec2 instance, it seems that the status of the machines never do the transition from "pending" to something else | 16:55 |
| rogpeppe | frankban: that's right | 16:55 |
| rogpeppe | frankban: machine agents don't currently set status | 16:55 |
| gary_poster | rogpeppe, ah! we used it in deploy | 16:55 |
| gary_poster | where it made more sense perhaps | 16:56 |
| gary_poster | like this | 16:56 |
| rogpeppe | frankban: just proposed: https://codereview.appspot.com/8630043/ | 16:56 |
| gary_poster | juju deploy --repository=./charms local:buildbot-master --config=./charms/oneiric/buildbot-master/examples/pyflakes.yaml | 16:56 |
| frankban | rogpeppe: great! | 16:56 |
| rogpeppe | gary_poster: just had first successful live test pass with auth enabled, BTW | 16:57 |
| gary_poster | rogpeppe, so the config file format makes more sense there | 16:57 |
| rogpeppe | gary_poster: kinda. i'm still pretty meh. | 16:57 |
| gary_poster | rogpeppe, awesome! no changes needed in GUI? | 16:57 |
| rogpeppe | gary_poster: none | 16:57 |
| gary_poster | whee! | 16:57 |
| rick_h_ | hatch: see 3.9.1 notes? | 16:57 |
| gary_poster | rogpeppe, with you on the meh | 16:57 |
| gary_poster | rogpeppe, but I say compatibility now, discussions later :-) | 16:58 |
| rogpeppe | gary_poster: thanks for your sane suggestion yesterday re: passwords. it pulled the wool from my eyes. | 16:58 |
| rogpeppe | gary_poster: i guess | 16:58 |
| gary_poster | rogpeppe, glad the suggestion made sense :-) | 16:58 |
| gary_poster | for config... | 16:58 |
| gary_poster | I dunno | 16:58 |
| gary_poster | the GUI doesn't have a lot to do with it | 16:59 |
| gary_poster | we just pass it to you | 16:59 |
| gary_poster | for a given deploy | 16:59 |
| gary_poster | I don't *think* we expose doing it for a set yet | 16:59 |
| gary_poster | so do what you think and let us know how it turns out, rogpeppe ;-) | 17:00 |
| rogpeppe | gary_poster: ok | 17:00 |
| hatch | rick_h_: hmm? | 17:04 |
| rick_h_ | hatch: release notes for 3.9.1, big handlebars bug | 17:04 |
| rick_h_ | hatch: I guess we must not hit it much since our tests didn't go boom, but heads up | 17:05 |
| hatch | aren't we using 3.9.1? | 17:05 |
| rick_h_ | hatch: hmm, just saw it on the rss feed so assumed it was 3.9.0 | 17:05 |
| hatch | 3.9.1 was released at the end of Match I think | 17:06 |
| rick_h_ | hatch: heh, google reader fail...nvm :) | 17:06 |
| hatch | :) | 17:08 |
| hatch | who are you going to switch to once Reader is nomore? | 17:08 |
| hatch | I just use it as an API for gReader | 17:08 |
| rick_h_ | hatch: no idea. Everything is fail so hoping they get better in the next couple of months | 17:08 |
| hatch | isn't Feedly the goto one? | 17:08 |
| rick_h_ | :( | 17:08 |
| rick_h_ | needs some major UX help | 17:09 |
| rick_h_ | same with newsblur | 17:09 |
| hatch | ahh, have you seen the gReader app? I really like it (paid $5 for it lol) | 17:09 |
| rick_h_ | hatch: no, web or bust | 17:10 |
| hatch | oh - I only ever read my feeds on my N7 | 17:10 |
| rick_h_ | I do all three, phone, N10, and laptop | 17:11 |
| hatch | and it's UI is very functional | 17:11 |
| hatch | rick_h_ while we are on the topic of YUI updates...did you see the massive performance gains in 3.10? | 17:13 |
| frankban | rogpeppe: EOD, do you want me to leave the ec2 env alive? | 17:13 |
| rogpeppe | frankban: no, you can take it down, thanks | 17:14 |
| frankban | rogpeppe: ok, cool, have a nice evening | 17:14 |
| rogpeppe | frankban: and you | 17:14 |
| rick_h_ | hatch: yea, looking forward to running the test suite with it :) | 17:17 |
| rogpeppe | gary_poster: https://codereview.appspot.com/8626044/ | 17:20 |
| === matsubara-lunch is now known as matsubara | ||
| === deryck[lunch] is now known as deryck | ||
| gary_poster | rogpeppe, great :-) | 17:48 |
| hatch | bcsaller: so what's supposed to happen if the fakebackend has multiple possibilities for the inferred relation? | 17:49 |
| bcsaller | hatch: it has to throw an error | 17:49 |
| bcsaller | in Python we threw an ambigious relation error | 17:50 |
| * bcsaller looks at what python.js does | 17:50 | |
| hatch | ok and that's before adding juju-info to the provides? | 17:50 |
| bcsaller | hatch: no, because I think that the charms won't define a requires juju-info as an example so it won't automatically match (its provided but not required) | 17:52 |
| rogpeppe | i'm off now. g'night all | 17:52 |
| * hatch really wishes these rules were documented somewhere | 17:52 | |
| gary_poster | night rogpeppe | 17:52 |
| gary_poster | hatch, on the bright side, arguably that's what we are doing. kinda. or I tell myself this. :-P | 17:53 |
| hatch | haha - I still think not requring an interface is wako | 17:53 |
| hatch | :P | 17:53 |
| bcsaller | requiring an implicit interface always would be bad | 17:56 |
| hatch | how do you figure? | 17:57 |
| bcsaller | everything would relate to everything | 18:02 |
| hatch | oh I misread - I was saying to always require an explicit interface | 18:03 |
| gary_poster | Makyo, do you have some time to pair with me on some d3 stuff? If you're in the middle of something, np | 18:09 |
| Makyo | gary_poster, Sure. | 18:09 |
| hatch | oh awesome I found a new feature in Sublime.... | 18:09 |
| gary_poster | thanks Makyo guichat | 18:09 |
| hatch | 'New View Into File' | 18:09 |
| hatch | allows you to open up the same file in multiple windows | 18:10 |
| hatch | w00t I always missed that feature from Netbeans | 18:10 |
| rick_h_ | *cough*vim*cough* | 18:10 |
| hatch | rick_h_: if it was in vim I never would have found it because it would have been under some obscure config property that I'd need to enable :P | 18:10 |
| hatch | s/obscure/obsure and undocumented | 18:11 |
| hatch | :P | 18:11 |
| rick_h_ | hatch: not if you watched: http://lococast.net/archives/category/screencast :P | 18:11 |
| hatch | haha oh boy | 18:11 |
| rick_h_ | hatch: see the first one in the list. Guy that does those is pretty good hehe | 18:12 |
| rick_h_ | well, last in the list, first released | 18:12 |
| hatch | bcsaller: is ['wordpress:db', 'mysql'] allowed? or if you specify one you have to specify the other interface? | 18:16 |
| bcsaller | hatch: knowing one side can force a match on the other, its allowed | 18:17 |
| hatch | ok and one more...are subordinate relations always juju-info? | 18:18 |
| BradCrittenden | guihelp: i've deployed the charm against juju-core. can't login. tried the admin-secret password from environments.yaml. what've i missed? | 18:18 |
| === BradCrittenden is now known as bac | ||
| gary_poster | bac, not sure, it has worked for me with passwd and random stuff | 18:19 |
| gary_poster | rogpeppe has branch coming that actually turns on authentication but has not landed I think | 18:19 |
| benji | bac: did you change the user name to "user-admin"? | 18:19 |
| gary_poster | ah right | 18:20 |
| bac | benji: that is what it is fixed to. cannot change | 18:20 |
| gary_poster | that should be automatic | 18:20 |
| rick_h_ | hatch: https://codereview.appspot.com/8618044 is the second branch of the day. Huw's overnight work if you get a chance. | 18:20 |
| benji | oh, for the charm; I always forget to do it when hand-editing the config | 18:20 |
| bac | gary_poster: so you were able to use just any old password? | 18:20 |
| bac | benji: charm from charm store | 18:21 |
| gary_poster | bac you may need to edit charm config? not sure | 18:21 |
| gary_poster | user-admin was in francesco's version today | 18:21 |
| hatch | bcsaller: are subordinate relations always juju-info? | 18:22 |
| hatch | or can they be anything? | 18:22 |
| bcsaller | hatch: they can be any container scoped interface, but often juju-info implying that the principal service wasn't aware of the subordinate in any special way | 18:24 |
| hatch | bcsaller: ok I think I have all of the possibilities as per the documentation and your responses http://pastebin.ubuntu.com/5696204/ | 18:26 |
| hatch | can you check that over and let me know if I missed one | 18:26 |
| hatch | oh darn you can't edit these pastes | 18:30 |
| bcsaller | hatch: that looks good though | 18:32 |
| bcsaller | I believe when you show that fakebackend will cover our usecases | 18:33 |
| hatch | ok great I'll link to that list from somewhere so we have a list of valid inputs documented | 18:33 |
| bcsaller | the tests can self document that pretty easily | 18:33 |
| hatch | rick_h_ done | 18:41 |
| rick_h_ | hatch: ty much kind sir | 18:41 |
| bac | gary_poster: what did you mean by "edit charm config"? you mean 'juju set' one of the variables? | 18:49 |
| gary_poster | bac, no, ssh to the machine and mutate the config | 18:49 |
| gary_poster | bac, I think we probably ought to handle this in the login; not sure why we are not | 18:50 |
| gary_poster | because I think "user-" is supposed to always be prefix | 18:50 |
| gary_poster | and should be hidden from end user | 18:50 |
| bac | gary_poster: any particular part of the config you have in mind? the name is presented as 'user-admin' | 18:50 |
| gary_poster | bac, what is your ec2 instance? | 18:50 |
| bac | https://ec2-184-72-198-219.compute-1.amazonaws.com | 18:50 |
| gary_poster | I like the cows | 18:51 |
| bac | gary_poster: note i already poked at 'login-help' to see if i could mutate things with 'juju set'. the current help is not very helpful now. :) | 18:51 |
| gary_poster | :-) | 18:51 |
| bac | cows are nice, but not helpful | 18:51 |
| gary_poster | bac, I don't know. I saw it working this morning. :-( You could try frankban's script from his blog to make sure that the old recipe for logging in works at all | 18:54 |
| gary_poster | http://pastebin.ubuntu.com/5676102/ | 18:54 |
| gary_poster | bac you could also use debug log to verify that GUI is sending what you expect, again per his blog | 18:56 |
| bac | gary_poster: i used debug log but didn't see anything interesting | 18:57 |
| gary_poster | bac did you see the authentication attempt? | 18:57 |
| bac | gary_poster: i did this time. it returned an "invalid entity tag" | 18:59 |
| gary_poster | bac...uh. oh. | 18:59 |
| gary_poster | bac did you try frankban's script? | 19:00 |
| gary_poster | bac you could also try roger's authentication branch of go | 19:00 |
| gary_poster | I think he got experimental success with that | 19:00 |
| bac | gary_poster: yes, francesco's script seems to login using user-admin/passwd | 19:01 |
| gary_poster | bac, ah! what's the diff in the debug log? | 19:01 |
| bac | gary_poster: http://pastebin.ubuntu.com/5696289/ | 19:03 |
| gary_poster | bac, weird. EntityName -> AuthTag? | 19:04 |
| gary_poster | Try hacking gui bac? | 19:05 |
| bac | gary_poster: login in go.js already sends AuthTag | 19:12 |
| bac | or so it says | 19:13 |
| gary_poster | bac, why does debug log say differently then? yeah | 19:13 |
| bac | gary_poster: charm is different? | 19:13 |
| * bac looks at log | 19:13 | |
| gary_poster | bac did you deploy with juju-gui-source=lp:juju-gui ? | 19:13 |
| bac | gary_poster: no. | 19:15 |
| gary_poster | bac that's it then | 19:15 |
| bac | gah | 19:15 |
| bac | gary_poster: you don't mean "deploy with" you mean deploy and then juju set, correct? | 19:16 |
| gary_poster | bac correct, you can do it now, go out and get some tea, and then come back | 19:17 |
| bac | an iced tea would be pleasant | 19:17 |
| bac | with a nutty chocolate treat | 19:18 |
| bac | gary_poster: ok, now that works. unfortunately i've now forgotten what i was trying to do | 19:27 |
| gary_poster | bac, you were trying to have some tea | 19:27 |
| bac | no, i've done that | 19:27 |
| bac | :) | 19:27 |
| gary_poster | :-) | 19:27 |
| gary_poster | I predict it had something or other to do with bug 1167344 | 19:28 |
| _mup_ | Bug #1167344: Deployment via the juju-core API fails with a "no options to set" error in some cases <juju-core:New> <juju-gui:Triaged> < https://launchpad.net/bugs/1167344 > | 19:28 |
| gary_poster | I'm psychic | 19:28 |
| hatch | just wrote 125lns.....'pass my tests maybe?' | 19:38 |
| hatch | :) | 19:38 |
| hatch | or syntax error, that's cool too lol | 19:38 |
| gary_poster | hatch or anyone, any idea why the heck the yui3-g class has -0.31em letter-spacing? | 19:58 |
| hatch | yup | 19:58 |
| gary_poster | pray tell! | 19:59 |
| hatch | to get rid of inline block spacing | 19:59 |
| hatch | sorry eating an apple...typing with one hand :) | 19:59 |
| gary_poster | lol hatch we have some ugly rendering now in a variety of places. what's the recommended way around that | 20:00 |
| gary_poster | I mean I can think of plenty :-) | 20:00 |
| gary_poster | but I figure there's a suggestion | 20:00 |
| hatch | yeah generally people just reset the letter-spacing back within their elements | 20:00 |
| gary_poster | hatch is there a place to read what the intended use of various classes is (e.g. yui3-u-1) | 20:01 |
| hatch | yep sec | 20:01 |
| hatch | http://yuilibrary.com/yui/docs/cssgrids/ | 20:01 |
| gary_poster | perfect thanks hatch | 20:02 |
| hatch | no problemo | 20:03 |
| hatch | ugh was just totally burnt by if (myvar) instead of if (myvar === ...) | 20:44 |
| hatch | figures i'd get caught breaking my own rule | 20:44 |
| bcsaller | that how we make the rules | 20:44 |
| hatch | haha | 20:45 |
| hatch | I coudlnt find any documentation but does Mocha have a test return status for 'todo' or 'wip' instead of fail? | 20:49 |
| hatch | so in the output they would show as such but not cause a failure? | 20:49 |
| bcsaller | hatch: pending tests? allowing you to TDD | 20:52 |
| hatch | yeah exactly | 20:52 |
| bcsaller | it('should do xxx'); // no function | 20:52 |
| hatch | ahhh nice!! | 20:52 |
| hatch | thanks | 20:53 |
| hatch | brb taking a break, maybe grab some lunch | 20:58 |
| gary_poster | benji, anecdotal and I'm trying to finish something up so not investigating, but it seems doc linter did not complain when I added new function without doc. | 21:02 |
| benji | hmm, that's not good | 21:03 |
| benji | I swore that the next time I touched it I would add tests. I didn't though. | 21:04 |
| gary_poster | bac did you have to manually change user id in config? | 21:19 |
| gary_poster | review request: https://codereview.appspot.com/8640043 | 21:22 |
| gary_poster | back later | 21:23 |
| bac | gary_poster: no, i just 'juju set' to the lp:gui source | 21:49 |
| hatch | ooo soo close to getting this done | 23:20 |
| hatch | annnnnnd done | 23:24 |
| hatch | now to document and clean it up | 23:24 |
| hatch | so....many....tests | 23:44 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!