intel / cri-resource-manager

Kubernetes Container Runtime Interface proxy service with hardware resource aware workload placement policies
Apache License 2.0
176 stars 57 forks source link

e2e: ignore terminated processes on fetching allowed resources #1065

Closed askervin closed 1 year ago

askervin commented 1 year ago

Fetching allowed resources (needed by "verify" and "report allowed") caused test failures when a process found by pgrep got terminated before reading its /proc/pid/files. This change omits pids whose files could not be read without errors.

codecov-commenter commented 1 year ago

Codecov Report

Merging #1065 (18a74bc) into master (d659cbd) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1065   +/-   ##
=======================================
  Coverage   31.73%   31.73%           
=======================================
  Files          64       64           
  Lines        9816     9816           
=======================================
  Hits         3115     3115           
  Misses       6410     6410           
  Partials      291      291           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more