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

Config-Option to deactivate rsyslog if syslogng are used #8046

Closed
ps20renar opened this issue May 14, 2024 · 3 comments · Fixed by #8094
Closed

Config-Option to deactivate rsyslog if syslogng are used #8046

ps20renar opened this issue May 14, 2024 · 3 comments · Fixed by #8094
Assignees
Labels
Non Containerized Non containerized Type:Enhancement New suggestions for behaviours
Milestone

Comments

@ps20renar
Copy link

Is your feature request related to a problem? Please describe.
Currently the health check script monitor rsyslog and syslobng if we use the sub option --check_syslog_ng. In our linux platform we had rsyslog masked and use only syslogng for this logging request
(checked Version are 5.15.3)

Describe the solution you'd like
generate a config.json option to use syslogng as the only one logging framework. Like ENABLE_SYSLOG_NG: "True"
this should the effect that rsyslog should no more checked. In the health.js script. It will be also great if this issue generate an example (opt/syslog-ng/etc/conf.d/nsfs.conf that defined the filter what should filtered in a separat log-file. (/var/log/nsfs.log)

Describe alternatives you've considered
Description of alternative solutions or features you've considered.

Additional context
An example of these logconf-file can be:

NSFS syslog-ng settings

template nsfs_t {
template("$R_ISODATE $HOST $MSGHDR $MSG\n");
};

filter f_nsfs { match('node[ID0-9][0-9]*[EWID]') and level(info..emerg); };
destination d_nsfs { file("/var/log/nsfs.log" template(nsfs_t) frac-digits(6) ts_format(iso)); };
log { source(s_sys); filter(f_nsfs); destination(d_nsfs); };

As an example it should be verified.

@ps20renar ps20renar added the Type:Enhancement New suggestions for behaviours label May 14, 2024
@naveenpaul1
Copy link
Contributor

Duplicated issue : #8027

@romayalon
Copy link
Contributor

Hey @ps20renar @guymguym
I created a PR in which I add --check_rsyslog flag, without passing this flag we won't check rsyslog status.
Guy and I discussed last week if checking rsyslog/syslog_ng service status during this health check is appropriate, So for now I decided to put all the rsyslog/syslog_ng checks as optional, and the status of these won't affect the overall status. WDYT?

@ps20renar
Copy link
Author

Hallo @romayalon,
i think this is ok for me.
If someone want to check rsyslog or syslogng, there ist an option to decide wich one.
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non Containerized Non containerized Type:Enhancement New suggestions for behaviours
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants