AlertsManagement Module TSP Migration - #29969
Draft
JiaSeng-v wants to merge 1 commit into
Draft
Conversation
Contributor
|
Azure Pipelines: Successfully started running 1 pipeline(s). There may be pipelines that require an authorized user to comment /azp run to run. |
Member
|
/azp run |
Contributor
There was a problem hiding this comment.
Pull request overview
Migrates the AlertsManagement PrometheusRuleGroups AutoRest submodule toward the TypeSpec (TSP) generation workflow by introducing TSP configuration/location files and refreshing generated artifacts (module metadata, help/docs, and solution wiring).
Changes:
- Added TypeSpec generation configuration (
tspconfig.yaml) and pinned spec location (tsp-location.yaml) for PrometheusRuleGroups. - Updated/regenerated module metadata and documentation (assembly attributes, markdown help/docs, module manifest, generate-info).
- Updated AlertsManagement solution wiring (project GUID) and added a changelog note about internal generation updates.
Reviewed changes
Copilot reviewed 17 out of 17 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| src/AlertsManagement/PrometheusRuleGroups.Autorest/tspconfig.yaml | Adds TypeSpec PowerShell emitter configuration and directives. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/tsp-location.yaml | Pins the source spec directory and commit for reproducible generation. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/README.md | Removes legacy AutoRest generation instructions/config block. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/Properties/AssemblyInfo.cs | Updates assembly metadata for regenerated output. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/generate-info.json | Updates generation identifier. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/Update-AzPrometheusRuleGroup.md | Regenerated cmdlet reference content. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/Remove-AzPrometheusRuleGroup.md | Regenerated cmdlet reference content. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/New-AzPrometheusRuleGroup.md | Regenerated cmdlet reference content. |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/Get-AzPrometheusRuleGroup.md | Regenerated cmdlet reference content (parameter sets/metadata). |
| src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/Az.PrometheusRuleGroups.md | Regenerated module index/reference content. |
| src/AlertsManagement/AlertsManagement/help/Update-AzPrometheusRuleGroup.md | Regenerated in-module help content. |
| src/AlertsManagement/AlertsManagement/help/Remove-AzPrometheusRuleGroup.md | Regenerated in-module help content. |
| src/AlertsManagement/AlertsManagement/help/New-AzPrometheusRuleGroup.md | Regenerated in-module help content. |
| src/AlertsManagement/AlertsManagement/help/Get-AzPrometheusRuleGroup.md | Regenerated in-module help content (parameter sets/metadata). |
| src/AlertsManagement/AlertsManagement/ChangeLog.md | Adds an “internal generation updated” upcoming release note. |
| src/AlertsManagement/AlertsManagement/Az.AlertsManagement.psd1 | Updates manifest metadata and dependency version; refreshes formatting. |
| src/AlertsManagement/AlertsManagement.sln | Updates the PrometheusRuleGroups project GUID mapping. |
Suppressed comments (4)
src/AlertsManagement/AlertsManagement/help/Update-AzPrometheusRuleGroup.md:47
- The description/example text repeats the grammatical error "Update an Prometheus"; this should be "Update a Prometheus".
Update an Prometheus rule group definition.
## EXAMPLES
### Example 1: Update an Prometheus rule group definition.
src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/Update-AzPrometheusRuleGroup.md:44
- The description/example text repeats the grammatical error "Update an Prometheus"; this should be "Update a Prometheus".
Update an Prometheus rule group definition.
## EXAMPLES
### Example 1: Update an Prometheus rule group definition.
src/AlertsManagement/AlertsManagement/help/Update-AzPrometheusRuleGroup.md:171
- Grammar: use "a UUID" (pronounced "you-"), not "an UUID".
The value must be an UUID.
src/AlertsManagement/PrometheusRuleGroups.Autorest/docs/Update-AzPrometheusRuleGroup.md:168
- Grammar: use "a UUID" (pronounced "you-"), not "an UUID".
The value must be an UUID.
|
|
||
| ## SYNOPSIS | ||
| update an Prometheus rule group definition. | ||
| Update an Prometheus rule group definition. |
|
|
||
| ## SYNOPSIS | ||
| update an Prometheus rule group definition. | ||
| Update an Prometheus rule group definition. |
| [assembly: System.Reflection.AssemblyTitleAttribute("Microsoft Azure PowerShell - PrometheusRuleGroups")] | ||
| [assembly: System.Reflection.AssemblyFileVersionAttribute("0.6.3")] | ||
| [assembly: System.Reflection.AssemblyVersionAttribute("0.6.3")] | ||
| [assembly: System.Reflection.AssemblyTitleAttribute("Microsoft Azure PowerShell - AzureAlertsManagementServiceResourceProvider")] |
|
|
||
| ### [Update-AzPrometheusRuleGroup](Update-AzPrometheusRuleGroup.md) | ||
| update an Prometheus rule group definition. | ||
| Update an Prometheus rule group definition. |
|
|
||
| ### -SubscriptionId | ||
| The ID of the target subscription. | ||
| The value must be an UUID. |
|
|
||
| ### -SubscriptionId | ||
| The ID of the target subscription. | ||
| The value must be an UUID. |
|
|
||
| ### -SubscriptionId | ||
| The ID of the target subscription. | ||
| The value must be an UUID. |
|
|
||
| ### -SubscriptionId | ||
| The ID of the target subscription. | ||
| The value must be an UUID. |
|
|
||
| ### -SubscriptionId | ||
| The ID of the target subscription. | ||
| The value must be an UUID. |
|
|
||
| ### -SubscriptionId | ||
| The ID of the target subscription. | ||
| The value must be an UUID. |
Contributor
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.