Skip to content

Commit

Permalink
Use X-Large VM instance for speeding up builds (#3043)
Browse files Browse the repository at this point in the history
Increase job parallelism to 8 since xl instances have 8 vcpus.

Co-authored-by: Pavel Merkis <pavel.merkis@mapbox.com>
  • Loading branch information
mandeepsandhu and merkispavel committed Apr 29, 2021
1 parent 01ca643 commit 3babbef
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 10 deletions.
22 changes: 12 additions & 10 deletions .circleci/config.yml
Expand Up @@ -45,6 +45,7 @@ jobs:
lint-build-debug:
docker:
- image: valhalla/valhalla:build-latest
resource_class: xlarge
steps:
- checkout
- run: |
Expand All @@ -67,11 +68,11 @@ jobs:
-DCMAKE_CXX_FLAGS="-fuse-ld=lld" \
-DENABLE_PYTHON_BINDINGS=On
- run: python3 ./scripts/valhalla_build_config
- run: make -C build -j3
- run: make -C build -j8
- run: make -C build utrecht_tiles
- run: make -C build -j2 tests
- run: make -C build -j2 benchmarks
- run: make -C build run-benchmarks
- run: make -C build -j8 tests
- run: make -C build -j8 benchmarks
- run: make -C build -j8 run-benchmarks
# Note: we save the cache here before doing linting so that if linting fails, we can rebuild quickly
# for follow-up fixes
- save_cache:
Expand All @@ -83,12 +84,13 @@ jobs:
- run: make -C build package
- run: |
# Note: run (compile) tests and make code coverage report.
make -C build -j4 coverage
make -C build -j8 coverage
- run: .circleci/vendored-codecov.sh || echo "Codecov did not collect coverage reports"

build-release:
docker:
- image: valhalla/valhalla:build-latest
resource_class: xlarge
steps:
- checkout
- run: |
Expand All @@ -106,13 +108,13 @@ jobs:
- run: |
cd build && cmake .. -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS=On -DENABLE_PYTHON_BINDINGS=On \
-DCPACK_GENERATOR=DEB -DCPACK_PACKAGE_VERSION_SUFFIX="-0ubuntu1-$(lsb_release -sc)" -DENABLE_ADDRESS_SANITIZER=ON
- run: make -C build -j2
- run: make -C build -j8
- run: make -C build utrecht_tiles
- run: make -C build -j2 tests
- run: make -C build -j8 tests
# leaks in glibc we cant control for
- run: export ASAN_OPTIONS=detect_leaks=0 && make -C build -j2 check
- run: make -C build -j2 benchmarks
- run: make -C build run-benchmarks
- run: export ASAN_OPTIONS=detect_leaks=0 && make -C build -j8 check
- run: make -C build -j8 benchmarks
- run: make -C build -j8 run-benchmarks
- save_cache:
key: ccache-release-linux-x86_64-{{ .Branch }}-{{ epoch }}
paths:
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Expand Up @@ -5,6 +5,7 @@
* **Enhancement**
* CHANGED: Refactor timedep forward/reverse to reduce code repetition [#2987](https://github.com/valhalla/valhalla/pull/2987)
* CHANGED: Sync translation files with Transifex command line tool [#3030](https://github.com/valhalla/valhalla/pull/3030)
* CHANGED: Use CircleCI XL instances for linux based builds [#3043](https://github.com/valhalla/valhalla/pull/3043)

## Release Date: 2021-04-21 Valhalla 3.1.1
* **Removed**
Expand Down

0 comments on commit 3babbef

Please sign in to comment.