os-autoinst-distri-opensuse
os-autoinst-distri-opensuse copied to clipboard
Test registration role use
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)
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.
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