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

Use golang builtin multierror #1023

Closed marquiz closed 1 year ago

marquiz commented 1 year ago

Go v1.20 supports wrapping multiple errors into one.

Depends on #1022

codecov-commenter commented 1 year ago

Codecov Report

Merging #1023 (f70c9d4) into master (0baf03f) will not change coverage. Report is 14 commits behind head on master. The diff coverage is 87.03%.

@@           Coverage Diff           @@
##           master    #1023   +/-   ##
=======================================
  Coverage   31.86%   31.86%           
=======================================
  Files          65       65           
  Lines        9835     9835           
=======================================
  Hits         3134     3134           
  Misses       6411     6411           
  Partials      290      290           
Files Changed Coverage Δ
...cy/builtin/topology-aware/topology-aware-policy.go 16.17% <0.00%> (ø)
pkg/blockio/blockio.go 58.98% <83.33%> (ø)
pkg/cgroups/cgroupblkio.go 84.51% <100.00%> (ø)

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