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

logparser on windows can't use * in the path #1517

Closed
dmitrysdm opened this issue Jul 19, 2016 · 3 comments · Fixed by #1821
Closed

logparser on windows can't use * in the path #1517

dmitrysdm opened this issue Jul 19, 2016 · 3 comments · Fixed by #1821
Labels
bug unexpected problem or unintended behavior platform/windows
Milestone
1.1.0

Comments

@dmitrysdm
Copy link

dmitrysdm commented Jul 19, 2016 •
edited by sparrc
Loading

Bug Report:

logparser working correctly with follow path
files = ["//hostname/FTPFetchFilesLog/2016_07/2016_07_19.txt"]
but, can't see anything with follow:
files = ["//hostname/FTPFetchFilesLog/2016_07/.txt"]
or follow:
files = ["//hostname/FTPFetchFilesLog//*.txt"]

config:

files = ["//hostname/FTPFetchFilesLog/2016_07/*.txt"]

result:

PS \telegraf-1.0.0-beta2_windows_amd64\telegraf\telegraf.exe -config telegraf-1.0.0-beta2_windows_amd64\telegraf\logparser_telegraf.conf
2016/07/19 11:50:27 Attempting connection to output: influxdb
2016/07/19 11:50:27 Successfully connected to output: influxdb
2016/07/19 11:50:27 Starting Telegraf (version 1.0.0-beta3)
2016/07/19 11:50:27 Loaded outputs: influxdb
2016/07/19 11:50:27 Loaded inputs: logparser
2016/07/19 11:50:27 Tags enabled: host=hostname
2016/07/19 11:50:27 Agent Config: Interval:10s, Debug:true, Quiet:false, Hostname:"hostname", Flush Interval:10s
2016/07/19 11:50:30 Input [logparser] gathered metrics, (10s interval) in 0
2016/07/19 11:50:40 Input [logparser] gathered metrics, (10s interval) in 0
2016/07/19 11:50:40 Output [influxdb] buffer fullness: 0 / 1000 metrics. Total gathered metrics: 0. Total dropped metrics: 0.
2016/07/19 11:50:50 Input [logparser] gathered metrics, (10s interval) in 0
2016/07/19 11:50:50 Output [influxdb] buffer fullness: 0 / 1000 metrics. Total gathered metrics: 0. Total dropped metrics: 0.

config:

files = ["//hostname/FTPFetchFilesLog/2016_07/2016_07_19.txt"]

result:

PS \telegraf-1.0.0-beta2_windows_amd64\telegraf\telegraf.exe -config \telegraf-1.0.0-beta2_windows_amd64\telegraf\logparser_telegraf.conf
2016/07/19 12:00:45 Attempting connection to output: influxdb
2016/07/19 12:00:45 Successfully connected to output: influxdb
2016/07/19 12:00:45 Starting Telegraf (version 1.0.0-beta3)
2016/07/19 12:00:45 Loaded outputs: influxdb
2016/07/19 12:00:45 Loaded inputs: logparser
2016/07/19 12:00:45 Tags enabled: host=hostname
2016/07/19 12:00:45 Agent Config: Interval:10s, Debug:true, Quiet:false, Hostname:"hostname", Flush Interval:10s
2016/07/19 12:00:45 Seeked //hostname/FTPFetchFilesLog/2016_07/2016_07_19.txt - &{Offset:0 Whence:0}
2016/07/19 12:00:50 Input [logparser] gathered metrics, (10s interval) in 0
2016/07/19 12:01:00 Input [logparser] gathered metrics, (10s interval) in 0
2016/07/19 12:01:00 Output [influxdb] buffer fullness: 885 / 1000 metrics. Total gathered metrics: 885. Total dropped metrics: 0.
2016/07/19 12:01:00 Output [influxdb] wrote batch of 885 metrics in 24.8388ms
@gcyre
Copy link

gcyre commented Sep 22, 2016

is there any comment on this? I've run into the same issue

@sparrc sparrc added the bug unexpected problem or unintended behavior label Sep 23, 2016
@sparrc sparrc added this to the 1.1.0 milestone Sep 23, 2016
@sparrc
Copy link
Contributor

sparrc commented Sep 23, 2016

I think I know the issue, here, I'm putting it into a bugfix for 1.1

@gcyre
Copy link

gcyre commented Sep 23, 2016

Awesome, thanks!

sparrc added a commit that referenced this issue Sep 28, 2016
@sparrc
globpath: only walk tree if ** is defined
9c8940a 
closes #1517
@sparrc sparrc mentioned this issue Sep 28, 2016
Merged
3 tasks
sparrc added a commit that referenced this issue Sep 28, 2016
@sparrc
globpath: only walk tree if ** is defined
5213455 
closes #1517
jackzampolin pushed a commit that referenced this issue Oct 7, 2016
@sparrc @jackzampolin
globpath: only walk tree if ** is defined
7c20e0f 
closes #1517
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug unexpected problem or unintended behavior platform/windows
Projects
None yet
Milestone
1.1.0
Development

Successfully merging a pull request may close this issue.

globpath: only walk tree if ** is defined influxdata/telegraf
3 participants
@gcyre @sparrc @dmitrysdm