Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrated blobfs smoke tests #2730

Merged
merged 9 commits into from
Jul 17, 2024
Merged

Migrated blobfs smoke tests #2730

merged 9 commits into from
Jul 17, 2024

Conversation

gapra-msft
Copy link
Member

@gapra-msft gapra-msft commented Jun 20, 2024

test_blobfs_sas_download_1Kb_file -> Scenario_DownloadFile
test_blobfs_sas_download_64MB_file -> Scenario_DownloadFile64MB
test_blobfs_sas_download_100_1Kb_file -> Scenario_Download100Files

test_blobfs_download_1Kb_file -> Scenario_DownloadFile
test_blobfs_download_64MB_file -> skipped anyway
test_blobfs_download_100_1Kb_file -> Scenario_Download100Files
test_blobfs_download_200_1Kb_file -> probaby fine to skip? covered by 100 files anyway

test_blobfs_sas_upload_1Kb_file -> Scenario_UploadFile
test_blobfs_sas_upload_64MB_file -> Scenario_UploadFile64MB
test_blobfs_sas_upload_100_1kb_file -> Scenario_Upload100Files

test_blobfs_upload_1Kb_file_with_oauth -> Scenario_UploadFile
test_blobfs_upload_64MB_file_with_oauth -> skipped anyway
test_blobfs_upload_100_1Kb_file_with_oauth -> Scenario_Upload100Files
test_blobfs_upload_uneven_multiflush_with_oauth -> Scenario_UploadFileUnevenMultiflushOAuth
test_blobfs_upload_even_multiflush_with_oauth -> Scenario_UploadFileEvenMultiflushOAuth

adreed-msft
adreed-msft previously approved these changes Jul 16, 2024
Copy link
Member

@adreed-msft adreed-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, could be compressed down, but otherwise fine.

Base automatically changed from gapra/smokeTestAutoDetectBlobType to main July 16, 2024 22:24
@gapra-msft gapra-msft dismissed adreed-msft’s stale review July 16, 2024 22:24

The base branch was changed.

@gapra-msft gapra-msft merged commit f1121bf into main Jul 17, 2024
17 checks passed
@gapra-msft gapra-msft deleted the gapra/smokeTestBlobFs branch July 17, 2024 01:33
@vibhansa-msft vibhansa-msft added this to the 10.26.0 milestone Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants