kubevirt / hyperconverged-cluster-operator / 14240046899

72%

main: 76%

Pull #3379

github

Fix a bug when setting a CPU quentity without a type

When setting, for example, the
`spec.resourceRequirements.storageWorkloads.limit` field to `"1.5"`, the
client set the value in CDI to `"1500m"`. Then HCO will always find this
value as not matches to the required value of `"1.5"`, and will
constantly try to update CDI.

This PR fixes this issue by turning the required objects to json and back
to objects. The json marshaling peforms the same change in the object.
Now when HCO will compare the required value with the existing one, both
will be at the newer format, with the quntity type, and HCO will
understand that no change was done, and it will not try to update the CR
again.

Also. This PR adds a mutex protection to the cache in the operand
handlers.

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

53 of 62 new or added lines in 7 files covered. (85.48%)

6524 of 9024 relevant lines covered (72.3%)

0.8 hits per line