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.

Sign up for GitHub

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
+30 −18
Merged

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

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
48 changes: 30 additions & 18 deletions ansible/playbooks/registration.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
---
- hosts: all
- name: Register
hosts: all
remote_user: cloudadmin
become: true
become_user: root
Expand All @@ -10,6 +11,10 @@
tasks:

# Pre flight checks
- name: Check for SUSEConnect binary presence
ansible.builtin.command: which SUSEConnect
changed_when: false

# Do we have repos? If not, we need to register
- name: Check for registration
ansible.builtin.command: SUSEConnect -s
Expand All @@ -19,28 +24,37 @@

# Check if there are instances of `Not Registered` in it
- name: Check for 'Not Registered'
set_fact:
ansible.builtin.set_fact:
not_registered_found: "{{ 'Not Registered' in repos.stdout }}"
ignore_errors: true

# Is registercloudguest available?
# only run it if:
# - there's at least one 'Not Registered' module
# - the user does not require only use SUSEConnect with `use_suseconnect`
- name: Check for registercloudguest
ansible.builtin.command: which registercloudguest
register: rcg
register: is_registercloudguest_bin
failed_when: false
changed_when: false
when:
- not_registered_found
- not use_suseconnect | bool

# Execute Section
- name: registercloudguest pre-run cleaning

# Start by pre-cleaning all. Only run it if:
# - the registercloudguest binary is available
# - there's at least one 'Not Registered' module
# - the user does not require only use SUSEConnect with 'use_suseconnect'
- name: Pre-run cleaning registercloudguest
ansible.builtin.command: registercloudguest --clean
when:
- not_registered_found
- rcg.rc == 0
- is_registercloudguest_bin.rc == 0
- not use_suseconnect | bool

- name: registercloudguest registration
- name: Run registercloudguest registration
ansible.builtin.command: registercloudguest --force-new -r "{{ reg_code }}" -e "{{ email_address }}"
register: result
until: result is succeeded
Expand All @@ -49,10 +63,13 @@
failed_when: result.rc != 0 or result.stderr != ""
when:
- not_registered_found
- rcg.rc == 0
- is_registercloudguest_bin.rc == 0
- not use_suseconnect | bool

# If registercloudguest is not present fall back on SUSEConnect
# Fall back on SUSEConnect if:
# - registercloudguest is not present
# or
# - the user explicitly require using SUSEConnect
- name: SUSEConnect registration
ansible.builtin.command: SUSEConnect -r "{{ reg_code }}" -e "{{ email_address }}"
register: result
Expand All @@ -61,9 +78,10 @@
delay: 60
when:
- not_registered_found
- "(rcg.rc != 0) or (use_suseconnect | bool)"
- "(is_registercloudguest_bin.rc != 0) or (use_suseconnect | bool)"

# There are additional repos to add. These are handled differently for SLES 15 and SLES12
# There are additional repos to add.
# These are handled differently for SLES 15 and SLES12
- name: Add SLES 12 Advanced Systems Modules
ansible.builtin.command: SUSEConnect -p sle-module-adv-systems-management/12/{{ ansible_facts['architecture'] }} -r "{{ reg_code }}"
register: result
Expand All @@ -72,8 +90,6 @@
delay: 60
when:
- ansible_facts['distribution_major_version'] == "12"
- not_registered_found
- "(rcg.rc != 0) or (use_suseconnect | bool)"

- name: Add SLES 12 public cloud module
ansible.builtin.command: SUSEConnect -p sle-module-public-cloud/12/{{ ansible_facts['architecture'] }}
Expand All @@ -84,7 +100,7 @@
when:
- ansible_facts['distribution_major_version'] == "12"
- not_registered_found
- "(rcg.rc != 0) or (use_suseconnect | bool)"
- "(is_registercloudguest_bin.rc != 0) or (use_suseconnect | bool)"

- name: Add SLES 15 public cloud module
ansible.builtin.command: SUSEConnect -p sle-module-public-cloud/{{ ansible_facts['distribution_version'] }}/{{ ansible_facts['architecture'] }}
Expand All @@ -95,7 +111,7 @@
when:
- ansible_facts['distribution_major_version'] == "15"
- not_registered_found
- "(rcg.rc != 0) or (use_suseconnect | bool)"
- "(is_registercloudguest_bin.rc != 0) or (use_suseconnect | bool)"

# Latest version of cloud-regionsrv-client is needed in PAYG, and image
# needs to be registered in order for zypper up to work.
Expand All @@ -105,7 +121,6 @@
name: cloud-regionsrv-client
state: latest
when:
- not not_registered_found
- sles_modules is defined and sles_modules | length > 0

- name: Add additional authenticated modules
Expand All @@ -115,7 +130,6 @@
retries: 10
delay: 60
when:
- not_registered_found
- sles_modules is defined and sles_modules | length > 0
loop: "{{ sles_modules }}"
loop_control:
Expand All @@ -125,5 +139,3 @@
ansible.builtin.command: zypper lr -u
register: repos_after
failed_when: repos_after.rc != 0
when:
- not_registered_found
Loading