New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for Billing CloudWatch metrics and alarms #659
Add support for Billing CloudWatch metrics and alarms #659
Conversation
PR is now waiting for a maintainer to run the acceptance tests. This PR will only perform build and linting. |
…metrics-and-alarms
PR is now waiting for a maintainer to run the acceptance tests. This PR will only perform build and linting. |
/run-acceptance-tests |
Please view the PR build - https://github.com/pulumi/pulumi-awsx/actions/runs/751562172 |
* Valid CloudWatch statistics: Average, Maximum (recommended), Minimum | ||
*/ | ||
export function estimatedCharges(change?: BillingMetricChange) { | ||
return metric("EstimatedCharges", { statistic: "Maximum", currency: "USD", ...change }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the question for me here is can "Statistic" and "currency" be anything other than these values above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Statistic looks like it accepts the usual values, however, I was unable to find a list of supported currencies, and the only thing I've found is "USD".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks so much for this @catmeme :)
Usage:
Adds #658