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

🌱 Remove permanent error after bm reboot #1396

Merged
merged 1 commit into from
Jul 25, 2024
Merged

Conversation

janiskemper
Copy link
Contributor

What this PR does / why we need it:
In the provisioning of bare metal servers, we added permanent errors that appear after a server times out while rebooting.

This turned out to be quite unreliable, because the server reboots are very unreliable. Sometimes they take forever to reboot, in other situations they are fast and the server provisions successfully.

Therefore, we should rather try again and not stop completely.

TODOs:

  • squash commits
  • include documentation
  • add unit tests

In the provisioning of bare metal servers, we added permanent errors
that appear after a server times out while rebooting.

This turned out to be quite unreliable, because the server reboots are
very unreliable. Sometimes they take forever to reboot, in other
situations they are fast and the server provisions successfully.

Therefore, we should rather try again and not stop completely.
@syself-bot syself-bot bot added size/XS Denotes a PR that changes 0-20 lines, ignoring generated files. area/code Changes made in the code directory labels Jul 25, 2024
@batistein batistein merged commit 475cb3a into main Jul 25, 2024
9 checks passed
@batistein batistein deleted the remove-failed-boot branch July 25, 2024 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/code Changes made in the code directory size/XS Denotes a PR that changes 0-20 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants