Skip to content

Set skip rows and columns in csv parser#5336

Merged
danielnelson merged 1 commit intomasterfrom
bugfix/5334
Jan 23, 2019
Merged

Set skip rows and columns in csv parser#5336
danielnelson merged 1 commit intomasterfrom
bugfix/5334

Conversation

@glinton
Copy link
Copy Markdown
Contributor

@glinton glinton commented Jan 23, 2019

Resolves #5334

Turns out skip_rows and skip_columns were never getting set in the csv parser.

Tested with:

[[inputs.exec]]
  timeout = "1s"
  command = "echo -en 'version=13.0\ndate,instance,Incoming Requests,Outgoing Requests,Incoming Answers 2xxx,Outgoing Answers 2xxx,Incoming Answers UTD,Outgoing Answers UTD,Incoming Answers Redirect,Outgoing Answers Redirect,Incoming Answers Other,Outgoing Answers Other,Retransmit Requests,Rejected Requests Filtering,Rejected Requests In-gress Filtering,Rejected Requests Other,Timeout Requests,Discarded Answers\n2018-01-23-12:04,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1\n'"
  data_format = "csv"
  csv_skip_rows = 1
  csv_header_row_count = 1
  csv_delimiter = ","
  csv_trim_space = true
  csv_tag_columns = ["instance"]
  csv_timestamp_column = "date"
  csv_timestamp_format = "2006-01-02-15:04"

@glinton glinton added the fix pr to fix corresponding bug label Jan 23, 2019
@glinton glinton added this to the 1.9.4 milestone Jan 23, 2019
@danielnelson danielnelson merged commit 458d310 into master Jan 23, 2019
@danielnelson danielnelson deleted the bugfix/5334 branch January 23, 2019 19:37
danielnelson pushed a commit that referenced this pull request Jan 23, 2019
trevorwhitney pushed a commit to trevorwhitney/telegraf that referenced this pull request Feb 14, 2019
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
otherpirate pushed a commit to otherpirate/telegraf that referenced this pull request Mar 15, 2019
dupondje pushed a commit to dupondje/telegraf that referenced this pull request Apr 22, 2019
bitcharmer pushed a commit to bitcharmer/telegraf that referenced this pull request Oct 18, 2019
athoune pushed a commit to bearstech/telegraf that referenced this pull request Apr 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix pr to fix corresponding bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants