implement internalTrafficPolicy for daemonsets

[Joerit]

Hi 👋

we're using fluentbit as a logstash dropin (i.e. tcp input, output to ES)

deploying as a daemonset with a service causes applications to just push to a random pod running on a random node

internalTrafficPolicy makes the service route to the instance running on the local node (so avoids cross-node traffic)

this change works for my usecase.

Greetings,
Joeri

[stevehipwell]

Thanks for the PR @Joerit. I agree that this is a desirable capability. I've added a couple of suggestions, and to get this merged you'd also need to bump the chart version and update the changelog annotations.

[Joerit]

merged latest main in, and bumped the chart version + annotation

[stevehipwell]

@Joerit could you sign the DCO?

[Joerit]

@stevehipwell i have
whatever automated process you use to check this seems to be unhappy i signed off with my actuall full name

Summary Author: Joeri, Committer: Joeri; Expected "Joeri [[email protected]](mailto:[email protected])", but got "Joeri Temmerman [[email protected]](mailto:[email protected])".

[stevehipwell]

@Joerit did you amend all of your commits to be signed off? My suggestion would be to squash your commits and make sure the resulting commit has the correct sign off.

[Joerit]

@stevehipwell
squashed and forced it up.
that should do it then?

[stevehipwell]

@Joerit the DCO looks good now, but you're out of date with the main branch. Could you rebase your branch please (personally I'd suggest clicking the drop-down in the UI and choosing a rebase)?

[Joerit]

@stevehipwell sorry, rebased.

[stevehipwell]

@Joerit it looks like the chart tests are failing due to an implementation detail, #466 should fix this and then if you rebase your PR should be able to be merged.

[stevehipwell]

@Joerit if you can rebase your automation should work correctly.

[stevehipwell]

LGTM