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

Cherry-pick #16586 to 7.x: Add cloudfoundry input for filebeat #16614

Merged
merged 1 commit into from
Feb 27, 2020

Conversation

blakerouse
Copy link
Contributor

Cherry-pick of PR #16586 to 7.x branch. Original message:

What does this PR do?

Add cloudfoundry input to x-pack filebeat.

Why is it important?

To support Cloud Foundry.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works

Author's Checklist

  • Sends http access, log messages, and error logs to elasticsearch.

How to test this PR locally

Run x-pack build of filebeat with the following configuration, with a deploy cloudfoundry. Substitute the configuration variables based on the cloudfoundry deployment and authentication.

filebeat.inputs:
  - type: cloudfoundry
    api_address: https://api.dev.cfdev.sh
    client_id: firehose-to-filebeat
    client_secret: verysecret
    ssl:
      verification_mode: none

output.console:
  pretty: true

Related issues

Use cases

Screenshots

Logs

* Add cloudfoundry input.

* Fix issue with http transport.

* Add input documentation.

(cherry picked from commit ed693e9)
@blakerouse
Copy link
Contributor Author

I forgot the changelog entry in the original PR, I have a follow up PR that has the changelog for this change.

Copy link
Member

@ChrsMark ChrsMark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@blakerouse blakerouse merged commit b18b864 into elastic:7.x Feb 27, 2020
@blakerouse blakerouse deleted the backport_16586_7.x branch February 27, 2020 14:00
@blakerouse blakerouse added the test-plan Add this PR to be manual test plan label Mar 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport review test-plan Add this PR to be manual test plan
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants