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

Add FAQ topic about bandwidth throttling #2225

Merged
merged 1 commit into from
Aug 15, 2016

Conversation

dedemorton
Copy link
Contributor

@tsg Here's my first pass at resolving issue #2164. Please review. I wasn't sure if we wanted a generic topic for all beats, or only for Filebeat, so I added it to all beats.

@andrewkroh Is there something we can say about bandwidth throttling on Windows (I don't think the Linux example is useful for Winlogbeat), or should I remove this topic from Winlogbeat entirely?

@andrewkroh
Copy link
Member

I have never tried this on Windows so maybe we should omit it for Winlogbeat. But it seems like it is possible to implement a traffic shaping using a Group Policy. I found this http://www.windowsnetworking.com/articles-tutorials/windows-server-2012/QoS-Windows-Server-2012-Part3.html

@tsg
Copy link
Contributor

tsg commented Aug 15, 2016

LGTM, thanks @dedemorton. Requires a rebase.

@dedemorton
Copy link
Contributor Author

@tsg This should be ready to merge now. In addition to the rebase, I moved the content to a separate asciidoc file to make it easier to exclude the content from Winlogbeat (I'm trying to avoid conditional coding in favor of asciidoc include statements because I think the include statements are easier for maintainers to understand).

@tsg tsg merged commit b53e50c into elastic:master Aug 15, 2016
dedemorton added a commit to dedemorton/beats that referenced this pull request Aug 15, 2016
ruflin pushed a commit that referenced this pull request Aug 16, 2016
* Make FAQs a single topic (#2250)

* Add FAQ topic about bandwidth throttling (#2225)
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