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.

Sign up for GitHub

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

Jump to bottom

Continue after prepare failure in tmt try #3505

Draft

lukaszachy wants to merge 1 commit into main from try-prepare-nonfatal

Collaborator

lukaszachy commented Jan 31, 2025

Useful if machine needs tweaks before prepare can work, e.g. enable repos.
Fix: #2685

WDYT ? Error message needs tweaks (e.g write it in red...) but any objections against this behaviour change?

I often hit this if test requires something from beaker task library or crb isn't enabled in the image.

Pull Request Checklist


          Continue after prepare failure in tmt try

7c47bf1

Useful if machine needs tweaks before prepare can work, e.g. enable
repos.

Fix: #2685

lukaszachy added the command | try label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

psss Awaiting requested review from psss psss will be requested when the pull request is marked ready for review psss is a code owner

happz Awaiting requested review from happz happz will be requested when the pull request is marked ready for review happz is a code owner

thrix Awaiting requested review from thrix thrix will be requested when the pull request is marked ready for review thrix is a code owner

janhavlin Awaiting requested review from janhavlin janhavlin will be requested when the pull request is marked ready for review janhavlin is a code owner

martinhoyer Awaiting requested review from martinhoyer martinhoyer will be requested when the pull request is marked ready for review martinhoyer is a code owner

At least 2 approving reviews are required to merge this pull request.

Labels