Skip to content

Commit

Permalink
Merge pull request #587 from mikedep333/fix_pulp_tests
Browse files Browse the repository at this point in the history
Fix pulp CI tests failing for 2 different reasons
  • Loading branch information
mikedep333 authored Feb 5, 2024
2 parents 768b19e + 0ae887c commit f5bb168
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGES/584.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fix pulp CI tests failing with "pulp: command not found"
1 change: 1 addition & 0 deletions CHANGES/588.bugfix
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Workaround pulp_tests.sh failing to build the latest branch of openshift-examples/web.git
8 changes: 7 additions & 1 deletion images/s6_assets/pulp_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@ else
cd pulp-cli
fi
pip install -r test_requirements.txt || pip install --no-build-isolation -r test_requirements.txt
pip install pulp-cli
if [ -e tests/cli.toml ]; then
mv tests/cli.toml "tests/cli.toml.bak.$(date -R)"
fi
Expand Down Expand Up @@ -54,7 +55,12 @@ podman exec -u pulp pulp bash -c "pulpcore-manager add-signing-service --class d

# Test buildah for pulp_container's usage
podman exec -u pulp pulp podman system migrate
podman exec -u pulp pulp podman build https://github.com/openshift-examples/web.git
# This is currently not working due to a bug in it, so workaround below
# podman exec -u pulp pulp podman build https://github.com/openshift-examples/web.git
podman exec -u pulp pulp git clone https://github.com/openshift-examples/web.git /tmp/web
podman exec -u pulp pulp git -C /tmp/web checkout 7a5e95bbf7111f32be27b5fc9bc0070f844a03a3
podman exec -u pulp pulp podman build /tmp/web

# Test skopeo for pulp_container's usage with an image with the nobody uid 65534
# (And the image that pulp_container CI actually tests with)
podman exec -u pulp pulp podman pull docker.io/library/busybox
Expand Down

0 comments on commit f5bb168

Please sign in to comment.