608033 - Random failures in the build system trying to remove the repo directory

Status: VERIFIED FIXED

(Tamarin Graveyard :: Build Config, defect)

Description

[Brent Baker]

Attachment: usePTY="False" when calling sync_Clean

There have been a couple of random failures in the build system happening when trying to remove the "repo" directory. Logging into the machine after the failure shows that the directory was in fact deleted and appears that there should not have been any issue.

logfile:
2010-10-28 12:25:01-0400 [Broker,client]  startCommand:shell [id 18509]
2010-10-28 12:25:01-0400 [Broker,client] ShellCommand._startCommand
2010-10-28 12:25:01-0400 [Broker,client]  rm -Rf repo
2010-10-28 12:25:01-0400 [Broker,client]   in dir /home/build/buildbot/tamarin-redux/linux/./sandbox-linux2-test/../ (timeout 1200 secs)
2010-10-28 12:25:01-0400 [Broker,client]   watching logfiles {}
2010-10-28 12:25:01-0400 [Broker,client]   argv: ['rm', '-Rf', 'repo']
2010-10-28 12:25:01-0400 [Broker,client]   closing stdin
2010-10-28 12:25:01-0400 [Broker,client]   using PTY: True
2010-10-28 12:25:02-0400 [-] command finished with signal 1, exit code None, elapsedTime: 1.448545


Doing a little searching it appears that this has been a known issue that can happen in buildbot and was tracked via 2 separate bugs [1][2]

Since the version of buildbot that we are currently using does not have this patch, and the fact that we use our own "clean" step, we just need to create the build step with usePTY="False".

[1] http://buildbot.net/trac/ticket/255
[2] http://buildbot.net/trac/ticket/284

Comment 1

[Brent Baker]

Attachment: v2. usePTY="False" when calling sync_Clean

add the same check to the bb_slaveupdate build step that just does a quick 'cp' call, which is also mentioned as a possible problem in the comments of the buildbot trac issue (found while reading entire thread)

Comment 2

[Brent Baker]

Comment on attachment 486665 [details] [diff] [review]
v2. usePTY="False" when calling sync_Clean

Patch pushed and deployed as 5462:5a7d5f01296a

Comment 3

[Brent Baker]

final patch pushed as 5475:33edebac2097

Comment 4

[Brent Baker]

Marking as verified, the build system has been running for 5 days without the same issue.