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

rdt: stop trying to get container cgroup dir #975

Closed marquiz closed 1 year ago

marquiz commented 1 year ago

We don't use that information for anything. In an error case we'll get the same error from container.GetProcesses() later anyway.

codecov-commenter commented 1 year ago

Codecov Report

Merging #975 (61acbd7) into master (4674927) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #975   +/-   ##
=======================================
  Coverage   31.96%   31.96%           
=======================================
  Files          65       65           
  Lines        9833     9833           
=======================================
  Hits         3143     3143           
  Misses       6402     6402           
  Partials      288      288

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

marquiz commented 1 year ago

ping @jukkar