1
84%
main: 76%

Ran 23 Nov 2020 02:08PM UTC

Files 1

Run time 0s

Badge

Embed ▾

Committed 23 Nov 2020 01:14PM UTC coverage: 84.304% (-0.4%) from 84.743%

Job # 4532.1

Build Type

push

travis-ci-com

Committed by

web-flow

Commit Message

Add a mutating webhook to protect hco namespace (#949)

Introducing a mutating admission webhooks, deployed by
the OLM, bound to the namespace where HCO is running
to protect it from uneducated deletion the can
potentially made it stuck.
The webhook scope is limited to that specific namespace
by the use of an objectSelector.

In this first step the mutating webhook is simply refusing,
with a clear error message,to admit the deletion of the
namespace if the Hyperconverged CR is still there.
A future enhancement can try to delete the Hyperconverged CR
as a side effect (that's why a mutating webhook with side effects)
and, only when it and all its descendants are correctly
and safely deleted, admit the deletion of the namespace.
This requires other ancillary work to make the deletion safe
avoiding race conditions.
So let's simply refuse to delete the namespace as a first
and safer step.

Test it in CI at the end of upgrade test lanes:
the test will try to delete the namespace when the Hyperconverged CR
is still there looking for a specific error message,
the test then will delete the hyperconverged CR and finally
successfully delete the namespace.

Fixes: https://bugzilla.redhat.com/1881874

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

Run Details

521 of 618 relevant lines covered (84.3%)

25.95 hits per line

kubevirt / hyperconverged-cluster-operator / 4532 / 1
84%
main: 76%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4532.1

kubevirt / hyperconverged-cluster-operator / 4532 / 1 84% main: 76%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4532.1

kubevirt / hyperconverged-cluster-operator / 4532 / 1
84%
main: 76%

README BADGES
x