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

Добавить возможность перенаправлять логи узлов в отдельные файлы #40

Closed
VifleY opened this issue Jan 22, 2025 · 1 comment
Labels
enhancement New feature or request

Comments

@VifleY
Copy link

VifleY commented Jan 22, 2025

User story:
Я разрабатываю плагин для Picodata. У меня полно разного рода логов в плагине. Зачастую количество узлов в кластере и количество логов превышает то количество, когда это можно безболезненно просматривать в консоли.
Я хочу иметь возможность перенаправить вывод логов узлов в файлы, чтобы мог потом их отдельно анализировать, грепать и так далее.

Что делать:

  1. Добавить в параметры запуска параметр --log-path со значением по умолчанию tmp/cluster
  2. При выполнении команды pike run формировать добавить в команду старта узла пикодаты параметр вида: --log '<log-path>/<instance_name>/picodata.log'. Таким образом при запуске пайка со значением по умолчанию, логи будут литься в tmp/cluster/<instance_dir>
  3. При вызове pike clean очищать log

При это нет необходимости добавлять в пайк режим демонизации - просто мы в консоли будем видеть только лог самого пайка.

Неочевидно:

  1. Надо ли задавать это в topology.yaml?
  2. Есть ли другой конфигурационный файл для пайка, чтобы не прокидывать каждый раз через консоль этот параметр?
  3. Надо ли добавлять переменную окружения для управления этим в пайке?
@lowitea lowitea added the enhancement New feature or request label Jan 22, 2025
@VifleY VifleY changed the title Как пользователь, я хочу чтобы логи каждого узла писались в его директорию в tmp Добавить возможность перенаправлять логи узлов в отдельные файлы Jan 22, 2025
@lowitea
Copy link
Collaborator

lowitea commented Jan 22, 2025

Начать надо с задачи демонизации, она фактически закроет и этот кейс и свой собственный #41

lowitea added a commit that referenced this issue Jan 27, 2025
- add demon mode
- improve logs
- remove logs while enable plugin
- write logs to files per instance
- add more robust wait for child processes
- add colored instance name as prefix for logs in stdout

Closes #40, #20, #10
lowitea added a commit that referenced this issue Jan 28, 2025
- add demon mode
- improve logs
- remove logs while enable plugin
- write logs to files per instance
- add more robust wait for child processes
- add colored instance name as prefix for logs in stdout

Closes #40, #20, #10
lowitea added a commit that referenced this issue Jan 28, 2025
- add demon mode
- improve logs
- remove logs while enable plugin
- write logs to files per instance
- add more robust wait for child processes
- add colored instance name as prefix for logs in stdout

Closes #40, #20, #10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants