kubevirt / hyperconverged-cluster-operator / 14485009561 / 1
76%
main: 76%

DEFAULT BRANCH: main
Ran
Files 83
Run time 2s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 72.413% (+0.08%) from 72.333%
14485009561.1

push

github

web-flow 
Check descheduler according to new profile (#3476)

* Check descheduler according to new profile

On the descheduler we now have a KubeVirt
dedicated profile named KubeVirtRelieveAndMigrate.
Enforce its presence if the descheduler is in use.

Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>

* Move the KubeDescheduler validation to the controller

The clusterInfo interface's  `IsDeschedulerMisconfigured()` and
`RefreshDeschedulerCR()` functions are only called from the controller,
and should not be globally available. Also, the current logic is using a
cache mechanism that is never actually used, becase the user call always
calls `RefreshDeschedulerCR()` (to refresh the cache) before calling to
`IsDeschedulerMisconfigured()` (to read the cache). So the cache saves
nothing.

In addition, the ClusterInfo is huge: it contains 21 functions, while
golang best practice is to write interfaces as smaller as possible. It
is hard to implement and maintain such a big interface, where any change
in any function, need to be implemented in all the implementations.

This commit moves the KubeDescheduler validation logic from the
ClusterInfo, to the descheduler controller, where it actually been used,
to get more effective, more readable code, and to reduce the number of
functions in the ClusterInfo interface.

Signed-off-by: Nahshon Unna-Tsameret <nunnatsa@redhat.com>

* Align e2e tests

Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>

---------

Signed-off-by: Simone Tiraboschi <stirabos@redhat.com>
Signed-off-by: Nahshon Unna-Tsameret <nunnatsa@redhat.com>
Co-authored-by: Nahshon Unna-Tsameret <nunnatsa@redhat.com>

6536 of 9026 relevant lines covered (72.41%)

0.95 hits per line

Source Files on job Unit - 14485009561.1