[Buildbot-commits] [buildbot/buildbot] e32c49: moved master argument parsing to CreateSlaveOption...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/buildbot/buildbot
  Commit: e32c49e95c3085f59835e2f33bc4f1cc4d52d95c
      https://github.com/buildbot/buildbot/commit/e32c49e95c3085f59835e2f33bc4f1cc4d52d95c
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/scripts/create_slave.py
    M slave/buildslave/scripts/runner.py
    M slave/buildslave/test/unit/test_scripts_runner.py

  Log Message:
  -----------
  moved master argument parsing to CreateSlaveOptions class

Moved the code that validates and parses 'buildbot create-slave'
master argument to runner.CreateSlaveOptions class.


  Commit: 75b357f54f7ace905a5a9fff8c37cff2055f8766
      https://github.com/buildbot/buildbot/commit/75b357f54f7ace905a5a9fff8c37cff2055f8766
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/scripts/create_slave.py

  Log Message:
  -----------
  slave cli: replaced Maker class with local functions

No point in creating a single instance of Maker class, we can as well
call some local functions.


  Commit: f9cbc992d9ded1529d230cd95905360d50187025
      https://github.com/buildbot/buildbot/commit/f9cbc992d9ded1529d230cd95905360d50187025
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/scripts/create_slave.py

  Log Message:
  -----------
  catch IO errors when doing 'buildslave create-slave'

When creating buildslave directory with 'buildslave create-slave'
command, catch IO errors and print user friendly error messages.


  Commit: 26ca38096272ea86cfcb32416ee22d87fab9b243
      https://github.com/buildbot/buildbot/commit/26ca38096272ea86cfcb32416ee22d87fab9b243
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/test_extra_coverage.py
    A slave/buildslave/test/unit/test_scripts_create_slave.py

  Log Message:
  -----------
  added unit tests on buildslave.scripts.create_slave.createSlave()


  Commit: 919d668d9b06b3060a9b30584225155813949078
      https://github.com/buildbot/buildbot/commit/919d668d9b06b3060a9b30584225155813949078
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/unit/test_scripts_create_slave.py

  Log Message:
  -----------
  added unit tests on buildslave.scripts.create_slave._makeBaseDir()


  Commit: a9612a1fc508d681f89444ebf24367564e673a80
      https://github.com/buildbot/buildbot/commit/a9612a1fc508d681f89444ebf24367564e673a80
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/unit/test_scripts_create_slave.py
    M slave/buildslave/test/util/misc.py

  Log Message:
  -----------
  added unit tests on buildslave.scripts.create_slave._makeBuildbotTac()


  Commit: a3de85b4107fe825cd286e2608bfdd08029a71eb
      https://github.com/buildbot/buildbot/commit/a3de85b4107fe825cd286e2608bfdd08029a71eb
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/util/misc.py

  Log Message:
  -----------
  changed indention to follow PEP8


  Commit: 6a9011e8a43c103924c2a623213779ce51926856
      https://github.com/buildbot/buildbot/commit/6a9011e8a43c103924c2a623213779ce51926856
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/unit/test_scripts_create_slave.py
    M slave/buildslave/test/util/misc.py

  Log Message:
  -----------
  buildslave.test.misc.OpenFileMixin: specify default errno

In methods that patches open() to simulate open, read and write errors
use default errno for the exception raised.


  Commit: 89cfe563a5025f6a0da2aaf5b9454e158571e1a1
      https://github.com/buildbot/buildbot/commit/89cfe563a5025f6a0da2aaf5b9454e158571e1a1
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/unit/test_scripts_create_slave.py

  Log Message:
  -----------
  added unit tests on buildslave.scripts.create_slave._makeInfoFiles()


  Commit: 8df3483d884b6de684aa7e63a8c1927cf2761b67
      https://github.com/buildbot/buildbot/commit/8df3483d884b6de684aa7e63a8c1927cf2761b67
  Author: Elmir Jagudin <elmir at axis.com>
  Date:   2013-05-07 (Tue, 07 May 2013)

  Changed paths:
    M slave/buildslave/test/unit/test_scripts_base.py
    M slave/buildslave/test/unit/test_scripts_create_slave.py
    M slave/buildslave/test/unit/test_scripts_stop.py
    M slave/buildslave/test/unit/test_scripts_upgrade_slave.py
    M slave/buildslave/test/util/misc.py

  Log Message:
  -----------
  renamed buildslave.test.misc.OpenFileMixin class to FileIOMixin

The FileIOMixin class name is more descriptive, as the mixin now patches
more then just open() function


  Commit: 8226ae1ca603b0957babd7bae0cb044498191aad
      https://github.com/buildbot/buildbot/commit/8226ae1ca603b0957babd7bae0cb044498191aad
  Author: Dustin J. Mitchell <dustin at mozilla.com>
  Date:   2013-05-12 (Sun, 12 May 2013)

  Changed paths:
    M slave/buildslave/scripts/create_slave.py
    M slave/buildslave/scripts/runner.py
    M slave/buildslave/test/test_extra_coverage.py
    M slave/buildslave/test/unit/test_scripts_base.py
    A slave/buildslave/test/unit/test_scripts_create_slave.py
    M slave/buildslave/test/unit/test_scripts_runner.py
    M slave/buildslave/test/unit/test_scripts_stop.py
    M slave/buildslave/test/unit/test_scripts_upgrade_slave.py
    M slave/buildslave/test/util/misc.py

  Log Message:
  -----------
  Merge branch 'create-slave-cli' of git://github.com/elmirjagudin/buildbot


Compare: https://github.com/buildbot/buildbot/compare/97c3ef8bbe53...8226ae1ca603