Project

General

Profile

Actions

action #101213

closed

coordination #90086: [epic] Refactor container tests

Merge `podman_3rd_party_images.pm` and `docker_3rd_party_images.pm` into single module "3rd_party_images.pm"

Added by jlausuch over 2 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
Start date:
2021-10-20
Due date:
% Done:

0%

Estimated time:

Description

The only meaningful diff between these 2 files is:

podman_3rd_party_images.pm:

script_run("echo 'Container base image tests:' > /var/tmp/podman-3rd_party_images_log.txt");
...
install_podman_when_needed($host_distri);
...
upload_3rd_party_images_logs('podman');

docker_3rd_party_images.pm :

script_run("echo 'Container base image tests:' > /var/tmp/docker-3rd_party_images_log.txt");
...
install_docker_when_needed($host_distri);
...
upload_3rd_party_images_logs('docker');

Since we have the variable CONTAINER_RUNTIME, which can have docker, podman or docker,podman, it doesn't make sense to keep duplicated code doing the same thing.


Related issues 1 (0 open1 closed)

Related to Containers - action #101210: Merge `podman_image.pm` and `docker_image.pm` into single module "image.pm"Resolvedilausuch2021-10-20

Actions
Actions

Also available in: Atom PDF