/srv/irclogs.ubuntu.com/2012/03/06/#launchpad-yellow.txt

frankbanmorning gmb09:14
gmbHi frankban09:14
* gmb -> lunch11:52
frankbangmb: thanks for the review12:58
gmbnp13:01
gary_posterbenji frankban gmb call in 213:08
gmback13:09
benjigary_poster: I'm about to send this to the -tech list: https://pastebin.canonical.com/61706/; feedback?14:17
benjigmb and frankban: have you used the terminal broadcasting hack?  If so (or not, even), do you have any feedback on my future message to the tech list (https://pastebin.canonical.com/61706/) or on the wiki page describing the hack (https://dev.launchpad.net/yellow/RemoteTerminalBroadcasting)?14:21
frankbanbenji: not used, taking a look14:22
* gmb neither14:25
* gmb looking14:25
gmbbenji, Both look good to me. Looking forward to having something to try this with properly :)14:28
gary_posterbenji, sorry, didn't see notification. looking.  Also, I'm looking at testrepository to see feasibility of adding a -v14:36
gary_postermore important is to help you if that is valuable14:36
gary_posterlemme know14:36
benjigary_poster: the slave just started so I'm looking at it now; I'll know more in a few minutes14:36
gary_posterbenji, pastebin looks good.  one possible change: terminal size doesn't need to match.  viewer terminal must be same size or bigger.  Maybe what you said is simpler for an email.  I think it is more nuanced on the web page itself14:38
benjigary_poster: I changed it on the wiki; I /think/ that under some circumstances being exactly the same size is important14:39
gary_posterhuh, ok14:39
frankbanbenji: maybe I am missing something, but, in the first scenario, if 2 users have ssh access to a machine, isn't it easier to just use tmux with a shared socket?14:50
benjifrankban: it may; how do you do that?14:50
gary_posterwhee14:56
gary_posterbenji, am back post-crash-and-update if you were looking for me14:56
benjigary_poster: I wasn't ;)14:57
gary_poster:-) k14:57
frankbanbenji: http://readystate4.com/2011/01/02/sharing-remote-terminal-session-between-two-users-with-tmux/15:01
benjifrankban: thanks!  I'll take a look.15:01
gmbEvery day's a school day on the Launchpad Yellow Squad :)15:01
benjifrankban: that's a little different than what we're doing here, in one good way and one bad way.  Good: both users can control the session.  Bad: both users have to be able to SSH into the host box, which is often an issue (either of security or hassle)15:02
gary_posterheh :-)15:02
gary_posterbut that's your first case though, yeah?15:03
gary_posterthe heh was to school day comment15:03
gary_posterthe first case comment was an aside to benji's reply to frankban15:03
benjigary_poster: nope; using the tmux (or screen) approach means that we'd have to *work* on chinstrap, not just use it as a shared conduit15:03
gary_poster:-)15:03
gary_posterah!15:04
frankbanbenji: ah, I see15:06
gary_posterbenji, I'd expect a lot of comments of the sort "why don't you just use X".  Positioning the tool maybe should come first (i.e., why not screen, why not tmux, and so on).  Just a thought15:07
benjigary_poster: good point; by "positioning" do you mean describing it relative to the other solutions?  I tried to do that vs. hangouts, but I only glanced at screen/tmux.15:08
benjigary_poster: oh, that is what you meant (if only I could read)15:09
gary_posterMaybe "I was interested in finding a screen-equivalent that didn't do X; a tmux-equivalent that didn't do Y; a hangout/skype screen sharing that allowed Z; and a tool that was easy to set up and use.  Unix to the rescue."15:09
benjigary_poster: I like that, but I already wrote it another way and I'm lazy ;) https://pastebin.canonical.com/61714/15:14
benjigary_poster: the bit starting at "We've tried using screen" is new.15:14
gary_posterbenji, second sentence needs editing (probably just delete "The one drawback is" and capitalize "That."  Reading more...15:21
gary_posterI mean15:22
gary_posterdelete "that" too15:22
gary_posterand capitalize "sharing"15:22
gary_posterbenji, "lessoned" -> "lessened"15:23
benjigary_poster: good catch, I wanted to use a contrasting word, so I deleted "The one drawback is that" and added "Unfortunately, "15:23
* benji fixes spelling.15:23
gary_posterI would mention talking about tmux and...what's that canonical thing in the same space that has an asian name meaning foldable temporary wall...15:24
gary_posterin the same area as screen15:24
gary_postersince AIUI the arguments are the same15:25
benjib... something15:25
gary_postergmb, do you remember the name of that canonical I-am-not-tmux thing?  juju uses it too; looking there15:26
benjibyobu15:26
gary_posterthat's it15:26
gary_postersorry gmb nm :-)15:26
gary_posterotherwise looks good benji15:26
gary_postergmb, I know you have gone down this maze of twisty passages before: following code path through subunit + testtools + testrepository = think about doing something else instead :-)15:27
gmbgary_poster, benji: Sorry, was getting a cup of tea. Can I assume that I can just raise an eyebrow at this conversation and move on, or is there still a question dangling...?15:31
gary_postergmb, you can move on, but the very last comment was directly addressed to you and had nothing to do with the foregoing...and is not particularly important15:32
gmbOh, er. Right.15:32
benjiheh15:33
gmbgary_poster, And yes, it's a knotty and horrible place to be.15:33
gary_poster:-)15:33
gmbOftentimes the only way to navigate out of it is to ask one of {lifeless,jml} for a map.15:33
gary_posterah15:33
gary_posterok15:33
gmblifeless, of course, is like the guy at a crossroads who gives you really long-winded directions and then, when he's finished, goes back and corrects himself on one of the more esoteric but fundamentally important things that you've already forgotten he said.15:34
gary_posterlol, yes15:34
gary_posterwell described15:34
gmbNote that I may be jaded about asking him for questions when I'm not capable of physically stopping him.15:34
gmbs/questions/answers/15:34
gary_posterheh15:34
* gary_poster goes to branch subunit...15:35
gmbBest of luck. We'll let your children know you love them if you don't come back.15:36
gary_posterlol thank you15:36
benjilol15:36
benjigary_poster: email to -tech away15:36
gary_postercool benji15:36
benjigary_poster: the tests are very quietly running at http://ec2-107-21-150-52.compute-1.amazonaws.com:8010/builders/lucid_lp/builds/0/steps/shell_8/logs/stdio while I work on the tracking issue report15:37
gary_postercool benji.  yeah, no errors so far...15:37
benjigary_poster: that's an interesting exception coming from the stdlib tempfile module: http://ec2-107-21-150-52.compute-1.amazonaws.com:8010/builders/lucid_lp/builds/0/steps/shell_8/logs/stdio16:05
gary_posterbenji, smells like another overlayfs error on the face of it, yeah?16:05
benjigary_poster: yep, that's what I was thinking16:06
benjigary_poster: tracking issue report: https://pastebin.canonical.com/61724/16:09
benjilet me know if there is something else you'd like on it16:09
gary_posterbenji, for the test failure, also looks like pub_config.htaccessroot was not made, and that's the problem that led to the traceback, so we can't simply expect to run "tempfile.mkstemp" to trigger the problem.  tracking issue: excellent thanks.  I have the RT info, as you'd guess.16:24
benjigary_poster: re. failure: hmmm; re. report: cool16:25
frankbangary_poster (or someone else): the MP for shell toolbox is ready: https://code.launchpad.net/~frankban/python-shell-toolbox/helpers/+merge/9618016:52
frankban... and I am sorry for the long diff16:53
gary_posterfrankban, interesting, so you opted to put run later down in the file to keep things alphabetized?17:07
frankbangary_poster: yes, i've reordered functions inside the module17:07
gary_postercool17:07
gmb gary_poster, Er, I *think* I just uploaded a new version of charm-tools to my PPA... If this works I can push it to the yellow PPA, too...17:32
gary_postergmb, cool.  Do you want me to check something or other?  Uh, and charm-tools...oh, so you mean, this includes the new binary of python-charm-tools or whatever it is?17:34
gmbgary_poster, Well, AIUI, all we actually needed to do was and a python charm helpers package, which bac already did. My job was to package it up for us to use, which I think I just acheived.17:35
gmb... after spending some time fighting build deps on precise, which was fun.17:36
gary_postergmb, right I forgot bac had gotten pretty far on that17:37
gmbgary_poster, The good news is that once you're system's not being a dork, bzr builddeb is mind-blowingly excellent.17:38
gary_posterso, yeah, gmb, I think we ought to put it in yellow, and then propose it to clint17:38
gmbI must remember to by jamesw a beer.17:38
gary_poster:-)17:38
gmbgary_poster, Okay, let me make sure it's uploaded to mine first... it's in the ether somewhere at the moment.17:38
gmb(i.e. dput says "yay", Launchpad says "No")17:38
gary_posterack17:38
gary_posterfrankban, approved with comments https://code.launchpad.net/~frankban/python-shell-toolbox/helpers/+merge/9618017:45
frankbanthank you gary_poster17:45
gary_posterthank you!17:46
gary_posterbenji, sabdfl is following you :-)17:48
gmbArgh, silly man. Trying to upload a binary to a PPA. Nevergunnahappen.17:53
* gmb rebuilds with -S17:53
gmb\0/ accepted.17:54
gmbgary_poster, I've copied the package to the Yellow PPA.17:55
benjigary_poster: heh, it seems so17:56
gary_postergmb, yay!  So...if we need to do this again, is the knowledge on bac's page, or in your brain, or some combination?17:56
gmbgary_poster, I'm going to update bac's page.17:56
gary_posterawesome, thanks gmb17:56
gmbhow do you strikethrough an entire wiki page again? ;)17:57
gmbAh, although builddeb mightn't help us with the no-such-package-exists-yet case, actually.17:59
gary_postergmb, are you going to have the MP for the change to charm tools today?  If so, I'd love to have a URL18:02
gary_posterIf not, np, but I won't wait for it.18:02
gmbgary_poster, I will, yes.18:02
gary_postergreat, thank you.  Lemme know and I will include it in our review response to ~charmers18:03
gary_posterbenji, we have changes for the master and slave charm, right?  If you can make MPs for them, that would be good to get them landed before I ask the ~charmers to approve our charm.18:06
gary_posterOr ask me to make MPs, pointing me to the branches18:06
benjigary_poster: I can do MPs momentarily.18:06
gary_posterthank you18:06
gmbgary_poster, https://code.launchpad.net/~gmb/charm-tools/add-charm-helpers/+merge/9620418:10
gary_posterthank you gmb, great18:10
gmbnp.18:11
* gmb -> dinner, but will check back later w.r.t build status of the package, etc.18:11
gary_postergmb, could you quickly remove "7+# NOTE TO LAUNCHPAD DEVELOPERS: This is a bootstrapping file from the18:11
gary_poster8+# setuptools project.  It is imported by our setup.py."18:11
gary_poster:-)18:11
gary_posterwell, do it later18:12
gary_poster:-)18:12
gary_posterbenji, do you recall details as to why we rejected the idea of combining the master and slave charm into one?  My memory is that we felt that overlap was not significant, and that the idea of flexibly becoming the master or the slave did not jibe with our need to have a time-consuming preparation of slaves, and...there was something else about how a charm decided to be the mysql master or slave that didn't make s18:14
gary_posterense for us.  Any other recollections from you?18:14
benjigary_poster: the "no overlap at all" but was the biggest, but also our desire to do as little dynamic setup as possible was another.  With the two-in-one approach everything would have been dynamic18:15
gary_posterright, thanks benji18:16
benjigary_poster: it turns out that all of my changes to the slave have already been landed or superseeded, so the trunk is good to go as-is18:46
gary_posterbenji, even to args?18:46
benjigary_poster: as for setuplxc: I might have been a little light on this MP, but I think the diff is small enough for me to get away with it: https://code.launchpad.net/~benji/launchpad/more-integration/+merge/9621118:46
gary_posterbenji, I saw it; will be happy to get to it very soon18:46
benjigary_poster: surprisingly, yes; I think frankban must have done it18:46
gary_posterah ok...there were two changes to args...one was ssh key18:47
gary_posterwhat was the other18:47
benjigary_poster: user (-u buildbot)18:48
benjimaybe I accidentally pushed to the slave trunk and that's how those got there... I don't know (being able to commit to the trunk isnt' good if we want to do reviews)18:48
gary_poster:-)18:49
gary_posterok thanks benji18:49
gary_posterwas there anything for the master?18:49
gary_posterI thought there was18:50
* gary_poster checks notes18:50
benjinope18:50
gary_posterNo18:50
gary_posterRight ok18:50
gary_posterI'll go look at setuplxc18:50
gary_posterbtw benji, thoughts on https://docs.google.com/a/canonical.com/document/d/1UuW2T7Vhmp4zqa5Rz31TErg46EnO936Yehddp8p10AY/edit welcome18:50
benjigary_poster: I've already corrected a typo and am almost done reading it18:51
gary_posterCool thank you :-)18:51
benjigary_poster: bikeshed: "DONE?" sounds weak, I'd say either "DONE" or remove it alltoghether so it's just "BLOCKED ON..."18:52
benjigary_poster: looks good18:53
gary_posterbenji fair enough, changed.  The deployment plan might actually be done; not sure if we need more approval than Robert.  Since I don't know that, I guess it is not done either way :-)18:54
benji:)18:54
* benji works on bash completion for juju for a few more minutes before delving back into the tests-on-slave issues.18:58
gary_posterbenji, ow: http://ec2-107-21-150-52.compute-1.amazonaws.com:8010/builders/lucid_lp/builds/0/steps/shell_8/logs/stdio18:59
benjigary_poster: yeah, I'm not sure what's going on there; I /think/ there was a timeout or other ssh failure between the host and container that caused at least a restart, but it may have caused some of that flood18:59
gary_posterhuh19:00
gary_postermaybe so19:00
gary_posterbenji, setuplxc approved with trivial comment19:21
gary_posterhttps://code.launchpad.net/~benji/launchpad/more-integration/+merge/9621119:21
benjigary_poster: I now know as much about bash completion as I do about zsh (which isn't saying much) :)19:36
benjigary_poster: if you're looking for something to pair on, I'm going to start looking at those test failures now19:36
gary_posterbenji, cool.  I'll be available in 12 or so19:48
benjigary_poster: k19:48
gary_posterbenji, I'm in goldenhorde, and for some reason my camera is working again.  <shrug>20:13
gary_posterbenji, what we did with the sleep is in the wrong place21:12
gary_posterit needs to be on the ssh side21:12
benjigary_poster: ah! you're right21:12
gary_posterbenji flacoste suggests protocol keep alive21:13
gary_posteran ssh setting21:13
benjigary_poster: that rings a bell... let me look21:14
gary_posterbenji ProtocolKeepAlives 24021:29
benjigary_poster: I'm not sure that does what we want; that's more about keeping an interviening party from cutting the connection (like some firewalls do)21:30
gary_posterbenji, we can move the xvfb card to landing, right?  it will be done when that branch I reviewed lands right?21:31
benjigary_poster: right, we believe xvfb is happy now21:31
benjido we have a card for "make all the tests pass"?21:31
gary_posterbenji, no that's the final stage21:31
benjigary_poster: broken pipe error with ProtocolKeepAlives (actually, ServerAliveInterval which is it's real name, ProtocolKeepAlives is an alias for what was once a Debian-only feature)21:53
gary_posterbenji, :-/22:15

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