Skip to content

[CI][DOCS] Document that known-failing checks must not run in pre-commit - #39541

Closed
mryzhov wants to merge 3 commits into
openvinotoolkit:masterfrom
mryzhov:ci/doc-no-failing-precommit
Closed

mryzhov wants to merge 3 commits into
openvinotoolkit:masterfrom
mryzhov:ci/doc-no-failing-precommit

Conversation

@mryzhov

@mryzhov mryzhov commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Details:

  • Add a "Known-Failing and Non-Blocking Checks" section to docs/dev/ci/github_actions/adding_tests.md, explaining why known-failing builds/tests should not run automatically on the initial pre-commit attempt, even if they are non-blocking or outside Overall_Status: they waste runner capacity, create noise, provide no per-change signal and misrepresent validation.
  • Recommend running work-in-progress checks on a schedule, either in a dedicated workflow or as a schedule-gated job in an existing workflow. For shared workflows, skip unrelated jobs on the experimental schedule and use a distinct cron expression when the workflow has other schedules.
  • Allow opt-in PR validation on a workflow re-run using github.run_attempt > 1, as with draft PR validation. A full re-run also launches the regular workflow, and the condition can match automated re-runs; Smart CI skip rules still apply.
  • Recommend tracking and fixing scheduled failures, then enabling the check automatically in pre-commit as a required check in Overall_Status only after it passes reliably. Add the matching pitfall to the ov-github-actions-ci skill.
  • Context: [CI] Run C++20 build checks daily instead of in pre-commit #39540 moves the non-blocking C++20 builds to a daily schedule while allowing workflow re-runs.

Tickets:

AI Assistance:

  • AI assistance used: yes
  • The text was drafted with an AI assistant; human validation has not yet been reported.
  • git diff --cached --check: passed. This is a documentation-only change, so no build or tests were run.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mryzhov
mryzhov requested a review from a team as a code owner November 5, 2026 13:22
@mryzhov
mryzhov requested review from michalzx and removed request for a team November 5, 2026 13:22
@github-actions github-actions Bot added the category: docs OpenVINO documentation label Nov 5, 2026
mryzhov and others added 2 commits November 5, 2026 14:26
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mryzhov mryzhov closed this Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: docs OpenVINO documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant