From 04be01e1a2271fda45b36f84f84a67475d866471 Mon Sep 17 00:00:00 2001 From: Philip Potter Date: Mon, 25 Nov 2019 16:15:39 +0000 Subject: [PATCH] make manifests match reality The manifests for spotlight set one name for the app, but etc/deploy.sh overrides this to a different name. This is confusing and has left us with multiple copies of spotlight running, bound to the same route. To avoid this footgun in future, I think we should have the manifests use the actual app name that we actually deploy. I hope this is not a controversial opinion. --- manifest.production.yml | 2 +- manifest.staging.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/manifest.production.yml b/manifest.production.yml index 4dedbd0d4..ba46c9a0d 100644 --- a/manifest.production.yml +++ b/manifest.production.yml @@ -2,7 +2,7 @@ services: - logit-ssl-drain applications: - - name: performance-platform-spotlight + - name: performance-platform-spotlight-production memory: 1G instances: 6 buildpack: nodejs_buildpack diff --git a/manifest.staging.yml b/manifest.staging.yml index dcedc243b..35c683e1b 100644 --- a/manifest.staging.yml +++ b/manifest.staging.yml @@ -1,6 +1,6 @@ --- applications: - - name: performance-platform-spotlight + - name: performance-platform-spotlight-staging memory: 1G instances: 2 buildpack: nodejs_buildpack