Inconsistent naming of NSGs IDs in compute/virtualmachines

[AaronSaikovski]

Describe the bug

In the file ResourceModules/arm/Microsoft.Compute/virtualMachines/deploy.bicep
is currently
networkSecurityGroupId: contains(nicConfiguration, 'nsgId') ? (!empty(nicConfiguration.nsgId) ? nicConfiguration.nsgId : '') : ''

but should be:
networkSecurityGroupId: contains(nicConfiguration, 'networkSecurityGroupId') ? (!empty(nicConfiguration.networkSecurityGroupId) ? nicConfiguration.networkSecurityGroupId : '') : ''

To reproduce

see desc

Code snippet

No response

Relevant log output

No response

[AlexanderSehr]

We could even go a step further and call it networkSecurityGroupResourceId.

In any case I agree that we should update this, but will remove the label bug and replace it with enhancement.

Thanks for the submission Aaron Saikovski (@AaronSaikovski)