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

armconsumption.ModernReservationRecommendationProperties missing documented ResourceType field #22786

Open
joshk0 opened this issue Apr 26, 2024 · 10 comments
Assignees
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. Mgmt This issue is related to a management-plane library. needs-team-attention This issue needs attention from Azure service team or SDK team Service Attention This issue is responsible by Azure service team.

Comments

@joshk0
Copy link

joshk0 commented Apr 26, 2024

The armconsumption package currently uses api version 2021-10-01. However, according to docs, version 2023-05-01 is available, and has several added fields that are valuable to me.

Could armconsumption get regenerated with this new API version?

More generally speaking, is there a methodology for using different released API versions with the Go client?

@github-actions github-actions bot added customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-triage This is a new issue that needs to be triaged to the appropriate team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Apr 26, 2024
@joshk0 joshk0 changed the title Upgrade armconsumption to new version Upgrade armconsumption to version 2023-05-01 Apr 26, 2024
@jhendrixMSFT jhendrixMSFT added Mgmt This issue is related to a management-plane library. feature-request This issue requires a new behavior in the product in order be resolved. and removed question The issue doesn't require a change to the product in order to be resolved. Most issues start as that needs-triage This is a new issue that needs to be triaged to the appropriate team. labels Apr 26, 2024
@github-actions github-actions bot added the needs-team-attention This issue needs attention from Azure service team or SDK team label Apr 26, 2024
@Alancere
Copy link
Member

Hi @joshk0 thanks for this issue!

Releasing a new package will require the service to submit a new SDK release request, I have added the label on this issue so that the corresponding service team could take this a look.
I also sent some internal emails to the contact of the corresponding service team about this.

@Alancere Alancere added the Service Attention This issue is responsible by Azure service team. label Apr 28, 2024
Copy link

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @ArcturusZhang @lirenhe @tadelesh.

@Alancere
Copy link
Member

Hi @joshk0, you could use different API versions through ClientOptions.APIVersion

@joshk0
Copy link
Author

joshk0 commented Apr 29, 2024

I've taken advantage of that for another API. But if the response structure is different, there's no way to override that; or is there?

@Alancere
Copy link
Member

I've taken advantage of that for another API. But if the response structure is different, there's no way to override that; or is there?

I compared api version 2021-10-01 and 2023-11-01 and the definition of response is the same.

You could also configure the environment variable AZURE_SDK_GO_LOGGING=all to determine whether the specified api version is being used

@Alancere Alancere added the needs-author-feedback More information is needed from author to address the issue. label Apr 30, 2024
@github-actions github-actions bot removed the needs-team-attention This issue needs attention from Azure service team or SDK team label Apr 30, 2024
Copy link

Hi @joshk0. Thank you for opening this issue and giving us the opportunity to assist. To help our team better understand your issue and the details of your scenario please provide a response to the question asked above or the information requested above. This will help us more accurately address your issue.

Copy link

github-actions bot commented May 7, 2024

Hi @joshk0, we're sending this friendly reminder because we haven't heard back from you in 7 days. We need more information about this issue to help address it. Please be sure to give us your input. If we don't hear back from you within 14 days of this comment the issue will be automatically closed. Thank you!

@github-actions github-actions bot added the no-recent-activity There has been no recent activity on this issue. label May 7, 2024
@joshk0
Copy link
Author

joshk0 commented May 7, 2024

I compared api version 2021-10-01 and 2023-11-01 and the definition of response is the same.

So I think you're right that it's the same spec wise, but the real problem is that there's a missing field in the Go SDK compared to this JSON spec.

It's several layers deep, but it's ModernReservationRecommendationProperties that's the difference. I am looking for the resourceType field within that; you can see it here in the public doc as well.

The field is present in the swagger definition of both 2021-10-01 and 2023-11-01, but it is not present within the Go SDK. Only the "Legacy" version of the Properties has it.

Why is this field missing from the Go SDK despite being on the public doc as well as the swagger spec that you build this from?

@github-actions github-actions bot added needs-team-attention This issue needs attention from Azure service team or SDK team and removed needs-author-feedback More information is needed from author to address the issue. no-recent-activity There has been no recent activity on this issue. labels May 7, 2024
@joshk0 joshk0 changed the title Upgrade armconsumption to version 2023-05-01 armconsumption.ModernReservationRecommendationProperties missing documented ResourceType field May 7, 2024
@Alancere
Copy link
Member

Alancere commented May 8, 2024

I compared api version 2021-10-01 and 2023-11-01 and the definition of response is the same.

So I think you're right that it's the same spec wise, but the real problem is that there's a missing field in the Go SDK compared to this JSON spec.

It's several layers deep, but it's ModernReservationRecommendationProperties that's the difference. I am looking for the resourceType field within that; you can see it here in the public doc as well.

The field is present in the swagger definition of both 2021-10-01 and 2023-11-01, but it is not present within the Go SDK. Only the "Legacy" version of the Properties has it.

Why is this field missing from the Go SDK despite being on the public doc as well as the swagger spec that you build this from?

armconsumption@v1.2.0 generated from this swagger, there was no resourceType at that time. I will request a release-request as soon as possible

@joshk0
Copy link
Author

joshk0 commented May 8, 2024

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. feature-request This issue requires a new behavior in the product in order be resolved. Mgmt This issue is related to a management-plane library. needs-team-attention This issue needs attention from Azure service team or SDK team Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

4 participants