Skip to content

Commit 934c9ae

Browse files
authored
infra: Make docker folder read only, remove unused tests. (#384)
* Remove container tests. * Rename buildspec.yml -> buildspec-container-pr.yml. * Rename buildspec-toolkit.yml -> buildspec.yml. * Rename test-toolkit/ -> test/. * Refactor tests.
1 parent c9e7b4b commit 934c9ae

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

60 files changed

+139
-2721
lines changed

.flake8

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
[flake8]
2-
application_import_names = image_utils, integration, sagemaker_tensorflow_container, test, test-toolkit, timeout, utils
2+
application_import_names = image_utils, integration, sagemaker_tensorflow_container, test, timeout, utils
33
import-order-style = google

buildspec-container-pr.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
version: 0.2
2+
3+
phases:
4+
pre_build:
5+
commands:
6+
- PR_NUM=$(echo $CODEBUILD_SOURCE_VERSION | grep -o '[0-9]\+')
7+
- echo 'Pull request number:' $PR_NUM '. No value means this build is not from pull request.'
8+
9+
build:
10+
commands:
11+
12+
- error_cmd="echo 'In order to make changes to the docker files, please, use https://github.com/aws/deep-learning-containers repository.' && exit 1"
13+
- execute-command-if-has-matching-changes "$error_cmd" "docker/"

buildspec-toolkit.yml

Lines changed: 0 additions & 90 deletions
This file was deleted.

buildspec-unit.yml

Lines changed: 0 additions & 8 deletions
This file was deleted.

0 commit comments

Comments
 (0)