Skip to content

Commit a630478

Browse files
fix: Increase timeout for starting model server to 10 minutes (#92)
* Increase timeout for starting model server to 10 minutes * fix: Increase timeout for starting model server to 10 minutes * Fix tabs * Trigger build * Modify test to wait for 10 minutes * Sleep initially for 3 sec * Add later sleep within the except block * Print number of attempts * Print number of attempts * Disable output capturing * Print more debuggin info * Add wait_fixed for retry * Trigger build * Revert test changes Co-authored-by: David Thomas <[email protected]>
1 parent 7fcb805 commit a630478

File tree

5 files changed

+15
-5
lines changed

5 files changed

+15
-5
lines changed

buildspec-release.yml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
version: 0.2
22

3+
env:
4+
variables:
5+
LC_ALL: "C.UTF-8"
6+
LANG: "C.UTF-8"
7+
38
phases:
49
pre_build:
510
commands:

buildspec.yml

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
version: 0.2
22

3+
env:
4+
variables:
5+
LC_ALL: "C.UTF-8"
6+
LANG: "C.UTF-8"
7+
38
phases:
49
pre_build:
510
commands:
@@ -30,4 +35,4 @@ phases:
3035
- cd ../..
3136

3237
# run local integration tests
33-
- IGNORE_COVERAGE=- tox -e py27,py36,py37 -- test/integration/local
38+
- IGNORE_COVERAGE=- tox -e py27,py36,py37 -- -s test/integration/local

src/sagemaker_inference/model_server.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -189,8 +189,8 @@ def _install_requirements():
189189
raise ValueError("failed to install required packages")
190190

191191

192-
# retry for 10 seconds
193-
@retry(stop_max_delay=10 * 1000)
192+
# retry for 10 minutes
193+
@retry(wait_fixed=1000, stop_max_delay=10 * 60 * 1000)
194194
def _retrieve_mms_server_process():
195195
mms_server_processes = list()
196196

test/integration/local/test_dummy_multi_model.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ def container():
3838
proc = subprocess.Popen(command.split(), stdout=sys.stdout, stderr=subprocess.STDOUT)
3939

4040
attempts = 0
41-
while attempts < 5:
41+
while attempts < 10:
4242
time.sleep(3)
4343
try:
4444
requests.get(PING_URL)

test/integration/local/test_mxnet_multi_model.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ def container():
3838
proc = subprocess.Popen(command.split(), stdout=sys.stdout, stderr=subprocess.STDOUT)
3939

4040
attempts = 0
41-
while attempts < 5:
41+
while attempts < 10:
4242
time.sleep(3)
4343
try:
4444
requests.get(PING_URL)

0 commit comments

Comments
 (0)