Conversation
Refactors the `vsphere-clone` example for Alpine. - Variable ordering. - Missing variable (`datastore`) - Remove unused variables (`guest_os_type`) Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the Alpine vsphere-clone example to make it operational by adding a missing required datastore configuration and improving code organization. The changes include removing an unused guest_os_type variable, reordering variables and source block attributes for better clarity, and adding the previously missing datastore variable and attribute.
Key Changes:
- Added missing
datastorevariable and corresponding attribute in the source block - Removed unused
guest_os_typevariable - Reordered variables and source block attributes for improved organization
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Description
Refactors the
vsphere-cloneexample for Alpine.datastore)guest_os_type)Resolved Issues
Example is not operable with the missing
datastoreattribute.Rollback Plan
Revert commit.
Changes to Security Controls
None.