-
Notifications
You must be signed in to change notification settings - Fork 119
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
Implement --env for testcloud provisioner #148
Conversation
This pull request introduces 1 alert and fixes 1 when merging cd3fce4 into 83f699d - view on LGTM.com new alerts:
fixed alerts:
|
This pull request introduces 1 alert and fixes 1 when merging e792f45 into 83f699d - view on LGTM.com new alerts:
fixed alerts:
|
This pull request fixes 1 alert when merging eee8835 into 83f699d - view on LGTM.com fixed alerts:
|
eee8835
to
db779a4
Compare
This pull request introduces 1 alert and fixes 1 when merging db779a4 into 50d8a5a - view on LGTM.com new alerts:
fixed alerts:
|
db779a4
to
99836fd
Compare
This pull request introduces 1 alert and fixes 1 when merging 99836fd into a543b3b - view on LGTM.com new alerts:
fixed alerts:
|
Along the way also: * improve a bit how image is detected, so the user can just pass a path, and tmt adds testcloud required prefix 'file://' by itself. * make sure finish runs only if there is an instance provisioned Signed-off-by: Miroslav Vadkerti <mvadkert@redhat.com>
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.
Looks good and works as expected. There was just some wrong indent which caused syntax error. I've fixed that and renamed the shared function to better match what it does.
Along the way also:
improve a bit how image is detected, so the user can just pass a path, and tmt adds testcloud required prefix 'file://' by itself.
make sure finish runs only if there is an instance provisioned
Note: also bundles #132, pls review only the commit with the same name as the description of the PR