| hatch | any bash experts in? | 00:44 |
|---|---|---|
| bcsaller | hatch: not an expert but I might be able to help | 01:03 |
| hatch | bcsaller: oh np I figured it out already | 01:03 |
| hatch | thanks though | 01:03 |
| bcsaller | heh, ok | 01:03 |
| hatch | in fact I think I found a bug in the Hadoop bash script | 01:03 |
| hatch | copied it wholesale into a script to test and it failed...so I was like wtf hah | 01:04 |
| hatch | but since you're here mind taking a look at a diff? | 01:04 |
| hatch | https://gist.github.com/hatched/fc70d29198df22cf32ad | 01:08 |
| hatch | I'm assuming that the size of the diff is because there were trailing whitespaces? | 01:08 |
| bcsaller | hatch: I'm guessing line 174 was where your question was :) | 01:20 |
| hatch | nope, see 126 vs 132 | 01:21 |
| hatch | that being on the next line caused it to not work in my test script | 01:21 |
| hatch | so I wasn't sure how to debug it | 01:21 |
| hatch | bash -x filename | 01:21 |
| hatch | the -x was what I needed | 01:21 |
| hatch | now I have to figure out how the heck to deploy a charm from my launchpad account | 01:21 |
| hatch | :/ | 01:22 |
| bcsaller | ahh, that I could have told you, you put set -xe at the top of scripts to get the echo/exit on error behavior | 01:22 |
| hatch | oh cool | 01:22 |
| hatch | what does the -e do? | 01:22 |
| bcsaller | exit on error | 01:22 |
| hatch | ahh cool | 01:22 |
| bcsaller | x is the echo | 01:22 |
| hatch | learn something new every day | 01:22 |
| hatch | :) | 01:22 |
| bcsaller | looks like it should work, you verified that was all the usage of config-get/set? its not in the relation hooks or anything? | 01:23 |
| bcsaller | or are they all symlinks? | 01:23 |
| hatch | all symlinks | 01:23 |
| hatch | interesting approach but smart | 01:23 |
| hatch | now I'm trying to test it but I can't figure out how to deploy it lol | 01:24 |
| hatch | looks like it's gota be local | 01:25 |
| hatch | hmm I want to deploy a local charm via the GUI | 01:32 |
| hatch | I wonder if this is a rare occurance | 01:32 |
| bcsaller | yeah, it doesn't work by default | 01:33 |
| hatch | I'm not actually sure how you would do it withou a local charmstore | 01:34 |
| hatch | but it would be cool if we could select a dir to deploy | 01:34 |
| bcsaller | hatch: we've talked about a number of longer term plans for how to do this, but currently I think your instinct was correct a local charm store and maybe changing the config settings to point the API endpoint there | 01:45 |
| hatch | yeah I'm definitely not going to attempt setting up a local charmstore | 01:49 |
| hatch | :D | 01:49 |
| hatch | I gota go cook supper soon haha | 01:49 |
| bcsaller | me too | 02:13 |
| rick_h_ | hatch: it's a charm, you can push your branch up and get it in the official store | 03:06 |
| rick_h_ | hatch: just 'juju publish' it. It'll be under your username, etc. | 03:06 |
| rick_h_ | https://jujucharms.com/fullscreen/search/?text=apache look at the 'more charms' section :) | 03:07 |
| hatch | rick_h_: oh cool thanks good to know | 04:14 |
| hazmat | hatch, thanks | 06:38 |
| hazmat | anyone know how to get personal charms to show up in the gui search? | 06:39 |
| hazmat | hmm.. looks like its a manage.jujucharms.com issue.. | 06:42 |
| rick_h_ | benji: is charmworld still sans lbox? | 12:10 |
| benji | rick_h_: some people have claimed that it works for them, but I haven't been able to get it to work. The problem may be the way I have lbox installed on my LXC container. | 12:11 |
| rick_h_ | benji: hmm, that's what I'd be doing so if you're willing to do LP review I'll skip it then. | 12:12 |
| rick_h_ | benji: writing things up and will ping once done if you've got time to peek at it. | 12:12 |
| benji | rick_h_: sure | 12:13 |
| rick_h_ | benji: https://code.launchpad.net/~rharding/charmworld/bundle-metadata/+merge/188905 (one line lint fix just pushed and will sync in a min) | 12:18 |
| rick_h_ | thanks! | 12:18 |
| rick_h_ | benji: and so the idea is that with this we get support for the trailing bits of a url after the id, so going to add a bundle/some/id/charmdetails to provide the different payload format withthe charm details per our previous discussion? | 12:20 |
| rick_h_ | does that jive with what you recollection of our talk on Tues? | 12:20 |
| benji | rick_h_: makes sense | 12:22 |
| benji | rick_h_ (and gary_poster): I need to run an unscheduled child to daycare, I'll be back in about 15 minutes | 12:23 |
| gary_poster | ok benji | 12:23 |
| rick_h_ | thanks benji | 12:23 |
| frankban | luca__: hi, could you point me to any css/html resources for building a webpage similar to juju.ubuntu.com? | 12:53 |
| rick_h_ | benji: when you get back want to see if you've got a sec for a chat. A question came up from my call with gary | 12:54 |
| rick_h_ | frankban: there's the base resources. /me goes to look where they shared that. | 12:54 |
| frankban | rick_h_: thanks | 12:55 |
| benji | rick_h_: I'm available now. | 12:55 |
| rick_h_ | benji: want to use your chat thing? | 12:55 |
| benji | rick_h_: sure: benjiyork.com/chat | 12:56 |
| rick_h_ | frankban: so there's a lp project of shared base css/etc resources and I'll have to dig for it. If luca__ comes by he might remember the link. The web team in HQ runs it | 12:56 |
| rick_h_ | it's like ubuntu bootstrap | 12:56 |
| luca__ | frankban: rick_h_ heya | 12:57 |
| luca__ | frankban: rick_h_ this is the link: https://dl.dropboxusercontent.com/u/46840621/Web%20style%20guide/v3/get-started.html | 12:58 |
| luca__ | frankban: rick_h_ it's work in progress but should be what your looking for | 12:58 |
| rick_h_ | benji: http://bazaar.launchpad.net/~bac/charms/bundles/wiki/bundle/view/head:/bundles.yaml | 12:58 |
| frankban | hi luca__, thanks! | 12:58 |
| luca__ | frankban: rick_h_ if you need any help or information, ping Anthony Dillon (ant on IRC) and he should be able to fix anything | 12:59 |
| rick_h_ | http://charmworld:2464/~bac/bundle/wiki/wiki | 13:02 |
| rick_h_ | https://manage.jujucharms.com/~bac/bundle/wiki/wiki | 13:03 |
| rick_h_ | frankban: luca__ http://bazaar.launchpad.net/~ya-bo-ng/ubuntu-brand-guidelines/trunk/files is what I was thinking of I think | 13:32 |
| gary_poster | hey jujugui, I feel not good. Going to go lie down for a few. back when I can; will periodically check email | 13:33 |
| rick_h_ | gary_poster: good luck | 13:34 |
| gary_poster | ty | 13:34 |
| jcastro | when we remove stuff in the GUI | 13:44 |
| jcastro | does that destroy-service or destroy-unit? | 13:44 |
| rick_h_ | jcastro: there's the option for each, not following you. | 13:46 |
| jcastro | http://askubuntu.com/questions/353114/how-can-i-know-which-machine-juju-is-actually-using | 13:47 |
| jcastro | I am wondering if people think they're removing units | 13:47 |
| jcastro | but are not | 13:47 |
| rick_h_ | I know lxc has a thing where destroy service doesn't remove the machine, maybe azure has an issue there? | 13:49 |
| * rick_h_ checks some code quick | 13:49 | |
| jcastro | yeah, since this is an azure one I was wondering that too | 13:50 |
| jcastro | nice to know people are trying it on azure! | 13:50 |
| rick_h_ | well, so destroy service calls destroy service. I'd be curious to know if the gui instance was recent enough to have the inspector. In the older UI I could see where destroy unit might be what he clicked vs service destroy. | 13:52 |
| rick_h_ | when you destroy a unit does the underlying machine get removed? | 13:52 |
| rick_h_ | guess, the only real way to chase it would be to try it out and see if we could dupe it. | 13:53 |
| jcastro | I'm going to ask him to mail me, I have a bunch of other questions to ask him | 14:01 |
| jcastro | more about "hey so did our docs suck?" and so on | 14:01 |
| rick_h_ | jcastro: ok cool. Let us know if you can help when you get more details. Knowing more of what version of the Gui he was using, what he actually did, etc | 14:03 |
| rick_h_ | I don't know if any of us have tried things on azure yet so cool to know it at least got going and working :) | 14:04 |
| frankban | jcastro, rick_h_ : from the GUI we send a DestroyService API call. ServiceDestroy does not destroy the machine (and I believe this behavior is shared by all providers). AFAICT there is no API for destroying a machine. DestroyServiceUnits also seems to leave the machine up. | 14:09 |
| jcastro | yeah the thing is this guy thought he was running X VMs, but was really running X+whatever. | 14:13 |
| jcastro | that's going to cost people money | 14:14 |
| jcastro | then they're going to say "wait, but I told the GUI I didn't want those services anymore, why are there blank machines running up my bill?" | 14:14 |
| frankban | jcastro, rick_h_: That's right. I believe we will handle that as part of our machine/containerization story. Right now the GUI does not manage/expose machines at all, and as I mentioned my understanding is that there is no API to do that (we will need an API for all the provisioning CLI commands). | 14:17 |
| * jcastro nods | 14:19 | |
| frankban | jcastro: you might want to ping gary_poster when he's back, he surely have a better knowledge of the problem | 14:20 |
| jcastro | I think in the meantime ... would it be a bad thing to note this in the docs or the charm readme? Like in huge letters? | 14:20 |
| jcastro | I am just envisioning some guy trying out juju at work and running up this huge bill | 14:20 |
| jcastro | and his boss getting stuck with the check with a mental image of juju... that thing that cost me money. | 14:20 |
| frankban | jcastro: yes, that's a good idea, I'll make a card | 14:22 |
| rick_h_ | frankban: oh, I thought it was just an lxc thing which is how I end up doing things. Knowing is half the battle I guess | 14:22 |
| frankban | rick_h_: yeah, that's the same on ec2. It's the way juju-core works when you just destroy a service | 14:23 |
| frankban | rick_h_: pyJuju did that differently: destroy-service == bye bye machine | 14:25 |
| jcastro | yeah, I think we should bring in design at the sprint too. We need to ensure that the GUI always is upfront on how many instances are running for real | 14:25 |
| jcastro | we can't let that be hidden or put behind somewhere where it won't be discoverable | 14:25 |
| rick_h_ | frankban: ahhhh, maybe that's where I'm confused then. I only moved to juju-core once lxc came out | 14:25 |
| rick_h_ | jcastro: yea, there was work on ideas for a 'machine view' during the last sprint, but that's work that's a little bit out atm | 14:26 |
| benji | abentley: I have a demo bundle I want to get into charmworld, will it be picked up automatically, or is there something I have to do? | 14:28 |
| frankban | rick_h_: created a card in maintenance high: GUI Charm: document that machines are not removed when you destroy services from the GUI | 14:31 |
| jcastro | rick_h_: yeah I get that, but maybe there's some low hanging fruit somewhere | 14:32 |
| jcastro | rick_h_: if you don't know how juju works it looks like the gui is lying to you | 14:32 |
| rick_h_ | jcastro: yea, just meant the idea is on the radar. | 14:32 |
| abentley | benji: It will be picked up automatically, as long as the naming conventions are right. | 14:32 |
| benji | abentley: great, thanks | 14:33 |
| abentley | benji: file must be named "bundles.yaml", series must be "bundles", branch must be named "bundle" | 14:33 |
| abentley | benji: This just got picked up automatically yesterday: http://manage.jujucharms.com/~abentley/bundle/charmworld/staging | 14:33 |
| benji | abentley: I think I have it right. I branched from bac's bundle, which appears to work | 14:33 |
| Makyo | 40 minutes -> 10 minutes. I kind of hate this battery. | 14:45 |
| rick_h_ | Makyo: replacement battery? | 14:45 |
| Makyo | rick_h_, I guess so. It's been less than a year, though, so I'm mostly just disappointed. | 14:45 |
| rick_h_ | Makyo: gotcha, yea mine is just over a year and getting to that time so battery on the brain | 14:46 |
| rick_h_ | Makyo: http://paste.mitechie.com/show/1027/ ? | 14:57 |
| Makyo | .182222 | 15:02 |
| rick_h_ | wow! | 15:02 |
| rick_h_ | so when you charge it up now, it's only going to 18% of the original capacity | 15:02 |
| rick_h_ | and it's less than a year old?! | 15:02 |
| Makyo | Yeah, I got it just before Copenhagen. | 15:03 |
| rick_h_ | mine's at 75% and on the replacement block. | 15:03 |
| rick_h_ | hit 1yr around july I think | 15:03 |
| rick_h_ | crazy | 15:03 |
| Makyo | Oh well. Sent in a support ticket. If the solution is to buy a new battery, that's fine, will just have to wait. | 15:09 |
| rick_h_ | yea, if less than a year that seems a warranty issue for sure | 15:10 |
| Makyo | I checked, it's just about a year, got it about this time last year, from the looks of it. I didn't do the "periodically drain the battery" thing, though I'm not sure if you need to do that anymore. | 15:11 |
| rick_h_ | yea, it should destroy it that bad :/ | 15:11 |
| Makyo | Damn. | 15:12 |
| rick_h_ | heh, warranty die yesterday? | 15:12 |
| Makyo | Last week. | 15:12 |
| Makyo | Oh well. | 15:12 |
| hatch | Makyo: well time to get the Air setup for dev :) | 15:13 |
| Makyo | Haha, I guess! | 15:14 |
| Makyo | I'll have it set up by Oakland, for sure. | 15:14 |
| Makyo | It kinda works now, just not ideal. | 15:14 |
| hatch | it has 8GB or 4GB? | 15:14 |
| Makyo | 4 | 15:14 |
| Makyo | So lboxing will take a while. | 15:15 |
| hatch | ahh - well you might want to run Ubuntu on metal then if you want some better perf outa it :) | 15:15 |
| Makyo | It works fine for actual dev work, tbh. | 15:15 |
| hatch | oh that's cool - I was a little worried the vm would use too much ram | 15:16 |
| Makyo | I have it somewhat constrained, but it's also a very light-weight installation of ubuntu server, rather than desktop. | 15:17 |
| benji | abentley: how long should I wait for my bundle to be picked up before becoming concerned? It has been well past the 15 minute enqueue/ingest cycle. | 15:20 |
| abentley | benji: It should only take 15 minutes, so I'd be concerned now. | 15:21 |
| benji | darn | 15:21 |
| abentley | benji: What's the branch? | 15:21 |
| benji | abentley: https://code.launchpad.net/~benji/charms/bundles/wiki/bundle | 15:22 |
| abentley | benji: Nothing obviously wrong to me. | 15:24 |
| benji | abentley: thanks for looking | 15:24 |
| hatch | well I have now spilt two cups of coffee on my desk today....w t h | 15:26 |
| abentley | benji: So, it's up on staging, kinda. http://staging.jujucharms.com/~benji/bundle/wiki/wiki | 15:27 |
| benji | abentley: the kinda part being the "Internal Server Error"? :) | 15:28 |
| abentley | benji: Yes. I think it's because your bundle doesn't have a series. | 15:28 |
| benji | ah | 15:29 |
| benji | I added a series, we'll see how that works. | 15:30 |
| benji | I'm confused though because it worked on my local charmworld | 15:30 |
| abentley | benji: /home/webops_deploy/charmworld/charmworld/templates/bundle.pt refers to ${bundle.data.series}. I don't know how it worked on yours. | 15:31 |
| benji | abentley: well, I didn't view the page, I was hitting the API with the GUI, but that isn't working on production either so I'm still confused | 15:32 |
| abentley | benji: Your bundle isn't showing on production, so that's a separate issue. | 15:32 |
| benji | mmm, good point | 15:33 |
| benji | I'll try pointing my GUI at staging. | 15:33 |
| sinzui | bac: benji, jcsackett : I jot hate mail from charmworld staging: https://bugs.launchpad.net/charmworld/+bug/1234780 | 15:34 |
| abentley | sinzui: Yes, I triggered it by visiting benji's bundle: http://staging.jujucharms.com/~benji/bundle/wiki/wiki | 15:34 |
| benji | yep, we'll have to fix... something so bad bundles won't go down in flames like that | 15:35 |
| abentley | sinzui: But there's a separate issue: that bundle isn't appearing on production, even though it's had plenty of time to ingest. | 15:35 |
| sinzui | abentley, from when? | 15:35 |
| abentley | benji: ^^ | 15:36 |
| sinzui | benji, abentley we killed a stale proc from before the bad release to get charms ingesting again. | 15:37 |
| benji | sinzui: 09:11:01-0500 | 15:37 |
| sinzui | okay, that is just minutes after we completed. I think the cron or run-forever queuing is broken in production | 15:38 |
| sinzui | abentley, I think we just need supervisord running to queue. and cron will ingest about every 2 to 4 times per hour | 15:39 |
| abentley | I'm pretty sure cron does the queueing and supervisord manages the ingest. | 15:40 |
| abentley | sinzui: ^^ | 15:40 |
| sinzui | abentley, yep. This is what webops created to make cron happy (i think) https://pastebin.canonical.com/98319/ | 15:42 |
| abentley | sinzui: Looks sane to me. | 15:43 |
| sinzui | I will enquirer about it | 15:44 |
| hatch | jujugui call in 10 | 15:50 |
| * hatch is pretty pumped that he beat Makyo to it today | 15:51 | |
| Makyo | Whoa whoa whoa. | 15:52 |
| Makyo | Coffee is WAY more important than this. | 15:52 |
| hatch | rick_h_: so I've noticed that `bzr grep` doesn't give line numbers or syntax highlighting - am I missing a flag or something? | 15:57 |
| sinzui | abentley, benji looks like mthaddon found the issue, crontab was manually put in the wrong place in production. He is fixing it | 15:57 |
| rick_h_ | hatch: no, that's ack stuff. I just open the file and use vim /xxx | 15:58 |
| hatch | oh ok, then back to ack it is :) | 15:58 |
| Makyo | jujugui call in 2 | 15:59 |
| hatch | damnnn | 15:59 |
| Makyo | Hahaha | 15:59 |
| rick_h_ | benji: my new desktop env pulled your charm in | 16:01 |
| benji | rick_h_: good; I think production is on its way to being fixed | 16:02 |
| rick_h_ | cool | 16:02 |
| Makyo | luca__, ping | 16:02 |
| sinzui | abentley, benji : I misspoke. I still have no idea what is not always running to ensure we ingest | 16:03 |
| benji | mmm | 16:03 |
| luca__ | Makyo: heya | 16:03 |
| luca__ | Makyo: I need to check the upgrade stuff | 16:03 |
| abentley | sinzui: You can try running bin/ingest-queued manually to see if the queue's getting stuff. Remember to set INI and run as ~charmworld. | 16:04 |
| Makyo | luca__, yeah, have a sec after daily meeting? | 16:06 |
| luca__ | Makyo: sure, just ping when your free | 16:06 |
| sinzui | abentley, I don't want to manually run. that is what got the last ingest sorted. I want to know what is not runnung | 16:13 |
| abentley | sinzui: I was suggesting manually running as a way to determine what's not running. If the queue is full, then you know that cron is happy. If it's empty, then you know that it's not happy. | 16:14 |
| sinzui | okay, I will ask again if the queue is full | 16:15 |
| Makyo | luca__, https://plus.google.com/hangouts/_/9ad52af076c648af1c3c61fcda614bb034cfc336?hl=en | 16:19 |
| luca__ | Makyo: give me 2 mins :) | 16:19 |
| Makyo | luca__, okay | 16:19 |
| luca__ | Makyo: coming now | 16:22 |
| hatch | well I finally had to turn the heat on... 16C is too cold :) | 16:24 |
| gary_poster | hey rick_h_ & benji, am I right that this is something we need to address asap? https://bugs.launchpad.net/bugs/1234780 | 16:34 |
| _mup_ | Bug #1234780: Dict has no attribute series <bundles> <charmworld:Triaged> <https://launchpad.net/bugs/1234780> | 16:34 |
| sinzui | abentley, I think we will ingest https://pastebin.canonical.com/98468/ | 16:41 |
| sinzui | abentley, I think queuing via cron is bad | 16:41 |
| sinzui | I could ask for a manual queue, wait 15 minutes and verify all it good. then return to the question of why cron is not | 16:42 |
| abentley | sinzui: I'm pretty sure I changed the default delay to 1 minute, not 15. | 16:42 |
| sinzui | abentley, not on staging and that is where we stole the crontab from | 16:43 |
| abentley | sinzui: I meant the default delay for ingest-queued. | 16:43 |
| sinzui | oh | 16:44 |
| rick_h_ | gary_poster I'm not sure. Do we have the idea of a proof for bundles? this seems like if it's a required attribute then proof/ingest would catch/ignore it | 16:44 |
| rick_h_ | gary_poster: so I guess yes we should adress but unsure how atm | 16:45 |
| gary_poster | rick_h_, doesn't that look like a problem in the charmworld code, not the bundle? | 16:48 |
| gary_poster | AFAICT the traceback is saying that the template is trying to look at the data as an object, not a dict | 16:49 |
| rick_h_ | gary_poster: well, series is expected to exist. The question is, can a bundle not provide it? (reasonable to expect it to always be there) or is it ok to not have and thus the template/charmworld needs to update to deal with the lack of it | 16:49 |
| gary_poster | rick_h_, the bundle has a series: http://bazaar.launchpad.net/~benji/charms/bundles/wiki/bundle/view/head:/bundles.yaml | 16:50 |
| rick_h_ | gary_poster: ah ok then. Yea. | 16:50 |
| gary_poster | rick_h_, but data is a dict, so the code would have to say /series | 16:50 |
| rick_h_ | strange if it's an issue there it dodn't show befoer | 16:50 |
| gary_poster | or ['series'] | 16:50 |
| gary_poster | cool rick_h_ , thanks.I'll make a card for bug | 16:51 |
| rick_h_ | gary_poster: cool, will coodinate with benji when he's back around then and peek at the code after I fix up these review changes | 16:51 |
| gary_poster | cool | 16:52 |
| gary_poster | thank you | 16:52 |
| abentley | gary_poster: benji just added the series. | 17:09 |
| gary_poster | abentley, oh! so it was what rick_h_ said: bundle had the problem, not code? | 17:10 |
| gary_poster | other than proof equivalent needed in charmworld | 17:11 |
| abentley | gary_poster: I think series is supposed to be mandatory. But the error handling could stand improvement. | 17:11 |
| rick_h_ | gary_poster: yea, I think we had a chat with marcoceppi once about the idea of updating the proof tools to be able to detect or be told to proof using rules for charm or bundle in one command | 17:11 |
| rick_h_ | and then run proof against both types on ingest | 17:11 |
| gary_poster | got it, thanks abentley and rick_h_ | 17:11 |
| abentley | gary_poster: (Now that it has a series, it's not broken: http://staging.jujucharms.com/~benji/bundle/wiki/wiki) | 17:12 |
| gary_poster | gotcha | 17:12 |
| abentley | sinzui: successful test run, from creating the container, through running the test suite, to destroying the container. | 17:13 |
| sinzui | abentley, there is a fix for that in 1.15.1 | 17:13 |
| abentley | sinzui: A fix for succcessful test runs? | 17:14 |
| sinzui | abentley, sorry. I am in too many places | 17:14 |
| sinzui | any I just put my mouth the the production firehose. prod will email me | 17:15 |
| rick_h_ | abentley: benji is charmworld landing still just mark approved and tarmac comes along and grabs it? I've not been following it lately. | 17:17 |
| abentley | rick_h_: It's still just mark approved, but it's all-Jenkins, no Tarmac. | 17:18 |
| rick_h_ | abentley: ah, very cool. Thanks. | 17:18 |
| rick_h_ | abentley: hmm, I can't see the "approved" in the drop down for status here? https://code.launchpad.net/~rharding/charmworld/bundle-metadata/+merge/188905 | 17:19 |
| rick_h_ | ah, pending approval to the charmworl developers team. | 17:20 |
| rick_h_ | can anyone approve me in please or add the Juju Gui Hackers team? sinzui abentley ^ | 17:21 |
| abentley | rick_h_: You should see it if you're a member of the branch owner team, Juju-Jitsu Hackers. | 17:21 |
| rick_h_ | abentley: looking | 17:21 |
| rick_h_ | abentley: ah, I am not. Asking back in now. | 17:21 |
| rick_h_ | will need an ok on that one as well. | 17:22 |
| * rick_h_ is feeling the repercussions of losing super orange powers | 17:22 | |
| * rick_h_ wonders how benji got to submit | 17:22 | |
| benji | rick_h_: that (just mark it approved) is my understanding | 17:24 |
| rick_h_ | benji: right, but I'm not member enough to have permission to makr it approved | 17:24 |
| rick_h_ | benji: I only see the options to mark it as WIP, needs review, merged | 17:24 |
| benji | rick_h_: hmm, want me to do it? | 17:25 |
| abentley | rick_h_: He's a member of yellow squad. | 17:25 |
| rick_h_ | benji: I'm not seeing you in the group abentley mentioned either though. Chasing the permission chain | 17:25 |
| rick_h_ | benji: if you don't mind until I get admins to approve my requests to join their teams | 17:25 |
| sinzui | rick_h_, I have approved you | 17:25 |
| rick_h_ | sinzui: thanks | 17:26 |
| rick_h_ | I'll ask gary_poster to update yellow squad when he's got time. | 17:26 |
| rick_h_ | thanks sinzui got it marked approved now. | 17:26 |
| gary_poster | rick_h_, everybody is in yellow. probably should retire yellow instead but enh | 17:30 |
| rick_h_ | gary_poster: thanks for the update | 17:30 |
| gary_poster | rick_h_, I mean, I added you and everyone else in gui | 17:30 |
| gary_poster | welcome | 17:30 |
| hatch | rick_h_: is 'editorial' the token list? | 17:32 |
| hatch | it feels like it should be the details view? | 17:33 |
| rick_h_ | hatch: editorial is the default content in the side panel which is the three containers (featured, popular, new) | 17:33 |
| rick_h_ | they're 'editorial' selected charms to show on page load by default | 17:33 |
| hatch | uhh wut? | 17:34 |
| rick_h_ | hatch: so in the sidebar View you can load two bits of content: editorial content or search results content | 17:34 |
| hatch | lol | 17:34 |
| rick_h_ | which are each a View | 17:34 |
| hatch | so editorial is not editorial | 17:34 |
| rick_h_ | hatch: call? | 17:34 |
| hatch | it's the token list | 17:34 |
| hatch | sure | 17:34 |
| rick_h_ | hatch: sure it is, it's three containers (and used to have the listed categories) and such | 17:34 |
| === matsubara is now known as matsubara-lunch | ||
| gary_poster | hey bcsaller I'm playing around with the visualization via tests and would like to make faster progress. you have a sec for a hangout? | 17:55 |
| marcoceppi | rick_h_: how close are bundles to being ready? | 18:01 |
| marcoceppi | I'm planning on pressing a release before cloud sprint, if you'd like that in there I need to know sooner than later | 18:01 |
| marcoceppi | also, can someone fill in this guy with a roadmap to being able to use "--to" in the gui? http://askubuntu.com/q/353262/41 | 18:02 |
| rick_h_ | marcoceppi: so we can ingest/show them in manage.jujucharms.com now. hatch will have a details view in the gui next week, and so parts of it will be giong on next week | 18:02 |
| rick_h_ | marcoceppi: so yea, bundles will be pre-sprint and so bundle support in proof would be greatly appreciated :) | 18:03 |
| rick_h_ | marcoceppi: for the ask ubuntu gary_poster would have to provide the best answer, but I think it's part of a big chunk of work for 14.04? | 18:05 |
| marcoceppi | rick_h_: if you haven't already, I need deets! https://bugs.launchpad.net/charm-tools/+bug/1222833 | 18:07 |
| _mup_ | Bug #1222833: charm tools should proof bundles as well <Juju Charm Tools:New> <https://launchpad.net/bugs/1222833> | 18:07 |
| marcoceppi | sample no longer works | 18:07 |
| rick_h_ | bcsaller: are you a deployer expert to know which all fields are required/optional in a file format? Can you comment in ^^ bug? | 18:09 |
| rick_h_ | or maybe frankban since I know he did a lot of that in the server bits. | 18:09 |
| gary_poster | marcoceppi, rick_h_ I answered the askubuntu thing. | 18:10 |
| rick_h_ | gary_poster: thank you. | 18:10 |
| gary_poster | rick_h_, benji is your best bet. frankban has EoD'd | 18:10 |
| rick_h_ | gary_poster: right, no hurry. I figured I'd shoot for someone that's used the deployer/messed with that code itself wouldhave the best knowledge | 18:11 |
| gary_poster | cool rick_h_. thx | 18:11 |
| benji | rick_h_: I would have to read the code to see what is required, but I would assume "services" is the only required key ("services" should be made optional by charmworld) | 18:14 |
| benji | oops, that second "services" should be "series" | 18:14 |
| === matsubara-lunch is now known as matsubara | ||
| gary_poster | benji, I'd guess proof could go farther if desired--validating the deployer format generally, yeah? | 18:22 |
| rick_h_ | gary_poster: yea, one items was to check it's valid yaml/json | 18:22 |
| gary_poster | yeah, reasonable | 18:22 |
| benji | gary_poster: yep; for one thing, it could check for a bug I have seen, which is relation names not being correct in the bundle (it will obviously need net access to load charm details to do that, though) | 18:23 |
| rick_h_ | stuff like that I think charmworld can do easier. I'm not sure proof should be online? | 18:24 |
| rick_h_ | proof doesn't check everything ingest does, but it's a basic linter | 18:24 |
| rick_h_ | curious what marcoceppi thinks on that part though. | 18:25 |
| gary_poster | marcoceppi, I am making a card for us to give you some details. how soon do you need them? and thank you, btw. :-) | 18:26 |
| rick_h_ | benji: isn't there an issue with setting kwarg defaults to mutable types? Is dict() considered immutable? | 18:31 |
| benji | rick_h_: dict() just returns an empty dict, so it is the same (mostly) as using {}, so yeah, you wouldn't want to use that as a default (unless you really do, but still you probably shouldn't because it will confuse people) | 18:33 |
| rick_h_ | benji: ok, just noticed it in get_bundle_data and hasn't seen dict() used and wondered if it somehow got around the badness of using {}/[] for kwarg defaults | 18:34 |
| benji | rick_h_: the "issue" isn't so much a bug or anything like that, it's that people often don't understand default keyword arguments are evaluated when the function is read, not every time it is called, so a mutable default could be changed by the function from one call to another | 18:35 |
| rick_h_ | benji: rgr, which from a factory method creating different versions seems like a particularly troubled spot to reply on those defaults | 18:35 |
| rick_h_ | benji: but yea, all good. Thanks for the sanity check. I'll tweak in a drive by in this branch | 18:36 |
| benji | rick_h_: yeah, for the code in question it will technically work, but it is an attractive nuisance; I agree that it should be tweaked | 18:37 |
| bcsaller | rick_h_: looks like you have that resolved? | 18:41 |
| rick_h_ | bcsaller: rgr, thanks | 18:41 |
| bcsaller | gary_poster: did you still want to chat about the vis stuff? | 18:42 |
| gary_poster | bcsaller, yeah, thanks. 1 sec... | 18:42 |
| gary_poster | bcsaller, calling on https://plus.google.com/hangouts/_/58de968d2a9ae12f362336b9c9a13fb586d4c581?hl=en | 18:42 |
| bcsaller | very odd | 18:44 |
| marcoceppi | gary_poster: sooner the better. By Wednesday if you want me to have time to get it in the release | 18:50 |
| gary_poster | ok thanks marcoceppi | 18:50 |
| marcoceppi | rick_h_: proof probably shouuld, orange has opened a bunch of bugs about this IIRC | 18:50 |
| gary_poster | hey hatch, is your branch working enough for me to use as a basis for playing around with integrating Ben's visualization? If so, could you push it somewhere and let me know the address? | 19:08 |
| hatch | gary_poster: I have 12 failing tests I'm just finishing up now, then it'll be ready to propose | 19:08 |
| hatch | so can you give me a few minutes to eval these failures then I'll get back to you | 19:08 |
| gary_poster | hatch, cool. if it is working in qa then I can use it, test failures or no. so, please push it soon-ish however that turns out | 19:09 |
| hatch | oh ok, yeah as long as you aren't going to base any real code off of it, just incase the tests expose changes that need to be done | 19:09 |
| hatch | gary_poster: lp:~hatch/juju-gui/bundle-view-url http://localhost:8888/bundle/~benji/wiki/wiki/:flags:/charmworldv3/ to view | 19:11 |
| gary_poster | ok thanks hatch! | 19:11 |
| hatch | no problemo | 19:13 |
| hatch | I -really- wish I knew why phantom crashes when being run in the foreground | 19:14 |
| hatch | it's gota be these 'get' logs | 19:14 |
| benji | abentley: please let me know when you have time for a short call about the ingest process and backfilling | 19:33 |
| abentley | benji: Okay. Probably in 30. | 19:33 |
| benji | abentley: thanks | 19:34 |
| hatch | jujugui could I get a quick review and qa on https://codereview.appspot.com/14355043 thanks | 19:45 |
| bcsaller | hatch: pulling it down now | 19:46 |
| hatch | thanks | 19:46 |
| abentley | benji: I'm free now. | 19:58 |
| benji | abentley: ok, one second; I left my camera outside | 19:58 |
| hatch | bcsaller: thanks for the review - I'm not really clear on the direction you want me to go with your comments | 19:59 |
| benji | abentley: try benjiyork.com/chat | 20:00 |
| bcsaller | hatch: sounds like you want to make changes in later branches then and go as is? | 20:00 |
| hatch | bcsaller: yeah I want to try and do atomic changes and work towards the final goal - I think the reviews/qa's will be easier this way | 20:01 |
| bcsaller | then I'll +1 it | 20:01 |
| abentley | benji: I get a black screen that says "Initializing..." with Firefox | 20:01 |
| benji | abentley: ok; I'll create a hangout then | 20:01 |
| abentley | benji: Chrome works. | 20:02 |
| abentley | benji: Or at least it says Connecting... and shows my face. | 20:02 |
| hatch | bcsaller: cool thanks | 20:03 |
| hatch | gary_poster: I'm going to be submitting that branch now just FYI | 20:03 |
| gary_poster | cool hatch thanks! | 20:03 |
| gary_poster | I'm playing with integrating. I'll put my face on the card. | 20:03 |
| hatch | Makyo: Loving OneTab :) thanks for the tip | 20:04 |
| Makyo | Yeah! | 20:05 |
| Makyo | It's not perfect, but it's a pretty good grouping solution to cut down on memory footprint from multiple windows. | 20:05 |
| hatch | yeah I have 54 windows in it right now haha | 20:05 |
| hatch | tabs* | 20:05 |
| Makyo | Yeah. It's also kind of neat that you can restore one, or a whole group, or publish the list. | 20:07 |
| Makyo | Oh, so these aren't battery crashes. 23 LXCs crashing the machine. | 20:19 |
| Makyo | BEcause I'm not destroying machines. | 20:20 |
| bcsaller | ouch | 20:20 |
| Makyo | I have everything tuned way down to try and increase battery, though, so I guess it's both. | 20:20 |
| hatch | lol | 20:21 |
| abentley | sinzui: Complete test run in lxc: http://162.213.35.27:8080/job/charmworld-autoland-lxc/lastBuild/ | 20:25 |
| gary_poster | bcsaller, good news is that I have it working. (we do need fakebackend, it turns out, to load bundle into db). Not as good news is that the one good bundle we have doesn't render very well. I suspect that the root cause is that the bundle does not have x-y annotations. The symptoms are that I see a lot of errors trying to draw relation lines ('Invalid value for <line> attribute y2="NaN"') and I only see a single | 20:25 |
| gary_poster | charm (because they are all on top of one another. So... | 20:25 |
| bcsaller | gary_poster: wp-deployer? I thought that did have annotations, we even test the positions | 20:26 |
| bcsaller | oh... from the store | 20:26 |
| bcsaller | yeah... | 20:26 |
| bcsaller | there isn't really any auto layout | 20:26 |
| bcsaller | the assumption being these things will be placed, but thats an area to work on for sure | 20:27 |
| gary_poster | so a few options/things to consider... | 20:27 |
| gary_poster | (1) we can ask marco to enforce that bundles have x-y coords | 20:27 |
| abentley | benji, rick_h_, sinzui, jcsackett: I'm switching over to doing the landings with testing "make sysdeps". (by using an lxc). Sorry in advance for any bumpiness. | 20:27 |
| gary_poster | that will probably be the cheapest option and a reasonable short term solution | 20:28 |
| gary_poster | (2) auto layout | 20:28 |
| benji | abentley: so it will be a "fresh" system each time the tests are run? | 20:28 |
| gary_poster | maybe there are only two ;-) | 20:28 |
| abentley | benji: Right. | 20:28 |
| benji | very cool | 20:28 |
| bcsaller | gary_poster: well... auto layout is a spectrum | 20:28 |
| gary_poster | fair enough | 20:28 |
| bcsaller | we could pack them like we do now but the layouts are un-attractive | 20:29 |
| gary_poster | bcsaller, if we can do a "it doesn't fall over" story for this case it would be a win | 20:29 |
| gary_poster | but bcsaller I think we ought to say that #1 is required. WDYT? | 20:29 |
| gary_poster | that way all bundles from store at least will be ok | 20:30 |
| bcsaller | #1 +1 | 20:30 |
| gary_poster | although | 20:30 |
| gary_poster | we should have a nice story for loading | 20:30 |
| gary_poster | bcsaller, does the "drag file on GUI" import story work for deployer files again? | 20:30 |
| hatch | rick_h_: any chance you are about to land the token bundle click branch? | 20:30 |
| bcsaller | they are supposed to capture additional human thinking about a deployment and position conveys some information | 20:30 |
| bcsaller | gary_poster: it works with things like test/data/wp-deployer.yaml that have a single import target | 20:31 |
| bcsaller | gary_poster: otherwise you need the bundle target name and a UI to select it | 20:31 |
| bcsaller | but if you provide that it works that way too | 20:31 |
| gary_poster | bcsaller, 'cause if so, then we could enforce #1, and say "to add position information, go to jujucharms.com, import, and the export". But yeah, that's not good enough for the multiple bundle story :-/ | 20:32 |
| gary_poster | that is... | 20:32 |
| bcsaller | gary_poster: but that isn't really how the store will expose them anyway, right? | 20:32 |
| gary_poster | even if we build the UI to choose which bundle you want... | 20:32 |
| bcsaller | they index and return as singles? | 20:32 |
| gary_poster | bcsaller, yes, but the point of aggregation was to make it easier to develop deployer files with multiple bundles--that is, to support the inheritance story in order to encourage better maintainability of similar bundles | 20:33 |
| gary_poster | if you inherit | 20:34 |
| gary_poster | then the import/export story is broken | 20:34 |
| gary_poster | unless we allow you to import, choose bundles one at a time and position, and then export together | 20:34 |
| bcsaller | our fake importer should support it, but yeah, still no ux | 20:34 |
| gary_poster | well, no | 20:35 |
| bcsaller | and for export, it only does a single | 20:35 |
| gary_poster | bcsaller, I'm not expressing myself well. I blame my sleep condition. :-) Can we try a hangout very quickly? | 20:35 |
| bcsaller | sure | 20:35 |
| hatch | :/ I would really like to figure out what's up with the CI | 20:40 |
| === _mup__ is now known as _mup_ | ||
| gary_poster | hey benji you around? | 21:02 |
| gary_poster | bcsaller, Makyo, hatch I am not going to make it to tonight's Australian call. I will update calendar. Sorry about that. | 21:03 |
| Makyo | Alright. | 21:03 |
| Makyo | Feel better! | 21:03 |
| gary_poster | Thank you Makyo | 21:03 |
| benji | gary_poster: yep | 21:04 |
| gary_poster | benji, yay. Could you do the following quickly? (1) go to jujucharms.com (or comingsoon) (2) drag the file of your official bundle into comingsoon | 21:05 |
| gary_poster | (3) position the services | 21:05 |
| gary_poster | (4) export it again | 21:05 |
| gary_poster | (5) push to your branch | 21:05 |
| gary_poster | the idea is that then you will have position information for your services | 21:06 |
| gary_poster | which will make rendering the bundle much better | 21:06 |
| gary_poster | *and...* | 21:06 |
| gary_poster | jujugui, esp people working on bundle bits, you might be interested in my comment #3 of https://bugs.launchpad.net/charm-tools/+bug/1222833 | 21:06 |
| _mup_ | Bug #1222833: charm tools should proof bundles as well <Juju Charm Tools:New> <https://launchpad.net/bugs/1222833> | 21:06 |
| benji | gary_poster: well, the flag doesn't seem to "take" on jujucharms.com, but I can do it locally (I wonder if that is a bug, maybe something to do with the rewrites) | 21:07 |
| gary_poster | benji, you mean annotations don't carry over | 21:07 |
| gary_poster | if so, another thing we have to fix | 21:08 |
| benji | gary_poster: nope, I mean that my bundle doesn't show up in the search, presumably because the flag is being ignored | 21:09 |
| benji | gary_poster: but, indeed, the annotations are not included in the export I just did | 21:09 |
| gary_poster | benji, pastebin it for me pls? | 21:09 |
| benji | gary_poster: http://paste.ubuntu.com/6189871/ | 21:09 |
| gary_poster | bcsaller, ^^^ | 21:10 |
| benji | (the raw export has some other issues too, but I assume those are known) | 21:11 |
| gary_poster | benji, only issue I saw was js undefined. others? | 21:11 |
| benji | gary_poster: yep, that's the one I was referring to | 21:11 |
| * bcsaller checks, I thought we had tests around that and that file came from an export | 21:11 | |
| bcsaller | hmm, I'll look into this, I don't see the annotations either | 21:13 |
| bcsaller | and I suspect why | 21:13 |
| rick_h_ | hatch: no, the one branch landed and I'm workingin charmworld for a bit | 21:13 |
| rick_h_ | benji: do you nkow how to get access to the jenkins results? Looks like I've got failing CI tests? | 21:13 |
| rick_h_ | benji: https://code.launchpad.net/~rharding/charmworld/bundle-metadata/+merge/188905/comments/433299 | 21:13 |
| hatch | rick_h_: ok do you have anything I can use? Or can I just implement it? | 21:14 |
| rick_h_ | abentley: oh ok, so maybe my failures are results of changes going on then? | 21:14 |
| rick_h_ | hatch: kind of. Where are you headed atm? | 21:14 |
| benji | rick_h_: I have done it but don't remember much about it. The main thing is to set up a VPN into its private world, then those 10.xxx.xxx.xxx URLs will actually go somewhere | 21:15 |
| benji | the VPN setup is documented.... on the wiki? | 21:15 |
| hatch | rick_h_: well if it's not that then I'll continue on with the bundle view stuff | 21:15 |
| rick_h_ | ugh, ok. I'll look for that then. vpn to the QA lab bits? | 21:15 |
| abentley | rick_h_: Yes. I've given up for the day and switch back to not testing "make sysdeps". Just making sure your branch lands. | 21:15 |
| hatch | but would like to keep this moving forward and not clash with bcsaller stuff | 21:15 |
| rick_h_ | abentley: ok, looks like it failed a few times but no helpful info in the note. | 21:16 |
| rick_h_ | hatch: call? | 21:16 |
| hatch | sure | 21:16 |
| * Makyo -> dogwalk | 21:16 | |
| abentley | rick_h_: Sure there was info. One of them points out that you didn't enter a commit message. The rest link to the actual failed builds so that you can see exactly what happened. | 21:16 |
| rick_h_ | abentley: right, thanks for the commit message. | 21:17 |
| rick_h_ | abentley: I mean the failures give the link but I've got to get the QA vpn notes to get that info. I'll work on that. Thanks | 21:17 |
| gary_poster | bcsaller, when I drag a file with the contents from http://pastebin.ubuntu.com/6189887/ onto the GUI sandbox, I get "Deployer Import Unsupported | 21:17 |
| gary_poster | Your environment is too old to support deployer file imports directly. Please consider upgrading to use this feature." | 21:17 |
| abentley | rick_h_: No you don't. It's a public IP on canonistack. | 21:17 |
| gary_poster | am I doing something wrong? | 21:17 |
| rick_h_ | abentley: oh, it must have shut down. It's merged now | 21:18 |
| abentley | rick_h_: Oh, wierd. Sometimes it's using the public ip, sometimes the private. | 21:18 |
| rick_h_ | I swear I was just looking at it | 21:18 |
| abentley | rick_h_: Anyhow, congrats on landing revno 404 | 21:18 |
| rick_h_ | abentley: woot | 21:18 |
| abentley | rick_h_: The public site is http://162.213.35.27:8080/ | 21:19 |
| abentley | rick_h_: I'll have to see what IPs it's dropping in the merge proposals later. | 21:20 |
| bcsaller | ok, so QA issue one, manually doing a wp/mysql pair and moving them and then in the console, app.db.services.get('annotations') gives me two empty objects so that appears broken | 21:20 |
| bcsaller | I'm looking at the import issue now | 21:20 |
| rick_h_ | abentley: ok cool thanks. Glad tests passed, I got nervous with failed CI runs | 21:20 |
| gary_poster | bcsaller, could you make "urgent" cards for what you find and tackle 'em? | 21:21 |
| bcsaller | gary_poster: that file imported for me with trunk | 21:21 |
| gary_poster | bcsaller, you just drag the file on, right? | 21:21 |
| bcsaller | gary_poster: are you using api3 flag? I wasn't | 21:21 |
| gary_poster | bcsaller, did you try comingsoon? I was trying that, and normal jujucharms, without api3 flag | 21:21 |
| gary_poster | will try trunk | 21:22 |
| bcsaller | gary_poster: yes, dnd, import with successful notification | 21:22 |
| gary_poster | k | 21:22 |
| gary_poster | bcsaller, confirmed. it works on trunk locally, but not on comingsoon or jujucharms for me | 21:23 |
| bcsaller | neither the gui db or the fakebackend have position annotations on the service models, something is wrong there, the export code should still pick them up at a glance though | 21:24 |
| bcsaller | wonder how we introduced a bug here w/o seeing it in the tests :( | 21:24 |
| bcsaller | I will add a card and start on it though | 21:24 |
| gary_poster | thanks | 21:31 |
| gary_poster | g'night all | 21:31 |
| gary_poster | hey hatch | 21:35 |
| hatch | ahoy! | 21:36 |
| gary_poster | could you ask huw to look at the styling of the bundle detail pages we have now | 21:36 |
| gary_poster | and change/style your page accordingly? also | 21:36 |
| hatch | sure thing | 21:36 |
| gary_poster | I'm heading out | 21:36 |
| gary_poster | but | 21:36 |
| hatch | yeah no problem | 21:36 |
| gary_poster | thanks! | 21:36 |
| hatch | get better! | 21:37 |
| gary_poster | if you want to play with the bundle vis yourself, and take over, here's what I did | 21:37 |
| gary_poster | lp:~gary/juju-gui/bundle-detail-view-2 | 21:37 |
| gary_poster | it works nominally | 21:37 |
| gary_poster | but we need position annotations for the canonical example we have to not look horrible (with broken relations and charms on top of one another) | 21:37 |
| hatch | cool will look | 21:38 |
| gary_poster | I don't know what to do with it. if you have any ideas, hatch, go for it. test? land? I dunno. :-) | 21:38 |
| gary_poster | thanks! | 21:38 |
| gary_poster | ttyl | 21:38 |
| hatch | cya! | 21:38 |
| === matsubara is now known as matsubara-afk | ||
| hatch | morning huwshimi | 22:28 |
| huwshimi | Morning | 22:28 |
| hatch | so gary isn't feeling well so he cancled our meeting, but would like you to change/style the bundle detail pages which can now be accessed in the application | 22:29 |
| huwshimi | hatch: Sure I can take a look at that. | 22:30 |
| huwshimi | hatch: Is it in trunk but not on comingsoon? | 22:31 |
| hatch | ok so on a fresh trunk you will need to use the url to access them directly so that is.... http://localhost:8888/bundle/~benji/wiki/wiki/:flags:/charmworldv3/ | 22:31 |
| hatch | and you'll need to change to the staging charmworld in your config-debug.js file | 22:32 |
| hatch | as per benji's email "Working bundle for developing visuals" | 22:32 |
| huwshimi | Ah I see | 22:32 |
| hatch | I'll be around so if you have any trouble getting that going I can help | 22:35 |
| huwshimi | hatch: Do you know if the tabs are supposed to work yet? | 22:38 |
| huwshimi | The only thing wired up appears to be the name... | 22:38 |
| hatch | right that's all that is wired up so far | 22:39 |
| hatch | I'll get you the link to the mockups | 22:39 |
| huwshimi | hatch: Oh, well not really anything I can do then yet :) | 22:39 |
| hatch | https://drive.google.com/a/canonical.com/#folders/0B7XG_QBXNwY1d3VVX0FFVEJ3S2s | 22:40 |
| hatch | well you could hook the stuff up :) | 22:40 |
| hatch | it's all changes in the markup | 22:40 |
| hatch | the data is being passed to the template | 22:40 |
| hatch | you can see it in subapps/browser/views/bundle.js | 22:40 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!