-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[api] [monitors] Add option for force delete. #6595
Conversation
@@ -8,3 +8,6 @@ monitor_id=<YOUR_MONITOR_ID> | |||
|
|||
# Delete a monitor | |||
curl -X DELETE "https://api.datadoghq.com/api/v1/monitor/${monitor_id}?api_key=${api_key}&application_key=${app_key}" | |||
|
|||
# Force delete a monitor | |||
curl -X DELETE "https://api.datadoghq.com/api/v1/monitor/${monitor_id}?force=true&api_key=${api_key}&application_key=${app_key}" |
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.
Nit: missing new line
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.
👍
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 for the update!
can we also update the code example for ruby client? and also fix typo in the ruby example https://a.cl.ly/z8unyXKr |
Probably not a typo 😀 , valid ruby syntax, underscores can be used to separate digits, typically done as a replacement for commas for every 3 digits for readability. |
ah, thanks for checking that! my bad, will be more careful next time when I comment on any ruby code. |
Hi guys, I'm not able to force delete the monitor (that has reference in composite monitor) via API:
|
Fixed by DD already. |
What does this PR do?
Adds documentation and examples of using monitor delete with force, these options were added to clients and released:
Motivation
MA-17
Preview link
https://docs-staging.datadoghq.com/pb/mon-del-force/api
Additional Notes