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

feat: filter by PID in process tree #87

Merged
merged 1 commit into from
Jul 20, 2023

Conversation

snprajwal
Copy link
Member

Currently, the entire process tree is rendered in the tree view. This allows for the tree to be pruned for a specific process using the --pid or -p flag.

Fixes #86

@github-actions
Copy link

github-actions bot commented Jul 10, 2023

Test Results

38 tests  +2   38 ✔️ +2   1s ⏱️ ±0s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 1d39519. ± Comparison against base commit 29d4681.

♻️ This comment has been updated with latest results.

@snprajwal snprajwal force-pushed the filter-pid branch 2 times, most recently from 941dbfc to c7682d1 Compare July 12, 2023 15:13
@codecov-commenter
Copy link

codecov-commenter commented Jul 12, 2023

Codecov Report

Patch coverage: 85.36% and project coverage change: +0.02 🎉

Comparison is base (29d4681) 82.40% compared to head (0e8d1b4) 82.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
+ Coverage   82.40%   82.43%   +0.02%     
==========================================
  Files           4        4              
  Lines         523      558      +35     
==========================================
+ Hits          431      460      +29     
- Misses         68       72       +4     
- Partials       24       26       +2     
Impacted Files Coverage Δ
tree.go 87.17% <80.00%> (-3.15%) ⬇️
checkpointctl.go 92.17% <100.00%> (+0.51%) ⬆️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

tree.go Outdated Show resolved Hide resolved
tree.go Show resolved Hide resolved
tree.go Show resolved Hide resolved
checkpointctl.go Outdated Show resolved Hide resolved
tree.go Outdated Show resolved Hide resolved
Currently, the entire process tree is rendered in the tree view. This
allows for the tree to be pruned for a specific process using the `--pid`
or `-p` flag.

Signed-off-by: Prajwal S N <prajwalnadig21@gmail.com>
@snprajwal
Copy link
Member Author

cc @adrianreber PTAL?

@adrianreber adrianreber merged commit 5d84ffa into checkpoint-restore:main Jul 20, 2023
9 checks passed
@snprajwal snprajwal deleted the filter-pid branch July 27, 2023 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Prune process tree by PID if specified
4 participants