[AutoPR azure-resourcemanager-dashboard]-generated-from-SDK Generation - Java-5112177 #46078
Conversation
…ig.yaml', API Version: 2024-11-01-preview, SDK Release Type: beta, and CommitSHA: '737d276c61674f351c2d8ff06c69341ee0f7a598' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs'
github-actions
bot
added
the
Mgmt
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
|
/azp run java - dashboard - mgmt - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
|
/azp run java - dashboard - mgmt - tests |
|
Azure Pipelines successfully started running 1 pipeline(s). |
weidongxu-microsoft
requested review from
weidongxu-microsoft,
haolingdong-msft,
XiaofeiCao and
ArthurMa1978
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR contains an auto-generated SDK update for the Azure Dashboard Resource Manager client library, updating from commit 4c09d8bbeea862f6a84a75f703c3a30ed48fc5cc to 737d276c61674f351c2d8ff06c69341ee0f7a598 for API version 2024-11-01-preview. The primary purpose is to fix a bug related to the update operation for ManagedGrafana resources by converting it from a synchronous operation to a long-running operation (LRO) pattern.
Key changes include:
- Converting ManagedGrafana update operations to use LRO patterns with
beginUpdatemethods - Restructuring the ManagedGrafanaInner class to extend ProxyResource instead of Resource
- Reordering fluent builder interface stages to make region specification optional in the creation flow
Reviewed Changes
Copilot reviewed 12 out of 54 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tsp-location.yaml | Updates commit SHA reference for spec generation |
| DashboardManagerTests.java | Reorders method calls in test to match new fluent interface |
| azure-resourcemanager-dashboard_metadata.json | Updates cross-language definitions mappings |
| azure-resourcemanager-dashboard_apiview_properties.json | Adds beginUpdate method and removes updateWithResponse |
| ManagedGrafana.java | Restructures fluent interface to make region optional in creation flow |
| ManagedGrafanaImpl.java | Updates to use LRO update methods instead of direct updateWithResponse |
| GrafanasClientImpl.java | Implements LRO pattern for update operations |
| ManagedGrafanaInner.java | Changes base class from Resource to ProxyResource |
| GrafanasClient.java | Updates interface to include LRO update methods |
| SAMPLE.md | Reorders method calls to match new fluent interface pattern |
| README.md | Updates version and reorders method calls in code example |
| CHANGELOG.md | Documents the bug fix for ManagedGrafana update operations |
|
Hi @weidongxu-microsoft @haolingdong-msft @XiaofeiCao @ArthurMa1978 Changes look good, would you be able to approve when you have the time? Thank you. |
Configurations: 'specification/dashboard/Dashboard.Management/tspconfig.yaml', API Version: 2024-11-01-preview, SDK Release Type: beta, and CommitSHA: '737d276c61674f351c2d8ff06c69341ee0f7a598' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs'