search

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

pkg/sysfs: drop discovery flags #1083

Closed marquiz closed 10 months ago

marquiz commented 10 months ago

Simplify the code by dropping unused functionality. Effectively, we always used the default flags.

Depends on #1082

codecov-commenter commented 10 months ago

Codecov Report

Attention: 15 lines in your changes are missing coverage. Please review.

Comparison is base (da26501) 32.11% compared to head (5c6eed8) 32.12%. Report is 3 commits behind head on master.

Files Patch % Lines
pkg/cgroups/cgroupid.go 0.00% 2 Missing :warning:
pkg/cri/resource-manager/cache/cache.go 0.00% 2 Missing :warning:
pkg/cri/resource-manager/cache/container.go 0.00% 2 Missing :warning:
pkg/cri/resource-manager/cache/utils.go 0.00% 1 Missing :warning:
...ource-manager/control/page-migrate/page-migrate.go 0.00% 1 Missing :warning:
...esource-manager/policy/builtin/balloons/cputree.go 0.00% 1 Missing :warning:
pkg/dump/flags.go 0.00% 1 Missing :warning:
pkg/instrumentation/flags.go 0.00% 1 Missing :warning:
pkg/log/flags.go 0.00% 1 Missing :warning:
pkg/log/grpc-logger.go 0.00% 1 Missing :warning:
... and 2 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1083 +/- ## ======================================= Coverage 32.11% 32.12% ======================================= Files 64 64 Lines 9938 9937 -1 ======================================= Hits 3192 3192 + Misses 6450 6449 -1 Partials 296 296 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.