Skip to content
This repository was archived by the owner on May 23, 2024. It is now read-only.

Commit bd41141

Browse files
author
Jinpeng Qi
committed
fix sanity test failure
1 parent 1d28892 commit bd41141

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

docker/build_artifacts/sagemaker/python_service.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,8 +69,10 @@ def __init__(self):
6969
# during the _handle_load_model_post()
7070
self.model_handlers = {}
7171
else:
72-
self._tfs_grpc_ports = self._reconstruct_sagemaker_selected_ports(TFS_GRPC_SELECTED_PORTS)
73-
self._tfs_rest_ports = self._reconstruct_sagemaker_selected_ports(TFS_REST_SELECTED_PORTS)
72+
self._tfs_grpc_ports = self._reconstruct_sagemaker_selected_ports(
73+
TFS_GRPC_SELECTED_PORTS)
74+
self._tfs_rest_ports = self._reconstruct_sagemaker_selected_ports(
75+
TFS_REST_SELECTED_PORTS)
7476

7577
self._channels = {}
7678
for grpc_port in self._tfs_grpc_ports:

docker/build_artifacts/sagemaker/serve.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -97,9 +97,9 @@ def __init__(self):
9797
# select non-overlapping grpc and rest ports based on tfs instance count
9898
# record the init and end value in order to reconstruct them in python service
9999
self._tfs_grpc_selected_ports = "{}-{}".format(low,
100-
low + 2 * self._tfs_instance_count)
100+
low + 2 * self._tfs_instance_count)
101101
self._tfs_rest_selected_ports = "{}-{}".format(low + 1,
102-
low + 2 * self._tfs_instance_count + 1)
102+
low + 2 * self._tfs_instance_count + 1)
103103
for i in range(self._tfs_instance_count):
104104
self._tfs_grpc_port.append(str(low + 2 * i))
105105
self._tfs_rest_port.append(str(low + 2 * i + 1))
@@ -199,7 +199,8 @@ def _setup_gunicorn(self):
199199
"python_service:app").format(self._gunicorn_worker_class,
200200
self._gunicorn_workers, self._gunicorn_threads,
201201
python_path_option, ",".join(python_path_content),
202-
self._tfs_grpc_selected_ports, self._tfs_rest_selected_ports,
202+
self._tfs_grpc_selected_ports,
203+
self._tfs_rest_selected_ports,
203204
self._tfs_enable_multi_model_endpoint,
204205
self._sagemaker_port_range,
205206
self._tfs_wait_time_seconds)

0 commit comments

Comments
 (0)