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

ServerClaim does not propagate power state to Server #61

Closed
afritzler opened this issue Jul 2, 2024 · 3 comments · Fixed by #50
Closed

ServerClaim does not propagate power state to Server #61

afritzler opened this issue Jul 2, 2024 · 3 comments · Fixed by #50
Assignees
Labels
bug Something isn't working

Comments

@afritzler
Copy link
Member

Describe the bug

Once a Server has been claimed and reserved, the change of the Power field in the ServerClaim is not propagated to the Server resource due to an early exit in the claim reconciliation loop. This posts a problem as you can not power of a Server via it's claim.

@afritzler afritzler added the bug Something isn't working label Jul 2, 2024
@afritzler afritzler moved this from Backlog to Ready in Metal Automation Jul 2, 2024
@stefanhipfel stefanhipfel self-assigned this Jul 17, 2024
@stefanhipfel stefanhipfel moved this from Ready to In progress in Metal Automation Jul 17, 2024
@stefanhipfel
Copy link
Contributor

What is the exact purpose of setting the powerState in the serverClaim?

do I want to have the powerState applied after my server has booted the desired bootconfig?

Is this the only way a "customer" which has claimed a server to power cycle his server?

@afritzler
Copy link
Member Author

Yes that is exactly the intention here: The owner of a ServerClaim can typically not modify the Server object directly.

@afritzler
Copy link
Member Author

Fixed with #50

@github-project-automation github-project-automation bot moved this from In progress to Done in Metal Automation Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants