Skip to content

Logstash support for logs#41

Merged
mxsnq merged 7 commits intoispras:masterfrom
MatthewZMSU:MZ_log
Apr 18, 2024
Merged

Logstash support for logs#41
mxsnq merged 7 commits intoispras:masterfrom
MatthewZMSU:MZ_log

Conversation

@MatthewZMSU
Copy link
Copy Markdown
Contributor

@MatthewZMSU MatthewZMSU commented Mar 15, 2024

Description

As some companies use ELK-stack for storing logs we propose Logstash support for the service.
Added new modules in order to add the feature and also commented several important functions.

New dependencies:

  • Winston-Logstash

Type of change

  • New feature (non-breaking change which adds functionality).
  • This change requires a documentation update.

How Has This Been Tested?

I raised my own ELK-stack up using docker-compose. It worked well.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@MatthewZMSU
Copy link
Copy Markdown
Contributor Author

MatthewZMSU commented Mar 15, 2024

As I have shown in the checklist I haven't changed our README.md file. I think we need to add information about environment variables we use.
What do you think?

@mxsnq
Copy link
Copy Markdown
Member

mxsnq commented Mar 28, 2024

As I have shown in the checklist I haven't changed our README.md file. I think we need to add information about environment variables we use. What do you think?

I agree

@MatthewZMSU MatthewZMSU requested a review from mxsnq April 12, 2024 08:24
@mxsnq mxsnq merged commit 9a7b5e4 into ispras:master Apr 18, 2024
@MatthewZMSU MatthewZMSU deleted the MZ_log branch May 30, 2024 09:43
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.

2 participants