[Buildbot-devel] contrib/git_buildbot.py reports connection lost handling multiple revisions

[Daniel THOMPSON]

Hi Folks

I have a buildbot monitoring a git based project. It is fairly new and I
am in the process of hooking up the change monitoring automatically
(works great by hand though but I'm getting bored).

We are having problems when multiple revisions are handed to the tool.

In summary if I hand it three lines like the following (except without
shortened SHA1 to fit on a single line in an email):
--- cut here ---
c8c2d694 47e004b5 refs/heads/master
47e004b5 f94e79f6 refs/heads/master
f94e79f6 b9e1dd84 refs/heads/master
--- cut here ---

The changes are collected into a single list and the framework tries to
transmit all three changes in a single session. Unfortunately I then
observe the following error:
--- cut here ---
/usr/lib/python2.6/site-packages/twisted/spread/pb.py:30:
DeprecationWarning: the md5 module is deprecated; use hashlib instead
  import md5
Unhandled error in Deferred:
Traceback (most recent call last):
Failure: twisted.spread.pb.PBConnectionLost: [Failure instance:
Traceback (failure with no frames): <class
'twisted.internet.error.ConnectionLost'>: Connection to the other side
was lost in a non-clean fashion: Connection lost.
]
--- cut here ---

The buildmaster records the first change so it appears that the server
is hanging up after the first change is notified.

I can probably modify the script to make a new connection for each
change we want to process but I'm not familiar enough with the design to
assess if this is an elegant approach. I suspect not...

Can anyone suggest a better solution?


Version info is shown below:
# Buildbot: 0.7.12
# Twisted: 8.2.0
# Python: 2.6.2 (r262:71600, Aug 21 2009, 12:22:21)
    [GCC 4.4.1 20090818 (Red Hat 4.4.1-6)]
# Buildmaster platform: linux2

-- 
Daniel Thompson (STMicroelectronics) <daniel.thompson at st.com>
1000 Aztec West, Almondsbury, Bristol, BS32 4SQ. 01454 462659

If a car is a horseless carriage then is a motorcycle a horseless horse?