Skip to content

Commit fc802c8

Browse files
authored
Merge pull request #14125 from guardian/aa/access-logs-prefix
feat: Update load balancer access logs prefix
2 parents 45d767d + 737d72a commit fc802c8

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

dotcom-rendering/cdk/lib/__snapshots__/renderingStack.test.ts.snap

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1061,7 +1061,7 @@ exports[`The RenderingCDKStack matches the snapshot 1`] = `
10611061
},
10621062
{
10631063
"Key": "access_logs.s3.prefix",
1064-
"Value": "ELBLogs/frontend/article-rendering/PROD",
1064+
"Value": "application-load-balancer/PROD/frontend/article-rendering",
10651065
},
10661066
{
10671067
"Key": "idle_timeout.timeout_seconds",

dotcom-rendering/cdk/lib/renderingStack.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,8 @@ export class RenderingCDKStack extends CDKStack {
210210
},
211211
accessLogging: {
212212
enabled: true,
213-
prefix: `ELBLogs/${guStack}/${guApp}/${stage}`,
213+
// This is the prefix pattern DevX assume so that the logs can be shown on the Availability dashboard.
214+
prefix: `application-load-balancer/${stage}/${guStack}/${guApp}`,
214215
},
215216
applicationLogging: {
216217
enabled: true,

0 commit comments

Comments
 (0)