Skip to content

Commit

Permalink
Fix nightly build metrics job to add shared builds
Browse files Browse the repository at this point in the history
  • Loading branch information
kgpai committed Jan 14, 2025
1 parent 5c770df commit 236eb57
Showing 1 changed file with 24 additions and 3 deletions.
27 changes: 24 additions & 3 deletions .github/workflows/build-metrics.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ jobs:
matrix:
runner: ["16-core-ubuntu"]
type: ["debug", "release"]
link-type: ["shared", "static"]
defaults:
run:
shell: bash
Expand All @@ -57,7 +58,8 @@ jobs:
# it doesn't work
run: git config --global --add safe.directory ${GITHUB_WORKSPACE}

- name: Make ${{ matrix.type }} Build
- name: Make ${{ matrix.link-type }} - ${{ matrix.type }} Build
if: ${{ matrix.link-type == 'static' }}
env:
MAKEFLAGS: 'MAX_HIGH_MEM_JOBS=8 MAX_LINK_JOBS=4'
run: |
Expand All @@ -73,6 +75,25 @@ jobs:
)
make '${{ matrix.type }}'
- name: Make ${{ matrix.link-type }} - ${{ matrix.type }} Build
if: ${{ matrix.link-type == 'shared' }}
env:
MAKEFLAGS: 'MAX_HIGH_MEM_JOBS=8 MAX_LINK_JOBS=4'
run: |
EXTRA_CMAKE_FLAGS=(
"-DVELOX_ENABLE_BENCHMARKS=ON"
"-DVELOX_ENABLE_ARROW=ON"
"-DVELOX_ENABLE_PARQUET=ON"
"-DVELOX_ENABLE_HDFS=ON"
"-DVELOX_ENABLE_S3=ON"
"-DVELOX_ENABLE_GCS=ON"
"-DVELOX_ENABLE_ABFS=ON"
"-DVELOX_ENABLE_REMOTE_FUNCTIONS=ON"
"-DVELOX_MONO_LIBRARY=ON"
"-DVELOX_BUILD_SHARED=ON"
)
make '${{ matrix.type }}'
- name: Log binary sizes
run: |
mkdir -p /tmp/metrics
Expand Down Expand Up @@ -110,8 +131,8 @@ jobs:
CONBENCH_PROJECT_COMMIT: "${{ inputs.ref || github.sha }}"
run: |
./scripts/build-metrics.py upload \
--build_type "${{ matrix.type }}" \
--run_id "BM-${{ matrix.type }}-${{ github.run_id }}-${{ github.run_attempt }}" \
--build_type "${{ matrix.link-type }}-${{ matrix.type }}" \
--run_id "BM-${{ matrix.link-type }}-${{ matrix.type }}-${{ github.run_id }}-${{ github.run_attempt }}" \
--pr_number "${{ github.event.number }}" \
--sha "${{ inputs.ref || github.sha }}" \
"/tmp/metrics"
Expand Down

0 comments on commit 236eb57

Please sign in to comment.