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

[BUG] Microsoft.Maps @ 2023-06-01 - PUT and DELETE operations for Accounts should be marked as LRO #29501

Open
stephybun opened this issue Jun 19, 2024 · 2 comments
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. Maps Mgmt This issue is related to a management-plane library. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Service Attention Workflow: This issue is responsible by Azure service team.

Comments

@stephybun
Copy link

stephybun commented Jun 19, 2024

API Spec link

"responses": {
"200": {
"description": "The Account was successfully updated.",
"schema": {
"$ref": "#/definitions/MapsAccount"
}
},
"201": {
"description": "The Account was successfully created.",
"schema": {
"$ref": "#/definitions/MapsAccount"
}
},

API Spec version

2023-06-01

Describe the bug

The PUT and DELETE methods in the spec for Maps are not marked as Long Running Operations

However creating a Maps Account returns a provisioningState of Updating

...
"properties": {
    "uniqueId": "36b85746-cbda-45e5-8f60-7090a13ccc3c",
    "provisioningState": "Updating",
...

And attempting to delete results in the following error:

unexpected status 409 (409 Conflict) with error: InvalidProvisoningState: The current provisioningState must transition to a terminal state before the resource can be updated.

I believe these operations should be Long Running operations and return an async URL in the header that we can poll on to know when a Maps Account resource is ready to be updated or deleted.

Expected behavior

When creating a Maps Account I should receive an async operation URL in the header that will allow me to poll the provisioning status of the resource.

Actual behavior

Creating a Maps Account returns a 201 with no async operation operation URL in the header for us to poll on. Issuing an update to the account or a delete right after results in a 409 error.

Reproduction Steps

Create a Map Account then issue an update or a delete immediately afterwards.

Environment

No response

@stephybun stephybun added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Jun 19, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added question The issue doesn't require a change to the product in order to be resolved. Most issues start as that customer-reported Issues that are reported by GitHub users external to the Azure organization. labels Jun 19, 2024
@v-jiaodi v-jiaodi added Mgmt This issue is related to a management-plane library. Service Attention Workflow: This issue is responsible by Azure service team. Maps labels Jun 20, 2024
@v-jiaodi
Copy link
Member

@gigigoo0 Can you help take a look?

@gigigoo0
Copy link
Member

Will take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. Maps Mgmt This issue is related to a management-plane library. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that Service Attention Workflow: This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

3 participants