Project

Profile

Help

Issue #8703

closed

Backport to 2.5: if wait is set on Throttled() exponential backoff is not enabled

Added by ipanova@redhat.com almost 3 years ago. Updated almost 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 96
Quarter:

Description

It appears that if having wait set for 5 seconds, the client will always back-off for 5 seconds https://github.com/pulp/pulp_container/blob/master/pulp_container/app/registry_api.py#L661, the exponential backoff in this case won't be triggered.

https://github.com/pulp/pulp_container/pull/289#issuecomment-820524896


Related issues

Copied from Container Support - Issue #8576: if wait is set on Throttled() exponential backoff is not enabledCLOSED - CURRENTRELEASEmdellwegActions
Actions #1

Updated by ipanova@redhat.com almost 3 years ago

  • Copied from Issue #8576: if wait is set on Throttled() exponential backoff is not enabled added
Actions #2

Updated by ipanova@redhat.com almost 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee changed from mdellweg to ipanova@redhat.com
Actions #3

Updated by pulpbot almost 3 years ago

  • Status changed from ASSIGNED to POST

Added by mdellweg almost 3 years ago

Revision 01c33d4a | View on GitHub

No wait in Throttled

Stop to tell the client to wait for a specific amount. This way, well behaved clients will increase their waiting time exponentially while retrying.

closes #8703 backports #8576

(cherry picked from commit 3796cab986a8ca2892d70802279ab01896018fbe)

Added by mdellweg almost 3 years ago

Revision 01c33d4a | View on GitHub

No wait in Throttled

Stop to tell the client to wait for a specific amount. This way, well behaved clients will increase their waiting time exponentially while retrying.

closes #8703 backports #8576

(cherry picked from commit 3796cab986a8ca2892d70802279ab01896018fbe)

Actions #4

Updated by mdellweg almost 3 years ago

  • Status changed from POST to MODIFIED
Actions #5

Updated by pulpbot almost 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF