os-autoinst-distri-opensuse icon indicating copy to clipboard operation
os-autoinst-distri-opensuse copied to clipboard

Add argument to customize qesap log filename

Open mpagot opened this issue 1 year ago • 1 comments

Add argument to qesap_execute about the log filename. Use it in qesap regression Ansible and Terraform retry. Add verbosity args with no verbose default in qesap_ansible_script_output_file. Improve debug message in peering delete function.

  • Related ticket:

Verification run:

All VR with a simulated failure use https://github.com/SUSE/qe-sap-deployment/pull/184

No failure

http://openqaworker15.qa.suse.cz/tests/248633 No qesap.py failure this code version

Simulate non zero rc but without special error messages

http://openqaworker15.qa.suse.cz/tests/248634 QESAP_SIM_RC=42 configured after qesap.py ...configure and before qesap.py ... terraform. Job just die, as expected no retry is attempted. :green_circle:

http://openqaworker15.qa.suse.cz/tests/248638 QESAP_SIM_RC=42 configured after qesap.py ... terraform and before qesap.py ... ansible. Terraform runs properly, Ansible is simulated and never executed. From the point of view of the test, qesap.py ... ansible return non 0 exit code. Code to look for special Ansible message in the log is executed and does not find anything (as expected), code execution falls here :green_circle: Failure is due to the fact that qesap_cluster_logs() is using Ansible to get logs from the cluster and the failure simulation also prevent initial ssh key shearing to be executed.

Simulate non zero rc with timeout error message

  • sle-15-SP5-Qesap-Azure-Byos-x86_64-BuildLATEST_AZURE_SLE15_5_BYOS-qesap_azure_saptune_test@64bit -> http://openqaworker15.qa.suse.cz/tests/248684 Timeout error properly detected in the Ansible log http://openqaworker15.qa.suse.cz/tests/248684#step/deploy/47 :green_circle:

Simulate zero rc with timeout error message

Corner case that should never happens

mpagot avatar Oct 17 '23 13:10 mpagot

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • [ ] Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

github-actions[bot] avatar Oct 17 '23 13:10 github-actions[bot]