[Buildbot-commits] [Buildbot] #827: ChangeSources may produce changes containing non-ascii bytestrings

[Buildbot]

#827: ChangeSources may produce changes containing non-ascii bytestrings
------------------------------+---------------------
Reporter:  dustin             |       Owner:
    Type:  defect             |      Status:  closed
Priority:  critical           |   Milestone:  0.8.+
 Version:  0.7.12             |  Resolution:  fixed
Keywords:  encoding database  |
------------------------------+---------------------

Comment (by birbilakos):

 Sure,

 Here's a full traceback of this issue. It happens every day depending on
 the special characters that get passed to the Perforce changelists.

 2011-07-15 12:31:01+0300 [-] P4 poll failed
         Traceback (most recent call last):
           File "/usr/local/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 269, in errback
             self._startRunCallbacks(fail)
           File "/usr/local/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 312, in _startRunCallbacks
             self._runCallbacks()
           File "/usr/local/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 328, in _runCallbacks
             self.result = callback(self.result, *args, **kw)
           File "/usr/local/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 630, in gotResult
             _deferGenerator(g, deferred)
         --- <exception caught here> ---
           File "/usr/local/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 605, in _deferGenerator
             result = g.next()
           File "/usr/local/lib/python2.6/site-
 packages/buildbot-0.8.4p1-py2.6.egg/buildbot/changes/p4poller.py", line
 186, in _poll
             wfd.getResult()
           File "/usr/local/lib/python2.6/site-
 packages/twisted/internet/defer.py", line 584, in getResult
             self.result.raiseException()
           File "/usr/local/lib/python2.6/site-
 packages/twisted/python/threadpool.py", line 210, in _worker
             result = context.call(ctx, function, *args, **kwargs)
           File "/usr/local/lib/python2.6/site-
 packages/twisted/python/context.py", line 59, in callWithContext
             return self.currentContext().callWithContext(ctx, func, *args,
 **kw)
           File "/usr/local/lib/python2.6/site-
 packages/twisted/python/context.py", line 37, in callWithContext
             return func(*args,**kw)
           File "/usr/local/lib/python2.6/site-
 packages/buildbot-0.8.4p1-py2.6.egg/buildbot/db/pool.py", line 112, in thd
             rv = callable(conn, *args, **kwargs)
           File "/usr/local/lib/python2.6/site-
 packages/buildbot-0.8.4p1-py2.6.egg/buildbot/db/changes.py", line 141, in
 thd
             project=project))
           File "/usr/local/lib/python2.6/site-
 packages/SQLAlchemy-0.7.1-py2.6-linux-i686.egg/sqlalchemy/engine/base.py",
 line 1358, in execute
             params)
           File "/usr/local/lib/python2.6/site-
 packages/SQLAlchemy-0.7.1-py2.6-linux-i686.egg/sqlalchemy/engine/base.py",
 line 1491, in _execute_clauseelement
             compiled_sql, distilled_params
           File "/usr/local/lib/python2.6/site-
 packages/SQLAlchemy-0.7.1-py2.6-linux-i686.egg/sqlalchemy/engine/base.py",
 line 1599, in _execute_context
             context)
           File "/usr/local/lib/python2.6/site-
 packages/SQLAlchemy-0.7.1-py2.6-linux-i686.egg/sqlalchemy/engine/base.py",
 line 1592, in _execute_context
             context)
           File "/usr/local/lib/python2.6/site-
 packages/SQLAlchemy-0.7.1-py2.6-linux-i686.egg/sqlalchemy/engine/default.py",
 line 325, in do_execute
             cursor.execute(statement, parameters)
         sqlalchemy.exc.ProgrammingError: (ProgrammingError) You must not
 use 8-bit bytestrings unless you use a text_factory that can interpret
 8-bit bytestrings (like text_factory = str). It is highly recommended that
 you instead just switch your application to Unicode strings. u'INSERT INTO
 changes (author, comments, is_dir, branch, revision, revlink,
 when_timestamp, category, repository, project) VALUES (?, ?, ?, ?, ?, ?,
 ?, ?, ?, ?)' ('build', 'Change 83699 by build at build1.1 on 2011/07/14
 03:37:24\n\n\tIntegrating //depot/1.1_maint/... to //depot/1.1/... as of
 change 83680\n\nJobs fixed ...\n\nbug31060 by relengr
 *REOPENED*\n\n\Release Bug\n\nbug36707 by xxx *VERIFIED-
 FIXED*\n\n\tPlayback fails for MBM file
 http://sph_breast_MBM.mp4\n\nbug36759 by xxx *VERIFIED-FIXED*\n\n\tChanges
 needed in Flexible Redirection to support Day Pass at Alltel.\n\nbug36767
 by xxx *VERIFIED-FIXED*\n\n\t\x91307\x92 responses from the server are
 sent to the client as \x91206\x92.\n\nbug36796 by xxx *RESOLVED-
 FIXED*\n\n\tRadius enhancement to be able to handle and log the ULI
 VSA\n\nbug36833 by xxx *VERIFIED-FIXED*\n\n\tKPI report inconsistent
 throughput with KPI Parser log errors\n\nbug36972 by xxx *RESOLVED-
 FIXED*\n\n\tWrong value for 3GPP-User-Location-Info registered in the
 sdb\n\n', '83699', '', 1310639844, None, '', '')

-- 
Ticket URL: <http://trac.buildbot.net/ticket/827#comment:16>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation