-
Notifications
You must be signed in to change notification settings - Fork 492
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
Stress/perf automation #1077
base: main
Are you sure you want to change the base?
Stress/perf automation #1077
Conversation
Note: includes #1066 |
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
File upload tests likely fixed by #1065 |
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 2 pipeline(s). |
Unrelated multi-connection test failures in Canary. |
/azp run |
Commenter does not have sufficient privileges for PR 1077 in repo Azure/azure-iot-sdk-csharp |
@drwill-ms @prmathur-microsoft @abhipsaMisra what's the status on this one? |
@CIPop , I am not able to update your branch to pull changes from master. Could you update your branch; I'll then run the E2E tests and that'll give us an idea if we need to update this PR to incorporate the changes we've made to the SDK since then. |
Firewall, adding KPI and exception faulting.
… before exiting the process.
Ah, sorry about that - I've protected my branch to avoid deleting it by mistake. I have updated it locally, will unlock then push new changes. |
/azp run csharp-westcentralus |
Azure Pipelines successfully started running 1 pipeline(s). |
@CIPop Hey Chris, do you see |
My recommendation would be to run them nightly and monitor the results or tune the KPI parameters to avoid perf regressions (i.e. with each perf enhancement). |
DevOps perf/stress automation.
Checklist
master
branch.Description of the changes