/srv/irclogs.ubuntu.com/2013/09/30/#juju-gui.txt

rick_h_the email it burns!00:35
gary_posterlol01:14
gary_posterwelcome back rick_h_ 12:17
gary_postergood to have you back. hope you had a good time12:17
bacgary_poster: on stable and dev ~juju-gui-charmers PPA doesn't have saucy builds.  want a card for that?12:19
gary_posterbac, sure thank you. high priority has room, and that seems about right.12:20
gary_posterrick_h_, when you get a chance, please add this to your list of emails to slog through, or similar :-) : http://jujugui.wordpress.com/2013/09/27/bugs-vs-kanban/12:27
gary_posterrick_h_, that is what we agreed, but we are happy to revisit if you disagree and want to be convincing. :-)12:28
=== TheRealMue is now known as TheMue
rick_h_gary_poster: sure thing, I read through that and +1. I think it reads kind of what we had talked about where kanban is committed work and the only 'change' is let's do better to syncing the cards with bugs we've gotten done12:31
gary_posterok cool rick_h_ thanks12:31
rick_h_and yea, good time by all, DC Zoo was awesome. VA mountains are awesome. My own bed is awesome lol12:32
gary_posterlol great :-)12:32
rick_h_and the touareg has a 'spirited' side taking us through back mountain passes with > 8 deg grades and emergency runaway truck lanes and fog so thick you can't see the brake lights of a guy 2 car lengths in front of you. 12:33
rick_h_but she loved powering through those 12:33
gary_posterlol, sounds fun in a "I hope I don't drive us all off the side of a mountain" kind of way :-)12:34
rick_h_yea, let's just say my wife wants me to start double checking the GPS the car produces more often12:35
gary_posterheh, yeah, DC has brought that conversation up for us more than once actually.  DC appears to be an anti-GPS zone12:41
rick_h_well that part worked well. We got rerouted twice due to traffic conditions. It was crazy, towing a 5,000lb trailer to get the notice "In 1 mile, take the exit on the left" in a 6 lane wide highway12:42
bactoo much vietnamese food?  i type 'make tet' way too often.13:35
gary_posterhey frankban, running make in the charm on saucy is downloading and trying to build bzrlib, even though I have bzr 2.6.0 installed.  This then fails because I do not have libpython-dev installed, afaict.  Shoudl I install libpython-dev and add that to the charm dependencies, or should the charm not being trying to build bzrlib?13:39
frankbangary_poster: hi! bzrlib is required by juju-deployer. so, if libpython-dev is not listed in make sysdeps it should be added13:41
gary_posterfrankban, cool, on it, thanks13:41
rick_h_yay, readthedocs fixed their bzr stuff http://charmworld.readthedocs.org/en/latest/api.html13:53
frankbanguihelp: could anyone please review https://codereview.appspot.com/14125043 ? 1 review + 1 QA. thanks!13:58
gary_posterjujugui if you are coming to the resolve/retry mtg with luca it is in less than one minute in https://plus.google.com/hangouts/_/eeadd54f49162797f746e11bc0adc95c87a0e96313:59
gary_poster(see your calendar for detaisl)14:00
gary_posterluca__, you coming?14:05
luca__just grabbing my charger14:05
luca__gary_poster: ^14:05
gary_postercool14:05
benjifrankban: which file in the GUI has the tests for the deployer integration?14:17
gary_posterbenji, the only gui deployer integration we have so far is from bcsaller14:18
frankbanbenji: did not work on the client side deployer support, you might ask ben14:18
gary_posterand it is on the env14:18
benjiok; I think I see it now14:18
bachey benji can you review https://code.launchpad.net/~bac/charmworld/fix-icons/+merge/188341 ?  it is a critical charmworld fix for broken icons.14:24
benjibac: I'll look right now.14:24
bacty14:24
rick_h_gary_poster: so I'll update #1229215 with the first step we talked about14:33
_mup_Bug #1229215: Confusing unit retry/resolve UX <juju-gui:Triaged> <https://launchpad.net/bugs/1229215>14:33
gary_posterawesome thanks rick_h_ 14:38
benjibac: apparently my "right now" is a little off today; the branch looks good.14:50
bacthanks benji.  will land and release soon.14:50
gary_posterfrankban, hey.  I'm still having trouble making the charm.  I'm on saucy fwiw, but I think this issues is a chicken and egg problem.  http://pastebin.ubuntu.com/6175862/  It looks like bzr is trying to build itself, and found a plugin in my .bazaar directory and so tried to include that in docs, but the rvsubmit plugin expected launchpadlib, which exists in my system just fine but has not yet been built in the virtu14:58
gary_posteralenv (because it needs bzr to be built).  So, it's a loop.  The solution would seem to be to exclude my plugins from the bzr build step.  I don't see an automatic way to do this, so I just plan to move my ~/.bazaar directory aside manually, temporarily.  Agree, or do you have another idea?14:58
bacabentley: i just had a charmworld MP approved.  lander being tested.14:58
bacabentley: does it run at :00 ?14:58
abentleybac: I don't understand.  I'm trying to restore the lander.  AFAIK, it's not running successfully.14:59
bacabentley: last week you asked to be alerted when we had branches to be landed.  i wasn't aware that the new lander was not working.15:00
abentleybac: The trigger runs every minute, does that answer your question?15:00
bacabentley: that does15:00
frankbangary_poster: looking15:00
gary_poster :-P biabfrankban, going to another call15:01
gary_poster supposed to be this: frankban, going to another call :-P biab15:01
abentleybac: Okay, so jcsackett reported problems Friday afternoon.  I tried to fix them, but ended up locking myself out of the instance.15:01
bacabentley: ok.  please let me know what i should do, if anything, to get my branch merged.  i'm happy to wait a little while but i'd like to do a charmworld release today.15:02
abentleybac: Run the tests locally, merge into trunk locally, commit and push trunk.15:03
bacabentley: ok, i understand the manual procedure.  i was asking more if you wanted me to wait to see if the lander would work on my branch.  as i said, i'm happy to do that for a limited amount of time.15:05
abentleybac: I can't give you an estimate for how long this will take.  The floor keeps collapsing underneath me with every step I take.15:05
bacok.  i'll land manually.  good luck abentley.15:06
rick_h_lol benji you don't like my python docstrings with params? :)15:19
rick_h_gary_poster: so I think I'm close enough on email atm, if I was sick of email and wanted to code something did you have a place you wanted me to jump into or just grab a bug and go forth?15:19
benjinope :)15:19
rick_h_benji: do you displlike it on the js @param business as well or python in particular is buggy?15:20
hazmatgary_poster, incidentally there are a few bugs around the notifications in the last release.. once you enter the full screen errors you cant go back15:20
gary_posterrick_h_, "Make bundle token realer": make the bundle token thing actually do the right thing15:20
rick_h_gary_poster: rgr15:20
gary_posterhazmat, yeah, thank you, known bug.  for next release (this week) we are simply removing the "all notifications"15:20
benjirick_h_: I dislike it in JS too, but at least we have tools that actually parse it there, we're not doing anything like that on the Python side.15:21
hazmatgary_poster, cool15:21
rick_h_benji: ah, ok.15:21
frankbangary_poster: make BZR_PLUGIN_PATH='-user' should prevent bzr from importing plugins from HOME.15:25
gary_posterfrankban, ah, nice!  I'll add that to Makefile too15:25
gary_posterthank you15:25
frankbangary_poster: yeah, if it works, adding that to the Makefile sounds great, thank you15:26
gary_postercool15:26
bachey benji i had to fix a ton of pre-existing lint problems in my branch.  could you re-approve https://code.launchpad.net/~bac/charmworld/fix-icons/+merge/188341 ?15:28
benjibcsaller: hi, are you in?15:29
benjibac: sure15:29
bacactually since i'm landing it by hand it doesn't matter15:29
bcsallerbenji: yes15:29
benjibcsaller: cool, a question about the deployer bits: is it (more or less) finished?  I.e., given a deployer file will it deploy the described services, relations, config, etc.?15:30
bcsallerbenji: It should fire off the env call to an actual backend with the proper data (though that could use more testing) but that call is also implemented in the fakebackend and that will simulate a deploy now15:31
benjibac: looks good [but man I dislike a couple of the whitespace rules that linter insists upon, especially the two spaces before inline indents; who does that?)15:32
benjibcsaller: ok.  I'm asking because (in my in-development branch that may well be doing something wrong) I don't get correct results when deploying a bundle using its deployer-formatted description.  I'll look deeper.15:33
hatchmorning15:33
benjibcsaller: are there any demo deployer files that are known to work that I can reference?15:33
benjihatch: it seems your jetlag-induced early-bird desires have been defeated15:34
bcsallerbenji: you can export something on the canvas today and re-import that but there is test/data/wp-deployer.yaml as well15:34
benjibcsaller: cool, thanks15:34
hatchbenji: haha, nah had eye doc appt15:34
benjiah15:34
hatch20/20....like a boss15:35
benjiI was 20/13 the last time I had mine checked, but I'm probably down to 20/20 by now.15:36
abentleybac: I am ready to try an auto landing.  Will that interfere with you?15:36
bacabentley: i just pushed mine15:36
bacabentley: so, no.15:37
abentleybac: Great.15:37
gary_posterI'm, like, 20/500000015:37
hatchlol15:37
hatchblind like bat?15:37
gary_postercontacts work for me because glasses would tilt the earth off its orbit15:37
hatchrofl15:38
* hatch is gona review/qa frankban's branch15:39
benjiheh15:39
frankbanhatch: thanks15:39
benjigary_poster: except that your contacts are so thick that you have to take them out to blink15:39
gary_posterbenji, lol, no, I just have callouses under my eyelids... (might not entirely be a joke, not sure... :-P)15:40
hatchhaha15:41
benjiyow15:41
gary_posterand, yes, yuck :-)15:41
benjiI'll make my millions by inventing eye lube for the overly-thick contact market.15:42
bcsallerbenji: did that file import for you?15:42
benjibcsaller: I'm just visually comparing it with my data right now, I haven't tried to import it... one sec an I'll try.15:43
benjibcsaller: "Import Environment Failed"15:46
benjibcsaller: let me try again on trunk, I may have broken my branch15:47
bcsallertesting locally, I didn't expect that to fail15:47
abentleyjcsackett: I'm afraid bac landed changes that conflict with yours.  However, once you've fixed the conflicts, I believe jenkins will land your changes.15:49
bcsallerbenji: that did work for me here15:49
benjibcsaller: it failed on trunk for me too; this is by dragging and dropping the file; are you using some other mechanism to load the file?15:50
Makyojujugui call in 1015:50
bcsallerDnD15:50
benjihmm15:50
benjibcsaller: oh!  I figured it out: I had the Charmworld API v3 flag enabled, which apparently causes the failure when importing.  I'll look at figuring out why that breaks import.15:52
bcsallerbenji: ahh, I bet its the change to charm revision searching15:52
benjiI'll keep that in mind.15:52
bacabentley: does your lander run 'make lint' on charmworld?15:53
abentleybac: No.15:54
jcsackettabentley: thanks!15:57
gary_posterjujugui call in 215:58
gary_posterbug 16:01
gary_posterbug 122090916:01
_mup_Bug #1220909: Searching for "rabbit" doesn't return the rabbitmq charm <charmworld:Triaged> <https://launchpad.net/bugs/1220909>16:01
rick_h_luca__: still around?16:13
luca__rick_h_: for another 15 mins or so :)16:13
rick_h_luca__: so I'm working on updating the bundle token to be 'correctish' and I'm looking over assets/etc16:14
luca__rick_h_: ok16:14
rick_h_luca__: the one thing I notice is that we have several sizes of the old charm token and curious if there's thought on that on the bundle side?16:14
rick_h_in particular for the autocomplete/quicksearch results display?16:14
rick_h_right now I'd imagine they all get the same generic icon + the name of the bundle?16:14
luca__rick_h_: thats right16:14
rick_h_luca__: the other thing I've not verified yet is 'related charms' ?16:15
luca__rick_h_: they only show in the auto-complete if its recommended16:15
rick_h_there's related bundles I'd assume, but not verified. They also just get the same generic image 16:15
luca__rick_h_: related charms is "charms within this bundle"16:15
rick_h_luca__: is that how we treat autcomplete currently? I thought we ditched the idea of search with recommended? /me dbl checks16:15
luca__rick_h_: it's a list of the charms within the bundle, it includes summary of each service16:16
rick_h_luca__: it seems odd that I can search for rack and get results but a rack bundle would not because it's not recommended?16:16
luca__rick_h_: afaik the auto-complete only shows recommended charms16:16
rick_h_luca__: ok, that's not true atm16:16
luca__rick_h_: wait wait wait :)16:16
rick_h_luca__: and since we've been heading down the path of autcomplete being more of a 'quick search' I don't think it should be true16:16
luca__rick_h_: the auto-complete is the drop down part which shows results as you type16:16
rick_h_luca__: right16:17
Makyohatch, https://codereview.appspot.com/1398604316:17
rick_h_http://comingsoon.jujucharms.com/16:17
rick_h_enter 'rack'16:17
hatchMakyo: thanks will take a look shortly16:17
rick_h_and see three options, none of which are reviewed charms16:17
Makyohatch, cool, thanks.16:17
rick_h_luca__: ^^16:17
luca__rick_h_: it should only recommended stuff…I thought it was only showing recommended stuff hehe16:17
rick_h_luca__: now recommended charms are given a higher 'score' in autocomplete and get showed at the top of the list16:17
luca__rick_h_: because the add to my canvas in auto-complete is missing16:18
rick_h_luca__: but with the idea of it being 'quicksearch' vs 'autocomplete' does that idea still hold true?16:18
rick_h_luca__: true, I need to get that in. I'll try to squeeze it in soon. I might be able to drive-by it. It should be a quick/easy update16:18
luca__rick_h_: mostly, I can't remember the decision to why it's not just recommended, I think it's like that because we couldn't implement the rest of the UX16:19
* bac lunches16:20
rick_h_luca__: ok, so bundle tokens in the sidebar will be larger than charm tokens and show the icons below. They'll be a simple icon/name in autocomplete and be the generic icon for now.16:20
luca__rick_h_: thats fine16:20
rick_h_so really just the three sizes, sidebar, fullscreen (for now), and autocomplete16:20
luca__rick_h_: ok16:20
luca__rick_h_: do you need assets?16:21
rick_h_luca__: no, I can use the bundle token svg. I think that's the only *asset* needed right now. 16:21
rick_h_luca__: I'm also noticing the orange 'selected' is moved to the left?16:21
luca__rick_h_:  ok16:21
luca__rick_h_:  that is correct16:21
luca__rick_h_: due to the orange triangle16:22
rick_h_luca__: ok cool. I think I'm good for now. I'm sure something else will come up. 16:22
rick_h_luca__: ah, that makes sense16:22
luca__rick_h_: which you should be implementing soon :D16:22
rick_h_luca__: glad to be back :P16:22
luca__rick_h_: /nudge /nudge /wink /wink16:22
luca__rick_h_: rofl welcome back, you was missed :D16:22
Makyobac, I...uh...had other problems with lightweight checkouts.  I may have been lying about what I found.16:23
* Makyo quietly rebuilds dev evn <.< >.>16:23
bacMakyo: ok.16:23
gary_posterfrankban, hey https://plus.google.com/hangouts/_/fcf9a8c4670d9c371763980ac16e8f3ba7de77b216:24
gary_posteroh frankban actually lets go to the meeting one16:24
gary_posterthe next one16:24
frankbangary_poster: ok joining16:24
gary_posterfrankban, if not clear, https://plus.google.com/hangouts/_/4dc220a25e4e00a204f48f8e64644c392635a78d16:24
hatchMakyo: much better :) just qa'ing16:28
hatchbcsaller: so are you going to tackle the bundle details page or did you want me to take that branch back?16:35
bcsallerhatch: Oh, feel free, I still have a little time on this one16:36
hatchok I'm going to move the visualization integration into a separate card16:37
bcsallerhatch: there is a different card for that already16:38
hatchoh ok16:38
hatchrefreshing16:38
Makyohatch, thanks! Big diff against trunk, re-qaing before I land.16:47
hatchMakyo: oh ok yes plz do :)16:47
hatchrick_h_: do you have a second to explain to me what's going on with this token? I'm trying to find what handles the click event? It doesn't appear to be the container or the token?16:51
rick_h_hatch: sure thing16:52
rick_h_bac: can you join us for a sec? https://plus.google.com/hangouts/_/59268b0ddde3ebb3799023f43c35e7d5d1a2f80b?hl=en16:59
gary_posterMakyo, congrats on eagle landing :-)17:07
Makyogary_poster, haha, thanks :)  it'll be good to have in the GUI17:08
gary_posterdefinitely!17:08
hatchjujugui does anyone know what a bundle id is?17:12
hatchhttp://staging.jujucharms.com/api/3/bundle/~bac/wiki/wiki says ~bac/wiki/3/wiki17:12
hatchso does that mean there is no reference to the Ubuntu version?17:12
hatchor that it's a bundle?17:12
gary_posterhatch currently that would be jc:~bac/wiki/wiki IIRC.  Also, see this page:17:13
gary_posterhttp://staging.jujucharms.com/~bac/bundle/wiki/wiki17:14
gary_posteroh.  apparently it is jc:~bac/wiki/3/wiki  ?17:14
gary_posterthat's a bit odd17:14
gary_postersince it is against charms17:14
rick_h_gary_poster: yea, looking at the api to see what charmworld wants for an ID to be passed to locate a charm and get its details17:15
rick_h_so /api/3/xxx/yyy/zzz17:15
hatchyeah basically I need to know what the ID is so that I know how to parse it17:15
gary_posterhatch do you have url for an api 3 search result?  that would probably tell us what we need to know17:16
gary_posteroh there we go17:16
hatchhttp://staging.jujucharms.com/api/3/bundle/~bac/wiki/wiki17:16
rick_h_gary_poster: do they not have a series info on them?17:17
gary_posterno rick_h_ there is only one series, bundle.  So here I have something for you17:17
rick_h_gary_poster: yea, trying to figure out from the api docs http://charmworld.readthedocs.org/en/latest/api.html#terminology to actual calls how things map up to build urls17:18
gary_posterthis is the kind of result you get from a search: http://pastebin.ubuntu.com/6176395/17:18
gary_postere.g. you go to http://staging.jujucharms.com/api/3/search and search for "bac" (don't do this, it loads everything :-P)17:18
rick_h_gary_poster: right, what's a 'basket' then? Just a grouping so I might have a bunch of bundles in my rharding basket?17:19
gary_posterrick_h_, basket is deployer file17:19
gary_posterrick_h_, deployer files can contain multiple bundles17:19
gary_posterso they decided to call deployer files baskets17:19
rick_h_so you might have multiple bundles in a single deployer file...ah that starts to fit17:19
rick_h_ok, so the bundle details view might have multiple bundles in it?17:20
gary_posterso if you look at ~bac/wiki/3/wiki17:20
rick_h_or are we only working on one at a time?17:20
gary_poster~bac is the owner of course17:20
gary_posterwiki is the basket17:20
gary_postersorry, the first one is.  3 is the basket revision17:20
rick_h_gary_poster: right, ok. so we're dealing with one at a time. 17:20
gary_posterand the second "wiki" is the bundle name17:20
gary_posteryes17:20
hatchso what should the url be then? on jujucharms.com ?17:20
hatchjujucharms.com/bundle/~bac/wiki/3/wiki ?17:21
rick_h_so to ID it we'd need to know it's a bundle, the basket, the owner, the revision, and the name of the bundle in the basket17:21
gary_posteryeah17:21
gary_posterrick_h_, both of these work:17:23
gary_posterhttp://staging.jujucharms.com/api/3/bundle/~bac/wiki/3/wiki17:23
gary_posterhttp://staging.jujucharms.com/api/3/bundle/~bac/wiki/wiki17:23
gary_posterso if you don't have revision you can get head17:23
rick_h_gary_poster: right, because rev is HEAD17:24
gary_posteryeah17:24
abentleyjcsackett: Looks like it landed fine, I'll update the revno now.17:25
jcsackettabentley: yes; i'm sorry were you waiting for me to confirm?17:25
abentleyjcsackett: No, I was on lunch.17:26
jcsackettabentley: well ok then. :-)17:26
rick_h_gary_poster: can you join us then for a sec?17:27
rick_h_hatch: has a valid question17:28
rick_h_gary_poster: or not if you're busy :)17:28
hatchrare...I know17:28
hatchlol17:28
gary_poster:-)17:28
rick_h_or benji or bac if they're free. 17:28
gary_posterwhat url rick_h_ 17:28
rick_h_https://plus.google.com/hangouts/_/59268b0ddde3ebb3799023f43c35e7d5d1a2f80b?hl=en17:28
rick_h_gary_poster: are all bundles required to have an owner? e.g. no ~charmers default? Or is that not true17:40
hatchjujugui I've noticed that the failures for api restarting have lasted a very long time now, do we know if any changes were made to alter that code or is canonistack just being itself?17:49
MakyoWe could just use https://www.google.com/webdesigner/ for the GUI.17:49
hatchgoogle is making Dreamweaver, what? lol17:50
gary_posterrick_h_, don't quite understand your question.  ~charmers may own a bundle.  The "promulgated" flag determines whether that bundle is the recommended one, as with charms17:52
gary_posterhatch that looks like Canonistack to me :-(17:52
gary_postereither that or juju17:52
gary_posterbut probably canonistack17:53
rick_h_gary_poster: ok, so having a bundle without a ~owner is legit as it is in the case of a charm. 17:53
gary_posterrick_h_, oh I see, yes.  promugated.17:53
gary_posterl17:53
rick_h_gary_poster: since bundles weren't in LP I wasn't sure how that worked out. If all bundle urls would requie an explicit ~owner17:53
gary_posterrick_h_, bundles are in LP.  See "Branch" in http://staging.jujucharms.com/~bac/bundle/wiki/wiki for instance17:55
gary_poster(links to https://code.launchpad.net/~bac/charms/bundles/wiki/bundle)17:55
* gary_poster needs to go to lunch :-)17:55
* gary_poster goes to lunch and a walk17:57
hatchenjoy!17:57
abentleybac, jcsackett: I believe Jenkins is fully restored now.  Please ping if you have any problems.17:57
bacthanks abentley18:01
abentleybac: np18:02
bacrick_h_, hatch: have you seen https://docs.google.com/a/canonical.com/document/d/17bgbReU6JJMoUHSJeo8egG5zoSM0fpPIs7U8F1-Piyk/edit18:02
rick_h_bac: no, good to know18:03
hatchrick_h_: looks like they do want `bundle` in the url18:05
rick_h_hatch: so they want it in the charmworld url, doesn't mean we need it in the gui url18:05
hatchsee Actual Example URLS18:06
rick_h_hatch: so the browser Store object will need to build that correctly18:06
rick_h_hatch: right, those are all charmworld urls18:06
rick_h_hatch: staging/manage are charmworld18:07
hatchno they are on jujucharms.com it says18:07
rick_h_http://staging.jujucharms.com/ is not the gui18:07
hatchohh damn woops I totally didn't even click on that18:07
rick_h_hatch: ah, I see that there's a GUI url there18:07
hatchand yeah18:07
hatchlol18:07
rick_h_hatch: but I'm sure we can negotiate18:07
rick_h_hatch: the smaller we can make it the better 18:08
hatchmy issue with these user readable urls is that they aren't really readable at all haha18:08
hatchthey might be 'pretty' but not really readable18:08
hatchunless of course you know what you are looking for18:08
rick_h_hatch: well, sharable, twitter, printable on flyer -able18:08
hatch:)18:08
rick_h_hatch: but yea, that's been the curse with what we HAVE to have vs what is really human readable18:09
hatchrick_h_: see (where your cursor is) the valid urls18:11
hatch(or so I am assuming)18:11
hatchI don't know if we can do all of those without /bundle/18:11
rick_h_hatch: just closed it. 18:11
hatchhttps://docs.google.com/a/canonical.com/document/d/17bgbReU6JJMoUHSJeo8egG5zoSM0fpPIs7U8F1-Piyk/edit#18:12
rick_h_hatch: calling18:12
rick_h_bac benji gary_poster so there's a bundle series but it's not part of the id?18:17
benjirick_h_: are you asking if that is the case or why that is the case?18:18
rick_h_benji: I guess both. Is this true and intentionally true18:18
bacthe 'series' is part of the id.  it is just always 'bundle'18:19
benjiyes to both18:19
benjisince it will never change, making it part of the ID didn't make much sense18:19
rick_h_http://staging.jujucharms.com/api/3/bundle/~bac/wiki/wiki series is precise?18:19
bacah, i was referring to the url18:19
rick_h_so can I get a precise and a saucy version of the bundle for bac's wiki?18:20
rick_h_it would seem no since series isn't part of the url/id?18:20
rick_h_unlike charms18:20
benjiright, bundles aren't segmented into series18:21
hatchbut they have a series property?18:21
hatchso you need a new deployer or bundle for different serieses?18:22
benjiI don't know why they would have a series property.  I can't see what it would mean.18:23
hatchhttp://staging.jujucharms.com/api/3/bundle/~bac/wiki/wiki18:23
rick_h_benji: that's our confusion18:23
hatchit's there?18:23
benjihatch: define "it" and "there"18:23
hatchseries18:23
hatchin the result of the api call18:23
benjihatch: yep, it's returned from the URL you gave18:24
hatchbenji: so bundles DO have a series, but the only way for a user to determine that is not by the ID but by inspecting its details18:29
hatchan author will need to make different bundles for different series18:30
rick_h_benji: yea, but you said it makes no sense for bundles to have a revision so it confused us that the api call provided one, yet it's not part of the id to have multiple revisions of the same bundle18:30
benjiI'd say that bundles having a series is a bug.18:30
rick_h_benji: ok, makes sense now. Thanks for the clear up18:30
hatchbenji: ok so would the author make a new deployer or new bundles for series?18:31
hatchprecise-wordpress, saucy-wordpress etc etc18:31
bcsallerare we sure bundles are limited to a series?18:31
bcsallerI'm not18:31
hatchwell the parts in it are18:32
hatchso if you're using 'precise' charms the bundle only works on precise18:32
hatchwell18:32
benjihatch: it would depend on their goal; I wouldn't be surprised if some name them according to series18:33
bcsallerhatch: a bundle is more than one charm, they might not all be the same series, maybe LTS with the DB but new series for some new appserver18:33
benjithat makes me wonder if juju supports mixed series environments18:33
hatchbcsaller: right - but is that supported?18:34
bcsallerI thought it did18:34
bcsallerwe can check on another channel, but we did used to be able to do this18:34
hatchbenji: is the charm api going to match the bundle one at some point? re the revision being /3/ instead of -3 ?18:35
rick_h_bcsaller: yea, I don't think anything stops it. It's just confusing and as benji says can be considered a bug18:35
rick_h_bcsaller: the charms define what series they are in their definitinos and can be corss series, though I'm not sure how juju would do with bringing up the 'right' series  images for each charm18:35
benjihatch: I'm not aware of any plans to do that.  I'm also not aware of the difference being intentional, but I would suspect it was.  18:35
bcsallerbundles can include a default series to resolve charms from, but they can include explicit charm urls as well 18:35
rick_h_bcsaller: but urls incldue the series. It's never optional18:36
benjiah, so "series" is really "default charm series"18:36
bcsallerrick_h_: it used to be, but maybe not anymore 18:36
bcsallerrick_h_: and what if they say -HEAD18:36
bcsallerI guess that would still have series though18:36
hatchthen you're cray-cray18:37
hatchbut yeah :)18:37
bcsallerreally? package.json says things like that 18:37
rick_h_bcsaller: so revision is different. We have allowed that to be ignored, -HEAD, or an int18:37
hatchbcsaller: I know - and I wish that they would delete every module without locked down deps from npm :)18:37
bcsallerhatch: and maybe for packages that makes sense, but charms are configuration and runtime flow around packages and usually not a specific version but any in that series18:38
bcsallerand so the newest is usually expected to be the best one18:38
hatchwell that's the same everywhere but that's not always the case18:39
hatchbuut anyways - as long as Juju can deploy each charm to the proper series that's alright18:40
hatchnot sure how it'll know which one to deploy on the providers18:40
hatchbut I'm sure there is an answer for that18:41
gary_posterthe charm is for a series19:06
hatchgary_poster: right, but if a bundle contains charms for multiple series does Juju handle deploying it to the proper series?19:07
gary_posterhatch, yes.19:07
hatchthat's pretty coooooool19:08
gary_poster:-)19:08
BradCrittendenbenji: for a bundle to be 'featured' it must first be promulgated, right?19:17
=== BradCrittenden is now known as bac
benjibac: that is my understanding19:18
bacbenji: and what's the process for marking it promulgated?19:18
benjibac: If I recall correctly, there is none. :/19:19
bac\o/19:19
bacwell, my work here is done19:19
bacbenji: ok, so i'll write a killer test and we'll just hope it works in real life19:19
benjiheh19:19
hatchrick_h_: do you remember where the app makes the XHR request to get the charm data?19:24
rick_h_hatch: that's all in the Store object. 19:24
rick_h_hatch: the Store is theonly thing allowed to make calls to charmworld19:24
hatchok because this is what I have right now https://manage.jujucharms.com/api/3/charm/bundle/~bac/wiki/3/wiki19:25
hatchso I need to make the 'charm' part optional19:25
rick_h_hatch: so api3 should already provide the calls19:25
rick_h_hatch: just a matter of getting it the data19:25
rick_h_hatch: actually, don't see bundle calls, so maybe it needs a new call for bundle vs charm()19:26
gary_posterbenji, bac, to make a charm promulgated, you log onto charm world as a charmer and you have the ability to bless a charm.  I would guess it is the same for bundles.  There is a UX for that now for charms.19:26
rick_h_gary_poster: that's for 'featured' but not promulgated. I thought that was part of the LP/branch process?19:27
gary_posterI don't know whether it exists for bundles. Do you all?19:27
hatchwho chose the word promulgated? lol19:28
gary_posterrick_h_, I may be confused then.  I don't know what promulgated means then.  bac, benji, I think this is something to clarify with Rick and/or Orange squad.19:28
rick_h_gary_poster: yea, abentley do you know what the rules are for promulgated and bundles?19:28
rick_h_gary_poster: originally, any charm owned by ~charmers was promulgated19:28
rick_h_and the UI is to feature things which show up in the featured section on the front page19:29
rick_h_then the promulgation rules changed so that the juju gui charm could be 'reviewed' even though ~charmers didn't own it19:29
rick_h_I'm not sure how that was implemented in the end19:29
gary_posterrick_h_, oh and by featured you mean added to the list of editorially gardened featured charms19:30
abentleyrick_h_: I believe the charms in any basket that's linked to a series are considered promulgated.19:30
rick_h_gary_poster: correct19:30
abentleyrick_h_: s/charms/bundles/19:30
rick_h_gary_poster: benji bac ^19:30
gary_posterabentley, rick_h_ how do you link something to a series?19:30
gary_poster(in this case the bundles series?)19:31
rick_h_gary_poster: well that gets interesting when bundles don't have a series. Except benji said they did have a series and it was always bundle19:31
gary_posterright, they do, they have a single static series19:31
rick_h_so that definition/workings might need some shake down19:31
abentleygary_poster: I don't know.  sinzui? ^^19:31
rick_h_might need two? bundle/promulgated?19:31
rick_h_no sinzui here atm19:31
abentleyrick_h_: No, there's a different between having a series and being linked to it.19:32
rick_h_I've ping'd sinzui to jump in here for a sec if he's got a minute19:32
rick_h_abentley: gotcha19:32
abentleyrick_h_: All sourcepackage branches *have* a series, but only one branch is linked to it, typically ~charmers/*/trunk19:32
rick_h_thanks sinzui 19:33
rick_h_sinzui: so the question came up about how to mark a bundle as promulgated19:33
sinzuiYou don't19:33
rick_h_sinzui: and abentley mentioned that linking a branch to a series does it currently, and we were wondering how that was done/worked?19:34
rick_h_currently meaning for charms19:34
abentleyrick_h_: Oh, for charms they use a script.19:34
sinzuiMembers of ~charmers can run charm-promulgate to that links a branch to series package in Lp19:34
sinzuionly they can run it.19:34
sinzuiThey can also do in the Lp UI, though I doubt they know that19:35
rick_h_sinzui: ah ok. And is/was there a plan for bundles and this? 19:35
sinzuiyes, though I have not read the script to know if it does something charmy19:35
gary_postersinzui, we could ask them to do it via LP for bundles, though, it sounds like?19:36
rick_h_sinzui: cool, is the script in charmworld src to look at or just handed to someone? e.g. where can we find it? 19:36
sinzuidoesn't look like it enforces trunk19:37
bacrick_h_: not in charmworld afaict19:37
sinzuirick_h_, gary_poster, bac: I think they run charm-tools promulgate  -owner-branch lp:~user/charms/bundles/bundle-package/bundle --series bundles19:39
rick_h_sinzui: ah, part of charm tools. Very cool then. 19:40
gary_postercool, thanks sinzui!19:40
sinzuiIf it does not work, I think we can fix the script in a matter of minutes!19:41
bacthanks sinzui19:49
hatch I'm looking for ideas as to what to call a bundle and a charm because 'charmview' is no longer a good name20:09
hatchchundle? :D20:09
gary_posterhatch, DeployableView20:11
gary_posterDeploymentView20:11
bcsallerBundleView?20:11
gary_posterIt's supposed to handle both IIUC20:11
gary_postercharms and bundles20:11
hatchyeah it looks like they might share a lot of the same code20:12
hatchI'll know once I'm done to be sure20:12
gary_posterBundleOrCharmView might actually not be as bad as it seems :-)20:12
hatchhaha20:12
hatchnew BundleOrCharmView(yourGuessIsAsGoodAsMine); //mohohahahaha20:12
bcsallerlol20:13
bcsallerCharmworldEntityView 20:13
gary_posterheh20:13
gary_poster(to the mohahaha etc.)20:13
gary_posteryeah bcsaller's is nice.  I like the stupid simplicity of BundleOrCharmView though :-)20:14
hatchyeah but everythign else we do is with big fancy words, like promulgated20:15
bcsalleror Stack ;)20:15
hatchso we can't just go lame on this one20:15
hatchhahaha20:15
hatchstack20:15
hatchlol20:15
bcsallerBundleOrCharmOrStackView ;)20:15
hatchlol20:16
hatchCharmworldEntityView sounds good, so does DeployableVIew though20:16
hatchwell I guess I'll see if they do share enough code for it to matter20:16
hatchwe don't need fullscreen views for this bundle view do we?20:17
gary_posterwhy wouldn't we hatch?20:17
hatchoh I thought it was going away20:17
hatchor not soon enough I guess20:17
gary_posterhatch yeah it is, but...exactly20:17
* gary_poster updates hoping it doesn't kill his system, prior to restarting20:21
gary_posterIf I never come back, I left a GUI charm branch for Antonio on Launchpad...20:21
gary_poster...Jeff can have my horse...20:22
hatchyusssssss20:22
gary_poster...and I leave my six-shooter to Benji...20:22
bacgary_poster: it looks like the webops have their hands full today and won't get to our charmworld RT.  benji has agreed to shepherd it tomorrow.20:23
gary_posterok cool thanks bac.  Have a fantastic vacation!20:23
bacgary_poster: not done yet20:23
bac:)20:23
bacbut thanks20:23
gary_posterbac, oh, ok, I take it back then ;-)20:23
bacwell, just premature20:23
gary_poster:-)20:23
hatchit's been put on a 2h setTimeout20:27
hatchok so now when you visit localhost:8888/bundle/~bac/wiki/3/wiki/:flags:/charmworldv3/ you get a details page20:30
hatchas much crap as I give rick_h_ for the browser not making any sense......well no, it still doesn't make any sense20:31
hatchEHHHH seee what I did there? :P20:31
hatchkik20:31
hatchlol20:31
hatchno...nothing? well I thought it was funny :)20:35
gary_poster:-P20:45
* gary_poster restarts20:46
=== gary_poster is now known as gary_poster|away
=== gary_poster|away is now known as gary_poster
hatchman it's frustrating that bzr doesn't deal with renaming files well20:57
bachatch: huh?21:05
hatchbac: it shows the whole file as a diff instead of a rename21:07
hatchso you lose all of your 'real' diff21:08
bachatch: i think you've done something wrong.  did you use 'bzr mv'?21:08
hatchbac: no I just renamed the file21:08
baceven that should work.  put it back and then use 'bzr mv' to rename it.21:08
hatchok will try that21:09
bachatch: i'm not trying to convert you but generally when you find bzr doing something incredibly stupid then you may have done something where there is a different way that gives expected results.21:10
gary_poster+1 on all of that fwiw21:10
hatchre-proposing will see if it works this time21:11
hatchthanks21:11
hatchand yeah - you're probably right :)21:11
hatchthe issue is that when you have a git problem, a quick google search solves everything, there just aren't that many q&a's for bzr I guess21:11
hatchmost just point to the docs which don't help if you don't know what's happening :)21:11
hatchand actually I've seen this rename issue from others so I just assumed that's how it was :D21:12
bacbenji or jujugui: one review of  https://codereview.appspot.com/14151043 please?21:14
hatchsure21:14
* bac walks dog. will be back later.21:14
* benji is too late.21:15
hatchbac: done21:16
hatchbac: doesn't look like that technique worked either https://codereview.appspot.com/14153043 diff is still the whole file21:16
hatchoh well I'll just update the description to point to where the changes were21:19
bachatch: what does bzr diff say on your command line?21:21
hatchbac: `bzr diff --old ../../trunk | vim - ` shows me the same as what codereview says21:22
hatchso it must just deal with mv's as delete/create21:22
hatchinstead of rename21:23
bacMakefile => makefile21:23
bacrenamed:21:23
bac  Makefile => makefile21:23
bac=== renamed file 'Makefile' => 'makefile'21:23
bacer, that's not right21:23
hatchand the diff shows nothing?21:24
bachttp://paste.ubuntu.com/6177338/21:24
hatchhmm that's odd21:25
hatchI did the exact same thing21:25
hatchI'll try reverting it and re-applying21:25
hatchbzr mv charm.js charmworld.js21:26
hatchbac:  odly enough https://gist.github.com/hatched/1ed747f2c33ba1a629ba21:28
hatchthe output is very similar to what you said21:28
hatchs/said/had21:28
hatchbut the diff still shows a full add/delete21:28
gary_posterthat's just how it represents it in the diff, IIRC21:28
gary_posterI'm running away.  Juju local conquered me again. :-/21:29
gary_posterttyl21:29
hatchgary_poster: ohhh ok I gotcha21:29
hatchcya21:29
hatchwow they have some big hardware outside digging up the road, shaking the whole house haha21:29
hatchrick_h_: hey if you pop your head in here could you take a look at https://codereview.appspot.com/14153043 and let me know if this is on the right track for your impl21:39
rick_h_hatch: dinner and such atm but will peek later tonight.22:43
hatchyeah no rush22:43
huwshimiMorning23:00
hatchmornin huwshimi23:22
hatchwhat's Tuesday like?23:22
huwshimihatch: Rainy23:23
hatchyeah that's what I'm hearing :/23:23

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