-
Notifications
You must be signed in to change notification settings - Fork 311
Revise VM watch description and add signal categories #405
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
base: main
Are you sure you want to change the base?
Conversation
Updated the VM watch section to clarify its purpose and functionality, including a new introduction to VMWatch and its signal categories.
|
@tfishler1 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
Learn Build status updates of commit 0a1a178: ✅ Validation status: passed
For more details, please refer to the build report. |
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.
Pull request overview
This pull request updates the Azure VM watch documentation to reflect the service's general availability (removing the "preview" designation) and provides enhanced descriptions of its capabilities. The changes introduce a more comprehensive explanation of VMWatch as a testing framework and add a new section describing signal categories.
Key changes:
- Removed "(preview)" from the main title to reflect general availability status
- Completely rewrote the introductory description to position VMWatch as an open-sourced testing framework with expanded details about its purpose and capabilities
- Added a new "VMWatch Signal Categories" section that introduces the different types of health signals collected
| # VM watch: Enhancing VM health monitoring | ||
|
|
||
| VM watch is a standardized, lightweight, and adaptable service offering for virtual machines (VMs) and virtual machine scale sets. It runs health checks within a VM at configurable intervals and sends the results via a uniform data model to Azure. The AI operations (AIOps) engines for production monitoring in Azure consume these health results for regression detection and prevention. | ||
| VMWatch, a standardized, lightweight, and open-sourced testing framework designed to enhance the monitoring and management of guest VMs on the Azure platform, including both 1P and 3P instances. VMWatch is engineered to collect vital health signals across multiple dimensions, which will be seamlessly integrated into Azure's quality systems. By leveraging these signals, VMWatch will enable Azure to swiftly detect and prevent regressions induced by platform updates or configuration changes, identify gaps in platform telemetry, and ultimately improve the guest experience for all Azure customers. |
Copilot
AI
Jan 6, 2026
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.
This paragraph is not a complete sentence - it lacks a main verb. The text begins with "VMWatch, a standardized..." but never completes the sentence structure. Consider rephrasing to form a complete sentence, such as "VMWatch is a standardized, lightweight, and open-sourced testing framework..."
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.
@copilot open a new pull request to apply changes based on this feedback
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.
@copilot open a new pull request to apply changes based on this feedback
| ### VMWatch Signal Categories | ||
|
|
||
| VMWatch measures VM health by emitting Signals in different categories, including Network, Disk, CPU, Process, IMDS, Clock, AzBlob, Hardware. |
Copilot
AI
Jan 6, 2026
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.
Inconsistent terminology: The title and rest of the document use "VM watch" (two words, lowercase 'watch'), but this new section header and content use "VMWatch" (one word, camelCase). Maintain consistency throughout the document by using "VM watch" as established in the title and other sections.
|
|
||
| ### VMWatch Signal Categories | ||
|
|
||
| VMWatch measures VM health by emitting Signals in different categories, including Network, Disk, CPU, Process, IMDS, Clock, AzBlob, Hardware. |
Copilot
AI
Jan 6, 2026
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.
According to the Microsoft Writing Style Guide, avoid using "and so on" or leaving lists open-ended. Either provide the complete list of signal categories or rephrase to indicate these are examples by using "such as" instead of "including". For example: "VMWatch measures VM health by emitting signals in different categories, such as Network, Disk, CPU, Process, IMDS, Clock, AzBlob, and Hardware."
|
Hi @tfishler1 - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you! Can you review the proposed changes? Important: When the changes are ready for publication, adding a |
|
Hi Diana,
Thank you for sharing the information. I’ll be making additional changes soon, so let’s keep it open for now.
I’m still getting familiar with this system and policy, so apologies for any confusion.
…________________________________
From: Diana Richards ***@***.***>
Sent: Tuesday, January 6, 2026 1:44 PM
To: MicrosoftDocs/azure-compute-docs ***@***.***>
Cc: Tal Fishler ***@***.***>; Mention ***@***.***>
Subject: Re: [MicrosoftDocs/azure-compute-docs] Revise VM watch description and add signal categories (PR #405)
[https://avatars.githubusercontent.com/u/103777760?s=20&v=4]v-dirichards left a comment (MicrosoftDocs/azure-compute-docs#405)<#405 (comment)>
Hi @tfishler1- This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. We can keep this PR open for review and merge, but would you make future content updates in the private repo? Thank you!
________________________________
@iamwilliew<https://github.com/iamwilliew>
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team<https://github.com/orgs/MicrosoftDocs/teams/public-repo-pr-review-team>
—
Reply to this email directly, view it on GitHub<#405 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BHQNWPFX3IWCMYHLLGPMQN34FQUFLAVCNFSM6AAAAACQ4CIGUKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTOMJWGQ3TKMRTG4>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
@tfishler1 If you need help getting set up to work in the private repo, see Get started contributing to the Microsoft Learn platform. |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
Learn Build status updates of commit 908e195: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Learn Build status updates of commit 399ec83: ✅ Validation status: passed
For more details, please refer to the build report. |
Updated signal names, types, and descriptions for various metrics and checks related to Azure VM monitoring. Enhanced clarity and consistency across sections.
|
Learn Build status updates of commit 2289b30: ✅ Validation status: passed
For more details, please refer to the build report. |
@microsoft-github-policy-service agree |
|
Learn Build status updates of commit 83ae212: ✅ Validation status: passed
For more details, please refer to the build report. |
|
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
Updated the VM watch section to clarify its purpose and functionality, including a new introduction to VMWatch and its signal categories.