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

Merge dockerized rsyslog. config split into multiple files. #4

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

t0ffel
Copy link

@t0ffel t0ffel commented May 19, 2016

Please wait for @portante to check this before merging.

@portante everything here as discussed today.

I didn't fix ipaddr4.
pid - is very suspicious. a lot of cases need to be checked.

t0ffel added 3 commits May 19, 2016 19:15
use Peter's copr repository for rsyslog 8.17,
Use external config file outside of the container
Apply new data model v2016.03.10.0

EXPOSE 10514

ENV SYSLOG_LISTEN_PORT=10514 \
ES_HOST=bitscout-elasticsearch \
ES_PORT=9200

RUN yum install -y rsyslog rsyslog-elasticsearch rsyslog-gssapi \
RUN curl https://copr.fedorainfracloud.org/coprs/portante/rsyslog-8.17/repo/epel-7/portante-rsyslog-8.17-epel-7.repo > /etc/yum.repos.d/portante-rsyslog-v8.17-epel-7.repo && \
yum install -y rsyslog rsyslog-elasticsearch rsyslog-gssapi \
rsyslog-mmjsonparse rsyslog-mmsnmptrapd && \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can drop the mmsnmptrapd module here.

Added 4 worker threads in the main queue.
fixed typo in the config,
default to non-debug rsyslog
@t0ffel
Copy link
Author

t0ffel commented May 19, 2016

@portante please check

@richm
Copy link
Member

richm commented May 19, 2016

looks good to me - @portante ?

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.

3 participants