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

default-network-drivers should support log-iw-size #4170

Closed
kaiw opened this issue Oct 10, 2022 · 3 comments · Fixed by #4328
Closed

default-network-drivers should support log-iw-size #4170

kaiw opened this issue Oct 10, 2022 · 3 comments · Fixed by #4328
Labels

Comments

@kaiw
Copy link

kaiw commented Oct 10, 2022

syslog-ng

Version of syslog-ng

3.33.2 (but code appears identical in current master)

Platform

CentOS 7

Issue

The default-network-drivers block supports max-connections, but does not support log-iw-size, even though both of these need to be adjusted simultaneously. Passing through log-iw-size to the TCP sources is straightforward and lets us increase both it and max-connections to avoid configuration warnings.

Steps to reproduce

  1. Configure default-network-drivers with an increased max-connections
  2. Restart syslog-ng, and receive a warning like WARNING: window sizing for tcp sources were changed in syslog-ng 3.3...
  3. Add log-iw-size configuration as suggested, and observe that the warning still appears
source s_sys {
    system();
    default-network-drivers(
        ip(127.0.0.1)
        log-iw-size(10000)
        max-connections(100)
    );
    internal();
};
@kaiw kaiw added the bug label Oct 10, 2022
@bazsi
Copy link
Collaborator

bazsi commented Oct 11, 2022 via email

@kovgeri01
Copy link
Contributor

Hi @bazsi, will it be possible if i work on this as my first issue in the project?

@bazsi
Copy link
Collaborator

bazsi commented Feb 13, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants