Merge pull request #59 from yuvipanda/fixes

Ensure `credHelpers` key exists in ~/.docker/config.json
berkeley-dsep-infra · Mar 27, 2020 · 5071b96 · 5071b96
2 parents 63c668d + 1b57576
commit 5071b96
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/hubploy/auth.py b/hubploy/auth.py
@@ -88,8 +88,9 @@ def registry_auth_aws(deployment, project, zone, service_key):
         with open(docker_config, 'r') as f:
             config = json.load(f)
     else:
-        config = {'credHelpers': {}}
-    config['credHelpers'][registry] = 'ecr-login'
+        config = {}
+
+    config.setdefault('credHelpers', {})[registry] = 'ecr-login'
     with open(docker_config, 'w') as f:
         json.dump(config, f)