Skip to content
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

Monitoring examples are outdated #11271

Closed
estensen opened this issue May 6, 2020 · 11 comments
Closed

Monitoring examples are outdated #11271

estensen opened this issue May 6, 2020 · 11 comments
Assignees
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Docs issue-addressed The Azure SDK team member assisting with this issue believes it to be addressed and ready to close. Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@estensen
Copy link

estensen commented May 6, 2020

  • Package Name: azure-mgmt-monitor
  • Package Version: 0.9.0
  • Operating System: macOS
  • Python Version: 3.8.1

Describe the bug
Azure Monitoring libraries for python are outdated and doesn't work with the new alerts.

Is this going to be fixed or is it possible to label examples like this as deprecated/outdated so people don't have to spend time working with examples that don't work?

Tried to open an issue in azure-docs-sdk-python, but was redirected here

@ghost ghost added needs-triage This is a new issue that needs to be triaged to the appropriate team. customer-reported Issues that are reported by GitHub users external to the Azure organization. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels May 6, 2020
@lmazuel lmazuel added Docs Monitor Monitor, Monitor Ingestion, Monitor Query labels May 6, 2020
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label May 6, 2020
@lmazuel lmazuel added the Mgmt This issue is related to a management-plane library. label May 6, 2020
@lmazuel lmazuel added this to the Backlog milestone May 6, 2020
@RodgeFu RodgeFu assigned nickzhums and unassigned zikalino May 9, 2020
@RodgeFu
Copy link
Member

RodgeFu commented May 9, 2020

Assign to Nick for the doc issue

@nickzhums
Copy link
Contributor

Will add this to doc work backlog

@nickzhums
Copy link
Contributor

@estensen could you give a bit more details on this?

@estensen
Copy link
Author

@nickzhums have you tried running the examples I've linked?

The problem is that this example creates the legacy (classic) alerts which you are not allowed to do anymore.

Some things that are broken:

  • You have to use client.metric_alerts.create_or_update instead of client.alert_rules.create_or_update
  • location has to be global
  • alert_rule_resource_name is renamed

@nickzhums
Copy link
Contributor

@estensen thanks, will look into this

@mukulgupta2138
Copy link

Hey @nickzhums, @RodgeFu any update on this ?

Hey @estensen, Can you please share the example that worked for you ? I'm trying to use this but not able to get any help around this :(

@nickzhums
Copy link
Contributor

@mukulgupta2138
Copy link

Hi @nickzhums , thanks for the revert. I'm trying to set up action group action rule and there is no sample example which I can find which can help in setting up these. Can you share some code pointers (if any) for same ?

@nickzhums
Copy link
Contributor

@00Kai0 do you know if we have samples for this case?

@RAY-316 RAY-316 added the issue-addressed The Azure SDK team member assisting with this issue believes it to be addressed and ready to close. label Dec 15, 2021
@ghost
Copy link

ghost commented Dec 15, 2021

Hi @estensen. Thank you for opening this issue and giving us the opportunity to assist. We believe that this has been addressed. If you feel that further discussion is needed, please add a comment with the text “/unresolve” to remove the “issue-addressed” label and continue the conversation.

@ghost
Copy link

ghost commented Dec 22, 2021

Hi @estensen, since you haven’t asked that we “/unresolve” the issue, we’ll close this out. If you believe further discussion is needed, please add a comment “/unresolve” to reopen the issue.

@ghost ghost closed this as completed Dec 22, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Docs issue-addressed The Azure SDK team member assisting with this issue believes it to be addressed and ready to close. Mgmt This issue is related to a management-plane library. Monitor Monitor, Monitor Ingestion, Monitor Query question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

7 participants