-
Notifications
You must be signed in to change notification settings - Fork 278
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
Cleanup logic improvements #19829
Cleanup logic improvements #19829
Conversation
83f2ef2
to
f59c7c0
Compare
003c60d
to
c0e2d46
Compare
@@ -9,48 +9,28 @@ | |||
# Post run hooks are generally disabled during normal module run so the infrastructure persists between test modules. | |||
# Cleanup is triggered only with B<SDAF_DO_CLEANUP> set to true, which is done by scheduling this module at the end of test flow. | |||
|
|||
use parent 'sles4sap::sap_deployment_automation_framework::basetest'; |
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.
This is probably not crucial, but it might remove some unnecessary inheritance.
b4873c5
to
89ac82d
Compare
@@ -142,7 +142,7 @@ sub az_login { | |||
assert_script_run("source $temp_file"); | |||
|
|||
my $login_cmd = 'while ! az login --service-principal -u ${ARM_CLIENT_ID} -p ${ARM_CLIENT_SECRET} -t ${ARM_TENANT_ID}; do sleep 10; done'; | |||
assert_script_run($login_cmd, timeout => 5 * 60); | |||
assert_script_run($login_cmd, timeout => 30); |
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.
5*60 for login is just too much.
89ac82d
to
79fe840
Compare
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.
LGTM
LGTM, thanks |
This PR adds improvements for SDAF cleanup routine:
snapshot based deployer VM cleanup
cleanup behavior at different stages of test failures
removing redundant cleanup code in post fail hook and standard cleanup test module
Related ticket: https://jira.suse.com/browse/TEAM-9459
Verification runs
Successful run:
https://openqaworker15.qa.suse.cz/tests/294657#
Failure before SDAF deployment starts:
Failure before deployer VM is created: