Skip to content

Fix finish/shell scripts never running in image mode - #5125

Merged
happz merged 2 commits into
mainfrom
fvagner-image-mode
Aug 20, 2026
Merged

Fix finish/shell scripts never running in image mode#5125
happz merged 2 commits into
mainfrom
fvagner-image-mode

Conversation

@therazix

Copy link
Copy Markdown
Contributor

This PR fixes finish/shell scripts being skipped on image-mode guests. Finish scripts now run immediately, prepare/shell is unchanged.

Fixes #4917

Pull Request Checklist

  • implement the feature
  • include a release note

@therazix therazix added this to the 1.78 milestone Aug 17, 2026
@therazix therazix added step | finish Stuff related to the finish step ci | full test Pull request is ready for the full test execution labels Aug 17, 2026
@github-project-automation github-project-automation Bot moved this to backlog in planning Aug 17, 2026
@therazix therazix moved this from backlog to review in planning Aug 17, 2026
Comment thread tmt/steps/prepare/shell.py Outdated
Comment thread tmt/guest/__init__.py

@thrix thrix left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, no commens from me. Thanks for fixing this!

@thrix
thrix requested a review from happz August 18, 2026 11:11
@therazix
therazix force-pushed the fvagner-image-mode branch 2 times, most recently from 0c5ee89 to 709214b Compare August 18, 2026 14:30
@LecrisUT

Copy link
Copy Markdown
Member

/packit build

@tcornell-bus tcornell-bus moved this from review to merge in planning Aug 19, 2026
@happz
happz merged commit c8f36c6 into main Aug 20, 2026
35 checks passed
@github-project-automation github-project-automation Bot moved this from merge to done in planning Aug 20, 2026
@happz
happz deleted the fvagner-image-mode branch August 20, 2026 16:09
tcornell-bus pushed a commit to tcornell-bus/tmt that referenced this pull request Aug 21, 2026
This PR fixes `finish/shell` scripts being skipped on image-mode guests.
Finish scripts now run immediately, `prepare/shell` is unchanged.

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

Labels

area | image mode ci | full test Pull request is ready for the full test execution step | finish Stuff related to the finish step

Projects

Status: done

Development

Successfully merging this pull request may close these issues.

In image-mode all shell finish steps have "Command collected for deferred execution"

5 participants