AWS ECR Auth bug fixed

ezra-buckingham · Jan 19, 2023 · 53a3b05 · 53a3b05
1 parent 7008ab7
commit 53a3b05
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/core/__init__.py b/core/__init__.py
@@ -480,6 +480,7 @@ def validate_credentials(ctx_obj, check_containers=True):
                     check_for_required_value('container_registry')
                     check_for_required_value('container_registry_username', hide_input=True)
                     check_for_required_value('container_registry_password', hide_input=True)
+                    check_for_required_value('container_registry_strategy', hide_input=False)
                     container_registry_credentials_checked = True
 
     ctx_obj['required_providers'] = []

diff --git a/terry.py b/terry.py
@@ -53,7 +53,7 @@
 @click.option('-cRP', '--container_registry_password', help='''
     Password used to authenticate to the container registry (required if deploying containers)
     ''')
-@click.option('-cRS', '--container_registry_strategy', default='default', help='''
+@click.option('-cRS', '--container_registry_strategy', help='''
     Strategy to use for container registry auth (use "aws" when using AWS ECR or "default" for standard username / password auth)
     ''')
 @click.option('-eS', '--elastic_server', help='''