Skip to content

Comments

Add basic tests in packit#921

Draft
ekohl wants to merge 2 commits intotheforeman:developfrom
ekohl:add-tmt-tests
Draft

Add basic tests in packit#921
ekohl wants to merge 2 commits intotheforeman:developfrom
ekohl:add-tmt-tests

Conversation

@ekohl
Copy link
Member

@ekohl ekohl commented Mar 19, 2024

This uses tmt to run basic tests on packit builds.

Inspired by #918 without duplicating the whole test setup. I've chosen tmt for its integration in packit. It would be interesting to see if this can also be used standalone with a development install.

@ehelms
Copy link
Member

ehelms commented Mar 19, 2024

Do you have any docs on what tmt is, why we want to use it and where we draw the line between GHA and whatever tmt is?

@ekohl
Copy link
Member Author

ekohl commented Mar 19, 2024

It utilizes tmt and Packit in turn uses Testing Farm to run those. The idea is that you run some tests after you've built your package on a real system. If this works, we can give a much better guarantee our packages actually work. Unlike the %check phase in RPM, this runs on actually machines (containers or VMs).

@nforro
Copy link

nforro commented Apr 2, 2024

@ekohl You need to set use_internal_tf: true on the tests job if you want to use RHEL composes.

ekohl added 2 commits April 2, 2024 18:29
This uses tmt to run basic tests on packit builds.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants