Skip to content

Conversation

@matmair
Copy link
Contributor

@matmair matmair commented Jan 11, 2026

Reverses #11099 as we are getting much more resources by codspeed so we should not run into issues anymore

@matmair matmair added this to the 1.2.0 milestone Jan 11, 2026
@matmair matmair self-assigned this Jan 11, 2026
@matmair matmair added the CI CI / unit testing ecosystem label Jan 11, 2026
@netlify
Copy link

netlify bot commented Jan 11, 2026

Deploy Preview for inventree-web-pui-preview canceled.

Name Link
🔨 Latest commit c0f6884
🔍 Latest deploy log https://app.netlify.com/projects/inventree-web-pui-preview/deploys/696753a3b73ff20008ddf350

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 11, 2026

CodSpeed Performance Report

Merging this PR will degrade performance by 67.94%

Comparing matmair:re-enable-codspeed-runner (c0f6884) with master (8715151)

Summary

❌ 44 regressed benchmarks
⏩ 29 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
WallTime test_api_list_performance[/api/build/item/] 196 ms 603.1 ms -67.51%
WallTime test_api_options_performance[/api/order/po-line/] 155.6 ms 457.3 ms -65.98%
WallTime test_api_list_performance[/api/part/category/] 17 ms 48.9 ms -65.2%
WallTime test_api_options_performance[/api/user/roles/] 6.3 ms 18.4 ms -65.81%
WallTime test_api_list_performance[/api/stock/location/] 21 ms 59.9 ms -65.05%
WallTime test_api_list_performance[/api/parameter/template/] 9 ms 26.3 ms -65.85%
WallTime test_api_list_performance[/api/build/] 42.9 ms 129.2 ms -66.81%
WallTime test_api_options_performance[/api/order/po/] 48 ms 143.6 ms -66.59%
WallTime test_api_options_performance[/api/part/category/] 14.7 ms 43.5 ms -66.21%
WallTime test_api_list_performance[/api/stock/] 159.8 ms 475 ms -66.35%
WallTime test_api_list_performance[/api/part/] 77.9 ms 221.2 ms -64.78%
WallTime test_api_list_performance[/api/parameter/] 11.8 ms 34.4 ms -65.61%
WallTime test_api_options_performance[/api/parameter/template/] 9.1 ms 27.6 ms -66.94%
WallTime test_api_list_performance[/api/order/so/shipment/] 21.7 ms 62.8 ms -65.41%
WallTime test_api_list_performance[/api/user/roles/] 9 ms 28.2 ms -67.94%
WallTime test_api_list_performance[/api/order/so/] 42.1 ms 119.1 ms -64.62%
WallTime test_api_list_performance[/api/company/] 19.1 ms 55.1 ms -65.42%
WallTime test_search_performance[part] 62.5 ms 184.6 ms -66.15%
WallTime test_api_options_performance[/api/company/] 21.7 ms 67.1 ms -67.69%
WallTime test_api_options_performance[/api/stock/location/] 24.9 ms 77.3 ms -67.76%
... ... ... ... ... ...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

Footnotes

  1. 29 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@codecov
Copy link

codecov bot commented Jan 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.16%. Comparing base (8715151) to head (c0f6884).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11120      +/-   ##
==========================================
- Coverage   88.17%   88.16%   -0.01%     
==========================================
  Files        1290     1290              
  Lines       58183    58183              
  Branches     1969     1969              
==========================================
- Hits        51300    51299       -1     
- Misses       6392     6393       +1     
  Partials      491      491              
Flag Coverage Δ
backend 89.46% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
Backend Apps 92.01% <ø> (ø)
Backend General 93.51% <ø> (ø)
Frontend 70.84% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@SchrodingersGat
Copy link
Member

Any idea why the performance has degraded so much on this change?

@matmair
Copy link
Contributor Author

matmair commented Jan 13, 2026

@SchrodingersGat in my testing the codspeed runners seem to be less performant than the machines by GitHub.

@SchrodingersGat
Copy link
Member

@matmair some conflicts have snuck in here

@matmair
Copy link
Contributor Author

matmair commented Jan 14, 2026

@SchrodingersGat conflicts resolved

@SchrodingersGat SchrodingersGat merged commit 22d6f7d into inventree:master Jan 14, 2026
38 checks passed
@SchrodingersGat SchrodingersGat deleted the re-enable-codspeed-runner branch January 14, 2026 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI CI / unit testing ecosystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants