-
Notifications
You must be signed in to change notification settings - Fork 49
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
Echo and KVcounter integration benchmark tests #552
Conversation
Would this be helpful? I.e. writing out benchmark results to a file and attaching it to a run as an artifact? https://github.com/actions/upload-artifact |
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.
I only noticed this was still in draft when I went to submit the review, so ignore me if these comments aren't relevant :)
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
d5465ad
to
cc9b0a6
Compare
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.
Just nits. LGTM!
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Signed-off-by: Brooks Townsend <brooks@cosmonic.com>
Feature or Problem
This PR aims to add benchmark integration tests for the echo and kvcounter sample actors so that we can compare performance across new features and bug fixes in the OTP host.
Release Information
next
releaseConsumer Impact
The only impact this PR has is that it might slow down some of our integration tests by about a minute since they will be benchmarking.
Testing
Tested on platform(s)
Unit Test(s)
Modified echo, added echo_wasi benchmark
Acceptance or Integration
Ideally, I'd like to include a way to upload benchmark results so we can compare over time.
Manual Verification
I'm just running this locally:
MIX_ENV=test mix test test/host_core/benchmark/echo_test.exs