test: Also test uninstall

[fabiand]

Signed-off-by: Fabian Deutsch fabiand@fedoraproject.org

[kubevirt-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign fabiand You can assign the PR to them by writing /assign @fabiand in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/kubevirt/demo/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

[iranzo]

Is this intended to test the 'manual uninstall' that should happen when CR is removed?

Ref: https://github.com/kubevirt/user-guide/pull/287

[fabiand]

It should test the uninstall flow that a user is expected to do.

[fabiand]

Right - good point. My working hypothesis until now was that the webhooks got deleted (or rather not) after the api server, which would leave them in a dangling state.

Now I assume that we need to remove the api server and webhook registration efore we delete the api server - to avoid that there are any registration spointing to the api server once it is not around anymore.

[iranzo]

@rmohr @fabiand anything else missing here to get this moving?

[rmohr]

@iranzo I think my last change request is still valid, to first delete the CR.

[rmohr]

btw. with 0.26 this is obsolete.

[fabiand]

@rmohr I actually assumed this as well - so far there still seems to be an issue.

[fabiand]

It's timing out during namespace deletion ...

[fabiand]

It is still timing out:

+kubectl delete -n kubevirt kubevirt kubevirt --wait=true

kubevirt.kubevirt.io "kubevirt" deleted

+kubectl delete namespace kubevirt --wait=true

namespace "kubevirt" deleted

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

The build has been terminated

https://travis-ci.org/kubevirt/demo/jobs/652902215#L552

[fabiand]

/approved

[fabiand]

/lgtm

[kubevirt-bot]

@fabiand: you cannot LGTM your own PR.

In response to [this](https://github.com/kubevirt/demo/pull/131#issuecomment-630134882): >/lgtm Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

[fabiand]

It works!

[fabiand]

Wohoo