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

[WIP] Debug Perf test/framework with storage-blob #12653

Closed

Conversation

HarshaNalluru
Copy link
Member

@HarshaNalluru HarshaNalluru commented Nov 20, 2020

TO REPRO

  • Checkout this branch
  • Create a storage account
  • Add a .env file in test-utils/perfstress/ folder with ACCOUNT_NAME and ACCOUNT_KEY variables
  • Build - rush update && rush build -t test-utils-perfstress
  • To run the test (this seems to fail at the fourth request with 404 ContainerNotFound)
    • rushx build && npm run perfstress-test:node -- StorageBlobDownloadTest --warmup 2 --duration 7 --iterations 2 --parallel 2
  • To run the test without the parallel flag
    • rushx build && npm run perfstress-test:node -- StorageBlobDownloadTest --warmup 2 --duration 7 --iterations 2

@HarshaNalluru
Copy link
Member Author

Closing since the issue is resolved.

ghost pushed a commit that referenced this pull request Dec 2, 2020
Reference - #12653

This PR mainly focuses on 
- improving the type inference for the options(uses typescript magic to achieve it)
- introduces a new parsedOptions that can be consumed in the tests rather than overriding and consuming the original options bag
- required for #12737 which adds tests for storage-blob

Any issues with the functionality of the options or otherwise will be caught while we add new tests in upcoming PRs.
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.

1 participant