Enums entries missing extension `x-ms-enum`

[ArcturusZhang]

We have some swaggers that are missing x-ms-enum extension. For instance, this:

azure-rest-api-specs/specification/compute/resource-manager/Microsoft.ContainerService/stable/2017-01-31/containerService.json

Line 359 in ff541d8

"enum": [

In code generation, a code-name with number will be given to this enum type (for instance Enum47), and during the update of the swaggers, this code-name might change and cause unnecessary breaking changes.

Related issue: Azure/autorest.go#571

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Drewm3, @avirishuv, @Vaibhav-Agar, @amjads1.

Issue Details

---

We have some swaggers that are missing x-ms-enum extension. For instance, this:

azure-rest-api-specs/specification/compute/resource-manager/Microsoft.ContainerService/stable/2017-01-31/containerService.json

Line 359 in ff541d8

"enum": [

In code generation, a code-name with number will be given to this enum type (for instance Enum47), and during the update of the swaggers, this code-name might change and cause unnecessary breaking changes.

Related issue: Azure/autorest.go#571

Author:	ArcturusZhang
Assignees:	ruowan
Labels:	Compute, Service Attention, needs-triage
Milestone:	-

[axayjo]

@ArcturusZhang, I see that the issue is related to Microsoft.ContainerService, but it's assigned to Compute. Any Compute swaggers with the issue?

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @dkkapur.

Issue Details

---

We have some swaggers that are missing x-ms-enum extension. For instance, this:

azure-rest-api-specs/specification/compute/resource-manager/Microsoft.ContainerService/stable/2017-01-31/containerService.json

Line 359 in ff541d8

"enum": [

In code generation, a code-name with number will be given to this enum type (for instance Enum47), and during the update of the swaggers, this code-name might change and cause unnecessary breaking changes.

Related issue: Azure/autorest.go#571

Author:	ArcturusZhang
Assignees:	-
Labels:	Compute, Container Instances, Service Attention
Milestone:	-

[Drewm3]

Correcting labels

[ArcturusZhang]

Out of curiosity, do we have plans to remove the containservices swaggers from the compute directory? @Drewm3 @axayjo