search

fabric8io / fluent-plugin-kubernetes_metadata_filter

Enrich your fluentd events with Kubernetes metadata
Apache License 2.0
351 stars 166 forks source link

Add 410 Gone handling for namespace watch #247

Closed Ghazgkull closed 4 years ago

Ghazgkull commented 4 years ago

@jcantrill This is a follow-up to the PR I submitted last week to add handling for 410 Gone responses on the pod watch.

This PR makes two changes:

  1. It adds the same handling for the namespace watch.
  2. I changed the log level of the message from debug to info. Based on my experience running this in the wild, this is a message that we should print with the default log level. It's important for folks to know this is happening and that we're handling it.

This fixes issue #246

Ghazgkull commented 4 years ago

Please hold on this. I'm adding more tests and will update the PR with a new commit shortly.

Ghazgkull commented 4 years ago

New commit is attached, which includes tests for the controlling thread loop as well. Ready for review.