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

[docs] Add missing HTTP JSON input #17999

Merged
merged 2 commits into from
May 5, 2020
Merged

Conversation

dedemorton
Copy link
Contributor

@dedemorton dedemorton commented Apr 26, 2020

Adds HTTP input to the list of includes.

@alakahakai I'm assuming that you meant to build this content, but let me know if for some reason you don't want it to show up in the published docs.

Also let me know which versions this needs to be backported to.

(Note that I've also alphabetized the list of inputs because there are too many entries to sort by common usage. Put it in a separate commit.)

@dedemorton dedemorton added docs needs_backport PR is waiting to be backported to other branches. v7.7.0 labels Apr 26, 2020
@dedemorton dedemorton requested a review from alakahakai April 26, 2020 20:38
@dedemorton dedemorton self-assigned this Apr 26, 2020
@alakahakai
Copy link

alakahakai commented Apr 26, 2020

Adds HTTP input to the list of includes.

@alakahakai I'm assuming that you meant to build this content, but let me know if for some reason you don't want it to show up in the published docs.

@dedemorton Yes, thank you.

Also let me know which versions this needs to be backported to.

7.x and 7.7.

@dedemorton
Copy link
Contributor Author

@alakahakai Great! If I can get your review approval, I'll get this merged and backported to the relevant branches. Thanks.

@dedemorton
Copy link
Contributor Author

jenkins run the tests please

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview stats

Expand to view the summary

Build stats

  • Build Cause: [dedemorton commented: jenkins run the tests please]

  • Start Time: 2020-04-29T18:37:16.785+0000

  • Duration: 52 min 24 sec (3143978)

  • Commit: 16516cf

Test stats 🧪

Test Results
Failed 0
Passed 1589
Skipped 280
Total 1869

@dedemorton
Copy link
Contributor Author

@alakahakai I'm going to take your comment above as approval for this PR and merge since there aren't actual content changes in this PR. Thanks!

@dedemorton dedemorton merged commit 5d66251 into elastic:master May 5, 2020
@dedemorton dedemorton deleted the add_httpjson branch May 5, 2020 19:15
dedemorton added a commit to dedemorton/beats that referenced this pull request May 5, 2020
dedemorton added a commit to dedemorton/beats that referenced this pull request May 5, 2020
@dedemorton dedemorton removed the needs_backport PR is waiting to be backported to other branches. label May 5, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants