-
Notifications
You must be signed in to change notification settings - Fork 122
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
Support Fedora Image Mode #3229
base: main
Are you sure you want to change the base?
Conversation
With tmt from this MR: teemtee/tmt#3229 Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
8c1c5db
to
b447a6a
Compare
tmt/steps/execute/__init__.py
Outdated
@@ -58,6 +59,9 @@ | |||
# Scripts source directory | |||
SCRIPTS_SRC_DIR = tmt.utils.resource_files('steps/execute/scripts') | |||
|
|||
# Scripts destination directory, hardcoded in the tmt.sh script | |||
SCRIPTS_DEST_DIR = "/var/tmp/tmt/bin" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a path, it shall be a Path
instance.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And I'd add a way to override this path right from the beginning, check out effective_pidfile_root()
, but that would conflict with easy hard-coding the path into tmt.sh
... Still, hard-coded paths turned out to not be generally usable, sooner or later, might be a good investment to get this one configurable while we're not [ushed by a looming regression...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. Let's make it configurable.
e48ac2b
to
c8115ad
Compare
Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
Pull Request Checklist