Skip to content
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 files for LTSS in qesap regression #20624

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

mpagot
Copy link
Contributor

@mpagot mpagot commented Nov 13, 2024

Add conf.yaml and setting to manage LTSS in qesap regression for Azure.

Verification run:

BYOS

sle-12-SP5-Qesap-Azure-Byos-x86_64-BuildLATEST_AZURE_SLE12_5_BYOS-qesap_azure_saptune_test
http://openqaworker15.qa.suse.cz/tests/302738

With QESAP_CONFIG_FILE=qesap_azure_ltss.yaml SCC_REGCODE_LTSS=****** QESAPDEPLOY_SCC_LTSS_MODULE='SLES-LTSS/12.5/x86_64'

http://openqaworker15.qa.suse.cz/tests/302739 🟢 --> this log is particularly important http://openqaworker15.qa.suse.cz/tests/302739/logfile?filename=deploy-ansible.registration.log.txt

PAYG

sle-12-SP5-Qesap-Azure-Payg-x86_64-BuildLATEST_AZURE_SLE12_5_PAYG-qesap_azure_saptune_test With QESAP_CONFIG_FILE=qesap_azure_ltss.yaml SCC_REGCODE_LTSS=****** QESAPDEPLOY_SCC_LTSS_MODULE='SLES-LTSS/12.5/x86_64'

Add conf.yaml and setting to manage LTSS in qesap regression for Azure.
Copy link
Contributor

@alvarocarvajald alvarocarvajald left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@lilyeyes lilyeyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

# diff qesap_azure_ltss.yaml qesap_azure.yaml 
37c37
<     - %REGISTRATION_PLAYBOOK%.yaml -e reg_code='%SCC_REGCODE_SLES4SAP%' -e email_address='testing@suse.com' -e sles_modules='[{"key":"%SCC_LTSS_MODULE%","value":"%SCC_LTSS_REGCODE%"}]'
---
>     - %REGISTRATION_PLAYBOOK%.yaml -e reg_code='%SCC_REGCODE_SLES4SAP%' -e email_address='testing@suse.com'

Is it possible to merge qesap_azure_ltss.yaml and qesap_azure.yaml together?

@mpagot
Copy link
Contributor Author

mpagot commented Nov 14, 2024

LGTM.

# diff qesap_azure_ltss.yaml qesap_azure.yaml 
37c37
<     - %REGISTRATION_PLAYBOOK%.yaml -e reg_code='%SCC_REGCODE_SLES4SAP%' -e email_address='testing@suse.com' -e sles_modules='[{"key":"%SCC_LTSS_MODULE%","value":"%SCC_LTSS_REGCODE%"}]'
---
>     - %REGISTRATION_PLAYBOOK%.yaml -e reg_code='%SCC_REGCODE_SLES4SAP%' -e email_address='testing@suse.com'

Is it possible to merge qesap_azure_ltss.yaml and qesap_azure.yaml together?

I do not want to keep the perl code simpler.
For HanaSR the perl code to dynamically compose that part of the conf.yaml is already in place so for HanaSR I'll not duplicate the conf.yaml.

@mpagot mpagot merged commit a9fed7a into os-autoinst:master Nov 14, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants