Skip to content

{Monitor} az monitor sli: Add SLI command group models (2025-03-01-preview)#1015

Draft
saleelpk wants to merge 1 commit into
Azure:mainfrom
saleelpk:monitor-sli
Draft

{Monitor} az monitor sli: Add SLI command group models (2025-03-01-preview)#1015
saleelpk wants to merge 1 commit into
Azure:mainfrom
saleelpk:monitor-sli

Conversation

@saleelpk
Copy link
Copy Markdown

What this PR doesnnAdds aaz models for the tenant-scoped Microsoft.Monitor SLI resource (CRUD on serviceGroups/{}/providers/microsoft.monitor/slis), 2025-03-01-preview.nn## Design notesnn- The sli-signal-preview peer operation in the same swagger tag is intentionally excluded. It is not exposed via CLI now or in the future.n- The identitysubgroup is intentionally excluded; identity rotation is handled viaaz monitor sli update --set/--add/--remove in azure-cli to keep the three identity references in the resource body consistent in a single PATCH.nn## Companion PRnn- azure-cli: https://github.com/Azure/azure-cli/pull/33432nn## Notes for future regenerationsnnaaz-dev command-model generate-from-swaggeralways emits a model for every operation in a swagger tag (no--include/--excludeflag). For incremental updates tomonitoringservice, prefer the workspace flow (aaz-dev run) so peer resources (pipeline-group, sli-signal-preview`, etc.) aren't re-touched.

…eview)

Adds aaz models for the tenant-scoped Microsoft.Monitor SLI resource (CRUD on serviceGroups/{}/providers/microsoft.monitor/slis).

Notes:

- The 'sli-signal-preview' peer operation is intentionally excluded from this PR. It is not exposed via CLI now or in the future.

- The 'identity' subgroup is intentionally excluded; identity rotation is handled via 'az monitor sli update --set/--add/--remove' to keep the three identity references in the resource body consistent in a single PATCH.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant