intel_oneapi_{ccl,compilers,dnn,ipp{,cp},dpl,mkl,tbb,vtune}: new versions#5625
Open
rliulin wants to merge 5 commits into
Open
intel_oneapi_{ccl,compilers,dnn,ipp{,cp},dpl,mkl,tbb,vtune}: new versions#5625rliulin wants to merge 5 commits into
rliulin wants to merge 5 commits into
Conversation
spackbot-triage
Bot
added
new-version
bernhardkaindl
changed the title
Contributor
There was a problem hiding this comment.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| repos/spack_repo/builtin/packages/intel_oneapi_vtune/package.py | Adds vtune 2026.2.0 version metadata. |
| repos/spack_repo/builtin/packages/intel_oneapi_tbb/package.py | Adds oneTBB 2023.1.0 version metadata. |
| repos/spack_repo/builtin/packages/intel_oneapi_mpi/package.py | Adds MPI installer metadata (but currently introduces a duplicate version entry). |
| repos/spack_repo/builtin/packages/intel_oneapi_mkl/package.py | Adds oneMKL 2026.1.0 version metadata. |
| repos/spack_repo/builtin/packages/intel_oneapi_ippcp/package.py | Adds ippcp 2026.0.1 version metadata. |
| repos/spack_repo/builtin/packages/intel_oneapi_ipp/package.py | Adds IPP 2026.0.1 version metadata. |
| repos/spack_repo/builtin/packages/intel_oneapi_dpl/package.py | Adds a 2026.0.1 entry (but currently points to the IPP installer URL/checksum). |
| repos/spack_repo/builtin/packages/intel_oneapi_dnn/package.py | Adds oneDNN 2026.0.1 version metadata. |
| repos/spack_repo/builtin/packages/intel_oneapi_compilers/package.py | Adds compilers 2026.1.0 URLs/checksums to the versions list. |
| repos/spack_repo/builtin/packages/intel_oneapi_ccl/package.py | Adds oneCCL 2022.1.0 version metadata. |
Comments suppressed due to low confidence (1)
repos/spack_repo/builtin/packages/intel_oneapi_mpi/package.py:40
- There are two
version("2021.18.0", ...)entries. Spack requires unique version identifiers; the second definition will overwrite the first (or can lead to confusion about which installer/checksum is intended). If this PR is updating the 2021.18.0 installer build, replace the existing entry instead of adding a duplicate.
version(
"2021.18.0",
url="https://registrationcenter-download.intel.com/akdlm/IRC_NAS/f62e2cfe-82a9-480f-b6ca-51ad7cc799fc/intel-mpi-2021.18.0.749_offline.sh",
sha256="a2afb95b3b9f85b9ddd32171e940a7b7cc02d7bbd6dca9d77a7bd405f3d62b73",
expand=False,
)
version(
"2021.18.0",
url="https://registrationcenter-download.intel.com/akdlm/IRC_NAS/1710a04f-08b7-4b46-a0a5-bd46a4ef4436/intel-mpi-2021.18.0.748_offline.sh",
sha256="a1d9bebb9112f166c1911371971ae03fecb447f58d9f28f5c8ecdf78fb697f29",
expand=False,
)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
|
@rliulin, thank you for updating the Intel ONEAPI packages! |
bernhardkaindl
marked this pull request as draft
July 17, 2026 09:53
bernhardkaindl
changed the title
Contributor
|
AFAICS, you just fixed up the duplicated intel_oneapi_mpi@2021.18.0 version. I agree that it is more economical to put the related version updates into one PR, as long as GitLab CI does not run problems when changing many things at once. Otherwise, it might indeed be easier to branch off some changes. I was initially confused by the change, so I'd like to suggest for easier review of such changes, it would be good to re-base this PR on top of the latest develop instead of merging |
rscohn2
marked this pull request as ready for review
July 17, 2026 11:09
Member
|
It looks good with the changes. Now we need to see if the new versions break anything in e4s so I removed draft. It would be better if your automated tool started with an up to date base version rather than merging changes afterwards. |
Contributor
|
I install-tested the new versions: passed intel-oneapi-ccl@=2022.1.0
passed intel-oneapi-compilers
passed intel-oneapi-dnn@=2026.0.1
passed intel-oneapi-dpl@=2022.13.0
passed intel-oneapi-ipp@=2026.0.1
passed intel-oneapi-ippcp@=2026.0.1
passed intel-oneapi-mkl@=2026.1.0
passed intel-oneapi-mpi
passed intel-oneapi-tbb@=2023.1.0
passed intel-oneapi-vtune@=2026.2.0 |
Member
|
Fails looks like system issue with gitlab. I will rerun |
Member
|
@spackbot run pipeline |
|
I've started that pipeline for you! |
rscohn2
approved these changes
Jul 21, 2026
rscohn2
enabled auto-merge (squash)
July 21, 2026 11:36
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull request overview (moved from Copilot Review)
This PR updates Spack’s built-in Intel oneAPI package recipes to add newly released component versions (vtune, tbb, mpi, mkl, ipp/ippcp, dpl, dnn, compilers, ccl), keeping installer URLs/checksums current for oneAPI offline installers.
Changes:
version(...)entries across multipleintel_oneapi_*packages.intel_oneapi_compilersversionsmetadata with the new 2026.1.0 compiler release.It changes
intel-oneapi-mpi@2021.18.0without changing the version string from 748 to 749 sub-release: