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

Onboard Tmt #1405

Merged
merged 2 commits into from
Nov 10, 2023
Merged

Onboard Tmt #1405

merged 2 commits into from
Nov 10, 2023

Conversation

inknos
Copy link
Contributor

@inknos inknos commented Nov 9, 2023

This PR onboards tmt to test DNF5 only.

The checks will run only when requested via /packit tests or /packit tests rpm-software-management/dnf5#PRNUMBER.

The PR also checks tmt plans with pre-commit tests.

Dockerfile Outdated Show resolved Hide resolved
plans/main.fmf Outdated
BASE: "fedora:39"
- when: distro == fedora-rawhide
environment:
BASE: "fedora:rawhide"
Copy link
Member

Choose a reason for hiding this comment

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

This adjustments are a bit unfortunate - we will need to adjust them every time a new Fedora is branched. Would it be possible to use original distro value and do something like this?

...container-test build --base $(echo "$BASE" | tr '-' ':') ...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

for fedora no problem but we'll need to adjust stuff for centos, rhel etc in the future

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@m-blaha I tweaked the --base option but we will need to be smarter if we'll want to add Centos tests too

Copy link
Member

Choose a reason for hiding this comment

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

Thank you.

@inknos
Copy link
Contributor Author

inknos commented Nov 10, 2023

/packit test

@m-blaha m-blaha merged commit de35295 into main Nov 10, 2023
7 of 9 checks passed
@m-blaha m-blaha deleted the tmt-test branch November 10, 2023 15:02
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