Skip to content

Commit

Permalink
beaker: Make vagrant installation optional
Browse files Browse the repository at this point in the history
When we run beaker jobs for vagrant_libvirt on dedicated servers, we
control the host. That means we can already install the apt repo +
packages. We even have to do it because when multiple beaker jobs start
and they want to call `apt` in parallel, it fails.
  • Loading branch information
bastelfreak committed Aug 19, 2024
1 parent 4e7257a commit 4bd99a1
Showing 1 changed file with 7 additions and 1 deletion.
8 changes: 7 additions & 1 deletion .github/workflows/beaker.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,11 @@ on:
default: docker
required: false
type: string
install_vagrant_dependencies:
description: When true and beaker_hypervisor==vagrant_libvirt, we will install vagrant + libvirt
default: true
required: false
type: boolean
beaker_facter:
description: Expand the Beaker matrix based on a fact
default: 'false'
Expand Down Expand Up @@ -163,7 +168,7 @@ jobs:
sudo apt-get update
sudo apt-get install -y ${{ inputs.additional_packages }}
- name: Setup libvirt for Vagrant
if: ${{ inputs.beaker_hypervisor == 'vagrant_libvirt' }}
if: ${{ inputs.beaker_hypervisor == 'vagrant_libvirt' && inputs.install_vagrant_dependencies == true }}
run: |
sudo add-apt-repository ppa:evgeni/vagrant
sudo apt-get update
Expand All @@ -176,6 +181,7 @@ jobs:
bundler-cache: true
cache-version: ${{ inputs.cache-version }}
working-directory: ${{ inputs.working-directory }}
self-hosted: ${{ inputs.acceptance_runs_on != 'ubuntu-24.04' }}
- name: Run tests
run: bundle exec rake beaker
env: ${{ matrix.env }}
Expand Down

0 comments on commit 4bd99a1

Please sign in to comment.