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

Run extension registration independently by the initial main image registration status #293

Merged
merged 1 commit into from
Nov 14, 2024

Conversation

mpagot
Copy link
Collaborator

@mpagot mpagot commented Nov 14, 2024

Both update registercloudguest and enable requested additional extensions independently by the initial registration state: so both for BYOS and PAYG. It is possible as at the playbook point that enable extensions, the image is already registered.
Fix a couple of lint errors and improve the inline documentation.

Related ticket: https://jira.suse.com/browse/TEAM-9808

Verification

qesap regression

Both VR are using os-autoinst/os-autoinst-distri-opensuse#20624 and are triggered with

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

BYOS

Highlights:

  • new task [Check for SUSEConnect binary presence]
  • task TASK [Ensure cloud-regionsrv-client is on latest version.] is executed
  • LTSS enabled in TASK [Add additional authenticated modules]

BYOS with LTSS-ES

sle-12-SP5-Qesap-Azure-Byos-x86_64-Buildmpagot_VR-qesap_azure_saptune_test

PAYG

Important file is http://openqaworker15.qa.suse.cz/tests/302830/logfile?filename=deploy-ansible.registration.log.txt
Highlights:

  • Fails in Add additional authenticated modules . It could be external issue but at least it is wrongly trying registering LTSS with SUSEConnect in place of registercloudguest.
 "SUSEConnect -p SLES-LTSS/12.5/x86_64 -r *********"

HanaSR

All of them without LTSS

BYOS

sle-15-SP6-HanaSr-Aws-Byos-x86_64-Build15-SP6_2024-11-14T03:03:15Z-hanasr_aws_test_fencing_sbd_crash ec2_r4.8xlarge

-> http://openqaworker15.qa.suse.cz/tests/302831 🟢

sle-15-SP6-HanaSr-Azure-Byos-x86_64-Build15-SP6_2024-11-14T03:03:15Z-hanasr_azure_test_saptune_msi az_Standard_E4s_v3

sle-15-SP6-HanaSr-Gcp-Byos-x86_64-Build15-SP6_2024-11-14T03:03:15Z-hanasr_gcp_test_fencing_sbd_crash gce_n1_highmem_8

-> http://openqaworker15.qa.suse.cz/tests/302833 🟢 fails later in Crash_site_a-primary

sle-12-SP5-HanaSr-Azure-Byos-x86_64-Build12-SP5_2024-11-14T03:03:15Z-hanasr_azure_test_msi az_Standard_E4s_v3

-> http://openqaworker15.qa.suse.cz/tests/302835

PAYG

…gistration status

Both update registercloudguest and enable requested additional
extensions independently by the initial registration state: so both for
BYOS and PAYG. It is possible as at the playbook point that enable
extensions, the image is already registered.
Fix a couple of lint errors and improve the inline documentation.
Copy link
Collaborator

@BillAnastasiadis BillAnastasiadis 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
Collaborator

@lpalovsky lpalovsky 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
Collaborator

@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

@mpagot mpagot merged commit 694343f into SUSE:main Nov 14, 2024
1 check 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.

4 participants