(ARMSubscription) fixes Azure/azure-sdk-for-go#18620

[navba-MSFT]

fixes #18620

The subscriptions Model struct includes a number of fields, but not all the fields available in the REST API according to the documentation. Specifically, the following fields are missing in the Go struct:

tenantId - The subscription tenant ID.
tags - The tags attached to the subscription.

Note that these fields are returned by the API and are documented here: https://docs.microsoft.com/en-us/rest/api/resources/subscriptions/list?tabs=HTTP#subscription

• The purpose of this PR is explained in this or a referenced issue.
• The PR does not update generated files.
  • These files are managed by the codegen framework at Azure/autorest.go.
• Tests are included and/or updated for code changes.
• Updates to CHANGELOG.md are included.
• MIT license headers are included in each file.

[hermanschaaf]

@navba-MSFT Thank you for looking into this issue 🙇

I'm not super familiar with this repo, but as far as I can tell, the fix will need to be in the underlying OpenAPI JSON spec first, and then the code here should be regerenated. As it stands, it looks like this PR's change will be lost the next time the code is regenerated.

[navba-MSFT]

@navba-MSFT Thank you for looking into this issue 🙇

I'm not super familiar with this repo, but as far as I can tell, the fix will need to be in the underlying OpenAPI JSON spec first, and then the code here should be regerenated. As it stands, it looks like this PR's change will be lost the next time the code is regenerated.

@hermanschaaf You are right. I have created another PR in the RestAPi specs repo:
Azure/azure-rest-api-specs#19870

[tadelesh]

@navba-MSFT We do not change the SDK manually. This PR will be closed and after the swagger updated, we will release the new SDK according to service team's release request.