Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow TinkerbellMachine to be removed when no Hardware exists: #421

Merged
merged 2 commits into from
Nov 25, 2024

Conversation

jacobweinstock
Copy link
Member

Description

When a Hardware object corresponding to a tinkerbellmachine object no longer exists, allow the tinkerbellmachine object to be removed and clean up associated template and workflow. This means BMC operations to power the machine off and modify the Hardware object will all be skipped.

While the tinkerbellmachine object can be deleted if the finalizer is removed manually this improves the user experience.

Why is this needed

Fixes: #368

How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • added unit or e2e tests
  • provided instructions on how to upgrade

Signed-off-by: Jacob Weinstock <[email protected]>
When a Hardware object corresponding to a tinkerbellmachine
object no longer exists, allow the tinkerbellmachine object
to be removed and clean up associated template and workflow.
At the moment this will skip any BMC operations to power the
machine off and modify the Hardware object. This allows the
tinkerbellmachine to be deleted without needing to know that
it's ok to remove the finalizer and delete the object manually.

Signed-off-by: Jacob Weinstock <[email protected]>
@jacobweinstock jacobweinstock merged commit 2c72446 into tinkerbell:main Nov 25, 2024
7 checks passed
@jacobweinstock jacobweinstock deleted the orphan-machine branch November 25, 2024 18:53
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.

TinkerbellMachine's cannot be deleted when the bound Hardware doesn't exist
1 participant