cpuallocator: fix cpu priority discovery

[marquiz]

The CPU priorities were discovered correctly but not stored in the cpuallocator so they were effectively discarded.

[fmuyassarov]

LGTM

[codecov-commenter]

Codecov Report

Merging #971 (daa821d) into master (01cf8c2) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #971   +/-   ##
=======================================
  Coverage   33.68%   33.69%           
=======================================
  Files          61       61           
  Lines        9156     9157    +1     
=======================================
+ Hits         3084     3085    +1     
  Misses       5789     5789           
  Partials      283      283           

Impacted Files	Coverage Δ
pkg/cpuallocator/allocator.go	48.32% <100.00%> (+0.15%)	⬆️

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

[askervin]

LGTM.

[klihub]

LGTM

Should we have a unit test with a mocked sysfs exercising and verifying this feature, using exclusively the publicly visible interfaces ?

[marquiz]

Should we have a unit test with a mocked sysfs exercising and verifying this feature, using exclusively the publicly visible interfaces ?

I think we should. But not just for this feature but for the cpuallocator overall. I'd suggest to leave this as a future exercise as we don't currently have any unit test infra in the cpuallocator to quickly add a new testcase like this