quidame

@satken2 , you removed integration tests in your last commit (https://github.com/ansible-collections/ansible.posix/pull/206/commits/34b467719e0dab10d062813cdef11b49f3925ed6), whereas they're more than welcome. Could you please revert this and just fix the last `assert` (missing quotes around...

> Is there a way to solve this problem with `add_file_common_args=True`? I would appreciate it if you could give me some advice :) Yes, there is :) 1. Add this...

@saito-hideki I think CI failures are not really related to this PR. It seems there is some issue somewhere else... In the failed tests on Docker, all tasks in the...

> Also, added needs/privileged to tests/integration/targets/sysctl/aliases file to avoid "Read-only file system" error during integration tests on opensuse15. Now CI tests succeeded without errors. Now CI tests succeeded without error...

It would probably be clearer to separate *configuration* and *running state*, and split them into two module parameters, the same way it is done for `service` or `systemd` modules, with...

A part of the trouble with the `state` option of this module, is that two of its values describe (or lead to) the same state, but depend on the initial...

Hi @satken2, thanks for your contribution ! IMHO, this PR doesn't fix #163, as said in its description: the new parameter only applies to the mountpoint itself, not its parents,...

> In #163, author requests to add `umask` option but when the mount point does not exist, this module creates directories, not files. So I think `mode` is more appropriate...

> The `mount` module only creates directories, so I don't think it is necessary to consider creating files, I guess. > So I added `mode` option. There is no need...

@bmillemathias is this work still in progress ?