-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
salt-cloud unable to create proxmox profile #25652
Comments
@niq000, thanks for the report. |
@niq000 Thanks for the pull request. Does that fix resolve this issue for you? |
I installed a wildcard certificate on my proxmox server, and still wasn't able to get things working when salt-cloud was verifying ssl. I'm not sure if that is an issue with my configuration, or something else. In regards to the pull request, I was able to successfully get things working with the changes I made. In the little bit of research I did, it looks like salt cloud never used to verify ssl for proxmox (which is why i never had any problems in the past), but then that was changed with this commit (860d4b7). All I changed in the code was give an option in the proxmox provider file to give the user the choice to verify SSL or not instead of having it hard-coded on or off. |
Best security practices dictate that SSL be the default where ever it is available. Making it optional is acceptable, so long as the default is still to have it on. |
@techhat the default is to verify ssl, even if it's not defined in the provider config file, but now the user has a choice to disable the verification by adding "verify_ssl: False" in the proxmox provider config file |
Thanks for the clarifications and for the fix! |
I'm not able to create a new openVZ container via proxmox. I should mention that in the past (one or two salt versions ago, I was able to with the same provider and profiles).
/etc/salt/cloud.providers.d/proxmox.conf
/etc/salt/cloud.profiles.d/proxmox.conf
Output:
The text was updated successfully, but these errors were encountered: