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
Ssh keygen trials #78655
Ssh keygen trials #78655
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
you still need a changelog and it would be a good idea to add tests |
lib/ansible/modules/user.py
Outdated
@@ -3078,7 +3095,8 @@ def main(): | |||
bits=0, | |||
type='rsa', | |||
passphrase=None, | |||
comment='ansible-generated on %s' % socket.gethostname() | |||
comment='ansible-generated on %s' % socket.gethostname(), | |||
trials=100 |
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.
if we are changing the name to rounds, i would try to keep it consistent everywhere, it can be very confusing to the next person that has to debug that trials == rounds
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.
Fixed it
Great idea. I would have needed the feature before. How do you proceed with the PR now? Is there anything missing in the implementation? @bcoca @mkrizek @crasyhorse |
SUMMARY
Suggest to Add ssh-keygen -a (Trials) Option. Fixes #78473
ISSUE TYPE
COMPONENT NAME
user