Move set_pxe_boot utility from utils to orchestrator domain - #5037
Merged
sujit-jadhav merged 3 commits intoAug 27, 2026
Conversation
Migrates PXE boot roles (collect_pxe_credentials, idrac_pxe_boot, verify_phone_home), playbook, and config into orchestrator under playbooks/setpxe/. Updates orchestrator.yml workflow, adds dellemc.openmanage dependency, and cleans all PXE references from utils domain docs and playbooks. Signed-off-by: venu <236371043+Venu-p1@users.noreply.github.com>
Signed-off-by: venu <236371043+Venu-p1@users.noreply.github.com>
- Replace inventory-based input with pxe_mapping_file.csv - Leverage orchestrator credential store (bmc_username/password) - Tag-based execution: ansible-playbook orchestrator.yml --tags setpxe - Remove enable_pxe_boot config flag requirement - Add README with usage examples and ansible.cfg Signed-off-by: venu <236371043+Venu-p1@users.noreply.github.com>
Venu-p1
requested review from
abhishek-sa1 and
sujit-jadhav
and removed request for
sujit-jadhav
August 26, 2026 03:57
abhishek-sa1
approved these changes
Aug 27, 2026
Rajeshkumar-s2
approved these changes
Aug 27, 2026
sujit-jadhav
approved these changes
Aug 27, 2026
sujit-jadhav
merged commit Aug 27, 2026
0be3dad
into
dell:issue-4849-omnia-modernization
13 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Migrated 3 PXE boot roles (collect_pxe_credentials, idrac_pxe_boot, verify_phone_home) from roles to roles
Created setpxe directory with adapted set_pxe_boot.yml playbook
Updated orchestrator.yml to import setpxe/set_pxe_boot.yml after prepare phase
Moved set_pxe_boot_config.yml to input
Added dellemc.openmanage >= 10.0.3 to orchestrator requirements.yml
Updated role meta/main.yml files to match orchestrator Galaxy conventions
Updated all path references from utils to orchestrator (input/output dirs)
Removed PXE boot playbook, roles, config, and samples from utils domain (clean break)
Updated utils utils.yml entry point to remove PXE boot flow
Updated documentation in both domains (READMEs, architecture, contracts)