/srv/irclogs.ubuntu.com/2012/12/18/#juju-gui.txt

hazmatbcsaller, for tomorrow.. could use a review on the annotation support for persistent positions https://codereview.appspot.com/6936063/05:21
bachi frankban12:48
frankbanmorning bac12:53
bacmorning frankban.  can we have a quick chat to discuss your branch and getting mine out of your way?12:54
frankbanbac: sure12:56
bacjuju-ui then12:56
frankbanfrankban: joining12:56
frankbanbac: bzr+ssh://bazaar.launchpad.net/~frankban/charms/precise/juju-gui/bug-1088618-serve-releases/13:03
frankbanbac: https://code.launchpad.net/~frankban/charms/precise/juju-gui/bug-1088618-serve-releases13:04
gary_posterfrankban, bac, I hope I'm not making unnecessary problems.  Happy to do whatever you think might help13:05
bacgary_poster: join G+ juju-ui13:06
bacplease13:06
gary_poster:-)13:06
gary_posterhey benji.  I wanted to make sure we were on the same page for your new card.  you have a second for juju-ui hangout?13:23
benjigary_poster: sure, let me plug up my camera13:23
gary_postercool thanks13:23
gary_postergoodspud, hey.  any timeline for getting the designer assets (hopefully largely CSS) for the login page?  We don't have to have it this week, but it would be convenient if we had it today or tomorrow morning at the latest.13:36
gary_posterhazmat, just to let you know so you are not surprised: you made a "read-only mode" card in the primary story tasks.  that card/bug already exists in the secondary story.  it is in the secondary story because the refactoring is supposed to assist in that card, and so it is one of the user-facing improvements I associated with the refactoring.  shared position is also down in the secondary story for the same logic.13:40
goodspudgary_poster - quite conveniently I've just been handed them13:40
gary_posterTherefore I deleted the primary lane's version13:40
gary_postergoodspud, yay :-)13:40
goodspudgary_poster, just in the process of uploading.... please hold caller13:41
gary_poster:-) goodspud, benji is working on this. As a heads up, I encouraged him to divide this up into a first cut effort that incorporated the functionality plus the low-hanging fruit of the design in one pass, and then a full-on design match in a second pass.  (Someone else might even do the second pass.)13:43
goodspudgary_poster, benji - all the assets and guides are here: https://drive.google.com/a/canonical.com/#folders/0B1IM--9A1RkTd21pN2RxUHU3WEU13:47
goodspudI'm about to leave the office for the afternoon but buzz Alejandra if you need any help. Greg (designer) isn't set up on IRC or email yet so best to go through her13:48
gary_posterack goodspud & thanks.  good luck at the dr13:49
benjigoodspud: thanks13:49
hazmatgary_poster, cool, thanks13:50
hazmatthe kanban search is kind of cool13:51
hazmatthe pan to card effect13:51
gary_posteryeah13:51
gary_posterhttps://docs.google.com/a/canonical.com/file/d/0B1IM--9A1RkTRmw5NVBxeUgwZzA/edit looks liek an old-style macbook pro trackpad, upsidedown.13:52
hazmatif anyone is up for a pyjuju review... this is the annotation support for persistent positions and landscape..  https://codereview.appspot.com/6936063/13:52
gary_posterI'll look at it, at least for a non-pyjuju-reviewer review13:53
hazmatgary_poster, on the subject of kanban org.. in the primary story we have (proto, code, & review) in the secondary (proto, code, review code).. are those meant to be the same?13:53
gary_posterhazmat, yeah13:54
gary_posterchanged hazmat, thanks13:54
hazmatnp13:55
hazmati'm going to be out for our daily today to attend an omnibus review, for my next gui task i'm looking at working on read-only support 13:56
gary_posterhazmat in GUI or in rapi?  That's only in GUI, do I remember correctly?13:56
gary_posterIn GUI, the read-only support would conflict with the refactoring work; that will either be largely done by Jan 3 (per blog post definitions of this week's goals) or pushed aside for later if we have time in the future13:58
hazmatgary_poster, it shouldn't conflict13:58
gary_posteryou would only be doing env changes?13:58
hazmatgary_poster, i'll be adding another store impl13:58
hazmats/store/env13:58
gary_posterright, ok cool13:58
gary_posterthanks13:58
bacgary_poster: i've updated my branch and just tested it.  the new nginx upstart config seems to have fixed the issue i talked about.13:59
gary_posteryay, bac!  Want an official approve from me?13:59
hazmatgary_poster, re rapi support, long term vision would be the cli uses rapi too, so read only support there would still be advisory based on the gui13:59
hazmatswitching the conn to read only mode13:59
baci changed the juju-gui-branch out from under it and also switch from production to staging.  all worked.  this isn't to be confused with thorough testing, of course.14:00
bacgary_poster: yes but waiting for a second from frankban14:00
hazmatbac, cool, real config support!14:00
baci think "real" needs scare quotes.  it is a good start.14:00
gary_posteroh, hazmat, I had a question about config-changed.  I saw confirmation from you somewher on the web that config-changed is called after install and before start14:05
gary_posterif that's the case14:06
gary_posterthen in the common case install can be very very small, I would think14:06
gary_posterbecause there's no need to dupe work in install and config14:06
gary_posterMore controversially,14:06
gary_posterit also can replace start in some cases14:07
gary_posterbecause that's in fact the simple way to implement it if config-changed has to restart14:07
gary_postera more sophisticated config-changed in that case woudl notice that nothing had been started yet14:07
gary_posterand not restart14:07
gary_posterbut the super simple implementation is to just start/restart always14:08
gary_posterwhich means that in that case start is redendant14:08
gary_posterredundant14:08
gary_posterhazmat, could you confirm or correct the above?14:08
hazmatgary_poster, confirmed config-changed called b4 start14:13
hazmathook logic ordering is whatever make sense to the charm14:13
hazmatmany charms just have a single file symlinked to hook names so they can share logic easily14:14
gary_posterso what I said makes sense to you, but is not the only way to look at it, is what you are saying?14:15
frankbanbac: is your branch ready for review?14:20
benjihazmat: ping; I am trying to get the latest version of rapi-rollup to send "state": "login-required" and I am not having any luck14:26
hazmatbenji, poking14:29
hazmatbenji, using improv?14:29
hazmatbenji, atm login isn't required14:31
gary_posterhazmat, out of curiosity, if we were doing it within the charm, how would you turn login on?14:31
gary_poster/usr/bin/python -m juju.agents.api --help shows --secure14:31
hazmatsince it would break all the extant without gui support, what's there is to support implementing the gui bits14:31
hazmatgary_poster, there will be a separate branch to require login for all rapi access14:32
hazmatgary_poster, secure is for tls14:32
gary_posterhazmat I thought we had a switch to turn on the login so we could test?14:32
hazmatgary_poster, you can still login now14:33
gary_posterhazmat, oh, but there is no way to turn on a challenge as in line 7 of http://paste.ubuntu.com/1397723/14:33
gary_poster?14:33
gary_posterI was asking benji to make the GUI able to handle either the challenge14:33
gary_posteror not the challenge14:34
gary_posterwhich ought to be no problem14:34
hazmatgary_poster, no... but the gui also needs to keep track of a user state14:34
gary_posterAFAIK :-)14:34
hazmatmeeting..14:34
gary_posterok14:34
gary_posterbenji, I'm not sure id this is what hazmat means by user state, but one other aspect of this is that we want to stash the authentication credentials in memory14:35
benjiok, I can fake it for the time being14:35
gary_posterthat way, if the socket dies14:35
gary_posterwe can reconnect14:35
benjimakes sense14:35
gary_poster(but if the user refreshes the browser window, they have to reauthenticate)14:35
hazmatgary_poster, auth creds in browser is what i meant by user state14:36
hazmatgary_poster, session storage14:36
hazmator not...14:36
gary_posterhazmat, we said session storage then you changed your mind in my memory14:36
gary_posterbrowser in memory makes sense to me fwiw14:37
hazmatgary_poster, yeah.. i recall as well. thinking now it would be nice to give  a voice to ux.. unless security is paramount14:37
gary_posterhazmat, I think we can have rapi have a switch turn on login challenge.  let's talk when you are out of meeting14:37
gary_posteragree on ux14:37
gary_posterthough a reasonable and easy first cut is browser in-memory14:38
hazmatgary_poster, i'd rather avoid a switch for something not meant to be configurable...14:38
hazmatbut let's talk  in 1hr14:38
gary_posterhazmat, right it would be for the transition period14:38
gary_posterok cool14:38
gary_posterOIC14:39
gary_posterSo benji, I guess hazmat is saying that you can just always show the challenge14:39
gary_posterit doesn't do anything really now14:39
gary_posterbut that's the transition story14:39
gary_posterthat's fine too14:40
gary_postermake sense benji?14:40
benjiso we will always prompt for login but only relay it to the server if it asks?14:40
benjiwill we land it that way (without validating that it actually works against the real server)?14:41
gary_posterbenji, no the server accepts the login now is what Kapil is saying14:43
gary_posterbenji so after line 7 of http://paste.ubuntu.com/1397723/14:43
gary_posterI mean14:43
gary_posterline 7 would not have  a challenge14:44
gary_posterbut you would send line 11 anyway14:44
gary_posterexpect lines 15-1714:44
gary_posteror 2114:44
gary_posterbenji, I'm sure you will verify experimentally, but that's my understanding14:44
benjioh, it accepts it but it does not challenge?  so my branch would always prompt for credentials but only send them if... what?14:45
gary_posteryou'd always send them after the greeting14:45
benjior send them and ignore any "what you talkin' about Willis" errors?14:45
gary_posteryou'd treat the greeting response as if it had a challenge--as if it looked like line 714:45
gary_postereverything else (except for the rejection in line 28) should behave as described14:46
gary_posterAIUI14:46
gary_posterIf I'm correct, there should be no "what you talkin' 'bout Willis" errors when you send line 11, benji14:47
gary_posterif there are, we'll need to circle back around with Kapil14:47
gary_posteris that clear?14:47
benjigary_poster: even if the server does not understand credentials?  or are we ignoring non-credential-understanding backends?14:47
gary_posteryes benji14:47
gary_posterwe control that particular horizontal/vertical14:48
gary_posterwe are ignoring non-credential-understanding backends14:48
benjik14:48
* gary_poster withholds backend jokes14:48
benjiheh14:49
gary_posterhey bcsaller, sorry. are you already in another hangout or do you want to join me in our calendar hangout?14:50
teknicogary_poster, I have to leave in ten minutes, I likely won't be back in time for the daily call, only shortly thereafter14:50
gary_posterok teknico.  can you give me status on your branch?14:50
teknicogary_poster, I gathered together the commands and config changes needed, am now tracking bac's branch to put my changes in14:51
gary_posterteknico, oh!  you have to modify charm too, of course. :-/14:52
teknicogary_poster, yep14:52
bacgary_poster: mine is ready to land pending a second review14:53
gary_posterteknico, if bac's branch lands in the next hour or so will you be able to land soon after, or will integration be more involved that that?14:53
gary_posterbac, other than mine?14:53
bacgary_poster: yes.  two required, right14:53
gary_postery14:53
gary_posterbac, so do you know of any more issues lurking in your branch, or is everything hunky dory as far as you know?  You expressed reservations earlier today14:54
bacgary_poster: i think it is good14:55
teknicogary_poster, I hope to be able to propose today, yes14:55
gary_posterteknico have you followed bac's branch closely, enough to give it a fast review before you leave, or should we just wait for frankban's return?14:55
gary_posterteknico, great thanks14:55
teknicogary_poster, yes, I was trying to do that14:55
teknicobac, is there a rietveld proposal, or just lp?14:56
gary_posterteknico, and I am keeping you from it. :-P sorry.  Just lp teknico 14:56
bacjust lp teknico14:56
baclbox blew up on me when authenticating to google14:56
teknicook, thanks14:56
frankbangary_poster, bac: I am here14:57
gary_posterhey frankban, we've got a second review from teknico, thank you.  You can keep on doing good stuff on your own branch :-)14:58
teknicogary_poster, frankban, well, mine is more like a half review, though :-)14:59
teknicoand now I gotta go14:59
gary_posterok teknico see you in a bit14:59
gary_posterbac, I think it's two reviews. ;-)15:02
bacgary_poster: yep.  landing now.15:03
gary_posterthanks bac15:03
gary_posterbcsaller, ping15:03
bcsallergary_poster: hey15:03
gary_posteroh hey, there you are bcsaller.  I was running late, then couldn't connect with you earlier.  Meet briefly in our usual hangout, or are you already somewhere else?15:04
bcsallergary_poster: usual is fine, grabbing the link15:05
gary_postercool15:05
bacgary_poster, hazmat: i just pushed my branch.  unfortunately, since lbox is broken for me atm i had to do it manually and i inadvertantly polluted the revision history by including all of my development revisions.15:08
gary_poster:-/15:09
gary_posterbac, I don't know how to do that surgery, butr hazmat does.  mm, I think you can uncommit bac15:14
gary_posterbac, try that: get a checkout of the trunk15:14
gary_posteruncommit15:14
bacgary_poster: i think i could fix it locally and do an --overwrite15:14
gary_posterbac, if you have a plan then go for it15:14
bacall ears if there is a better way15:15
gary_posterbac, fwiw, I always do manual merges with a checkout.  That always seems to be more obvious to me15:15
gary_posterMaybe just me though15:15
bacgary_poster: yes, it is.  but i was lulled by lbox and then when it failed i messed up15:16
gary_posterbac, cool.  but anyway, it is worth fixing, and it soulds like you have a reasonable plan for doing so, so go for it15:16
gary_posterand let us know when you are done :-)15:16
gary_posterbac note that you may need to fix up staging15:16
gary_posteruistage15:17
gary_postermanually15:17
bacok15:17
bacgary_poster: my plan won't work as i'd merged from trunk last night and those revisions that it pulled in are not easily recovered as discrete checkins15:22
gary_posterbac my plan was this:15:22
gary_posterget checkout15:22
gary_posteruncommit until we are where we were before you pushed15:22
gary_postermerge your branch15:23
gary_posterwould that work?15:23
bacno, #2 is the problem since one of my revisions is a merge from trunk.  it collapsed several revisions15:23
bacyou can see it here: http://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/changes15:24
bacat r 2015:24
gary_posterright15:24
gary_posterum15:24
gary_posterbac, I have trunk as of r15, but lbox messages are not there for r 14 and r1515:26
gary_posterfrankban do you have a pristine charm trunk hanging around on your machine anywhere by chance?15:27
frankbangary_poster: yes15:28
gary_posterbac bcsaller benji frankban hazmat Makyo teknico (not here): call in 215:28
teknicook, I made it :-)15:28
frankbangary_poster: last revision: 15: Gary Poster 2012-12-17 [merge] Move markdown files to .md extension15:28
gary_posterfrankban, yeah that's it, I think.  bac, frankban has pristine charm trunk.  Could he push --overwrite to bzr+ssh://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/ and get us back to a good place?15:29
bacgary_poster: yes, i think so15:29
frankbanbac, gary_poster, trying: bzr push --overwrite bzr+ssh://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/15:30
gary_posterbac, frankban has fixed the world15:31
gary_posterhttp://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/changes15:31
gary_posterbenji, hazmat starting without you15:32
gary_posterhttp://jujucharms.com/~juju-gui/precise/juju-gui15:33
bacbcsaller: where is the lbox auth file?15:35
bcsallerbac: I think its .lpad.auth or similar, checking15:35
bcsallerbac: ~/.lpad_oauth15:35
bacbcsaller: hmm, don't have one of those15:36
benjigary_poster: my apologies for missing the meeting; I really wish my sound worked when not plugged in15:42
gary_posters'ok benji.  if it happens again, try an alarm :-)  16:09
gary_posterhazmat, I did a two stage review of https://codereview.appspot.com/6936063/but am done.  I saw at least one thing you might want to address, but everything I wrote is just for your consideration, since I don't know all the details.16:44
hazmatgary_poster, thanks, thats awesome.. naming is hard :-)16:45
gary_poster:-)16:45
hazmatgary_poster, i went with add because its clearly cumulative.. while set might be taken for an equality/total value16:45
* hazmat heads to reitveld to respond16:46
gary_postermm, I see what you mean :-/16:46
hazmatgary_poster, s/(add|set)/update ?16:50
gary_posterupdate_annotations...hazmat I like it better than add, but I think it falls into the same trap you mentioned for set: it is not clear that the annotations are merged16:50
hazmattrue though update tends to imply a merge to existing more than set16:52
gary_posterhazmat, I think it is an improvement.  like you said, naming is hard16:53
bachi frankban, want to pair in a little while?17:06
frankbanbac, sure17:06
bacfrankban: ok, i'll ping you in a few minutes17:06
frankbangary_poster: it seems that "make distfile" produces a tarball containing the "build" dir but not the "build-prod" and "build-debug" ones (which, AFAICT, are  required by the charm)17:17
bacfrankban: i started a hangout and invited you17:18
frankbanbac: joining17:19
hazmatbac, just noticed your comment from earlier17:43
hazmatre history.. eek17:43
bachazmat: it is cleaned now17:43
hazmatbac, cool17:43
hazmatbac, is that possible without an push --overwrite?17:44
hazmatbac, just ping me when your done w/ frankban .. i've got some idle curiosity about it17:44
hazmatgary_poster, benji did you want to discuss login challenges?17:45
benjihazmat: I think I have a handle on things now.17:50
hazmatbenji, cool17:51
hazmatbenji, i was figuring the app needs a user object if it doesn't its doing login form, and retaining the creds, and using them on the conn, on a reconnect, it can just reauth with those creds on challenge17:52
hazmatthe reauth bit might need another branch after login-required lands in rapi17:53
hazmatalthough possibly hanging off the reconnect logic callback in lib/reconnecting-ws.js17:53
benjiat the moment I don't have a distinct user object, but am retaining the creds so they can be resubmitted17:53
hazmatexplicit seems nicer though17:53
hazmatbenji, sounds good17:53
bachi hazmat18:02
hazmatbac, greetings.. so this was an accidental push instead of merge that augmented the trunk history with your branch commits?18:03
bachazmat: yes, since lbox was broken for me i accidentally did a push from my dev branch.18:04
hazmatbac, what went wrong with lbox?18:04
bachazmat: i've moved a vm to a new computer and here lbox throws exceptions when trying to authenticate with google18:04
bacanyway, had i been working manually i would've merged into a local clean version of trunk and then pushed it.18:05
hazmatbac, it should recover from that.. but you can manually delete the lbox google cache @ ~/.goetveld_codereview.appspot.com18:09
teknicobac, remember that "build" function in the charm "install" hook? I lengthened it a bit :-) (but still no test)18:11
bacteknico: that's ok, frankban has redone it anyway18:11
teknicobac, oh no...18:11
hazmatbac, if it happens again its probably helpful to pastebin the  lbox -debug output18:11
hazmatbac, thanks 18:12
teknicook, more conflicts, get 'em all to me, I can handle them!18:12
gary_posterteknico, don't you need to change exposing 80 to 44318:12
gary_posterteknico, you will win :-)18:12
teknicogary_poster, I'm sure there's broken stuff in there; unfortunately, I cannot seem to run charm tests :-/18:12
frankbanbrace yourself, conflicts are coming...18:13
gary_poster:-)18:13
* gary_poster hangs on to treadmill desk for dear life18:13
frankban:-)18:13
teknicogary_poster, and yes, that's one of them18:14
gary_posterteknico, I had another few small comments from a code review, which I posted.  You should receive them soon if you have not already.  trying the charm itself now.18:17
hazmatteknico, incidentally the rapi tls branches from frankban are merged and support specifying the cert directory18:17
teknicogary_poster, yes, I just pushed a few changes18:18
gary_postercool18:19
teknicohazmat, great, that's useful for the next card18:19
hazmatteknico, if your having issues with the test runner jimbaker might be able to help18:24
jimbakergary_poster, i've been thinking of getting a treadmill desk in the new year18:24
teknicohazmat, there were a few problems in my code, and then it's the usual slowness18:25
gary_posterjimbaker, lately it's been a standing desk, but I generally like it and have gotten a lot of use out of it.  I had an existing treadmill and then added the desk on top of it18:25
teknicoI'm hoping frankban is fixing that, or has already fixed it by now18:25
jimbakerteknico, re test runner (i assume using jitsu test), definitely interested in feedback18:26
teknicoI sure hope all those conflicts won't be for nothing!18:26
gary_posterteknico, you fixing the conditional I pointed out too?18:26
jimbakernote that there's some outstanding work on it, but seems like everything has a current workaround18:26
teknicogary_poster, yes, thanks for that too18:26
jimbakerbefore i push in the next version18:26
gary_postercool teknico 18:26
gary_posterwelcome18:26
jimbakergary_poster, i use the bar in my kitchen as my standing desk18:27
hazmatjimbaker, we've been heavy users of it i think, doing tdd on the gui charm18:27
jimbakerhazmat, excellent to hear that18:27
hazmati suspect some of the issues are just slowness around juju.. it would be nice if the apt_update/upgrade cloud config thing where an option.. because that eats up significant time on an instance startup and adds to test time18:27
jimbakerhazmat, is that a controllable option from juju?18:28
bachi benji, got a second for a quick call regarding making releases?18:28
hazmatjimbaker, no.. its a hard code18:28
jimbakerhazmat, got it18:28
benjibac: sure18:28
jimbakeriirc there was a thread on this18:29
hazmatjimbaker, we originally defaulted to not doing, do to an accidental key mispeling, but spampas fixed it in oct i think, now its pretty arn slow to do things.18:29
teknicook, I'm running tests again, let's see what happens18:29
bacbenji: normal hangout18:29
benjik18:29
hazmatit magnified the hp cloud instance startup time by almost double18:29
jimbakerhazmat, got it18:29
frankbanbac: email sent18:29
bactahnks frankban18:30
gary_posterack jimbaker. :-)  re: speed, a big part of it is building the gui itself, we are pretty sure, so using releases should help a bunch.  re: testing with jitsu test, we have http://bazaar.launchpad.net/~juju-gui/charms/precise/juju-gui/trunk/view/head:/HACKING.md jimbaker, which might be good for other people to follow to get experience with the test command18:30
gary_posterbefore it lands18:30
teknicofrankban, is your card out of the woo... I mean, prototyping now? ;-) Are those conflicts coming? :-P18:31
jimbakergary_poster, btw that version of jitsu test is in the ppa; however, i believe running it that way gets around a problem w/ autotools18:32
gary_posterjimbaker, you mean running it the way we document gets around a problem?18:32
jimbakergary_poster, correct18:32
benjibac: one of us froze 18:33
gary_posterok cool.18:33
frankbanteknico: yes and yes, moving the card to "coding"18:33
jimbakergary_poster, i'm sad to see the 40m timeout !18:33
jimbakerbut purely a function of the env18:34
gary_posterjimbaker, yeah :-/ like I said, hopefully that is largely on our side.  I'm sure it will be slower than we want still, but we think a good chunk of that time is spent over the network getting things our build needs18:35
gary_posterso we are fixing that18:35
jimbakergary_poster, yeah, it's definitely on the extreme side of things. i believe the default timeout for jitsu test is 10m, and this was based on empirical observation18:35
gary_posterteknico: nice to have would be redirect from 80 to 44318:35
jimbakermediawiki on ec218:35
gary_posterteknico, easy nginx?18:36
gary_postereasy on nginx I mean?18:36
gary_posterjimbaker, yeah 10 would be a lot better than 40 :-)18:36
jimbaker:)18:36
gary_posterteknico, starting charm fwiw, to see how it works in practice18:37
teknicogary_poster, yes, we can redirect from 80 to 443 if useful, it's easy18:46
teknicogary_poster, I started it 25 minutes ago, still no dice :-/18:46
gary_posterteknico, let's do the redirect.  I have an install-error.  want to look at it with me?  I have a call in 1018:50
gary_posterteknico, http://pastebin.ubuntu.com/1448193/18:52
gary_postera naughty, naughty s18:52
teknicogary_poster, looking18:52
teknicooh, nice :-/18:53
teknicogary_poster, no, the "s" is intended18:54
bacbenji: man, even having a shelve makes 'make distfile' unhappy18:54
gary_poster?18:54
bacshelf?18:54
benjibac: it may be that the current approach is too paranoid about checking cleanlyness; we may need to dial it back18:54
teknicodrats, it was os.makedirs :-P18:55
gary_posterright18:55
gary_posterchanging locally and retrying18:55
bacyeah, i think having shelves is pretty clean18:55
teknicogary_poster, in what logfile did you get that output?18:56
gary_posterteknico, on unit in /var/lib/juju/units/juju-gui/charm.log, or something like that18:57
gary_posterteknico, /var/lib/juju/units/juju-gui-0/charm.log18:58
gary_postertrying install again, from debug-hooks...18:58
teknicouhm, why do I have no /var/lib/juju/ ?18:58
teknicooh, on unit18:59
gary_posterteknico, did you start on unit?  IOW, juju -hooks18:59
gary_posterright18:59
gary_posterjuju debug-hooks, that is18:59
gary_posterdum da dum.  waiting on install.19:00
gary_posteralsohave call now19:00
teknicopushed the directory creation fix, gotta go now19:06
teknicoI'll add the redirect tomorrow, with the conflict resolutions19:07
=== gary_poster|away is now known as gary_poster
gary_posterbac approved19:46
gary_posterhttps://code.launchpad.net/~bac/juju-gui/1091787/+merge/14052819:46
bachazmat: lbox errors  https://pastebin.canonical.com/80719/ and https://pastebin.canonical.com/80720/19:46
bacgary_poster: thanks.19:46
bacbenji: quick look ^^ ?19:46
* benji looks.19:46
bachazmat: it says auth fails but it is different if i use a clearly bad password19:47
benjibac: my two-factor auth is broken, I will look if you put them somewhere I don't have to log in19:47
* benji goes to ask to get that fixed19:47
bacsorry benji, i meant the MP link that gary posted19:48
benjiah19:48
benjisure19:48
baci hope you can still get to LP19:48
=== gary_poster is now known as gary_poster|away
benjimy browser has stored credentials19:51
=== gary_poster|away is now known as gary_poster
bachazmat: it looks like the same lbox problem y'all discussed here: http://irclogs.ubuntu.com/2012/10/11/%23juju-dev.html19:53
bacgary_poster: what version of the lbox package do you have?19:56
bacbenji: is there still a 'make release' target?  i don't see it19:59
bacmaybe it is now 'make dist'19:59
benjibac: right!  it is "make dist"; thanks for realizing my mistake20:00
bacso i need to s/make distfile/make dist/ and then add a description of 'make distfile'20:01
benjiexactly20:02
gary_posterbac, distfile is still valuable20:03
gary_posterdistfile is what frankban will need to make a local, temporary dist in the charm20:04
bacgary_poster: yes.  that is included in what i said20:04
bacyes20:04
gary_posterok sorry20:04
bacbuttinsky!  :)20:04
gary_postertrying to see version of lbox now.  was on call...20:04
gary_poster:-)20:04
baclooks like the bug may have been fixed in precise but not quantal.  are you on Q?20:04
gary_posterbac, I am on lbox 1.0-57.64.39.11~quantal120:05
bachmm, me too20:05
bacweirdness20:05
bacam going to try to build my own20:06
gary_posterk20:06
hazmatbac, as i recall that was an issue with a golang broken http impl.. afaik that's been resolved (though it regressed and was subsequently fixed again)20:17
hazmatbac, if you have a pastebin output of the error that with -v or -debug flags that would be helpful20:18
bachazmat: well, it sure looks the same20:18
bachazmat: i pasted them above20:18
hazmatah20:18
* hazmat backtracks more20:18
bac15:4620:18
hazmatrogpeppe, ping20:19
hazmatdoh.. he's done already20:19
hazmatbac, for your local compile what's the version from $ go version 20:21
bachazmat: haven't gotten a local compile to work yet20:22
baci set GOPATH=/usr/lib/go but that isn't working20:22
bachazmat: but it is go1.0.220:23
hazmatbac, hmm... i show go1.0.320:24
bachazmat: it looks like my golang did not come from the PPA20:26
bachazmat: assuming i get 1.0.3, how do i actually build lbox?20:26
bacmy lbox is from the gophers ppa20:26
hazmatbac, i think the all in one command for it is..20:26
hazmat set gopath to a user writeable directory and then just ->  go get -v launchpad.net/lbox20:26
bachazmat: oh, i downloaded it and tried to use 'make'20:27
* hazmat tries it out as well20:27
bacwhich throws up20:27
hazmatbac, yeah.. the go toolchain has evolved from makefiles to a builtin tool20:27
bachazmat: so the Makefile is just a trap?20:28
* hazmat nods20:28
hazmatbac, for compiling on disk src.. go install 20:28
hazmator go build20:29
hazmatIANAGE20:29
hazmati am not a go expert ;-)20:29
hazmatits got some magic syntax for recursive compile of dependencies .. like go install ... 20:29
hazmattriple dot is recurse to deps20:29
hazmatanyways... the all in one command is much nicer ;-)20:30
hazmatyou have to set GOBIN to a place it can install lbox binary20:30
hazmatyup.. that worked for me... GOBIN=$HOME/bin GOPATH=$HOME/golib  go get launchpad.net/lbox20:31
hazmatthe nice part is that it pull source, so you can modify it.. and then just go install launchpad.net/lbox and it does the recompile install thing20:33
bachazmat: i made a local version and it still fails: https://pastebin.canonical.com/80724/20:35
hazmatbac, yeah.. i think go1.0.2 is the one with the regression around http client20:35
hazmatbac, i'd try the go version from the ppa20:35
bacoh, right, i need to grab 1.03 first20:36
bacrighto20:36
hazmatbac, you might have to blow away the $GOPATH/pkg dir.. its got the link library for the go reitveld lib that's barfiing and needs recompilation20:38
bacyep20:38
bacso if gary has the same lbox from PPA for quantal, i wonder why he isn't seeing this problem?20:39
gary_posterso, the periodic issue we have with d3.v2.min.js is something to do with the symlink not being created properly...20:50
gary_postercharm/juju-gui/app/assets/javascripts/d3.v2.min.js -> /var/lib/juju/units/juju-gui-0/charm/node_modules/d3/d3.v2.min.js20:50
gary_posterThat second path should have 'juju-gui' in it, like /var/lib/juju/units/juju-gui-0/charm/juju-gui/node_modules/d3/d3.v2.min.js20:51
gary_posters/periodic/intermittent/20:51
gary_posterit makes me suspect that the problem is the makefile not being run from the working directory we expect20:52
gary_posterwhich could be addressed pretty simply in the charm20:52
gary_posterthe charm tries to address it...20:57
gary_posterI wonder if -c would work better20:57
gary_postersome tools don't honor the form of cd that we use20:57
bachazmat: got go1.0.3 from golang-stable PPA, rebuilt lbox, same issue.  frustrating.21:14
hazmatbac, argh!21:14
baclands by hand.  stand back.21:14
hazmatbac, sorry, i'll rendevous with some go folks in the AM and try to dig deeper.. afaics their predominantly europe based @ juju-core21:16
gary_posterVoila21:24
gary_poster>>> with shelltoolbox.environ(NO_BZR='1'):21:24
gary_poster...     with shelltoolbox.cd('juju-gui'):21:24
gary_poster...         shelltoolbox.run('make', 'pwd')21:24
gary_poster... 21:24
gary_poster'/var/lib/juju/units/juju-gui-0/charm\n'21:24
gary_posteractually make -C juju-gui isn't any better21:26
gary_posterinteresting21:26
baccool21:26
gary_poster>>> with shelltoolbox.environ(NO_BZR='1'):21:27
gary_poster...     shelltoolbox.run('make', '-C', 'juju-gui', 'pwd')21:27
gary_poster... 21:27
gary_poster"make: Entering directory `/var/lib/juju/units/juju-gui-0/charm/juju-gui'\n/var/lib/juju/units/juju-gui-0/charm\nmake: Leaving directory `/var/lib/juju/units/juju-gui-0/charm/juju-gui'\n"21:27
gary_posterWe want to see /var/lib/juju/units/juju-gui-0/charm/juju-gui21:27
gary_posteroh, wow, weird21:28
gary_poster>>> with shelltoolbox.environ(NO_BZR='1'):21:28
gary_poster...     with shelltoolbox.cd('juju-gui'):21:28
gary_poster...         shelltoolbox.run('pwd')21:28
gary_poster... 21:28
gary_poster'/var/lib/juju/units/juju-gui-0/charm/juju-gui\n'21:28
gary_posterso it's the makefile's use of pwd in a variable that breaks things21:28
gary_posterThis21:29
gary_posterifeq ($(PWD),)21:29
gary_poster        PWD=$(shell pwd)21:29
gary_posterendif21:29
gary_posterah ha21:32
gary_poster$(shell pwd) is always correct21:32
gary_poster$(PWD), if it is set, may be wrong21:32
gary_posterSo simply setting PWD=$(shell pwd) should fix this21:33
gary_posterunconditionally21:34
benjigary_poster: it's not so much that PWD may be wrong, it is that sudo clenses the environment before invoking the subprocess, removing PWD21:41
gary_posterbenji, no21:41
gary_posterthis is a separate issue21:41
benjioh21:42
benjihow is PWD wrong?21:42
gary_posterit shows the PWD that you ran Make in, benji.  It ignores -C tricks, like make -C juju-gui, and it ignores the trick we use in our shelltoolbox to change the current directory21:43
benjiah, yep21:43

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