change: update no-p2 and no-p3 regions. (#230)

Add placeholder in test commands for cpu-instance-type and aws-id.

Author: lianyiding

buildspec-release.yml

@@ -73,7 +73,8 @@ phases:
             "dest": ["1.13.1-gpu-py3", "1.13-gpu-py3", "1.13.1-gpu-py3-'${CODEBUILD_BUILD_ID#*:}'"]
           }],
           "test": [
-            "IGNORE_COVERAGE=- tox -e py36 -- -m deploy_test test/integration/sagemaker -n 4 --region {region} --account-id 520713654638 --docker-base-name sagemaker-tensorflow-scriptmode --framework-version 1.13.1 --processor cpu,gpu --py-version 2,3"
+            "IGNORE_COVERAGE=- tox -e py36 -- -m deploy_test test/integration/sagemaker -n 4 --region {region} --account-id {aws-id} --instance-type {cpu-instance-type} --docker-base-name sagemaker-tensorflow-scriptmode --framework-version 1.13.1 --processor cpu --py-version 2,3",
+            "IGNORE_COVERAGE=- tox -e py36 -- -m deploy_test test/integration/sagemaker -n 4 --region {region} --account-id {aws-id} --docker-base-name sagemaker-tensorflow-scriptmode --framework-version 1.13.1 --processor gpu --py-version 2,3"
           ]
         }]' > deployments.json
 

test/integration/__init__.py

@@ -21,6 +21,26 @@
 RESOURCE_PATH = os.path.join(os.path.dirname(__file__), '..', 'resources')
 
 # these regions have some p2 and p3 instances, but not enough for automated testing
-NO_P2_REGIONS = ['ca-central-1', 'eu-central-1', 'eu-west-2', 'us-west-1']
-NO_P3_REGIONS = ['ap-southeast-1', 'ap-southeast-2', 'ap-south-1', 'ca-central-1',
-                 'eu-central-1', 'eu-west-2', 'us-west-1']
+NO_P2_REGIONS = [
+    'ca-central-1',
+    'eu-central-1',
+    'eu-west-2',
+    'us-west-1',
+    'eu-west-3',
+    'eu-north-1',
+    'sa-east-1',
+    'ap-east-1'
+]
+NO_P3_REGIONS = [
+    'ap-southeast-1',
+    'ap-southeast-2',
+    'ap-south-1',
+    'ca-central-1',
+    'eu-central-1',
+    'eu-west-2',
+    'us-west-1'
+    'eu-west-3',
+    'eu-north-1',
+    'sa-east-1',
+    'ap-east-1'
+]

tox.ini

@@ -34,6 +34,7 @@ ignore =
     FI15,
     FI16,
     FI17,
+    FI18,  # __future__ import "annotations" missing -> check only Python 3.7 compatible
     FI50,
     FI51,
     FI52,