Project

Profile

Help

Issue #8703

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

Added by ipanova@redhat.com 6 months ago. Updated 5 months 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 - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 01c33d4a View on GitHub
Added by mdellweg 6 months ago

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)

Revision 01c33d4a View on GitHub
Added by mdellweg 6 months ago

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)

History

#1 Updated by ipanova@redhat.com 6 months ago

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

#2 Updated by ipanova@redhat.com 6 months ago

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

#3 Updated by pulpbot 6 months ago

  • Status changed from ASSIGNED to POST

#4 Updated by mdellweg 6 months ago

  • Status changed from POST to MODIFIED

#5 Updated by pulpbot 5 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF