From f816677f32f054497730c594d4f92c4a2ee9f100 Mon Sep 17 00:00:00 2001 From: Karim Nakad Date: Wed, 25 Dec 2019 09:52:37 -0800 Subject: [PATCH 1/2] change: reduce test's describe frequency to eliminate throttling error. --- tests/integ/test_data_capture_config.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/integ/test_data_capture_config.py b/tests/integ/test_data_capture_config.py index 94b7db3fcc..fe236b0716 100644 --- a/tests/integ/test_data_capture_config.py +++ b/tests/integ/test_data_capture_config.py @@ -74,11 +74,11 @@ def test_enabling_data_capture_on_endpoint_shows_correct_data_capture_status( predictor.enable_data_capture() # Wait for endpoint to finish updating - # Endpoint update takes ~7min. 50 retries * 30s sleeps = 25min timeout + # Endpoint update takes ~7min. 25 retries * 60s sleeps = 25min timeout for _ in retries( - max_retry_count=50, + max_retry_count=25, exception_message_prefix="Waiting for 'InService' endpoint status", - seconds_to_sleep=30, + seconds_to_sleep=60, ): new_endpoint = sagemaker_session.sagemaker_client.describe_endpoint( EndpointName=predictor.endpoint From 6decf34c7e4fbbd176ef3567bbf0cf6659c9f6d3 Mon Sep 17 00:00:00 2001 From: Karim Nakad Date: Wed, 25 Dec 2019 10:19:37 -0800 Subject: [PATCH 2/2] change: fixing 2 other occurrences --- tests/integ/test_data_capture_config.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/integ/test_data_capture_config.py b/tests/integ/test_data_capture_config.py index fe236b0716..78c17a2f26 100644 --- a/tests/integ/test_data_capture_config.py +++ b/tests/integ/test_data_capture_config.py @@ -159,11 +159,11 @@ def test_disabling_data_capture_on_endpoint_shows_correct_data_capture_status( predictor.disable_data_capture() # Wait for endpoint to finish updating - # Endpoint update takes ~7min. 50 retries * 30s sleeps = 25min timeout + # Endpoint update takes ~7min. 25 retries * 60s sleeps = 25min timeout for _ in retries( - max_retry_count=50, + max_retry_count=25, exception_message_prefix="Waiting for 'InService' endpoint status", - seconds_to_sleep=30, + seconds_to_sleep=60, ): new_endpoint = sagemaker_session.sagemaker_client.describe_endpoint( EndpointName=predictor.endpoint @@ -228,11 +228,11 @@ def test_updating_data_capture_on_endpoint_shows_correct_data_capture_status( ) # Wait for endpoint to finish updating - # Endpoint update takes ~7min. 50 retries * 30s sleeps = 25min timeout + # Endpoint update takes ~7min. 25 retries * 60s sleeps = 25min timeout for _ in retries( - max_retry_count=50, + max_retry_count=25, exception_message_prefix="Waiting for 'InService' endpoint status", - seconds_to_sleep=30, + seconds_to_sleep=60, ): new_endpoint = sagemaker_session.sagemaker_client.describe_endpoint( EndpointName=predictor.endpoint