[Storage] Lease operations for BlobClient and BlobContainerClient, feature parity for AppendBlobClient
#2739
Conversation
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
…s bag extension methods
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
vincenttran-msft
changed the title
BlobClient and BlobContainerClientBlobClient and BlobContainerClient, feature parity for AppendBlobClient
vincenttran-msft
requested review from
LarryOsterman,
RickWinter,
heaths and
jalauzon-msft
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR introduces lease management operations to BlobClient and BlobContainerClient and adds full append-blob support to AppendBlobClient. It includes new API methods, updated models, and end-to-end tests for these features.
- Added
acquire_lease,break_lease,change_lease,release_lease, andrenew_leasemethods (and related options/results) for blobs and containers - Completed
AppendBlobClientwithappend_block,append_block_from_url, andsealmethods - Expanded test suites to cover leasing and append-blob workflows
Reviewed Changes
Copilot reviewed 9 out of 13 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tsp-location.yaml | Updated commit hash for spec linkage |
| tests/blob_container_client.rs | Added lease operation imports and new container lease test |
| tests/blob_client.rs | Added blob lease tests and lease-aware operation tests |
| tests/append_blob_client.rs | Added append-block, append-from-url, and seal tests for append blobs |
| src/models/mod.rs | Expanded model exports for lease and append-blob operations |
| src/clients/blob_container_client.rs | Implemented container lease methods and docs |
| src/clients/blob_client.rs | Implemented blob lease methods and docs |
| src/clients/append_blob_client.rs | Implemented append-blob methods and docs |
| assets.json | Bumped SDK assets tag |
heaths
approved these changes
Jul 10, 2025
jalauzon-msft
approved these changes
Jul 10, 2025
jalauzon-msft
approved these changes
Jul 11, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
.tsp: Azure/azure-rest-api-specs#35564Leasing Operations
BlobClientandBlobContainerClientacquire_lease,break_lease,change_lease,release_lease, andrenew_leaseBlobClientLease Operations:acquire_leasebreak_leasechange_leaserelease_leaserenew_leaseBlobContainerClientLease Operations:acquire_leasebreak_leasechange_leaserelease_leaserenew_leaseAppendBlobClient APIs
AppendBlobClientto make it more usable beyond initially introducedcreateAPIappend_block,append_block_from_url, andsealappend_blockappend_block_from_urlseal