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

Add packit job copr and tests #996

Merged
merged 1 commit into from
Nov 10, 2023
Merged

Add packit job copr and tests #996

merged 1 commit into from
Nov 10, 2023

Conversation

inknos
Copy link
Collaborator

@inknos inknos commented Nov 3, 2023

Requires rpm-software-management/ci-dnf-stack#1405
Tests will fail. Once merged test with /packit test before merging.

Requirements to onboard:

  • wrap our ci so that we don't have to reinvent the wheel
    • use the latest upstream ci-dnf-stack and the same scripts provided by us
    • respect the requirements of running destructive tests in separate containers
  • provide copr builds for inspections

Key advantages:

  • build rpms for all fedoras using copr
  • run tests for all fedoras
  • reproduce the tests locally (provided by tmt)
  • possibility to run tests directly from ci-dnf-stack via /packit test rpm-software-management/dnf5#973 (provided that the pr has a packit build) Test tmt/packit integration ci-dnf-stack#1398

Future possible improvements:

  • possibility to rerun part of the tests in the future via /packit test <sometest>
  • possibility to split the test ci and run separate tiers for different systems

Once this PR gets merged and works, the old actions can be removed: #1008

@inknos inknos marked this pull request as ready for review November 9, 2023 15:39
@inknos inknos force-pushed the tmt-test branch 2 times, most recently from 40e96d7 to 86a78f0 Compare November 9, 2023 15:41
@inknos inknos requested a review from m-blaha November 9, 2023 15:55
@inknos inknos force-pushed the tmt-test branch 2 times, most recently from c639d90 to 5166af7 Compare November 9, 2023 16:02
@inknos inknos changed the title Add tests using tmt and packit Add packit job copr and tests Nov 9, 2023
@m-blaha
Copy link
Member

m-blaha commented Nov 10, 2023

/packit tests

@m-blaha
Copy link
Member

m-blaha commented Nov 10, 2023

/packit test

Copy link
Member

@m-blaha m-blaha left a comment

Choose a reason for hiding this comment

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

Thanks!

@m-blaha m-blaha added this pull request to the merge queue Nov 10, 2023
Merged via the queue into main with commit ceb2bc1 Nov 10, 2023
13 of 15 checks passed
@m-blaha m-blaha deleted the tmt-test branch November 10, 2023 16:31
@inknos inknos mentioned this pull request Dec 5, 2023
5 tasks
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