Skip to content

Swagger changes for Paginated List DDM rules API#36819

Merged
ericshape merged 1 commit intoAzure:publish-sql-Microsoft.Sql-2025-02-01-previewfrom
skpadhy33:dev/santoshpadhy/ListDDMAPIPagination
Sep 2, 2025
Merged

Swagger changes for Paginated List DDM rules API#36819
ericshape merged 1 commit intoAzure:publish-sql-Microsoft.Sql-2025-02-01-previewfrom
skpadhy33:dev/santoshpadhy/ListDDMAPIPagination

Conversation

@skpadhy33
Copy link
Contributor

We are introducing pagination for the List Database Masking Rules API in version 2025-02-01. The API would accept a $skip (int64) query parameter. In this PR, we introduce the corresponding Swagger changes, including some clean-up, etc.

@microsoft-github-policy-service microsoft-github-policy-service bot added customer-reported Issues that are reported by GitHub users external to the Azure organization. SQL labels Aug 20, 2025
@github-actions
Copy link

github-actions bot commented Aug 20, 2025

Next Steps to Merge

⌛ Please wait. Next steps to merge this PR are being evaluated by automation. ⌛

Comment generated by summarize-checks workflow run.

@github-actions
Copy link

github-actions bot commented Aug 20, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Language API Review for Package
Go sdk/resourcemanager/sql/armsql
Java com.azure.resourcemanager:azure-resourcemanager-sql-generated
C# Azure.ResourceManager.Sql
JavaScript @azure/arm-sql

@github-actions github-actions bot added the brownfield Brownfield services will soon be required to convert to TypeSpec. See https://aka.ms/azsdk/typespec. label Aug 25, 2025
@ericshape
Copy link
Member

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@ericshape
Copy link
Member

@skpadhy33 could you reply this message above to unblock the merge?

image

@skpadhy33
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@skpadhy33 skpadhy33 closed this Aug 28, 2025
@skpadhy33
Copy link
Contributor Author

@skpadhy33 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement
Reopening.

@skpadhy33 skpadhy33 reopened this Aug 28, 2025
@ericshape
Copy link
Member

@skpadhy33 it seems you need to reply that comment as
@microsoft-github-policy-service agree company="Microsoft"

@skpadhy33
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@skpadhy33
Copy link
Contributor Author

@skpadhy33 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@microsoft-github-policy-service agree [company="{your company}"]

Options:

  • (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
  • (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement
Reopening.

@microsoft-github-policy-service agree company="Microsoft"

@ericshape ericshape merged commit dab6566 into Azure:publish-sql-Microsoft.Sql-2025-02-01-preview Sep 2, 2025
67 of 80 checks passed
jeremyfrosti added a commit that referenced this pull request Mar 4, 2026
* Create base of release-sql-Microsoft.Sql-2025-02-01-preview via Copy-ApiVersion.ps1 (#36370)

* Copy files from preview/2024-11-01-preview

Copied the files in a separate commit.
This allows reviewers to easily diff subsequent changes against the previous spec.

* Update version to preview/2025-02-01-preview

Updated the API version from preview/2024-11-01-preview to preview/2025-02-01-preview.

* Added tag for 2025-02-01-preview in readme file

---------

Co-authored-by: Mitesh Vishwakarma <mvishwakarma@microsoft.com>

* Swagger changes for Paginated List DDM rules API (#36819)

Co-authored-by: Santosh Padhy <santoshpadhy@microsoft.com>

* Datasync UAMI support API in 2025-02-01-preview (#35876)

* Datasync UAMI support API in 2025-02-01-preview

* Updated json to fix validation failure

* Spell, lint fix

* Fix error message

* Updating error codes

---------

Co-authored-by: Hareesh Kumar Gajulapalli <hgajulapalli@microsoft.com>

* Azure SQL API Parity (#38442)

* Azure SQL API Parity

* avocado error update

* fixes lint diff errors

---------

Co-authored-by: mvishwakarma <mvishwakarma@microsoft.com>

* Removing datasync breaking change because of required fields (#39059)

Co-authored-by: Hareesh Kumar Gajulapalli <hgajulapalli@microsoft.com>

* Added back the legacy params (#39317)

Co-authored-by: Santosh Padhy <santoshpadhy@microsoft.com>

* 20250201-preview breaking change fix for databases.json (#40027)

* breaking change fix

* fix model validation failure

---------

Co-authored-by: mvishwakarma <mvishwakarma@microsoft.com>

* fixing model validation errors (#40364)

* fixing model validation errors

* adds model validation fix

* fixes model validation errors

* Fixes model validation errors

* fixes model validation failures

* model valdation fixes '

* model validation fix

* model validation fixes

* fixes model validation errors

---------

Co-authored-by: mvishwakarma <mvishwakarma@microsoft.com>

* fix semantic error

* fix semantic error

* add v2

* fix breaking change (#40593)

Co-authored-by: mvishwakarma <mvishwakarma@microsoft.com>

* Adding suppressions previously approved in #38470 (#40764)

* Removing the list import export operation results from Servers API (#40836)

* Adding suppressions previously approved in #38470

* Removing the list import export operation results from Servers API

* Fix for lint-diff errors (#40792)

* Fix for lint-diff errors

* Rectifies suppression for ResourceNamePattern

* Remove ResourceNameRestriction validation code

Removed ResourceNameRestriction code from parameters.

* Fix formatting issue

Fix formatting issue

---------

Co-authored-by: mvishwakarma <mvishwakarma@microsoft.com>
Co-authored-by: Jeremy Frosti <111934615+jeremyfrosti@users.noreply.github.com>

* Add suppression for ElasticPools to match existing suppression in Databases (#40862)

* Adding suppressions previously approved in #38470

* Removing the list import export operation results from Servers API

* Adding suppression for new currentSku in ElasticPool update

---------

Co-authored-by: Mitesh Vishwakarma <mvishwakarma@microsoft.com>
Co-authored-by: skpadhy33 <35034191+skpadhy33@users.noreply.github.com>
Co-authored-by: Santosh Padhy <santoshpadhy@microsoft.com>
Co-authored-by: hareeshghk <hareeshghk@users.noreply.github.com>
Co-authored-by: Hareesh Kumar Gajulapalli <hgajulapalli@microsoft.com>
Co-authored-by: zedy <zedy@wicresoft.com>
Co-authored-by: Jeremy Frosti <111934615+jeremyfrosti@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

brownfield Brownfield services will soon be required to convert to TypeSpec. See https://aka.ms/azsdk/typespec. customer-reported Issues that are reported by GitHub users external to the Azure organization. SQL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants