tests: Show clearer error message when cleanup tests fail

kubevirt / ssp-operator

Operator that deploys additional KubeVirt resources

Apache License 2.0

29 stars 47 forks source link

tests: Show clearer error message when cleanup tests fail #1038

Open akrejcir opened 2 months ago

akrejcir commented 2 months ago

What this PR does / why we need it: Previously the error message contained the whole object, but the text was too long and it was truncated, so often we could not read the name and namespace of the objects.

Now we show only group, version, kind, namespace and name.

Release note:

None

akrejcir commented 2 months ago

/cc @0xFelix @codingben

sonarcloud[bot] commented 2 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

jcanocan commented 2 months ago

There's any chance to also print the status or something that will help to understand why it wasn't deleted?

kubevirt-bot commented 2 months ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 0xFelix

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/kubevirt/ssp-operator/blob/main/OWNERS)~~ [0xFelix] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment