Skip to content

Commit

Permalink
Update LLVM toolchain to version 17 on hosted-runners (#2280)
Browse files Browse the repository at this point in the history
  • Loading branch information
mewim committed Oct 27, 2023
1 parent ece824f commit 1276a9b
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 12 deletions.
20 changes: 14 additions & 6 deletions .github/workflows/ci-workflow.yml
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,8 @@ jobs:
env:
NUM_THREADS: 32
TEST_JOBS: 16
CC: clang-14
CXX: clang++-14
CC: clang
CXX: clang++
steps:
- uses: actions/checkout@v3

Expand Down Expand Up @@ -216,7 +216,7 @@ jobs:
header-include-guard-check:
name: header include guard check
runs-on: kuzu-self-hosted-testing
runs-on: ubuntu-22.04
steps:
- uses: actions/checkout@v3

Expand All @@ -225,8 +225,13 @@ jobs:

clang-formatting-check:
name: clang-format check
runs-on: kuzu-self-hosted-testing
runs-on: ubuntu-22.04
steps:
- name: Install clang-format
run: |
sudo apt-get update
sudo apt-get install -y clang-format-11
- uses: actions/checkout@v3

- name: Check source format
Expand All @@ -237,11 +242,14 @@ jobs:

rustfmt-check:
name: rustfmt check
runs-on: kuzu-self-hosted-testing
runs-on: ubuntu-22.04
steps:
- name: Update Rust
run: rustup update

- uses: actions/checkout@v3

- name: Check api format
- name: Check Rust API format
working-directory: tools/rust_api
run: cargo fmt --check

Expand Down
17 changes: 11 additions & 6 deletions scripts/dockerized-ci-tests-runner/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -10,15 +10,12 @@ RUN curl -fsSL https://deb.nodesource.com/gpgkey/nodesource-repo.gpg.key | gpg -
RUN echo "deb [signed-by=/etc/apt/keyrings/nodesource.gpg] https://deb.nodesource.com/node_20.x nodistro main" | tee /etc/apt/sources.list.d/nodesource.list

RUN apt-get update && apt-get install -y \
build-essential \
ccache \
clang-14 \
clang-format-11 \
clang-tidy \
cmake \
curl \
g++ \
gcc \
git \
gnupg \
jq \
lcov \
libcurl4-openssl-dev \
Expand All @@ -30,9 +27,17 @@ RUN apt-get update && apt-get install -y \
python-is-python3 \
python3-dev \
python3-pip \
sudo \
software-properties-common \
wget

RUN wget https://apt.llvm.org/llvm.sh && \
chmod +x llvm.sh && \
yes | ./llvm.sh 17 all && \
rm llvm.sh && \
ln /usr/bin/clang++-17 /usr/bin/clang++ && \
ln /usr/bin/clang-17 /usr/bin/clang && \
ln /usr/bin/run-clang-tidy-17 /usr/bin/run-clang-tidy

ENV JAVA_HOME=/usr/lib/jvm/java-17-openjdk-amd64
RUN useradd --create-home runner
USER runner
Expand Down

0 comments on commit 1276a9b

Please sign in to comment.