Skip to content

Add playbook to prepare seed SNO before seeed gen#544

Merged
shaior merged 1 commit into
openshift-kni:mainfrom
shaior:prepare_ibu_seed
Jun 24, 2026
Merged

Add playbook to prepare seed SNO before seeed gen#544
shaior merged 1 commit into
openshift-kni:mainfrom
shaior:prepare_ibu_seed

Conversation

@shaior

@shaior shaior commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@openshift-ci openshift-ci Bot requested review from ccardenosa and rdiscala June 24, 2026 08:21
@shaior shaior force-pushed the prepare_ibu_seed branch from a5cb656 to 6af8d6e Compare June 24, 2026 08:29

@kononovn kononovn left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please check comments

Comment thread playbooks/ran/ibu-poweroff-seed-spoke.yml Outdated
Comment thread playbooks/ran/ibu-poweroff-seed-spoke.yml Outdated
Comment thread playbooks/ran/ibu-poweroff-seed-spoke.yml Outdated
Comment thread playbooks/ran/prepare-ibu-seed-sno.yml Outdated
Comment thread playbooks/ran/prepare-ibu-seed-sno.yml
changed_when: false
become: true

- name: Attach containers disk if not already attached

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What is containers disk? This operation is dangerous because it affect hypervisor which is shared between different CI setups

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This is only for dev process, later spokes will be switched to BMs and this will be removed.

Comment thread playbooks/ran/ibu-poweroff-seed-spoke.yml Outdated
Comment on lines +31 to +40
- name: Power off seed spoke node via SSH
ansible.builtin.command:
cmd: >
ssh -o StrictHostKeyChecking=no
-o ConnectTimeout=10
core@{{ spoke_node_ip.stdout }}
sudo systemctl poweroff
async: 15
poll: 0
changed_when: true

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

not mandatory - but you can use the inventory to shutdown the host / VM via redfish
it will be more reliable and faster

    - name: Force power off node via Redfish
      community.general.redfish_command:
        category: Systems
        command: PowerForceOff
        baseuri: "{{ hostvars[item].bmc_address }}"
        username: "{{ hostvars[item].bmc_user }}"
        password: "{{ hostvars[item].bmc_password }}"

@openshift-ci

openshift-ci Bot commented Jun 24, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from eifrach. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@shaior shaior force-pushed the prepare_ibu_seed branch from 26ca204 to 37f3a0e Compare June 24, 2026 09:26
@eifrach

eifrach commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 24, 2026
@shaior shaior merged commit c90af15 into openshift-kni:main Jun 24, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants