From 2cb2571127e3d82f59133f63fca97951691859aa Mon Sep 17 00:00:00 2001 From: Jonathan Lebon Date: Wed, 4 Jul 2018 11:59:18 -0400 Subject: [PATCH] tests/installed: Add NOTE when re-using RPMs One gotcha here is that we don't invalidate the RPMs if we're not sitting on the same commit anymore. Shouldn't be too hard to fix, though let's at least make a note of it for now. Closes: #1668 Approved by: cgwalters --- tests/installed/playbook-run.sh | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tests/installed/playbook-run.sh b/tests/installed/playbook-run.sh index 37572bd4..af906c3d 100755 --- a/tests/installed/playbook-run.sh +++ b/tests/installed/playbook-run.sh @@ -7,6 +7,10 @@ dn=$(cd $(dirname $0) && pwd) if ! test -d build; then mkdir -p build (cd build && ${dn}/../../ci/build-rpm.sh) +else + # XXX: we should invalidate based on `git describe` or something + echo "NOTE: Re-using prebuilt RPMs:" + find build -name '*.rpm' fi # https://fedoraproject.org/wiki/CI/Tests