Repository navigation
[SQL] Add reader endpoint APIs for 2026-11-01-preview - #47504
rajashree012 wants to merge 5 commits into
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Thank you for your contribution rajashree012! We will review the pull request and get back to you soon. |
|
Azure Pipelines: Successfully started running 7 pipeline(s). 3 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
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. |
Require tracked-resource location, expose provisioning state, retain nested properties and optional PATCH databases, and add minimal and tags-only examples. Emit the read-only state inline so Swagger validators preserve its visibility. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
/arm-review |
TypeSpec suppressions requiring reviewStatus: ❌ Approval required — 1 suppression This PR adds or updates the TypeSpec suppressions listed below. Suppressions are strongly discouraged — they bypass linter rules that protect API quality and consistency. Authors should avoid adding new suppressions and prefer fixing the underlying issue; reviewers should approve only when there is a clear, compelling justification and no reasonable alternative. Review each linked rule and source location, then apply New suppressions (1)
For an overview of TypeSpec linting rules click here. 💬 Have feedback on the TypeSpec suppression flow? Let us know. |
|
/arm-review |
|
/arm-review |
ARM API Review SummaryReviewed PR #47504 at head SHA Approval labels observed: none.
No issues found. The ReaderEndpoint TypeSpec, generated ARM OpenAPI, LRO behavior, CRUD/list operations, pagination, mutability, examples, version registration, and scoped suppressions are consistent. posted-by: arm-api-reviewer-agent | rule: summary | category: summary | severity: suggestion | classification: new | critic: pass | head-sha: 4842c33
|
There was a problem hiding this comment.
ARM API Review
Posting findings from the ARM API Reviewer agent (critic-verified, 1 iteration, no findings) against commit 4842c335955cf4e694c5743acc30831c71682a29. All findings are reported in the summary comment.
Approval labels observed: none.
posted-by: arm-api-reviewer-agent | rule: summary | category: summary | severity: suggestion | classification: existing | critic: pass | head-sha: 4842c33
🔍 ARM API review by ARM API Review: Automated Workflow
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Document completed 200, pending 202, and absent 204 DELETE responses with targeted exceptions requiring ARM approval. Reuse the existing generator-only feature registry without changing generated contracts. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
rajashree012 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
2026-11-01-previewSQL release branch, with matching TypeSpec source and generated OpenAPI.Summary
Adds six reader endpoint operations for November 2026 preview only:
Includes six request/response examples, reader endpoint models, and the November input-file registrations. Preserves request requirements, response codes, Location-based long-running operations, pagination, property mutability, and read-only location. The create/update request example omits read-only location; responses retain it.
No older API versions or workflow files are changed. The public schema uses
ReaderEndpointas its model name and the SQL project's existing common-type version. Narrow TypeSpec suppressions preserve the existing contract and feature-splitting conventions; their approval remains part of API review.Validation
tsp compile .completed successfully with the AutoRest emitter.azsdk_run_typespec_validationreturnedSucceeded.common.jsononly gains reader definitions.Due diligence checklist
Draft / release readiness
This is intentionally a draft. Service deployment, release readiness, required API/suppression reviews, and CI results must be confirmed before merging. This PR does not assert production availability or authorize release publication.
The change is split into a generated Swagger commit followed by its matching TypeSpec, examples, and registration commit.