DEVX-146: Scale check_test_speed limits on CI runners #233

Merged
emo merged 2 commits from DEVX-146-check-test-speed-ci-runner-independent into master 2026-08-03 21:40:19 +00:00
Owner

Problem

CI runners (Gitea Actions Docker containers) are 3-4x slower than local dev machines due to shared CPU, fewer cores, and container overhead. This caused false CI failures — the same 3085 tests took 37.98s on CI vs 4.76s local, exceeding the 18s limit.

This blocked PR #945 (OBL-INFRA-511-outage-post-mortem-fixes) from merging.

Solution

When the CI or GITEA_ACTIONS env var is set, both total and per-test limits are multiplied by CI_SCALE_FACTOR (default 4, overridable via DEVX_CI_SCALE_FACTOR env var). This keeps local budgets strict while preventing false failures on slower CI runners.

Closes DEVX-146

## Problem CI runners (Gitea Actions Docker containers) are 3-4x slower than local dev machines due to shared CPU, fewer cores, and container overhead. This caused false CI failures — the same 3085 tests took 37.98s on CI vs 4.76s local, exceeding the 18s limit. This blocked PR #945 (OBL-INFRA-511-outage-post-mortem-fixes) from merging. ## Solution When the CI or GITEA_ACTIONS env var is set, both total and per-test limits are multiplied by CI_SCALE_FACTOR (default 4, overridable via DEVX_CI_SCALE_FACTOR env var). This keeps local budgets strict while preventing false failures on slower CI runners. Closes DEVX-146
emil added 1 commit 2026-08-03 21:23:04 +00:00
fix: scale check_test_speed limits on CI runners
CI / validate (pull_request) Failing after 1m27s
CI / auto-merge (pull_request) Skipped
a912c619ce
CI runners (Gitea Actions Docker containers) are 3-4x slower than
local dev machines due to shared CPU, fewer cores, and container
overhead. This caused false CI failures (37.98s on CI vs 4.76s local
for the same 3085 tests).

When the CI or GITEA_ACTIONS env var is set, both total and per-test
limits are multiplied by CI_SCALE_FACTOR (default 4, overridable via
DEVX_CI_SCALE_FACTOR). This keeps local budgets strict while
preventing false failures on slower CI runners.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
emil added 1 commit 2026-08-03 21:31:52 +00:00
fix: correct DEVX_VIKUNJA_PROJECT_ID from 2 to 8
CI / validate (pull_request) Successful in 1m26s
CI / auto-merge (pull_request) Successful in 23s
d40b16af3d
Project 2 is "Oblachno", project 8 is "devx". The wrong project ID
caused pre-merge validation to fail with "Could not find Vikunja
task DEVX-146 in project 2".

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
emo reviewed 2026-08-03 21:33:14 +00:00
emo left a comment
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emo reviewed 2026-08-03 21:36:44 +00:00
emo left a comment
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emo approved these changes 2026-08-03 21:37:03 +00:00
Dismissed
emo left a comment
Owner

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emo reviewed 2026-08-03 21:39:54 +00:00
emo left a comment
Owner

Automated PR Review

  • Architecture compliance: OK
  • Best practices: OK
  • Security: OK
  • i18n: OK
  • Resource management: OK
  • Documentation: WARNING — source files changed but no docs updated
  • Tests: OK
  • Commit conventions: OK

No issues found by automated checks.


Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: WARNING — source files changed but no docs updated - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.
emo approved these changes 2026-08-03 21:40:16 +00:00
emo left a comment
Owner

Auto-approved: all CI checks passed (validate job).

Auto-approved: all CI checks passed (validate job).
emo merged commit 9bb461e12f into master 2026-08-03 21:40:19 +00:00
emo deleted branch DEVX-146-check-test-speed-ci-runner-independent 2026-08-03 21:40:19 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: oblachno-oss/devx#233