Davide Girardi

Results 8 comments of Davide Girardi

cc @ravikesarwani I think it would be good to have your input on the docs-related changes

Thanks @legoguy1000 for adding this :) . I am currently keeping this PR on hold as we might have some refactoring to do, as follow-up of https://github.com/elastic/beats/pull/33105 (cc @kaiyan-sheng )

@tommyers-elastic Yes, we'd need to document the new parameter, if we go this route. However, based on Joao's comment here https://github.com/elastic/obs-infraobs-team/issues/810#issuecomment-1266687847, this PR might not be needed anymore.

Working directly on this plugin, we can avoid adding additional metadata introducing a new config value, and use it as flag to add / not add the header medatata, together...

I think we can close this as not needed anymore :) (cc @yaauie @roaksoax @tommyers-elastic )

Closing as not needed anymore (see https://github.com/elastic/obs-infraobs-team/issues/842 )