Merge pull request #40 from Fgruntjes/initial-build

deploy test
Fgruntjes · Oct 12, 2022 · 2e51093 · 2e51093
2 parents b46ad13 + ad1c71f
commit 2e51093
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/deploy.yaml b/.github/workflows/deploy.yaml
@@ -73,7 +73,8 @@ jobs:
         -   uses: mshick/fast-envsubst@v1
             env:
                 SERVICE: ${{ matrix.project.service }}-${{ env.environment_lowercase }}
-                DEPLOY_ENV: ${{ steps.environment.outputs.result }}
+                DEPLOY_ENV: ${{ env.environment_lowercase }}
+                DOTNET_ENVIRONMENT: ${{ steps.environment.outputs.result }}
                 IMAGE: gcr.io/${{ secrets.GOOGLE_PROJECT_ID }}/${{ matrix.project.service }}:${{env.slug}}
                 Database__ConnectionString: ${{ secrets.DATABASE__CONNECTIONSTRING }}
                 Database__DatabaseName: ${{steps.environment.outputs.result}}

diff --git a/App.Function.Banktransaction.Import/service.yaml b/App.Function.Banktransaction.Import/service.yaml
@@ -1,7 +1,7 @@
 apiVersion: serving.knative.dev/v1
 kind: Service
 metadata:
-  name: ${SERVICE}
+  name: ${SERVICE}-${DEPLOY_ENV}
   labels:
     environment: ${DEPLOY_ENV}
 spec:
@@ -12,7 +12,7 @@ spec:
         - image: ${IMAGE}
           env:
             - name: DOTNET_ENVIRONMENT
-              value: ${DEPLOY_ENV}
+              value: ${DOTNET_ENVIRONMENT}
             - name: Database__ConnectionString
               value: ${Database__ConnectionString}
             - name: Database__DatabaseName