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

chore(docker): include python suffix tags on rc images #1194

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

jansegre
Copy link
Member

Motivation

I couldn't use -python3.12 suffix on the release candidate image for testing.

Acceptance Criteria

  • Include -python{Version} tags even for rc images

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

@jansegre jansegre requested review from glevco and luislhl December 11, 2024 16:30
@jansegre jansegre self-assigned this Dec 11, 2024
@jansegre jansegre requested a review from msbrogli as a code owner December 11, 2024 16:30
glevco
glevco previously approved these changes Dec 11, 2024
Copy link

github-actions bot commented Dec 11, 2024

🐰 Bencher Report

Branchchore/rc-include-python-suffix
Testbedubuntu-22.04

🚨 1 Alert

BenchmarkMeasure
Units
ViewBenchmark Result
(Result Δ%)
Lower Boundary
(Limit %)
sync-v2 (up to 20000 blocks)Latency
nanoseconds (ns)
📈 plot
🚨 alert (🔔)
🚷 threshold
82,874,414,981.34
(-16.59%)
89,423,483,320.23
(107.90%)
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
nanoseconds (ns)
(Result Δ%)
Lower Boundary
nanoseconds (ns)
(Limit %)
Upper Boundary
nanoseconds (ns)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚨 view alert (🔔)
🚷 view threshold
82,874,414,981.34
(-16.59%)
89,423,483,320.23
(107.90%)
109,295,368,502.50
(75.83%)
🐰 View full continuous benchmarking report in Bencher

luislhl
luislhl previously approved these changes Dec 11, 2024
@jansegre jansegre dismissed stale reviews from luislhl and glevco via c57826b December 11, 2024 18:29
@jansegre jansegre force-pushed the chore/rc-include-python-suffix branch from d6d34f7 to c57826b Compare December 11, 2024 18:29
@jansegre jansegre force-pushed the chore/rc-include-python-suffix branch from c57826b to f893765 Compare December 11, 2024 21:13
Copy link

codecov bot commented Dec 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.87%. Comparing base (f6bf33a) to head (f893765).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1194      +/-   ##
==========================================
- Coverage   84.95%   84.87%   -0.08%     
==========================================
  Files         323      323              
  Lines       25115    25115              
  Branches     3859     3859              
==========================================
- Hits        21336    21316      -20     
- Misses       3036     3050      +14     
- Partials      743      749       +6     

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

@jansegre jansegre merged commit 191dac6 into master Dec 11, 2024
9 of 10 checks passed
@jansegre jansegre deleted the chore/rc-include-python-suffix branch December 11, 2024 22:14
@jansegre jansegre mentioned this pull request Dec 19, 2024
2 tasks
@jansegre jansegre mentioned this pull request Jan 9, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants