nasuni-labs / nasuni-edgeappliance-automation

Scripts for automating the deployment of Nasuni Edge Appliances
MIT License
2 stars 0 forks source link

nasuni-edge-appliance-automation script error #5

Closed agrancea closed 2 years ago

agrancea commented 2 years ago

nasuni-edge-appliance-automation.ps1 has a error from the start

Thanks, Adrian

screenshot r

cdillahunty-nasuni commented 2 years ago

Hello Adrian,

I'm sorry for the trouble. I wasn't able to reproduce the issue--PowerShell prompted me for the path to the script if I didn't reference variables.ps1:

PS C:\Users\Administrator> C:\scripts\AutodeployEA.ps1
Please enter the path to the variables input file:  

Can you run the following command so that I can see your PowerShell version?

Get-Host | Select-Object Version

agrancea commented 2 years ago

Hello Coan,

Version

5.1.19041.1320

cdillahunty-nasuni commented 2 years ago

Hello Adrian,

The PowerShell version could be the issue. We built them and tested them to support PowerShell 6 or higher. The current PowerShell version is 7.2.2.

agrancea commented 2 years ago

Hello Coan, It works like a charm in ver 7.2.2. One suggestion for improvement, prompt for credentials rather than leave them on variables file, our security team won't like admin credentials in clear text

Thanks for your help, Adrian

alexanderwillacker commented 2 years ago

...I made a wrapper around the powershell script which checks for version and retrieves credentials then I call the script.

agrancea commented 2 years ago

Thank you for all your help

cdillahunty-nasuni commented 2 years ago

closing this issue since it's resolved.