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

Test registration role use

Open BillAnastasiadis opened this issue 1 year ago • 2 comments

This PR adds conditional clauses for the use of the registration role from Steven's repo () in qe-sap-deployment. See https://github.com/SUSE/qe-sap-deployment/pull/164 for details.

  • Related ticket: https://jira.suse.com/browse/TEAM-8142
  • Verification runs:

run double vrs with+without USE_ROLES openqa var for hanasr regression and qesap_regression: for qesap_regression the repo was downloaded, but since the ansible create section is hardcoded in the config file and not dynamically created, the old registration.yaml is used, so both VRs are identical and have the same behaviour as before. for hanasr regression, with USE_ROLES the new registration role is used, and without USE_ROLES the behaviour is the same as it was before. qesap regression:

  • with USE_ROLES: http://openqaworker15.qa.suse.cz/tests/201365
  • without: http://openqaworker15.qa.suse.cz/tests/201366 (same behavior for both) hanasr regression:
  • with USE_ROLES: https://openqaworker15.qa.suse.cz/tests/201367 (the registration role from steven's repo is used)
  • without: http://openqaworker15.qa.suse.cz/tests/201364 (our old registration.yaml task-based playbook is used)

BillAnastasiadis avatar Jul 12 '23 11:07 BillAnastasiadis

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 Jul 12 '23 11:07 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. If it's still needed, you can add labels WIP or notready to it

stale[bot] avatar Dec 15 '23 07:12 stale[bot]