Allow one to override the hardcoded value "ansible_syslogger" in the syslogger module.

[mivanov-its]

SUMMARY

Copied from ansible/ansible#69428.

It is useful to allow one to specify the application name in the syslogger module. It is hardcoded to "ansible_syslogger" at the moment, see:

https://github.com/ansible-collections/community.general/blob/master/plugins/modules/notification/syslogger.py#L145
https://github.com/ansible-collections/community.general/blob/master/plugins/modules/notification/syslogger.py#L149

Moreover, this hardcoded string is not mentioned in the documentation:

https://docs.ansible.com/ansible/latest/modules/syslogger_module.html

ISSUE TYPE

• Feature Idea

COMPONENT NAME

plugins/modules/notification/syslogger.py

ADDITIONAL INFORMATION

One could override it with:

- name: Test syslog
  syslogger:
    appname: "myapp"
    msg: "Hello from ansible"
    priority: "err"
    facility: "daemon"
    log_pid: true

or in some other way.

[ansibullbot]

Files identified in the description:

• plugins/modules/notification/syslogger.py

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

[ansibullbot]

cc @garbled1
click here for bot help

[Akasurde]

resolved_by_pr #340

[Akasurde]

@mivanov-itsonix, Could you please check PR #340 and let us know if it works for you ?

Please check How to test a PR for more help.

Thanks.

needs_info

[mivanov-its]

@Akasurde , that works for me, thank you for your time.

PS:

You can remove the alias "appname", I do not insist on that as it was just an example. Ident works fine for me.

[Akasurde]

@mivanov-itsonix Thanks for testing and providing the feedback. I implemented the required changes.