This is an automated email from the ASF dual-hosted git repository.

wusheng pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/skywalking-rover.git


The following commit(s) were added to refs/heads/main by this push:
     new d0cf7d9  Remove unnecessary process exist check (#138)
d0cf7d9 is described below

commit d0cf7d9619ec600fd9bbc944669582b1b24b166a
Author: mrproliu <[email protected]>
AuthorDate: Tue Aug 13 23:05:04 2024 +0800

    Remove unnecessary process exist check (#138)
---
 pkg/process/finders/kubernetes/finder.go | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/pkg/process/finders/kubernetes/finder.go 
b/pkg/process/finders/kubernetes/finder.go
index d4e37c8..29b5b1d 100644
--- a/pkg/process/finders/kubernetes/finder.go
+++ b/pkg/process/finders/kubernetes/finder.go
@@ -382,21 +382,6 @@ func (f *ProcessFinder) ParseProcessID(ps 
api.DetectedProcess, downstream *v3.EB
 }
 
 func (f *ProcessFinder) ShouldMonitor(pid int32) bool {
-       pidList, err := process.Pids()
-       if err != nil {
-               return false
-       }
-       pidExist := false
-       for _, p := range pidList {
-               if p == pid {
-                       pidExist = true
-                       break
-               }
-       }
-       if !pidExist {
-               return false
-       }
-
        newProcess, err := process.NewProcess(pid)
        if err != nil {
                return false

Reply via email to