-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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 backup feature to user module #41854
Add backup feature to user module #41854
Conversation
lib/ansible/modules/system/user.py
Outdated
@@ -194,6 +194,14 @@ | |||
type: bool | |||
default: 'no' | |||
version_added: "2.4" | |||
backup: |
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 would not even make this an option, specially since it behaves differntly than other 'backup' options, we should just 'do this'
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.
Ok. I'll just add a note then and remove the option.
The test
|
Remove the option to enable/disable backups and make it automatic. Add note to docs describing this behavior. Change tests to account for new module behavior. Change section name in changelog fragment since minor_features is not a valid section.
rebuild_merge |
SUMMARY
Add feature to backup shadow file only on platforms where the module modifies the shadow file directly, which are currently SunOS variants.
Fixes #40696
ISSUE TYPE
COMPONENT NAME
user.py
ANSIBLE VERSION