Was combine_logs deprecated and removed?
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 43.3k
- Forks
- 2.7k
- PR merge metrics
- No merged PRs in 30d
Description
What's going wrong?
using pm2 5.3
combine_logs option stopped working and we were wondering why our logs were blanked despite pm2 logs being able to tail logs as they come in. switching to merge_logs works.
How could we reproduce this issue?
set combine_logs: true without merge_logs and have error_log and out_log set to the same file.
Supporting information
combine_logs seems to still be documented as an alias https://pm2.keymetrics.io/docs/usage/application-declaration/ but is now absent in https://pm2.keymetrics.io/docs/usage/log-management/
I did a search on the codebase for combine_logs while it still exists in the schema it doesn't seem to be set as deprecated or anything. It's not wired up as an alias to merge_logs anymore.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by checking the PM2 schema and the log-management documentation for combine_logs and merge_logs, then reproduce the issue with both log options targeting the same file. Compare the observed behavior with the documented alias behavior; done means the option is either restored as an alias or its deprecation/removal is clearly documented.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, nodejs
- Domain
- observability-sre
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100