[OperationalInsights] Add new 2026-03-01 stable API for OperationalInsights RP#42662
[OperationalInsights] Add new 2026-03-01 stable API for OperationalInsights RP#42662
Conversation
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
Comment generated by After APIView workflow run. |
| "readOnly": true, | ||
| "description": "List of associations for the workspace. Indicates if the workspace is associated with any of the following experiences: MDC, Sentinel, SentinelGraph, etc." | ||
| }, | ||
| "dataAuthorizationMode": { |
There was a problem hiding this comment.
[NEW] 🟡 Warning [Section 8.1 - Enums over booleans]
dataAuthorizationMode is modeled as boolean, which limits forward compatibility if additional modes are introduced later.
Why this matters: adding a third state later becomes a breaking shape change for SDKs and policy logic.
Recommended fix: model this as an extensible string enum (for example Disabled/Enabled, with x-ms-enum.modelAsString: true) instead of boolean.
| }, | ||
| "dataAuthorizationMode": { | ||
| "type": "boolean", | ||
| "x-nullable": true, |
There was a problem hiding this comment.
[NEW] 🟡 Warning [Section 6 - Null values]
This new response property is marked x-nullable: true.
Why this matters: ARM guidance prefers omitting missing values rather than returning explicit null for response fields.
Recommended fix: remove x-nullable: true and have the service omit the field when not set.
| "x-ms-enum": { | ||
| "name": "SummaryLogsIdentityType", | ||
| "modelAsString": false | ||
| } |
There was a problem hiding this comment.
[NEW] 🔴 Blocking [Section 7 - Enum extensibility]
SummaryLogsIdentityType uses x-ms-enum.modelAsString: false.
Why this matters: closed enums are brittle for SDKs and break forward compatibility when the service adds a new value.
Recommended fix: set x-ms-enum.modelAsString to true.
ARM (Control Plane) API Specification Update Pull Request
Tip
Overwhelmed by all this guidance? See the
Getting helpsection at the bottom of this PR description.PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.
Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!
Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.
Additional information
Viewing API changes
New stable API version 2026-03-01 for OperationalInsights with two additions:
Added "protectionLevel" property to table properties. A string enum with values "General" (table data accessible through standard permissions) and "Protected" (table data isolated by default, requires explicit scoped authorization).
Added "dataAuthorizationMode" property to workspace features. A nullable boolean that enables data authorization mode for the workspace.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.
Getting help
Purpose of this PRandDue diligence checklist.write accessper aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositoriesNext Steps to Mergecomment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.and https://aka.ms/ci-fix.
queuedstate, please add a comment with contents/azp run.This should result in a new comment denoting a
PR validation pipelinehas started and the checks should be updated after few minutes.