-
Notifications
You must be signed in to change notification settings - Fork 130
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
Add support for SSH BMC provider #470
Conversation
Looks like you need to silence the quoted boolean puppet-lint check. Other than that I think this looks good. |
Added ignore:quoted_booleans around poweron and powerstatus commands. |
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.
I think it's correct now. Restarted the failing acceptance test since there were issues in master but that now passes. Waiting for that result now.
It turns out these parameters are incorrect: theforeman/smart-proxy#626. Going to send a PR to fix it before releasing it. |
Since this module is apparently in no wide use yet, I suggest to fix smart-proxy codebase and keep puppet and config as is. Rebased the PR. |
This commit adds support for the SSH BMC provider and all configuration options that come with it. Defaults have been copied from the official foreman docs.