Skip to content

Conversation

@fwiesel
Copy link
Contributor

@fwiesel fwiesel commented Oct 10, 2025

No description provided.

@fwiesel fwiesel force-pushed the fix-cleanup-revisited branch 2 times, most recently from 1448519 to 1ce6aa3 Compare October 13, 2025 13:58
@fwiesel fwiesel marked this pull request as ready for review October 13, 2025 13:59
@fwiesel fwiesel requested a review from notandy October 13, 2025 13:59
@fwiesel fwiesel force-pushed the fix-cleanup-revisited branch from 1ce6aa3 to 5134854 Compare October 14, 2025 08:49
@fwiesel fwiesel force-pushed the fix-cleanup-revisited branch from 5134854 to de8c5a2 Compare October 14, 2025 15:49
@fwiesel fwiesel force-pushed the fix-cleanup-revisited branch from de8c5a2 to e59a74b Compare October 22, 2025 10:10
@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/cobaltcore-dev/openstack-hypervisor-operator/internal/controller 37.17% (-0.16%) 👎

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/cobaltcore-dev/openstack-hypervisor-operator/internal/controller/onboarding_controller.go 4.00% (+1.45%) 250 (+15) 10 (+4) 240 (+11) 👍

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Delete all servers, and do not abort on the first failure.
The spec can disable the life-cycle or we can get a terminating
condition, even while we are testing.
We better clean up the test instances then too.
The log message was the same as in another delete case,
and that makes it hard to identify where it gets deleted.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants