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

No error code for inactive syslog-ng service #8024

Open
Roushan45 opened this issue May 8, 2024 · 2 comments · May be fixed by #8052
Open

No error code for inactive syslog-ng service #8024

Roushan45 opened this issue May 8, 2024 · 2 comments · May be fixed by #8052
Assignees
Labels
Non Containerized Non containerized
Milestone

Comments

@Roushan45
Copy link

Roushan45 commented May 8, 2024

Environment info

  • NooBaa Version: noobaa-core-5.15.3-20240502.el9.x86_64
  • Platform: RHEL9

Actual behavior

  1. [root@s3test-11 bin]# systemctl show -p ActiveState --value syslog-ng
    inactive
    [root@s3test-11 bin]# systemctl show -p ActiveState --value rsyslog
    inactive

both are inactive but in health command /usr/local/noobaa-core/bin/node /usr/local/noobaa-core/src/cmd/health --check_syslog_ng output error code of showing only for rsyslog, what if customer using syslog-ng then it should report syslog-ng as error. How does noobaa knows which logger service being used for noobaa services?

{"service_name":"noobaa","status":"NOTOK","memory":"340.1M","error":{"error_code":"RSYSLOG_SERVICE_FAILED","error_message":"RSYSLOG service is not started properly, Please verify the service with status command."},"checks":{"services":[{"name":"noobaa","service_status":"active","pid":"1172983","error_type":"PERSISTENT"},{"name":"rsyslog","service_status":"inactive","pid":"0","error_type":"PERSISTENT"},{"name":"syslog-ng","service_status":"inactive","pid":"0","error_type":"PERSISTENT"}],"endpoint":{"endpoint_state":{"response":{"response_code":"RUNNING","response_message":"Endpoint running successfuly."},"total_fork_count":2,"running_workers":[2,1]},"error_type":"TEMPORARY"}}}

Expected behavior

  1. error code should be as per logger service which is being used as default for noobaa

Steps to reproduce

  1. /usr/local/noobaa-core/bin/node /usr/local/noobaa-core/src/cmd/health --check_syslog_ng

More information - Screenshots / Logs / Other output

@guymguym guymguym added this to the 5.15.4 milestone May 8, 2024
@romayalon
Copy link
Contributor

@Roushan45 Thank you for bringing this to our attention. We will add it to our task list and address it based on our priorities.

@romayalon romayalon added the Non Containerized Non containerized label May 9, 2024
@romayalon romayalon linked a pull request May 19, 2024 that will close this issue
2 tasks
@romayalon
Copy link
Contributor

Hey @Roushan45
I created a PR in which the status of rsyslog/syslog_ng won't affect the overall status. WDYT?
syslog_ng status is still available on the checks.services - {"name":"syslog-ng","service_status":"inactive","pid":"0","error_type":"PERSISTENT"}

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

Successfully merging a pull request may close this issue.

3 participants