Tags no longer passed in due to change in Client

[shortjared]

I was experimenting with Telegraf, and I am pretty impressed with the ease of getting it setup and the potential for adding plugins. That said, I noticed that Tags don't seem to be passed at the point level. Not even the default "host" as I would have expected after finding issue #4 .

I think I tracked down the cause.

It looks like a recent change in the influxdb client ( influxdata/influxdb@e6c36d5 ) has made it so that passing common parameters at the batch level is no longer allowed.

If we look at the agent.go code, it appears that the "Tags" were passed at the top of the batch level and expected to be inherited down through the "sub" points. This seems to be leveraging the now removed common batch parameters functionality.

    close(points)

    var acc BatchPoints
    acc.Tags = a.Config.Tags
    acc.Time = time.Now()
    acc.Database = a.Config.Database

    for sub := range points {
        acc.Points = append(acc.Points, sub.Points...)
    }

I am not familiar enough with Go to effectively fix and properly add unit testing, but hopefully this is enough for someone to jump in and take it from here.

[dever860]

+1

[mgresser]

+1 Also seeing this issue.