[Bug]: principalId and tenantId in ManagedServiceIdentity should be of type uuid ARM consistency

[XiaofeiCao]

Describe the bug

BUG

TypeSpec definition:

typespec-azure/packages/typespec-azure-resource-manager/lib/common-types/managed-identity.tsp

Lines 20 to 27 in ac594eb

	model ManagedServiceIdentity {
	@doc("The Active Directory tenant id of the principal.")
	@visibility("read")
	tenantId?: string;
	@doc("The active directory identifier of this principal.")
	@visibility("read")
	principalId?: string;

Though in Swagger's common-types v5, it's defined as

"principalId": {
  "readOnly": true,
  "format": "uuid",
  "type": "string",
  "description": "The service principal ID of the system assigned identity. This property will only be provided for a system assigned identity."
},
"tenantId": {
  "readOnly": true,
  "format": "uuid",
  "type": "string",
  "description": "The tenant ID of the system assigned identity. This property will only be provided for a system assigned identity."
},

v3, v4 are uuids also.

Java mgmt SDK will generate type UUID for "format": "uuid". Lacking this mark will break Java SDK.

Expected behavior

These two properties should have @format("uuid") be of type uuid:

@doc("The Active Directory tenant id of the principal.")
@visibility("read")
//@format("uuid")
//tenantId?: string;
tenantId?: uuid;

@doc("The active directory identifier of this principal.")
@visibility("read")
//@format("uuid")
//principalId?: string;
principalId?: uuid;

Reproduction

In the description

Checklist

• Follow our Code of Conduct
• Check that this issue is about the Azure libraries for typespec. For bug in the typespec language or core libraries file it in the TypeSpec repo
• Check that there isn't already an issue that request the same bug to avoid creating a duplicate.
• The provided reproduction is a minimal reproducible example of the bug.

[timotheeguerin]

Should change to have uuid type not @format("uuid")

[XiaofeiCao]

@timotheeguerin This will impact MPG for Java and Go(@tadelesh for confirmation). I can help with a PR.

[pshao25]

@timotheeguerin This will impact MPG for Java and Go(@tadelesh for confirmation). I can help with a PR.

Could you do this together? #863

[timotheeguerin]

The change is ok, PR welcome, the only thing is this can be a breaking change for the providerhub controller generation

[XiaofeiCao]

@timotheeguerin Another discrepancy for ManagedServiceIdentity: #931

I don't have much context for providerhub controller. I'll put them together in a PR as a start.

[allenjzhang]

fixed in pending PR. #945