kubevirt / hyperconverged-cluster-operator / 21704553524

76%

Build Type

push

github

Committed by web-flow

Commit Message

CNV-74931: Split TLS Security Profile out of ClusterInfo (#3996)

* Fix TLSSecurityProfile validation

Also validate that if the type is "Custom", the custom field must not be
empty.

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

* Add a new package to manage the TLSSecurityProfile

This will replace the logic in the validating webhook and in the
ClusterInfo.

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

* use the new tlssecconfig on boot

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

* use the tlssecprofile pkg in handllers

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

* use the tlssecprofile pkg in wh apiserver-controller

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

* introduce new apiserver controller for the operator

This new controller is watching the APIServer CR in openshift and
refresh the TLS Security Config if it was changed.

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

* use the apiserver controller in the operator

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

* Clean the ClusterInfo interface

Remove the `GetTLSSecurityProfile` and `RefreshAPIServerCR` functions from the interface.

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

* Add debug information for the hack/deploy.sh script

The script should print the pod log in case of failure, but this is not
working.

This commit fixes this issue, and the failed pods logs are now properly
printed out.

When running this script in CI, the script writes the logs and the log
description (the output of `kubectl describe pod ...`), to the test
artifacts, so it won't fload the test log, and to ease the access to
each log, separately.

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

---------

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

Run Details

212 of 224 new or added lines in 12 files covered. (94.64%)

14 existing lines in 2 files now uncovered.

8604 of 11275 relevant lines covered (76.31%)

1.78 hits per line