mirror of
https://github.com/VictoriaMetrics/VictoriaMetrics.git
synced 2024-11-23 20:37:12 +01:00
lib/promscrape: add a comment explaining the logic behind adding exported_
perfix to metric names
This is a follow-up for 7b87fac8e7
Updates https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3557
Updates https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3406
This commit is contained in:
parent
7b87fac8e7
commit
8b9ebf625a
@ -904,14 +904,24 @@ func (sw *scrapeWork) addAutoTimeseries(wc *writeRequestCtx, name string, value
|
||||
|
||||
func (sw *scrapeWork) addRowToTimeseries(wc *writeRequestCtx, r *parser.Row, timestamp int64, needRelabel bool) {
|
||||
metric := r.Metric
|
||||
if needRelabel && isAutoMetric(metric) {
|
||||
if !sw.Config.HonorLabels && len(r.Tags) == 0 {
|
||||
bb := bbPool.Get()
|
||||
bb.B = append(bb.B, "exported_"...)
|
||||
bb.B = append(bb.B, metric...)
|
||||
metric = bytesutil.InternBytes(bb.B)
|
||||
bbPool.Put(bb)
|
||||
}
|
||||
|
||||
// Add `exported_` prefix to metrics, which clash with the automatically generated
|
||||
// metric names only if the following conditions are met:
|
||||
//
|
||||
// - The `honor_labels` option isn't set to true in the scrape_config.
|
||||
// If `honor_labels: true`, then the scraped metric name must remain unchanged
|
||||
// because the user explicitly asked about it in the config.
|
||||
// - The metric has no labels (tags). If it has labels, then the metric value
|
||||
// will be written into a separate time series comparing to automatically generated time series.
|
||||
//
|
||||
// See https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3557
|
||||
// and https://github.com/VictoriaMetrics/VictoriaMetrics/issues/3406
|
||||
if needRelabel && !sw.Config.HonorLabels && len(r.Tags) == 0 && isAutoMetric(metric) {
|
||||
bb := bbPool.Get()
|
||||
bb.B = append(bb.B, "exported_"...)
|
||||
bb.B = append(bb.B, metric...)
|
||||
metric = bytesutil.InternBytes(bb.B)
|
||||
bbPool.Put(bb)
|
||||
}
|
||||
labelsLen := len(wc.labels)
|
||||
targetLabels := sw.Config.Labels.GetLabels()
|
||||
|
Loading…
Reference in New Issue
Block a user