Skip to content

Commit

Permalink
Merge #63
Browse files Browse the repository at this point in the history
63: Add buildkite yml r=kmdeck a=charleskawczynski

This PR
 - adds buildkite jobs for each of the unit tests
 - Fixes a few leaky tests
 - Adds dependencies to the test environment

The 3D job seems to require more memory than the default on central, so the 3D job in the pipeline yaml may need to be changed to

```
      - label: "soil_test_3d"
        command: "julia --color=yes --project=test test/Soil/soil_test_3d.jl"
        artifact_paths: "soil_test_3d/"
        agents:
          slurm_mem: "32GB"
```

`@kmdeck,` can you try this?

Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
  • Loading branch information
bors[bot] and charleskawczynski authored Jul 5, 2022
2 parents 0f087f2 + 19bafc2 commit c00bd76
Show file tree
Hide file tree
Showing 4 changed files with 84 additions and 1 deletion.
80 changes: 80 additions & 0 deletions .buildkite/pipeline.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,80 @@
env:
JULIA_VERSION: "1.7.0"
OPENMPI_VERSION: "4.0.4"
CUDA_VERSION: "11.2"
OPENBLAS_NUM_THREADS: 1
BUILDKITE_COMMIT: "${BUILDKITE_COMMIT}"
BUILDKITE_BRANCH: "${BUILDKITE_BRANCH}"

agents:
config: cpu
queue: central
slurm_ntasks: 1

steps:
- label: "init environment :computer:"
key: "init_cpu_env"
command:
- "julia --project -e 'using Pkg; Pkg.instantiate(;verbose=true)'"
- "julia --project -e 'using Pkg; Pkg.precompile()'"
- "julia --project -e 'using Pkg; Pkg.status()'"

- "julia --project=test -e 'using Pkg; Pkg.instantiate(;verbose=true)'"
- "julia --project=test -e 'using Pkg; Pkg.precompile()'"
- "julia --project=test -e 'using Pkg; Pkg.status()'"

- wait

- group: "Unit tests"
steps:

# - label: "Unit tests"
# command: "julia --color=yes --project=test test/runtests.jl"
# artifact_paths: "test/*"

- label: "bucket_test"
command: "julia --color=yes --project=test test/Bucket/bucket_test.jl"
artifact_paths: "bucket_test/"

- label: "root_test"
command: "julia --color=yes --project=test test/Vegetation/root_test.jl"
artifact_paths: "root_test/"

- label: "lsm_test"
command: "julia --color=yes --project=test test/LSM/lsm_test.jl"
artifact_paths: "lsm_test/"

- label: "domains"
command: "julia --color=yes --project=test test/domains.jl"
artifact_paths: "domains/"

- label: "variable_types"
command: "julia --color=yes --project=test test/variable_types.jl"
artifact_paths: "variable_types/"

- label: "pond_test"
command: "julia --color=yes --project=test test/SurfaceWater/pond_test.jl"
artifact_paths: "pond_test/"

- label: "pond_soil_lsm"
command: "julia --color=yes --project=test test/LSM/pond_soil_lsm.jl"
artifact_paths: "pond_soil_lsm/"

- label: "soil_test_3d"
command: "julia --color=yes --project=test test/Soil/soil_test_3d.jl"
artifact_paths: "soil_test_3d/"
agents:
slurm_mem_per_cpu: 8G

- label: "soiltest"
command: "julia --color=yes --project=test test/Soil/soiltest.jl"
artifact_paths: "soiltest/"

- label: "soil_parameterizations"
command: "julia --color=yes --project=test test/Soil/soil_parameterizations.jl"
artifact_paths: "soil_parameterizations/"

- label: "soil_bc"
command: "julia --color=yes --project=test test/Soil/soil_bc.jl"
artifact_paths: "soil_bc/"

1 change: 1 addition & 0 deletions bors.toml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
status = [
"buildkite/climalsm-ci",
"ci 1.7.0 - ubuntu-latest",
"ci 1.7.0 - macOS-latest",
"docbuild",
Expand Down
3 changes: 2 additions & 1 deletion test/Soil/soil_bc.jl
Original file line number Diff line number Diff line change
@@ -1,9 +1,10 @@
using Test
using ClimaCore

if !("." in LOAD_PATH)
push!(LOAD_PATH, ".")
end
using ClimaLSM

using ClimaLSM.Domains: HybridBox, SphericalShell
FT = Float64

Expand Down
1 change: 1 addition & 0 deletions test/Soil/soil_test_3d.jl
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
using Test
using UnPack
using Statistics
using ClimaCore
import CLIMAParameters as CP

Expand Down

0 comments on commit c00bd76

Please sign in to comment.