dom
/
telegraf
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix for tags in the config not being applied to the agent.

This commit is contained in:
Ellison Marks 2015-10-22 11:22:54 -07:00
parent c8852339c9
commit 8230b7d7bc
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
config.go
View File

@ -21,7 +21,7 @@ import (
// specified // specified
type Config struct { type Config struct {
// This lives outside the agent because mergeStruct doesn't need to handle maps normally. // This lives outside the agent because mergeStruct doesn't need to handle maps normally.
// We just copy the elements manually in applyAgent. // We just copy the elements manually in ApplyAgent.
Tags map[string]string Tags map[string]string
agent *Agent agent *Agent
@ -129,11 +129,11 @@ func (c *Config) ApplyOutput(name string, v interface{}) error {
// Overrides only values in the given struct that were set in the config. // Overrides only values in the given struct that were set in the config.
func (c *Config) ApplyAgent(a *Agent) error { func (c *Config) ApplyAgent(a *Agent) error {
if c.agent != nil { if c.agent != nil {
for key, value := range c.Tags {
a.Tags[key] = value
}
return mergeStruct(a, c.agent, c.agentFieldsSet) return mergeStruct(a, c.agent, c.agentFieldsSet)
} }
for key, value := range c.Tags {
a.Tags[key] = value
}
return nil return nil
} }

1
config_test.go
View File

@ -142,7 +142,6 @@ func (s *MergeStructSuite) TestFullMerge() {
s.T().Error(err) s.T().Error(err)
} }
s.Equal(result, s.FullStruct, fmt.Sprintf("Full merge of %v onto FullStruct failed.", s.AnotherFullStruct)) s.Equal(result, s.FullStruct, fmt.Sprintf("Full merge of %v onto FullStruct failed.", s.AnotherFullStruct))
s.T().Log("hi")
} }
func (s *MergeStructSuite) TestPartialMergeWithoutSlices() { func (s *MergeStructSuite) TestPartialMergeWithoutSlices() {