sysctl_file required?

[landam]

When playing with ansible pg tune I got following error:

fatal: [gislab_vagrant]: FAILED! => {"changed": false, "failed": true,
"module_stderr": "Shared connection to 127.0.0.1 closed.\r\n",
"module_stdout": "Traceback (most recent call last):\r\n  File
\"/tmp/ansible_p6EVdJ/ansible_module_postgresql_tune.py\", line 374,
in <module>\r\n    main()\r\n  File
\"/tmp/ansible_p6EVdJ/ansible_module_postgresql_tune.py\", line 369,
in main\r\n    has_changed, config = tune(module.params)\r\n  File
\"/tmp/ansible_p6EVdJ/ansible_module_postgresql_tune.py\", line 332,
in tune\r\n    with open(data[\"sysctl_file\"], 'w') as
confile:\r\nTypeError: coercing to Unicode: need string or buffer,
NoneType found\r\n", "msg": "MODULE FAILURE", "rc": 0}

In documentation (1) is set sysctl_file param only for PG 9.1 (but, even in the vars, version 9.6 is mentioned). The ansible playbook was run against PG 9.5. It's not clear when sysctl_file is required (2).

See probably related #5.

(1) https://github.com/imincik/ansible-postgresql_tune#postgres-91-with-changes-to-kernel-shm-settings-and-setting-a-max_connections
(2) https://github.com/imincik/ansible-postgresql_tune#postgresql-96