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

cannot set spool size? #17

Closed
sysmonk opened this issue Jul 15, 2014 · 2 comments
Closed

cannot set spool size? #17

sysmonk opened this issue Jul 15, 2014 · 2 comments
Assignees
Labels
Milestone

Comments

@sysmonk
Copy link
Contributor

sysmonk commented Jul 15, 2014

Hi,

Trying the newest log-courier, and i'm not able to set the spool size. I get this error:

# /usr/local/bin/log-courier -config /etc/log-courier.conf -log-to-syslog=true
Configuration error: Option /general/spool size must be int64 or similar

The configuration i use:

{
        "general": {
                "spool size": 1024,
                "spool timeout": 5,
                "log level" : "debug"
        },
        "network": {
                "servers": [ ... ],
                "ssl ca": "...",
                "timeout": 15,
                "reconnect": 10
        },
        "includes": [
                "/etc/log-courier.d/*.conf"
        ]
}

By the way, i liked the ability to set the spool size on the command line, that way i can have different spool sizes based on the server role / hostname when starting the script from init script. Now, i need to have separate configs for different servers :(

@driskell
Copy link
Owner

Hi

Will get this fixed ASAP.

Regarding cmd line options it made sense to me to keep as much in the config file as possible as it's easier to modify than the init script. When I get around to packaging it'll simplify the init script too and make it something that can be updated on package upgrades and is the same on all installations.

If you've done testing with different spool sizes it'll be good to know what positive/negative effects you've seen too. I've only tweaked it a few times and not seen much difference.

Jason

@driskell driskell added the bug label Jul 15, 2014
@driskell driskell self-assigned this Jul 15, 2014
@driskell driskell added this to the 0.12 milestone Jul 15, 2014
@sysmonk
Copy link
Contributor Author

sysmonk commented Jul 15, 2014

Re the testing with different spool sizes - i didn't do any tests on log-courier, but i tried different values on logstash-forwarder. On servers with 3k+ of log lines per second i found that having bigger (i.e. 4096) values would perform a bit better. At the low values (<1024) if there was some hiccup (i.e. increase from 3k to 10k log lines / s for a few minutes), it would take a while for logstash-forwarder to get back in sync, and with higher values it was faster. No real tests done on performance though.

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

No branches or pull requests

2 participants