← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/server.py

  • Committer: Aaron Bentley
  • Date: 2012-07-13 19:32:33 UTC
  • mto: This revision was merged to the branch mainline in revision 6540.
  • Revision ID: aaron@aaronbentley.com-20120713193233-l6y0l0twwhd3wmka
Switch to much simpler implementation of restore_uncommitted.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/smart/server.py
1
 
# Copyright (C) 2006, 2007 Canonical Ltd
 
1
# Copyright (C) 2006-2011 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
"""Server for smart-server protocol."""
18
18
 
 
19
from __future__ import absolute_import
 
20
 
19
21
import errno
 
22
import os.path
20
23
import socket
 
24
import sys
 
25
import time
21
26
import threading
22
27
 
23
28
from bzrlib.hooks import Hooks
24
29
from bzrlib import (
25
30
    errors,
26
31
    trace,
27
 
    transport,
 
32
    transport as _mod_transport,
28
33
)
29
 
from bzrlib.smart.medium import SmartServerSocketStreamMedium
 
34
from bzrlib.i18n import gettext
 
35
from bzrlib.lazy_import import lazy_import
 
36
lazy_import(globals(), """
 
37
from bzrlib.smart import (
 
38
    medium,
 
39
    signals,
 
40
    )
 
41
from bzrlib.transport import (
 
42
    chroot,
 
43
    pathfilter,
 
44
    )
 
45
from bzrlib import (
 
46
    config,
 
47
    urlutils,
 
48
    )
 
49
""")
30
50
 
31
51
 
32
52
class SmartTCPServer(object):
38
58
    hooks: An instance of SmartServerHooks.
39
59
    """
40
60
 
41
 
    def __init__(self, backing_transport, host='127.0.0.1', port=0):
 
61
    # This is the timeout on the socket we use .accept() on. It is exposed here
 
62
    # so the test suite can set it faster. (It thread.interrupt_main() will not
 
63
    # fire a KeyboardInterrupt during socket.accept)
 
64
    _ACCEPT_TIMEOUT = 1.0
 
65
    _SHUTDOWN_POLL_TIMEOUT = 1.0
 
66
    _LOG_WAITING_TIMEOUT = 10.0
 
67
 
 
68
    _timer = time.time
 
69
 
 
70
    def __init__(self, backing_transport, root_client_path='/',
 
71
                 client_timeout=None):
42
72
        """Construct a new server.
43
73
 
44
74
        To actually start it running, call either start_background_thread or
45
75
        serve.
46
76
 
 
77
        :param backing_transport: The transport to serve.
 
78
        :param root_client_path: The client path that will correspond to root
 
79
            of backing_transport.
 
80
        :param client_timeout: See SmartServerSocketStreamMedium's timeout
 
81
            parameter.
 
82
        """
 
83
        self.backing_transport = backing_transport
 
84
        self.root_client_path = root_client_path
 
85
        self._client_timeout = client_timeout
 
86
        self._active_connections = []
 
87
        # This is set to indicate we want to wait for clients to finish before
 
88
        # we disconnect.
 
89
        self._gracefully_stopping = False
 
90
 
 
91
    def start_server(self, host, port):
 
92
        """Create the server listening socket.
 
93
 
47
94
        :param host: Name of the interface to listen on.
48
95
        :param port: TCP port to listen on, or 0 to allocate a transient port.
49
96
        """
54
101
        from socket import error as socket_error
55
102
        self._socket_error = socket_error
56
103
        self._socket_timeout = socket_timeout
57
 
        self._server_socket = socket.socket()
58
 
        self._server_socket.bind((host, port))
 
104
        addrs = socket.getaddrinfo(host, port, socket.AF_UNSPEC,
 
105
            socket.SOCK_STREAM, 0, socket.AI_PASSIVE)[0]
 
106
 
 
107
        (family, socktype, proto, canonname, sockaddr) = addrs
 
108
 
 
109
        self._server_socket = socket.socket(family, socktype, proto)
 
110
        # SO_REUSERADDR has a different meaning on Windows
 
111
        if sys.platform != 'win32':
 
112
            self._server_socket.setsockopt(socket.SOL_SOCKET,
 
113
                socket.SO_REUSEADDR, 1)
 
114
        try:
 
115
            self._server_socket.bind(sockaddr)
 
116
        except self._socket_error, message:
 
117
            raise errors.CannotBindAddress(host, port, message)
59
118
        self._sockname = self._server_socket.getsockname()
60
119
        self.port = self._sockname[1]
61
120
        self._server_socket.listen(1)
62
 
        self._server_socket.settimeout(1)
63
 
        self.backing_transport = backing_transport
 
121
        self._server_socket.settimeout(self._ACCEPT_TIMEOUT)
 
122
        # Once we start accept()ing connections, we set started.
64
123
        self._started = threading.Event()
 
124
        # Once we stop accept()ing connections (and are closing the socket) we
 
125
        # set _stopped
65
126
        self._stopped = threading.Event()
 
127
        # Once we have finished waiting for all clients, etc. We set
 
128
        # _fully_stopped
 
129
        self._fully_stopped = threading.Event()
66
130
 
67
 
    def serve(self):
68
 
        self._should_terminate = False
69
 
        # for hooks we are letting code know that a server has started (and
70
 
        # later stopped).
 
131
    def _backing_urls(self):
71
132
        # There are three interesting urls:
72
133
        # The URL the server can be contacted on. (e.g. bzr://host/)
73
134
        # The URL that a commit done on the same machine as the server will
74
135
        # have within the servers space. (e.g. file:///home/user/source)
75
136
        # The URL that will be given to other hooks in the same process -
76
 
        # the URL of the backing transport itself. (e.g. chroot+:///)
 
137
        # the URL of the backing transport itself. (e.g. filtered-36195:///)
77
138
        # We need all three because:
78
139
        #  * other machines see the first
79
140
        #  * local commits on this machine should be able to be mapped to
80
 
        #    this server 
 
141
        #    this server
81
142
        #  * commits the server does itself need to be mapped across to this
82
143
        #    server.
83
144
        # The latter two urls are different aliases to the servers url,
84
 
        # so we group those in a list - as there might be more aliases 
 
145
        # so we group those in a list - as there might be more aliases
85
146
        # in the future.
86
 
        backing_urls = [self.backing_transport.base]
 
147
        urls = [self.backing_transport.base]
87
148
        try:
88
 
            backing_urls.append(self.backing_transport.external_url())
 
149
            urls.append(self.backing_transport.external_url())
89
150
        except errors.InProcessTransport:
90
151
            pass
 
152
        return urls
 
153
 
 
154
    def run_server_started_hooks(self, backing_urls=None):
 
155
        if backing_urls is None:
 
156
            backing_urls = self._backing_urls()
91
157
        for hook in SmartTCPServer.hooks['server_started']:
92
158
            hook(backing_urls, self.get_url())
 
159
        for hook in SmartTCPServer.hooks['server_started_ex']:
 
160
            hook(backing_urls, self)
 
161
 
 
162
    def run_server_stopped_hooks(self, backing_urls=None):
 
163
        if backing_urls is None:
 
164
            backing_urls = self._backing_urls()
 
165
        for hook in SmartTCPServer.hooks['server_stopped']:
 
166
            hook(backing_urls, self.get_url())
 
167
 
 
168
    def _stop_gracefully(self):
 
169
        trace.note(gettext('Requested to stop gracefully'))
 
170
        self._should_terminate = True
 
171
        self._gracefully_stopping = True
 
172
        for handler, _ in self._active_connections:
 
173
            handler._stop_gracefully()
 
174
 
 
175
    def _wait_for_clients_to_disconnect(self):
 
176
        self._poll_active_connections()
 
177
        if not self._active_connections:
 
178
            return
 
179
        trace.note(gettext('Waiting for %d client(s) to finish')
 
180
                   % (len(self._active_connections),))
 
181
        t_next_log = self._timer() + self._LOG_WAITING_TIMEOUT
 
182
        while self._active_connections:
 
183
            now = self._timer()
 
184
            if now >= t_next_log:
 
185
                trace.note(gettext('Still waiting for %d client(s) to finish')
 
186
                           % (len(self._active_connections),))
 
187
                t_next_log = now + self._LOG_WAITING_TIMEOUT
 
188
            self._poll_active_connections(self._SHUTDOWN_POLL_TIMEOUT)
 
189
 
 
190
    def serve(self, thread_name_suffix=''):
 
191
        # Note: There is a temptation to do
 
192
        #       signals.register_on_hangup(id(self), self._stop_gracefully)
 
193
        #       However, that creates a temporary object which is a bound
 
194
        #       method. signals._on_sighup is a WeakKeyDictionary so it
 
195
        #       immediately gets garbage collected, because nothing else
 
196
        #       references it. Instead, we need to keep a real reference to the
 
197
        #       bound method for the lifetime of the serve() function.
 
198
        stop_gracefully = self._stop_gracefully
 
199
        signals.register_on_hangup(id(self), stop_gracefully)
 
200
        self._should_terminate = False
 
201
        # for hooks we are letting code know that a server has started (and
 
202
        # later stopped).
 
203
        self.run_server_started_hooks()
93
204
        self._started.set()
94
205
        try:
95
206
            try:
101
212
                        pass
102
213
                    except self._socket_error, e:
103
214
                        # if the socket is closed by stop_background_thread
104
 
                        # we might get a EBADF here, any other socket errors
105
 
                        # should get logged.
106
 
                        if e.args[0] != errno.EBADF:
107
 
                            trace.warning("listening socket error: %s", e)
 
215
                        # we might get a EBADF here, or if we get a signal we
 
216
                        # can get EINTR, any other socket errors should get
 
217
                        # logged.
 
218
                        if e.args[0] not in (errno.EBADF, errno.EINTR):
 
219
                            trace.warning(gettext("listening socket error: %s")
 
220
                                          % (e,))
108
221
                    else:
109
 
                        self.serve_conn(conn)
 
222
                        if self._should_terminate:
 
223
                            conn.close()
 
224
                            break
 
225
                        self.serve_conn(conn, thread_name_suffix)
 
226
                    # Cleanout any threads that have finished processing.
 
227
                    self._poll_active_connections()
110
228
            except KeyboardInterrupt:
111
229
                # dont log when CTRL-C'd.
112
230
                raise
113
231
            except Exception, e:
114
 
                trace.error("Unhandled smart server error.")
115
 
                trace.log_exception_quietly()
 
232
                trace.report_exception(sys.exc_info(), sys.stderr)
116
233
                raise
117
234
        finally:
118
 
            self._stopped.set()
119
235
            try:
120
236
                # ensure the server socket is closed.
121
237
                self._server_socket.close()
122
238
            except self._socket_error:
123
239
                # ignore errors on close
124
240
                pass
125
 
            for hook in SmartTCPServer.hooks['server_stopped']:
126
 
                hook(backing_urls, self.get_url())
 
241
            self._stopped.set()
 
242
            signals.unregister_on_hangup(id(self))
 
243
            self.run_server_stopped_hooks()
 
244
        if self._gracefully_stopping:
 
245
            self._wait_for_clients_to_disconnect()
 
246
        self._fully_stopped.set()
127
247
 
128
248
    def get_url(self):
129
249
        """Return the url of the server"""
130
 
        return "bzr://%s:%d/" % self._sockname
131
 
 
132
 
    def serve_conn(self, conn):
 
250
        return "bzr://%s:%s/" % (self._sockname[0], self._sockname[1])
 
251
 
 
252
    def _make_handler(self, conn):
 
253
        return medium.SmartServerSocketStreamMedium(
 
254
            conn, self.backing_transport, self.root_client_path,
 
255
            timeout=self._client_timeout)
 
256
 
 
257
    def _poll_active_connections(self, timeout=0.0):
 
258
        """Check to see if any active connections have finished.
 
259
 
 
260
        This will iterate through self._active_connections, and update any
 
261
        connections that are finished.
 
262
 
 
263
        :param timeout: The timeout to pass to thread.join(). By default, we
 
264
            set it to 0, so that we don't hang if threads are not done yet.
 
265
        :return: None
 
266
        """
 
267
        still_active = []
 
268
        for handler, thread in self._active_connections:
 
269
            thread.join(timeout)
 
270
            if thread.isAlive():
 
271
                still_active.append((handler, thread))
 
272
        self._active_connections = still_active
 
273
 
 
274
    def serve_conn(self, conn, thread_name_suffix):
133
275
        # For WIN32, where the timeout value from the listening socket
134
 
        # propogates to the newly accepted socket.
 
276
        # propagates to the newly accepted socket.
135
277
        conn.setblocking(True)
136
278
        conn.setsockopt(socket.IPPROTO_TCP, socket.TCP_NODELAY, 1)
137
 
        handler = SmartServerSocketStreamMedium(conn, self.backing_transport)
138
 
        connection_thread = threading.Thread(None, handler.serve, name='smart-server-child')
 
279
        thread_name = 'smart-server-child' + thread_name_suffix
 
280
        handler = self._make_handler(conn)
 
281
        connection_thread = threading.Thread(
 
282
            None, handler.serve, name=thread_name)
 
283
        self._active_connections.append((handler, connection_thread))
139
284
        connection_thread.setDaemon(True)
140
285
        connection_thread.start()
 
286
        return connection_thread
141
287
 
142
 
    def start_background_thread(self):
 
288
    def start_background_thread(self, thread_name_suffix=''):
143
289
        self._started.clear()
144
290
        self._server_thread = threading.Thread(None,
145
 
                self.serve,
 
291
                self.serve, args=(thread_name_suffix,),
146
292
                name='server-' + self.get_url())
147
293
        self._server_thread.setDaemon(True)
148
294
        self._server_thread.start()
154
300
        self._should_terminate = True
155
301
        # close the socket - gives error to connections from here on in,
156
302
        # rather than a connection reset error to connections made during
157
 
        # the period between setting _should_terminate = True and 
 
303
        # the period between setting _should_terminate = True and
158
304
        # the current request completing/aborting. It may also break out the
159
305
        # main loop if it was currently in accept() (on some platforms).
160
306
        try:
183
329
        These are all empty initially, because by default nothing should get
184
330
        notified.
185
331
        """
186
 
        Hooks.__init__(self)
187
 
        # Introduced in 0.16:
188
 
        # invoked whenever the server starts serving a directory.
189
 
        # The api signature is (backing urls, public url).
190
 
        self['server_started'] = []
191
 
        # Introduced in 0.16:
192
 
        # invoked whenever the server stops serving a directory.
193
 
        # The api signature is (backing urls, public url).
194
 
        self['server_stopped'] = []
 
332
        Hooks.__init__(self, "bzrlib.smart.server", "SmartTCPServer.hooks")
 
333
        self.add_hook('server_started',
 
334
            "Called by the bzr server when it starts serving a directory. "
 
335
            "server_started is called with (backing urls, public url), "
 
336
            "where backing_url is a list of URLs giving the "
 
337
            "server-specific directory locations, and public_url is the "
 
338
            "public URL for the directory being served.", (0, 16))
 
339
        self.add_hook('server_started_ex',
 
340
            "Called by the bzr server when it starts serving a directory. "
 
341
            "server_started is called with (backing_urls, server_obj).",
 
342
            (1, 17))
 
343
        self.add_hook('server_stopped',
 
344
            "Called by the bzr server when it stops serving a directory. "
 
345
            "server_stopped is called with the same parameters as the "
 
346
            "server_started hook: (backing_urls, public_url).", (0, 16))
 
347
        self.add_hook('server_exception',
 
348
            "Called by the bzr server when an exception occurs. "
 
349
            "server_exception is called with the sys.exc_info() tuple "
 
350
            "return true for the hook if the exception has been handled, "
 
351
            "in which case the server will exit normally.", (2, 4))
195
352
 
196
353
SmartTCPServer.hooks = SmartServerHooks()
197
354
 
198
355
 
199
 
class SmartTCPServer_for_testing(SmartTCPServer):
200
 
    """Server suitable for use by transport tests.
201
 
    
202
 
    This server is backed by the process's cwd.
203
 
    """
204
 
 
205
 
    def __init__(self):
206
 
        SmartTCPServer.__init__(self, None)
207
 
        
208
 
    def get_backing_transport(self, backing_transport_server):
209
 
        """Get a backing transport from a server we are decorating."""
210
 
        return transport.get_transport(backing_transport_server.get_url())
211
 
 
212
 
    def setUp(self, backing_transport_server=None):
213
 
        """Set up server for testing"""
214
 
        from bzrlib.transport.chroot import ChrootServer
215
 
        if backing_transport_server is None:
216
 
            from bzrlib.transport.local import LocalURLServer
217
 
            backing_transport_server = LocalURLServer()
218
 
        self.chroot_server = ChrootServer(
219
 
            self.get_backing_transport(backing_transport_server))
220
 
        self.chroot_server.setUp()
221
 
        self.backing_transport = transport.get_transport(
222
 
            self.chroot_server.get_url())
223
 
        self.start_background_thread()
224
 
 
225
 
    def tearDown(self):
226
 
        self.stop_background_thread()
227
 
        self.chroot_server.tearDown()
228
 
 
229
 
    def get_bogus_url(self):
230
 
        """Return a URL which will fail to connect"""
231
 
        return 'bzr://127.0.0.1:1/'
232
 
 
233
 
 
234
 
class ReadonlySmartTCPServer_for_testing(SmartTCPServer_for_testing):
235
 
    """Get a readonly server for testing."""
236
 
 
237
 
    def get_backing_transport(self, backing_transport_server):
238
 
        """Get a backing transport from a server we are decorating."""
239
 
        url = 'readonly+' + backing_transport_server.get_url()
240
 
        return transport.get_transport(url)
241
 
 
 
356
def _local_path_for_transport(transport):
 
357
    """Return a local path for transport, if reasonably possible.
 
358
    
 
359
    This function works even if transport's url has a "readonly+" prefix,
 
360
    unlike local_path_from_url.
 
361
    
 
362
    This essentially recovers the --directory argument the user passed to "bzr
 
363
    serve" from the transport passed to serve_bzr.
 
364
    """
 
365
    try:
 
366
        base_url = transport.external_url()
 
367
    except (errors.InProcessTransport, NotImplementedError):
 
368
        return None
 
369
    else:
 
370
        # Strip readonly prefix
 
371
        if base_url.startswith('readonly+'):
 
372
            base_url = base_url[len('readonly+'):]
 
373
        try:
 
374
            return urlutils.local_path_from_url(base_url)
 
375
        except errors.InvalidURL:
 
376
            return None
 
377
 
 
378
 
 
379
class BzrServerFactory(object):
 
380
    """Helper class for serve_bzr."""
 
381
 
 
382
    def __init__(self, userdir_expander=None, get_base_path=None):
 
383
        self.cleanups = []
 
384
        self.base_path = None
 
385
        self.backing_transport = None
 
386
        if userdir_expander is None:
 
387
            userdir_expander = os.path.expanduser
 
388
        self.userdir_expander = userdir_expander
 
389
        if get_base_path is None:
 
390
            get_base_path = _local_path_for_transport
 
391
        self.get_base_path = get_base_path
 
392
 
 
393
    def _expand_userdirs(self, path):
 
394
        """Translate /~/ or /~user/ to e.g. /home/foo, using
 
395
        self.userdir_expander (os.path.expanduser by default).
 
396
 
 
397
        If the translated path would fall outside base_path, or the path does
 
398
        not start with ~, then no translation is applied.
 
399
 
 
400
        If the path is inside, it is adjusted to be relative to the base path.
 
401
 
 
402
        e.g. if base_path is /home, and the expanded path is /home/joe, then
 
403
        the translated path is joe.
 
404
        """
 
405
        result = path
 
406
        if path.startswith('~'):
 
407
            expanded = self.userdir_expander(path)
 
408
            if not expanded.endswith('/'):
 
409
                expanded += '/'
 
410
            if expanded.startswith(self.base_path):
 
411
                result = expanded[len(self.base_path):]
 
412
        return result
 
413
 
 
414
    def _make_expand_userdirs_filter(self, transport):
 
415
        return pathfilter.PathFilteringServer(transport, self._expand_userdirs)
 
416
 
 
417
    def _make_backing_transport(self, transport):
 
418
        """Chroot transport, and decorate with userdir expander."""
 
419
        self.base_path = self.get_base_path(transport)
 
420
        chroot_server = chroot.ChrootServer(transport)
 
421
        chroot_server.start_server()
 
422
        self.cleanups.append(chroot_server.stop_server)
 
423
        transport = _mod_transport.get_transport_from_url(chroot_server.get_url())
 
424
        if self.base_path is not None:
 
425
            # Decorate the server's backing transport with a filter that can
 
426
            # expand homedirs.
 
427
            expand_userdirs = self._make_expand_userdirs_filter(transport)
 
428
            expand_userdirs.start_server()
 
429
            self.cleanups.append(expand_userdirs.stop_server)
 
430
            transport = _mod_transport.get_transport_from_url(expand_userdirs.get_url())
 
431
        self.transport = transport
 
432
 
 
433
    def _get_stdin_stdout(self):
 
434
        return sys.stdin, sys.stdout
 
435
 
 
436
    def _make_smart_server(self, host, port, inet, timeout):
 
437
        if timeout is None:
 
438
            c = config.GlobalStack()
 
439
            timeout = c.get('serve.client_timeout')
 
440
        if inet:
 
441
            stdin, stdout = self._get_stdin_stdout()
 
442
            smart_server = medium.SmartServerPipeStreamMedium(
 
443
                stdin, stdout, self.transport, timeout=timeout)
 
444
        else:
 
445
            if host is None:
 
446
                host = medium.BZR_DEFAULT_INTERFACE
 
447
            if port is None:
 
448
                port = medium.BZR_DEFAULT_PORT
 
449
            smart_server = SmartTCPServer(self.transport,
 
450
                                          client_timeout=timeout)
 
451
            smart_server.start_server(host, port)
 
452
            trace.note(gettext('listening on port: %s') % smart_server.port)
 
453
        self.smart_server = smart_server
 
454
 
 
455
    def _change_globals(self):
 
456
        from bzrlib import lockdir, ui
 
457
        # For the duration of this server, no UI output is permitted. note
 
458
        # that this may cause problems with blackbox tests. This should be
 
459
        # changed with care though, as we dont want to use bandwidth sending
 
460
        # progress over stderr to smart server clients!
 
461
        old_factory = ui.ui_factory
 
462
        old_lockdir_timeout = lockdir._DEFAULT_TIMEOUT_SECONDS
 
463
        def restore_default_ui_factory_and_lockdir_timeout():
 
464
            ui.ui_factory = old_factory
 
465
            lockdir._DEFAULT_TIMEOUT_SECONDS = old_lockdir_timeout
 
466
        self.cleanups.append(restore_default_ui_factory_and_lockdir_timeout)
 
467
        ui.ui_factory = ui.SilentUIFactory()
 
468
        lockdir._DEFAULT_TIMEOUT_SECONDS = 0
 
469
        orig = signals.install_sighup_handler()
 
470
        def restore_signals():
 
471
            signals.restore_sighup_handler(orig)
 
472
        self.cleanups.append(restore_signals)
 
473
 
 
474
    def set_up(self, transport, host, port, inet, timeout):
 
475
        self._make_backing_transport(transport)
 
476
        self._make_smart_server(host, port, inet, timeout)
 
477
        self._change_globals()
 
478
 
 
479
    def tear_down(self):
 
480
        for cleanup in reversed(self.cleanups):
 
481
            cleanup()
 
482
 
 
483
 
 
484
def serve_bzr(transport, host=None, port=None, inet=False, timeout=None):
 
485
    """This is the default implementation of 'bzr serve'.
 
486
 
 
487
    It creates a TCP or pipe smart server on 'transport, and runs it.  The
 
488
    transport will be decorated with a chroot and pathfilter (using
 
489
    os.path.expanduser).
 
490
    """
 
491
    bzr_server = BzrServerFactory()
 
492
    try:
 
493
        bzr_server.set_up(transport, host, port, inet, timeout)
 
494
        bzr_server.smart_server.serve()
 
495
    except:
 
496
        hook_caught_exception = False
 
497
        for hook in SmartTCPServer.hooks['server_exception']:
 
498
            hook_caught_exception = hook(sys.exc_info())
 
499
        if not hook_caught_exception:
 
500
            raise
 
501
    finally:
 
502
        bzr_server.tear_down()