Skip to content

Conversation

@ricardobranco777
Copy link
Contributor

@ricardobranco777 ricardobranco777 commented Aug 20, 2025

Add tests for python3-docker & python3-podman.

This brings lots of benefits:

These tests take only ~7 minutes.

TODO (in another PR):

Verification runs:

Using GIT_REPO: https://openqa.opensuse.org/tests/5258587

@ricardobranco777 ricardobranco777 added the WIP Work in progress label Aug 20, 2025
@ricardobranco777 ricardobranco777 marked this pull request as draft August 20, 2025 22:59
@github-actions
Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@ricardobranco777 ricardobranco777 force-pushed the runtime_py branch 23 times, most recently from e7687a9 to 48730a9 Compare August 22, 2025 17:39
@ricardobranco777 ricardobranco777 force-pushed the runtime_py branch 13 times, most recently from 9de37d7 to 3c1cb31 Compare August 24, 2025 16:49
@ricardobranco777 ricardobranco777 marked this pull request as ready for review August 24, 2025 19:51
@ricardobranco777 ricardobranco777 removed the WIP Work in progress label Aug 24, 2025
Copy link
Member

@pdostal pdostal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is pretty long. I wonder if it really makes sense to keep docker&podman together here.

@ricardobranco777
Copy link
Contributor Author

ricardobranco777 commented Aug 25, 2025

This is pretty long. I wonder if it really makes sense to keep docker&podman together here.

There's no need to review the backported fixes. Only the new module and the scheduling.

Splitting would be a mess because they share common code.

@ricardobranco777 ricardobranco777 merged commit 7734c29 into os-autoinst:master Aug 25, 2025
12 checks passed
@ricardobranco777 ricardobranco777 deleted the runtime_py branch August 25, 2025 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants