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

Allow running next plan in queue when one fails to complete #2034

Merged
merged 1 commit into from
Jun 29, 2023

Conversation

happz
Copy link
Collaborator

@happz happz commented Apr 28, 2023

The default behavior is to let the exception bubble up and quit. The patch adds an option to change the behavior, the exception is logged and the next plan in line is processed.

@happz happz added the run label Apr 28, 2023
@happz happz added this to the 1.24 milestone Apr 28, 2023
tmt/base.py Outdated Show resolved Hide resolved
@happz happz marked this pull request as ready for review May 26, 2023 13:20
@happz happz requested review from psss and thrix as code owners May 26, 2023 13:20
@happz happz force-pushed the plan-on-error-continue branch 2 times, most recently from 88076ec to 3c29873 Compare June 6, 2023 10:11
@happz happz modified the milestones: 1.24, 1.25 Jun 6, 2023
@happz happz force-pushed the plan-on-error-continue branch 2 times, most recently from c3bfd56 to 27212f7 Compare June 16, 2023 11:09
Copy link
Collaborator

@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.

Nice improvement! Thanks for implementing this.

The default behavior is to let the exception bubble up and quit. The
patch adds an option to change the behavior, the exception is logged and
the next plan in line is processed.
@psss psss self-assigned this Jun 29, 2023
@psss psss added the plans label Jun 29, 2023
@psss psss merged commit dbc69b8 into main Jun 29, 2023
16 checks passed
@psss psss deleted the plan-on-error-continue branch June 29, 2023 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants