Skip to content

ci: contain narrow python test silos#1692

Merged
lmeyerov merged 2 commits into
masterfrom
ci/test-silos-low-risk-gates
Jul 6, 2026
Merged

ci: contain narrow python test silos#1692
lmeyerov merged 2 commits into
masterfrom
ci/test-silos-low-risk-gates

Conversation

@lmeyerov

@lmeyerov lmeyerov commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Summary

Third slice for #1689, stacked on #1691. This is the first behavior-changing slice and is intentionally conservative:

  • add networkx_test_only, docs_test_only, benchmarks_only, and aggregate narrow_python_only
  • keep unknown Python edits fail-closed with python && narrow_python_only != true
  • keep shared/core/tooling paths fanout via core / infra
  • carve out known narrow Python cases from unrelated GFQL/pandas/DGL/docs/schema gates

For the #1686 shape (CHANGELOG.md + graphistry/tests/plugins/test_networkx.py), the modified gates select the networkx policy job and skip unrelated GFQL/pandas/DGL/docs/schema gates.

Why

The target overrun happened because a plugin test-only Python edit made python=true, and heavy jobs treated python as sufficient. This PR preserves conservative behavior for ambiguous Python edits while containing known narrow silos.

Validation

Residual risk

GitHub Actions needs skip behavior should be confirmed in CI. Follow-up PRs can address trickier hidden fanout and changed-line coverage artifact dependencies.

Stack: 3/3. Previous: #1691.

@lmeyerov
lmeyerov marked this pull request as ready for review July 5, 2026 22:50
Base automatically changed from ci/test-silos-classifiers to master July 6, 2026 00:40
@lmeyerov
lmeyerov merged commit 1fa0de2 into master Jul 6, 2026
151 of 153 checks passed
@lmeyerov
lmeyerov deleted the ci/test-silos-low-risk-gates branch July 6, 2026 02:36
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