Skip to content

audit_all_for_package 120s default too tight under CI matrix load (deterministic timeout) #276

Description

@ywatanabe1989

Problem

The audit conformance test generated by scitex-dev ecosystem write-audit-test calls scitex_dev.testing.audit_all_for_package(pkg) with the default timeout=120.0. In figrecipe's pytest-matrix CI (apptainer SIF, coverage instrumentation + parallel load), scitex-dev ecosystem audit-all figrecipe deterministically exceeds 120s and raises subprocess.TimeoutExpired, failing the matrix on all 3 pythons (2544 other tests pass).

  • Standalone audit-all runs clean in ~55s.
  • Under the matrix SIF load it tips over 120s every run (confirmed across original + full reruns).
  • Not a hang, not violations — just slow under load past a too-tight cap.

Immediate workaround (in figrecipe)

Hand-edited the generated test to audit_all_for_package("figrecipe", timeout=300). This is fragile — a future write-audit-test regeneration reverts it.

Requested durable fix (either)

  1. Bump write-audit-test's emitted default timeout (e.g. 300s), or
  2. Make audit_all_for_package read the timeout from .scitex/dev/config.yaml (e.g. audit.timeout) so packages can set it without hand-editing a generated file.

Optional (perf)

audit-all at ~55s standalone / >120s under load is a growing cost — if 2.30.x figure-lint/FM rules added weight, worth a profiling pass. (Same theme as the testmon bottleneck.)

Filed per the audit output's own escalation footer. Reported by the figrecipe agent while shipping 0.29.10.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions