-
Notifications
You must be signed in to change notification settings - Fork 230
action: add benchmark on schedule #1254
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
Conversation
d59bbe9
to
f113feb
Compare
switch into daily or every two days? |
Codecov Report
@@ Coverage Diff @@
## master #1254 +/- ##
==========================================
- Coverage 45.08% 45.06% -0.03%
==========================================
Files 126 126
Lines 37395 37395
Branches 37395 37395
==========================================
- Hits 16860 16851 -9
- Misses 19648 19656 +8
- Partials 887 888 +1 |
Is that too frequent? We want to compare this benchmark with the last time benchmark. |
44bd80f
to
06d0b79
Compare
I'm afraid it's hard to find the real commit which causes regression with a per-week benchmark test. |
You are right. I will switch to every two days. |
06d0b79
to
df3bccc
Compare
df3bccc
to
9d40b75
Compare
956b085
to
d98cfc7
Compare
4d42dcf
to
34c2d45
Compare
we will run benchmark twice per-week, and compare result with the last by cache. Signed-off-by: Yadong Ding <[email protected]>
supprot the benchmark on schedule by new mode in benchmark_summary. Signed-off-by: Yadong Ding <[email protected]>
34c2d45
to
83fbf80
Compare
@Desiki-high , the title has been updated, so a new test job has been submitted. Please wait in patience. The test job url: https://tone.openanolis.cn/ws/nrh4nnio/test_result/70728 |
@Desiki-high , The CI test is completed, please check result:
Congratulations, your test job passed! |
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.
LGTM!
Add the benchmark workflow
Add benchmark-schedule mode on benchmark summary
You can see the result in 1, 2, 3, 4.
Ref:#1220