Skip to content

Update cloudwatch query to include records that were accidently being…#12207

Merged
sb-2011 merged 1 commit intomainfrom
1005-update-cw-query
May 29, 2025
Merged

Update cloudwatch query to include records that were accidently being…#12207
sb-2011 merged 1 commit intomainfrom
1005-update-cw-query

Conversation

@sb-2011
Copy link
Contributor

@sb-2011 sb-2011 commented May 23, 2025

To fix the stale data check bug that is causing mismatches between idp log group and redshift tables.

Related MRs:

🎫 Ticket

issue-1005

🛠 Summary of changes

Remove filter from the cloudwatch query.

📜 Testing Plan

  • Step 1: Deploy to personal env
  • Step 2: Run the entire stale data check workflow and confirm that the cloudwatch row count check passes

Copy link
Contributor

@colter-nattrass colter-nattrass left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple enough. Looks like you need a changelog message to merge it

… dropped from DW

changelog: Internal, DataWarehouse Alerts, Update the TableSummaryStatsExportJob cloudwatch query to include set of previously excluded records (Issue 1005)
@sb-2011 sb-2011 force-pushed the 1005-update-cw-query branch from 8e01f67 to d7c8a2b Compare May 23, 2025 23:45
Copy link
Contributor

@samathad2023 samathad2023 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sb-2011 sb-2011 merged commit 9e5aa31 into main May 29, 2025
1 check passed
@sb-2011 sb-2011 deleted the 1005-update-cw-query branch May 29, 2025 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants