-
Notifications
You must be signed in to change notification settings - Fork 127
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
Enable profiling in perf piplines. #4398
Conversation
ignore cspell when reviewing for the moment, Daniel is looking into why it failed, as there are no real failures, it seems to have grabbed the wrong version of cspell for some reason |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks reasonable to me.
The spell checker issue is fixed. Rebase this PR to get a more successful run of the spell checker. |
Co-authored-by: Mike Harder <mharder@microsoft.com>
/azp run cpp - core - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
* profiling * dsadsada * Update eng/pipelines/templates/jobs/perf.yml Co-authored-by: Mike Harder <mharder@microsoft.com> --------- Co-authored-by: Mike Harder <mharder@microsoft.com> # Conflicts: # sdk/keyvault/azure-security-keyvault-certificates/perf.yml # sdk/keyvault/azure-security-keyvault-keys/perf.yml # sdk/keyvault/azure-security-keyvault-secrets/perf.yml
* profiling * dsadsada * Update eng/pipelines/templates/jobs/perf.yml Co-authored-by: Mike Harder <mharder@microsoft.com> --------- Co-authored-by: Mike Harder <mharder@microsoft.com>
To enable profiling , along with chnges in the automation the changes to the pipelines are required
Pull Request Checklist
Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:
See the detailed list in the contributing guide.