← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/smart/server.py

(jameinel) Allow 'bzr serve' to interpret SIGHUP as a graceful shutdown.
 (bug #795025) (John A Meinel)

Show diffs side-by-side

added added

removed removed

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