Skip to content

ci: remove all timeout-minutes (owner: no CI time limits) - #147

Merged
luisleo526 merged 1 commit into
mainfrom
data-vp/ci-no-timeouts-pfdata
Oct 20, 2026
Merged

luisleo526 merged 1 commit into
mainfrom
data-vp/ci-no-timeouts-pfdata

Conversation

@luisleo526

@luisleo526 luisleo526 commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

What

Removes every timeout-minutes from this repository's workflows (owner's no-time-limits rule; L6 bulletins #826/#827): .github/workflows/ci.yml changes (5) and .github/workflows/native-feed.yml release matrix leg, asan-ubsan, tsan (30 each) — 4 deleted lines, 0 added, nothing else changed.

The platform maximum (6 h per job) still catches a hang, and the canon authz job already carries no timeout. No pinned test asserts a timeout value.

Stacked on #146 (data-vp/ci-pool-canon-tests).

🤖 Generated with Claude Code

@luisleo526
luisleo526 force-pushed the data-vp/ci-no-timeouts-pfdata branch from 079f38c to 102c3aa Compare October 20, 2026 08:42
@luisleo526
luisleo526 force-pushed the data-vp/ci-pool-canon-tests branch from 442d24f to b21be27 Compare October 20, 2026 09:07
@luisleo526
luisleo526 force-pushed the data-vp/ci-no-timeouts-pfdata branch from 102c3aa to 396091b Compare October 20, 2026 09:07
@luisleo526
luisleo526 changed the base branch from data-vp/ci-pool-canon-tests to main October 20, 2026 10:41
…e limits)

Owner rule (10-12, bulletins #826/#827): no time limit on any CI job or step;
GitHub's own 6 h maximum stays. Remove the four job-level timeout-minutes
(ci.yml changes, native-feed.yml release, asan-ubsan, tsan). docs.yml,
release.yml, release-report.yml and pineforge-release.yml had none. Docker
--health-timeout options are service health checks and stay. The workflow test
that adds timeout-minutes to authz as a fault is kept: the canonical authz has
none. No other line changes.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@luisleo526
luisleo526 force-pushed the data-vp/ci-no-timeouts-pfdata branch from 396091b to 972e6d9 Compare October 20, 2026 10:44
@luisleo526
luisleo526 merged commit 9590725 into main Oct 20, 2026
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant