-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add fully patch system ansible playbook module #175
Add fully patch system ansible playbook module #175
Conversation
Fully patch system before test start on public cloud qesap-deployment test cases TEAM-8437 - [qesap-deployment] Fully patch system before test start
015ac0f
to
69ce9ec
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
Please add a check if there is a patch for zypper itself. In that case zypper patch will only update zypper and return 103 (ZYPPER_EXIT_INF_RESTART_NEEDED) |
I think the playbook handles this I see this in the log: |
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 |
Add fully patch system before test start on public cloud qe-sap-deployment
Related ticket:
TEAM-8437 - [qesap-deployment] Fully patch system before test start
VRs: (see deploy_qesap_ansible-qesap_exec_ansible.log.txt file for the execution of this new added playbook)
https://openqaworker15.qa.suse.cz/tests/229811 (passed, (02:02 hours))
http://openqaworker15.qa.suse.cz/t229807 (Command "sudo SAPHanaSR-showAttr" passed)
(The "Fenced database 'vmhana01' is not offline" failure is a known issue)
https://openqaworker15.qa.suse.cz/tests/229815 (passed)