Add engine for fluent

[mchugh19]

What does this PR do?

Adds new engine to export events to fluentd

What issues does this PR fix or reference?

None

Tests written?

No

Commits signed with GPG?

No

[garethgreenaway]

@mchugh19 Looks like there are some lint failures and a merge conflict.

[mchugh19]

Fixed the junky commit. Should be lint clean now.

[twangboy]

@mchugh19 Hey, thanks for the PR! Could we get some tests for this?

[mchugh19]

Could we get some tests for this?

Having a bit of trouble here. The other engine test examples are the sqs engine which validates sqs messages can be resent on the salt event bus, while the slack tests validate the complex configuration. For the fluent engine, it really is just:

while True:
    salt_event = event_bus.get_event_block()
    if salt_event:
        event.Event(app, salt_event)

sender.close()

where I'm struggling to come up with a meaningful test. As long as the fluent event module is functional, and the event_bus.get_event_block() works as expected, there isn't much else to validate here.

Thoughts?

[twangboy]

@mchugh19 Looks like we have some lint failures here.

fluent.py:95, PyLint, Priority: Low
--
Trailing whitespace

[mchugh19]

github editor created lint issue fixed :)

[mchugh19]

Bump. I think this is all set

[mchugh19]

Can this make it in for Neon?

[waynew]

Looks like got out of date again... I'm on my phone right now but if I'm reading it correctly it's been approved, so with an update we should be able to get this in... As soon as it's all up to date and passing 😝

[waynew]

Well, let's update this again 😛

We've also been having some struggles with our CI hitting the GitHub rate limits (good growing pains??) but I think we've got that sorted. I'll keep an eye on this throughout the day for the builds to finish 🤞

[dwoz]

@mchugh19 can you rebase this PR to master and update the base branch please?

[mchugh19]

Closing this out for the master branch based #55711