Skip to content

Commit

Permalink
Fix target port in ACA deployment (change 80 to 8080) (#91)
Browse files Browse the repository at this point in the history
  • Loading branch information
mburumaxwell authored Nov 20, 2023
1 parent 2ab79e3 commit 6c1c164
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions main.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ resource app 'Microsoft.App/containerApps@2023-05-01' = {
properties: {
managedEnvironmentId: hasProvidedAppEnvironment ? appEnvironmentId : appEnvironment.id
configuration: {
ingress: { external: true, targetPort: 80, traffic: [ { latestRevision: true, weight: 100 } ] }
ingress: { external: true, targetPort: 8080, traffic: [ { latestRevision: true, weight: 100 } ] }
secrets: concat(
[
{ name: 'connection-strings-application-insights', value: appInsights.properties.ConnectionString }
Expand Down Expand Up @@ -200,8 +200,8 @@ resource app 'Microsoft.App/containerApps@2023-05-01' = {
]
resources: { cpu: json('0.25'), memory: '0.5Gi' } // these are the least resources we can provision
probes: [
{ type: 'Liveness', httpGet: { port: 80, path: '/liveness' } }
{ type: 'Readiness', httpGet: { port: 80, path: '/health' } }
{ type: 'Liveness', httpGet: { port: 8080, path: '/liveness' } }
{ type: 'Readiness', httpGet: { port: 8080, path: '/health' } }
]
}
]
Expand Down
6 changes: 3 additions & 3 deletions main.json
Original file line number Diff line number Diff line change
Expand Up @@ -187,7 +187,7 @@
"configuration": {
"ingress": {
"external": true,
"targetPort": 80,
"targetPort": 8080,
"traffic": [
{
"latestRevision": true,
Expand Down Expand Up @@ -244,14 +244,14 @@
{
"type": "Liveness",
"httpGet": {
"port": 80,
"port": 8080,
"path": "/liveness"
}
},
{
"type": "Readiness",
"httpGet": {
"port": 80,
"port": 8080,
"path": "/health"
}
}
Expand Down

0 comments on commit 6c1c164

Please sign in to comment.