Skip to content

[MAINT] Add chron job for updating OS compiler dependencies - #2408

Merged
ndgrigorian merged 2 commits into
masterfrom
chron-job-to-update-os-compiler-dependencies
Nov 3, 2026
Merged

ndgrigorian merged 2 commits into
masterfrom
chron-job-to-update-os-compiler-dependencies

Conversation

@ndgrigorian

Copy link
Copy Markdown
Collaborator

This PR proposes the addition of a chron job for updating the dependencies of the OS compiler workflow

  • Have you provided a meaningful PR description?
  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • Have you checked performance impact of proposed changes?
  • Have you added documentation for your changes, if necessary?
  • Have you added your changes to the changelog?
  • If this PR is a work in progress, are you opening the PR as a draft?

@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Deleted rendered PR docs from intelpython.github.com/dpctl, latest should be updated shortly. 🤞

@coveralls

coveralls commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

Coverage Status

coverage: 74.893% (-0.1%) from 74.989% — chron-job-to-update-os-compiler-dependencies into master

antonwolfy
antonwolfy previously approved these changes Nov 3, 2026

@antonwolfy antonwolfy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A couple of minor nits. In overall, LGTM,

Comment thread .github/workflows/bump-sycl-deps.yml Resolved
Cache key hashed a path outside the workspace, so it never changed.
Reject '..' in the llvm-ref input and check the resolved sha is 40 hex.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues block the dependency-update automation and shared-pin changes.

No open issues 🎉

What changed in this PR

Adds weekly dependency-update automation for the open-source LLVM SYCL build, moving the CPU OpenCL runtime and oneTBB pins into a shared file.

Changes:

  • Adds a scheduled and manually triggered workflow that validates upstream dependency metadata and opens pin-update pull requests.
  • Updates the build workflow to load shared pins, key its cache by their contents, and refresh dependencies independently of the compiler nightly.
  • Enables manual build runs to validate bot-created pull requests, which do not automatically trigger CI.
File Description
.github/​workflows/​os-llvm-sycl-build.yml Loads shared pins, updates dependency caching and fetching, and enables manual runs.
.github/​workflows/​bump-sycl-deps.yml Adds weekly upstream dependency checks and automated pin-update pull requests.
.github/​sycl-build-deps.env Centralizes the existing CPU OpenCL runtime and oneTBB dependency pins.

Review effort: Balanced

@ndgrigorian
ndgrigorian merged commit d52046c into master Nov 3, 2026
62 of 68 checks passed
@ndgrigorian
ndgrigorian deleted the chron-job-to-update-os-compiler-dependencies branch November 3, 2026 20:33
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.

4 participants