Merge pull request #56 from populationgenomics/prod_deploy-show-full-…

…batch-url [prod deploy]: show batch url in the terminal
daniel-goldstein · Feb 26, 2021 · a4452bb · a4452bb
2 parents 32452b0 + 127237c
commit a4452bb
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/ci/ci/ci.py b/ci/ci/ci.py
@@ -148,9 +148,10 @@ async def prod_deploy(request, unused_userdata):
         0, FQBranch.from_short_str('populationgenomics/hail:main'), True
     )
     watched_branch.sha = 'HEAD'
-    await watched_branch._start_deploy(request.app['batch_client'], steps)
+    await watched_branch._start_deploy(app['batch_client'], steps)
 
-    url = deploy_config.external_url('ci', '/batches')
+    batch_id = watched_branch.deploy_batch.id
+    url = deploy_config.external_url('ci', f'/batches/{batch_id}')
     return web.Response(text=f'{url}\n')