Skip to content

Commit fba61c1

Browse files
anthony-nhsanthony-nhs
committed
set state machine log level to lower in prod
1 parent 88cb743 commit fba61c1

7 files changed

Lines changed: 27 additions & 4 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ jobs:
114114
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
115115
ENABLE_ALERTS: true
116116
RUN_REGRESSION_TEST: true
117+
STATE_MACHINE_LOG_LEVEL: ALL
117118
secrets:
118119
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_DEPLOY_ROLE }}
119120
DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE }}
@@ -142,6 +143,7 @@ jobs:
142143
DEPLOY_APIGEE_CPSU: true
143144
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
144145
RUN_REGRESSION_TEST: false
146+
STATE_MACHINE_LOG_LEVEL: ALL
145147
secrets:
146148
REGRESSION_TESTS_PEM: ${{ secrets.REGRESSION_TESTS_PEM }}
147149
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_DEPLOY_ROLE }}
@@ -167,6 +169,7 @@ jobs:
167169
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
168170
ENABLE_ALERTS: true
169171
RUN_REGRESSION_TEST: true
172+
STATE_MACHINE_LOG_LEVEL: ALL
170173
secrets:
171174
REGRESSION_TESTS_PEM: ${{ secrets.REGRESSION_TESTS_PEM }}
172175
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.QA_CLOUD_FORMATION_DEPLOY_ROLE }}

‎.github/workflows/pull_request.yml‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ jobs:
7777
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
7878
ENABLE_ALERTS: false
7979
RUN_REGRESSION_TEST: true
80+
STATE_MACHINE_LOG_LEVEL: ALL
8081
secrets:
8182
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_DEPLOY_ROLE }}
8283
PROXYGEN_ROLE: ${{ secrets.PROXYGEN_PTL_ROLE }}
@@ -101,6 +102,7 @@ jobs:
101102
DEPLOY_APIGEE_CPSU: true
102103
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
103104
RUN_REGRESSION_TEST: false
105+
STATE_MACHINE_LOG_LEVEL: ALL
104106
secrets:
105107
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_DEPLOY_ROLE }}
106108
PROXYGEN_ROLE: ${{ secrets.PROXYGEN_PTL_ROLE }}

‎.github/workflows/release.yml‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@ jobs:
133133
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
134134
ENABLE_ALERTS: true
135135
RUN_REGRESSION_TEST: true
136+
STATE_MACHINE_LOG_LEVEL: ALL
136137
secrets:
137138
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_DEPLOY_ROLE }}
138139
DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE }}
@@ -161,6 +162,7 @@ jobs:
161162
DEPLOY_APIGEE_CPSU: true
162163
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
163164
RUN_REGRESSION_TEST: false
165+
STATE_MACHINE_LOG_LEVEL: ALL
164166
secrets:
165167
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_DEPLOY_ROLE }}
166168
PROXYGEN_ROLE: ${{ secrets.PROXYGEN_PTL_ROLE }}
@@ -194,6 +196,7 @@ jobs:
194196
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
195197
ENABLE_ALERTS: true
196198
RUN_REGRESSION_TEST: false
199+
STATE_MACHINE_LOG_LEVEL: ERROR
197200
secrets:
198201
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.REF_CLOUD_FORMATION_DEPLOY_ROLE }}
199202
PROXYGEN_ROLE: ${{ secrets.PROXYGEN_PTL_ROLE }}
@@ -227,6 +230,7 @@ jobs:
227230
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
228231
ENABLE_ALERTS: true
229232
RUN_REGRESSION_TEST: true
233+
STATE_MACHINE_LOG_LEVEL: ALL
230234
secrets:
231235
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.QA_CLOUD_FORMATION_DEPLOY_ROLE }}
232236
PROXYGEN_ROLE: ${{ secrets.PROXYGEN_PTL_ROLE }}
@@ -255,6 +259,7 @@ jobs:
255259
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
256260
ENABLE_ALERTS: true
257261
RUN_REGRESSION_TEST: true
262+
STATE_MACHINE_LOG_LEVEL: ALL
258263
secrets:
259264
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.INT_CLOUD_FORMATION_DEPLOY_ROLE }}
260265
DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE }}
@@ -283,6 +288,7 @@ jobs:
283288
DEPLOY_APIGEE_CPSU: true
284289
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: true
285290
RUN_REGRESSION_TEST: false
291+
STATE_MACHINE_LOG_LEVEL: ALL
286292
secrets:
287293
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.INT_CLOUD_FORMATION_DEPLOY_ROLE }}
288294
PROXYGEN_ROLE: ${{ secrets.PROXYGEN_PROD_ROLE }}
@@ -318,6 +324,7 @@ jobs:
318324
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: false
319325
ENABLE_ALERTS: true
320326
RUN_REGRESSION_TEST: false
327+
STATE_MACHINE_LOG_LEVEL: ERROR
321328
secrets:
322329
CLOUD_FORMATION_DEPLOY_ROLE: ${{ secrets.PROD_CLOUD_FORMATION_DEPLOY_ROLE }}
323330
DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE: ${{ secrets.DEV_CLOUD_FORMATION_CHECK_VERSION_ROLE }}

‎.github/workflows/run_release_code_and_api.yml‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,10 @@ on:
6666
default: true
6767
RUN_REGRESSION_TEST:
6868
required: true
69-
type: boolean
69+
type: boolean
70+
STATE_MACHINE_LOG_LEVEL:
71+
required: true
72+
type: string
7073
secrets:
7174
CLOUD_FORMATION_DEPLOY_ROLE:
7275
required: true
@@ -150,6 +153,7 @@ jobs:
150153
DYNAMODB_AUTOSCALE: ${{ inputs.DYNAMODB_AUTOSCALE }}
151154
DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE: ${{ inputs.DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE }}
152155
ENABLE_ALERTS: ${{ inputs.ENABLE_ALERTS }}
156+
STATE_MACHINE_LOG_LEVEL: ${{ inputs.STATE_MACHINE_LOG_LEVEL }}
153157
run: ./release_code.sh
154158

155159
- name: get mtls secrets

‎Makefile‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,8 @@ sam-deploy-package: guard-artifact_bucket guard-artifact_bucket_prefix guard-sta
9696
LogRetentionInDays=$$LOG_RETENTION_DAYS \
9797
Environment=$$TARGET_ENVIRONMENT \
9898
DeployCheckPrescriptionStatusUpdate=$$DEPLOY_CHECK_PRESCRIPTION_STATUS_UPDATE \
99-
EnableAlerts=$$ENABLE_ALERTS
99+
EnableAlerts=$$ENABLE_ALERTS \
100+
StateMachineLogLevel=$$STATE_MACHINE_LOG_LEVEL
100101

101102
compile-node:
102103
npx tsc --build tsconfig.build.json

‎SAMtemplates/main_template.yaml‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,8 @@ Parameters:
8686
Description: When the database usage crosses this percentage of provisioned capacity, trigger an alarm
8787
Default: 90
8888

89+
StateMachineLogLevel:
90+
Type: String
8991

9092
Resources:
9193
Tables:
@@ -145,6 +147,7 @@ Resources:
145147
ConvertRequestToFhirFormatFunctionArn: !GetAtt Functions.Outputs.ConvertRequestToFhirFormatFunctionArn
146148
LogRetentionInDays: !Ref LogRetentionInDays
147149
EnableSplunk: !Ref EnableSplunk
150+
StateMachineLogLevel: !Ref StateMachineLogLevel
148151

149152
Alarms:
150153
Type: AWS::Serverless::Application

‎SAMtemplates/state_machines/main.yaml‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,9 @@ Parameters:
3434
Type: String
3535
Default: fhir-validator
3636

37+
StateMachineLogLevel:
38+
Type: String
39+
3740
Resources:
3841
UpdatePrescriptionStatusStateMachine:
3942
Type: AWS::Serverless::StateMachine
@@ -53,7 +56,7 @@ Resources:
5356
- CloudWatchLogsLogGroup:
5457
LogGroupArn: !GetAtt UpdatePrescriptionStatusStateMachineResources.Outputs.StateMachineLogGroupArn
5558
IncludeExecutionData: true
56-
Level: ALL
59+
Level: !Ref StateMachineLogLevel
5760
Tracing:
5861
Enabled: true
5962

@@ -90,7 +93,7 @@ Resources:
9093
- CloudWatchLogsLogGroup:
9194
LogGroupArn: !GetAtt Format1UpdatePrescriptionsStatusStateMachineResources.Outputs.StateMachineLogGroupArn
9295
IncludeExecutionData: true
93-
Level: ALL
96+
Level: !Ref StateMachineLogLevel
9497
Tracing:
9598
Enabled: true
9699

0 commit comments

Comments
 (0)