Merge pull request #14 from andresionek91/raise-error-docker-push-ecr

Raise error before jobs getting stuck in  CREATING STACK cfn-airflow-…

Author: andresionek91

deploy_docker.py

@@ -16,6 +16,9 @@
 IMAGE_TAG = 'latest'
 
 
+class DockerException(Exception): pass
+
+
 def connect_to_ecr():
     client = boto3.client('ecr')
     token = client.get_authorization_token()
@@ -47,6 +50,10 @@ def tag_and_push_to_ecr(image, tag):
     ecr_repo_name = '{}/{}'.format(registry.replace('https://', ''), ECR_REPO_NAME)
     image.tag(ecr_repo_name, tag)
     push_log = docker_client.images.push(ecr_repo_name, tag=tag)
+
+    if 'errorDetail' in push_log:
+        logging.error(push_log)
+        raise DockerException()
     logging.info(push_log)
 
 

service.yml

@@ -97,8 +97,6 @@ metadataDb:
   parameters:
     maxConnections: 100
 
-
-
 celeryBackend:
   port: 6379
   azMode: single-az