Undeprecate attributes module about node_exporter HOT 5 CLOSED

The textfile module is a better way to do this, and we want to get rid of the config file.

For example you can drop in a file to it with the content:
attributes{myauoracluster="A"} 1

from node_exporter.

That doesn't fo what I described. I want to know what the total used disk space is across nodes in cluster 'a', for instance, so I want the fs metrics to all have a cluster label. Your solution doesn't help with that.

Also, I only have a couple of such labels, so putting them in a flag is fine.

from node_exporter.

For that you need to configure that as target labels in the prometheus config, no feature of the node exporter can do that for you.

Set labels in the prometheus config to do this, see https://github.com/prometheus/prometheus/blob/master/config/config.proto#L48

from node_exporter.

Setting custom server-side labels only works for statically configured targets at the moment (that will change when we add more generalized SD support).

What @swsnider suggests would add a separate set of time series that would contain attribute information about this exporter. These could be correlated via and and the instance label with the other time series outputted by the node exporters to select only a certain group of exporters. That might be fine too?

But yeah, there's currently no node exporter feature to just add a label to every time series exported, if that is what you want.

from node_exporter.

These days we have both all the support needed to do this via the attribute approach, and service discovery in Prometheus would also allow you to do this in few ways including file_sd discovery. Setting target labels on an exporter is an anti-pattern we want to avoid, that concern belongs on the Prometheus server.

from node_exporter.