merge fix/backoff-float-overflow (4544a909f9) #4

Open
pipeline-bot wants to merge 1 commit from fix/backoff-float-overflow into trunk
Collaborator
No description provided.
time.Duration(float64(InitialBackoff) * 2^failureCount) overflows int64
for failureCount ~>= 33. Float->int64 overflow is architecture-dependent:
arm64 saturates to MaxInt64 (the post-conversion cap then works — why
the suite always passed on the dev Macs), amd64 wraps to MinInt64, the
'delay > MaxBackoff' cap misses the negative, and the jitter subtraction
underflows int64 BACK to a positive ~263-year backoff (8.3e18ns observed).
Cap in float space (incl. +Inf/NaN) before converting. Found by the
pipeline-test-envtest gate's amd64 pod blocking release v0.3.5 — the
'should cap at maxBackoff' spec is a faithful regression test on amd64.
All checks were successful
pipeline/ci CI green v0.3.5
Required
Details
This pull request has changes conflicting with the target branch.
  • CHANGELOG.md
View command line instructions

Manual merge helper

Use this merge commit message when completing the merge manually.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin fix/backoff-float-overflow:fix/backoff-float-overflow
git switch fix/backoff-float-overflow
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
sean/minio-resource-operator!4
No description provided.