Skip to content

NSFS | Log events to stderr when stderr logging is enabled #8207

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

Merged
merged 1 commit into from
Aug 21, 2024
Merged

NSFS | Log events to stderr when stderr logging is enabled #8207

merged 1 commit into from
Aug 21, 2024

Conversation

naveenpaul1
Copy link
Contributor

@naveenpaul1 naveenpaul1 commented Jul 16, 2024

Explain the changes

  1. Event are always logged to syslog(/var/log/noobaa_events.log), If customer choose to enable stderr also for logging event log should print in stderr
  2. Eent json is generated from debug module, rsyslog templete is removed
  3. removed syslog specific property from event, updated event log

{"timestamp":"2024-08-07T09:29:00.286Z","host":"4c8c451cfb17","event":{"code":"noobaa_started","message":"Noobaa started","description":"Noobaa started running","entity_type":"NODE","event_type":"STATE_CHANGE","scope":"NODE","severity":"INFO","state":"HEALTHY","pid":477}}

Issues: Fixed #xxx / Gap #xxx

  1. NC | NSFS | Log events to stderr if stderr is enabled #8206

Testing Instructions:

  1. check event are getting logged to stderr after enabling stder(LOG_TO_STDERR_ENABLED=true)

Jul-16 16:01:17.591 [/170454] [EVENT]{"code":"noobaa_account_created","message":"Account created","description":"Noobaa Account created","entity_type":"NODE","event_type":"INFO", "scope":"NODE","severity":"INFO","state":"HEALTHY","arguments":{"account":"account-test2"},"pid":170454}

  • Doc added/updated
  • Tests added

@romayalon romayalon requested review from guymguym and romayalon July 16, 2024 10:55
@naveenpaul1 naveenpaul1 changed the title NSFS | Log events to stderr if stderr is enabled NSFS | Log events to stderr when stderr logging is enabled Jul 16, 2024
@naveenpaul1 naveenpaul1 merged commit 6cb5714 into noobaa:master Aug 21, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants