Skip to content

Add ARM lease entries for KubernetesConfiguration services and refresh DesktopVirtualization lease#44242

Merged
tejaswiMinnu merged 4 commits into
Azure:mainfrom
vikeshi26:vikeshi26/arm-lease-postgresql-remove-uppercase
Jun 26, 2026
Merged

Add ARM lease entries for KubernetesConfiguration services and refresh DesktopVirtualization lease#44242
tejaswiMinnu merged 4 commits into
Azure:mainfrom
vikeshi26:vikeshi26/arm-lease-postgresql-remove-uppercase

Conversation

@vikeshi26

Copy link
Copy Markdown
Member

Summary

This PR includes the 3 requested lease updates:

  1. Add lease for kubernetesResources under Microsoft.KubernetesConfiguration
  2. Add lease for upgradeAssessments under Microsoft.KubernetesConfiguration
  3. Update DesktopVirtualization lease start date and duration to current/P180D, reviewer @vikeshi26

Files changed

  • .github/arm-leases/kubernetesconfiguration/Microsoft.KubernetesConfiguration/kubernetesResources/lease.yaml
  • .github/arm-leases/kubernetesconfiguration/Microsoft.KubernetesConfiguration/upgradeAssessments/lease.yaml
  • .github/arm-leases/desktopvirtualization/Microsoft.DesktopVirtualization/DesktopVirtualization/lease.yaml

Vikeshi Tiwari added 3 commits June 10, 2026 13:34
…llision 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
https://github.com/Azure/azure-rest-api-specs-pr/pull/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.
@github-actions

github-actions Bot commented Jun 26, 2026

Copy link
Copy Markdown

Next Steps to Merge

Next steps that must be taken to merge this PR:
  • ❌ The required check named Protected Files has failed. Refer to the check in the PR's 'Checks' tab for details on how to fix it and consult the aka.ms/ci-fix guide


Comment generated by summarize-checks workflow run.

@tejaswiMinnu tejaswiMinnu merged commit fc12374 into Azure:main Jun 26, 2026
45 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants