intel / intel/granulate-utils

Merge duplicate logic of NoSuchProcess detection

Open
#136 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1
Forks
13
PR merge metrics
No merged PRs in 30d

Description

We have 2 different implementations that detect a "NoSuchProcess" state when reading ELF files of processes - https://github.com/Granulate/granulate-utils/blob/ddd5f79b2b7683af431c5b592f548fe6eff38c28/granulate_utils/linux/elf.py#L24 and https://github.com/Granulate/granulate-utils/blob/ddd5f79b2b7683af431c5b592f548fe6eff38c28/granulate_utils/linux/process.py#L122.

Personally I prefer the latter, and it seems more complete. Let's remove either and fix all sites to use a single one.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.