[01:42] huwshimi: morning [01:42] huwshimi: how goes? [02:02] rick_h: Hey good thanks. Have you travelled already? [02:05] huwshimi: yea, traveled and gone out for the night and back [02:05] huwshimi: I'd like to setup a time to chat. I had to pull the breadcrumb stuff in the process of trying to get the home button stuff landed for friday late. [02:06] huwshimi: I think we can work on getting it back in and supported globally in a bit, but wanted to give you the heads up on why it was pulled out for now [02:06] rick_h: OK [02:06] huwshimi: sorry for the quick change without discussing. Not usually how I'd like to do that kind of thing. [02:07] rick_h: It's all good. [02:08] huwshimi: cool [02:09] rick_h: Well, in that I'm not the stakeholder here :) [02:10] yea, I wanted to give you the heads up though and let you know that I want to help get it back in and such. Felt a bit rude in a friday rush to yank it. :P [02:11] rick_h: No, it's all fine, really. [12:58] alejandraobregon, luca, bcsaller has a demo of bars and icons on deployed services, replacing the current pie chart. We are wondering whether to try and land that as default--without the inspector. The downside is that it really only makes sense with the inspector as is, because unit counts are not included. bcsaller added unit counts but they are teeny teeny tiny in the given (designed) bar size. do you have a moment to look [12:58] at his branch and give some thoughts on whether we should land it for OSCON (pre-inspector)? [13:00] Makyo, https://bugs.launchpad.net/juju-gui/+bug/1201416 [13:07] gary_poster: hi gary... we're at a meeting right now, we'll take a look once we're out (cc.luca) [13:07] cool thanks luca [13:26] jujugui review please https://codereview.appspot.com/11276043 sinzui jcsackett as well [14:17] gary_poster: Hi Gary, the bar is meant to represent an action point, it's meant to get you to the inspector. Unit counts aren't really needed at this view just now, they might come into play for us when we have a progressive zoom but it's meant to just move users on. [14:17] bcsaller: ^ [14:18] luca: its a little funny to have to open the inspector to see how many units there are, but I'm fine with it for now [14:18] turning labels on is easy enough, or including a hover state unit count some other way is fine too [14:21] luca, bcsaller +1 on what bcsaller said, though to clarify, IIUC you are saying that the bar change does not make sense to you without the inspector at least without further discussion? If we had a unit count indicator I would feel it would be fine [14:23] gary_poster: bcsaller the pie chart has always tested really badly, we're interested in moving on from it. It works better than the pie chart irrespective if we have the inspector or not. Unit count isn't really important, it's nice to know your unit count but generally you should know roughly how many units you have on a service in your environment (if its large, medium or small). What we want to do is direct people. [14:23] ahh, I'd also removed the unit scaling, but we could put that back in. I like it w/o but sounds like you still expect the block to scale with unit size? [14:24] gary_poster: bcsaller It also is really hard to design solution which can handle a text input of 10 or 10,000. We need a nice big dedicated space for it, we don't really have that kind of flexibility. Not until we redesign the service block completely and look at power tools. [14:24] * bcsaller is eager for that [14:24] bcsaller: we actually don't want the blocks to scale with unit... [14:24] ok, good, I thought that as well [14:24] I like it better w/o [14:24] :) [14:27] luca, bcsaller, ok, so we want the bar + icon, without unit count and without block scaling, before the inspector? [14:28] those icons are pretty nice ;) [14:28] That's what I understand, in which case, let's get your branch reviewed and landed bcsaller :-) [14:28] agreed [14:28] its proposed [14:28] bcsaller: gary_poster do you have the assets for the bar? etc [14:28] I saw [14:28] luca we didn't think we needed any assets :-) [14:28] luca: no, I render an animated bar with d3 [14:30] bcsaller: gary_poster ah, cool :) [14:30] gary_poster: bcsaller Then I look forward to seeing it! hehe [14:30] cool, thanks luca. :-) I had a few pending questions for you...lemme see if I can find them [14:31] luca, right one easy question, one hard :-) [14:31] We don't have the jujucharms.com switch of the log out [14:31] button to...what was it? Maybe we can squeeze that in, if you can [14:31] remind me of the plan. [14:31] luca ^^^ that was #1 [14:31] the easy one :-) [14:31] hard one: [14:32] can you share plans for any of the following inspector bits yet: [14:32] - making a relation [14:32] - moves of expose and delete per MS [14:33] moves of constraints per MS [14:33] ? [14:33] luca ^^^ that was #2 [14:33] the hard one [14:34] gary_poster: When do you guys need them? I've been watching the user testing today. I know you'll need them asap but whens the uber cut off date? [14:35] luca, for #1, today if at all possible. for #2, tomorrow would be ideal, wed still maybe ok [14:35] gary_poster: #1 the button should say "Get Juju" [14:36] gary_poster: and link to https://juju.ubuntu.com/get-started/ [14:36] awesome thanks luca [14:36] gary_poster: no worries, I'll get to work on #2 :) [14:37] cool thanks again :-) [14:37] bcsaller, lp:~makyo/juju-gui/unit-view [14:59] jujugui could i get a quick review on https://codereview.appspot.com/11283043/ plz [15:50] jujugui call in 10 kanban now :) [16:05] then again, maybe not? :-) [16:45] arosales: Am I correct in thinking that not all charms will get a charm rating? [16:46] jcastro: [16:46] jcastro: ^^ [16:47] luca, correct we are just shooting for all featured, popular, and one charm/category [16:47] arosales: I see [16:47] arosales: I'm requesting the copy writer should write a message for it [16:47] luca: they should all have one [16:47] all new ones will [16:47] luca, I think it currently states [16:47] Quality Rating: [16:47] This charm does not have a Quality Rating [16:47] It will be rated soon. [16:48] but like personal ones I doubt people will care about [16:48] jcastro, are you saying all reviewed charms will have a rating by week end [16:48] arosales: The current message says that it will be rated soon [16:48] arosales: It would be nicer to have it more personable [16:48] arosales: no [16:49] luca, for "reviewed" charms it should have a rating soon [16:49] all charms from now on going in should [16:49] jcastro, ok I was confused by, "they should all have one." [16:49] jcastro: arosales I see, so maybe it's not so important [16:49] luca, so to summerize [16:49] they are supposed to have one [16:49] the thing is when we decided the ratings we already had like 100 some charms in the store [16:49] so they should have one, but the reality backlog is different than what we really want [16:49] 1. all featured, popular, and one charm/category will have a rating by week end [16:50] hey sinzui [16:50] http://uistage.jujucharms.com:8086/fullscreen/precise/ceph-13/ [16:50] 2. All reviewed charm ~should~ have a rating, and that is a work in progress but won't be done before OSCON [16:50] do we really need the revision control in the URL? [16:50] the "ceph-13" [16:50] luca, ^ does that make sense? [16:51] arosales: yes :) [16:51] arosales: thank you [16:53] luca: if you look at logs I also confirmed non-reviewed charms have the correct wording "Only reviewed charms have a quality rating" [17:15] jcastro: yes we do === matsubara is now known as matsubara-lunch [17:21] rick_h: can we make something like /current then or something? [17:21] I can't expect we'd get good google juice on ceph-13 [17:21] and then people pass around ceph-14 [17:21] and so on [17:22] sinzui: https://code.launchpad.net/~abentley/charmworld/store-data-on-enqueue/+merge/174827 [17:23] jcastro: I'd think so myself, but would want to work that through the guys working on getting things to support multiple versions [17:24] who would that be? [17:25] jcastro: sinzui and gary_poster probably need to chat. [17:33] jcastro, I want the default urls to be without versions. However, we will want versions to exist and work. Right now we are in a weird in-between land in which, for instance, juju-gui-GHJGJHG takes you to the same info as juju-gui-66 and the same info as juju-gui-0; but juju-gui takes you to the same info as juju-0. Not ideal. I want specified versions to take you to the right info; unspecified versions to take you to the new [17:33] est info; and internal links and urls to generally use the unspecified versions. [17:37] jujugui jcsackett https://codereview.appspot.com/11271045 for the home button being non-stupid :) [17:38] gary_poster: yeah I am just saying when users want to share the URL etc it shouldn't care about the version [17:38] rick_h: ack. [17:38] jcastro, on the same page [17:38] if we do like a /current or whatever that's fine, as long as the canonical url doesn't change [17:38] I mean canonical as in the URL namespace, not our employer, heh [17:38] :-) [17:43] I have all sorts of dumb URL questions but I'll just bother rick_h later since you guys are sprinting [17:43] unless he's sprinting with you [17:45] * rick_h waves at jcastro from N.C. [17:45] :P [17:47] heh [17:48] jcastro, as a ~charmer I believe you can do help with this. otherwise I can ask webops to helps us: http://pastebin.ubuntu.com/5878176/ [17:49] rick_h: http://cdn.fd.uproxx.com/wp-content/uploads/2010/11/Oprahs-Bees.gif [17:50] sinzui: what would you like me to do? [17:51] I think you are one of the few that have permission to add a series. /me cannot see the URL [17:56] oh [17:56] ok so should I try to make one? [17:56] jcastro, yes please [17:58] https://launchpad.net/charms/bundles [17:58] \o/ [17:59] thank you very much jcastro [18:16] hazmat: can you explain relationship weights in deployer? === matsubara-lunch is now known as matsubara [18:20] bac no :-) that's legacy format imo. it was an odd syntax for ordering. simpler one is just a list [18:21] bac, see configs/*yaml [18:21] for the altenrate forms [18:21] * hazmat watches juju-core test suite bring his laptop to its knees. [18:24] hazmat: sorry, but where are these configs/*yaml? [18:25] bac lp:juju-deployer/darwin [18:25] configs dir in that branch [18:25] bac, what are you trying to do? [18:27] bac forwarded some docs on the different forms .. its also in doc/config.rst in that branch [18:27] hazmat: adding ability to fetch configs by url, e.g. -c http://... [18:27] cool [18:27] hazmat: so, if we're making changes to deployer should we be using darwin? [18:28] bac, yeah [18:28] good to know [18:28] bac, the goal is to have it merged to trunk this week (with py compat) rather than trying to keep a separate series [18:29] hazmat: k [19:01] Hey guys, there's a charm that was recently promulgated, but the icon isn't showing up http://manage.jujucharms.com/charms/precise/openvpn-as [19:01] Is this a known issue or should I open a bug? [19:08] the icon was there this morning btw [19:16] abentley, review? https://code.launchpad.net/~sinzui/charmworld/bundle-api-1/+merge/174837 [19:32] marcoceppi: icon is showing right now? [19:37] hatch: https://www.google.com/search?site=imghp&tbm=isch&source=hp&biw=1364&bih=659&q=w700ds&oq=w700ds&gs_l=img.3..0j0i24l6.1944.3345.0.3489.6.6.0.0.0.0.205.684.1j3j1.5.0...0.0.0..1ac.1.17.img.KQJmcASAcqQ [19:37] bah, "I will shorten urls...I will shorten urls" [20:03] hatch: the wip is here: lp:~bcsaller/juju-gui/header-slot [20:11] Makyo, chrome://settings/languages [20:46] jujugui: can i get two reviews on https://codereview.appspot.com/11301043 [20:47] i have to run; if it's all green i'll land tonight, if it's not all good i'll address concerns before tomorrow morning. [20:54] hazmat: trivial merge to deployer/darwin to get tests to work: https://code.launchpad.net/~bac/juju-deployer/darwin-test-fix/+merge/174862 === BradCrittenden is now known as bac [20:57] Makyo: lp:~rharding/charmworld/icon-api [21:18] thanks hazmat. can you merge it in now? :) [21:29] hazmat: I have a small deployer branch up for review that I think you've heard rumors of ealier today: https://code.launchpad.net/~benji/juju-deployer/add-fetching-configs/+merge/174869 [21:45] benji, thanks [21:45] hazmat: that branch also has bac's test fix in it, so if you want to save a merge you can just do mine ;) [21:46] benji, can you propose it to the lp:juju-deployer/darwin series [21:46] hazmat: hmm, I thought I did, let me check [21:47] darn; resubmitting... [21:50] hazmat: there you go [21:51] benji, cool, i'll pick it up before i eod. [21:52] awesome [22:59] Morning