Changelog¶
Changes by Version¶
1.3.2 (unreleased)¶
- Nothing changed yet.
1.3.1 (2018-06-11)¶
- Fixed a bug which caused servers to send requests to peers that sent requests to them.
1.3.0 (2017-11-20)¶
- Added OpenTracing client interceptor support for outbound requests.
1.2.0 (2017-10-19)¶
- Hook methods can now be implemented as coroutines.
- Added a new event (before_serialize_request_headers) that can be hooked. This is intended to allow application headers to be modified before requests are sent.
1.1.0 (2017-04-10)¶
- Added messages with ttl, service, and hostport information to TimeoutErrors
1.0.2 (2017-03-20)¶
- Fixed a race condition where the on_close callback for tchannel connections would not be called if the connection was already closed.
- Fixed a bug where the reference to the next node would not be cleared when nodes were pulled from message queues (Introducing a potential memory leak).
1.0.1 (2016-12-14)¶
- Add str functions to Peer and PeerClientOperation for easier debugging in exc_info
- Updated internal APIs to no longer depend on the PeerGroup add function and to use the get function for creating new peers instead.
- Fixed a bug where choosing a hostport directly for a downstream call would add that peer to the “core” peers which are used for regular calls. Now choosing the hostport directly will create a peer but will exclude it from selection.
1.0.0 (2016-11-17)¶
- Committing to existing API. We’re calling this a 1.0.
0.30.6 (2016-11-14)¶
- Fixed a bug which would cause handshake timeouts to bubble up to the caller rather than retry a different peer.
0.30.5 (2016-11-10)¶
- Fixed a bug which would cause assertion errors if a connection to a peer disconnected shortly after a handshake.
0.30.4 (2016-11-03)¶
- Time out handshake attempts for outgoing connections after 5 seconds.
- Fixed a regression where large requests would block small requests until they were completely written to the wire.
- Propagate message sending errors up to the caller. This should greatly reduce
the number of
TimeoutError: None
issues seen by users and show the root cause instead. - Fail
TChannel
instantiation if the service name is empty or None.
0.30.3 (2016-10-24)¶
- Revert 0.30.2. The previous release may have introduced a memory leak.
0.30.2 (2016-10-12)¶
- Propagate message sending errors up to the caller. This should greatly reduce
the number of
TimeoutError: None
issues seen by users and show the root cause instead. - Fail
TChannel
instantiation if the service name is empty or None.
0.30.1 (2016-10-05)¶
- Relax opentracing upper bound to next major.
- Never send requests to ephemeral peers.
0.30.0 (2016-09-29)¶
- Pass span.kind tag when calling start_span(), not after the span was started.
- Add jitter argument to advertise().
0.29.1 (2016-10-05)¶
- Never send requests to ephemeral peers.
- Relax opentracing upper bound to next major.
0.29.0 (2016-09-12)¶
- Change default setting for tracing to be enabled.
- You can now specify an override for a request’s
cn
transport header using thecaller_name
argument of thecall()
,json()
,raw()
, andthrift()
methods ofTChannel
.
0.28.3 (2016-10-05)¶
- Never send requests to ephemeral peers.
- Relax opentracing upper bound to next major.
0.28.2 (2016-09-12)¶
- Bug fix: Tracing headers will no longer be added for raw requests if the headers are unparsed.
0.28.1 (2016-08-19)¶
- Ignore tracing fields with empty/zero trace ID.
0.28.0 (2016-08-17)¶
- Don’t send more Hyperbahn advertise requests if an existing request is ongoing.
- Add jitter between Hyperbahn consecutive advertise requests.
- If the initial advertise request fails, propagate the original error instead of a timeout error.
0.27.4 (2016-10-05)¶
- Never send requests to ephemeral peers.
- Relax opentracing upper bound to next major.
0.27.3 (2016-08-19)¶
- Ignore tracing fields with empty/zero trace ID.
0.27.2 (2016-08-17)¶
- VCR should ignore tracing headers when matching requests. This will allow replaying requests with or without tracing regardless of whether the original request was recorded with it.
0.27.1 (2016-08-10)¶
- Bug fix: set Trace.parent_id to 0 if it’s None
0.27.0 (2016-08-08)¶
- Native integration with OpenTracing (for real this time)
- Replace tcollector and explicit trace reporting with OpenTracing
0.26.1 (2016-10-05)¶
- Never send requests to ephemeral peers.
0.26.0 (2016-07-13)¶
- VCR:
use_cassette
now uses cached copies of cassettes if their contents have not changed. This should improve performance for large cassette files.
0.25.2 (2016-10-05)¶
- Never send requests to ephemeral peers.
0.25.1 (2016-06-30)¶
- Fixed a bug where the application error status code was not being copied into Response objects.
0.25.0 (2016-06-16)¶
- Support for OpenTracing.
0.24.1 (2016-10-05)¶
- Never send requests to ephemeral peers.
0.24.0 (2016-04-19)¶
- Added
TChannel.host
andTChannel.port
. - Added
TChannel.close()
andTChannel.is_closed()
.
0.23.2 (2016-10-05)¶
- Never send requests to ephemeral peers.
0.23.1 (2016-04-14)¶
- Fixed tornado version constraint causing
reuse_port
to be missing, updated constraint totornado>=4.3,<5
. - Only pass
reuse_port
tobind_sockets
if it’s set to True.
0.23.0 (2016-04-14)¶
- Added an opt-in feature to use the
SO_REUSEPORT
socket option for TChannel servers. Usereuse_port=True
when instantiating aTChannel
.
0.22.4 (2016-10-05)¶
- Never send requests to ephemeral peers.
0.22.3 (2016-04-07)¶
- Fixed a bug where type mismatch for timeouts could cause a crash.
0.22.2 (2016-04-06)¶
- VCR now respects the timeout specified on the original request. Timeouts in
making the requests while recording now propagate as
TimeoutError
exceptions rather thanRemoteServiceError
. - Reduced a warning for unconsumed error messages to info.
- Made UnexpectedError’s message a little more debuggable.
0.22.1 (2016-04-06)¶
- Added a timeout to the VCR proxy call.
- Fixed a bug where tests would time out if the VCR server failed to start. The VCR server failure is now propagated to the caller.
0.22.0 (2016-03-31)¶
- Peer selection is now constant time instead of linear time. This should significantly reduce CPU load per request.
- Fixed a bug where certain errors while reading requests would propagate as TimeoutErrors.
- Attempting to register endpoints against a synchronous TChannel now logs an INFO level message.
- Reduced default advertisement interval to 3 minutes.
0.21.10 (2016-03-17)¶
- Zipkin traces now include a server-side ‘cn’ annotation to identify callers.
- Reduced “unconsumed message” warnings to INFO. These are typically generated when Hyperbahn garbage collects your process due to a timed-out advertisement.
- Handshake timeouts were incorrectly being surfaced as StreamClosedError but are now raised as NetworkError.
- Reduced default tracing sample rate from 100% to 1%.
0.21.9 (2016-03-14)¶
- Fixed a bug that caused silent failures when a write attempt was made to a closed connection.
- Reduce
StreamClosedError
log noisiness for certain scenarios. - Make
TChannel.advertise
idempotent and thread-safe.
0.21.8 (2016-03-10)¶
- Reduce read errors due to clients disconnecting to INFO from ERROR.
0.21.7 (2016-03-08)¶
- Fixed an unhelpful stack trace on failed reads.
0.21.6 (2016-03-08)¶
- Fixed a logging error on failed reads.
0.21.5 (2016-03-08)¶
- Tornado 4.2 was listed as a requirement but this was corrected to be 4.3 which introduced the locks module.
- Fixed in issue where clients could incorrectly time out when reading large response bodies. This was due to response fragments being dropped due to out-of-order writes; writes are now serialized on a per-connection basis.
0.21.4 (2016-02-15)¶
- Fixed noisy logging of late responses for requests that timed out locally.
0.21.3 (2016-01-22)¶
- Attempting to register endpoints against a synchronous TChannel is now a no-op instead of a crash.
0.21.2 (2016-01-05)¶
- The synchronous client will no longer start a thread when the
TChannel
instance is initialized. This resolves an issue where an application could hang indefinitely if it instantiated a synchronousTChannel
at import time.
0.21.1 (2015-12-29)¶
- Fixed a bug in Zipkin instrumentation that would cause CPU spikes due to an infinite loop during downstream requests.
0.21.0 (2015-12-10)¶
- Add support for zipkin trace sampling.
tchannel.TChannel.FALLBACK
may now be used to register fallback endpoints which are called for requests with unrecognized endpoints. For more information, see Can I register an endpoint that accepts all requests?- Expose
timeout
andservice
attributes onRequest
objects inside endpoint handlers. - Disable the retry for all zipkin trace submit.
- Fix Thrift service inheritance bug which caused parent methods to not be propagated to child services.
- VCR recording should not fail if the destination directory for the cassette does not exist.
- Fix bug which incorrectly encoded JSON arg scheme headers in the incorrect format.
- Add support for
rd
transport header. - BREAKING - Support unit testing endpoints by calling the handler
functions directly. This is enabled by changing
tchannel.thrift.register
to return the registered function unmodified. See Upgrade Guide for more details.
0.20.2 (2015-11-25)¶
- Lower the log level for Hyperbahn advertisement failures that can be retried.
- Include the full stack trace when Hyperbahn advertisement failures are logged.
- Include the error message for unexpected server side failures in the error returned to the client.
0.20.1 (2015-11-12)¶
- Fix bug which prevented requests from being retried if the candidate connection was previously terminated.
0.20.0 (2015-11-10)¶
- Support thriftrw 1.0.
- Drop explicit dependency on the
futures
library.
0.19.0 (2015-11-06)¶
- Add tchannel version & language information into init message header when initialize connections between TChannel instances.
0.18.3 (2015-11-03)¶
- Reduced Hyperbahn advertisement per-request timeout to 2 seconds.
- Removed an unncessary exception log for connection failures.
0.18.2 (2015-10-28)¶
- Reduced Hyperbahn advertisement failures to warnings.
0.18.1 (2015-10-28)¶
- Improved performance of peer selection logic.
- Fixed a bug which caused the message ID and tracing for incoming error frames to be ignored.
- Prefer using incoming connections on peers instead of outgoing connections.
0.18.0 (2015-10-20)¶
- Deprecated warnings will now sound for
tchannel.thrift.client_for
,tchannel.thrift_request_builder
, andtchannel.tornado.TChannel
- these APIs will be removed soon - be sure to move totchannel.thrift.load
in conjunction withtchannel.TChannel
. - Added singleton facility for maintaining a single TChannel instance per thread.
See
tchannel.singleton.TChannel
,tchannel.sync.singleton.TChannel
, or check the guide for an example how of how to use. Note this feature is optional. - Added Thrift support to
tcurl.py
and re-worked the script’s arguments. - Specify which request components to match on with VCR, for example, ‘header’,
‘body’, etc. See
tchannel.testing.vcr.use_cassette
. - Removed
tchannel.testing.data
module. - Changed minimum required version of Tornado to 4.2.
tchannel.tornado.TChannel.close
is no longer a coroutine.- BREAKING - headers for JSON handlers are not longer JSON blobs but are instead maps of strings to strings. This mirrors behavior for Thrift handlers.
- Fixed bug that caused server to continue listening for incoming connections despite closing the channel.
- Explicit destinations for
ThriftArgScheme
may now be specified on a per-request basis by using thehostport
keyword argument. - Only listen on IPv4, until official IPv6 support arrives.
0.17.11 (2015-10-19)¶
- Fix a bug that caused
after_send_error
event to never be fired. - Request tracing information is now propagated to error responses.
0.17.10 (2015-10-16)¶
- Support thriftrw 0.5.
0.17.9 (2015-10-15)¶
- Fix default timeout incorrectly set to 16 minutes, now 30 seconds.
0.17.8 (2015-10-14)¶
- Revert timeout changes from 0.17.6 due to client incompatibilities.
0.17.7 (2015-10-14)¶
- Network failures while connecting to randomly selected hosts should be retried with other hosts.
0.17.6 (2015-10-14)¶
- Fixed an issue where timeouts were being incorrectly converted to seconds.
0.17.5 (2015-10-12)¶
- Set default checksum to
CRC32C
.
0.17.4 (2015-10-12)¶
- Updated
vcr
to usethriftrw
-generated code. This should resolve some unicode errors during testing withvcr
.
0.17.3 (2015-10-09)¶
- Fixed uses of
add_done_callback
that should have beenadd_future
. This was preventing propper request/response interleaving. - Added support for
thriftrw
0.4.
0.17.2 (2015-09-18)¶
- VCR no longer matches on hostport to better support ephemeral ports.
- Fixed a bug with thriftrw where registering an endpoint twice could fail.
0.17.1 (2015-09-17)¶
- Made “service” optional for
thrift.load()
. The first argument should be a path, but backwards compatibility is provided for 0.17.0.
0.17.0 (2015-09-14)¶
- It is now possible to load Thrift IDL files directly with
tchannel.thrift.load
. This means that the code generation step using the Apache Thrift compiler can be skipped entirely. Check the API documentation for more details. - Accept host file in advertise:
TChannel.advertise()
now accepts a parameter,router_file
that contains a JSON stringified format of the router list. - Add
TChannel.is_listening
method to return whether the tchannel instance is listening or not.
0.16.10 (2015-10-15)¶
- Fix default timeout incorrectly set to 16 minutes, now 30 seconds.
0.16.9 (2015-10-15)¶
- Network failures while connecting to randomly selected hosts should be retried with other hosts.
0.16.8 (2015-10-14)¶
- Revert timeout changes from 0.16.7 due to client incompatibilities.
0.16.7 (2015-10-14)¶
- Fixed an issue where timeouts were being incorrectly converted to seconds.
0.16.6 (2015-09-14)¶
- Fixed a bug where Zipkin traces were not being propagated correctly in
services using the
tchannel.TChannel
API.
0.16.5 (2015-09-09)¶
- Actually fix status code being unset in responses when using the Thrift scheme.
- Fix request TTLs not being propagated over the wire.
0.16.4 (2015-09-09)¶
- Fix bug where status code was not being set correctly on call responses for application errors when using the Thrift scheme.
0.16.3 (2015-09-09)¶
- Make
TChannel.listen
thread-safe and idempotent.
0.16.2 (2015-09-04)¶
- Fix retry_limit in TChannel.call not allowing 0 retries.
0.16.1 (2015-08-27)¶
- Fixed a bug where the ‘not found’ handler would incorrectly return serialization mismatch errors..
- Fixed a bug which prevented VCR support from working with the sync client.
- Fixed a bug in VCR that prevented it from recording requests made by the sync
client, and requests made with
hostport=None
. - Made
client_for
compatible withtchannel.TChannel
. - Brought back
tchannel.sync.client_for
for backwards compatibility.
0.16.0 (2015-08-25)¶
- Introduced new server API through methods
tchannel.TChannel.thrift.register
,tchannel.TChannel.json.register
, andtchannel.TChannel.raw.register
- when these methods are used, endpoints are passed atchannel.Request
object, and are expected to return atchannel.Response
object or just a response body. The deprecatedtchannel.tornado.TChannel.register
continues to function how it did before. Note the breaking change to the top-level TChannel on the next line. - Fixed a crash that would occur when forking with an unitialized
TChannel
instance. - Add
hooks
property in thetchannel.TChannel
class. - BREAKING -
tchannel.TChannel.register
no longer has the same functionality astchannel.tornado.TChannel.register
, instead it exposes the new server API. See the upgrade guide for details. - BREAKING - remove
retry_delay
option in thetchannel.tornado.send
method. - BREAKING - error types have been reworked significantly. In particular,
the all-encompassing
ProtocolError
has been replaced with more granualar/actionable exceptions. See the upgrade guide for more info. - BREAKING - Remove third
proxy
argument from the server handler interface. - BREAKING -
ZipkinTraceHook
is not longer registered by default. - BREAKING -
tchannel.sync.client.TChannelSyncClient
replaced withtchannel.sync.TChannel
.
0.15.2 (2015-08-07)¶
- Raise informative and obvious
ValueError
when anything but a map[string]string is passed as headers to theTChannel.thrift
method. - First param, request, in
tchannel.thrift
method is required.
0.15.1 (2015-08-07)¶
- Raise
tchannel.errors.ValueExpectedError
when calling a non-void Thrift procedure that returns no value.
0.15.0 (2015-08-06)¶
- Introduced new top level
tchannel.TChannel
object, with new request methodscall
,raw
,json
, andthrift
. This will eventually replace the akwardrequest
/send
calling pattern. - Introduced
tchannel.thrift_request_builder
function for creating a request builder to be used with thetchannel.TChannel.thrift
function. - Introduced new simplified examples under the
examples/simple
directory, moved the Guide’s examples toexamples/guide
, and deleted the remaining examples. - Added ThriftTest.thrift and generated Thrift code to
tchannel.testing.data
for use with examples and playing around with TChannel. - Fix JSON arg2 (headers) being returned a string instead of a dict.
0.14.0 (2015-08-03)¶
- Implement VCR functionality for outgoing requests. Check the documentation
for
tchannel.testing.vcr
for details. - Add support for specifying fallback handlers via
TChannel.register
by specifyingTChannel.fallback
as the endpoint. - Fix bug in
Response
wherecode
expected an object instead of an integer. - Fix bug in
Peer.close
where a future was expected instead ofNone
.
0.13.0 (2015-07-23)¶
- Add support for specifying transport headers for Thrift clients.
- Always pass
shardKey
for TCollector tracing calls. This fixes Zipkin tracing for Thrift clients.
0.12.0 (2015-07-20)¶
- Add
TChannel.is_listening()
to determine iflisten
has been called. - Calling
TChannel.listen()
more than once raises atchannel.errors.AlreadyListeningError
. TChannel.advertise()
will now automatically start listening for connections iflisten()
has not already been called.- Use
threadloop==0.4
. - Removed
print_arg
.
0.11.2 (2015-07-20)¶
- Fix sync client’s advertise - needed to call listen in thread.
0.11.1 (2015-07-17)¶
- Fix sync client using
0.0.0.0
host which gets rejected by Hyperbahn during advertise.
0.11.0 (2015-07-17)¶
- Added advertise support to sync client in
tchannel.sync.TChannelSyncClient.advertise
. - BREAKING - renamed
router
argument torouters
intchannel.tornado.TChannel.advertise
.
0.10.3 (2015-07-13)¶
- Support PyPy 2.
- Fix bugs in
TChannel.advertise
.
0.10.2 (2015-07-13)¶
- Made
TChannel.advertise
retry on all exceptions.
0.10.1 (2015-07-10)¶
- Previous release was broken with older versions of pip.
0.10.0 (2015-07-10)¶
- Add exponential backoff to
TChannel.advertise
. - Make transport metadata available under
request.transport
on the server-side.
0.9.1 (2015-07-09)¶
- Use threadloop 0.3.* to fix main thread not exiting when
tchannel.sync.TChannelSyncClient
is used.
0.9.0 (2015-07-07)¶
- Allow custom handlers for unrecognized endpoints.
- Released
tchannel.sync.TChannelSyncClient
andtchannel.sync.thrift.client_for
.
0.8.5 (2015-06-30)¶
- Add port parameter for
TChannel.listen
.
0.8.4 (2015-06-17)¶
- Fix bug where False and False-like values were being treated as None in Thrift servers.
0.8.3 (2015-06-15)¶
- Add
as
attribute to the response header.
0.8.2 (2015-06-11)¶
- Fix callable
traceflag
being propagated to the serializer. - Fix circular imports.
- Fix
TimeoutError
retry logic.
0.8.1 (2015-06-10)¶
- Initial release.
Upgrade Guide¶
Migrating to a version of TChannel with breaking changes? This guide documents what broke and how to safely migrate to newer versions.
From 0.20 to 0.21¶
tchannel.thrift.register
returns the original function as-is instead of the wrapped version. This allows writing unit tests that call the handler function directly.Previously, if you used the
tchannel.thrift.register
decorator to register a Thrift endpoint and then called that function directly from a test, it would return aResponse
object if the call succeeded or failed with an expected exception (defined in the Thrift IDL). For example,# service KeyValue { # string getValue(1: string key) # throws (1: ValidationError invalid) # } @tchannel.thrift.register(kv.KeyValue) def getValue(request): key = request.body.key if key == 'invalid': raise kv.ValidationError() result = # ... return result response = getValue(make_request(key='invalid')) if response.body.invalid: # ... else: result = response.body.success
With 0.21, we have changed
tchannel.thrift.register
to return the unmodified function so that you can call it directly and it will behave as expected.@tchannel.thrift.register(kv.KeyValue) def getValue(request): # ... try: result = getValue(make_request(key='invalid')) except kv.ValidationError: # ...
From 0.19 to 0.20¶
- No breaking changes.
From 0.18 to 0.19¶
- No breaking changes.
From 0.17 to 0.18¶
request.headers
in a JSON handler is no longer a JSON blob. Instead it is a dictionary mapping strings to strings. This matches the Thrift implementation. If your headers include richer types like lists or ints, you’ll need to coordinate with your callers to no longer pass headers as JSON blobs. The same applies to JSON requsts; rich headers will now fail to serialize.If you were accessing
request_cls
orresponse_cls
directly from a service method in a module generated bytchannel.thrift.load
, you can no longer do that. Therequest_cls
andresponse_cls
attributes are internal details of the implementation and have been changed to protected. You should only ever use the service method directly.Before:
my_service.doSomething.request_cls(..)
After:
my_service.doSomething(..)
Note that
request_cls
gives you just an object containing the method arguments. It does not include any of the other information needed to make the request. So if you were using it to make requests, it wouldn’t have worked anyway.
From 0.16 to 0.17¶
- No breaking changes.
From 0.15 to 0.16¶
tchannel.TChannel.register
no longer mimickstchannel.tornado.TChannel.register
, instead it exposes the new server API like so:Before:
from tchannel.tornado import TChannel tchannel = TChannel('my-service-name') @tchannel.register('endpoint', 'json') def endpoint(request, response, proxy): response.write({'resp': 'body'})
After:
from tchannel import TChannel tchannel = TChannel('my-service-name') @tchannel.json.register def endpoint(request): return {'resp': 'body'} # Or, if you need to return headers with your response: from tchannel import Response return Response({'resp': 'body'}, {'header': 'foo'})
TChannelSyncClient
has been replaced withtchannel.sync.TChannel
. This new synchronous client has been significantly re-worked to more closely match the asynchronousTChannel
API.tchannel.sync.thrift.client_for
has been removed andtchannel.thrift_request_builder
should be used instead (tchannel.thrift.client_for
still exists for backwards compatibility but is not recommended). This new API allows specifying headers, timeouts, and retry behavior with Thrift requests.Before:
from tchannel.sync import TChannelSyncClient from tchannel.sync.thrift import client_for from generated.thrift.code import MyThriftService tchannel_thrift_client = client_for('foo', MyThriftService) tchannel = TChannelSyncClient(name='bar') future = tchannel_thrift_client.someMethod(...) result = future.result()
After:
from tchannel import thrift_request_builder from tchannel.sync import TChannel from tchannel.retry import CONNECTION_ERROR_AND_TIMEOUT from generated.thrift.code import MyThriftService tchannel_thrift_client = thrift_request_builder( service='foo', thrift_module=MyThriftService, ) tchannel = TChannel(name='bar') future = tchannel.thrift( tchannel_thrift_client.someMethod(...) headers={'foo': 'bar'}, retry_on=CONNECTION_ERROR_AND_TIMEOUT, timeout=1000, ) result = future.result()
from tchannel.tornado import TChannel
is deprecated.Removed
retry_delay
option fromtchannel.tornado.peer.PeerClientOperation.send
method.Before:
tchannel.tornado.TChannel.request.send(retry_delay=300)
After: no more
retry_delay
intchannel.tornado.TChannel.request.send()
If you were catching
ProtocolError
you will need to catch a more specific type, such asTimeoutError
,BadRequestError
,NetworkError
,UnhealthyError
, orUnexpectedError
.If you were catching
AdvertiseError
, it has been replaced byTimeoutError
.If you were catching
BadRequest
, it may have been masking checksum errors and fatal streaming errors. These are now raised asFatalProtocolError
, but in practice should not need to be handled when interacting with a well-behaved TChannel implementation.TChannelApplicationError
was unused and removed.Three error types have been introduced to simplify retry handling:
NotRetryableError
(for requests should never be retried),RetryableError
(for requests that are always safe to retry), andMaybeRetryableError
(for requests that are safe to retry on idempotent endpoints).
From 0.14 to 0.15¶
- No breaking changes.
From 0.13 to 0.14¶
- No breaking changes.
From 0.12 to 0.13¶
- No breaking changes.
From 0.11 to 0.12¶
- Removed
print_arg
. Userequest.get_body()
instead.
From 0.10 to 0.11¶
Renamed
tchannel.tornado.TChannel.advertise
argumentrouter
torouters
. Since this is a required arg and the first positional arg, only clients who are using as kwarg will break.Before:
tchannel.advertise(router=['localhost:21300'])
After:
tchannel.advertise(routers=['localhost:21300'])