Description

Standardizes and improves the clarity of log messages throughout the VMware builder codebase by updating the phrasing of log outputs and consistently adding log level prefixes (such as [INFO] and [WARN]). This helps make log output more uniform and easier to interpret, which is beneficial for both debugging and monitoring. Additionally, several test assertions have been updated to use clearer error messages.

• Added [INFO] and [WARN] prefixes to log messages in driver.go, driver_esxi.go, driver_workstation_windows.go, step_configure_vnc.go, step_output_dir.go, and step_remote_upload.go, ensuring consistent log levels and improved readability. [1] [2] [3] [4] [5] [6]
• Updated log message phrasing for error handling to use "Failed to..." instead of "Error..." or "Unable to...", making the messages more direct and standardized. [1] [2] [3] [4] [5] [6] [7]
• Changed error messages in test assertions from "Unable to..." to "Failed to..." for improved clarity and consistency in driver_parser_test.go and hw_config_test.go. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]
• Improved log messages to better describe actions and outcomes, such as device and MAC address discoveries, and cloning/uploading operations. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11]

These changes collectively make the codebase's logging and error reporting more consistent and easier to understand for future maintenance and debugging.

Rollback Plan

Revert commit.

Changes to Security Controls

None.