[exporter/splunkhecexporter] Support separate metric index for hec exporter

[spiffyy99]

Component(s)

exporter/splunkhec

Is your feature request related to a problem? Please describe.

Users of splunkhecexporter may be sending both logs and metrics data at the same time, so the config should provide a separate default metrics index to support this, since metrics and logs indexes support different data.

Describe the solution you'd like

To avoid regressions in existing clients, metrics index can be configured as such:

• Config introduces an extra optional param MetricsIndex
• Have an additional config flag UseMetricsIndex. If this is set, metrics exporter will set the default index to be MetricsIndex for metrics data.

Describe alternatives you've considered

No response

Additional context

No response

[github-actions[bot]]

Pinging code owners:

• exporter/splunkhec: @atoulme @dmitryax

  See Adding Labels via Comments if you do not have permissions to add labels yourself.

[atoulme]

It doesn't look to me like we would need a UseMetricsIndex field. We can just check if MetricsIndex is set.

If MetricsIndex is set, we use it when converting metrics to splunk events. If it is not set, we should default back to the Index config field to preserve the current behavior.

[spiffyy99]

It doesn't look to me like we would need a UseMetricsIndex field. We can just check if MetricsIndex is set.

If MetricsIndex is set, we use it when converting metrics to splunk events. If it is not set, we should default back to the Index config field to preserve the current behavior.

good point. that's fine as well.

[crobert-1]

Removing needs triaged based on response from code owner.

[atoulme]

@spiffyy99 are you still working on this?

[spiffyy99]

No, we decided to go with a different approach. Lets close this out.