search

intel / cri-resource-manager

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

test/e2e: fix incorrect Allocat[ion|or]Priority config key. #1077

Closed klihub closed 10 months ago

klihub commented 10 months ago

Fix incorrect config key in some balloons tests. Use more consistent test environment variables wrt. the actual config key.

codecov-commenter commented 10 months ago

Codecov Report

Merging #1077 (93b81e0) into master (c9bce6d) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1077   +/-   ##
=======================================
  Coverage   32.11%   32.11%           
=======================================
  Files          64       64           
  Lines        9938     9938           
=======================================
  Hits         3192     3192           
  Misses       6450     6450           
  Partials      296      296

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

klihub commented 10 months ago

Do you think we could do s/ALLOCATIONPRIORITY/ALLOCATORPRIORITY/g with the same commit, too?

To have consistent naming in the env vs. config ? Yeah, probably a good idea... I'll update.