Skip to content

Commit

Permalink
Do not execute Docker image tests as root
Browse files Browse the repository at this point in the history
- because newer versions of Firefox and Chrome can't deal with that
- and because we shouldn't have to anyway

- move beakerlib setup inside GHA because this is where it belongs
  • Loading branch information
atodorov committed Mar 7, 2024
1 parent 3867592 commit 837cf4a
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 11 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,15 @@ jobs:
- name: Setup
run: |
sudo apt-get update
sudo apt-get install firefox wrk
sudo apt-get install firefox git make wrk
# install beakerlib from source it doesn't ship DEB packages
if [ ! -f "/usr/share/beakerlib/beakerlib.sh" ]; then
git clone https://github.com/beakerlib/beakerlib.git
sudo make -C beakerlib/ install
fi
pip install -r requirements/devel.txt
- name: Docker version info
run: |
Expand All @@ -272,7 +280,6 @@ jobs:
- name: Execute tests
run: |
pip install -r requirements/devel.txt
make test-docker-image
docker images
Expand Down
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ docker-manifest:

.PHONY: test-docker-image
test-docker-image: docker-image
sudo --preserve-env env PATH=$$PATH ./tests/runner.sh
./tests/runner.sh

.PHONY: docs
docs:
Expand Down
8 changes: 0 additions & 8 deletions tests/runner.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,14 +6,6 @@
rm -rf /var/tmp/beakerlib-*/
export BEAKERLIB_JOURNAL=0

# install beakerlib from source b/c beakerlib doesn't ship
# .deb packages
if [ ! -f "/usr/share/beakerlib/beakerlib.sh" ]; then
sudo apt-get update
sudo apt-get install git make
git clone https://github.com/beakerlib/beakerlib.git
make -C beakerlib/ install
fi

# execute test scripts
./tests/test_docker.sh
Expand Down

0 comments on commit 837cf4a

Please sign in to comment.