Skip to content

Support loading tests from recipe file#4208

Merged
psss merged 9 commits intomainfrom
fvagner-recipe-loading
Feb 13, 2026
Merged

Support loading tests from recipe file#4208
psss merged 9 commits intomainfrom
fvagner-recipe-loading

Conversation

@therazix
Copy link
Contributor

@therazix therazix commented Oct 17, 2025

This PR introduces an experimental recipe loading feature. Recipes can be loaded using the tmt run --recipe command. Currently, only discover phases and tests are supported, and they will replace any existing discover phases in the plan. To use this feature, the current run must already include a plan with the same name as the one defined in the recipe. tmt will then update that plan by replacing its discover phases.

Resolves #4170

Pull Request Checklist

  • implement the feature
  • write the documentation
  • extend the test coverage
  • update the specification
  • adjust plugin docstring
  • modify the json schema
  • mention the version
  • include a release note

@therazix therazix added the area | recipe Related to the tmt recipe handling label Oct 17, 2025
@github-project-automation github-project-automation bot moved this to backlog in planning Oct 17, 2025
@therazix therazix added this to the 1.60 milestone Oct 17, 2025
@therazix therazix moved this from backlog to implement in planning Oct 17, 2025
@therazix therazix force-pushed the fvagner-recipe-loading branch from a1ce1ae to f3de3f7 Compare October 17, 2025 11:57
@happz happz removed this from the 1.60 milestone Oct 20, 2025
@therazix therazix added this to the 1.61 milestone Oct 23, 2025
@therazix therazix force-pushed the fvagner-recipe-loading branch 2 times, most recently from 0426fed to ce75d87 Compare October 31, 2025 08:44
@therazix therazix marked this pull request as ready for review October 31, 2025 09:17
@therazix therazix force-pushed the fvagner-recipe-loading branch from ce75d87 to 4efdcaa Compare October 31, 2025 09:17
@therazix therazix moved this from implement to review in planning Oct 31, 2025
@bajertom bajertom force-pushed the fvagner-recipe-loading branch from 2e95e9e to 13e0f14 Compare November 2, 2025 12:05
@therazix therazix force-pushed the fvagner-recipe-loading branch from 13e0f14 to 97faeca Compare November 4, 2025 09:53
@LecrisUT LecrisUT self-requested a review November 4, 2025 12:13
@bajertom bajertom removed this from the 1.61 milestone Nov 5, 2025
@LecrisUT LecrisUT self-assigned this Nov 10, 2025
@psss psss self-assigned this Nov 10, 2025
Copy link
Contributor

@LecrisUT LecrisUT left a comment

Choose a reason for hiding this comment

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

Some preliminary review

@therazix therazix force-pushed the fvagner-recipe-loading branch from 148554d to dfe8c46 Compare November 12, 2025 16:44
@therazix therazix changed the base branch from main to fvagner-serialization-helpers November 12, 2025 16:45
@happz
Copy link
Contributor

happz commented Nov 16, 2025

As discussed on Thursday afternoon, post-lunch session, we are going to take a slight detour first: move functionality that should be shared out of discover plugins - git cloning, policies, libraries, etc. (not distgit though, that one will be left for later). We would also modify the internal API, as we will expect a discover plugin to return more info than today. Eventually, we would not need to run discover plugins' go() method when recipe is given, as recipe would provide all needed info to "restore" the discover product.

@happz happz moved this from review to backlog in planning Nov 20, 2025
@psss psss requested a review from tcornell-bus February 9, 2026 14:18
Copy link
Contributor

@psss psss left a comment

Choose a reason for hiding this comment

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

Thanks a lot for implementing this! Looks good and works as expected. Added a just few minor comments.

@psss psss force-pushed the fvagner-recipe-loading branch from 37a026c to 4b8a1d8 Compare February 9, 2026 19:59
@therazix therazix force-pushed the fvagner-recipe-loading branch from 4b8a1d8 to fafe7d7 Compare February 10, 2026 10:03
@psss psss moved this from review to merge in planning Feb 10, 2026
@psss
Copy link
Contributor

psss commented Feb 10, 2026

/packit test

@psss
Copy link
Contributor

psss commented Feb 11, 2026

/packit retest-failed

@psss psss force-pushed the fvagner-recipe-loading branch 3 times, most recently from 4fcd2d5 to c78989a Compare February 12, 2026 09:25
@psss psss force-pushed the fvagner-recipe-loading branch 3 times, most recently from 011e2b6 to 1f7ae3c Compare February 12, 2026 17:27
@psss psss force-pushed the fvagner-recipe-loading branch from 1f7ae3c to 78a9dc6 Compare February 13, 2026 00:05
@psss psss merged commit b20289b into main Feb 13, 2026
31 checks passed
@psss psss deleted the fvagner-recipe-loading branch February 13, 2026 08:15
@github-project-automation github-project-automation bot moved this from merge to done in planning Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area | recipe Related to the tmt recipe handling ci | full test Pull request is ready for the full test execution status | postponed Let's postpone for the next sprint

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

Load tests from recipe.yaml file

7 participants