Skip to content

Commit 55c1037

Browse files
committed
Fixing race condition issue in SW queries
1 parent 4cb8398 commit 55c1037

1 file changed

Lines changed: 71 additions & 12 deletions

File tree

infrastructure/stacks/blue-green-link/cloudwatch-queries.tf

Lines changed: 71 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,13 @@ EOF
3232
}
3333
}
3434

35+
resource "time_sleep" "wait_after_search_for_errors" {
36+
depends_on = [aws_cloudwatch_query_definition.search_for_errors]
37+
create_duration = "15s"
38+
}
39+
3540
resource "aws_cloudwatch_query_definition" "search_by_correlation_id" {
36-
depends_on = [aws_cloudwatch_query_definition.search_for_errors]
41+
depends_on = [time_sleep.wait_after_search_for_errors]
3742
name = "${var.project_id}/${var.blue_green_environment}/search-by-correlation-id"
3843

3944
log_group_names = [
@@ -57,8 +62,13 @@ EOF
5762
}
5863
}
5964

65+
resource "time_sleep" "wait_after_search_by_correlation_id" {
66+
depends_on = [aws_cloudwatch_query_definition.search_by_correlation_id]
67+
create_duration = "15s"
68+
}
69+
6070
resource "aws_cloudwatch_query_definition" "search_by_correlation_id_expanded" {
61-
depends_on = [aws_cloudwatch_query_definition.search_by_correlation_id]
71+
depends_on = [time_sleep.wait_after_search_by_correlation_id]
6272
name = "${var.project_id}/${var.blue_green_environment}/search-by-correlation-id-expanded"
6373

6474
log_group_names = [
@@ -82,8 +92,13 @@ EOF
8292
}
8393
}
8494

95+
resource "time_sleep" "wait_after_search_by_correlation_id_expanded" {
96+
depends_on = [aws_cloudwatch_query_definition.search_by_correlation_id_expanded]
97+
create_duration = "15s"
98+
}
99+
85100
resource "aws_cloudwatch_query_definition" "search_by_odscode" {
86-
depends_on = [aws_cloudwatch_query_definition.search_by_correlation_id_expanded]
101+
depends_on = [time_sleep.wait_after_search_by_correlation_id_expanded]
87102
name = "${var.project_id}/${var.blue_green_environment}/search-by-odscode"
88103

89104
log_group_names = [
@@ -107,8 +122,13 @@ EOF
107122
}
108123
}
109124

125+
resource "time_sleep" "wait_after_search_by_odscode" {
126+
depends_on = [aws_cloudwatch_query_definition.search_by_odscode]
127+
create_duration = "15s"
128+
}
129+
110130
resource "aws_cloudwatch_query_definition" "search_by_odscode_expanded" {
111-
depends_on = [aws_cloudwatch_query_definition.search_by_odscode]
131+
depends_on = [time_sleep.wait_after_search_by_odscode]
112132
name = "${var.project_id}/${var.blue_green_environment}/search-by-odscode-expanded"
113133

114134
log_group_names = [
@@ -132,8 +152,13 @@ EOF
132152
}
133153
}
134154

155+
resource "time_sleep" "wait_after_search_by_odscode_expanded" {
156+
depends_on = [aws_cloudwatch_query_definition.search_by_odscode_expanded]
157+
create_duration = "15s"
158+
}
159+
135160
resource "aws_cloudwatch_query_definition" "search_for_invalid_postcode" {
136-
depends_on = [aws_cloudwatch_query_definition.search_by_odscode_expanded]
161+
depends_on = [time_sleep.wait_after_search_by_odscode_expanded]
137162
name = "${var.project_id}/${var.blue_green_environment}/search-for-invalid-postcode"
138163

139164
log_group_names = [
@@ -151,8 +176,13 @@ EOF
151176
}
152177
}
153178

179+
resource "time_sleep" "wait_after_search_for_invalid_postcode" {
180+
depends_on = [aws_cloudwatch_query_definition.search_for_invalid_postcode]
181+
create_duration = "15s"
182+
}
183+
154184
resource "aws_cloudwatch_query_definition" "search_for_invalid_opening_times" {
155-
depends_on = [aws_cloudwatch_query_definition.search_for_invalid_postcode]
185+
depends_on = [time_sleep.wait_after_search_for_invalid_postcode]
156186
name = "${var.project_id}/${var.blue_green_environment}/search-for-invalid-opening-times"
157187

158188
log_group_names = [
@@ -170,8 +200,13 @@ EOF
170200
}
171201
}
172202

203+
resource "time_sleep" "wait_after_search_for_invalid_opening_times" {
204+
depends_on = [aws_cloudwatch_query_definition.search_for_invalid_opening_times]
205+
create_duration = "15s"
206+
}
207+
173208
resource "aws_cloudwatch_query_definition" "search_by_email_correlation_id" {
174-
depends_on = [aws_cloudwatch_query_definition.search_for_invalid_opening_times]
209+
depends_on = [time_sleep.wait_after_search_for_invalid_opening_times]
175210
name = "${var.project_id}/${var.blue_green_environment}/search-by-email-correlation-id"
176211

177212
log_group_names = [
@@ -190,8 +225,13 @@ EOF
190225
}
191226
}
192227

228+
resource "time_sleep" "wait_after_search_by_email_correlation_id" {
229+
depends_on = [aws_cloudwatch_query_definition.search_by_email_correlation_id]
230+
create_duration = "15s"
231+
}
232+
193233
resource "aws_cloudwatch_query_definition" "search_by_update_request_success" {
194-
depends_on = [aws_cloudwatch_query_definition.search_by_email_correlation_id]
234+
depends_on = [time_sleep.wait_after_search_by_email_correlation_id]
195235
name = "${var.project_id}/${var.blue_green_environment}/update-request-success"
196236

197237
log_group_names = [
@@ -209,8 +249,13 @@ EOF
209249
}
210250
}
211251

252+
resource "time_sleep" "wait_after_search_by_update_request_success" {
253+
depends_on = [aws_cloudwatch_query_definition.search_by_update_request_success]
254+
create_duration = "15s"
255+
}
256+
212257
resource "aws_cloudwatch_query_definition" "search_by_update_request_failed" {
213-
depends_on = [aws_cloudwatch_query_definition.search_by_update_request_success]
258+
depends_on = [time_sleep.wait_after_search_by_update_request_success]
214259
name = "${var.project_id}/${var.blue_green_environment}/update-request-failed"
215260

216261
log_group_names = [
@@ -228,8 +273,13 @@ EOF
228273
}
229274
}
230275

276+
resource "time_sleep" "wait_after_search_by_update_request_failed" {
277+
depends_on = [aws_cloudwatch_query_definition.search_by_update_request_failed]
278+
create_duration = "15s"
279+
}
280+
231281
resource "aws_cloudwatch_query_definition" "search_by_dos_data_item_updates" {
232-
depends_on = [aws_cloudwatch_query_definition.search_by_update_request_failed]
282+
depends_on = [time_sleep.wait_after_search_by_update_request_failed]
233283
name = "${var.project_id}/${var.blue_green_environment}/dos-data-item-updates"
234284

235285
log_group_names = [
@@ -249,8 +299,13 @@ EOF
249299
}
250300
}
251301

302+
resource "time_sleep" "wait_after_search_by_dos_data_item_updates" {
303+
depends_on = [aws_cloudwatch_query_definition.search_by_dos_data_item_updates]
304+
create_duration = "15s"
305+
}
306+
252307
resource "aws_cloudwatch_query_definition" "search_for_report_warnings" {
253-
depends_on = [aws_cloudwatch_query_definition.search_by_dos_data_item_updates]
308+
depends_on = [time_sleep.wait_after_search_by_dos_data_item_updates]
254309
name = "${var.project_id}/${var.blue_green_environment}/search-for-report-warnings"
255310

256311
log_group_names = [
@@ -275,9 +330,13 @@ EOF
275330
}
276331
}
277332

333+
resource "time_sleep" "wait_after_search_for_report_warnings" {
334+
depends_on = [aws_cloudwatch_query_definition.search_for_report_warnings]
335+
create_duration = "15s"
336+
}
278337

279338
resource "aws_cloudwatch_query_definition" "search_for_quality_checker_logs_with_odscode" {
280-
depends_on = [aws_cloudwatch_query_definition.search_for_report_warnings]
339+
depends_on = [time_sleep.wait_after_search_for_report_warnings]
281340
name = "${var.project_id}/${var.blue_green_environment}/search-for-quality-checker-logs-with-odscode"
282341

283342
log_group_names = [

0 commit comments

Comments
 (0)