Windows Bootstrap Failing on New Installs

[doesitblend]

Description of Issue/Question

The latest version of the Windows bootstrap script checks for a registry key that will not exist if Salt is not already installed. The code producing the error seems to be on line 190 of the script.

Setup

Deploy a fresh Windows VM

Steps to Reproduce Issue

Run the bootstrap script with the options shown in the second image attached

Versions and Systems

Bootstrap version: 2022.03.15

[twangboy]

This probably needs to be wrapped in a Try/Except block...

[doesitblend]

@twangboy One recommendation I received on how to resolve this was the following:

The Get-ItemProperty call probably needs its -ErrorAction set to “SilentlyContinue”.

This may be an alternative to the existing code, but I'm not very familiar with Powershell and its best practices.

[twangboy]

That would probably work too, but the variable would be empty. So, set it first, then SilentlyContinue.

[austinbrowder]

I think the bigger issue here is that this causes problems for windows bootstraping in 3005.1+ because the default location the minion looks for confs is nolonger c:\salt.
https://docs.saltproject.io/salt/install-guide/en/latest/topics/install-by-operating-system/windows.html#install-windows

[dmurphy18]

@doesitblend Can you retry with the latest bootstrap script which has been heavily updated this July, noting that support for older EOL version of Salt has been dropped. If the latest version of the bootstrap script resolves your issues, please consider closing this issue.