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

bacfrankban: were you able to grab my branch?12:14
frankbanbac: yes12:15
bacfrankban: good.  i hope it is helpful.12:15
bacsee you tomorrow12:16
frankbanbac: it is, thank you, have a good day12:16
* frankban lunches12:17
gary_posterteknico, re-reviewed just for the heck of it.  Looks good.  I had an important thought for an upcoming branch.  Are tests running now?  Do you just need a second review?13:26
* gary_poster cheers bcsaller1 13:52
bcsaller1gary_poster: :) There may still be some issues but its a real improvement and Makyo already has a start on the next branch stacked on that one13:53
* gary_poster also cheers for a GUI branch that does not touch the Makefile :-P13:53
gary_postergreat bcsaller1 13:53
bcsaller1indeed13:53
gary_posterbcsaller1, for kanban board, which one is Makyo working on?  Will drag it into the coding lane13:54
bcsaller1relations13:54
gary_posterok thanks13:55
gary_posterbcsaller1, if you review hazmat's annotations branch then he will have two reviews and can land, and will clear up space in secondary story13:57
bcsaller1gary_poster: I'll do that this morning then13:57
=== bcsaller1 is now known as bcsaller
gary_posterthanks13:57
* teknico is officially back to work :-)14:01
gary_posterfrankban, benji or Makyo if you could give teknico his second review that would help move things along also14:01
gary_posterhey teknico :-)14:02
* benji looks14:02
teknicogary_poster, I did not yet run tests, only had time to make changes and push them this morning14:02
gary_posterack teknico14:02
teknicoI'd like to test that code a little more, at least manually14:02
teknicoprobably a unit test on the "build" function would also be appropriate14:03
teknicobut would delay things more14:03
teknicogary_poster, I'd also like to clear my head about something that puzzles me quite a bit, this: https://code.launchpad.net/~bac/juju-gui/1091787/+merge/14052814:04
gary_posterteknico, do what you think is necessary for this to be acceptable, pushing things to another branch where possible.  If you want to talk through that with someone, me or anyone else, feel free, but I leave that decision to you.14:05
gary_posterteknico, MP: which part?14:06
teknicogary_poster, all of it. Can we have a quick call on it, with bac and possibly you and/or benji?14:06
gary_posterteknico, bac is out today14:06
teknicooh, great :-/14:07
gary_posterteknico, I think bac did it because of some issues that frankban and he encountered.  frankban does that sound right?  anyway teknico, why don't you and I talk it through and then we can figure out if we need more people in on it.  let's see if juju-ui is open14:08
teknicogary_poster, anyway, you approved that branch, so you likely have some knowledge/opinion on it :-)14:08
gary_posteryes14:08
gary_posterI'm in juju-ui teknico 14:08
teknicogary_poster, sure, maybe frankban could have a say too, if bac discussed it with him14:08
gary_posterteknico, I'd like the two of us to talk about it first14:08
gary_posterthen we can bring others in if necessary14:09
goodspudQuick question: if I can download an installer for Juju to my Mac and run it form my desktop... why can't I do this in Ubuntu?14:42
gary_postergoodspud, juju is in software center/debs; would be the equivalent of being in the Mac app store15:20
gary_posterbcsaller, I got sidetracked.  Would a daily call be helpful today, or is the current happy state relatively obvious?15:21
bcsallergary_poster: I think we are doing fine today15:21
gary_postercool bcsaller, thanks.  I'll get back to review...at 11:30.  If you get someone else to beat me to it that's fine15:22
gary_posterbac bcsaller benji frankban goodspud hazmat Makyo teknico  call in 215:28
gary_posterbcsaller, also IMO and in yellow's past, pairing counted as a review.  If we all agree with that then your branch already has a review from Makyo, yeah?15:29
gary_posterbcsaller, starting without you15:31
gary_posterbcsaller, want to tell me the issues on hangout for your speed, or here?15:38
bcsallerback to hangout :)15:39
hazmatbac, so apparently its golang 1.0.3 which has the bug for http client.. sadly that s the latest version.. 1.0.2 should work15:42
frankbangary_poster: to create a stable release without needing to change trunk, it is ok to just modify CHANGES.yaml (replacing "unreleased" with a version number, and then "BRANCH_IS_CLEAN=1 FINAL=1 PROD=1 make dist"?15:52
gary_posterfrankban, I want to be helpful, but I don't like it as a precedent or as part of history (we want to be able to relate a release to a revision).  Maybe there's another approach we could have to make this fast.  Quick hangout in juju-ui?15:54
frankbangary_poster: sure15:54
frankbangary_poster: could you please review another Makefile quick fix? https://codereview.appspot.com/696504816:13
gary_posterfrankban, approved and I'm ok with you just landing16:16
frankbanthanks gary_poster 16:17
hazmatgary_poster, blame canada ;-)16:29
gary_poster:-)16:29
frankbangary_poster: release failed :-( the error seems to be related to BRANCH_IS_CLEAN, which executes "bzr missing --this". the latter raises a "bzr: ERROR: No peer location known or specified" failure.16:40
gary_poster:-( hm16:40
gary_posterfrankban, use BRANCH_IS_CLEAN=1 and I will file a bug16:41
frankbangary_poster: ok 16:41
frankbangary_poster: trying BRANCH_IS_GOOD, because also IS_TRUNK_BRANCH does not seem to work with checkouts16:45
frankban(it tries to grep for the parent branch)16:45
gary_posterfrankban, :-( it worked at one point.  benji ^^ I'll file bugs for this stuff16:46
benjihmm16:46
gary_posterfrankban, I'm guessing when you make a temporary release in the charm you already are saying BRANCH_IS_GOOD?16:46
frankbangary_poster: no, I only use NO_BZR=1. 16:49
benjifrankban: the release machinery is meant to be used with a branch of trunk, not a checkout16:49
gary_posterbenji, oh, that's not the way I had it16:49
frankbangary_poster: and it works...16:49
gary_posterbenji, sorry16:49
gary_posterfrankban, sorry16:49
benjigary_poster: we discussed this and decided branches would be at least as good as a checkout and arguably better 16:50
gary_posterfrankban, if it works then that's all I care about :-)16:50
frankbangary_poster: no. it works in the charm, when I create a local distfile from a lightweight checkout16:51
gary_posterbenji, ok.  Maybe you could connect with frankban after this is done to run though the releae process16:51
gary_posterthat we have documented16:51
gary_posterand figure out what could be documented better16:51
benjisure16:51
gary_posterthanks16:52
frankbangary_poster: ok. proceeding with BRANCH_IS_GOOD?16:52
gary_posterfrankban, yes.  If we have a revision that is distinctly associated with a release, I'm happy for now.16:53
gary_posterteknico, I filed low priority bug 1092199 and bug 1092204 per our discussions16:54
_mup_Bug #1092199: Makefile build target is misnamed <juju-gui:Triaged> < https://launchpad.net/bugs/1092199 >16:54
_mup_Bug #1092204: We copy YUI assets rather than symlinking them <juju-gui:Triaged> < https://launchpad.net/bugs/1092204 >16:54
teknicogary_poster, great, thanks!16:54
gary_posterwelcome, thank you16:54
gary_posterlp2kanban appears to be dead17:00
gary_posterI think bac is the one with the keys17:00
gary_posterbcsaller, I filed bug 1092208, and put both it and Nick's bug 1091616 in the secondary story tasks17:01
_mup_Bug #1092208: Pending relation line is not getting events <juju-gui:Triaged> < https://launchpad.net/bugs/1092208 >17:01
_mup_Bug #1091616: Unusual drag/zoom behaviour <regression> <juju-gui:Triaged> < https://launchpad.net/bugs/1091616 >17:01
bcsallergary_poster: thanks17:01
gary_posterwelcome17:02
frankbangary_poster: done! https://launchpad.net/juju-gui/stable17:03
gary_posterbenji, "Checklist for Making a Release" in trunk begins with  Get a checkout of the trunk:: ``bzr co lp:juju-gui``. :-) I am making a bug to update this, and will ask you to do it when you get a chance next17:03
gary_posteryay frankban! L0(17:03
gary_poster:-)17:03
gary_posterone-off smiley17:03
frankban:-)17:04
gary_posterbenji, filed bug 1092216.  I asked whoever tackles it to consult with you; alternatively you can clarify the process in the bug; alternatively you can consult with yourself :-)17:08
_mup_Bug #1092216: Release documentation is incorrect <juju-gui:Triaged> < https://launchpad.net/bugs/1092216 >17:08
gary_posteroh lp2kanban, where are thou17:09
benjiheh17:09
* gary_poster completes filing bugs and goes to have a bit of lunch before reviews17:10
teknicogary_poster, quick questio, how long did it take from deploy/expose commands to finish?17:12
teknicoI can't deploy a working environment, for some reason :-/17:12
gary_posterteknico, about 20 minutes17:29
gary_posterteknico, what does juju status say?17:29
teknicogary_poster, I need to wait a little more then17:30
teknicogary_poster, always this: http://pastebin.ubuntu.com/1450398/17:30
gary_posterteknico, it's been less than 20 minutes?  If not, I would ssh  to the machine and do a top or ps and see what's going on17:31
gary_posterlook at the log too of course17:31
gary_posterwhich might show that it is doeing something in the Makefile, which is another file you can tail17:32
gary_posterdoing17:32
teknicogary_poster, using juju-ssh juju-gui/0 , as you mentioned?17:32
gary_posterteknico ``juju ssh 1``17:32
gary_posterit's the machine number.  maybe unit works as well17:32
teknicook17:32
* gary_poster steps back to lunch break, trying to fix disposal :-)17:33
gary_posteryay frankban :-)18:09
gary_posterteknico, if you are heading to your EoD soon and you are still seeing problems with your branch, consider trying to pass it off to me to see if I can move it along any18:10
gary_posterI'm willing, and might have some time18:10
teknicogary_poster, that's probably a good idea18:11
teknicogary_poster, 40m now, not yet started, and "juju ssh 1" does not work either :-(18:11
teknicomaybe something wrong in my environments.yaml file?18:11
frankbangary_poster: heh, the diff is a bit long, I am sorry, but there are a lot of tests18:11
frankbanat least18:12
gary_posterteknico, weird :-/ if you want to have a call let me know18:12
teknicogary_poster, I need to fix this problem in short order, but you're right, no need to hold this up any more18:12
gary_posterfrankban, yeah, I think you are forgiven for the diff length ;-)18:12
teknicogary_poster, so consider the branch passed off to you :-)18:12
teknicogary_poster, I added the docs and fixed the other "build" invocation as discussed18:13
frankban:-)18:13
teknicogary_poster, yes, if you have time maybe a (hopefully short :-) ) call could help18:17
gary_posterteknico, join me in juju-ui whenever you are ready18:27
gary_posterbcsaller, for some reason every one of the rietveld pages for your branch shows "error: old chunk mismatch."  Example: https://codereview.appspot.com/6971045/diff/1/app/app.js18:39
gary_posterAny idea why?  Makes it difficult to review :-)18:39
hazmatgary_poster, i suspect it needs a merge from trunk.. i noticed that as well18:39
hazmatgary_poster, so its kinda of wierd18:40
gary_posterhazmat, ah ok.  bcsaller could you try and fix?18:40
hazmatgary_poster, you can got at it from the backside and it works..18:40
gary_posteror redo or something18:40
gary_posterhazmat, backside do you mean MP or bzr branch or someth?18:40
hazmatgary_poster, i think it might an issue in reitveld18:40
gary_posterah ok18:41
hazmatif you start from the index in the middle its okay https://codereview.appspot.com/6971045/patch/1/618:41
hazmatbut if you start at the top and use j/k to navigate they all show that error18:41
gary_posteroh, it is the side-by-side that is broken18:41
hazmatgary_poster, yeah18:41
gary_posterstill not so good for reviews :-P18:42
teknicohazmat, fyi, the eu-west-1 region does not seem to work with juju, at least for the juju-gui charm18:52
teknicoand now I'll go18:52
gary_posterhey jimbaker.  I asked jitsu test for verbose logging with -v and it's not been as verbose as I hoped for. This is all I've got, almost 40 minutes into the test: http://pastebin.ubuntu.com/1450714/20:03
gary_posterDid I do something wrong, or could I get better logs, or is this a place where jitsu test could do a better job in the future20:03
benjihas anyone else seen this?  I am running my tests by requesting "http://localhost:8084/test/?grep=login%20mechanism".  It only works intermittently.  Most of the time I get a completely blank page.20:14
gary_posterI have not benji, but I haven't used grep in a while.  bcsaller or Makyo would be people to ask20:14
gary_posterbenji, also a problem in trunk?20:15
benjiI haven't tried on trunk.20:15
gary_posterjimbaker, also, it looks like we don't have stdout/stdin from the test so we can't run debuggers, like pdb, which is a drag.  I think the debugging story right now is the most difficult20:51
gary_postergiven the logging and break point issues20:51
gary_posterI need to run.  I'm still trying to figure out the charm test situation with nicola's branch.  I hope frankban has some good tips21:11

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