Infrastructure/logto
0
Fork 0
mirror of https://github.com/logto-io/logto.git synced 2024-12-16 20:26:19 -05:00
Code Issues Activity

chore: allow setting appinsights string in core Dockerfile (#3717)

Browse source 
since we are tracking the main flow for Logto Cloud, it is required to pass
an ApplicationInsights connection string to make it works.
This commit is contained in:
Gao Sun 2023-04-19 18:05:43 +08:00 committed by GitHub
parent 9f50cc4a80
commit 4b6ad4c8c5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
.github/workflows/release.yml vendored
View file

@ -77,6 +77,7 @@ jobs:
labels: ${{ steps.meta.outputs.labels }} labels: ${{ steps.meta.outputs.labels }}
build-args: | build-args: |
additional_connector_args=--cloud additional_connector_args=--cloud
applicationinsights_connection_string=${{ (inputs.target || 'dev') == 'dev' && secrets.APPLICATIONINSIGHTS_CONNECTION_STRING || secrets.APPLICATIONINSIGHTS_CONNECTION_STRING_PROD }}
- name: Setup Depot - name: Setup Depot
if: ${{ startsWith(github.ref, 'refs/tags/') }} if: ${{ startsWith(github.ref, 'refs/tags/') }}

3
Dockerfile
View file

@ -16,6 +16,9 @@ COPY . .
### Install dependencies and build ### ### Install dependencies and build ###
RUN node .scripts/update-parcelrc.js RUN node .scripts/update-parcelrc.js
RUN pnpm i RUN pnpm i
ARG applicationinsights_connection_string
ENV APPLICATIONINSIGHTS_CONNECTION_STRING=${applicationinsights_connection_string}
RUN pnpm -r build RUN pnpm -r build
### Add official connectors ### ### Add official connectors ###