-
Notifications
You must be signed in to change notification settings - Fork 47
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
Onboard Tmt #1405
Conversation
plans/main.fmf
Outdated
BASE: "fedora:39" | ||
- when: distro == fedora-rawhide | ||
environment: | ||
BASE: "fedora:rawhide" |
There was a problem hiding this comment.
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 '-' ':') ...
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you.
/packit test |
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.