Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Parser processor #4551

Merged
merged 24 commits into from
Aug 22, 2018
Merged

Parser processor #4551

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
c823023
Add fieldparser processor
danielnelson Jul 31, 2018
683130a
Rename fieldparser to parser
Aug 6, 2018
ad54725
Add test for parsing one filed and multiple fields
Aug 8, 2018
f1373e6
Add test to parse one tag and to parse multiple tags
Aug 8, 2018
bc11ecc
WIP: Add test for parsing one and two tags
Aug 8, 2018
62c35f0
Remove duplicate files
Aug 8, 2018
60c0eb7
Fix test for parsing one and two tags
Aug 8, 2018
38733c1
Remove tests that handle parsing tags
Aug 9, 2018
c736252
Remove Parse Tag functionality
Aug 9, 2018
d805fc7
Add test that when a field fails to parse still parses other values
Aug 9, 2018
a0e2002
Fix test for parsing one and two fields
Aug 9, 2018
27c27b9
Rename fieldparser to parser
Aug 9, 2018
46173ee
Reformat
Aug 10, 2018
c04bc6a
WIP: README added
Aug 13, 2018
2a5da32
Add original measurement merge|replace|keep functionality
glinton Aug 16, 2018
4b0c00b
Add printer back to processors/all
glinton Aug 16, 2018
0938090
Update config in readme
glinton Aug 16, 2018
74a1eab
Fix tests and bad field parsing
glinton Aug 16, 2018
9e15fc6
Make config single level
glinton Aug 17, 2018
a0f1472
Update changelog
danielnelson Aug 22, 2018
83a969b
Split original option into drop_original and merge options
danielnelson Aug 22, 2018
d5aec65
Merge branch 'master' into fieldparser
danielnelson Aug 22, 2018
8ce4e98
Fixup merge issue with master CHANGELOG
danielnelson Aug 22, 2018
52b3b23
Use code block for output example
danielnelson Aug 22, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove tests that handle parsing tags
  • Loading branch information
Ayrdrie Palmer committed Aug 9, 2018
commit 38733c1f42febefebd3b6a55878e38ffe19d5100
56 changes: 1 addition & 55 deletions plugins/processors/fieldparser/fieldparser_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ func Metric(v telegraf.Metric, err error) telegraf.Metric {
func TestApply(t *testing.T) {
tests := []struct {
name string
parseTags []string
parseFields []string
config parsers.Config
input telegraf.Metric
Expand Down Expand Up @@ -130,61 +131,6 @@ func TestApply(t *testing.T) {
time.Unix(0, 0))),
},
},
{
name: "parse one tag",
config: parsers.Config{
DataFormat: "logfmt",
},
input: Metric(
metric.New(
"success",
map[string]string{},
map[string]interface{}{
"test_name": `ts=2018-07-24T19:43:40.275Z`,
},
time.Unix(0, 0))),
expected: []telegraf.Metric{
Metric(metric.New(
"success",
map[string]string{},
map[string]interface{}{
"test_name": `ts=2018-07-24T19:43:40.275Z`,
},
time.Unix(0, 0))),
Metric(metric.New(
"success",
map[string]string{},
map[string]interface{}{
"ts": "2018-07-24T19:43:40.275Z",
},
time.Unix(0, 0))),
},
},
{
name: "parse two tags",
config: parsers.Config{
DataFormat: "logfmt",
},
input: Metric(
metric.New(
"success",
map[string]string{},
map[string]interface{}{
"timestamp": `ts=2018-07-24T19:43:40.275Z`,
"message": `msg="http request"`,
},
time.Unix(0, 0))),
expected: []telegraf.Metric{
Metric(metric.New(
"success",
map[string]string{},
map[string]interface{}{
"timestamp": `ts=2018-07-24T19:43:40.275Z`,
"message": `msg="http request"`,
},
time.Unix(0, 0))),
},
},
{
name: "Fail to parse fields but still parses tag",
config: parsers.Config{
Expand Down