Project

General

Profile

Actions

action #101451

closed

coordination #90086: [epic] Refactor container tests

Allow running both runtimes in the same job using main_containers schedule

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

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

0%

Estimated time:

Description

Currently, container jobs that use the main_containers.pm schedule include the variable CONTAINER_RUNTIME.
That variable can have 3 values

  • CONTAINER_RUNTIME=docker
  • CONTAINER_RUNTIME=podman
  • CONTAINER_RUNTIME=docker,podman (or viceversa)

If we want to merge podman_image.pm and docker_image.pm into single module, we need to pass the runtime to this new module somehow.
One idea is using runargs in the main_containers.pm, similar to how it is done here:
https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/lib/main_common.pm#L1848

e.g.

my $runtime = get_required_var('CONTAINERS_RUNTIME');
# check if it's `podman`, `docker` or both, if it's both, we will need to do the call to the modules twice (one for podman, another for docker).
loadtest($test, run_args => $runtime;

or something similar.


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 #1

Updated by jlausuch over 2 years ago

  • Status changed from New to Workable
Actions #2

Updated by ilausuch over 2 years ago

  • Status changed from Workable to In Progress
  • Assignee set to ilausuch
Actions #4

Updated by ilausuch over 2 years ago

  • Related to action #101210: Merge `podman_image.pm` and `docker_image.pm` into single module "image.pm" added
Actions #5

Updated by ilausuch over 2 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF