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
Fix incorrect command-line parameters for Solaris #22865
Fix incorrect command-line parameters for Solaris #22865
Conversation
cc @jtyr would you mind reviewing 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.
After the requested change is in place I fine it to be merged. Unfortunately I cannot verify whether Solaris doesn't really need this option. I guess we must trust the usere here.
lib/ansible/modules/system/mount.py
Outdated
@@ -327,7 +327,7 @@ def unset_mount(module, args): | |||
def _set_fstab_args(fstab_file): | |||
result = [] | |||
|
|||
if fstab_file and fstab_file != '/etc/fstab': | |||
if fstab_file and fstab_file != '/etc/fstab' and get_platform().lower() != 'sunos': |
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.
Please could you change it to:
if (
fstab_file and
fstab_file != '/etc/fstab' and
get_platform().lower() != 'sunos'):
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.
Not sure if I did it correct, but it's updated according your request.
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.
It looks good!
rebuild_merge |
@jtyr Any other changes requested from you? |
Nop. I think if that works for you on Solaris then we can merge it now. @bcoca can you take care of it? |
* Fix incorrect command-line parameters for Solaris * Update mount.py
SUMMARY
Solaris 'mount' command does not have flag to point to fstab file.
So no such flags should be passed to the command.
Fixes #22135
ISSUE TYPE
COMPONENT NAME
mount module
ANSIBLE VERSION
ADDITIONAL INFORMATION