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

Fix #157 remove parameters causing warn on fluentd #159

Merged
merged 1 commit into from
Mar 11, 2015
Merged

Fix #157 remove parameters causing warn on fluentd #159

merged 1 commit into from
Mar 11, 2015

Conversation

uu59
Copy link
Member

@uu59 uu59 commented Mar 4, 2015

Remove add_newline, output_tag and output_time from s3 setting then I don't get any warnings with out_s3 0.5.6.

@repeatedly Now, generated config as below. looks good?

<match s3.**>
  type s3
  aws_key_id XXXXXXXXXXXXX
  aws_sec_key XXXXXXXX
  s3_bucket bbb
  s3_region us-west-1
  format out_file
  include_time_key false
  store_as gzip
  use_ssl true
  buffer_type memory
</match>

uu59 added a commit that referenced this pull request Mar 11, 2015
@uu59
Merge pull request #159 from fluent/remove_out_s3_warn_parameters
5cf8d17 
Fix #157 remove parameters causing warn on fluentd
@uu59 uu59 merged commit 5cf8d17 into master Mar 11, 2015
@uu59 uu59 deleted the remove_out_s3_warn_parameters branch March 11, 2015 04:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@uu59