Skip to content

Commit

Permalink
Fix CICD
Browse files Browse the repository at this point in the history
  • Loading branch information
benc-uk committed Sep 13, 2022
1 parent a633a37 commit 3b83173
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/cd-release-aks.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ on:
# Note. Required secrets: CLUSTER_KUBECONFIG

env:
CLUSTER_NAME: benc
CLUSTER_NAME: sandbox
HELM_RELEASE: nodejs
HELM_NAMESPACE: demoapps
INGRESS_DNS_HOST: nodejs-demoapp.kube.benco.io
Expand Down
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -171,6 +171,7 @@ If running in an Azure Web App, all of these values can be injected as applicati
| WEATHER_API_KEY | _none_ | OpenWeather API key. [Info here](https://openweathermap.org/api) |
| AAD_APP_ID | _none_ | Client ID of app registered in Azure AD |
| DISABLE_METRICS | _none_ | Set to truthy value if you want to switch off Prometheus metrics |
| REDIS_SESSION_HOST | _none_ | Point to a Redis host to hold/persist session cache |

## Deployment

Expand Down
4 changes: 3 additions & 1 deletion deploy/kubernetes/aks-live.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ image:
tag: latest
pullPolicy: Always

replicaCount: 3

service:
targetPort: 3000

Expand All @@ -13,7 +15,7 @@ secretEnv:

env:
AAD_APP_ID: 129e29d2-6c0a-4dbf-849e-320a3aa93d52
# APPINSIGHTS_INSTRUMENTATIONKEY: 45f5a949-518c-4216-80bd-6c5632a9342c
APPLICATIONINSIGHTS_CONNECTION_STRING: "InstrumentationKey=76ae0e4d-5b1f-4081-915a-34cd49c8c4c2;IngestionEndpoint=https://uksouth-1.in.applicationinsights.azure.com/;LiveEndpoint=https://uksouth.livediagnostics.monitor.azure.com/"

resources:
limits:
Expand Down

0 comments on commit 3b83173

Please sign in to comment.