Help plans/friends pick the correct version of tmt to test against#4841
Open
LecrisUT wants to merge 2 commits into
Open
Help plans/friends pick the correct version of tmt to test against#4841LecrisUT wants to merge 2 commits into
LecrisUT wants to merge 2 commits into
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
psss
pushed a commit
to teemtee/fmf
that referenced
this pull request
May 5, 2026
The state of `/plans/integration` has been out-of-sync for a while, namely that we do not use python `yq` anymore. Ideally we would use the `plans/friends` in tmt to have tmt handle the necessary dependencies. That is waiting on teemtee/tmt#4841. In the meantime we are doing some cleanup of the plan here partially reflecting the design described in teemtee/tmt#4841. - Drop the usage of `fedora` branch which is not being updated automatically. There are some ideas of how we could bring it back to track which should be discussed in teemtee/tmt#4841 and its followups - Track the `@teemtee/latest` copr (tmt `main` branch) - Add a design that is reusable in downstream packaging as well, basically taking the tmt files from the most relevant state of the dist-git
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The issue here is that if you just do a
plans.importof the friends plan there is a disconnect between what tmt version is installed (currently controlled by the importer) and the version of the test files used. However we have an explicit connection between@teemtee/latestand themainbranch which we use here to enforce that connection. Ideally we would do that for@teemtee/stablealso, but that requires some branch/mutable tag that follows the latest successful release, an issue to be resolved later.The tricky part is making this also available in the dist-git testing. It seems we can actually do that since the dist-git-source supports using a different package's dist-git, although we are blocked by the restriction in #4840. It magically just works in most cases except for inter-dependent dist-git PRs (multiple PRs that need to be landed in a side-tag), but there are enough failure points there anyway.
On the side of the users of plans/friends, they would have to remove their definition of the
@teemtee/*copr repo and let us handle it instead.Depends-on: #4840
Fixes #4539
CC @sopos @lsm5