[Buildbot] #2538: LibVirtSlave issues

Buildbot trac trac at buildbot.net
Mon Nov 10 16:52:39 UTC 2014


#2538: LibVirtSlave issues
-------------------+---------------------
Reporter:  gg0     |       Owner:
    Type:  defect  |      Status:  closed
Priority:  major   |   Milestone:  0.9.+
 Version:  master  |  Resolution:  fixed
Keywords:          |
-------------------+---------------------
Changes (by dustin):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 John checked his irc logs and found
 {{{
 11:50 <+Jc2k> 17:28 <+Jc2k> gg0: if you stick a "from __future__ import
 absolute_import" above the imports in
 master/buildbot/buildslave/libvirt.py does it work?
 }}}
 and the missing patch was only 421 bytes, so that seems reasonable.  And
 lo,
 {{{
 commit 8c27b44507fa9e0ea520cc26e5a1f73dad9c40f4
 Author: Gabriele Giacone <1o5g4r8o at gmail.com>
 Date:   Fri Aug 2 19:41:40 2013 +0200

     Fix libvirt module load after internal module rename. Thanks Jc2k.

     See #2538

 diff --git a/master/buildbot/buildslave/libvirt.py
 b/master/buildbot/buildslave/libvirt.py
 index e413314..5776f10 100644
 --- a/master/buildbot/buildslave/libvirt.py
 +++ b/master/buildbot/buildslave/libvirt.py
 @@ -9,16 +9,17 @@
  #
  # You should have received a copy of the GNU General Public License along
 with
  # this program; if not, write to the Free Software Foundation, Inc., 51
  # Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
  #
  # Portions Copyright Buildbot Team Members
  # Portions Copyright 2010 Isotoma Limited

 +from __future__ import absolute_import
  import os

  from twisted.internet import defer, utils, threads
  from twisted.python import log, failure
  from buildbot.buildslave.base import AbstractBuildSlave,
 AbstractLatentBuildSlave
  from buildbot.util.eventual import eventually
  from buildbot import config

 }}}

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


More information about the bugs mailing list