Influxdb 1.2. Collectd service. Multiple data sources for measurement don't work correctly.

[cinek810]

Bug report

System info: Influxdb 1.2, collectd-4.10

Steps to reproduce:

1. enable collectd service in influxdb
2. enable df plugin in collectd, configure collectd to send data to influx collectd service
3. check measurements created in influxdb
4. use types.db with:
   df used:GAUGE:0:1125899906842623, free:GAUGE:0:1125899906842623

Expected behavior:
Expected is to have one measurement df with values stored in fileds named "used" and "free".
At least it's how it's described on mailing list:
https://mailman.verplant.org/pipermail/collectd/2015-June/006551.html

Actual behavior:
Two separate measurements
df_used, df_free created

Additional info:
I've tried changing the ds names in types.db from used to df_used, the result was creation of measurement:
df_df_used
I suspect that data source is simply concatenated to the measurement name instead of beeing used as filed name in case of multiple values returned from measurement.

[rbetts]

@cinek810 I've spent some time reading the influxdb collectd service code. I think what is happening is by design - good or bad.

The construction of the InfluxDB measurement name is done here:
https://github.com/influxdata/influxdb/blob/master/services/collectd/service.go#L395 . This, as you guessed, is concatenating collectd's ValueList.Identifier.Plugin field and ValueList.DSName for each value in the ValueList.

Identifier.Plugin is from collectd here: https://github.com/collectd/go-collectd/blob/master/api/main.go#L47

DSName is from collected here: https://github.com/collectd/go-collectd/blob/master/api/main.go#L82

Changing this behavior seems pretty straight forward but would require a configuration flag to opt in to the new measurement name encoding.

[cinek810]

Since in new versions of influxdb we don't have joins it would be desired.. because now it's for example impossible to use collectd standard df plugin to display percent of drive used.. 2017-05-31 21:00 GMT+02:00 Ryan Betts <notifications@github.com>:

…

@cinek810 <https://github.com/cinek810> I've spent some time reading the influxdb collectd service code. I think what is happening is by design - good or bad. The construction of the InfluxDB measurement name is done here: https://github.com/influxdata/influxdb/blob/master/services/ collectd/service.go#L395 . This, as you guessed, is concatenating collectd's ValueList.Identifier.Plugin field and ValueList.DSName for each value in the ValueList. Identifier.Plugin is from collectd here: https://github.com/collectd/ go-collectd/blob/master/api/main.go#L47 DSName is from collected here: https://github.com/collectd/ go-collectd/blob/master/api/main.go#L82 Changing this behavior seems pretty straight forward but would require a configuration flag to opt in to the new measurement name encoding. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#8426 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGSwFwK5_V8k0O9SS8rTOCunX_vT4jnuks5r_biygaJpZM4NmJ2i> .

[rbetts]

Resolved by #8523