Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distro: add a functional test for Debian probes #6030

Merged

Conversation

clebergnu
Copy link
Contributor

While the standard "debian" container is not so easily suitable for being used in the existing functional tests for distros that run an Avocado job, the very standard "python" container is built on Debian and provides Python (of course).

This should serve to avoid regressions to the distro probes for Debian.

@clebergnu clebergnu added this to the 108 - Codename TBD milestone Sep 16, 2024
@clebergnu clebergnu self-assigned this Sep 16, 2024
@clebergnu clebergnu mentioned this pull request Sep 16, 2024
@clebergnu clebergnu force-pushed the distro_test_debian_12_7 branch from ba3bf2f to 9f42224 Compare September 16, 2024 08:50
@clebergnu clebergnu force-pushed the distro_test_debian_12_7 branch from 9f42224 to 5003e5a Compare September 24, 2024 11:22
@clebergnu clebergnu requested a review from richtja September 24, 2024 11:22
selftests/functional/utils/distro.py Outdated Show resolved Hide resolved
While the standard "debian" container is not so easily suitable for
being used in the existing functional tests for distros that run an
Avocado job, the very standard "python" container is built on Debian
and provides Python (of course).

This should serve to avoid regressions to the distro probes for
Debian.

Signed-off-by: Cleber Rosa <[email protected]>
@clebergnu clebergnu force-pushed the distro_test_debian_12_7 branch from 5003e5a to ef2474c Compare October 4, 2024 11:19
@clebergnu clebergnu requested a review from richtja October 4, 2024 11:20
Copy link
Contributor

@richtja richtja left a comment

Choose a reason for hiding this comment

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

It LGTM, thanks.

@clebergnu
Copy link
Contributor Author

Thanks for the review @richtja! Merging it!

@clebergnu clebergnu merged commit 24b15fe into avocado-framework:master Oct 4, 2024
58 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants