[Buildbot-commits] [Buildbot] #2451: multiple events_ already exist

[Buildbot]

#2451: multiple events_ already exist
----------------------+------------------------
Reporter:  virgilg    |       Owner:
    Type:  undecided  |      Status:  new
Priority:  major      |   Milestone:  undecided
 Version:  0.8.7p1    |  Resolution:
Keywords:             |
----------------------+------------------------

Comment (by dustin):

 The patch in #2450 seems to fix this - or at least make it not occur.
 virgilg's going to add
 {{{
 #! patch
 diff --git a/master/buildbot/status/persistent_queue.py
 b/master/buildbot/status/persistent_queue.py
 index 0106a21..a4e5c11 100644
 --- a/master/buildbot/status/persistent_queue.py
 +++ b/master/buildbot/status/persistent_queue.py
 @@ -151,6 +151,7 @@ class DiskQueue(object):
          self._nbItems = 0
          # The actual items id start at one.
          self.firstItemId = 0
 +        print "DQ %d - init - lastItemId = 0" % (id(self),)
          self.lastItemId = 0
          self._loadFromDisk()

 @@ -164,6 +165,7 @@ class DiskQueue(object):
              self.firstItemId = id + 1
          else:
              self._nbItems += 1
 +        print "DQ %d - pushItem - lastItemId += 1 -> %d" % (id(self),
 self.lastItemId+1)
          self.lastItemId += 1
          path = os.path.join(self.path, str(self.lastItemId))
          if os.path.exists(path):
 @@ -245,6 +247,7 @@ class DiskQueue(object):
          if self._nbItems:
              self.firstItemId = files[0]
              self.lastItemId = files[-1]
 +            print "DQ %d - _loadFromDisk - lastItemId = %d" % (id(self),
 self.lastItemId)


  class PersistentQueue(object):
 }}}
 and reproduce.

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