Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrades to spack v0.20.1 #792

Merged
merged 142 commits into from
Sep 14, 2023
Merged

Upgrades to spack v0.20.1 #792

merged 142 commits into from
Sep 14, 2023

Conversation

dominichofer
Copy link
Contributor

@dominichofer dominichofer commented Jul 18, 2023

Modifications for v0.20.1

Packages

All packages

  • change ssh links from ssh://git@github.com to git@github.com:

ICON

  • adapt custom build extensions to new version of spack
  • Pop PYTHONHOME only if set while testing
  • Change delimiter of variant fcgroup from ; to .
  • Limit out-of-source builds to spec with dev_path

COSMO

COSMO-Dycore

  • add missing include through patch
  • change test_dycore.py through patch

py-gt4py

py-icon4py

  • adapt custome build extensions to new version of spack

Infrastructure

  • rearange software in upstream for Daint
  • remove devbuildcosmo and installcosmo
  • spack clean -a before any test run

Todo before merge

  • Adapt upstream paths for Daint

@github-actions
Copy link
Contributor

github-actions bot commented Jul 18, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-09-14 14:56 UTC

@C2SM C2SM deleted a comment from jenkins-apn Jul 18, 2023
@C2SM C2SM deleted a comment from jenkins-apn Jul 18, 2023
@C2SM C2SM deleted a comment from jenkins-apn Jul 18, 2023
@AnnikaLau
Copy link
Contributor

@AnnikaLau, I gave this an other try. Maybe if we tackle this together we will succeed.
On Balfrin cosmo-dycore ran into this: spack/spack#31214

Ah great! I also did another fix yesterday (before I saw your new pull request) to fix the configure phase for icon on Daint. If you agree with my change (60962be), we can add it here and I can close my pull request?

@dominichofer
Copy link
Contributor Author

@AnnikaLau Yes, sounds good to me.

@AnnikaLau
Copy link
Contributor

When running the system test from Jenkins to build ICON, the following error appears for daint_cpu_cce, daint_cpu_gcc, daint_cpu_nvhpc, daint_cpu_nvhpc_out_of_source, daint_gpu_nvhpc at the step spack install -j 4 --until build -n -v (https://jenkins-mch.cscs.ch/job/Spack/job/spack_PR/1150/):

==> Error: develop does not satisfy 2. You could consider setting concretizer:unify to when_possible or false to allow multiple versions of some packages.

However, when running the test on my login nodes, I don't get this error.

@jenkins-apn
Copy link

balfrin

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🟢 system test
Test
🟢Claw-install_default

@jenkins-apn
Copy link

daint

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🔴 system test
Test
🔴Claw-install_default

WARNING: Serial tests did not run for system tests

@jenkins-apn
Copy link

tsa

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🟢 system test
Test
🟢Claw-install_default

@jonasjucker
Copy link
Contributor

launch jenkins claw

@jenkins-apn
Copy link

balfrin

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec

@jenkins-apn
Copy link

tsa

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec

@jenkins-apn
Copy link

daint

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🟢 system test
Test
🟢Claw-install_default

@jonasjucker
Copy link
Contributor

launch jenkins claw

@jenkins-apn
Copy link

balfrin

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🟢 system test
Test
🟢Claw-install_default

@jenkins-apn
Copy link

tsa

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🟢 system test
Test
🟢Claw-install_default

@jenkins-apn
Copy link

daint

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢claw-spack_info
🟢claw-spack_spec
🟢 system test
Test
🟢Claw-install_default_build_only

@dominichofer
Copy link
Contributor Author

launch jenkins icon

@jonasjucker
Copy link
Contributor

launch jenkins icon balfrin

@jenkins-apn
Copy link

tsa

🟢 unit test
Test
🟢summary

@jenkins-apn
Copy link

daint

🟢 unit test
Test
🟢summary

@jenkins-apn
Copy link

balfrin

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢icon-spack_info
🟢icon-spack_spec
🟢dace_icon.-O1-spack_spec
🟢icon_serialization=create_claw=std-spack_spec
🟢 system test
Test
🟢Icon-install_2_6_6_gcc
🟢Icon-install_2_6_6_nvhpc
🟢Icon-install_nwp_gpu

jonasjucker and others added 2 commits September 14, 2023 16:37
Co-authored-by: Jonas Jucker <jonas.jucker@c2sm.ethz.ch>
@jonasjucker
Copy link
Contributor

launch jenkins onnx-runtime

@jenkins-apn
Copy link

tsa

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢onnx-runtime-spack_info
🟢onnx-runtime-spack_spec
🟢 system test
Test
🟢OnnxRuntime-install_default

@jenkins-apn
Copy link

balfrin

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢onnx-runtime-spack_info
🟢onnx-runtime-spack_spec
🟢 system test
Test
🟢OnnxRuntime-install_default

@jenkins-apn
Copy link

daint

🟢 unit test
Test
🟢summary
🟢 integration test
Test
🟢onnx-runtime-spack_info
🟢onnx-runtime-spack_spec
🟢 system test
Test
🟢OnnxRuntime-install_default

@jonasjucker jonasjucker merged commit 8fc904d into main Sep 14, 2023
@jonasjucker jonasjucker deleted the dev0.20.1 branch September 14, 2023 14:55
@jonasjucker jonasjucker restored the dev0.20.1 branch September 14, 2023 14:55
@dominichofer dominichofer deleted the dev0.20.1 branch September 15, 2023 14:16
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.

5 participants