[12:33] morning party people [13:28] * rick_h_ runs off to dentist, back in a bit [14:01] wow, atlassian is an aggressive web stalker... [14:18] bac they really want you to use Jira [14:21] i wonder if benji will make it back today. [14:23] hey marcoceppi it'd be great if you could look at my charmworldlib proposal. https://codereview.appspot.com/69430043 [14:23] bac: hey, working thorugh the queue now, will have eyes on today [14:23] marcoceppi: thx [14:24] bac: oh, right, let me look at this now, i think this is something you guys wanted sooner rather than later [14:24] marcoceppi: sooner is better [14:24] bac: how do you guys use charmworldlib? deb? pip? trunk? [14:25] marcoceppi: we bundle a tgz downloaded from pypi [14:25] bac: cool, so you'll need this reviewed merged and released then? [14:26] marcoceppi: yes, that would be great. we *can* make our own tgz for the interim but would prefer not to [14:29] bac: that's fine, I can roll another release [14:57] bac: I don't understand removing ez_setup.py [14:58] oh I love it when tests pass in isolation and fail when running in the suite [14:58] marcoceppi: the ez_setup.py file was not being distributed by the package. so to install charmworldlib, the target machine would have to already have it somewhere or setup.py would fail. [14:59] bac: ah, thanks [14:59] bac: this lgtm, going to merge and release [14:59] marcoceppi: i then noticed since cwl relies on python-virtualenv being installed, then python-distribute is guaranteed to be there, since it is a dependency. so, just relying on py-distribute is all that is needed. [14:59] marcoceppi: cool, thanks [15:00] bac: the only thing that'd I'd comment on is the version bump [15:00] since it won't have the -1 in the 1.2.5 release [15:00] ok, so just 0.2.5 [15:01] er, 1.2.5? [15:01] err, yeah 0.2.5, but I'm not sure if merges are supposed to bump versions [15:01] was it a bug fix? [15:01] technically, in a semantic world, it' would be a 0.3.0 [15:01] http://semver.org/ [15:02] see the first three rules :) [15:02] the point is, it seems weird that a merge proposal for a feature would bump software version [15:02] but I'm not sure of all the different ways things are done [15:02] marcoceppi: so you wish i'd not changed the version and let the package maintainer do it? [15:02] bac: yes, ideally [15:02] marcoceppi: i can do that in the future [15:02] bac: cool, thanks [15:02] marcoceppi: thought i was helping... :) [15:03] bac: psh, you are, believe me [15:03] marcoceppi: i just wanted to ensure it didn't get merged with the old version number [15:06] bac: can you take a look at https://code.launchpad.net/~jcsackett/charmworld/bad-average/+merge/209272 for me? [15:08] jcsackett: sure [15:10] Makyo you in yet? [15:10] jcsackett: qaed_charms is my least favorite name of the day. [15:13] promulgated is worse [15:13] :P [15:14] hatch: promulgated is bad. [15:15] jcsackett: change looks good. could you whip up a test to show we're now using the data we want? [15:15] bac: oh, i suppose. [15:15] :p [15:15] ping you when that's up. [15:15] okey doke [15:15] thanks [15:23] bac: 0.3.0 is on pypi [15:27] marcoceppi: nice, thanks [15:45] jujugui lf a review/qa https://github.com/juju/juju-gui/pull/157 [15:46] hatch: i'll look [15:50] jujugui call in 10 [15:50] thanks bac [15:50] back from dentist ugh [15:50] Makyo I created a bug this morning re the new relation lines that you may want to comment on [15:50] rick_h_ feelin good? [15:51] "oh this is a small thing we can do it now and no anesthetic needed [15:51] "oh, it's deeper than I thought, are you feeling it yet?" [15:51] lol [15:51] ouch! [15:52] all that pain and then a big bill at the end [15:54] hey guys, is quickstart available for OSX and windows? [15:54] jcastro nope [15:54] not for OSX either? [15:54] osx probably woudln't be a -huge- amount of work [15:54] but windows would be [15:55] jcastro: it's a todo [15:55] is there a bug for OSX support or should I open one? [15:55] it's on the todo list :) [15:55] jcastro: we've got ok from the ramm to make it so [15:55] yeah... what rick said [15:55] :) [15:55] I think a windows port won't be too bad. [15:55] jcastro: and it's one of the reasons I own an air now to help test/qa it [15:55] windows is :P [15:56] awesome, do we have a bug report? [15:57] jcastro: looking, I think it was just something that was in email to get permission to add it [15:57] jcastro: feel free to add a bug report and I'll update it [15:57] benji: how's trusty? you have a home directory? [15:57] ack, I'll make one for each platform then? I just want to track it. [15:57] jcastro: ok [15:57] I submitted all the docs to use quickstart, and didn't think to think of OSX or windows [15:58] bac: trusty seems fine; I managed to find my ssh keys on an old hard drive, so I have that goin' for me [15:58] ssh and a terminal, what more do you need? [15:58] benji: i just write mine down on a postit [15:58] lol [15:58] that's one big postit [15:58] i write small [15:59] jujugui call in 2 [15:59] I seriously need to print it out and put it in a safe deposit box. [15:59] hah, that'll be a tough day "please enter the characters of your ssh key" [15:59] benji: you should rot-13 it so the banker won't impersonate you [15:59] "crap, one of these is bad...but which one?" [16:00] 2fa, one sec [16:00] rick_h_: installing hangout browser plugin [16:14] lll [16:16] jcastro is there a install of discourse somewhere that's getting a ton of traffic? [16:17] I'm curious to see how it works with a large number of posts every hour [16:17] hatch, boingboing is probably your best bet [16:18] cool thanks [16:19] I was just wondering how a user would use it on the scale of something like xda forums for example [16:37] rick_h_ did you qa as well? [16:46] hatch: no, just did a review [16:46] ok, bac are you still going to qa my branch? [16:46] hatch: I was curious how much there was, did you still need a qa? [16:46] yeah qa is the most important to make sure I didn't introduce any obscure issues [16:46] I think I caught them all but not sure... [16:48] k, will look after I get some lunch made up [16:48] no breakfast for dental days makes for hungry techie [16:51] yup that it would [16:51] hmm this converting inspector viewlets to views is a little more challenging [16:53] guihelp: anyone available for reviewing https://github.com/juju/juju-gui/pull/158 ? thanks! [16:56] frankban I can [16:56] hatch: thanks [17:38] hatch: can you verify you can dupe https://bugs.launchpad.net/juju-gui/+bug/1287819 ? [17:38] <_mup_> Bug #1287819: d3 error when deploying a service from the sidebar. [17:38] looking [17:39] rick_h_ was this in my branch or comingsoon? [17:39] hatch: local deploy, on develop [17:39] ok checking comingsoon [17:39] I'm mid branch right now [17:39] hatch: saw it when trying to qa your branch [17:39] hatch: all good thanks [17:40] hmm not on comingsoon....ok trying my branch [17:42] rick_h_ I cannot reproduce [17:42] hatch: hmm, ok. Maybe something in my chrome then thanks [17:42] you're still running on beta? [17:43] hmm, I get it in FF as well [17:43] hatch: yea, devel [17:43] I'll try ff [17:43] [12:43:31.490] Unexpected value translate(undefined,undefined) parsing transform attribute. @ http://gui:8888/juju-ui/assets/javascripts/d3.min.js:4 [17:43] ok I get it in firefox [17:44] ok, added the bug marked it as high. [17:44] thanks hatch [17:45] rick_h_ yeah it happens on comingsoon so it's not from my branch [17:45] doesn't look like it happens on osx chrome [17:45] hatch: gotcha, yea nothing to do with your branch. I got it on develop as well. [17:45] ok cool [18:17] hatch: did you see the CI failure on your branch? [18:17] yeah I re-triggered it [18:18] now I have to wait :) [18:19] or try to run it manually from your machine and see if you can dupe :P [18:20] APP_URL=http://yourip:8888 PORT=8888 JUJU_GUI_TEST_BROWSER="ie" make test_browser-mocha [18:22] ehh I'm lunching [18:22] :) [18:23] :) [18:35] hatch: sorry i got delayed. i see rick already did your review [18:37] yeah he picked up your slack :P [18:38] rick_h_ looks like it was a CI failure, it's running properly now [18:38] hatch: that was a different branch, frencesco's [18:39] :) [18:39] http://ci.jujugui.org:8080/job/juju-gui-merge/160/console [18:39] http://ci.jujugui.org:8080/job/juju-gui-merge/159/ [18:39] it's still running [18:39] his landed [18:39] hasn't runthe IE tests yet [18:40] but yes, I'm watching [18:40] frankban's landed ok. Your's is now running again. this will help tell CI or branch [18:40] ohh I thought IE was first [18:40] I forgot we changed that [18:42] ooo safari is in our ci now [18:42] fancy [18:42] yep [18:43] cool well looks like it was a CI failure [18:43] crazy java [18:43] looks like it's ok and from the exception it looks like some selenium issue java.util.concurrent.locks.AbstractQueuedSynchronizer.doAcquireSharedInterruptibly [18:43] so glad it's all good [18:43] thanks for rebooting your branch [18:43] I either found a bug in the GUI or deployer or bundles [18:43] not sure [18:43] shoot [18:43] jorge@jilldactyl:~/src/bundles/hadoop-cluster$ juju bundle proof [18:43] E: hadoop: The requested relation jobtracker to tasktracker is incompatible between services.Hint: if order of the relation were reversed it would be valid. [18:43] E: hadoop: The requested relation namenode to datanode is incompatible between services.Hint: if order of the relation were reversed it would be valid. [18:43] however, the entire thing deploys and works fine [18:44] jcastro relations are directional? [18:44] lp:~jorge/charms/bundles/hadoop-cluster/bundle is the bundle [18:44] jcastro: the deployer doesn't validate the order but we do. It's legic [18:44] "" if order of the relation were reversed it would be valid"" this doesn't really make any sense [18:44] at least as far as I understand it [18:44] jcastro: they should be reversed to be correct. Relations are directional. We checked on this as we hit it when we added the proof [18:44] yeah I don't get why they are directional [18:45] they are directional? what??????? [18:45] adding a nagios relation between a nagios and the app, which side is which matters [18:45] lol [18:45] bac: was there any docs to link to on this matter from when you added the check? ^ [18:45] oh requires/provides matters you mean [18:45] yeah [18:45] hatch: right [18:46] exactly, one thing requires something that the other provides [18:46] I thought it meant, dragging from one to the other worked haha [18:46] rick_h_: i think benji did that work [18:47] bac: ah sorry [18:47] I was pretty sure the GUI parses the charms properly [18:47] benji: you up and running today? [18:48] * rick_h_ worried benji has been swallowed up by trusty [18:48] http://imgur.com/EMXmUwN [18:48] sorry couldn't help it [18:48] :) [18:48] lol [18:48] they are pretty looking cables [18:48] good thing relations aren't ethernet cables [18:49] or I guess per that pic they are :) [18:49] ok so literally I should just switch them in my text editor and I should be good [18:49] jcastro: +1 [18:49] ohh right in the GUI we just say 'can these relate' whereas proof actually checks that the directions are proper [18:50] the GUI basically half-asses it :P [18:50] ok this isn't as simple though [18:50] in the bundle [18:50] relations: [18:50] - - "hive:jobtracker" [18:50] - "hadoop-master:jobtracker" [18:50] - - "hive:namenode" [18:50] - "hadoop-master:namenode" [18:50] - - "hive:db" [18:51] - "mysql:db" [18:51] - - "hadoop-master:jobtracker" [18:51] - "hadoop-slavecluster:tasktracker" [18:51] - - "hadoop-master:namenode" [18:51] - "hadoop-slavecluster:datanode" [18:51] oh poo [18:51] hmm [18:52] so if you invert the jobtracker and tasktracker relations it doesn't fix it? [18:52] for hadoop-master you mean? [18:52] jcastro: so what's not as simple. I feel like I'm getting half the info. [18:53] E: hadoop [18:53] the error states the service, then the interfaces, and tells you what to fix. Does that not work? [18:53] I don't see where to map what the error says to the bundle [18:54] "The requested relation jobtracker to tasktracker is incompatible between services" 3 things mention jobtracker, which one do I switch? [18:54] hadoop service is related jobtracker to tasktracker [18:54] jcastro: the one in the hadoop service related to the tasktracker [18:54] that's only in the 3-4th from the bottom lines [18:54] you have to pull together all the info [18:55] the first is hive to hadoop relatoing jobtracker to jobtracker [18:55] relating * [18:55] only one tasktracker in here [18:56] so when proof says "reversed", does that mean I put the tasktracker line in front of that one? [18:57] yes [18:57] the 2nd error was easy, that one I fixed [18:57] k [19:14] Makyo: when you get to a break point can I bug you about this bug please? [19:15] Now's good, prepping lunch. [19:16] Makyo: k, invite otw [19:17] https://plus.google.com/hangouts/_/72cpjtb1s2ev44ajjv1g1gmoj8?hl=en Makyo [19:21] jujugui charm review? https://codereview.appspot.com/71230043 [19:21] rick_h_: when you're done with Makyo can we chat for a second? [19:25] bac: sure thing [19:25] bac: https://plus.google.com/hangouts/_/76cpipat9od2quu1p2qqjti45k?hl=en [19:34] rick_h_, if I only want to look for bundles that are official, aka owned by ~charmers, I look at: ? [19:34] https://code.launchpad.net/charms/bundles has all of them [19:34] but lets say I only want to see the "official" ones [19:35] jcastro: when you search for bundles all of the ~charmers ones should show up in the top search result for recommended [19:35] jcastro: since they're promulgated [19:37] https://code.launchpad.net/~charmers/charms/bundles/mediawiki-scalable/bundle [19:37] was pushed 2 hours ago [19:37] so I am assuming you haven't ingested it yet? [19:37] looking [19:38] jcastro: it's ingested. Checking to make sure it's promulgated then. [19:39] yeah I still see Recommended (0) [19:40] yep, the gui isn't setting it. We've got it supported in the back end but not the front. [19:40] jcastro: will get it updated tomorrow [19:41] ok but it's ingested in the store right? aka I can keep submitting? [19:41] jcastro: yep [19:41] jcastro: it's purely a gui issue and a small fix [19:41] cool [19:48] ugh [19:48] swapping around the relations now breaks hadoop-master [19:49] breaks it in what way? Does it not deploy? Give an error? [19:49] tasktracker-relation-changed hook fails [19:57] benji: ping, around? [19:58] rick_h_: yep, running tests even [19:58] rick_h_, well, at least I am passing proof! [19:59] benji: can you help jcastro out with this bundle? he's hitting the issue where the relations required order and it's giving him trouble getting it to work now [19:59] sure [19:59] is there a case where proof might be wrong? [19:59] it is possible [19:59] https://code.launchpad.net/~jorge/charms/bundles/hadoop-cluster/bundle [19:59] jcastro: it's possible. This was a more recent change but I know benji had to do some research and got some confirmation from core on what it's doing. [19:59] ok so, rev 4 works, but fails proof [20:00] rev 5 passes proof, but tasktracker-relation-changed hook fails [20:00] bbiab, daily standup [20:00] jujugui looking for a review for the charm-details viewlet to view conversion https://github.com/juju/juju-gui/pull/159 [20:01] jcastro: let me know when your call is over [20:02] hatch: can look in a bit [20:02] thanks [20:03] I need this one to land because it has some extra fixes [20:04] ok, make sure to call those out please [20:08] rick_h_ reviewer notes added [20:08] hatch: thanks [20:09] heh, thinkpad ran out of battery, now onto the air [20:09] forget a power cord, just grab multiple laptops [20:09] lol [20:09] richy rich [20:09] :P [20:10] just have to buy smaller cheaper laptops :) [20:10] haha [20:10] you spent on yours what I did on my desktop with 32gb of ram [20:10] yeah but I can take mine on a plane :P [20:10] that would be hilarious seeing someone using a desktop on a plane haha [20:11] long powercable running from the overhead bin [20:12] to a car battery in their carry on [20:12] lol [20:12] 'sorry sir you are over your weight limit' [20:21] doh, qa will take a few min hatch. VM on here is way out of date [20:22] curse you air! [20:24] no problemo [20:26] rick_h_: by "way out of date" you mean a day or two ago? [20:27] bac: heh, probably a week at this point on trusty so the big thing is the nodejs update we nee [20:46] jcastro: I think I figured out your problem. In r4 you had [20:47] In r4 you had [20:47] - - "hadoop-master:jobtracker" [20:47] - "hadoop-slavecluster:tasktracker" [20:47] - - "hadoop-master:namenode" [20:47] - "hadoop-slavecluster:datanode" [20:47] In r5 you changed that to [20:47] - - "hadoop-master:tasktracker" [20:47] - "hadoop-slavecluster:jobtracker" [20:47] - - "hadoop-master:datanode" [20:47] - "hadoop-slavecluster:namenode" [20:47] But that just moved the relation names, not the service names. [20:47] Instead you need this: [20:47] - - "hadoop-slavecluster:tasktracker" [20:47] - "hadoop-master:jobtracker" [20:47] - - "hadoop-slavecluster:datanode" [20:47] - "hadoop-master:namenode" [20:51] the bundle relation syntax is so confusing [20:54] thanks benji, hopefully that gets him setup. [20:54] np [20:54] * rick_h_ runs away see you all later. [20:55] now, if I can just figure out why "make devel" is dying for the GUI [20:55] benji: trace? [20:55] benji: you have to have the latest nodejs from the ppa to get all the deps. We hit this issue last week though you should be set if you followed the hacking docs as it says to install that [20:56] yeah, I was copy-pasting from HACKING, so it should hwave worked [20:56] I don't have the error at-hand because I decided to nuke it from orbit and try again [20:56] gotcha [21:09] benji, OHHHHHHHH. [21:09] I get it now [21:09] * jcastro tests [22:21] rick_h_ async stack traces are now available in experimental [22:42] hatch: woot [22:43] hatch: ethernet adapter arrived today so will try to update tonight hopefully :) [22:43] NOICE [22:43] * hatch is hoping for good things [22:43] I'm finding all sorts of bugs doing this conversion heh [22:43] https://bugs.launchpad.net/juju-gui/+bug/1287959 [22:43] <_mup_> Bug #1287959: sidebar doesn't open after viewing charm details twice [23:38] oh man do we ever need to set up pagination on the search results [23:38] 'rabbitmq-server' search locks the gui up [23:39] yea, we might have to do some optimizations after all. Had been hoping to EOL charmworld and not put time into it [23:50] I thought we fixed this https://bugs.launchpad.net/juju-gui/+bug/1287977 [23:50] <_mup_> Bug #1287977: searching for rabbitmq locks the GUI up until all 400 results are loaded and rendered [23:50] we send ALLLLLLLL of the charm data then make more requests for it lol [23:50] rick_h_ is charmworld going Go? [23:51] hatch: not currently, python. Things will be moved to the juju core store in go [23:51] hatch: huh, not sure on that. Will have to look [23:52] I think that this could be fixed by fixing the db query [23:52] it shouldn't be requesting/returning so much information [23:53] that in itself will probably solve 80% of it heh [23:53] charmworld moving into juju-core? [23:54] how's that going to work? Or do you mean it'll be a project in juju-core written in Go but not part of juju-core? [23:54] sorry, cooking dinner at the stove [23:54] we can talk it over during 1-1 tomorrow [23:55] haha np :) yeah sure [23:55] oh crap it's almost 6 [23:55] * hatch is EOD'ing [23:55] have a good night [23:55] night