Skip to content

Consolidate Blob and File Share client configuration - #7281

Draft
Jinming-Hu wants to merge 1 commit into
mainfrom
stgxx_fix_issue_6952
Draft

Consolidate Blob and File Share client configuration#7281
Jinming-Hu wants to merge 1 commit into
mainfrom
stgxx_fix_issue_6952

Conversation

@Jinming-Hu

Copy link
Copy Markdown
Member

Purpose

  • Add internal BlobClientConfiguration and ShareClientConfiguration value objects, following the existing Data Lake configuration pattern.
  • Propagate client-level options through one configuration object instead of passing and storing multiple independent fields.
  • Update Data Lake rename paths for the consolidated private Blob client constructor.
  • Leave Queue unchanged because its child-client construction already passes only the URL and shared pipeline; it has no duplicated configuration bundle.

Validation

  • Built the Blob, File Share, and Data Lake test targets in Release mode on Windows.
  • Blob playback: 131 passed, 20 skipped after excluding shared-key-only tests and one baseline playback failure.
  • File Share playback: 93 passed, 4 skipped after excluding shared-key-only tests and one baseline playback failure.
  • Data Lake playback: 79 passed, 10 skipped after excluding shared-key-only tests.
  • Confirmed the excluded BlockBlobClientTest.AbortCopy_PLAYBACKONLY_ and FileShareFileClientTest.DestinationOAuthUploadRangeFromUri_PLAYBACKONLY_ failures reproduce unchanged on the unmodified baseline branch.

Closes #6952

Group Blob and File Share client-level options into internal configuration objects so child clients propagate one coherent state bundle. Adapt Data Lake rename paths to the consolidated Blob constructor.

Closes #6952

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot added the Storage Storage Service (Queues, Blobs, Files) label Aug 5, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add client-level configurations for Blobs/FIles/Queues Client to avoid multi-paramters passing around

1 participant