Use send instead of sendv on the systemd.journal (fixes #9886)

[lmacken]

On systemd-216-13.fc21 with ansible-1.8.2-1.fc21 the simple example from #9886 is triggering journald assertions. For some reason the ansible module arguments being passed into journal.sendv are causing the problem, but passing them into journal.send as keyword arguments seems to work around the issue.

[lmacken]

Using journal.send instead of sendv also has the added benefit of systemd automatically populating the CODE_FILE, CODE_LINE, and CODE_FUNC fields for us.

[lmacken]

I also filed a bug against systemd https://bugzilla.redhat.com/1177184

[ghost]

ooh that'd be super cool if ansible had this fix to populate those lines!

[abadger]

Looks good. Straightforward switching of which API function we use.

Merged

Hi!

This has been merged in, and will also be included in the next major release.

If you or anyone else has any further questions, please let us know by stopping by one of the two mailing lists, as appropriate:

• https://groups.google.com/forum/#!forum/ansible-project - for user questions, tips, and tricks
• https://groups.google.com/forum/#!forum/ansible-devel - for strategy, future planning, and questions about writing code

Because this project is very active, we're unlikely to see comments made on closed tickets, but the mailing list is a great way to ask questions, or post if you don't think this particular
issue is resolved.

Thank you!

[lmacken]

Thanks Toshio! 🍻

[abadger]

Applied change to the v2 tree as well: 2a04663

Thanks lmacken!