Azure Storage Sample Automation

[Yionse]

By changing the current Storage samples, can enable them to run automatically in the pipeline.

For the latest pipeline results: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4713055&view=results

The supported packages are as follows:

• azure-storage-blob
• azure-storage-file-datalake
• azure-storage-file-share
• azure-storage-queue

The specific modifications are as follows:

• Skipped azure-storage-blob/blob_samples_proxy_configuration.py because it uses a proxy and is not suitable for running on pipelines.

• Change the environment variable name in samples to be consistent with that exported in test-resources.json.

• Change the path of the local files used in the samples to an absolute address.
  Update

  SOURCE_FOLDER = "./sample-blobs/"
  

  to

  current_dir = os.path.dirname(os.path.abspath(__file__))
  SOURCE_FOLDER = os.path.join(current_dir, "./sample-blobs/")
  

• Comment out the following code snippet in blob_samples_container_access_policy.py, as setting the allowBlobPublicAccess attribute in test-resources.json is unstable and may not always take effect.

  container_client.set_container_access_policy(signed_identifiers=identifiers, public_access=public_access)
  

• Create required resources and add output in test-resources.json.

@xiangyan99 for notification.

[azure-sdk]

API change check

API changes are not detected in this pull request.

[Copilot]

Pull Request Overview

This PR automates Azure Storage samples for pipeline execution by standardizing environment variable names, adjusting local file paths to absolute paths, and commenting out unstable code sections.

• Environment variables have been renamed from AZURE_STORAGE_* to STORAGE_* in most samples.
• File paths have been converted from relative to absolute paths in several samples.
• Unstable container access policy code has been commented out in async samples.

Reviewed Changes

Copilot reviewed 67 out of 67 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
sdk/storage/azure-storage-blob/samples/blob_samples_client_side_encryption_keyvault.py	Updated env var names and added key generation logic for client-side encryption.
sdk/storage/azure-storage-blob/samples/blob_samples_enumerate_blobs_async.py	Renamed environment variable and updated container name for async enumeration.
sdk/storage/azure-storage-blob/samples/blob_samples_enumerate_blobs.py	Renamed environment variable and updated container name for blob enumeration.
sdk/storage/azure-storage-blob/samples/blob_samples_directory_interface*.py	Renamed environment variable and hardcoded container name for directory interface samples.
scripts/devops_tasks/test_run_samples.py	Added proxy configuration sample to the list of skipped samples.
sdk/storage/azure-storage-blob/samples/blob_samples_container_access_policy_async.py	Renamed environment variable and commented out unstable access policy setting.
sdk/storage/azure-storage-blob/samples/blob_samples_copy_blob*.py	Updated env var names and container names; added container creation call.
sdk/storage/azure-storage-blob/samples/blob_samples_containers*.py	Renamed environment variable, updated file path for source file, and changed container names.
sdk/storage/azure-storage-blob/samples/blob_samples_authentication*	Updated environment variable names for connection string and account keys.
sdk/storage/azure-storage-blob/samples/blob_samples_client_side_encryption.py	Updated environment variable name in sample header and connection string retrieval.
sdk/storage/azure-storage-blob/samples/blob_samples_common*.py	Renamed env var and updated local file path for sample source file.

Comments suppressed due to low confidence (2)

sdk/storage/azure-storage-blob/samples/blob_samples_common.py:32

• The environment variable used here is 'STORAGE_CONNECTION_STRING_SOFT', which is inconsistent with other samples that reference 'STORAGE_CONNECTION_STRING'. Please unify the variable naming.

connection_string = os.getenv("STORAGE_CONNECTION_STRING_SOFT")

sdk/storage/azure-storage-blob/samples/blob_samples_common_async.py:31

• The use of 'STORAGE_CONNECTION_STRING_SOFT' here is inconsistent with other parts of the code that expect 'STORAGE_CONNECTION_STRING'. Consider using the same environment variable name across all samples.

connection_string = os.getenv("STORAGE_CONNECTION_STRING_SOFT")

[jalauzon-msft]

This looks good to me! Thanks for your work on this.

I will start a run of the test pipeline on this PR and assuming that looks good, this should be good to go.

[jalauzon-msft]

/azp run python - storage - tests

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).