Resource manager statusMessage property has incorrect type.

[jhendrixMSFT]

In https://github.com/Azure/azure-rest-api-specs/blob/master/specification/resources/resource-manager/Microsoft.Resources/stable/2017-05-10/resources.json#L2068. The property statusMessage is defined as object however the service doesn't always return an object (sometimes a string). This results in broken behavior for the Go SDK. The type must be either a string or an object and the service must abide by the specified contract. See Azure/azure-sdk-for-go#938 for more details.

[jhendrixMSFT]

After recent discoveries in JSON schema I believe the correct thing to do here is remove the "type" so that statusMessage is treated as a wildcard type. @olydis @fearthecowboy do you agree?

[fearthecowboy]

I believe that should be the case; I think we're still waiting for everyone to fix the generators to get the right behavior everywhere.

[jhendrixMSFT]

Do we need to wait for that before updating the swagger?

[fearthecowboy]

@olydis is going to verify that there isn't any breaking change to the code model if you do this now.

[olydis]

Confirmed: The current modeler (the one that does not handle object types properly) generates identical output for type: object vs no type on statusMessage, so indeed it can be safely removed without inducing breaking changes across existing generators. That was what my assumption, but wanted to confirm 😋

[zhenglaizhang]

Looks like the issue is fixed in #2486, I will close it, if any guys has concern or questions, pls reopen it. thanks!