-
Notifications
You must be signed in to change notification settings - Fork 106
fix: always apply hardware configuration #561
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
Conversation
f86aa22 to
dd7e68c
Compare
dd7e68c to
bfdf005
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes hardware configuration to always run even when the hardware config is empty, ensuring template settings are preserved during VM provisioning.
- Removes the check that skipped hardware configuration when config was empty
- Adds a helper method to determine if custom hardware config is provided to display appropriate UI messages
- Updates test to verify hardware configuration runs even with empty config
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| builder/vsphere/common/step_hardware.go | Implements logic to always run hardware configuration and adds helper method to detect custom config |
| builder/vsphere/common/step_hardware_test.go | Updates test case to verify hardware configuration runs with empty config |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
bfdf005 to
31d14bd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
31d14bd to
a66fef3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
a66fef3 to
d492bc5
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
d492bc5 to
1de6426
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
- Only set non-zero or enabled fields, preventing unnecessary changes. - Always applies the configuration to ensure template settings are preserved. Signed-off-by: Ryan Johnson <[email protected]>
1de6426 to
2a256db
Compare
Description
Resolved Issues
Closes #522
Rollback Plan
Revert commit.
Changes to Security Controls
None.