Skip to content

Commit

Permalink
Serialize on fetch
Browse files Browse the repository at this point in the history
  • Loading branch information
Jake-Shadle committed Mar 1, 2024
1 parent 42e8e32 commit 6ee6307
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 4 deletions.
8 changes: 8 additions & 0 deletions build/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,14 @@ version:
# Run all tests
test: ensure-build-image test-quilkin test-examples test-docs

# In CI with split jobs that both fetch they will fail if run in parallel since
# cargo will be fighting with itself for some the same host directory that is
# mapped into the container, so instead just split it out to its own job
fetch-quilkin: ensure-build-image
docker run --rm $(common_rust_args) \
--network=host \
-e RUST_BACKTRACE=1 --entrypoint=cargo $(BUILD_IMAGE_TAG) fetch

check-quilkin: ensure-build-image
docker run --rm $(common_rust_args) \
--entrypoint=cargo $(BUILD_IMAGE_TAG) deny check
Expand Down
20 changes: 16 additions & 4 deletions cloudbuild.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,34 +26,46 @@ steps:
- name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
dir: ./build
args:
- test-quilkin
id: test-quilkin
- fetch-quilkin
id: fetch-quilkin
waitFor:
- fetch-git-submodules
- pull-build-image
- name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
dir: ./build
args:
- test-quilkin
id: test-quilkin
waitFor:
- fetch-quilkin
- name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
dir: ./build
args:
- check-quilkin
id: check-quilkin
waitFor:
- fetch-git-submodules
- pull-build-image
- fetch-quilkin
- name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
dir: ./build
args:
- test-examples
id: test-examples
waitFor:
- fetch-quilkin
- name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
dir: ./build
args:
- test-docs
id: test-docs
waitFor:
- fetch-quilkin
- name: us-docker.pkg.dev/$PROJECT_ID/ci/make-docker
dir: ./build
args:
- build-image
id: build
waitFor:
- fetch-quilkin

#
# Run the built images for 5 seconds in a few standard configurations, to test basic common scenarios
Expand Down

0 comments on commit 6ee6307

Please sign in to comment.