Azure / Azure/azure-rest-api-specs

[Monitor] Autoscale timezone property should be an enum

Open
#2,530 1 comment 1 reaction 0 assignees View on GitHub
Monitor - Operational Insights Service Attention
Dominant language
TypeSpec
Stars
3.1k
Forks
5.9k
Avg merge
3d 37m
Merged PRs (30d)
446

Description

https://github.com/Azure/azure-rest-api-specs/blob/master/specification/monitor/resource-manager/microsoft.insights/stable/2015-04-01/autoscale_API.json#L1040

The valid values are C# standard time zone names, of which there are about 136. The problem is that these "Standard Names" are only standard for C#. Python, Java, Ruby, don't have the same names and there is no good way to map to them.

This should be modeled as an enum with all the valid values.

Contributor guide

Open the contributing guide

Research direction

Open specification/monitor/resource-manager/microsoft.insights/stable/2015-04-01/autoscale_API.json at line 1040 and inspect the autoscale timezone property. Confirm the complete set of C# standard time zone names, model those values as an enum, and verify that the generated API definition exposes the enum rather than an unrestricted string.

Written by the indexing model from the issue text.

Assessment

Tech stack
json, openapi
Domain
api
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.