-
Notifications
You must be signed in to change notification settings - Fork 133
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
refs #3272 - print new user/password after installation #116
Conversation
# Foreman UI? | ||
if module_enabled? 'foreman' | ||
say " * <%= color('Foreman', :info) %> is running at <%= color('#{get_param('foreman','foreman_url')}', :info) %>" | ||
say " Credentials are '<%= color('#{get_param('foreman', 'admin_user')} / #{get_param('foreman', 'admin_password')}', :info) %>'" if get_param('foreman','authentication') == true |
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.
One problem I do see is rerunning the installer to upgrade for example, but these will still only be the default parameters and the user could have changed the password.
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.
That's possible - the rake task is designed not to overwrite or change the admin user if one already exists, so it's actually an initial password. I've added the word "initial", which should clarify.
👍 pending other patch. |
Nice, our installer is pure awesomness. |
Depends on theforeman/puppet-foreman#192 at the moment. |
Tested, works great, 👍 will merge when the core PR goes in. |
PR is merged, merge at will |
Depends on theforeman/puppet-foreman#190.
The change to print the details all the time might be needed if we had a partial failure, but enough that the web UI works - so as not to leave the user with no idea what their credentials are.