=== schwuk_away is now known as schwuk
frankbanhi rogpeppe: is statecmd still the preferred package where to put API calls logic? or is it preferred to put everything in apiserver/client/client.go?10:39
rogpeppefrankban: statecmd is the place to put logic which is called by both cmd/juju and the API server10:39
rogpeppefrankban: the aim is to eliminate it eventually, when we make the command line tools use the API10:40
frankbanrogpeppe: ack, I guessed so. So, if I need to implement several helper functions called by an API call, is it ok to put them in client.go?10:41
rogpeppefrankban: yes10:43
frankbanthank you10:43
rogpeppefrankban: which API call are you implementing, BTW?10:43
frankbanThis week (maybe to day) I'd like to start working on ServiceUpdate. It will include the MinUnits parameters.10:44
frankbanrogpeppe: ^^^10:44
frankbans/to day/today/10:44
rogpeppefrankban: what else will it include?10:45
rogpeppefrankban: i guess i'd naively expect a ServiceSetMinUnits call10:45
frankbanrogpeppe: as discussed with William, some weeks ago, everything: charmulr, settings(yaml), constraints10:45
rogpeppefrankban: ah, seems reasonable10:46
frankbanrogpeppe: he mentioned that an API like this could help in the process of making this kind of calls transactional, IIRC10:46
frankbanrogpeppe: so, maybe a good strategy could be implementing some helper functions that will be call by the new ServiceUpdate and by the other API call that share the same functionality (e.g.ServiceSetCharm). does it make sense?10:48
rogpeppefrankban: that seems reasonable. presumably this new call is going to overlap in functionality with some existing calls.10:49
frankbans/will be call/will be called/: writing today is particularly difficult...10:49
frankbanrogpeppe: yes, as I said, ServiceSetCharm is one example, and others are ServiceSet/ServiceSetYAML10:50
rogpeppefrankban: is the aim that the GUI actually changes several of these things in the same call?10:51
frankbanrogpeppe: the only difference is that in ServiceUpdate every parameter is optional. This seems easily achievable for strings and structs. For ints, perhaps pointers can help.10:51
frankbanrogpeppe: I am not aware the GUI needs this (it only needs a way for setting MinUnits atm), but in the future hopefully can take advantage of this new unified call. 10:52
rogpeppefrankban: yeah, i'm just wondering what thing we're aiming to making transactional. if we never want to change a service's charm URL in the same transaction as setting its constraints, i'm not really sure i see the point of conflating the two calls10:52
frankbanrogpeppe: maybe I am wrong, and I misunderstood what William told me some weeks ago: maybe the goal was to prevent partial successes?. anyway, I am pretty sure that he requested the new ServiceUpdate API to be implemented, and this will allow the GUI to set the MinUnits for a service. I believe I will also need to take care of including service.MinUnits to the megawatcher for the service.10:57
rogpeppefrankban: ok. i'm concerned about the fact that this means we'll want to deprecate the old calls, and i'm really wanting to keep the client api as stable as possible, but i guess that's just a hit we'll have to take11:00
frankbanrogpeppe: yeah. I guess ServiceSetMinUnits would work for us as well, but I was clearly pointed to this other direction11:03
rogpeppefrankban: yeah, i'd like to bring this up with william but he's incommunicado. you'd better go with what he said.11:04
frankbanrogpeppe: cool, thank you11:04
gary_posterantdillon, hi12:52
antdillonHi gary_poster 12:52
gary_posterantdillon, I have a few options for you12:53
gary_poster1) style the zoom widget12:53
gary_poster2) fix the export widget in the inspector (it doesn't upgrade; haven't checked out why)12:54
antdillongary_poster, Sure I'll grab the zoom widget. That shouldnt take me long12:55
gary_poster3) put the purple circle in for landscape issues in the inspector12:56
* frankban biab12:56
rick_hgary_poster: do you know who can answer gnuoy's question on the deployer in #juju? 12:56
rick_hgary_poster: http://paste.mitechie.com/raw/993/12:56
gary_poster(luca can tell you where that is :-) )12:57
gary_poster4) thanks rick_h 12:57
gary_posterheh that was not 412:57
gary_posterrick_h, on freenode #juju?12:57
rick_hgary_poster: yes, that's where he asked it. I figure it's an IS thing? 12:57
gary_posterthanks rick_h replied there12:58
rick_hgary_poster: yep, thanks. 12:59
gary_posterantdillon, #4 was "see if something else is broken and tell me about it and fix it if no one else is working on it" ;-)13:02
antdillongary_poster, Awesome, I'll work through them in that order I think, seems to make sense13:03
gary_postercool antdillon thanks13:03
=== teknico1 is now known as teknico
sinzuijujugui webops are urging us to provide nagios integration to the juju-gui charm. Does someone have time to review my branch? https://code.launchpad.net/~sinzui/charms/precise/juju-gui/nagios/+merge/17758813:39
benjisinzui: I'll be glad to take a look.13:39
benjisinzui: in way of quid pro quo I would like to know who the best person to talk about the jujucharms.com charm would be13:40
sinzuibenji, I and abentley are the most familiar in both authorship and deployments13:41
benjisinzui: I was thinking more on the ops side.  I need to verify (through some means that I have yet to devise) that the charm ops used for the GUI includes our latest changes (cache headers)13:42
teknicobenji: fyi, when I run "apache2ctl configtest" on a charm deployment, I get this output: http://pastebin.ubuntu.com/5950894/13:45
benjiteknico: that's something I should look into; thanks!13:46
benjiteknico: hmm, when I run it on a box with a deployed GUI charm I get "Syntax OK"13:47
teknicobenji: also, the config/apache-site.template file uses both tabs and spaces for indentation :-)13:47
benjiok, now THAT is a problem 13:47
sinzuibenji, gunoy and thedac are best. gnunoy is online now in fact.13:47
benjisinzui: cool, thanks13:47
teknicobenji: there might be some problem with my deployment then13:47
benjiteknico: I added a "a2emod" call to the dependencies setup step that should have added mod_headers13:48
teknicobenji: I have not seen that, are your cache changes in trunk already?13:49
benjiteknico: yep: http://bazaar.launchpad.net/~juju-gui-charmers/charms/precise/juju-gui/trunk/revision/7113:50
frankbanit's interesting that in Italy we use "quid pro quo" in a different way, meaning "misunderstanding". and we express the "exchange of services" meaning using another latin sentence: "do ut des"13:50
benjifrankban: interesting; it seems that the "substituting one thing for the other" (i.e., misunderstanding) meaning is closer to the original usage13:52
abentleybenji: I am thinking of going back to versioning, but before I do, is there anything left from Friday that we still need to put to bed?13:58
benjiabentley: not that I am aware of.  You might ask Gary if he needs anything for the demo [it may be done now, for all I know]13:59
abentleygary_poster: Is there anything you need for the demo (or is it over)?14:00
gary_posterabentley, it is over.  server was down again but it was ok.  people still very happy with progress.14:01
abentleygary_poster: That's a shame.  It's surprising it's so tempermental-- it's deployed exactly the same way as staging.14:02
abentleygary_poster: I'll rip it down now unless you want to keep it around for some reason.  I expect we'll have staging updated to work similarly soon.14:03
gary_posterjujugui, orangesquad, busy now but wanted to say generally that people (including MS ;-) were very pleased with the demo.  Juju group has a meeting this evening so may get more details this evening, but so far so very good.  Thank you!14:04
sinzuifrankban, :) In British English we use "to table" to mean lets talk now, but in America English the phrase means talk later.14:04
abentleygary_poster: Great to hear.14:04
rick_hgary_poster: woot!14:04
gary_posterabentley, yeah, rip it down.  sorry it didn't work out but glad for progress nonetheless14:04
benjisinzui: where is the best place to contact gunoy, I don't see him anywhere I normally /join14:20
rick_hbenji: he was in #juju eariler today14:21
rick_hearlier ugh14:21
sinzuibenji, #webops14:25
hatchmorning all14:52
antdillonHi, is there a way to clean the css its appending the new styles?14:53
hatchsay what?14:53
antdillonWhen I refresh a local juju gui it add's the combined css from the less to the end of the existing css14:54
antdillonDoes that not happen for everybody else?14:55
hatchso you're getting *.less content at the end of the css file?14:55
rick_hantdillon: I think you need to go a little deeper. 14:55
rick_hantdillon: yes, normal build is to process the less styles and build a combined stylesheet out of them14:56
rick_hantdillon: I don't think we have an 'unprocessed' version of it in single files available right now14:56
antdillonhatch, rick_h Not less content, this combining the css and adding it to the end of the css file14:57
antdillonrick_h, My juju-gui.css file is 27000+ line long at the moment14:58
rick_hantdillon: hmm, 16,221 here14:58
antdillonI guess just deleting the contents of juju-gui.css and re-running would do the trick14:58
rick_hantdillon: right, I understand what you're saying, but we're not following what you're asking. You want to rebuild the file? You want to not combine the file?14:59
hatchantdillon: maybe run `make clean`14:59
hatchrick_h: I 'think' that it's concatenating the contents to the file instead of remaking it14:59
hatchis what he is saying14:59
antdillonrick_h, Yes sorry I want to combine but it retaining the css from before the change15:00
rick_hhatch: oh hmmm, antdillon what commands are you running to 'rebuild' the css?15:00
antdillonhatch, Yes appending to new combined styles to the end15:00
antdillonrick_h, Just refreshing the localhost15:00
rick_hantdillon: if yuo wanted to try to keep your new stuff isolated I'd add a css file to index.html and hack on it. Then combine it into the right places in the less world. Any less file changes should auto-rebuild the css file for you. 15:01
antdillonhatch, So every refresh is adding over a 1k of lines to the css15:01
rick_hantdillon: want to chat on a hangout? maybe easier to follow/suggest something?15:01
hatchwell that's not possible hehe15:01
hatchso something is broken :)15:01
antdillonrick_h, Oh ok, I can split it off to its own. Just wondered if instead of concatenating the new combined styles it could replace15:02
rick_hantdillon: well that's why I ask what commands you're running? It should auto rebuild on its own as you edit the existing files.15:03
hatchi'm still curious as to how refreshing the browser is adding to the css file15:03
rick_hantdillon: if you manually run some less command, it might act strange15:03
rick_hhatch: yea, it can't be. So there's some step we're missing. Why I suggest a hangout at this point15:03
rick_hto see what's up15:03
antdillonrick_h, hatch No commands at all, I had a few changes to stylesheet.less and refreshed a few times and its combining the styles on each refresh but not replacing the content in juju-gui.css but appending it to the end15:04
antdillonrick_h, So If I give something some styles and then remove them it keeps that on earlier lines.15:05
antdillonrick_h, I have to set them back to defaults15:05
hatchantdillon: and if you run `make clean` and then `make devel` again?15:06
rick_hantdillon: ok, I see it. It's broken. 15:06
rick_hantdillon: it should not be doing that. 15:06
rick_hantdillon: so you found a bug :)15:06
antdillonrick_h, Woop lol15:07
antdillonhatch, Running them now15:07
rick_hfiling and will see what we can do about it. 15:07
rick_hantdillon: we'll get you a short command to use for the moment. Sec15:07
antdillonrick_h, Cool, are you not moving over to SASS?15:07
rick_hantdillon: trying to, but not today15:08
antdillonrick_h, COol15:08
antdillonrick_h, Oh by the way not sure if you've seen emails yet but I set up a frontend mailing list for canonical if your interested15:09
antdillonhatch, Worked a treat, thanks15:09
hatchantdillon: np - that's the goto 'fixall' :D there is also `make clean-all` which will also remove all of the node modules15:12
hatchantdillon: I joined your mailing list...do I get a grab bag or something now? :)15:12
rick_hantdillon: this will 'reset' faster than a make clean will. 15:15
rick_hrm build-shared/juju-ui/assets/juju-gui.css build-shared/juju-ui/templates.js  && make build-shared/juju-ui/templates.js15:15
rick_hantdillon: just keep that in your command history for doing a 'reset' on the css and should help. Filing a bug now15:15
hatchrick_h: do we know how to repro the bug?15:16
rick_hhatch: yea15:16
hatchoh ok cool15:16
rick_hhatch: #120850315:16
_mup_Bug #1208503: juju-gui.css rebuilds over itself improperly <juju-gui:Triaged> <https://launchpad.net/bugs/1208503>15:16
hatchrick_h: so you're telling me this has been happening for almost a year and noone has noticed?15:17
rick_hhatch: looks like the less build command in templates.js is doing a "appendFileSync" which I assume means it's appending vs rebuilding the file15:17
rick_hhatch: well think about it. cascading CSS15:17
rick_hit appends the new rules, which cascade over the old ones15:18
rick_hso it didn't 'effect' the layout/etc15:18
hatchthat would be 'cascading cascading style sheet'15:18
rick_hn + 115:18
rick_hhatch: so since it's in the node stuff I'll assign it to you :P15:18
hatchsure thing :)15:18
antdillonhatch, Thanks, yes its in the post!15:19
antdillonrick_h, Thanks15:19
hatchwell all of the stump grinders in the city are rented so I gota get back to digging this damn thing out :/15:21
Makyojujugui meeting in 10 according to calendar, kanban now15:21
rick_h<3 those things. 15:21
hatchoh standing up is in a couple minutes15:21
Makyo9 now.15:21
hatchmaybe I'll hang around then15:21
Makyojujugui call in 115:29
bcsallerhow did I miss this change? :-(15:29
rick_htrying to get in but no load on the page :(15:30
teknicodoes anyone fancy reviewing some more charm refactoring? :-) https://codereview.appspot.com/1246704415:55
gary_posterjujugui, orangesquad, hi.  I'm afraid I have a meeting with MS in conflict.  However...15:58
gary_posterI just forwarded an email to peeps15:59
gary_posterCould you start the next hour by reading its content? :-)16:00
rick_hgary_poster: so hangout is not taking place now?16:00
gary_posterrick_h, not with me, no16:00
rick_hhah "It's better to have something like 'the number of commits in the last month'"16:23
rick_hso what were they acutally using in this? Was it some sort of mockup?16:25
gary_posterrick_h, in part16:28
rick_hgary_poster: ok, I guess it says it used the 'prototype' in two states but assumed it was a juju-gui install16:29
gary_posterused juju-gui for part16:30
MakyoSome of these run counter to goals expressed to us as a team in Oakland.  Would be interested to discuss after IoM. 16:30
* gary_poster has not read yet; will do so tonight. We can hopefully talk about it this week, even.16:31
rick_hgary_poster: gotcha. Yea some of these confuse me and I assume it's how the mockup version worked. Not sure how they 'changed the tab' 2/3 way down. The back button works for going back to the charm after clicking a related one, hooks is gone, ti's source. 16:31
rick_hvery cool to read though. Some good info in here16:31
rick_halways fun to have testing back up things that have been said/brought up. 16:32
gary_posterjujugui, orangesquad, please be welcome to send comments to the mail16:34
abentleygary_poster: I don't know which mail you mean.  After you mentioned it, I looked at the juju-gui ML and didn't see anything.16:35
rick_habentley: PM16:38
gary_posterabentley, juju-gui-peeps16:40
gary_posterabentley, private16:40
=== schwuk is now known as schwuk_away
* benji reboots to do a little post-lunch RAM upgrade.17:28
benjiabentley: I'll be ready to join up in a couple of minutes if I'm not too late to the versioning party18:36
abentleybenji: Okay, let's hang out whenever you're ready.18:53
benjiabentley: guichat is open18:59
hatchhey all21:16
hatchmorning huwshimi23:53
hatchthanks for the update on the build script23:53
huwshimihatch: Hey23:54
huwshimihatch: I was going to push a branch up for that today23:55
huwshimihatch: It's a tiny fix23:55

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