-
Notifications
You must be signed in to change notification settings - Fork 271
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
foreman::cli::manage_root_config doesn't work because "replace => false" #813
Comments
The reason we do this is that in the foreman class we set the initial username and password, but have no guarantee that they remain the same. I know this is ugly and at a parameter for this would make sense. |
Maybe a rework of parameters, behavior, etc. of CLI setup could be a good way.
I'm OK to rework this part, if someone purpose a global approach. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
foreman::cli
class is configured not to replace/root/.hammer/cli.modules.d/foreman.yml
, even if$manage_root_config
is set to true, becausereplace => false
:I think this is confusing and contradictory; I know I was confused about why it wasn't working until I took a closer look at the module. I think if
$manage_root_config
is set to true then this file should automatically be updated with the specified config.The text was updated successfully, but these errors were encountered: