resourcegroups Should be resourceGroups

[greatbody]

Why this PR

While I am working on my project, I noticed that the terraform state I imported from Azure is not correct.

The "resourcegroups" string lies in the scope parameter, while the correct one should be "resourceGroup".

[tombuildsstuff]

hey @greatbody

Thanks for this PR.

Taking a look through here since this is changing the generated code from the Azure SDK for Go unfortunately we're unable to accept this change, since these files are vendored from the Azure SDK such that the change would instead need to be made in the Azure SDK for Go repository (or rather, the Azure/azure-rest-api-specs repository that it's generated from).

I've recently opened #20387 which switches this SDK from using Azure/azure-sdk-for-go to using hashicorp/go-azure-sdk, which is also generated from the Swagger - but instead requires the casing matches resourceGroups.

However in both SDKs we're not using this method - instead we're splitting the Resource ID into Scope and the Lock Name and then using the ByScope methods - meaning we use the user-provided Scope as this value, so unfortunately changing this method in the SDK won't have the desired effect here.

Based on the description/change you're proposing here, it sounds like you've imported this resource in a different casing, which is being flagged as a diff - in this case I'd recommend using terraform state rm (to remove the resource from your state file) and terraform import (to re-import this resource) using resourceGroups rather than resourcegroups, which should solve this - would you be able to take a look and see if that works for you?

However since this PR is changing a function in the upstream/vendored Azure SDK for Go (and isn't the method we're using) - whilst I'd like to thank you for this contribution, I'm going to close this PR for the moment.

Thanks!

[greatbody]

Thank you for your details response.

Respect.

[greatbody]

I think you are right. I checked the import command, and the id while I import is lowcase.

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.