Repository navigation
Conversation
praba2210
force-pushed
the
docs/dsql-cloudwatch-cost-alerts
branch
from
November 4, 2026 21:35
b3eaf23 to
5798e75
Compare
The skill had no guidance for cost alerts, and the Workflow 12 PromQL read only the latest one-minute sample, so a plain selector overcounted sporadic wait events and instant queries ignored the window. - Add cost-alerts.md: a per-cluster hourly TotalDPU alarm sized from a monthly budget, sent through an SNS topic on a customer managed KMS key, with a delivery test; TotalDPU already includes StreamDPU, and multi-Region budgets subtract storage in each peer Region - Run the topic, key and alarm lookups first and ask once before any change; merge into existing key and topic policies instead of replacing them, and subscribe the email for existing topics too - Average AAS with sum_over_time over each window or step, pass time at the window end, sort topk results, and keep span plus lookback within 7 days - Take Phase 1 shares from one instant query per window and the intra-hour chunks from a 600s step, so nothing is summed by hand - Batch the Phase 4 TotalTransactions and OccConflicts calls with a 1-hour period; a 1-minute period returns nothing for windows that start more than 15 days ago - Wait classes follow the AWS docs; check call count and latency before naming a cause of SingleRead growth - Name CloudWatch Database Insights and add a fallback for when the CloudWatch MCP server is not enabled: AWS CLI for commit and OCC metrics, Query Studio or Database Insights for AAS - Route cost questions and cost alarm problems to cost-alerts.md - Link the CloudWatch PromQL Region list instead of a stale copy - Fix "Active Average Sessions" to "Average Active Sessions" - Add cost evals 29-32 and one trigger case - Bump databases-on-aws to 1.11.0
praba2210
force-pushed
the
docs/dsql-cloudwatch-cost-alerts
branch
from
November 5, 2026 03:36
5798e75 to
7b98169
Compare
praba2210
marked this pull request as ready for review
November 5, 2026 18:42
praba2210
requested review from
a team,
arnewouters,
krokoko and
laithalsaadoon
November 5, 2026 18:42
praba2210
requested review from
Benjscho,
amaksimo,
anwesham-lab,
gxjx-x,
jaichabria,
pkale and
spencercorwin
November 5, 2026 18:42
praba2210
changed the title
praba2210
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds DPU cost alarm guidance to the DSQL skill and fixes how Workflow 12 averages Average Active Sessions (AAS).
Related
The evals below were graded with the eval runner from #296;
main's runner can't grade them on the current Claude CLI.Changes
references/cost-alerts.md: a per-cluster hourlyTotalDPUalarm sized from a monthly budget, sent through an SNS topic encrypted with a customer managed key, plus a delivery test. The agent looks up the existing topic, key and alarm first, asks once before changing anything, and merges into existing policies instead of replacing them.SKILL.md: routes cost questions and cost alarm problems tocost-alerts.md.sum_over_time. Phase 1 shares come from one instant query per window, so nothing is summed by hand.mcp/mcp-setup.md: links the PromQL Region list instead of a stale copy.1.10.0→1.11.0.Verification
mise run buildpasses.Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the project license.