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

Add benchmark job and update benchmark results #713

Closed

Conversation

coderbirju
Copy link
Contributor

WIP: Do not merge

This commit adds a new job 'benchmark' to the build.yml wrokflow, the benchmark workflow will run the make benchmark target. The Dco check needs to run on every code change regardless of whether it is a documentation or code change hence seperating it out into a different workflow, this will run for all pull and merge requests

Issue #, if available:

Description of changes:

  • added new workflow prebuild.yml
  • added new job benchmark to build.yml

Testing performed:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This commit adds a new job 'benchmark' to the build.yml wrokflow, the
benchmark workflow will run the make benchmark target. The Dco check
needs to run on every code change regardless of whether it is a
documentation or code change hence seperating it out into a different
workflow, this will run for all pull and merge requests

Signed-off-by: Arjun Raja Yogidas <arjunry@amazon.com>
@sparr
Copy link
Contributor

sparr commented Jul 6, 2023

Please make this two separate PRs.

@sparr sparr mentioned this pull request Jul 11, 2023
@coderbirju coderbirju changed the title Add new workflow and separate dco check Add benchmark job and update benchmark results Jul 12, 2023
This commit adds additional details like standard deviation,mean,p90
etc.. for individual events - 'Pull','Run Task 1' and 'Run Task
2(Running task without FUSE overhead)'

Signed-off-by: Arjun Raja Yogidas <arjunry@amazon.com>
@coderbirju
Copy link
Contributor Author

Closing this PR as it is a duplicate.

@coderbirju coderbirju closed this Jul 12, 2023
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.

2 participants