Skip to content

🌱 Backport hcloud rate-limit handling to v1.1.x#1903

Open
guettli wants to merge 3 commits intov1.1.xfrom
tg/backport-rate-limit-handling-to-v1.1.x
Open

🌱 Backport hcloud rate-limit handling to v1.1.x#1903
guettli wants to merge 3 commits intov1.1.xfrom
tg/backport-rate-limit-handling-to-v1.1.x

Conversation

@guettli
Copy link
Copy Markdown
Collaborator

@guettli guettli commented Mar 27, 2026

Summary

  • stop treating hcloud rate-limit failures during findServer() as if the server disappeared
  • let reconcile requeue while a not-yet-ready machine is rate-limited
  • keep deletion paths using the existing rate-limit helper

Main PRs

Why

Without this backport, v1.1.x can mis-handle a rate-limited findServer() call and fall into the missing-server path.

Testing

  • go test ./pkg/services/hcloud/server -run ^ -count=1
  • full server suite not run locally because envtest binaries are missing

@github-actions github-actions bot added size/XS Denotes a PR that changes 0-20 lines, ignoring generated files. area/code Changes made in the code directory labels Mar 27, 2026
@github-actions github-actions bot added size/M Denotes a PR that changes 50-200 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-20 lines, ignoring generated files. labels Mar 27, 2026
@guettli guettli requested a review from Dhairya-Arora01 March 27, 2026 08:18
@guettli guettli marked this pull request as ready for review March 27, 2026 08:18
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/M Denotes a PR that changes 50-200 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant