[Buildbot-commits] [buildbot/buildbot] 0cb6f1: Better error-handling in DuplicateSlaveArbitrator

[noreply at github.com]

  Branch: refs/heads/master
  Home:   https://github.com/buildbot/buildbot

  Commit: 0cb6f1e83e10e023711b326ef77c216e3b0d5291
      https://github.com/buildbot/buildbot/commit/0cb6f1e83e10e023711b326ef77c216e3b0d5291
  Author: Dustin J. Mitchell <dustin at mozilla.com>
  Date:   2011-07-31 (Sun, 31 Jul 2011)

  Changed paths:
    M master/buildbot/process/botmaster.py
  A master/buildbot/test/unit/test_process_botmaster_DuplicateSlaveArbitrator.py

  Log Message:
  -----------
  Better error-handling in DuplicateSlaveArbitrator

This includes full tests for the arbitrator as well.
Refs #2022.


  Commit: 995a6a7f4a18fe5b59cf57cd2a4afe3ab6234fc7
      https://github.com/buildbot/buildbot/commit/995a6a7f4a18fe5b59cf57cd2a4afe3ab6234fc7
  Author: Dustin J. Mitchell <dustin at mozilla.com>
  Date:   2011-07-31 (Sun, 31 Jul 2011)

  Changed paths:
    M master/buildbot/buildslave.py
  M master/buildbot/pbmanager.py
  M master/buildbot/process/botmaster.py
  A master/buildbot/test/integration/test_slave_comm.py
  M master/buildbot/test/unit/test_process_botmaster_DuplicateSlaveArbitrator.py

  Log Message:
  -----------
  More thorough testing of DuplicateSlaveArbitrator

This adds a subscribeToDetach method to build slaves, which the
arbitrator can use to wait for the old slave to detach, so there's no
longer any need to poll slave.isConnected.  Fixes #2022.


Compare: https://github.com/buildbot/buildbot/compare/6fdf5fb...995a6a7