fix(controller): use build image for publishing releases

Author: Matthew Fisher

api/views.py

@@ -493,7 +493,10 @@ def create(self, request, *args, **kwargs):
     def post_save(self, build, created=False):
         if created:
             release = build.app.release_set.latest()
-            new_release = release.new(build.owner, build=build)
+            source_version = 'latest'
+            if build.sha:
+                source_version = 'git-{}'.format(build.sha)
+            new_release = release.new(build.owner, build=build, source_version=source_version)
             initial = True if build.app.structure == {} else False
             try:
                 build.app.deploy(build.owner, new_release, initial=initial)

registry/private.py

@@ -59,7 +59,7 @@ def publish_release(source, config, target):
     image['id'] = _new_id()
     config['DEIS_APP'] = target_image
     config['DEIS_RELEASE'] = target_tag
-    image['config']['Env'] = _construct_env(image['config']['Env'], config)
+    image['config']['Env'] = _construct_env(config)
     # update and tag the new image
     _commit(target_image, image, _empty_tar_archive(), target_tag)
 
@@ -162,17 +162,10 @@ def _put_tag(image_id, repository_path, tag):
 
 # utility functions
 
-def _construct_env(env, config):
+def _construct_env(config):
     "Update current environment with latest config"
     new_env = []
-    # see if we need to update existing ENV vars
-    for e in env:
-        k, v = e.split('=', 1)
-        if k in config:
-            # update values defined by config
-            v = config.pop(k)
-        new_env.append("{}={}".format(encode(k), encode(v)))
-    # add other config ENV items
+    # add config ENV items
     for k, v in config.items():
         new_env.append("{}={}".format(encode(k), encode(v)))
     return new_env