Skip to content

Commit

Permalink
system test image: bump to 20220615
Browse files Browse the repository at this point in the history
Changes:
 - use --timestamp option to produce 'created' stamps
   that can be reliably tested in the image-history test

 - podman now supports manifest & multiarch run, so we
   no longer need buildah

 - bump up base alpine & busybox images

This turned out to be WAY more complicated than it should've been,
because:

 - alpine 3.14 fixed 'date -Iseconds' to include a colon in
   the TZ offset ("-07:00", was "-0700"). This is now consistent
   with GNU date's --iso-8601 format, yay, so we can eliminate
   a minor workaround.

 - with --timestamp, all ADDed files are set to that timestamp,
   including the custom-reference-timestamp file that many tests
   rely on. So we need to split the build into two steps. But:

 - ...with a two-step build I need to use --squash-all, not --squash, but:

 - ... (deep sigh) --squash-all doesn't work with --timestamp (containers#14536)
   so we need to alter existing tests to deal with new image layers.

 - And, long and sordid story relating to --rootfs. TL;DR that option
   only worked by a miracle relating to something special in one
   specific test image; it doesn't work with any other images. Fix
   seems to be complicated, so we're bypassing with a FIXME (containers#14505).

And, unrelated:

 - remove obsolete skip and workaround in run-basic test (dating
   back to varlink days)
 - add a pause-image cleanup to avoid icky red warnings in logs

Fixes: containers#14456

Signed-off-by: Ed Santiago <[email protected]>
  • Loading branch information
edsantiago committed Jun 15, 2022
1 parent 3109534 commit 0a202a9
Show file tree
Hide file tree
Showing 6 changed files with 75 additions and 39 deletions.
5 changes: 5 additions & 0 deletions test/system/010-images.bats
Original file line number Diff line number Diff line change
Expand Up @@ -158,6 +158,11 @@ Labels.created_at | 20[0-9-]\\\+T[0-9:]\\\+Z
# start here because this is the first one, fix this problem.
# You can (probably) ignore any subsequent failures showing '@sha'
# in the error output.
#
# WARNING! This test is likely to fail for an hour or so after
# building a new testimage (via build-testimage script), because
# two consecutive 'podman images' may result in a one-minute
# difference in the "XX minutes ago" output. This is OK to ignore.
run_podman images -a
is "$output" "$images_baseline" "images -a, after pull: same as before"

Expand Down
19 changes: 8 additions & 11 deletions test/system/030-run.bats
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,8 @@ echo $rand | 0 | $rand
# FIXME: The </dev/null is a hack, necessary because as of 2019-09
# podman-remote has a bug in which it silently slurps up stdin,
# including the output of parse_table (i.e. tests to be run).
run_podman $expected_rc run $IMAGE "$@" </dev/null

# FIXME: remove conditional once podman-remote issue #4096 is fixed
if ! is_remote; then
is "$output" "$expected_output" "podman run $cmd - output"
fi
run_podman $expected_rc run $IMAGE "$@"
is "$output" "$expected_output" "podman run $cmd - output"

tests_run=$(expr $tests_run + 1)
done < <(parse_table "$tests")
Expand Down Expand Up @@ -470,10 +466,10 @@ json-file | f
# dependent, we pick an obscure zone (+1245) that is unlikely to
# collide with any of our testing environments.
#
# To get a reference timestamp we run 'date' locally; note the explicit
# strftime() format. We can't use --iso=seconds because GNU date adds
# a colon to the TZ offset (eg -07:00) whereas alpine does not (-0700).
run date --date=@1600000000 +%Y-%m-%dT%H:%M:%S%z
# To get a reference timestamp we run 'date' locally. This requires
# that GNU date output matches that of alpine; this seems to be true
# as of testimage:20220615.
run date --date=@1600000000 --iso=seconds
expect="$output"
TZ=Pacific/Chatham run_podman run --rm --tz=local $IMAGE date -Iseconds -r $testfile
is "$output" "$expect" "podman run with --tz=local, matches host"
Expand Down Expand Up @@ -628,7 +624,8 @@ json-file | f
run_podman image mount $IMAGE
romount="$output"

run_podman run --rm --rootfs $romount echo "Hello world"
# FIXME FIXME FIXME: Remove :O once (if) #14504 is fixed!
run_podman run --rm --rootfs $romount:O echo "Hello world"
is "$output" "Hello world"

run_podman image unmount $IMAGE
Expand Down
7 changes: 6 additions & 1 deletion test/system/070-build.bats
Original file line number Diff line number Diff line change
Expand Up @@ -496,7 +496,12 @@ Labels.$label_name | $label_value
"image tree: third line"
is "${lines[3]}" "Image Layers" \
"image tree: fourth line"
is "${lines[4]}" ".* ID: [0-9a-f]\{12\} Size: .* Top Layer of: \[$IMAGE]" \
# FIXME: if #14536 is ever fixed, rebuild testimage & s/5/4/ below.
# Summary: this should be ${lines[4]}, not [5], and prior to 2022-06-15
# it was. Unfortunately, a nightmarish bug interaction makes it impossible
# for us to use --squash-all on our testimage. Unless/until that bug is
# fixed, we have an extra layer that all we can do is ignore.
is "${lines[5]}" ".* ID: [0-9a-f]\{12\} Size: .* Top Layer of: \[$IMAGE]" \
"image tree: first layer line"
is "${lines[-1]}" ".* ID: [0-9a-f]\{12\} Size: .* Top Layer of: \[localhost/build_test:latest]" \
"image tree: last layer line"
Expand Down
3 changes: 3 additions & 0 deletions test/system/500-networking.bats
Original file line number Diff line number Diff line change
Expand Up @@ -165,6 +165,9 @@ load helpers

run_podman pod rm $pod_name
is "$output" "$pid" "Only ID in output (no extra errors)"

# Clean up
run_podman rmi $(pause_image)
}

@test "podman run with slirp4ns assigns correct addresses to /etc/hosts" {
Expand Down
76 changes: 51 additions & 25 deletions test/system/build-testimage
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
# still need a fedora image for that.
#

# Buildah binary
BUILDAH=${BUILDAH:-buildah}
# Podman binary to use
PODMAN=${PODMAN:-$(pwd)/bin/podman}

# Tag for this new image
YMD=$(date +%Y%m%d)
Expand All @@ -25,7 +25,8 @@ if [ -z "$create_script" ]; then
fi

# Creation timestamp, Zulu time
create_time_z=$(env TZ=UTC date +'%Y-%m-%dT%H:%M:%SZ')
create_time_t=$(date +%s)
create_time_z=$(env TZ=UTC date --date=@$create_time_t +'%Y-%m-%dT%H:%M:%SZ')

set -ex

Expand Down Expand Up @@ -60,40 +61,51 @@ chmod 755 pause
# alpine because it's small and light and reliable
# - check for updates @ https://hub.docker.com/_/alpine
# busybox-extras provides httpd needed in 500-networking.bats
cat >Containerfile <<EOF
#
# Two Containerfiles, because we have to do the image build in two parts,
# which I think are easier to describe in reverse order:
# 2) The second build has to be run with --timestamp=CONSTANT, otherwise
# the Created test in 110-history.bats may fail (#14456); but
# 1) the timestamp of the testimage-id file must be preserved (see above),
# and 'build --timestamp' clobbers all file timestamps.
#
cat >Containerfile1 <<EOF
ARG REPO=please-override-repo
FROM docker.io/\${REPO}/alpine:3.13.5
FROM docker.io/\${REPO}/alpine:3.16.0
RUN apk add busybox-extras
ADD testimage-id pause /home/podman/
EOF

cat >Containerfile2 <<EOF
FROM localhost/interim-image:latest
LABEL created_by=$create_script
LABEL created_at=$create_time_z
WORKDIR /home/podman
CMD ["/bin/echo", "This container is intended for podman CI testing"]
EOF

# --squash-all : needed by 'tree' test in 070-build.bats
podman rmi -f testimage &> /dev/null || true
# Start from scratch
testimg_base=quay.io/libpod/testimage
testimg=${testimg_base}:$YMD
$PODMAN rmi -f $testimg &> /dev/null || true

# There should always be a testimage tagged ':0000000<X>' (eight digits,
# zero-padded sequence ID) in the same location; this is used by tests
# which need to pull a non-locally-cached image. This image will rarely
# if ever need to change, nor in fact does it even have to be a copy of
# this testimage since all we use it for is 'true'.
# However, it does need to be multiarch :-(
zerotag_latest=$(skopeo list-tags docker://quay.io/libpod/testimage |\
zerotag_latest=$(skopeo list-tags docker://${testimg_base} |\
jq -r '.Tags[]' |\
sort --version-sort |\
grep '^000' |\
tail -n 1)
zerotag_next=$(printf "%08d" $((zerotag_latest + 1)))

# We don't always need to push the :00xx image, but build it anyway.
zeroimg=quay.io/libpod/testimage:${zerotag_next}
buildah manifest create $zeroimg
zeroimg=${testimg_base}:${zerotag_next}
$PODMAN manifest create $zeroimg

# We need to use buildah because (as of 2021-02-23) only buildah has --manifest
# and because Dan says arch emulation is not currently working on podman
# (no further details).
# Arch emulation on Fedora requires the qemu-user-static package.
for arch in amd64 arm64 ppc64le s390x;do
# docker.io repo is usually the same name as the desired arch; except
Expand All @@ -104,31 +116,45 @@ for arch in amd64 arm64 ppc64le s390x;do
repo="${repo}v8"
fi

${BUILDAH} bud \
--arch=$arch \
--build-arg REPO=$repo \
--manifest=testimage \
--squash \
.
# First build defines REPO, but does not have --timestamp
$PODMAN build \
--arch=$arch \
--build-arg REPO=$repo \
--squash-all \
--file Containerfile1 \
-t interim-image \
.

# Second build forces --timestamp, and adds to manifest. Unfortunately
# we can't use --squash-all with --timestamp: *all* timestamps get
# clobbered. This is not fixable (#14536).
$PODMAN build \
--arch=$arch \
--timestamp=$create_time_t \
--manifest=$testimg \
--squash \
--file Containerfile2 \
.

# No longer need the interim image
$PODMAN rmi interim-image

# The zero-tag image
${BUILDAH} pull --arch $arch docker.io/$repo/busybox:1.33.1
${BUILDAH} manifest add $zeroimg docker.io/$repo/busybox:1.33.1
$PODMAN pull --arch $arch docker.io/$repo/busybox:1.34.1
$PODMAN manifest add $zeroimg docker.io/$repo/busybox:1.34.1
done

# Clean up
cd /tmp
rm -rf $tmpdir

# Tag image and push (all arches) to quay.
remote_tag=quay.io/libpod/testimage:$YMD
podman tag testimage ${remote_tag}
cat <<EOF
If you're happy with these images, run:
${BUILDAH} manifest push --all ${remote_tag} docker://${remote_tag}
${BUILDAH} manifest push --all ${zeroimg} docker://${zeroimg}
podman manifest push --all ${testimg} docker://${testimg}
podman manifest push --all ${zeroimg} docker://${zeroimg}
(You do not always need to push the :0000 image)
Expand Down
4 changes: 2 additions & 2 deletions test/system/helpers.bash
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,14 @@ PODMAN=${PODMAN:-podman}
PODMAN_TEST_IMAGE_REGISTRY=${PODMAN_TEST_IMAGE_REGISTRY:-"quay.io"}
PODMAN_TEST_IMAGE_USER=${PODMAN_TEST_IMAGE_USER:-"libpod"}
PODMAN_TEST_IMAGE_NAME=${PODMAN_TEST_IMAGE_NAME:-"testimage"}
PODMAN_TEST_IMAGE_TAG=${PODMAN_TEST_IMAGE_TAG:-"20210610"}
PODMAN_TEST_IMAGE_TAG=${PODMAN_TEST_IMAGE_TAG:-"20220615"}
PODMAN_TEST_IMAGE_FQN="$PODMAN_TEST_IMAGE_REGISTRY/$PODMAN_TEST_IMAGE_USER/$PODMAN_TEST_IMAGE_NAME:$PODMAN_TEST_IMAGE_TAG"
PODMAN_TEST_IMAGE_ID=

# Remote image that we *DO NOT* fetch or keep by default; used for testing pull
# This has changed in 2021, from 0 through 3, various iterations of getting
# multiarch to work. It should change only very rarely.
PODMAN_NONLOCAL_IMAGE_TAG=${PODMAN_NONLOCAL_IMAGE_TAG:-"00000003"}
PODMAN_NONLOCAL_IMAGE_TAG=${PODMAN_NONLOCAL_IMAGE_TAG:-"00000004"}
PODMAN_NONLOCAL_IMAGE_FQN="$PODMAN_TEST_IMAGE_REGISTRY/$PODMAN_TEST_IMAGE_USER/$PODMAN_TEST_IMAGE_NAME:$PODMAN_NONLOCAL_IMAGE_TAG"

# Because who wants to spell that out each time?
Expand Down

0 comments on commit 0a202a9

Please sign in to comment.