test/system: Bump secondary fedora image from 29 to 32

The fedora-toolbox:32 image is the first of images in the renamed
toolbox image repository[0]. With the change we can drop the
pull_image_old() function because it was kept only for the old image.

Seems like newer version of ShellCheck checks the validity of variable
names (SC2153). This caused a false positive, so I silenced it.

[0] https://github.com/containers/toolbox/pull/615

https://github.com/containers/toolbox/pull/780
This commit is contained in:
Ondřej Míchal 2021-05-27 00:13:18 +02:00
parent d36cf1cf43
commit a24c2f6dc1
5 changed files with 18 additions and 28 deletions

View file

@ -59,17 +59,17 @@
cmd: "skopeo copy containers-storage:registry.fedoraproject.org/fedora-toolbox:{{ ansible_distribution_version }} dir:{{ zuul.project.src_dir }}/fedora-toolbox-{{ ansible_distribution_version }}"
creates: "{{ zuul.project.src_dir }}/fedora-toolbox-{{ ansible_distribution_version }}/manifest.json"
- name: Pull registry.fedoraproject.org/f29/fedora-toolbox:29
command: podman pull registry.fedoraproject.org/f29/fedora-toolbox:29
- name: Pull registry.fedoraproject.org/fedora-toolbox:32
command: podman pull registry.fedoraproject.org/fedora-toolbox:32
register: _podman
until: _podman.rc == 0
retries: 5
delay: 10
- name: Copy registry.fedoraproject.org/f29/fedora-toolbox:29 to a directory
- name: Copy registry.fedoraproject.org/fedora-toolbox:32 to a directory
command:
cmd: "skopeo copy containers-storage:registry.fedoraproject.org/f29/fedora-toolbox:29 dir:{{ zuul.project.src_dir }}/fedora-toolbox-29"
creates: "{{ zuul.project.src_dir }}/fedora-toolbox-29/manifest.json"
cmd: "skopeo copy containers-storage:registry.fedoraproject.org/fedora-toolbox:32 dir:{{ zuul.project.src_dir }}/fedora-toolbox-32"
creates: "{{ zuul.project.src_dir }}/fedora-toolbox-32/manifest.json"
- name: Clean up the local containers storage
command: podman system reset --force

View file

@ -1,5 +1,6 @@
# shellcheck shell=sh
# shellcheck disable=SC2153
[ "$BASH_VERSION" != "" ] || [ "$ZSH_VERSION" != "" ] || return 0
[ "$PS1" != "" ] || return 0

View file

@ -27,10 +27,10 @@ teardown() {
assert_success
}
@test "create: Create a container with a custom image and name ('fedora29'; f29)" {
pull_image_old 29
@test "create: Create a container with a custom image and name ('fedora32'; f32)" {
pull_image 32
run $TOOLBOX -y create -c "fedora29" -i fedora-toolbox:29
run $TOOLBOX -y create -c "fedora32" -i fedora-toolbox:32
assert_success
}
@ -53,17 +53,17 @@ teardown() {
assert_line --index 2 "Run 'toolbox --help' for usage."
}
@test "create: Create a container with a distro and release options ('fedora'; f29)" {
pull_image 29
@test "create: Create a container with a distro and release options ('fedora'; f32)" {
pull_image 32
run $TOOLBOX -y create -d "fedora" -r f29
run $TOOLBOX -y create -d "fedora" -r f32
assert_success
assert_output --partial "Created container: fedora-toolbox-29"
assert_output --partial "Enter with: toolbox enter --release 29"
assert_output --partial "Created container: fedora-toolbox-32"
assert_output --partial "Enter with: toolbox enter --release 32"
# Make sure the container has actually been created
run podman ps -a
assert_output --regexp "Created[[:blank:]]+fedora-toolbox-29"
assert_output --regexp "Created[[:blank:]]+fedora-toolbox-32"
}

View file

@ -50,7 +50,7 @@ teardown() {
@test "list: Try to list images and containers (no flag) with 3 containers and 2 images (the list should have 3 images and 2 containers)" {
# Pull the two images
pull_default_image
pull_image_old 29
pull_image 32
# Create tree containers
create_default_container
create_container non-default-one
@ -61,7 +61,7 @@ teardown() {
assert_success
assert_output --partial "fedora-toolbox:${DEFAULT_FEDORA_VERSION}"
assert_output --partial "fedora-toolbox:29"
assert_output --partial "fedora-toolbox:32"
# Check containers
run $TOOLBOX list --containers
@ -76,7 +76,7 @@ teardown() {
assert_success
assert_output --partial "fedora-toolbox:${DEFAULT_FEDORA_VERSION}"
assert_output --partial "fedora-toolbox:29"
assert_output --partial "fedora-toolbox:32"
assert_output --partial "fedora-toolbox-${DEFAULT_FEDORA_VERSION}"
assert_output --partial "non-default-one"
assert_output --partial "non-default-two"

View file

@ -40,17 +40,6 @@ function pull_image() {
}
function pull_image_old() {
local version
local image
version="$1"
image="${REGISTRY_URL}/f${version}/fedora-toolbox:${version}"
$SKOPEO copy "dir:${PROJECT_DIR}/fedora-toolbox-${version}" "containers-storage:${image}"
$PODMAN images
}
function pull_default_image() {
pull_image "${DEFAULT_FEDORA_VERSION}"
}