eclipse-score / eclipse-score/lifecycle

Standardize ProcessInfoNode logs

Open
#570 0 comments 0 reactions 0 assignees View on GitHub
cleanup good first issue
Dominant language
C++
Stars
6
Forks
34
Avg merge
2d 3h
Merged PRs (30d)
80

Description

### What

There are 16 logs in ProcessInfoNode that describe which process the log relates to. However, most of these format information about the process differently. We have:
- "Process pid `pid_` ( `config_.name` )"
- "process `this`"
- `this` is just a pointer and logs as "True"
- "Process `process_index_` pid `pid_` ( `config_.name` ) for node `this` "
- "process `process_index_` ( `config_.name` )"
- "Process pid `pid_` ( `config_.name` ) process `process_index_`

It may be more readable to standardize how we log info about a process info node. It may be possible to provide a custom log formatter for ProcessInfoNode objects to keep the format consistent. One proposed format is the following:
- "Name: `config_.name` , PID: `pid_`"

### Acceptance Criteria (DoD)

- ProcessInfoNode log messages provide information in a consistent format

### How

_No response_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.