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

Implement logger #140

Open
1 task
ygrishajev opened this issue Apr 1, 2024 · 3 comments
Open
1 task

Implement logger #140

ygrishajev opened this issue Apr 1, 2024 · 3 comments
Labels
enhancement New feature or request P2 priority 2 feature/ enhancement

Comments

@ygrishajev
Copy link
Collaborator

Currently project utilises nodejs console.log for logging with doesn't provide a desired observability or debugging experience. Some of the issues:

  1. noisy tests outputs
  2. no way to filter or identify logs by levels which complicates decision making
  3. logs are simple strings which would complicate indexing if stored in tools like kibana or grafana

A good way to address this would be having a proper logger having common features. One of the good ones is pino.

  • replace all the logs in the backend services with a properly scoped pino logger
@ygrishajev ygrishajev added the enhancement New feature or request label Apr 17, 2024
@anilmurty
Copy link
Collaborator

  • consider adding a way to preserve and review logs after the deployment is closed

@anilmurty anilmurty added the P2 priority 2 feature/ enhancement label Apr 30, 2024
@ygrishajev
Copy link
Collaborator Author

@anilmurty this task is more about app level logs for cloudmos services. Infrastructure for managing logs is a different story. Clients' deployments logs is even more further topic.

@hiroyukikumazawa
Copy link

I will do this, assign me plz.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P2 priority 2 feature/ enhancement
Projects
Status: Backlog (not prioritized)
Development

No branches or pull requests

3 participants