splunk_modinput_prometheus icon indicating copy to clipboard operation
splunk_modinput_prometheus copied to clipboard

Splunk base update app

Open tioumen opened this issue 6 years ago • 6 comments

Hello,

Did you updated the splunk app on splunk base : https://splunkbase.splunk.com/app/4077/ Seems not based on the last update from github vs Splunk base.

Thanks.

tioumen avatar Feb 06 '19 15:02 tioumen

The Splunkbase version is behind Github. Please only use Github if you are able to deal with a few changes.

I will be changing some defaults and configuration keys and then releasing 1.0 to Splunkbase.

lukemonahan avatar Feb 07 '19 22:02 lukemonahan

Yeap, that's what I have done. I was needed the perfix and parser. I used your last update on github. What are the improvement you've in the pipe please ? :)

tioumen avatar Feb 08 '19 14:02 tioumen

The only further change I plan to make will be to make the dot-notation the default, with a new config key of "use_old_metric_name_conversion" to allow backwards compatibility.

lukemonahan avatar Feb 09 '19 08:02 lukemonahan

Hello, I compiled the code with the 2 new fields "MetricNamePrefix" & "MetricNameParse". It's works. But I'm getting 100% of CPU due to that parsing when ingesting metrics (that's not the case wtih previous version without these new fileds). I tried also to remove alternatively each option but I'm still getting 100% of CPU. I also tried to let the 2 new parameters empty and I'm getting the same amount of CPU load ... Do you have any advice please ?

Thanks.

tioumen avatar Feb 15 '19 11:02 tioumen

@tioumen Can you check if latest master fixes your issue?

ltmon avatar Mar 08 '19 21:03 ltmon

Sorry for the late answer. Still getting overload issue on the HF when I'm blacklisting / whitelisting metrics from Prometheus.

tioumen avatar Apr 15 '19 15:04 tioumen