-
Notifications
You must be signed in to change notification settings - Fork 14
docs: CRUD support for snow resource monitors #203
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
docs: CRUD support for snow resource monitors #203
Conversation
Deploying localstack-docs with
|
Latest commit: |
508dede
|
Status: | ✅ Deploy successful! |
Preview URL: | https://369170a7.localstack-docs.pages.dev |
Branch Preview URL: | https://docs-snow-resource-monitors.localstack-docs.pages.dev |
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.
Approving but there is one sentence that I find confusing/unclear.
|
||
Resource monitors in Snowflake allow administrators to track and control credit usage for warehouses and accounts. They help manage costs by defining limits and triggering actions (such as suspending warehouses) when thresholds are reached. | ||
|
||
The Snowflake emulator offers CRUD support for resource monitors, not the actual implementation or usage of them. This is useful for testing workload governance and cost control strategies in your local development environment. |
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 Snowflake emulator offers CRUD support for resource monitors, not the actual implementation or usage of them.
This sentence is confusing, particularly the latter half. And the word usage here adds confusing because we say above that they track usage, so it's unclear the context. I think you mean that the created monitors will not track usage or trigger actions?
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.
ty, I've rewritten this, do let me know if you still dislike it!
We need to document that we've added initial CRUD support for resource monitors.