cloud-bulldozer / go-commons

Code repository with all go common packages and libraries
Apache License 2.0
4 stars 9 forks source link

Fix worker node count logic #25

Closed rsevilla87 closed 1 year ago

rsevilla87 commented 1 year ago

Type of change

Description

The previous logic was counting nodes with the infra and worker labels (this happens in Classic ROSA clusters), as workers. With this patch, when the worker label is found, we ensure that the node does not contain any "workload" or "infra" label.

Related Tickets & Documents

Checklist before requesting a review

Testing

codecov[bot] commented 1 year ago

Codecov Report

Merging #25 (5e0d706) into main (ec9da22) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #25   +/-   ##
=======================================
  Coverage   85.13%   85.13%           
=======================================
  Files           6        6           
  Lines         343      343           
=======================================
  Hits          292      292           
  Misses         37       37           
  Partials       14       14