ci: Use docker-benchmark runner for benchmark-publish job #1084

Merged
CoreRasurae merged 1 commit from task/ci-benchmark-publish into master 2026-03-20 20:18:19 +00:00
Member

Job takes more time to run, needing a dedicated runner

ISSUES CLOSED: #1083

Description

Fixes a timeout in benchmark-publish that occurs because this jobs takes longer than the allowed 15 mins for the standard jobs.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (no functional changes)
  • Documentation update
  • Test improvements
  • CI/CD changes

Quality Checklist

  • Code follows the project's coding standards (see CONTRIBUTING.md)
  • All public/protected methods have argument validation
  • Static typing is complete (no Any unless justified)
  • nox -s typecheck passes with no errors
  • nox -s lint passes with no errors
  • Unit tests written/updated (Behave scenarios in features/)
  • Integration tests written/updated (Robot suites in robot/) if applicable
  • Coverage remains above 85% (nox -s coverage_report)
  • No security issues introduced (nox -s security_scan)
  • No dead code introduced (nox -s dead_code)
  • Documentation updated if behavior changed

Testing

Ensure that benchmark-publish job completes in the Forgejo CI, however this job only runs after the merge, however the present resolution has very high chances of fixing the issue.

Test Commands Run

nox -s unit_tests       # Behave tests
nox -s typecheck        # Type checking
nox -s lint             # Linting
nox -s coverage_report  # Coverage

Closes 1083

Implementation Notes

Forgejo CI changes only, no code changes

Job takes more time to run, needing a dedicated runner ISSUES CLOSED: #1083 ## Description Fixes a timeout in benchmark-publish that occurs because this jobs takes longer than the allowed 15 mins for the standard jobs. ## Type of Change - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [ ] Refactoring (no functional changes) - [ ] Documentation update - [ ] Test improvements - [X] CI/CD changes ## Quality Checklist - [X] Code follows the project's coding standards (see CONTRIBUTING.md) - [X] All public/protected methods have argument validation - [X] Static typing is complete (no `Any` unless justified) - [X] `nox -s typecheck` passes with no errors - [X] `nox -s lint` passes with no errors - [X] Unit tests written/updated (Behave scenarios in `features/`) - [X] Integration tests written/updated (Robot suites in `robot/`) if applicable - [X] Coverage remains above 85% (`nox -s coverage_report`) - [X] No security issues introduced (`nox -s security_scan`) - [X] No dead code introduced (`nox -s dead_code`) - [X] Documentation updated if behavior changed ## Testing Ensure that benchmark-publish job completes in the Forgejo CI, however this job only runs after the merge, however the present resolution has very high chances of fixing the issue. ### Test Commands Run ```bash nox -s unit_tests # Behave tests nox -s typecheck # Type checking nox -s lint # Linting nox -s coverage_report # Coverage ``` ## Related Issues Closes 1083 ## Implementation Notes Forgejo CI changes only, no code changes
ci: Use docker-benchmark runner for benchmark-publish job
Some checks failed
CI / build (pull_request) Successful in 21s
CI / lint (pull_request) Successful in 25s
CI / typecheck (pull_request) Successful in 43s
CI / quality (pull_request) Successful in 53s
CI / security (pull_request) Successful in 1m0s
CI / benchmark-publish (pull_request) Has been skipped
CI / integration_tests (pull_request) Successful in 2m58s
CI / unit_tests (pull_request) Successful in 3m20s
CI / docker (pull_request) Successful in 1m2s
CI / e2e_tests (pull_request) Successful in 5m23s
CI / coverage (pull_request) Successful in 8m3s
CI / benchmark-regression (pull_request) Successful in 36m31s
CI / lint (push) Successful in 14s
CI / quality (push) Successful in 26s
CI / build (push) Successful in 17s
CI / typecheck (push) Successful in 38s
CI / security (push) Successful in 38s
CI / benchmark-regression (push) Has been skipped
CI / integration_tests (push) Successful in 2m53s
CI / unit_tests (push) Successful in 3m0s
CI / docker (push) Successful in 1m3s
CI / e2e_tests (push) Successful in 5m32s
CI / benchmark-publish (push) Failing after 29m1s
CI / coverage (push) Successful in 6m8s
d206b0c5e6
Job takes more time to run,  needing a dedicated runner

ISSUES CLOSED: #1083
CoreRasurae added this to the v3.2.0 milestone 2026-03-20 19:37:59 +00:00
salcedo approved these changes 2026-03-20 19:44:38 +00:00
CoreRasurae deleted branch task/ci-benchmark-publish 2026-03-20 20:18:20 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
cleveragents/cleveragents-core!1084
No description provided.