Skip to content

fix(benchmark): fix CI on master#1117

Merged
glevco merged 1 commit intomasterfrom
fix/benchmark-ci
Aug 21, 2024
Merged

fix(benchmark): fix CI on master#1117
glevco merged 1 commit intomasterfrom
fix/benchmark-ci

Conversation

@glevco
Copy link
Contributor

@glevco glevco commented Aug 20, 2024

Motivation

PR #1112 introduced a regression in benchmark CI because it updated the PR workflow, but not the base branch workflow, and both should have been updated. This PR fixes it.

Acceptance Criteria

  • Move benchmark CI env vars to its own file so it can be used in both workflows.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@glevco glevco self-assigned this Aug 20, 2024
@glevco glevco marked this pull request as ready for review August 20, 2024 22:13
@github-actions
Copy link

github-actions bot commented Aug 20, 2024

🐰Bencher

ReportWed, August 21, 2024 at 00:05:21 UTC
Projecthathor-core
Branchfix/benchmark-ci
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyLatency Results
nanoseconds (ns) | (Δ%)
Latency Lower Boundary
nanoseconds (ns) | (%)
Latency Upper Boundary
nanoseconds (ns) | (%)
sync-v2 (up to 20000 blocks)✅ (view plot)101,406,596,606.58 (-0.01%)81,129,754,896.65 (80.00%)121,694,632,344.98 (83.33%)

Bencher - Continuous Benchmarking
View Public Perf Page
Docs | Repo | Chat | Help

@codecov
Copy link

codecov bot commented Aug 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.84%. Comparing base (7f03142) to head (e8f3aeb).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1117      +/-   ##
==========================================
- Coverage   84.85%   84.84%   -0.01%     
==========================================
  Files         315      315              
  Lines       24071    24071              
  Branches     3647     3647              
==========================================
- Hits        20425    20424       -1     
- Misses       2930     2931       +1     
  Partials      716      716              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glevco glevco merged commit 7ccc846 into master Aug 21, 2024
@glevco glevco deleted the fix/benchmark-ci branch August 21, 2024 01:06
@glevco glevco mentioned this pull request Aug 21, 2024
1 task
@jansegre jansegre mentioned this pull request Oct 4, 2024
2 tasks
@jansegre jansegre mentioned this pull request Dec 11, 2024
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants