Azure / azure-rest-api-specs

The source for REST API specifications for Microsoft Azure.
MIT License
2.67k stars 5.09k forks source link

[BUG] Recommendations List API missing attributes #28051

Open DevOpsStyle opened 8 months ago

DevOpsStyle commented 8 months ago

API Spec link

https://learn.microsoft.com/en-us/rest/api/advisor/recommendations/list?view=rest-advisor-2023-01-01&tabs=HTTP

API Spec version

2023-01-01

Describe the bug

Response not compliant with documentation, missing some attributes

Expected behavior

"value":[ { "id":"/resourceUri/providers/Microsoft.Advisor/recommendations/recGUID1", "name":"recGUID1", "properties":{ "category":"HighAvailability", "impact":"Medium", "impactedField":"Microsoft.Compute/availabilitySets", "impactedValue":"armavset", "lastUpdated":"2017-02-24T22:24:43.3216408Z", "risk":"Warning", "remediation":{ "additionalProperties":{ "httpMethod":"POST", "uri":"uri", "details":"link to document" } }, "shortDescription":{ "problem":"To ensure high availability add one or more virtual machines to this availability set", "solution":"To ensure high availability add one or more virtual machines to this availability set" }, "resourceMetadata":{ "resourceId":"/subscriptions/subscriptionId/resourceGroups/resourceGroup/providers/Microsoft.Compute/availabilitysets/abc", "action":{ "additionalProperties":{ "actionType":"Document", "link":"https://link3", "caption":"Enable Soft Delete to protect blob data", "description":"Enable Soft Delete to protect blob data" }, "metadata":{ "id":"/subscriptions/subscriptionId" } }, "singular":"Availability set", "plural":"Availability sets" }, "actions":[ { "additionalProperties":{ "actionType":"Document", "link":"https://link1", "caption":"Enable Soft Delete to protect blob data", "description":"Enable Soft Delete to protect blob data" }, "metadata":{ "id":"/subscriptions/subscriptionId" } } ], "description":"After enabling Soft Delete, deleted data transitions to a soft deleted state instead of being permanently deleted. When data is overwritten, a soft deleted snapshot is generated to save the state of the overwritten data. You can configure the amount of time soft deleted data is recoverable before it permanently expires.", "label":"Enable Soft Delete", "learnMoreLink":"https://link2", "potentialBenefits":"Save and recover your data when blobs or blob snapshots are accidentally overwritten or deleted" }, "type":"Microsoft.Advisor/recommendations" }

Actual behavior

{ "properties":{ "category":"HighAvailability", "impact":"High", "impactedField":"MICROSOFT.COMPUTE/DISKS", "impactedValue":"AADJoinVM_OsDisk_1_01c956f5872647f3ad36daee1c169805", "lastUpdated":"2024-02-25T20:59:57.0398805Z", "recommendationTypeId":"d4102c0f-ebe3-4b22-8fe0-e488866a87af", "shortDescription":{ "problem":"Use Azure Disks with Zone Redundant Storage for higher resiliency and availability", "solution":"Use Azure Disks with Zone Redundant Storage for higher resiliency and availability" }, "extendedProperties":{ "Location":"westus2" }, "resourceMetadata":{ "resourceId":"/subscriptions/subid/resourceGroups/AVD/providers/Microsoft.Compute/disks/AADJoinVM_OsDisk_1_01c956f5872647f3ad36daee1c169805" } }, "id":"/subscriptions/subid/resourceGroups/AVD/providers/Microsoft.Compute/disks/AADJoinVM_OsDisk_1_01c956f5872647f3ad36daee1c169805/providers/Microsoft.Advisor/recommendations/99c3c772-f9f1-8447-91c2-94102cc65f97", "type":"Microsoft.Advisor/recommendations", "name":"99c3c772-f9f1-8447-91c2-94102cc65f97" }

Reproduction Steps

Environment

zzhxiaofeng commented 8 months ago

@shavital7 Please help have a look, thank you.

shavital7 commented 8 months ago

@shavital7 Please help have a look, thank you.

@zzhxiaofeng this should be handled by Microsoft.Advisor owners, not Microsoft.RecommendationsService

zzhxiaofeng commented 8 months ago

@shavital7 Sorry, I see.

zzhxiaofeng commented 8 months ago

@brandonh1994 Please help have a look, thank you.

billyclaymyersmsft commented 3 months ago

This is behaving as expected. The metadata returned varies based on what each PG team provides per recommendation. The documentation needs to be updated and a task has been created internally and will be prioritized with the other already in-flight work.