Skip to content

Commit

Permalink
github: s/ubuntu-latest/ubuntu-24.04/
Browse files Browse the repository at this point in the history
Ubuntu 24.04 is now available, see
actions/runner-images#9848

so instead of building and running the tests in a fedora 40 container,
let's do the CI with the shinny ubuntu runner image, which preinstalls
some of the necessary building dependencies, so in theory, it should be
relatively faster than installing every package on fedora 40.

and since we are now using an image with git pre-installed, there
is no need to install `git` anymore. so, in this change, we drop the
step to install `git`.

because we are switching to a debian derivative distro, let's use
apt-get instead of dnf to install packages.

Signed-off-by: Kefu Chai <kefu.chai@scylladb.com>
  • Loading branch information
tchaikov committed Oct 1, 2024
1 parent 376f7eb commit ee3d140
Showing 1 changed file with 4 additions and 9 deletions.
13 changes: 4 additions & 9 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,13 +37,8 @@ on:
jobs:
test:
timeout-minutes: 40
runs-on: ubuntu-latest
container: fedora:40
runs-on: ubuntu-24.04
steps:
- name: Install Git
run: |
sudo dnf -y install git
- uses: actions/checkout@v4
with:
submodules: "${{ contains(inputs.enables, 'dpdk') }}"
Expand All @@ -55,17 +50,17 @@ jobs:
- name: Install clang++
if: ${{ inputs.compiler == 'clang++' }}
run: |
sudo dnf -y install clang
sudo apt-get -y install clang
- name: Install clang-scan-deps
if: ${{ contains(inputs.enables, 'cxx-modules') }}
run: |
sudo dnf -y install clang-tools-extra
sudo apt-get -y install clang-tools
- name: Install ccache
if: ${{ inputs.enable-ccache }}
run: |
sudo dnf -y install ccache
sudo apt-get -y install ccache
- name: Setup ccache
if: ${{ inputs.enable-ccache }}
Expand Down

0 comments on commit ee3d140

Please sign in to comment.