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

Update syslog-ng #7

Closed
wants to merge 1 commit into from
Closed

Update syslog-ng #7

wants to merge 1 commit into from

Conversation

joenoon
Copy link

@joenoon joenoon commented Feb 5, 2014

In my containers, syslog-ng was failing to start because of afsql being a default, even though it requires an extra package it be installed. The options were to either install that extra package, or add this argument

In my containers, syslog-ng was failing to start because of afsql being a default, even though it requires an extra package it be installed.  The options were to either install that extra package, or add this argument
@FooBarWidget
Copy link
Member

I believe this has already been fixed in version 0.9.4. See commit 6786e73.

Are you on an older version? If so could you check whether 0.9.4 fixes it?

@ruphin
Copy link
Contributor

ruphin commented Feb 14, 2014

This has not been fixed in 0.9.4, it is an unrelated issue.
See: https://bugs.launchpad.net/ubuntu/+source/syslog-ng/+bug/1009920

I recommend using the method described by Moritz Bunkus (moritz-bunkus) in this same thread to fix:

  1. Add --default-modules=... to sylog-ng's command line. This can be done by adding the following to /etc/default/syslog-ng:

SYSLOGNG_OPTS="--no-caps --default-modules=affile,afprog,afsocket,afuser,basicfuncs,csvparser,dbparser,syslogformat"

@FooBarWidget
Copy link
Member

Fixed in commit 9af37c0.

@kingdonb
Copy link
Collaborator

@FooBarWidget I take umbrage with the change --default-modules=...

In my fork of baseimage-docker, I have upgraded to trusty. This option is not supported and causes syslog-ng to fail, leaves lots of detritus in the runsvdir proc-info and results in a restart loop for syslog-ng. I found references to it online, so maybe it was an option in 12.04, but it must have been deprecated, and its absence will cause problems later for when 14.04 comes around as new-LTS.


I just checked on saucy/13.10 and the missing option is not an issue in the release prior to trusty/14.04, so maybe it's still a good change for the time being, but FWIW the original error "afsql" finally goes away for me with the more current trusty distro, I read that it was fixed in Debian upstream

I proceeded with just --no-caps, and accepting the remainder of your change commit, maybe recommended discarding this bottom-right part of image/config/syslog_ng_default if we can find a better way to do that or if an official trusty fork should emerge...

@FooBarWidget
Copy link
Member

I'm planning on upgrading baseimage-docker to Trusty once Trusty is officially released.

@kingdonb
Copy link
Collaborator

OK. Thanks for your reply, and from what it looks like on my end you won't have much to do. Works great with minimal changes.

@FooBarWidget
Copy link
Member

This change has been obsolete by PR #71. See commit 5cfdd42.

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

Successfully merging this pull request may close these issues.

4 participants