Skip to content

[ci] Rework SDL workflow - #4279

Open
AlexanderDokuchaev wants to merge 3 commits into
openvinotoolkit:developfrom
AlexanderDokuchaev:ad/sdl_rework
Open

AlexanderDokuchaev wants to merge 3 commits into
openvinotoolkit:developfrom
AlexanderDokuchaev:ad/sdl_rework

Conversation

@AlexanderDokuchaev

@AlexanderDokuchaev AlexanderDokuchaev commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Changes

  • Added nightly schedule trigger (0 0 * * *) to the sdl workflow.
  • Trivy job runs only on schedule, manual run (workflow_dispatch) and push to release_v* branches.
  • Trivy scans once with JSON output, then trivy convert generates:
    • HTML report (uploaded as artifact, same template as before);
    • table report printed to job logs.
  • Added "Trivy: packages to update" table to the GitHub job summary (target, package, installed version, fixed versions, vulnerability IDs with severity).

Reason for changes

  • Trivy results were available only in the HTML artifact, which is inconvenient to check.
  • Avoid blocking PRs and develop pushes by vulnerabilities found in requirements, while keeping regular scans via nightly runs and release branches.

Related tickets

Tests

https://github.com/openvinotoolkit/nncf/actions/runs/39744361579

@AlexanderDokuchaev AlexanderDokuchaev changed the title Ad/sdl rework Rework SDL workflow Oct 22, 2026
@AlexanderDokuchaev AlexanderDokuchaev changed the title Rework SDL workflow [ci] Rework SDL workflow Oct 22, 2026
@AlexanderDokuchaev
AlexanderDokuchaev marked this pull request as ready for review October 22, 2026 09:40
@AlexanderDokuchaev
AlexanderDokuchaev requested a review from a team as a code owner October 22, 2026 09:40
Copilot AI balanced review requested due to automatic review settings October 22, 2026 09:40

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No blocking issues remain, and the linked run confirms reporting and artifact upload succeed even after the scan step fails.

No open issues 🎉

What changed in this PR

Reworks NNCF’s SDL workflow to keep regular security scans while preventing Trivy findings from blocking PRs and develop pushes, and makes results easier to inspect.

Changes:

  • Adds a nightly schedule and limits Trivy execution to scheduled runs, manual runs, and release-branch pushes.
  • Converts a single JSON scan into HTML and table reports.
  • Prints scan results in logs and adds a package-update table to the job summary.
File Description
.github/​workflows/​sdl.yml Updates scan triggers and adds report conversion, log output, and summary generation.

Review effort: Balanced

This branch has not been deployed

No deployments
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