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

Merged
HAL9000 merged 1 commits 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.
HAL9000 added 1 commit 2026-06-02 20:55:46 +00:00
ci: quiet CI step logs to cut Forgejo server load
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>
HAL9000 added the controller-managed label 2026-06-02 21:00:25 +00:00
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 to 2d146ad9cd 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 to 9cd8a58852 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
Owner

Approved

Reviewed at commit 9cd8a58.

Confidence: high.

**✅ Approved** Reviewed at commit `9cd8a58`. Confidence: high. <!-- controller:fingerprint:79069a01114642c3 -->
HAL9000 added the auto/claimed-merge label 2026-06-02 22:16:20 +00:00
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 to f7de1672bb 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
Owner

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
HAL9000 removed the auto/claimed-merge label 2026-06-02 22:31:51 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cleveragents/cleveragents-core#11259