Remove duplicate-casing PostgreSql ARM lease file#43887
Closed
timotheeguerin with Copilot wants to merge 1 commit into
Closed
Remove duplicate-casing PostgreSql ARM lease file#43887timotheeguerin with Copilot wants to merge 1 commit into
timotheeguerin with Copilot wants to merge 1 commit into
Conversation
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
timotheeguerin
June 10, 2026 16:13
View session
Next Steps to MergeNext steps that must be taken to merge this PR:
Comment generated by summarize-checks workflow run. |
timotheeguerin
requested review from
AkhilaIlla,
MSEvanhi,
raosuhas and
tejaswiMinnu
as code owners
tejaswiMinnu
pushed a commit
that referenced
this pull request
Jun 10, 2026
…llision fix) (#43889) Removes .github/arm-leases/postgresql/Microsoft.DBforPostgreSQL/PostgreSQL/lease.yaml which case-collides with the sibling PostgreSql/lease.yaml on Windows/macOS filesystems and is no longer referenced by the spec layout. The lowercase PostgreSql/ folder matches the service folder casing introduced by Azure/azure-rest-api-specs-pr#28858 / #43777 (postgresql folder restructure), so keeping PostgreSql/ preserves lease coverage for that PR while eliminating the case-only duplicate. Supersedes #43887, which removed the lowercase variant; removing the lowercase one would re-trigger ARMModelingReviewRequired on the partner restructure PR because the lease checker looks up the exact-cased path. Co-authored-by: Vikeshi Tiwari <vikestiwari@microsoft.com>
glecaros
pushed a commit
to daniel-rodrigues809/azure-rest-api-specs
that referenced
this pull request
Jun 17, 2026
…llision fix) (Azure#43889) Removes .github/arm-leases/postgresql/Microsoft.DBforPostgreSQL/PostgreSQL/lease.yaml which case-collides with the sibling PostgreSql/lease.yaml on Windows/macOS filesystems and is no longer referenced by the spec layout. The lowercase PostgreSql/ folder matches the service folder casing introduced by Azure/azure-rest-api-specs-pr#28858 / Azure#43777 (postgresql folder restructure), so keeping PostgreSql/ preserves lease coverage for that PR while eliminating the case-only duplicate. Supersedes Azure#43887, which removed the lowercase variant; removing the lowercase one would re-trigger ARMModelingReviewRequired on the partner restructure PR because the lease checker looks up the exact-cased path. Co-authored-by: Vikeshi Tiwari <vikestiwari@microsoft.com>
tejaswiMinnu
added a commit
that referenced
this pull request
Jun 26, 2026
…h DesktopVirtualization lease (#44242) * Remove stale Microsoft.DBforPostgreSQL/PostgreSQL/ ARM lease (case-collision fix) Removes .github/arm-leases/postgresql/Microsoft.DBforPostgreSQL/PostgreSQL/lease.yaml which case-collides with the sibling PostgreSql/lease.yaml on Windows/macOS filesystems and is no longer referenced by the spec layout. The lowercase PostgreSql/ folder matches the service folder casing introduced by Azure/azure-rest-api-specs-pr#28858 / #43777 (postgresql folder restructure), so keeping PostgreSql/ preserves lease coverage for that PR while eliminating the case-only duplicate. Supersedes #43887, which removed the lowercase variant; removing the lowercase one would re-trigger ARMModelingReviewRequired on the partner restructure PR because the lease checker looks up the exact-cased path. * Add ARM leases for KubernetesConfiguration services and refresh DesktopVirtualization lease * Set DesktopVirtualization lease reviewer to @vikeshi26 --------- Co-authored-by: Vikeshi Tiwari <vikestiwari@microsoft.com> Co-authored-by: Tejaswi Salaigari <tsalaigari@microsoft.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This file is a duplicate with just a casing difference. It was added in this pr #43785