Skip to content

Commit

Permalink
Merge branch 'feature/use-native-number-of-cores-in-dune' into featur…
Browse files Browse the repository at this point in the history
…e/use-native-number-of-cores-in-dune-COMPATIBLE
  • Loading branch information
mrmr1993 committed Oct 15, 2024
2 parents 1b757eb + e63e130 commit 7f4c6d1
Show file tree
Hide file tree
Showing 5 changed files with 25 additions and 77 deletions.
63 changes: 0 additions & 63 deletions .xrefcheck.yaml

This file was deleted.

2 changes: 1 addition & 1 deletion buildkite/scripts/fuzzy-zkapp-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ export LIBP2P_NIXLESS=1 PATH=/usr/lib/go/bin:$PATH GO=/usr/lib/go/bin/go
# skip running all of the tests that have already succeeded, since dune will
# only retry those tests that failed.
echo "--- Run fuzzy zkapp tests"
time dune exec "${path}" --profile="${profile}" -j16 -- --timeout "${timeout}" --individual-test-timeout "${individual_test_timeout}" --seed "${RANDOM}"
time dune exec "${path}" --profile="${profile}" -- --timeout "${timeout}" --individual-test-timeout "${individual_test_timeout}" --seed "${RANDOM}"
STATUS=$?
if [ "$STATUS" -ne 0 ]; then
./scripts/link-coredumps.sh && exit "$STATUS"
Expand Down
8 changes: 4 additions & 4 deletions buildkite/scripts/unit-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ export LIBP2P_NIXLESS=1 PATH=/usr/lib/go/bin:$PATH GO=/usr/lib/go/bin/go
time make build

echo "--- Build all targets"
dune build "${path}" --profile="${profile}" -j8
dune build "${path}" --profile="${profile}"

echo "--- Check for changes to verification keys"
time dune runtest "src/app/print_blockchain_snark_vk" --profile="${profile}" -j8
time dune runtest "src/app/print_blockchain_snark_vk" --profile="${profile}"

# Turn on the proof-cache assertion, so that CI will fail if the proofs need to
# be updated.
Expand All @@ -35,8 +35,8 @@ export ERROR_ON_PROOF=true
# skip running all of the tests that have already succeeded, since dune will
# only retry those tests that failed.
echo "--- Run unit tests"
time dune runtest "${path}" --profile="${profile}" -j8 || \
time dune runtest "${path}" --profile="${profile}" || \
(./scripts/link-coredumps.sh && \
echo "--- Retrying failed unit tests" && \
time dune runtest "${path}" --profile="${profile}" -j8 || \
time dune runtest "${path}" --profile="${profile}" || \
(./scripts/link-coredumps.sh && false))
2 changes: 1 addition & 1 deletion buildkite/src/Constants/ContainerImages.dhall
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,6 @@
, ubuntu2004 = "ubuntu:20.04"
, postgres = "postgres:12.4-alpine"
, xrefcheck =
"serokell/xrefcheck@sha256:8fbb35a909abc353364f1bd3148614a1160ef3c111c0c4ae84e58fdf16019eeb"
"dkhamsing/awesome_bot@sha256:a8adaeb3b3bd5745304743e4d8a6d512127646e420544a6d22d9f58a07f35884"
, nixos = "gcr.io/o1labs-192920/nix-unstable:1.0.0"
}
27 changes: 19 additions & 8 deletions buildkite/src/Jobs/Lint/Xrefcheck.dhall
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,6 @@ let Cmd = ../../Lib/Cmds.dhall

let Command = ../../Command/Base.dhall

let Docker = ../../Command/Docker/Type.dhall

let Size = ../../Command/Size.dhall

let B/SoftFail = B.definitions/commandStep/properties/soft_fail/Type
Expand All @@ -23,7 +21,8 @@ in Pipeline.build
, spec = JobSpec::{
, dirtyWhen =
[ SelectFiles.strictly SelectFiles::{ exts = Some [ "md" ] }
, SelectFiles.strictly (SelectFiles.contains ".xrefcheck.yml")
, SelectFiles.strictlyStart
(SelectFiles.contains "buildkite/src/Jobs/Lint/Xrefcheck.dhall")
]
, path = "Lint"
, name = "Xrefcheck"
Expand All @@ -36,15 +35,27 @@ in Pipeline.build
, steps =
[ Command.build
Command.Config::{
, commands = [] : List Cmd.Type
, commands =
[ Cmd.runInDocker
Cmd.Docker::{
, image = (../../Constants/ContainerImages.dhall).xrefcheck
}
( "awesome_bot -allow-dupe "
++ "--allow-redirect "
++ "--allow 403,401 "
++ "--skip-save-results "
++ "--files "
++ "`find . -name \"*.md\" "
++ "! -path \"./src/lib/crypto/kimchi_bindings/*\" "
++ "! -path \"./src/lib/crypto/proof-systems/*\" "
++ "! -path \"./src/external/*\" "
++ "` "
)
]
, label = "Verifies references in markdown"
, key = "xrefcheck"
, target = Size.Small
, soft_fail = Some (B/SoftFail.Boolean True)
, docker = Some Docker::{
, image = (../../Constants/ContainerImages.dhall).xrefcheck
, shell = None (List Text)
}
}
]
}

0 comments on commit 7f4c6d1

Please sign in to comment.