ci: quiet CI step logs to cut Forgejo server load #11259

Merged
HAL9000 merged 1 commit from ci-quiet-logs into master 2026-06-02 22:31:50 +00:00
Owner

Applies the log-verbosity reduction from the controller-coverage-optimization branch to master.

Each heavy CI step now writes full output to a build/*-output.log file (still uploaded as an artifact) and streams to the live console only a one-line OK on success, or the COMPLETE log via cat on failure. Forgejo stores every streamed line as the live job log, and that aggregate volume across all controller-driven CI runs is what overloads the server; failure output is byte-identical to before, so no diagnostic data is suppressed.

Also: apt-get update -qq / apk add -q to quiet package-manager chatter; dockerd logs redirected to a file and cat'd only on a docker build failure; paired log-artifact uploads for build/docker/helm/push-validation; TEST_PROCESSES=8 cap on unit/integration tests to prevent OOM SIGKILL (exit 137) on high-core hosts.

The coverage gate (nox --fail-under, propagated via exit code) and the existing skip_coverage operator kill switch are preserved unchanged. The log-bundle job and skip_docker gate were intentionally excluded.

Applies the log-verbosity reduction from the controller-coverage-optimization branch to master. Each heavy CI step now writes full output to a build/*-output.log file (still uploaded as an artifact) and streams to the live console only a one-line OK on success, or the COMPLETE log via cat on failure. Forgejo stores every streamed line as the live job log, and that aggregate volume across all controller-driven CI runs is what overloads the server; failure output is byte-identical to before, so no diagnostic data is suppressed. Also: apt-get update -qq / apk add -q to quiet package-manager chatter; dockerd logs redirected to a file and cat'd only on a docker build failure; paired log-artifact uploads for build/docker/helm/push-validation; TEST_PROCESSES=8 cap on unit/integration tests to prevent OOM SIGKILL (exit 137) on high-core hosts. The coverage gate (nox --fail-under, propagated via exit code) and the existing skip_coverage operator kill switch are preserved unchanged. The log-bundle job and skip_docker gate were intentionally excluded.
ci: quiet CI step logs to cut Forgejo server load
All checks were successful
CI / lint (pull_request) Successful in 37s
CI / build (pull_request) Successful in 50s
CI / quality (pull_request) Successful in 56s
CI / helm (pull_request) Successful in 47s
CI / typecheck (pull_request) Successful in 1m3s
CI / push-validation (pull_request) Successful in 45s
CI / security (pull_request) Successful in 1m14s
CI / unit_tests (pull_request) Successful in 4m12s
CI / docker (pull_request) Successful in 2m25s
CI / integration_tests (pull_request) Successful in 8m21s
CI / coverage (pull_request) Successful in 11m0s
CI / status-check (pull_request) Successful in 5s
81412c40a4
Apply the log-verbosity reduction from controller-coverage-optimization:
each heavy step writes full output to a build/*-output.log file (still
uploaded as an artifact) and streams to the live console only a one-line
"OK" on success, or the COMPLETE log via `cat` on failure. The Forgejo
server stores every streamed line as the live job log, and that aggregate
volume across all controller-driven CI runs is what overloads it; failures
remain byte-identical to before, so no diagnostic data is suppressed.

Also:
- apt-get update -qq / apk add -q to quiet package-manager chatter
- dockerd logs redirected to a file, cat'd only on a docker build failure
- paired log-artifact uploads for build/docker/helm/push-validation
- TEST_PROCESSES=8 cap on unit/integration tests (prevents OOM SIGKILL
  / exit 137 from oversubscribing RAM on high-core hosts)

The coverage gate (nox --fail-under, propagated via exit code) and the
existing skip_coverage operator kill switch are preserved unchanged.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Author
Owner

🌱 Grooming: proceed — PR cleared for processing.

(check no_duplicates, category no_duplicates)

PR #11259 addresses a distinctive scope: reducing CI log verbosity to mitigate Forgejo server load. Comprehensive scan of 452 open PRs found 6 related CI-optimization PRs, all targeting different concerns (Helm caching, quality checks, benchmark tests, tooling, e2e parallelization, expression fixes). None address log-output reduction or server load mitigation. No duplicate detected.

**🌱 Grooming: proceed** — PR cleared for processing. (check `no_duplicates`, category `no_duplicates`) PR #11259 addresses a distinctive scope: reducing CI log verbosity to mitigate Forgejo server load. Comprehensive scan of 452 open PRs found 6 related CI-optimization PRs, all targeting different concerns (Helm caching, quality checks, benchmark tests, tooling, e2e parallelization, expression fixes). None address log-output reduction or server load mitigation. No duplicate detected. <!-- controller:fingerprint:2f55bfdcc24e553c -->
Author
Owner

📋 Estimate: tier 1.

Single CI workflow YAML file, +262/-77 lines. Change applies log-redirection shell patterns (tee to build/*.log, cat on failure), package-manager -q flags, dockerd log file redirect, paired artifact uploads, and TEST_PROCESSES=8 cap across many CI steps. Scope is single-file but the net LOC delta and breadth of step-by-step shell correctness requirements put this above tier 0. Calibration history confirms tier-0 fails on similarly-sized single-file CI edits. Not metadata-only (active CI shell logic). Tier 1, medium confidence — could plausibly be tier 0 if the porting is purely mechanical copy, but the risk of incorrect redirections makes tier 1 the safer call.

**📋 Estimate: tier 1.** Single CI workflow YAML file, +262/-77 lines. Change applies log-redirection shell patterns (tee to build/*.log, cat on failure), package-manager -q flags, dockerd log file redirect, paired artifact uploads, and TEST_PROCESSES=8 cap across many CI steps. Scope is single-file but the net LOC delta and breadth of step-by-step shell correctness requirements put this above tier 0. Calibration history confirms tier-0 fails on similarly-sized single-file CI edits. Not metadata-only (active CI shell logic). Tier 1, medium confidence — could plausibly be tier 0 if the porting is purely mechanical copy, but the risk of incorrect redirections makes tier 1 the safer call. <!-- controller:fingerprint:7914775f1ad12cb1 -->
HAL9000 force-pushed ci-quiet-logs from 81412c40a4
All checks were successful
CI / lint (pull_request) Successful in 37s
CI / build (pull_request) Successful in 50s
CI / quality (pull_request) Successful in 56s
CI / helm (pull_request) Successful in 47s
CI / typecheck (pull_request) Successful in 1m3s
CI / push-validation (pull_request) Successful in 45s
CI / security (pull_request) Successful in 1m14s
CI / unit_tests (pull_request) Successful in 4m12s
CI / docker (pull_request) Successful in 2m25s
CI / integration_tests (pull_request) Successful in 8m21s
CI / coverage (pull_request) Successful in 11m0s
CI / status-check (pull_request) Successful in 5s
to 2d146ad9cd
Some checks failed
CI / push-validation (pull_request) Successful in 23s
CI / unit_tests (pull_request) Successful in 5m38s
CI / helm (pull_request) Failing after 5m51s
CI / integration_tests (pull_request) Successful in 10m5s
CI / quality (pull_request) Failing after 10m53s
CI / security (pull_request) Failing after 11m0s
CI / lint (pull_request) Failing after 11m2s
CI / typecheck (pull_request) Failing after 11m3s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / build (pull_request) Failing after 11m5s
CI / status-check (pull_request) Failing after 5s
2026-06-02 21:18:31 +00:00
Compare
Author
Owner

(attempt #3, tier 1)

🔧 Implementer attempt — rebased.

Pushed 1 commit: 2d146ad.

_(attempt #3, tier 1)_ **🔧 Implementer attempt — `rebased`.** Pushed 1 commit: `2d146ad`. <!-- controller:fingerprint:a352fef9bd74ab91 -->
HAL9000 force-pushed ci-quiet-logs from 2d146ad9cd
Some checks failed
CI / push-validation (pull_request) Successful in 23s
CI / unit_tests (pull_request) Successful in 5m38s
CI / helm (pull_request) Failing after 5m51s
CI / integration_tests (pull_request) Successful in 10m5s
CI / quality (pull_request) Failing after 10m53s
CI / security (pull_request) Failing after 11m0s
CI / lint (pull_request) Failing after 11m2s
CI / typecheck (pull_request) Failing after 11m3s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / build (pull_request) Failing after 11m5s
CI / status-check (pull_request) Failing after 5s
to 9cd8a58852
All checks were successful
CI / lint (pull_request) Successful in 37s
CI / quality (pull_request) Successful in 1m1s
CI / push-validation (pull_request) Successful in 38s
CI / helm (pull_request) Successful in 41s
CI / security (pull_request) Successful in 1m10s
CI / typecheck (pull_request) Successful in 1m17s
CI / build (pull_request) Successful in 53s
CI / unit_tests (pull_request) Successful in 5m1s
CI / docker (pull_request) Successful in 1m38s
CI / integration_tests (pull_request) Successful in 8m31s
CI / coverage (pull_request) Successful in 8m51s
CI / status-check (pull_request) Successful in 3s
2026-06-02 21:40:03 +00:00
Compare
Author
Owner

(attempt #4, tier 1)

🔧 Implementer attempt — rebased.

Pushed 1 commit: 9cd8a58.

_(attempt #4, tier 1)_ **🔧 Implementer attempt — `rebased`.** Pushed 1 commit: `9cd8a58`. <!-- controller:fingerprint:5932633ebf5f2a03 -->
HAL9001 approved these changes 2026-06-02 22:12:18 +00:00
HAL9001 left a comment

Approved

Reviewed at commit 9cd8a58.

Confidence: high.

**✅ Approved** Reviewed at commit `9cd8a58`. Confidence: high. <!-- controller:fingerprint:79069a01114642c3 -->
Author
Owner

Claimed by merge_drive.py (pid 954997) until 2026-06-02T23:46:20.952005+00:00.

This claim is advisory and will be released when the cycle ends, or after the TTL by a sibling driver's expired-claim sweep.

<!-- merge_drive.py: claim --> Claimed by `merge_drive.py` (pid 954997) until `2026-06-02T23:46:20.952005+00:00`. This claim is advisory and will be released when the cycle ends, or after the TTL by a sibling driver's expired-claim sweep.
HAL9000 force-pushed ci-quiet-logs from 9cd8a58852
All checks were successful
CI / lint (pull_request) Successful in 37s
CI / quality (pull_request) Successful in 1m1s
CI / push-validation (pull_request) Successful in 38s
CI / helm (pull_request) Successful in 41s
CI / security (pull_request) Successful in 1m10s
CI / typecheck (pull_request) Successful in 1m17s
CI / build (pull_request) Successful in 53s
CI / unit_tests (pull_request) Successful in 5m1s
CI / docker (pull_request) Successful in 1m38s
CI / integration_tests (pull_request) Successful in 8m31s
CI / coverage (pull_request) Successful in 8m51s
CI / status-check (pull_request) Successful in 3s
to f7de1672bb
All checks were successful
CI / build (pull_request) Successful in 38s
CI / helm (pull_request) Successful in 41s
CI / push-validation (pull_request) Successful in 30s
CI / lint (pull_request) Successful in 53s
CI / quality (pull_request) Successful in 55s
CI / typecheck (pull_request) Successful in 1m22s
CI / security (pull_request) Successful in 1m23s
CI / unit_tests (pull_request) Successful in 4m53s
CI / docker (pull_request) Successful in 1m43s
CI / integration_tests (pull_request) Successful in 9m14s
CI / coverage (pull_request) Successful in 9m36s
CI / status-check (pull_request) Successful in 4s
2026-06-02 22:16:25 +00:00
Compare
HAL9001 approved these changes 2026-06-02 22:31:47 +00:00
HAL9001 left a comment

Approved by the controller reviewer stage (workflow 174).

Approved by the controller reviewer stage (workflow 174).
HAL9000 merged commit c4ea2f3f7a into master 2026-06-02 22:31:50 +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.

Dependencies

No dependencies set.

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