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

benchmarks: profiling #476

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

benchmarks: profiling #476

wants to merge 1 commit into from

Conversation

alecholmez
Copy link
Contributor

@alecholmez alecholmez commented Jul 30, 2021

No description provided.

@alecholmez alecholmez changed the title WIP: profiling Profiling Jul 31, 2021
@alecholmez alecholmez requested a review from snowp July 31, 2021 18:11
@alecholmez alecholmez assigned alecholmez and snowp and unassigned alecholmez Jul 31, 2021
@alecholmez alecholmez changed the title Profiling benchmarks: profiling Aug 2, 2021
@alecholmez
Copy link
Contributor Author

alecholmez commented Aug 2, 2021

This is the first step to running e2e benchmarks on this repo. I've integrated some profiling utilizing pprof so we can understand what the code is really doing and see where we can squeeze a bit more performance out of. The README in ./benchmarks demonstrates how to run the benchmarks and generate the artifacts, but this is only the first step for the e2e stuff. I still need to create test clients that can push this repo to the limits rather than just using envoy and the integration tests. Then we can start comparing/collecting the data and seeing what the results are

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. It will be closed in 7 days if no further activity occurs. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale label Sep 30, 2021
@github-actions
Copy link

github-actions bot commented Oct 7, 2021

This pull request has been automatically closed because it has not had activity in the last 37 days. Please feel free to give a status update now, ping for review, or re-open when it's ready. Thank you for your contributions!

@github-actions github-actions bot closed this Oct 7, 2021
@alecholmez alecholmez deleted the pprof branch October 26, 2021 18:31
@alecholmez alecholmez restored the pprof branch May 17, 2022 16:44
@alecholmez alecholmez reopened this May 17, 2022
@alecholmez alecholmez force-pushed the pprof branch 2 times, most recently from 44a0d6b to 632c311 Compare January 27, 2023 15:57
profiling

Signed-off-by: Alec Holmes <alecholmez@me.com>
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.

2 participants