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.

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

fix template for output filename and path #4475

Merged
merged 1 commit into from Jun 9, 2017

Conversation

Projects
None yet
4 participants
@jalvz
Copy link
Contributor

commented Jun 8, 2017

This fixes the generation of the yml file for system tests. After generating an "example" beat, the resulting tests/system/config/example.yml.j2 would look like:

   # Path to the directory where to save the generated files. The option is mandatory.
    path: {{ output_file_path|default(beat.working_dir + "/output") }}


    # Name of the generated files. The default is `example` and it generates
    # files: `example`, `example.1`, `example.2`, etc.
    filename: {{ output_file_filename|default("example") }}

@jalvz jalvz requested a review from ruflin Jun 8, 2017

@ruflin

ruflin approved these changes Jun 8, 2017

@jalvz jalvz added the review label Jun 8, 2017

@tsg tsg merged commit cde8f38 into elastic:master Jun 9, 2017

3 of 4 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
CLA Commit author is a member of Elasticsearch
Details
beats-ci Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@tsg

This comment has been minimized.

Copy link
Collaborator

commented Jun 9, 2017

Thanks for the fix, @jalvz!

@monicasarbu monicasarbu added the libbeat label Jun 12, 2017

ramon-garcia added a commit to ramon-garcia/beats that referenced this pull request Dec 5, 2017

athom added a commit to athom/beats that referenced this pull request Jan 25, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.