Skip to content

Scope CI workflow triggers by component - #1409

Closed
wanliAlex wants to merge 3 commits into
mainlinefrom
ci/scope-workflow-triggers-by-component
Closed

Scope CI workflow triggers by component#1409
wanliAlex wants to merge 3 commits into
mainlinefrom
ci/scope-workflow-triggers-by-component

Conversation

@wanliAlex

Copy link
Copy Markdown
Collaborator

Summary

  • Scope test workflows so only affected components run tests based on which directories changed
  • API (components/marqo/) changes → run marqo tests only
  • Inference orchestrator changes → run marqo + inference orchestrator tests
  • Model management or common changes → run all tests
  • Non-component changes (root files, .github/, etc.) → run all tests (safe default)
  • Rename Report-required-test-status jobs to include component name (Report-required-test-status-marqo-api, Report-required-test-status-inference-orchestrator, Report-required-test-status-model-management) for easier GitHub branch protection rule setup

Test plan

  • Verify marqo API workflow still runs for any non-doc change
  • Verify inference orchestrator workflow skips when only components/marqo/ files change
  • Verify model management workflow skips when only components/marqo/ and/or components/inference_orchestrator/ files change
  • Verify all workflows run when components/common/ or root-level files change
  • Verify workflow_dispatch still runs all tests
  • Update GitHub branch protection rules to use new job names

🤖 Generated with Claude Code

wanliAlex and others added 3 commits April 2, 2026 19:23
Only run tests for components affected by the change:
- API (marqo) changes: run marqo tests only
- Inference orchestrator changes: run marqo + inference tests
- Model management or common changes: run all tests
- Non-component changes (root, .github, etc.): run all tests

Also rename Report-required-test-status jobs per component for easier
GitHub branch protection rule setup.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@papa99do papa99do closed this Apr 10, 2026
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.

2 participants